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

Allow upgrade from 11.x to 12.x #233

Merged
merged 2 commits into from
Jul 19, 2022
Merged

Conversation

spilchen
Copy link
Collaborator

Upgrades to 12.x were broken because we had to accept updated terms to the EULA. This adds a path to the offline and online upgrade that will run the installer to re-accept the EULA if it changed.

We were hitting disk full errors in the GitHub CI, so this also does various things to save space in the CI:

  • delete the RPM once the vertica image has been built
  • use 11.1.1 as the base image for upgrades. We used to use 11.1.0 but that image is quite a bit bigger because its CentOS base.
  • don't pre-pull the vertica-k8s:11.1.0-0-minimal image for all e2e runs. This is only needed for the server upgrade tests.

Matt Spilchen added 2 commits July 18, 2022 09:08
In GitHub actions, we are limited with the amount of space we have.  We
have seen a few failures of the tests because disks were full.  Here are
some things we have done here to help reduce the amount of space:
- delete the RPM once the vertica image has been built
- use 11.1.1 as the base image for upgrades.  We used to use 11.1.0 but
  that image is quite a bit bigger because its CentOS base.
- don't pre-pull the vertica-k8s:11.1.0-0-minimal image for all e2e
  runs.  This is only needed for the server upgrade tests.
Upgrades to 12.x were broken because we had to accept updated terms to
the EULA.  This adds a path to the offline and online upgrade that will
run the installer to re-accept the EULA if it changed.
@spilchen spilchen requested a review from roypaulin July 19, 2022 12:59
@spilchen spilchen self-assigned this Jul 19, 2022
Copy link
Collaborator

@roypaulin roypaulin left a comment

Choose a reason for hiding this comment

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

Looks good.

@spilchen spilchen merged commit 08a8e9d into vertica:main Jul 19, 2022
@spilchen spilchen deleted the e2e-save-space branch July 19, 2022 15:11
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