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

don't check the passed version if we are in 1st stage #2408

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

ashraffouda
Copy link
Collaborator

Description

don't consider version in first stage of bootstrap

Changes

skip passed version if bootstrap is running in 1st stage

Related Issues

List of related issues

Checklist

  • Tests included
  • Build pass
  • Documentation
  • Code format and docstring

@iwanbk
Copy link
Member

iwanbk commented Aug 26, 2024

@ashraffouda maybe add the context/reason of this change?

@ashraffouda
Copy link
Collaborator Author

@ashraffouda maybe add the context/reason of this change?

bootstrap running in two stages, during 1st stage bootstrap only upgrades itself so if we added a new version like v4 in bootstrap and the running image has old version of bootstrap it will fail because bootstrap doesn't know the new version at the 1st stage, so it first needs to run and update itself to get the new changes which contains the newly added version

@@ -119,12 +119,15 @@ impl Config {
if stage == 0 {
bail!("invalid stage value 0, stages starting from 1");
}

let mut ver = Version::V3;
Copy link
Member

Choose a reason for hiding this comment

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

So, looks like it will become the version latest-1?

what happens when we upgrade zos with latest - 2 or latest version?
In this context, it means v2 or v4

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

at 1st step we set it to v3 this is the current supported version so it will not fail at 1st step

@ashraffouda ashraffouda merged commit cf82db1 into main Aug 26, 2024
25 of 26 checks passed
@ashraffouda ashraffouda deleted the main_bootstrap_versionfix branch August 26, 2024 09:55
maxux pushed a commit to threefoldtech/zoslight that referenced this pull request Sep 4, 2024
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