Skip to content

flatcar-postinst: Remove torcx sanity checks#30

Merged
t-lo merged 1 commit intoflatcar-masterfrom
t-lo/postinst-handle-torcx-removal
Oct 17, 2023
Merged

flatcar-postinst: Remove torcx sanity checks#30
t-lo merged 1 commit intoflatcar-masterfrom
t-lo/postinst-handle-torcx-removal

Conversation

@t-lo
Copy link
Copy Markdown
Member

@t-lo t-lo commented Oct 17, 2023

This change removes torcx sanity checks in the flatcar-postinst post-install script. This allows future major releases to of docker and containerd sysexts instead of torcx without the post-install script breaking.

An earlier version of this PR wrapped torcx sanity checks in a version check of the new OS version to be installed. If MAJOR > 3760 then torcx sanity checks are not run; this was removed during review.

If we need to issue updated update_engine releases to older major versions too, e.g. for bug fixes / security fixes, we'll create a branch with a back-port of the respective change.

This change removes torcx sanity checks in the `flatcar-postinst`
post-install script. This allows future major releases to of docker
and containerd sysexts instead of torcx without the post-install script
breaking.

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
@t-lo t-lo force-pushed the t-lo/postinst-handle-torcx-removal branch from e9a8477 to 0734e00 Compare October 17, 2023 16:05
@t-lo t-lo changed the title flatcar-postinst: don't check for torcx on major releases higher than 3760 flatcar-postinst: Remove torcx sanity checks Oct 17, 2023
@t-lo t-lo requested a review from krnowak October 17, 2023 16:07
@t-lo t-lo merged commit e4b5571 into flatcar-master Oct 17, 2023
@t-lo t-lo deleted the t-lo/postinst-handle-torcx-removal branch October 17, 2023 16:11
t-lo added a commit to flatcar/scripts that referenced this pull request Oct 17, 2023
Include PR flatcar/update_engine#30 to un-break
updates when torcx was removed in favour of sysext.

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
t-lo added a commit to flatcar/scripts that referenced this pull request Oct 19, 2023
Include PR flatcar/update_engine#30 to un-break
updates when torcx was removed in favour of sysext.

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
t-lo added a commit to flatcar/scripts that referenced this pull request Oct 19, 2023
Include PR flatcar/update_engine#30 to un-break
updates when torcx was removed in favour of sysext.

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
t-lo added a commit to flatcar/scripts that referenced this pull request Oct 20, 2023
Include PR flatcar/update_engine#30 to un-break
updates when torcx was removed in favour of sysext.

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
t-lo added a commit to flatcar/scripts that referenced this pull request Oct 20, 2023
Include PR flatcar/update_engine#30 to un-break
updates when torcx was removed in favour of sysext.

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
t-lo added a commit to flatcar/scripts that referenced this pull request Oct 23, 2023
Include PR flatcar/update_engine#30 to un-break
updates when torcx was removed in favour of sysext.

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
t-lo added a commit to flatcar/scripts that referenced this pull request Oct 26, 2023
Include PR flatcar/update_engine#30 to un-break
updates when torcx was removed in favour of sysext.

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
t-lo added a commit to flatcar/scripts that referenced this pull request Oct 27, 2023
Include PR flatcar/update_engine#30 to un-break
updates when torcx was removed in favour of sysext.

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
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