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

Stabilize spec 3.2.0 #1103

Merged
merged 5 commits into from Oct 14, 2020
Merged

Stabilize spec 3.2.0 #1103

merged 5 commits into from Oct 14, 2020

Conversation

arithx
Copy link
Contributor

@arithx arithx commented Sep 29, 2020

No description provided.

Copy link
Member

@sohankunkerkar sohankunkerkar left a comment

Choose a reason for hiding this comment

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

The external kola test (ext.config.root-reprovision.luks) is failing, but that's understood as we move from a 3.2.0-experimental to 3.2.0 spec which would break this change.

--- FAIL: ext.config.root-reprovision.luks (11.86s)

        harness.go:926: Cluster failed starting machines: machine cb8422ec-75ac-477a-ba4b-457f4e1c34ae entered emergency.target in initramfs

But overall, this change looks good to me. I used this doc for verification.

Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

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

Didn't dig into the details, but it looks reasonable to me!

Re. the breaking root reprovisioning tests, yeah that's unfortunate. Probably easiest to just merge on red here but sanity-check that the only tests that failed are the root reprovisioning ones, and locally verify that a cosa with the vendoring bumped to the stable spec and the tests updated to reference it does pass (or just manually boot up VMs directly with those Ignition configs really).

Once we do a new Ignition release, we'll have to merge the cosa version bump PR and the FCOS spec tweak and fast-track PR roughly at the same time.

Copy link
Contributor

@bgilbert bgilbert left a comment

Choose a reason for hiding this comment

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

LGTM aside from migration notes.

docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
@arithx arithx force-pushed the 3_2_0_stabilize branch 2 times, most recently from db8aee5 to 6a5745f Compare October 12, 2020 22:42
@arithx
Copy link
Contributor Author

arithx commented Oct 12, 2020

Updated ⬆️

docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
docs/migrating-configs.md Outdated Show resolved Hide resolved
@arithx arithx force-pushed the 3_2_0_stabilize branch 2 times, most recently from 16662ea to 75df5f1 Compare October 12, 2020 23:48
@arithx
Copy link
Contributor Author

arithx commented Oct 12, 2020

Updated ⬆️

@arithx
Copy link
Contributor Author

arithx commented Oct 14, 2020

@arithx arithx merged commit 1a87bee into coreos:master Oct 14, 2020
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.

None yet

4 participants