Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add mandatory annotations for OKD bundle #152

Merged

Conversation

clobrano
Copy link
Contributor

@clobrano clobrano commented Jul 23, 2024

  • Add OCP annotation to OKD bundle

OKD community pipeline is verifying the presence of OCP annotation.

Moreover, I fixed a typo in the release template.

Why we need this PR

Changes made

Add OCP annotation to OKD bundle

Which issue(s) this PR fixes

Test plan

Copy link
Contributor

openshift-ci bot commented Jul 23, 2024

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@clobrano clobrano mentioned this pull request Jul 23, 2024
Currently the release document created by CI uses github.ref_name in
place of the release version, but since we are not running this job on
tag creation anymore, this variable has the name of the branch instead
than the tag.

This change replaced {{github.ref_name}} with v{{inputs.version}}, which
contains the proper tag version.

Signed-off-by: Carlo Lobrano <[email protected]>
OKD community pipeline is verifying the presence of OCP annotation.

Signed-off-by: Carlo Lobrano <[email protected]>
@clobrano clobrano force-pushed the add-mandatory-annotation-for-okd-0 branch from 8538912 to aaa780e Compare July 25, 2024 11:47
@clobrano
Copy link
Contributor Author

/test 4.16-openshift-e2e

Copy link
Contributor

openshift-ci bot commented Jul 26, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: clobrano, slintes

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

The pull request process is described here

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

@clobrano clobrano marked this pull request as ready for review July 26, 2024 13:10
@openshift-ci openshift-ci bot requested review from razo7 and slintes July 26, 2024 13:10
@clobrano
Copy link
Contributor Author

@clobrano: 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/4.13-openshift-e2e aaa780e link true /test 4.13-openshift-e2e

Full PR test history. Your PR dashboard.

Deployment not completed (maybe network issue)

NodeCADaemonProgressing: The daemon set node-ca is deploying node pods  Operator progressing (DeploymentNotCompleted::NodeCADaemonUnavailable): Progressing: The deployment has not completed

@clobrano
Copy link
Contributor Author

/retest

@openshift-merge-bot openshift-merge-bot bot merged commit 0d727dc into medik8s:main Jul 26, 2024
27 checks passed
@clobrano clobrano deleted the add-mandatory-annotation-for-okd-0 branch July 29, 2024 12:34
@clobrano
Copy link
Contributor Author

I made a mistake here. I was somehow using go > 1.20 locally and didn't notice, but since FAR has version 1.20, YQ needs to be <= 4.42.1, not 4.44.2

I will fix this in a next PR

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

Successfully merging this pull request may close these issues.

2 participants