-
Notifications
You must be signed in to change notification settings - Fork 36
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
chore(ci): build images from 'main' #2653
Conversation
We are no longer getting updates to the `main` tag
@@ -588,7 +576,6 @@ jobs: | |||
just slt-bin 'sqllogictests_snowflake/*' | |||
just slt-bin --protocol=rpc 'sqllogictests_snowflake/*' | |||
|
|||
|
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.
All changes before this point were just automagic lint/fmt 🤷🏻
docker-push: | ||
name: Build and Push Docker Image | ||
if: github.event_name == 'push' && github.repository == 'GlareDB/glaredb' && github.ref == 'refs/heads/main' | ||
runs-on: ubuntu-latest-8-cores |
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.
depending on if we decide to reuse the previous artifacts or not we can downside this.
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.
I'd be happy to accomplish both. The code as-is was just copy-pasta'd from the release variant, but since this runs on every push to main
and is generally OK to take longer, I'm happy to downsize.
Co-authored-by: Grey <grey@glaredb.com>
We are no longer getting updates to the
main
tag