-
Notifications
You must be signed in to change notification settings - Fork 502
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
[JUJU-1283] Delete old logic inside the state and upgrades packages #14168
[JUJU-1283] Delete old logic inside the state and upgrades packages #14168
Conversation
/build |
1 similar comment
/build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is more stuff that can go.
@juanmanuel-tirado can you please update the commit message for 13baf18? Outside of the pr it has no context. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Upgrade from 2.9.17 to 3.0-beta1 is broken before this change, no way to test right now.
This PR removes all the logic required to run upgrades for versions earlier than 2.9.x. From now on all the logic to be maintained should be to upgrade from 2.9.x to 3.0.x. Whatever additional steps should be included to ensure an upgrade from 2.9.x to 3.0.x they have to be added in future PRs
Checklist
QA steps
NA
Documentation changes
NA
Bug reference
NA