Skip to content

Conversation

@damsien
Copy link
Contributor

@damsien damsien commented Aug 21, 2025

⚠️ This introduces an API definition breaking change.

This PR aims to use the APIServerPort field from the Cluster object instead of the previous Port from ScalewayCluster object to configure the LB exposing the kube-apiserver .

Copy link
Member

@Tomy2e Tomy2e left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add or update the existing e2e tests to use a custom frontend port?

@damsien damsien changed the base branch from main to custom-lb-port August 22, 2025 12:34
@damsien damsien changed the base branch from custom-lb-port to main August 25, 2025 13:24
@damsien damsien force-pushed the main branch 2 times, most recently from 2d45d48 to b834f5a Compare August 26, 2025 06:05
@damsien damsien changed the base branch from main to custom-lb-port August 26, 2025 06:05
@Tomy2e Tomy2e changed the title Move lb port definition from ScalewayCluster to Cluster Move lb frontend port definition from ScalewayCluster to Cluster Aug 26, 2025
@damsien damsien merged commit b59c847 into scaleway:custom-lb-port Aug 26, 2025
2 checks passed
damsien added a commit that referenced this pull request Aug 26, 2025
#30)

Move lb port definition from ScalewayCluster to Cluster

Signed-off-by: Damien DASSIEU <[email protected]>
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