Skip to content

Conversation

gardener-ci-robot
Copy link
Contributor

This PR contains the following updates:

Package Update Change
registry.k8s.io/coredns/coredns minor 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.


  • If you want to rebase/retry this PR, check this box

Release note:

The `registry.k8s.io/coredns/coredns` image has been updated to `v1.12.0`.

@gardener-ci-robot gardener-ci-robot added the kind/enhancement Enhancement, improvement, extension label Nov 24, 2024
@gardener-prow gardener-prow bot added the cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. label Nov 24, 2024
@gardener-prow gardener-prow bot requested review from acumino and ScheererJ November 24, 2024 20:05
@gardener-prow gardener-prow bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Nov 24, 2024
Copy link
Member

@marc1404 marc1404 left a 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-prow gardener-prow bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 25, 2024
@marc1404
Copy link
Member

TODO to check:

# TODO(DockToFuture): When updating coredns to v1.12.x check if the NET_BIND_SERVICE capability can be removed.

@marc1404 marc1404 self-assigned this Nov 26, 2024
@gardener-ci-robot
Copy link
Contributor Author

Edited/Blocked Notification

Renovate 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.

⚠️ Warning: custom changes will be lost.

@gardener-prow gardener-prow bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 26, 2024
@@ -32,6 +32,7 @@
'^example/.+\\.yaml$',
'^hack/.+\\.yaml$',
'^\\.test-defs/.+\\.yaml$',
'^charts/.+\\.yaml$',
Copy link
Member

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

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

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 😅

Copy link
Member

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)

@gardener-prow gardener-prow bot requested a review from oliver-goetz November 27, 2024 09:13
@marc1404
Copy link
Member

marc1404 commented Nov 27, 2024

Unfortunately, the NET_BIND_SERVICE capability introduced with #8945 can not be removed yet.

One of the related issues is still open:
coredns/coredns#6249

As well as a PR (coredns/coredns#6320) that was meant to remove the hardcoded NET_BIND_SERVICE capability inside CoreDNS' Dockerfile again.

I'll try friendly nudging the issue & PR and might try contributing a change as well.

@marc1404
Copy link
Member

/unhold

@gardener-prow gardener-prow bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 27, 2024
Copy link
Member

@ScheererJ ScheererJ left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-prow gardener-prow bot added the lgtm Indicates that a PR is ready to be merged. label Nov 27, 2024
Copy link
Contributor

gardener-prow bot commented Nov 27, 2024

LGTM label has been added.

Git tree hash: 090b5474f7a505bce6510a142ab1d159ac060134

@marc1404 marc1404 force-pushed the renovate/registry.k8s.io-coredns-coredns-1.x branch from 9278efd to 3e48f0a Compare November 28, 2024 08:31
@gardener-prow gardener-prow bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 28, 2024
@@ -389,7 +390,6 @@
{
// Ignore paths which most likely create false positives.
matchFileNames: [
'chart/**',
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

@oliver-goetz oliver-goetz left a comment

Choose a reason for hiding this comment

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

Thanks 🚀
/lgtm

@gardener-prow gardener-prow bot added the lgtm Indicates that a PR is ready to be merged. label Nov 28, 2024
Copy link
Contributor

gardener-prow bot commented Nov 28, 2024

LGTM label has been added.

Git tree hash: 04b21c64bb60f69e95edea71ccb27786003e1ed6

Copy link
Contributor

gardener-prow bot commented Nov 28, 2024

[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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gardener-prow gardener-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 28, 2024
@gardener-prow gardener-prow bot merged commit e151a63 into master Nov 28, 2024
19 checks passed
@gardener-prow gardener-prow bot deleted the renovate/registry.k8s.io-coredns-coredns-1.x branch November 28, 2024 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. kind/enhancement Enhancement, improvement, extension lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants