Skip to content

Conversation

@Jakob-Naucke
Copy link
Contributor

Verify the LUKS key is the one we'd expect as per the secret.

Use forced SSA to apply CRDs and revive parallel integration tests.

Picks one commit also included in #112.

Was `test-coreos-vm` for all.

Signed-off-by: Jakob Naucke <jnaucke@redhat.com>
@Jakob-Naucke Jakob-Naucke requested a review from alicefr January 5, 2026 14:57
@Jakob-Naucke Jakob-Naucke force-pushed the extend-integration-tests branch 2 times, most recently from 9b7f1c3 to 02c276c Compare January 5, 2026 15:13
vendored, direct definitions appear to be unavailable from the
internet

Signed-off-by: Jakob Naucke <jnaucke@redhat.com>
Signed-off-by: Jakob Naucke <jnaucke@redhat.com>
In integration tests, check not only that the disk is encrypted, but
that the encryption key matches.

Signed-off-by: Jakob Naucke <jnaucke@redhat.com>
This reverts commit a4c708a.
Use forced server-side apply to apply CRDs concurrently.

Signed-off-by: Jakob Naucke <jnaucke@redhat.com>
@Jakob-Naucke Jakob-Naucke force-pushed the extend-integration-tests branch from 02c276c to aa656e8 Compare January 5, 2026 16:23
@alicefr
Copy link
Contributor

alicefr commented Jan 6, 2026

Very nice, many thanks for the PR. Nothing to add if the tests are passing locally!

@openshift-ci
Copy link

openshift-ci bot commented Jan 6, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: alicefr, Jakob-Naucke

The full list of commands accepted by this bot can be found 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

@Jakob-Naucke Jakob-Naucke merged commit 9f5ab59 into trusted-execution-clusters:main Jan 6, 2026
8 checks passed
@Jakob-Naucke Jakob-Naucke deleted the extend-integration-tests branch January 7, 2026 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants