Skip to content

Don't use short-name - #486

Closed
jianzhangbjz wants to merge 1 commit into
openshift:mainfrom
jianzhangbjz:curl
Closed

Don't use short-name#486
jianzhangbjz wants to merge 1 commit into
openshift:mainfrom
jianzhangbjz:curl

Conversation

@jianzhangbjz

@jianzhangbjzjianzhangbjz commented Sep 25, 2025

Copy link
Copy Markdown
Contributor

Try to address https://storage.googleapis.com/test-platform-results/pr-logs/pull/openshift_operator-framework-operator-controller/484/pull-ci-openshift-operator-framework-operator-controller-main-openshift-e2e-aws/1970899227856867328/build-log.txt

 metrics_test.go:162:  Error Trace:	/go/src/github.com/openshift/operator-framework-operator-controller/test/e2e/metrics_test.go:162 /go/src/github.com/openshift/operator-framework-operator-controller/test/e2e/metrics_test.go:102 /go/src/github.com/openshift/operator-framework-operator-controller/test/e2e/metrics_test.go:48 Error: Received unexpected error: exit status 1 Test: TestOperatorControllerMetricsExportedEndpoint Messages: Error waiting for curl pod to be ready: error: timed out waiting for the condition on pods/oper-curl-metrics
 Events: Type Reason Age From Message ---- ------ ---- ---- ------- Normal Scheduled 60s default-scheduler Successfully assigned testns-swmtc7tc/oper-curl-metrics to ip-10-0-67-225.ec2.internal Normal AddedInterface 59s multus Add eth0 [10.128.2.19/23] from ovn-kubernetes Warning InspectFailed 13s (x6 over 59s) kubelet Failed to inspect image "": rpc error: code = Unknown desc = short name mode is enforcing, but image name curlimages/curl:8.15.0 returns ambiguous list Warning Failed 13s (x6 over 59s) kubelet Error: ImageInspectError

More on Slack: https://redhat-internal.slack.com/archives/C06KP34REFJ/p1758735242789949

@openshift-ci

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jianzhangbjz

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ciopenshift-ciBot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 25, 2025
@jianzhangbjz
jianzhangbjzforce-pushed the curl branch 2 times, most recently from 63706dd to e8b8e3eCompareSeptember 25, 2025 02:14
@jianzhangbjz

Copy link
Copy Markdown
ContributorAuthor

This fix has been added to #485

@tmshort

Copy link
Copy Markdown
Contributor

Since this has started, might as well see if it works

@tmshort

Copy link
Copy Markdown
Contributor

/hold
We'd want this to be a <drop> commit, because we'd likely do it upstream

@openshift-ciopenshift-ciBot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 25, 2025
@openshift-ci

Copy link
Copy Markdown
Contributor

@jianzhangbjz: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test nameCommitDetailsRequiredRerun command
ci/prow/openshift-e2e-aws6521ed6linktrue/test openshift-e2e-aws
ci/prow/openshift-e2e-aws-techpreview6521ed6linkfalse/test openshift-e2e-aws-techpreview

Full PR test history. Your PR dashboard.

Details

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. I understand the commands that are listed here.

@jianzhangbjz

Copy link
Copy Markdown
ContributorAuthor

Closed it since the fix has been added to operator-framework/operator-controller#2235

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by an approver from all required OWNERS files.do-not-merge/holdIndicates that a PR should not merge because someone has issued a /hold command.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@jianzhangbjz@tmshort