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

ci: automate bumping ZITADEL version #43

Merged
merged 33 commits into from
Mar 13, 2023
Merged

Conversation

eliobischof
Copy link
Member

@eliobischof eliobischof commented Oct 13, 2022

contributes to #29

What's happening:

  • The action consumes zitadel-released repository_dispatch events. With this PR on ZITADELs end, the webhook is called after a ZITADEL release.
  • The action compares the newly released ZITADEL version with the appVersion in Charts.yaml
  • If the appVersion is patched, the chart version is patched. If the appVersion has a new minor version, the charts minor version is bumped, The same for the major version.
  • The action creates or overwrites a pull request, approves the PR and enables auto-merge

I created the PR from a fork for testing purposes, because the workflow is just activated when it's on the default branch.

@eliobischof eliobischof added the enhancement New feature or request label Oct 13, 2022
@eliobischof eliobischof self-assigned this Oct 13, 2022
eliobischof and others added 6 commits October 13, 2022 10:21
Co-authored-by: eliobischof <eliobischof@users.noreply.github.com>
Co-authored-by: eliobischof <eliobischof@users.noreply.github.com>
@eliobischof eliobischof linked an issue Oct 13, 2022 that may be closed by this pull request
@hifabienne
Copy link
Member

@eliobischof @fforootd can we merge this? Or do we need something?

@eliobischof
Copy link
Member Author

@eliobischof @fforootd can we merge this? Or do we need something?

@livio-a why is it WIP again? https://github.com/zitadel/DevOps/issues/10#issue-1604411571

@livio-a
Copy link
Member

livio-a commented Mar 13, 2023

@eliobischof @fforootd can we merge this? Or do we need something?

@livio-a why is it WIP again? zitadel/DevOps#10 (comment)

Probably only because of the open discussion above (version parsing), but for me it's ok if we merge the current state

@eliobischof
Copy link
Member Author

@eliobischof @fforootd can we merge this? Or do we need something?

@livio-a why is it WIP again? zitadel/DevOps#10 (comment)

Probably only because of the open discussion above (version parsing), but for me it's ok if we merge the current state

I resolved it

@eliobischof eliobischof enabled auto-merge (squash) March 13, 2023 06:49
@eliobischof eliobischof merged commit 9eec101 into zitadel:main Mar 13, 2023
@kleberbaum kleberbaum mentioned this pull request Apr 30, 2024
11 tasks
kleberbaum pushed a commit to fhkit/zitadel-charts that referenced this pull request May 24, 2024
* add bump ZITADEL workflow

* use semanticoutput

* test action

* fix trigger

* remove if

* add token

* syntax and typos

* syntax

* typos

* typo

* fix references

* debug

* reading multiple yaml keys is not supported

* debug

* fix read yaml usage

* typo

* don't fail on semver conditionals

* fix action ref

* typo

* debug

* update Chart.yaml

* fix order

* automerge and approve

* [create-pull-request] automated change (zitadel#4)

Co-authored-by: eliobischof <eliobischof@users.noreply.github.com>

* Revert "[create-pull-request] automated change (zitadel#4)"

This reverts commit cced733.

* gh token scopes

* test bump

* [create-pull-request] automated change (zitadel#5)

Co-authored-by: eliobischof <eliobischof@users.noreply.github.com>

* Revert "[create-pull-request] automated change (zitadel#5)"

This reverts commit d5e7bc4.

* cleanup

* remove newline

* use app token

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: eliobischof <eliobischof@users.noreply.github.com>
@kleberbaum kleberbaum mentioned this pull request Jun 5, 2024
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update chart version automatically
4 participants