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

add some ignition/v3 test #301

Merged
merged 9 commits into from
Mar 23, 2022
Merged

add some ignition/v3 test #301

merged 9 commits into from
Mar 23, 2022

Conversation

tormath1
Copy link
Contributor

@tormath1 tormath1 commented Mar 3, 2022

In this PR, we add some Ignition V3 tests from coreos/coreos-assembler project in order to be closer from the upstream. It also brings new test for IgnitionV3 on top of UserDataV3 ones.

Tested with: http://localhost:9091/job/os/job/manifest/5049/cldsv/

NOTE: kargs test could be upstreamed

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)

@tormath1 tormath1 force-pushed the tormath1/ignition3 branch from e76bad9 to a376805 Compare March 4, 2022 09:14
@tormath1 tormath1 force-pushed the tormath1/ignition3 branch from ffa3306 to d1933f2 Compare March 7, 2022 10:01
@tormath1 tormath1 self-assigned this Mar 7, 2022
@tormath1 tormath1 marked this pull request as ready for review March 7, 2022 10:02
@tormath1 tormath1 requested a review from a team March 7, 2022 10:07
go.mod Outdated Show resolved Hide resolved
this test is backport from coreos-assembler to test systemd instantiated
services enabled via ignition.

Signed-off-by: Mathieu Tortuyaux <[email protected]>
this helper runs cmd via SSH and panics if stdout does not contain expected

Signed-off-by: Mathieu Tortuyaux <[email protected]>
this test creates a symlink via ignition and assert it's correct.

Signed-off-by: Mathieu Tortuyaux <[email protected]>
mainly to support config > 3.0.0

Signed-off-by: Mathieu Tortuyaux <[email protected]>
Signed-off-by: Mathieu Tortuyaux <[email protected]>
Signed-off-by: Mathieu Tortuyaux <[email protected]>
@tormath1 tormath1 force-pushed the tormath1/ignition3 branch from 8df4c83 to 6d831e3 Compare March 21, 2022 15:23
tormath1 and others added 2 commits March 21, 2022 16:40
Signed-off-by: Mathieu Tortuyaux <[email protected]>
Co-authored-by: Kai Lüke <[email protected]>
Signed-off-by: Mathieu Tortuyaux <[email protected]>
@tormath1 tormath1 force-pushed the tormath1/ignition3 branch from 1aa5de2 to 69b6d94 Compare March 21, 2022 15:44
@tormath1 tormath1 merged commit f85fbf8 into flatcar-master Mar 23, 2022
@tormath1 tormath1 deleted the tormath1/ignition3 branch March 23, 2022 17:52
}
}`),
Distros: []string{"cl"},
})
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tormath1 this need a MinVersion doesn't it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yes, version 3.0.0... Totally missed this one.

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.

3 participants