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

Update GlobalIngressIP internal service on exported service change #2828

Merged
merged 2 commits into from
Dec 11, 2023

Conversation

tpantelis
Copy link
Contributor

...specifically the ports. Also added an E2E test that changes the service port.

Fixes #1738

Depends on submariner-io/admiral#811

@tpantelis tpantelis self-assigned this Dec 8, 2023
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr2828/tpantelis/gn_service_update
🚀 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.

@tpantelis tpantelis added the ready-to-test When a PR is ready for full E2E testing label Dec 8, 2023
@github-actions github-actions bot removed the dependent label Dec 9, 2023
Copy link
Contributor

github-actions bot commented Dec 9, 2023

This PR/issue depends on:

@tpantelis tpantelis added the release-note-needed Should be mentioned in the release notes label Dec 11, 2023
@aswinsuryan aswinsuryan merged commit 54d6d18 into submariner-io:devel Dec 11, 2023
47 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr2828/tpantelis/gn_service_update]

tpantelis added a commit to tpantelis/submariner-website that referenced this pull request Dec 12, 2023
tpantelis added a commit to submariner-io/submariner-website that referenced this pull request Dec 21, 2023
@tpantelis tpantelis deleted the gn_service_update branch January 6, 2024 14:57
tpantelis added a commit to tpantelis/submariner-website that referenced this pull request Feb 27, 2024
tpantelis added a commit to submariner-io/submariner-website that referenced this pull request Feb 29, 2024
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
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

GlobalNet - exported service isn't reachable using desired port after service port edit
5 participants