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

platform/conf: add Butane config support #318

Merged
merged 8 commits into from
Apr 13, 2022
Merged

Conversation

tormath1
Copy link
Contributor

@tormath1 tormath1 commented Apr 11, 2022

In this PR, we provide conf.Butane to define userdata:

		UserData: conf.Butane(`---
variant: flatcar
version: 1.0.0
kernel_arguments:
  should_exist:
    - quiet`),

it should be used whenever we want to generate Ignition configuration, in place of CLC - except if we want to test specific CLC of course. It should support other version of Butane Flatcar's variant as long as we keep the vendor in sync with the upstream dependency since it's butane who takes care of finding the right translator for a given configuration.

1.1.0-experimental does not work currently because it generates Ignition-3.4.0 which is not supported by Ignition neither Flatcar.

Note for reviewers

  • The following commit: platform/conf: use coreos/ignition for ignition/v3 is required because CLC generates an Ignition types defined in flatcar-linux/ignition, so we need to keep flatcar-linux/ignition.

@tormath1 tormath1 self-assigned this Apr 11, 2022
@tormath1 tormath1 marked this pull request as ready for review April 11, 2022 15:09
@tormath1 tormath1 requested a review from a team April 11, 2022 15:09
@tormath1 tormath1 mentioned this pull request Apr 11, 2022
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
butane config will be translated to Ignition >= 3.3.0 before following
the regular Ignition process.

Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
it makes the configuration lighter - for example with luks test, it's
not required to define the "mount" system unit because it's generated by
Butane.

Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
@tormath1
Copy link
Contributor Author

CI passed.

@tormath1 tormath1 merged commit 83cdb53 into flatcar-master Apr 13, 2022
@tormath1 tormath1 deleted the tormath1/butane branch April 13, 2022 11:01
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.

2 participants