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: Added TLS configuration to bypass certificate verification in testenv #411

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

outscale-hmi
Copy link
Contributor

@outscale-hmi outscale-hmi commented Jan 12, 2025

For test Env:
We are using envtest (from controller-runtime) to start a Kubernetes test environment, which provides a fake Kubernetes API server for local testing. the call testEnv.Start() is to start this environment and obtain a cfg (Kubernetes config).

This PR include the:

  • Set InsecureSkipVerify to true in the TLS configuration to allow the use of self-signed certificates during testing.
  • Update the minimum TLS version to tls.VersionTLS12 to ensure secure communication while bypassing certificate verification.

=>This change enables the test environment to work with self-signed certificates without triggering certificate validation errors, ensuring smooth execution in environments with non-CA-signed certificates.

@outscale-hmi outscale-hmi changed the title Fix: Added TLS configuration to bypass certificate verification in te… Fix: Added TLS configuration to bypass certificate verification in tstenv Jan 12, 2025
@outscale-hmi outscale-hmi changed the title Fix: Added TLS configuration to bypass certificate verification in tstenv Fix: Added TLS configuration to bypass certificate verification in testenv Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant