Closed orenc1 closed 1 year ago
Hi @orenc1. Thanks for your PR.
I'm waiting for a nmstate member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
hi @qinqon , could you please comment here: /ok-to-test ? thanks
@orenc1: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test
message.
/ok-to-test
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: phoracek
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Is this a BUG FIX or a FEATURE ?:
What this PR does / why we need it: When a single-stack IPv6 cluster is being used, the nginx server isn't listening on IPv6 requests, just IPv4. Changing the nginx configuration
listen
directive to[::]:9443 ipv6only=off ssl
will accept connections both for IPv6 and IPv4. Also, parametrize the service port. Special notes for your reviewer:Release note: