-
Notifications
You must be signed in to change notification settings - Fork 527
chore(deps): update registry.k8s.io/coredns/coredns docker tag to v1.12.0 #10909
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
chore(deps): update registry.k8s.io/coredns/coredns docker tag to v1.12.0 #10909
Conversation
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.
/hold
Requires a change like this one:
2f63ea8
(#10224)
I'll check where Renovate's configuration could be updated to catch both occurrences.
gardener/imagevector/containers.yaml Line 395 in 34b7d92
|
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
@@ -32,6 +32,7 @@ | |||
'^example/.+\\.yaml$', | |||
'^hack/.+\\.yaml$', | |||
'^\\.test-defs/.+\\.yaml$', | |||
'^charts/.+\\.yaml$', |
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.
This includes YAML files in the charts
folder for Renovate to update image
tags such as this one:
2f63ea8
(#10224)
Regex test:
https://regex101.com/r/dvfQdb/1
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.
/cc @oliver-goetz
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.
You could also remove this line. It was meant to prevent false positives in chart updates. Apparently, it does not prevent anything anyway as there is a typo in the folder name 😅
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.
Fair enough and well spotted :D
Done via 3e48f0a
(#10909)
Unfortunately, the One of the related issues is still open: As well as a PR (coredns/coredns#6320) that was meant to remove the hardcoded I'll try friendly nudging the issue & PR and might try contributing a change as well. |
/unhold |
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
LGTM label has been added. Git tree hash: 090b5474f7a505bce6510a142ab1d159ac060134
|
…d from coredns" This reverts commit ad6d243.
…ERVICE capability
9278efd
to
3e48f0a
Compare
@@ -389,7 +390,6 @@ | |||
{ | |||
// Ignore paths which most likely create false positives. | |||
matchFileNames: [ | |||
'chart/**', |
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.
See: #10909 (comment)
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.
Thanks 🚀
/lgtm
LGTM label has been added. Git tree hash: 04b21c64bb60f69e95edea71ccb27786003e1ed6
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: oliver-goetz The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This PR contains the following updates:
v1.11.4
->v1.12.0
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
Release note: