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

NO-ISSUE: adding openssh package to recert image in order to generate ssh keys in IBI process #150

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

tsorya
Copy link
Member

@tsorya tsorya commented Jun 6, 2024

NO-ISSUE: adding openssh package to recert image in order to generate ssh keys in IBI process

@openshift-ci-robot
Copy link
Collaborator

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

In response to this:

NO-ISSUE: adding openssh package to recert image in order to generate ssh keys in IBI process

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-ci openshift-ci bot requested review from mresvanis and omertuc June 6, 2024 14:51
@tsorya
Copy link
Member Author

tsorya commented Jun 6, 2024

/test 4.14-images

Copy link

openshift-ci bot commented Jun 6, 2024

@rauhersu: GitHub didn't allow me to request PR reviews from the following users: fontivan.

Note that only rh-ecosystem-edge members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

/cc @fontivan

Steven, can you include this on our recert downstream images ?

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.

@javipolo
Copy link
Collaborator

javipolo commented Jun 6, 2024

/lgtm

@tsorya
Copy link
Member Author

tsorya commented Jun 6, 2024

/override e2e-aws-ovn-single-node-recert-serial
/override e2e-aws-ovn-single-node-recert-parallel

broken tests

Copy link

openshift-ci bot commented Jun 6, 2024

@tsorya: tsorya unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:.

In response to this:

/override e2e-aws-ovn-single-node-recert-serial
/override e2e-aws-ovn-single-node-recert-parallel

broken tests

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.

@omertuc
Copy link
Member

omertuc commented Jun 7, 2024

/override e2e-aws-ovn-single-node-recert-serial
/override e2e-aws-ovn-single-node-recert-parallel

@omertuc
Copy link
Member

omertuc commented Jun 7, 2024

/approve

Copy link

openshift-ci bot commented Jun 7, 2024

@omertuc: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • e2e-aws-ovn-single-node-recert-parallel
  • e2e-aws-ovn-single-node-recert-serial

Only the following failed contexts/checkruns were expected:

  • ci/prow/4.14-images
  • ci/prow/4.15-images
  • ci/prow/4.16-images
  • ci/prow/baremetalds-sno-recert-cluster-rename
  • ci/prow/cargo-check
  • ci/prow/cargo-clippy
  • ci/prow/cargo-test
  • ci/prow/e2e-aws-ovn-single-node-recert-parallel
  • ci/prow/e2e-aws-ovn-single-node-recert-serial
  • ci/prow/images
  • pull-ci-rh-ecosystem-edge-recert-main-4.14-images
  • pull-ci-rh-ecosystem-edge-recert-main-4.15-images
  • pull-ci-rh-ecosystem-edge-recert-main-4.16-images
  • pull-ci-rh-ecosystem-edge-recert-main-baremetalds-sno-recert-cluster-rename
  • pull-ci-rh-ecosystem-edge-recert-main-cargo-check
  • pull-ci-rh-ecosystem-edge-recert-main-cargo-clippy
  • pull-ci-rh-ecosystem-edge-recert-main-cargo-test
  • pull-ci-rh-ecosystem-edge-recert-main-e2e-aws-ovn-single-node-recert-parallel
  • pull-ci-rh-ecosystem-edge-recert-main-e2e-aws-ovn-single-node-recert-serial
  • pull-ci-rh-ecosystem-edge-recert-main-images
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override e2e-aws-ovn-single-node-recert-serial
/override e2e-aws-ovn-single-node-recert-parallel

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.

Copy link

openshift-ci bot commented Jun 7, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: omertuc, tsorya

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

@openshift-ci openshift-ci bot added the approved label Jun 7, 2024
@omertuc
Copy link
Member

omertuc commented Jun 7, 2024

/override pull-ci-rh-ecosystem-edge-recert-main-e2e-aws-ovn-single-node-recert-parallel
/override pull-ci-rh-ecosystem-edge-recert-main-e2e-aws-ovn-single-node-recert-serial

Copy link

openshift-ci bot commented Jun 7, 2024

@omertuc: Overrode contexts on behalf of omertuc: ci/prow/e2e-aws-ovn-single-node-recert-parallel, ci/prow/e2e-aws-ovn-single-node-recert-serial

In response to this:

/override pull-ci-rh-ecosystem-edge-recert-main-e2e-aws-ovn-single-node-recert-parallel
/override pull-ci-rh-ecosystem-edge-recert-main-e2e-aws-ovn-single-node-recert-serial

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-merge-bot openshift-merge-bot bot merged commit 8c3fc87 into rh-ecosystem-edge:main Jun 7, 2024
12 checks passed
@omertuc
Copy link
Member

omertuc commented Jun 7, 2024

Created https://issues.redhat.com/browse/MGMT-18091 to consider whether about this is a good idea or not

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.

4 participants