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

Fix validation of ESXi network configuration (backport) #698

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

ahadas
Copy link
Member

@ahadas ahadas commented Jan 4, 2024

Currently, all validations fail because we cannot access the hosts with secure connections. In #656 the code has changed to take the configuration of the client connection from the secret, however, the secret of the ESXi hosts is never set with the insecure flag.

This change practically makes the client to the hosts be always insecure. It's not that significant because the data we get from the host does not include data that could be confidential.

Currently, all validations fail because we cannot access the hosts with
secure connections. In kubev2v#656 the
code has changed to take the configuration of the client connection from
the secret, however, the secret of the ESXi hosts is never set with the
insecure flag.

This change practically makes the client to the hosts be always
insecure. It's not that significant because the data we get from the
host does not include data that could be confidential.

Signed-off-by: Arik Hadas <[email protected]>
@ahadas ahadas requested a review from liranr23 January 4, 2024 08:15
Copy link

sonarcloud bot commented Jan 4, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@liranr23 liranr23 changed the title Fix validation of ESXi network configuration Fix validation of ESXi network configuration (backport) Jan 4, 2024
@ahadas ahadas merged commit 6ad5a58 into kubev2v:release-v2.5.4 Jan 4, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants