-
Notifications
You must be signed in to change notification settings - Fork 720
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
[dev-v2.9] Bumps rancher pushprox to v0.1.4-rancher2 #4973
base: dev-v2.9
Are you sure you want to change the base?
Conversation
Validation steps
Ex:- longhorn-controller: repository: rancher/hardened-sriov-cni tag: v2.6.3-build20230913
|
12cc9e4
to
0330783
Compare
Validation steps
Ex:- longhorn-controller: repository: rancher/hardened-sriov-cni tag: v2.6.3-build20230913
|
0330783
to
c2f06ff
Compare
Validation steps
Ex:- longhorn-controller: repository: rancher/hardened-sriov-cni tag: v2.6.3-build20230913
|
c2f06ff
to
20d6a93
Compare
Validation steps
Ex:- longhorn-controller: repository: rancher/hardened-sriov-cni tag: v2.6.3-build20230913
|
...nitoring/rancher-monitoring/generated-changes/overlay/crds/crds/crd-alertmanagerconfigs.yaml
Outdated
Show resolved
Hide resolved
...ncher-monitoring/generated-changes/patch/templates/prometheus-operator/deployment.yaml.patch
Outdated
Show resolved
Hide resolved
20d6a93
to
2600bd4
Compare
Validation steps
Ex:- longhorn-controller: repository: rancher/hardened-sriov-cni tag: v2.6.3-build20230913
|
2600bd4
to
fc57b5d
Compare
Validation steps
Ex:- longhorn-controller: repository: rancher/hardened-sriov-cni tag: v2.6.3-build20230913
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I'm giving the PR an approval but since this is a backport for an issue @mallardduck worked on originally, it probably makes sense to get his approval as well before merging
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Think we still have to wait to merge until after freeze is lifted tomorrow (or Thursday)?
Issue: rancher/rancher#48704
Backport of #4856
Solution:
This PR bumps the PushProx image tag to: https://github.com/rancher/PushProx/releases/tag/v0.1.4-rancher2
And, creates a new chart for rancher-monitoring with updated pushprox image.