Skip to content

NO-ISSUE: Turn on more debugging for oc-mirror#729

Open
zaneb wants to merge 1 commit into
openshift:masterfrom
zaneb:oc-mirror-debug
Open

NO-ISSUE: Turn on more debugging for oc-mirror#729
zaneb wants to merge 1 commit into
openshift:masterfrom
zaneb:oc-mirror-debug

Conversation

@zaneb

@zaneb zaneb commented Jun 26, 2026

Copy link
Copy Markdown
Member

Increase the log-level for oc-mirror to debug, and log the output at
debug level on each attempt we make to run it, instead of just the first
one and only if all attempts fail.

This is more consistent with the code before retries were added, where
the output of oc-mirror was logged unconditionally at debug level.

@zaneb

zaneb commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

/test ?

@openshift-ci

openshift-ci Bot commented Jun 26, 2026

Copy link
Copy Markdown

@zaneb: The following commands are available to trigger required jobs:

/test e2e-compact-ipv4-static
/test images
/test lint
/test unit-test

The following commands are available to trigger optional jobs:

/test e2e-agent-compact-ipv4-iso-no-registry
/test e2e-sno-dualstack-dhcp

Use /test all to run all jobs.

Details

In response to this:

/test ?

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-ci

openshift-ci Bot commented Jun 26, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: zaneb

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-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 26, 2026
@zaneb zaneb force-pushed the oc-mirror-debug branch 3 times, most recently from 9c6516a to 2f66b83 Compare June 29, 2026 04:53
@zaneb zaneb changed the title Turn on more debugging for oc-mirror NO-ISSUE: Turn on more debugging for oc-mirror Jun 29, 2026
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jun 29, 2026
@openshift-ci-robot

Copy link
Copy Markdown

@zaneb: This pull request explicitly references no jira issue.

Details

In response to this:

Increase the log-level for oc-mirror to trace, and log the output on each of the 3 attempts we make to run it, not just the first one.

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.

@zaneb zaneb force-pushed the oc-mirror-debug branch from 2f66b83 to 5a3d79f Compare June 29, 2026 05:18
@openshift-ci openshift-ci Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 29, 2026
Increase the log-level for oc-mirror to debug, and log the output at
debug level on each attempt we make to run it, instead of just the first
one and only if all attempts fail.

This is more consistent with the code before retries were added, where
the output of oc-mirror was logged unconditionally at debug level.
@zaneb zaneb force-pushed the oc-mirror-debug branch from 5a3d79f to cc55c56 Compare June 29, 2026 23:42
@openshift-ci openshift-ci Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 29, 2026
@openshift-ci

openshift-ci Bot commented Jun 30, 2026

Copy link
Copy Markdown

@zaneb: all tests passed!

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.

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants