docs: TLS and DNS Policy user guides #322
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #322 +/- ##
==========================================
+ Coverage 64.28% 64.97% +0.68%
==========================================
Files 35 35
Lines 3794 3794
==========================================
+ Hits 2439 2465 +26
+ Misses 1158 1131 -27
- Partials 197 198 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
mikenairn
commented
Nov 21, 2023
mikenairn
commented
Nov 21, 2023
| @@ -4,9 +4,14 @@ apiVersion: kind.x-k8s.io/v1alpha4 | |||
| nodes: | |||
| - role: control-plane | |||
| image: kindest/node:v1.27.3 | |||
| # port forward 80 on the host to 80 on this node | |||
| extraPortMappings: | |||
| - containerPort: 30950 | |||
| hostPort: 9080 | |||
| - containerPort: 30951 | |||
| hostPort: 9443 | |||
There was a problem hiding this comment.
If we decide to use metalb in the default setup we can probably drop these ports and use 9080 -> 80, 9443 -> 443
mikenairn
force-pushed
the
tls_and_dns_guide_docs
branch
from
8889950 to
d5287a0
Compare
maleck13
reviewed
Nov 21, 2023
maleck13
reviewed
Nov 21, 2023
maleck13
reviewed
Nov 21, 2023
mikenairn
force-pushed
the
tls_and_dns_guide_docs
branch
from
392b962 to
f076e65
Compare
mikenairn
force-pushed
the
tls_and_dns_guide_docs
branch
from
f076e65 to
aac54d8
Compare
mikenairn
force-pushed
the
tls_and_dns_guide_docs
branch
from
aac54d8 to
f0b9534
Compare
* Add deploy-policy-controller and undeploy-policy-controller, deploys/undeploys the policy-controller into/from the kuadrant-system namespace. * Update dns and tls guides with extra step to install policy-controller.
mikenairn
force-pushed
the
tls_and_dns_guide_docs
branch
from
f0b9534 to
e9504db
Compare
|
@mikenairn the issue is with the 8080 port in the tests |
Yeah sure, 8080 was probably a bad choice |
maleck13
reviewed
Nov 28, 2023
maleck13
approved these changes
Nov 28, 2023
maleck13
pushed a commit
to maleck13/kuadrant-operator
that referenced
this pull request
Dec 12, 2023
* docs: TLS and DNS Policy user guides * docs: Update after removal of policy-controller from bundle * Add deploy-policy-controller and undeploy-policy-controller, deploys/undeploys the policy-controller into/from the kuadrant-system namespace. * Update dns and tls guides with extra step to install policy-controller. * Add script to generate MetalLB IPAddressPool for docker network. * Change kind cluster ports 80 -> 9081, 443 -> 9444
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add user guide documentation for TLS and DNS Policy focused on the single cluster context and using the simple routing strategy for DNS.
To support the DNS and TLS policy guides, updates were made to install and configure, metallb and the policy controller:
make install-metallb