OCPBUGS-33170: All containers must fallback to logs on error

[JoelSpeed]

In 4.17, we are enforcing this value for all OpenShift containers via tests, link in bug comment. This helps with debugging to ensure we have logs available.

[openshift-ci-robot]

@JoelSpeed: This pull request references Jira Issue OCPBUGS-33170, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug

• bug is open, matching expected state (open)
• bug target version (4.16.0) matches configured target version for branch (4.16.0)
• bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

In 4.17, we are enforcing this value for all OpenShift containers via tests, link in bug comment. This helps with debugging to ensure we have logs available.

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.

[nrb]

/lgtm

I'd like for @damdo to do final approval, since I don't understand the full impact of this change - what do we need to run in order to get all the manifests updated? A PR per provider to update these manifest ConfigMaps?

[damdo]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: damdo

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:

• OWNERS [damdo]

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

[damdo]

Let’s first merge the cluster-capi-operator change to the manifests-gen and then open a PR to vendor/exec generation for each provider.

[openshift-ci-robot]

/retest-required

Remaining retests: 0 against base HEAD ccded70 and 2 for PR HEAD d14e7d7 in total

[openshift-ci]

@JoelSpeed: The following tests 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/e2e-gcp-ovn-techpreview	d14e7d7	link	true	/test e2e-gcp-ovn-techpreview
ci/prow/e2e-azure-ovn-techpreview	d14e7d7	link	false	/test e2e-azure-ovn-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/test-infra repository. I understand the commands that are listed here.

[damdo]

This manifests-gen change does not affect the payload as it is a manually ran tool
/override ci/prow/e2e-gcp-ovn-techpreview

[openshift-ci]

@damdo: damdo unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight openshift-staff-engineers.

Details

In response to this:

This manifests-gen change does not affect the payload as it is a manually ran tool
/override ci/prow/e2e-gcp-ovn-techpreview

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.

[JoelSpeed]

/override ci/prow/e2e-gcp-ovn-techpreview

[openshift-ci]

@JoelSpeed: JoelSpeed unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight openshift-staff-engineers.

Details

In response to this:

/override ci/prow/e2e-gcp-ovn-techpreview

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.

[openshift-ci-robot]

@JoelSpeed: Jira Issue OCPBUGS-33170: All pull requests linked via external trackers have merged:

• openshift/cluster-capi-operator#172

Jira Issue OCPBUGS-33170 has been moved to the MODIFIED state.

Details

In response to this:

In 4.17, we are enforcing this value for all OpenShift containers via tests, link in bug comment. This helps with debugging to ensure we have logs available.

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-bot]

[ART PR BUILD NOTIFIER]

This PR has been included in build ose-cluster-capi-operator-container-v4.16.0-202405021515.p0.ga24f1ae.assembly.stream.el9 for distgit ose-cluster-capi-operator.
All builds following this will include this PR.