Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feat] Add update GitOps repository workflow #510

Merged
merged 19 commits into from
Feb 8, 2023

Conversation

saenyakorn
Copy link
Member

@saenyakorn saenyakorn commented Feb 6, 2023

Why did you create this PR

What did you do

  • Add update-gitops.yaml workflow for open PR to cugetreg-gitops repo
  • Add gitops-mode to choose between PR or commit

Workflow explain

  • When we push something to to dev, the action will calculate affected packages and push kustomize changed to gitios repo.
  • When we push something to beta, the action will do pre-release thing, and then open the Pre-release PR. After we merge the PR, the action will bump the package version, build the docker images and open kustomize changed PR to GitOps repo.
  • Similar to beta, when we push something to main, the action will do release thing, and the rest is the same as beta's flow.

Demo

Workflow https://github.com/thinc-org/cugetreg/actions/runs/4106806503

Success Commit - https://github.com/thinc-org/cugetreg-gitops/commit/ac296cee9f9ada87fc91369eda675950bbcbd62f

Success opened PR - https://github.com/thinc-org/cugetreg-gitops/pull/6

https://dev.cugetreg.com

Checklist

  • Deploy a demo
  • Check browsers compatibility
  • Wrote coverage tests

@saenyakorn saenyakorn self-assigned this Feb 6, 2023
@changeset-bot
Copy link

changeset-bot bot commented Feb 6, 2023

⚠️ No Changeset found

Latest commit: 737947d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

.github/workflows/deploy-docker.yaml Outdated Show resolved Hide resolved
.github/workflows/release-dev.yaml Outdated Show resolved Hide resolved
.github/workflows/deploy-docker.yaml Outdated Show resolved Hide resolved
.github/workflows/release-dev.yaml Show resolved Hide resolved
.github/workflows/release.yaml Outdated Show resolved Hide resolved
.github/workflows/release.yaml Show resolved Hide resolved
.github/workflows/update-gitops.yaml Show resolved Hide resolved
.github/workflows/update-gitops.yaml Show resolved Hide resolved
.github/workflows/update-gitops.yaml Show resolved Hide resolved
.github/workflows/update-gitops.yaml Show resolved Hide resolved
Copy link
Member

@suphon-t suphon-t left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more thing. The old syntax seems to be broken.

.github/workflows/release.yaml Outdated Show resolved Hide resolved
@saenyakorn saenyakorn merged commit 1537d22 into beta Feb 8, 2023
@saenyakorn saenyakorn deleted the yu/feat/update-gitops-workflow branch February 8, 2023 05:42
@bombnp bombnp mentioned this pull request Feb 9, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants