Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

Create separate module named matchbox-flatcar that provisions flatcar only. #1387

Merged

Conversation

ipochi
Copy link
Member

@ipochi ipochi commented Feb 18, 2021

This commit creates a new module named matchbox-flatcar which does the flatcar provisioning. Removes the code from baremetal and makes change s such that baremetal uses the new module.

Contains commits from #1374.

Please only review this commit 04bed78

@ipochi ipochi force-pushed the imran/split-baremetal-for-plain-flatcar-provisioning branch 8 times, most recently from ec67d43 to 04bed78 Compare February 23, 2021 13:56
@ipochi ipochi changed the title [WIP] - Do not review Split baremetal for plain flatcar provisioning Create separate module named matchbox-flatcar that provisions flatcar only. Feb 23, 2021
@ipochi ipochi requested a review from invidian February 23, 2021 14:02
invidian
invidian previously approved these changes Feb 23, 2021
Copy link
Member

@invidian invidian left a comment

Choose a reason for hiding this comment

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

Just some nits, otherwise looks good 👍 Really nice improvement.

assets/terraform-modules/matchbox-flatcar/profiles.tf Outdated Show resolved Hide resolved
assets/terraform-modules/matchbox-flatcar/variables.tf Outdated Show resolved Hide resolved
assets/terraform-modules/matchbox-flatcar/variables.tf Outdated Show resolved Hide resolved
assets/terraform-modules/matchbox-flatcar/variables.tf Outdated Show resolved Hide resolved
assets/terraform-modules/matchbox-flatcar/variables.tf Outdated Show resolved Hide resolved
assets/terraform-modules/matchbox-flatcar/variables.tf Outdated Show resolved Hide resolved
assets/terraform-modules/matchbox-flatcar/versions.tf Outdated Show resolved Hide resolved
@ipochi ipochi force-pushed the imran/split-baremetal-for-plain-flatcar-provisioning branch from 04bed78 to 361aae2 Compare February 23, 2021 15:23
@ipochi ipochi requested a review from invidian February 23, 2021 15:23
@ipochi ipochi force-pushed the imran/split-baremetal-for-plain-flatcar-provisioning branch from 361aae2 to 7375a58 Compare February 24, 2021 07:02
@ipochi ipochi requested a review from invidian February 24, 2021 07:02
invidian
invidian previously approved these changes Feb 24, 2021
assets/terraform-modules/matchbox-flatcar/variables.tf Outdated Show resolved Hide resolved
@ipochi ipochi force-pushed the imran/split-baremetal-for-plain-flatcar-provisioning branch 2 times, most recently from cce51f9 to 7899f02 Compare February 24, 2021 13:17
invidian
invidian previously approved these changes Feb 25, 2021
@ipochi ipochi force-pushed the imran/split-baremetal-for-plain-flatcar-provisioning branch from 7899f02 to fde1113 Compare March 9, 2021 08:50
This commit restructures the baremetal terraform module to split the
flatcar provisioning as a separate module `matchbox-flatcar`.

The new module takes care of setting up matchbox profiles and groups.

This makes it much easier to use this new module for other projects and
possibly the matchbox-flatcar module could maybe live in its own
repository in the future.

Signed-off-by: Imran Pochi <imran@kinvolk.io>
@ipochi ipochi force-pushed the imran/split-baremetal-for-plain-flatcar-provisioning branch from fde1113 to 15285e0 Compare March 9, 2021 09:10
Copy link
Member

@surajssd surajssd left a comment

Choose a reason for hiding this comment

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

Another LGTM 😆

@ipochi ipochi merged commit 0736a32 into master Mar 9, 2021
@ipochi ipochi deleted the imran/split-baremetal-for-plain-flatcar-provisioning branch March 9, 2021 10:39
ipochi added a commit that referenced this pull request Mar 11, 2021
…or-plain-flatcar-provisioning"

This reverts commit 0736a32, reversing
changes made to dc7324e.

Signed-off-by: Imran Pochi <imran@kinvolk.io>
ipochi added a commit that referenced this pull request Jun 1, 2021
…al-changes"

This reverts commit 106ea42, reversing
changes made to ac85cbb.

In effect this PR wants to re-revert the revert and put the following
baremetal changes back into master:
- #1374
- #1387
- #1398
- #1405
ipochi added a commit that referenced this pull request Jun 1, 2021
…al-changes"

This reverts commit 106ea42, reversing
changes made to ac85cbb.

In effect this PR wants to re-revert the revert and put the following
baremetal changes back into master:
- #1374
- #1387
- #1398
- #1405
ipochi added a commit that referenced this pull request Jun 2, 2021
…al-changes"

This reverts commit 106ea42, reversing
changes made to ac85cbb.

In effect this PR wants to re-revert the revert and put the following
baremetal changes back into master:
- #1374
- #1387
- #1398
- #1405
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants