OCPCLOUD-3347: tls: use centralized TLS profile (unrevert)#292
OCPCLOUD-3347: tls: use centralized TLS profile (unrevert)#292damdo wants to merge 3 commits intoopenshift:mainfrom
Conversation
|
@damdo: This pull request references OCPCLOUD-3347 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set. DetailsIn response to this:
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. |
|
/hold For HCP to sort out the RBAC permission issue |
|
/cc @neisw @bryan-cox |
|
[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 DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
2458331 to
357265a
Compare
|
/hold until the hypershift presubmit jobs are in place openshift/release#75309 |
|
/retest |
|
/testwith openshift/cluster-machine-approver/main/e2e-hypershift-aks openshift/hypershift#7802 |
|
/testwith openshift/cluster-machine-approver/main/e2e-hypershift-aws openshift/hypershift#7802 |
|
@damdo: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. I understand the commands that are listed here. |
|
Seems it is still failing |
In HyperShift, shouldn't we be querying the HostedCluster object instead of the APIServer one? At least the enhancement seems to suggest that for SLOs. Edit: Saying that, using APIServer object from the hosted cluster in HyperShift works just fine for my use case and the implementation is simpler than trying to find the HostedCluster object as suggested by the the enhancement. |
|
PR needs rebase. DetailsInstructions 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. |
Unrevert of the revert #291, to reintroduce #286
--
Start using centralized TLS profile fetched from the APIServer configuration.
Remove kube-rbac-proxy