Skip to content

ensure ingress port is available#368

Merged
nimakaviani merged 1 commit intocnoe-io:mainfrom
nabuskey:port-check
Aug 27, 2024
Merged

ensure ingress port is available#368
nimakaviani merged 1 commit intocnoe-io:mainfrom
nabuskey:port-check

Conversation

@nabuskey
Copy link
Copy Markdown
Collaborator

@nabuskey nabuskey commented Aug 26, 2024

fixes: #358, #366

With this we always insert the ingress port we need when a custom kind configuration is provided.

Signed-off-by: Manabu McCloskey <manabu.mccloskey@gmail.com>
Copy link
Copy Markdown
Contributor

@nimakaviani nimakaviani left a comment

Choose a reason for hiding this comment

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

LGTM

@nimakaviani nimakaviani merged commit 385c649 into cnoe-io:main Aug 27, 2024
@nabuskey nabuskey deleted the port-check branch August 27, 2024 16:47
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.

[Bug]: Validate and ensure the container port we need is available and correct.

2 participants