Skip to content

cluster: support setting externalTrafficPolicy for svc#1293

Merged
makhov merged 1 commit intok0sproject:mainfrom
clwluvw:externaltrafficpolicy
Nov 21, 2025
Merged

cluster: support setting externalTrafficPolicy for svc#1293
makhov merged 1 commit intok0sproject:mainfrom
clwluvw:externaltrafficpolicy

Conversation

@clwluvw
Copy link
Contributor

@clwluvw clwluvw commented Nov 16, 2025

Allow setting externalTrafficPolicy for the api service. useful when using NodePort and LoadBalancer types.

@clwluvw clwluvw requested a review from a team as a code owner November 16, 2025 19:07
@clwluvw clwluvw force-pushed the externaltrafficpolicy branch from ce173f7 to 50dccea Compare November 17, 2025 18:02
@clwluvw
Copy link
Contributor Author

clwluvw commented Nov 20, 2025

Hi @makhov @apedriza - shall we retry the failed job? I can't see this be related to my change.

@clwluvw
Copy link
Contributor Author

clwluvw commented Nov 20, 2025

@makhov seems to be failing for the same reason again.

@clwluvw
Copy link
Contributor Author

clwluvw commented Nov 20, 2025

@makhov still failing :D

@makhov
Copy link
Contributor

makhov commented Nov 21, 2025

Please, rebase you branch on top of the latest main. I think it should fix the issue.

Allow setting externalTrafficPolicy for the api service. useful
when using NodePort and LoadBalancer types.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
@clwluvw clwluvw force-pushed the externaltrafficpolicy branch from 50dccea to 9fc9028 Compare November 21, 2025 09:19
@clwluvw
Copy link
Contributor Author

clwluvw commented Nov 21, 2025

Please, rebase you branch on top of the latest main. I think it should fix the issue.

Thanks @makhov. I guess you need to approve the workflow now.

@clwluvw
Copy link
Contributor Author

clwluvw commented Nov 21, 2025

Thanks @makhov, it should be ready for a review now.

@makhov makhov merged commit 09207e5 into k0sproject:main Nov 21, 2025
54 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants