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

Honor publishNotReadyAddresses flag for headless services #2646

Merged
merged 3 commits into from
Aug 20, 2023

Conversation

tpantelis
Copy link
Contributor

@tpantelis tpantelis commented Aug 10, 2023

Lighthouse honors this flag when publishing endpoint addresses so Globalnet needs to as well when allocating a global IP for ingress.

Fixes submariner-io/lighthouse#1342

Lighthouse honors this flag when publishing endpoint addresses so
Globalnet needs to as well when allocating a global IP for ingress.

Fixes submariner-io/lighthouse#1342

Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr2646/tpantelis/globalip_pod_not_ready
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label Aug 16, 2023
@sridhargaddam
Copy link
Member

@tpantelis did we add a release note when the Lighthouse change got merged?

@skitt skitt enabled auto-merge (rebase) August 16, 2023 12:59
@tpantelis
Copy link
Contributor Author

@tpantelis did we add a release note when the Lighthouse change got merged?

yes - https://github.com/submariner-io/submariner-website/blob/c77237c3784b48b39cdffc2e3ab1b562dd231bd9/src/content/community/releases/_index.en.md?plain=1#L45

@skitt skitt merged commit c7d6aae into submariner-io:devel Aug 20, 2023
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr2646/tpantelis/globalip_pod_not_ready]

@dfarrell07 dfarrell07 added the release-note-needed Should be mentioned in the release notes label Aug 29, 2023
@tpantelis tpantelis deleted the globalip_pod_not_ready branch October 24, 2023 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-test When a PR is ready for full E2E testing release-note-handled release-note-needed Should be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GlobalIP not assigned in pod "init" phase
6 participants