Skip to content
This repository was archived by the owner on Dec 1, 2022. It is now read-only.

[CP][0.23]Add root ca for Controller HA test with https (#11471)#836

Merged
openshift-merge-robot merged 6 commits intoopenshift:release-v0.23.1from
nak3:cp-controller-ha
Jul 1, 2021
Merged

[CP][0.23]Add root ca for Controller HA test with https (#11471)#836
openshift-merge-robot merged 6 commits intoopenshift:release-v0.23.1from
nak3:cp-controller-ha

Conversation

@nak3
Copy link

@nak3 nak3 commented Jul 1, 2021

This patch adds test.AddRootCAtoTransport for prober in TestControllerHA.
It cherr-picked from upstream's knative@7253895 and changes test code.

It was backported for 0.22 #815 but slipped from backport for 0.23.

This patch adds `test.AddRootCAtoTransport` for prober in TestControllerHA.
@openshift-ci openshift-ci bot requested review from alanfx and mvinkler July 1, 2021 02:49
@openshift-ci openshift-ci bot added the area/test-and-release Issues or PRs related to test and release label Jul 1, 2021
@nak3
Copy link
Author

nak3 commented Jul 1, 2021

/hold

Test passed but pin to release-1.16 is wrong. I think I should update release label.

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 1, 2021
@nak3
Copy link
Author

nak3 commented Jul 1, 2021

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 1, 2021
Copy link

@markusthoemmes markusthoemmes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 1, 2021
@openshift-ci
Copy link

openshift-ci bot commented Jul 1, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: markusthoemmes, nak3

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 Jul 1, 2021
@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit 75ffe62 into openshift:release-v0.23.1 Jul 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/test-and-release Issues or PRs related to test and release lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants