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

Add ingress_port checks #25

Merged
merged 1 commit into from
Feb 3, 2021
Merged

Add ingress_port checks #25

merged 1 commit into from
Feb 3, 2021

Conversation

frenck
Copy link
Owner

@frenck frenck commented Feb 3, 2021

This PR adds additional checks for when the add-on has enabled Ingress.

  • When the add-on is running on the host network. The ingress_port should be 0.
  • When the add-on is NOT running on the host network. The ingress_port should not be 0.

@frenck frenck added the new-feature New features or options. label Feb 3, 2021
@frenck frenck merged commit 9ec248e into main Feb 3, 2021
@frenck frenck deleted the frenck-2021-0515 branch February 3, 2021 08:09
@frenck frenck restored the frenck-2021-0515 branch February 3, 2021 08:30
@github-actions github-actions bot locked and limited conversation to collaborators Feb 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new-feature New features or options.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant