-
Notifications
You must be signed in to change notification settings - Fork 686
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
Update docker images #4122
Update docker images #4122
Conversation
dde5522
to
cbe185a
Compare
Codecov Report
@@ Coverage Diff @@
## master #4122 +/- ##
==========================================
- Coverage 90.76% 88.20% -2.56%
==========================================
Files 187 186 -1
Lines 36201 35888 -313
==========================================
- Hits 32858 31656 -1202
- Misses 3343 4232 +889 |
8d26d63
to
2e2a0ff
Compare
3f9d4df
to
618f105
Compare
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.
Looks ok, left some comments, questions.
Once the images are pushed to the citusdata docker hub - and reflected here - I will approve the merge
- persist_to_workspace: | ||
root: . | ||
paths: | ||
- build-11/* |
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.
if we checkout at every stage, does this still need to be in the workspace?
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.
Yeah because we use this folder for running the tests:
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.
Is there a technical reason we can't run from the checkout?
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.
technically I think we should be able to do that. As far as I remember Makefile in build-X is already a symbolic link to src/.../Makefile so I am guessing that it should work but that is something to try. I would suggest trying that when updating pg-13 image from 13beta3 to 13.0, what do you think? I can create an issue for trying that on the-process repo.
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 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.
Thanks for the work here.
Love the view on circleci now.
🚢
The build image was a single one and it would contain pg11, pg12 and pg13. Now it is separated so that we can build each pg major independently. Tags are used as full postgres versions so that we can know which version we use by looking at the tag. For example exttester:11.9 would mean we are using pg11.9. pg11 is updated from 11.5 to 11.9. pg12 is updated from 12rc to 12.4.
743fcff
to
023dfdf
Compare
The build image was a single one and it would contain pg11, pg12 and
pg13. Now it is separated so that we can build each pg major
independently.
Tags are used as full postgres versions so that we can know which
version we use by looking at the tag. For example exttester:11.9 would
mean we are using pg11.9.
pg11 is updated from 11.5 to 11.9.
pg12 is updated from 12rc to 12.4.
pg13 is updated to 13beta3.
Ones the general structure is "approved", the repos will be updated as
citus
fromheisenberg302
.