Closed openshift-cherrypick-robot closed 2 months ago
@openshift-cherrypick-robot: Jira Issue OCPBUGS-33566 has been cloned as Jira Issue OCPBUGS-34117. Will retitle bug to link to clone. /retitle [release-4.16] OCPBUGS-34117: AWS STS should not error when a credentailsRequest does not have awsSTSIAMRoleARN
@openshift-cherrypick-robot: This pull request references Jira Issue OCPBUGS-34117, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
The bug has been updated to refer to the pull request using the external bug tracker.
Attention: Patch coverage is 50.00000%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 48.35%. Comparing base (
48b287d
) to head (2401b32
).
/retest-required /override ci/prow/security
@jstuever: Overrode contexts on behalf of jstuever: ci/prow/security
/jira refresh
@jianping-shu: This pull request references Jira Issue OCPBUGS-34117, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @jianping-shu
Verified with cluster-bot build. AWS STS cluster installed successfully. No error log "empty awsSTSIAMRoleARN" any more.
jianpingshu@jshu-mac ~ % oc get clusterversion NAME VERSION AVAILABLE PROGRESSING SINCE STATUS version 4.16.0-0.ci.test-2024-05-26-010010-ci-ln-6r9dy9t-latest True False 38m Cluster version is 4.16.0-0.ci.test-2024-05-26-010010-ci-ln-6r9dy9t-latest
jianpingshu@jshu-mac ~ % oc -n openshift-cloud-credential-operator get -o json credentialsrequests | jq -r '.items[] | select(tostring | contains("InfrastructureMismatch") | not) | .metadata.name as $n | .status.conditions // [{type: "NoConditions"}] | .[] | .type + "=" + .status + " " + $n + " " + .reason + ": " + .message' | sort NoConditions= aws-ebs-csi-driver-operator : NoConditions= cloud-credential-operator-iam-ro : NoConditions= openshift-cloud-network-config-controller-aws : NoConditions= openshift-image-registry : NoConditions= openshift-ingress : NoConditions= openshift-machine-api-aws :
jianpingshu@jshu-mac ~ % oc logs cloud-credential-operator-785c8f9445-g8xgx -n openshift-cloud-credential-operator -c cloud-credential-operator > cloud-credential-operator-785c8f9445-g8xgx-416.log
jianpingshu@jshu-mac OCPBUGS-34117 % grep "an empty awsSTSIAMRoleARN was found" cloud-credential-operator-785c8f9445-g8xgx-416.log jianpingshu@jshu-mac OCPBUGS-34117 %
/retest
/retest
/label cherry-pick-approved
@jstuever Kindly remind this PR since 4.16 GA is upcoming.
/cherry-pick release-6.15 release-6.14
@jstuever: once the present PR merges, I will cherry-pick it on top of release-6.15 in a new PR and assign it to you.
/cherrypick release-6.15 release-6.14
@jstuever: once the present PR merges, I will cherry-pick it on top of release-6.15 in a new PR and assign it to you.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstuever, openshift-cherrypick-robot
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/label backport-risk-assessed
/retest-required
Remaining retests: 0 against base HEAD 48b287d41e71b23c55b00cbcbfb38ac1eedfb9f4 and 2 for PR HEAD 2401b324ce9ee49b66b372d1ff73b7e89307d4e0 in total
/override ci/prow/security
@jstuever: Overrode contexts on behalf of jstuever: ci/prow/security
@openshift-cherrypick-robot: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
ci/prow/security | 2401b324ce9ee49b66b372d1ff73b7e89307d4e0 | link | true | /test security |
Full PR test history. Your PR dashboard.
@openshift-cherrypick-robot: Jira Issue OCPBUGS-34117: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-34117 has been moved to the MODIFIED state.
@jstuever: cannot checkout release-6.15
: error checking out "release-6.15": exit status 1 error: pathspec 'release-6.15' did not match any file(s) known to git
Fixing the presumed 6
vs 4
typo, to save Jeremiah some time.
/cherrypick release-4.15 release-4.14
@wking: new pull request created: #709
[ART PR BUILD NOTIFIER]
This PR has been included in build ose-cloud-credential-operator-container-v4.16.0-202406280629.p0.gc6c1b0b.assembly.stream.el9 for distgit ose-cloud-credential-operator. All builds following this will include this PR.
This is an automated cherry-pick of #703
/assign jstuever