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

Fix egress interfaces script to work on IPI clusters #87

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

simu
Copy link
Member

@simu simu commented Aug 6, 2024

We adjust the script to use the external API DNS record (api.) instead of api-int, so that it works without any problems on IPI clusters which provide the api-int record via in-cluster CoreDNS which isn't running yet before the kubelet is started.

Checklist

  • The PR has a meaningful title. It will be used to auto-generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Update the documentation.
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.

@simu simu added the bug Something isn't working label Aug 6, 2024
@simu simu force-pushed the fix/egress-interfaces-machineconfig branch 2 times, most recently from b63133c to a995b2a Compare August 6, 2024 11:50
@simu simu marked this pull request as ready for review August 6, 2024 12:30
@simu simu requested a review from a team as a code owner August 6, 2024 12:30
We adjust the script to use the external API DNS record
(api.<cluster-domain>) instead of api-int, so that it works without any
problems on IPI clusters which provide the api-int record via in-cluster
CoreDNS which isn't running yet before the kubelet is started.
@simu simu force-pushed the fix/egress-interfaces-machineconfig branch from a995b2a to 0425f8a Compare September 2, 2024 11:14
@simu simu added the bump:patch label Sep 2, 2024
Copy link

github-actions bot commented Sep 2, 2024

🚀 This PR has been released as v4.1.2

Triggering workflows Release

🛠️ Auto tagging enabled with label bump:patch

@haasad haasad merged commit 063dce5 into master Sep 3, 2024
18 checks passed
@haasad haasad deleted the fix/egress-interfaces-machineconfig branch September 3, 2024 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working bump:patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants