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
Bug 1794376: Remove IPv6 dualstack feature gate #758
Bug 1794376: Remove IPv6 dualstack feature gate #758
Conversation
* openshift/api@a63a88e3: Bug 1769015: Add kubebuilder annotations to the network types * openshift/api@e205a72a: Update generated files * openshift/api@3bf8cd9d: Update swagger docs * openshift/api@2b89f314: Call add-crd-gen in the Makefile * openshift/api@f92f2cd9: add deprecaction notice for build pipeline strategy * openshift/api@0bffa333: update/verify various generated * openshift/api@1772b9d5: Remove Description from CLI output to improve its display * openshift/api@74333963: Add CRD generator documentation to Readme * openshift/api@60d997e9: config/v1/types_proxy: Clarify trustedCA semantics * openshift/api@709bcf89: Bug 1784532: Clarify `from` behavior in builds * openshift/api@38db8664: make update * openshift/api@2dfe621f: config: disable IPv6DualStack feature flag * openshift/library-go@a3fec49c: Remove alpha-build-machinery * openshift/library-go@387645fc: add methods to manipulation observed configuration values * openshift/library-go@80c146a9: Migrate to openshift/build-machinery-go * openshift/library-go@6b4b515b: bump: openshift/build-machinery-go * openshift/library-go@39cf0ddc: Add alpha-build-machinery warning * openshift/library-go@9e4693a8: add substitution for NODE_NAME inside of static pods * openshift/library-go@89976f0a: add cpu limits to become guaranteed * openshift/library-go@58b2341b: bump(*): vendor update * openshift/library-go@042e7f26: adjust featuregates unit test for IPv6DualStack removal
/bugzilla refresh |
@mfojtik: This pull request references Bugzilla bug 1794376, which is valid. The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@mfojtik: This pull request references Bugzilla bug 1794376, which is valid. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@@ -106,7 +105,7 @@ func NewConfigObserver( | |||
[]string{"apiServerArguments", "cloud-config"}), | |||
featuregates.NewObserveFeatureFlagsFunc( | |||
nil, | |||
sets.NewString("IPv6DualStack"), // IPv6DualStack is bugged, don't turn it on | |||
nil, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @deads2k ... we removed that feature gate, so we don't have to black list this anymore...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/hold
for the prereq to merge first
/hold cancel @soltysh this does not need any pre-reqs |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mfojtik, soltysh, tnozicka The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest Please review the full test history for this PR and help us cut down flakes. |
4 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@mfojtik: All pull requests linked via external trackers have merged. Bugzilla bug 1794376 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Builds on #757