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(chart): bump snapshot-controller and snapshot-validation-webhook to v6.3.3 (backport #5107) #5672

Merged
merged 3 commits into from
Apr 29, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Apr 24, 2024

Test plan:

  1. Create a Harvester cluster.
  2. Check snapshot-controller and webhook is v6.3.3.
    This is an automatic backport of pull request feat(chart): bump snapshot-controller and snapshot-validation-webhook to v6.3.3 #5107 done by Mergify.

Copy link
Author

mergify bot commented Apr 24, 2024

Cherry-pick of 88d78c3 has failed:

On branch mergify/bp/v1.2/pr-5107
Your branch is up to date with 'origin/v1.2'.

You are currently cherry-picking commit 88d78c3c.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   deploy/charts/harvester/Chart.yaml
	deleted:    deploy/charts/harvester/charts/csi-snapshotter-0.1.0.tgz
	new file:   deploy/charts/harvester/charts/csi-snapshotter-0.2.0.tgz
	deleted:    deploy/charts/harvester/charts/snapshot-validation-webhook-0.1.0.tgz
	new file:   deploy/charts/harvester/charts/snapshot-validation-webhook-0.2.0.tgz
	modified:   deploy/charts/harvester/dependency_charts/csi-snapshotter/Chart.yaml
	modified:   deploy/charts/harvester/dependency_charts/csi-snapshotter/crds/volumesnapshotclasses.yaml
	modified:   deploy/charts/harvester/dependency_charts/csi-snapshotter/crds/volumesnapshotcontents.yaml
	modified:   deploy/charts/harvester/dependency_charts/csi-snapshotter/crds/volumesnapshots.yaml
	modified:   deploy/charts/harvester/dependency_charts/csi-snapshotter/templates/deployment.yaml
	modified:   deploy/charts/harvester/dependency_charts/csi-snapshotter/templates/rbac.yaml
	modified:   deploy/charts/harvester/dependency_charts/csi-snapshotter/templates/snapshotclass.yaml
	modified:   deploy/charts/harvester/dependency_charts/csi-snapshotter/values.yaml
	modified:   deploy/charts/harvester/dependency_charts/snapshot-validation-webhook/Chart.yaml
	modified:   deploy/charts/harvester/dependency_charts/snapshot-validation-webhook/templates/deployment.yaml
	modified:   deploy/charts/harvester/dependency_charts/snapshot-validation-webhook/templates/rbac.yaml
	modified:   deploy/charts/harvester/dependency_charts/snapshot-validation-webhook/templates/webhook.yaml
	modified:   deploy/charts/harvester/dependency_charts/snapshot-validation-webhook/values.yaml
	modified:   deploy/charts/harvester/templates/longhorn-volumesnapshotclass.yaml
	modified:   deploy/charts/harvester/values.yaml

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   deploy/charts/harvester/Chart.lock

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

FrankYang0529 and others added 3 commits April 24, 2024 14:46
… to v6.3.3

Signed-off-by: PoAn Yang <poan.yang@suse.com>
(cherry picked from commit 88d78c3)
…hook-deployment

rollback ci changes

(cherry picked from commit 0f7dd18)
Signed-off-by: PoAn Yang <poan.yang@suse.com>
(cherry picked from commit 51e8e74)
Copy link
Contributor

@Vicente-Cheng Vicente-Cheng left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link
Member

@WebberHuang1118 WebberHuang1118 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

@Vicente-Cheng Vicente-Cheng self-requested a review April 24, 2024 09:40
@bk201 bk201 removed the conflicts label Apr 26, 2024
Copy link
Contributor

@Vicente-Cheng Vicente-Cheng left a comment

Choose a reason for hiding this comment

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

verified with the test plan
Thanks!

Copy link
Member

@bk201 bk201 left a comment

Choose a reason for hiding this comment

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

lgtm!

@bk201 bk201 merged commit 642139a into v1.2 Apr 29, 2024
4 checks passed
@mergify mergify bot deleted the mergify/bp/v1.2/pr-5107 branch April 29, 2024 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants