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

Publish preview Docker image when release candidate is tagged #6787

Merged
merged 2 commits into from
Mar 1, 2024

Conversation

eradman
Copy link
Collaborator

@eradman eradman commented Feb 28, 2024

What type of PR is this?

  • Refactor
  • Feature
  • Bug Fix

Description

Every time a pull request is submitted a new docker image is uploaded. This should be fixed by only responding to changes that were merged. This proposal goes further and only publishes commits that were tagged.

  • Only respond to new tags ending with -dev
  • Use github account name to allow easier testing in a fork
  • Allow preview image to be referenced by a specific tag, or by latest tag

redash/preview:24.02.0-dev
redash/redash:preview

How is this tested?

  • Manually
git tag 24.03.0-dev
git push origin --tags 24.03.0-dev
# wait for build
docker pull eradman/preview:24.02.0-dev

* Only respond to new tags ending with -dev
* Use github account name to allow easier testing in a fork
* Allow preview image to be referenced by a specific tag, or by latest tag

redash/preview:24.02.0-dev
redash/redash:preview
Copy link

codecov bot commented Feb 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.41%. Comparing base (1200f98) to head (eb877f8).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6787   +/-   ##
=======================================
  Coverage   63.41%   63.41%           
=======================================
  Files         162      162           
  Lines       13186    13186           
  Branches     1822     1822           
=======================================
  Hits         8362     8362           
  Misses       4527     4527           
  Partials      297      297           

@justinclift justinclift enabled auto-merge (squash) March 1, 2024 15:08
justinclift
justinclift previously approved these changes Mar 1, 2024
Copy link
Member

@justinclift justinclift left a comment

Choose a reason for hiding this comment

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

Cool, lets try this out and see how it goes. 😄

@justinclift justinclift merged commit 7cd03c7 into getredash:master Mar 1, 2024
18 checks passed
@eradman eradman deleted the preview-image branch March 1, 2024 16:08
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.

3 participants