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

Revamp CI #239

Merged
merged 23 commits into from
Aug 7, 2022
Merged

Revamp CI #239

merged 23 commits into from
Aug 7, 2022

Conversation

spilchen
Copy link
Collaborator

@spilchen spilchen commented Aug 5, 2022

This is a major overhaul of the CI:

  • We now build all of the images first and store them in the GitHub container registry. The e2e tests in turn use the container registry, previously they each would create the containers themselves. The main benefit is that we preserve the containers that we tested with so that we can release the actual containers that were used in our CI.
  • Add security vulnerability scanning of the images. Scanning doesn't fail the built. But findings from the scans are now an artifact provided by the build and and can be found in the Security tab of the repo.
  • Allow the CI to be run with pre-built containers that are stored in a private repostiory in dockerhub. This will allow us to run the CI against unreleased versions of the Vertica server.
  • Add some standard labels to the containers

Matt Spilchen added 2 commits August 5, 2022 09:14
@spilchen spilchen self-assigned this Aug 5, 2022
@spilchen spilchen merged commit c4bdcb6 into vertica:main Aug 7, 2022
@spilchen spilchen deleted the ci-revamp branch August 7, 2022 22:36
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.

1 participant