Skip to content

ingress: add 443 as listener in cec#44229

Merged
joestringer merged 1 commit intocilium:mainfrom
xtineskim:ingress-shared-svc-fix
Feb 24, 2026
Merged

ingress: add 443 as listener in cec#44229
joestringer merged 1 commit intocilium:mainfrom
xtineskim:ingress-shared-svc-fix

Conversation

@xtineskim
Copy link
Copy Markdown
Member

@xtineskim xtineskim commented Feb 6, 2026

We saw that there was an issue where AWS' nlb on an ingress with a shared loadbalancer would report an unhealthy port 443 after an version upgrade from 1.14 to 1.17.

This adds port 443 to the cec for an ingress that opens the listener to receive healthchecks.
Testing with nc on port 80 and port 443 both now output a successful connection

Related to #34093, specifically this comment.

ingress: Ensure that the shared ingress exposes port 443 so that it can pass upstream loadbalancer health checks.

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Feb 6, 2026
@xtineskim xtineskim force-pushed the ingress-shared-svc-fix branch 5 times, most recently from b79a079 to 91b6411 Compare February 12, 2026 17:19
@xtineskim xtineskim requested a review from youngnick February 12, 2026 17:33
@xtineskim xtineskim added the area/servicemesh GH issues or PRs regarding servicemesh label Feb 12, 2026
@xtineskim xtineskim marked this pull request as ready for review February 12, 2026 17:35
@xtineskim xtineskim requested a review from a team as a code owner February 12, 2026 17:35
@xtineskim
Copy link
Copy Markdown
Member Author

/test

@xtineskim xtineskim force-pushed the ingress-shared-svc-fix branch 2 times, most recently from 3617e1d to 34eefc9 Compare February 18, 2026 19:30
@xtineskim xtineskim added needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch labels Feb 19, 2026
Signed-off-by: xtine <xtineskim@gmail.com>
@xtineskim xtineskim force-pushed the ingress-shared-svc-fix branch from 34eefc9 to f502b3f Compare February 19, 2026 20:26
@xtineskim
Copy link
Copy Markdown
Member Author

/test

@liyihuang liyihuang added affects/v1.17 This issue affects v1.17 branch release-note/bug This PR fixes an issue in a previous release of Cilium. labels Feb 24, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Feb 24, 2026
@joestringer joestringer added affects/v1.18 This issue affects v1.18 branch and removed needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch labels Feb 24, 2026
@joestringer
Copy link
Copy Markdown
Member

Would you mind updating the PR description with a section something like this for a one-liner release note communicating the user impact? This might make it a bit easier for users to know if they want this fix or not.

```release-note
Fix a bug where ...
```

@joestringer joestringer added this pull request to the merge queue Feb 24, 2026
Merged via the queue into cilium:main with commit b639b26 Feb 24, 2026
78 of 79 checks passed
@YutaroHayakawa YutaroHayakawa mentioned this pull request Feb 24, 2026
21 tasks
@YutaroHayakawa YutaroHayakawa added backport-pending/1.19 The backport for Cilium 1.19.x for this PR is in progress. and removed needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch labels Feb 24, 2026
@github-actions github-actions bot added backport-done/1.19 The backport for Cilium 1.19.x for this PR is done. and removed backport-pending/1.19 The backport for Cilium 1.19.x for this PR is in progress. labels Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

affects/v1.17 This issue affects v1.17 branch affects/v1.18 This issue affects v1.18 branch area/servicemesh GH issues or PRs regarding servicemesh backport-done/1.19 The backport for Cilium 1.19.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/bug This PR fixes an issue in a previous release of Cilium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants