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

kola: remove cmdline parameters for Tang and TPM tests #511

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

simoncampion
Copy link

Remove kernel cmdline parameters for Tang and TPM tests

Removes rd.luks... cmdline parameters that are unnecessary once this PR in flatcar/bootengine is merged. Removing the cmdline parameters should also fix an issue with tests failing because of failing superfluous systemd-cryptsetup units during the first boot.

How to use

Run the cl.tpm.* and cl.tang.* tests.

Testing done

See the description of testing done in this PR.

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

kola/tests/misc/tang.go Outdated Show resolved Hide resolved
kola/tests/misc/tpm.go Outdated Show resolved Hide resolved
@pothos
Copy link
Member

pothos commented Mar 25, 2024

Thanks! When we backport your bootengine changes this will even work for the old Alpha, but maybe we should still bump the minimum image version for the test to be 3913.0.1 (should work for all cases, a bugfix Alpha, a new Alpha, or Beta promotion).

@simoncampion
Copy link
Author

maybe we should still bump the minimum image version for the test to be 3913.0.1

Sounds good, I bumped the min version to 3913.0.1.

@pothos pothos merged commit 77adbbe into flatcar:flatcar-master Mar 27, 2024
1 of 2 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
2 participants