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

fix(upgrade-job): upgrade path validation and set flags #242

Merged
merged 4 commits into from
May 22, 2023

Conversation

niladrih
Copy link
Member

@niladrih niladrih commented May 22, 2023

  • use --set flags to set thin provisioning flags for 2.0.x releases, instead of 2.1.x release.
  • allow 'no-rollback' rule to be overridden by the --skip-upgrade-path-validation flag. This is not necessary for the openebs/openebs (Umbrella) chart because this upgrade-job does not support upgrade for the openebs chart, so might as well move that check to be a Core chart exclusive check.
  • allow the --set flags for 2.0.x release to be used with future 2.0.x releases as well, and not just till 2.0.1.

@sinhaashish
Copy link
Member

why is this required --app-tag?

@niladrih niladrih force-pushed the fix-upgrade-from-2.1 branch from e5beaec to 78e75f1 Compare May 22, 2023 09:27
@niladrih
Copy link
Member Author

why is this required --app-tag?

Oh, that's not supposted to be there..

@niladrih
Copy link
Member Author

bors try

bors bot pushed a commit that referenced this pull request May 22, 2023
@bors
Copy link
Contributor

bors bot commented May 22, 2023

try

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@niladrih niladrih changed the title fix(upgrade-job): upgrade path validation fix(upgrade-job): upgrade path validation and set flags May 22, 2023
@niladrih
Copy link
Member Author

bors merge

@bors
Copy link
Contributor

bors bot commented May 22, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 03ba9c7 into develop May 22, 2023
@bors bors bot deleted the fix-upgrade-from-2.1 branch May 22, 2023 13:55
bors bot pushed a commit that referenced this pull request May 22, 2023
244: Cherry-pick the commits from PR #242 to release/2.2 branch r=niladrih a=niladrih

This PR includes the commits from the following PR(s):
- #242 

Co-authored-by: Niladri Halder <[email protected]>
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