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

Helm chart patch release #322

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 4, 2023

Automatic Helm chart update.
This PR has been created by the automation used to automatically update the Helm charts when some Kubewarden component is released

REMEMBER IF YOU WANT TO MERGE IN A SINGLE COMMIT CHANGES AND VERSION BUMP, YOU MUST SQUASH THE COMMIT BEFORE MERGING THIS PR!


Update kubewarden-defaults chart patch version

Update container image in the chart-values.yaml file

key "$.policyServer.image.tag" updated from "v1.7.0" to "v1.7.1", in file "charts/kubewarden-defaults/chart-values.yaml"

Update container image in the values.yaml file

key "$.policyServer.image.tag" updated from "v1.7.0" to "v1.7.1", in file "charts/kubewarden-defaults/values.yaml"

Bump chart patch version

key "$.version" updated from "1.7.2" to "1.7.3", in file "charts/kubewarden-defaults/Chart.yaml"

Bump chart patch version in annotations

key "$.annotations.'catalog.cattle.io/upstream-version'" updated from "1.7.1" to "1.7.3", in file "charts/kubewarden-defaults/Chart.yaml"


Updatecli logo

Created automatically by Updatecli

Options:

Most of Updatecli configuration is done via its manifest(s).

  • If you close this pull request, Updatecli will automatically reopen it, the next time it runs.
  • If you close this pull request and delete the base branch, Updatecli will automatically recreate it, erasing all previous commits made.

Feel free to report any issues at github.com/updatecli/updatecli.
If you find this tool useful, do not hesitate to star our GitHub repository as a sign of appreciation, and/or to tell us directly on our chat!

@github-actions github-actions bot requested a review from a team as a code owner October 4, 2023 09:04
@@ -35,29 +35,27 @@ policyServer:
#
# Example of usage:
# imagePullSecret: "mysecret"
imagePullSecret:
Copy link
Member

Choose a reason for hiding this comment

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

I wonder why these changes are suggested now..

Copy link
Member

Choose a reason for hiding this comment

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

unmangled them with 340dd5c.

Copy link
Member

@viccuad viccuad left a comment

Choose a reason for hiding this comment

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

LGTM, but the CI is not running on this kind of PRs. I would like to have the CI running before merging.
Edit: opened #324 for fixing the CI triggering.

@viccuad
Copy link
Member

viccuad commented Oct 4, 2023

This consumes kubewarden/policy-server#561 for a CVE fix by bumping deps.

@viccuad
Copy link
Member

viccuad commented Oct 4, 2023

Closing, and immediately reopening, to have a CI trigger on the PR.
This is a workaround documented at https://github.com/peter-evans/create-pull-request/blob/main/docs/concepts-guidelines.md#workarounds-to-trigger-further-workflow-runs.

@viccuad viccuad closed this Oct 4, 2023
@viccuad viccuad reopened this Oct 4, 2023
Signed-off-by: Víctor Cuadrado Juan <[email protected]>
@viccuad viccuad self-assigned this Oct 4, 2023
@viccuad viccuad self-requested a review October 4, 2023 13:00
@@ -34,7 +34,7 @@ annotations:
# optional ones:
catalog.cattle.io/hidden: true # Hide specific charts. Only use on CRD charts.
catalog.cattle.io/auto-install: kubewarden-crds=1.4.1
catalog.cattle.io/upstream-version: 1.7.1
catalog.cattle.io/upstream-version: 1.7.3
Copy link
Member

Choose a reason for hiding this comment

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

heh, this is a fix, seems that in last update we didn't bump the upstream version.

Copy link
Member

@flavio flavio left a comment

Choose a reason for hiding this comment

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

LGTM

@viccuad viccuad merged commit 2614ed6 into main Oct 4, 2023
6 checks passed
@flavio flavio deleted the updatecli_625dfe2888771091da61253ea66f6dde5acc3c14f693c4195db42374fd56a14a branch October 4, 2023 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants