Skip to content

HIVE-3074: VSphere preflight check uses hive trusted CA bundle#2853

Open
2uasimojo wants to merge 1 commit intoopenshift:masterfrom
2uasimojo:HIVE-3074/vsphere-cabundle-preflight
Open

HIVE-3074: VSphere preflight check uses hive trusted CA bundle#2853
2uasimojo wants to merge 1 commit intoopenshift:masterfrom
2uasimojo:HIVE-3074/vsphere-cabundle-preflight

Conversation

@2uasimojo
Copy link
Member

For VSphere, when we set up to run the installer, we install the generated hive-trusted-cabundle as well as the configured VCenter certs. However, we weren't doing the former for the preflight authn check. Fix.

(This only seems to matter under particular network configurations around the VCenter, which are apparently quite rare given we only recently encountered problems in this space.)

For VSphere, when we set up to run the installer, we install the
generated hive-trusted-cabundle as well as the configured VCenter certs.
However, we weren't doing the former for the preflight authn check. Fix.

(This only seems to matter under particular network configurations
around the VCenter, which are apparently quite rare given we only
recently encountered problems in this space.)
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Feb 13, 2026
@openshift-ci-robot
Copy link

openshift-ci-robot commented Feb 13, 2026

@2uasimojo: This pull request references HIVE-3074 which is a valid jira issue.

Details

In response to this:

For VSphere, when we set up to run the installer, we install the generated hive-trusted-cabundle as well as the configured VCenter certs. However, we weren't doing the former for the preflight authn check. Fix.

(This only seems to matter under particular network configurations around the VCenter, which are apparently quite rare given we only recently encountered problems in this space.)

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 dlom and suhanime February 13, 2026 20:01
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 13, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 2uasimojo

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 Feb 13, 2026
@2uasimojo
Copy link
Member Author

/test e2e-vsphere

@2uasimojo
Copy link
Member Author

/test e2e-vsphere

May want to consider putting this file in a path with vsphere in it so changes kick off this test automatically.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 13, 2026

@2uasimojo: all tests passed!

Full PR test history. Your PR dashboard.

Details

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. I understand the commands that are listed here.

@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

❌ Patch coverage is 0% with 32 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.32%. Comparing base (7d52d80) to head (d044fc9).

Files with missing lines Patch % Lines
pkg/controller/utils/credentials.go 0.00% 32 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2853      +/-   ##
==========================================
- Coverage   50.33%   50.32%   -0.02%     
==========================================
  Files         280      280              
  Lines       34288    34296       +8     
==========================================
  Hits        17258    17258              
- Misses      15669    15677       +8     
  Partials     1361     1361              
Files with missing lines Coverage Δ
pkg/controller/utils/credentials.go 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants