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

[major] Add license terms in mas upgrade #1393

Merged
merged 129 commits into from
Dec 12, 2024
Merged

[major] Add license terms in mas upgrade #1393

merged 129 commits into from
Dec 12, 2024

Conversation

rawa-resul
Copy link
Contributor

@rawa-resul rawa-resul commented Dec 6, 2024

Description

  • Add license terms prompt in mas upgrade
    • Shows the license terms of the the next channel to upgrade to
  • --accept-license is now needed when --no-confirm is used
  • Add upgrade_paths from ansible-devops collection
    • Fails if there is no upgrade path for selected mas instance
  • Add step to install local version of python-devops in .github/workflows/build-cli.yml when tar.gz is available
  • Dynamically generate catalogTable from chosen catalog

Related Issues

Testing

  • Ran mas upgrade against my fyre cluster
Screenshot 2024-12-12 at 5 28 17 pm
  • Tried to upgrade 9.1.x-feature but there is no upgrade path and hence it errors out
Screenshot 2024-12-12 at 7 19 40 pm

@rawa-resul rawa-resul changed the title [patch] Add license terms in mas upgrade [major] Add license terms in mas upgrade Dec 12, 2024
@durera durera merged commit 38e78f2 into master Dec 12, 2024
12 checks passed
@durera durera deleted the MASR-2387 branch December 12, 2024 22:55
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.

7 participants