Skip to content

METAL-575: Revert "Mutate service monitor manifests to include tlsConfig cert an… #478

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

Merged

Conversation

derekhiggins
Copy link
Contributor

…d key file"

This reverts commit 4af508d.

Testing if this cause tls errors that started in metal-ipi jobs yesterday

@openshift-ci openshift-ci bot requested review from joelanford and oceanc80 April 5, 2023 09:04
@derekhiggins
Copy link
Contributor Author

/payload 4.14 nightly blocking

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 5, 2023

@derekhiggins: trigger 7 job(s) of type blocking for the nightly release of OCP 4.14

  • periodic-ci-openshift-release-master-nightly-4.14-e2e-aws-sdn-upgrade
  • periodic-ci-openshift-release-master-ci-4.14-e2e-azure-ovn-upgrade
  • periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-gcp-ovn-rt-upgrade
  • periodic-ci-openshift-release-master-ci-4.14-e2e-aws-ovn-upgrade
  • periodic-ci-openshift-release-master-nightly-4.14-e2e-aws-sdn-serial
  • periodic-ci-openshift-release-master-nightly-4.14-e2e-metal-ipi-ovn-ipv6
  • periodic-ci-openshift-release-master-nightly-4.14-e2e-metal-ipi-sdn

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/9600ee10-d393-11ed-8bf3-377c49f4f3e6-0

@derekhiggins
Copy link
Contributor Author

@derekhiggins derekhiggins changed the title TEST: Revert "Mutate service monitor manifests to include tlsConfig cert an… Revert "Mutate service monitor manifests to include tlsConfig cert an… Apr 5, 2023
@elfosardo
Copy link

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 5, 2023
@derekhiggins derekhiggins changed the title Revert "Mutate service monitor manifests to include tlsConfig cert an… METAL-575: Revert "Mutate service monitor manifests to include tlsConfig cert an… Apr 5, 2023
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Apr 5, 2023
@openshift-ci-robot
Copy link

openshift-ci-robot commented Apr 5, 2023

@derekhiggins: This pull request references METAL-575 which is a valid jira issue.

In response to this:

…d key file"

This reverts commit 4af508d.

Testing if this cause tls errors that started in metal-ipi jobs yesterday

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.

@derekhiggins
Copy link
Contributor Author

/test e2e-gcp-olm

@deepsm007
Copy link

/override ci/prow/e2e-gcp-olm
/skip

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 5, 2023

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

In response to this:

/override ci/prow/e2e-gcp-olm
/skip

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.

@neisw
Copy link

neisw commented Apr 5, 2023

/override ci/prow/e2e-gcp-olm
/skip

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 5, 2023

@neisw: Overrode contexts on behalf of neisw: ci/prow/e2e-gcp-olm

In response to this:

/override ci/prow/e2e-gcp-olm
/skip

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
Copy link
Contributor

openshift-ci bot commented Apr 5, 2023

@derekhiggins: all tests passed!

Full PR test history. Your PR dashboard.

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.

@neisw
Copy link

neisw commented Apr 5, 2023

/approve

@neisw
Copy link

neisw commented Apr 5, 2023

/approved

@neisw
Copy link

neisw commented Apr 5, 2023

/label approved

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 5, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 5, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: derekhiggins, neisw
Once this PR has been reviewed and has the lgtm label, please assign perdasilva for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@stbenjam
Copy link
Member

stbenjam commented Apr 5, 2023

/label bugzilla/valid-bug

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 5, 2023

@stbenjam: Can not set label bugzilla/valid-bug: Must be member in one of these teams: [openshift-patch-managers openshift-staff-engineers]

In response to this:

/label bugzilla/valid-bug

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.

@stbenjam
Copy link
Member

stbenjam commented Apr 5, 2023

/jira refresh

@openshift-ci-robot
Copy link

openshift-ci-robot commented Apr 5, 2023

@stbenjam: This pull request references METAL-575 which is a valid jira issue.

In response to this:

/jira refresh

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.

@stbenjam
Copy link
Member

stbenjam commented Apr 5, 2023

How about these?

/label px-approved
/label qe-approved
/label docs-approved

@openshift-ci openshift-ci bot added px-approved Signifies that Product Support has signed off on this PR qe-approved Signifies that QE has signed off on this PR docs-approved Signifies that Docs has signed off on this PR labels Apr 5, 2023
@openshift-merge-robot openshift-merge-robot merged commit 18655e5 into openshift:master Apr 5, 2023
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. docs-approved Signifies that Docs has signed off on this PR jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. px-approved Signifies that Product Support has signed off on this PR qe-approved Signifies that QE has signed off on this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants