[release-4.21] OCPBUGS-85781: Add AWS ISO domains to konnectivity IsCloudAPI#8531
Conversation
The konnectivity proxy was missing AWS ISO (classified) region domains from its cloud API detection. This prevented the ingress operator from adding these domains to the NO_PROXY list, blocking direct communication with endpoints in those namespaces. Add the following AWS ISO domain suffixes: - .c2s.ic.gov (AWS ISO / C2S) - .hci.ic.gov (AWS ISO / HCI) - .sc2s.sgov.gov (AWS ISO-B / SC2S) Fixes: OCPBUGS-77040 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@openshift-cherrypick-robot: Jira Issue OCPBUGS-85779 has been cloned as Jira Issue OCPBUGS-85781. Will retitle bug to link to clone. DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
@openshift-cherrypick-robot: This pull request references Jira Issue OCPBUGS-85781, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/jira refresh |
|
@csrwng: This pull request references Jira Issue OCPBUGS-85781, which is valid. The bug has been moved to the POST state. 7 validation(s) were run on this bug
DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: csrwng, openshift-cherrypick-robot The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/verified by unit test |
|
@csrwng: This PR has been marked as verified by DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
Analysis of NTO failure: NTO operator/operand version mismatch delayed PerformanceProfile processing beyond test timeout The test creates a PerformanceProfile ConfigMap and waits up to 10 minutes for the NTO to produce a corresponding status ConfigMap (labeled hypershift.openshift.io/nto-generated-performance-profile-status: "true"). However, the NTO's PerformanceProfile controller was stuck in a retry loop logging "operator and operand release versions do not match" for approximately 13 minutes (00:32:37 → 00:45:57), refusing to reconcile the PerformanceProfile until versions aligned. The status ConfigMap was eventually created at 00:46:07, but the test's 10-minute timeout had already expired around ~00:42:40. The test context was exhausted, causing even the client rate limiter to report context deadline exceeded. Root cause: During initial cluster convergence, the NTO operator image version and the operand (tuned daemon) version did not match. The NTO explicitly gates PerformanceProfile processing on version alignment (performanceprofile_controller.go:413). With 23 parallel sub-tests all operating on the same HostedCluster (including upgrade tests like TestNodePoolReplaceUpgrade, TestRollingUpgrade, and TestNodePoolInPlaceUpgrade), the convergence took over 13 minutes, exceeding the test's 10-minute budget. This is a flaky test / timing issue, not a code regression. The PerformanceProfile was ultimately processed correctly — the status ConfigMap exists in the artifact dump with valid conditions (Available=True, Degraded=False). Given that this is not related to the PR, overriding the e2e-aws test |
|
@csrwng: Overrode contexts on behalf of csrwng: ci/prow/e2e-aws DetailsIn 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-sigs/prow repository. |
|
@openshift-cherrypick-robot: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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-sigs/prow repository. I understand the commands that are listed here. |
6d6591b
into
openshift:release-4.21
|
@openshift-cherrypick-robot: Jira Issue Verification Checks: Jira Issue OCPBUGS-85781 Jira Issue OCPBUGS-85781 has been moved to the MODIFIED state and will move to the VERIFIED state when the change is available in an accepted nightly payload. 🕓 DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
This is an automated cherry-pick of #8447
/assign csrwng