Skip to content

Conversation

@davidweisse
Copy link
Contributor

This allows setting the --insecure-enable-debug-shell-access option for e2e tests, which gets passed through to contrast generate. This respects the debug variable when using just e2e or the associated option when running a manual e2e test. Additionally, the test matrices for the nightly and PR tests are extended to include all tests with and without the debug option.

@davidweisse davidweisse added the no changelog PRs not listed in the release notes label Nov 17, 2025
@katexochen katexochen self-assigned this Nov 17, 2025
@davidweisse davidweisse force-pushed the dav/e2e-debug-shell branch 2 times, most recently from 8cd2f58 to 9c9169b Compare November 18, 2025 12:14
@katexochen
Copy link
Member

@burgerdev for ac20943

@davidweisse
Copy link
Contributor Author

With the debugshell container, we have more than one main container, so the kubeclient in contrasttest does not know in which container to exec. The easiest solution I see is to just default to the first main container, which in our case always should be the workload container.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog PRs not listed in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants