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

chore(ci): Create docker images for commits in cd/staging #1090

Merged
merged 4 commits into from
Nov 16, 2022

Conversation

placer14
Copy link
Contributor

This PR causes commits to the branch cd/staging to trigger docker image build pipeline.

@placer14 placer14 added the kind/chore Technical debt due to be resolved label Nov 16, 2022
@placer14 placer14 self-assigned this Nov 16, 2022
@placer14 placer14 requested a review from frrist November 16, 2022 00:08
@@ -346,7 +303,7 @@ jobs:
# used in `build-docker-images` workflow below
branch_filters: &branch_filters
branches:
only: /.*deploy.*/
only: /cd\/staging/
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We are reusing prior behavior. This causes us to detect changes on cd/staging and trigger this job.

@@ -89,7 +55,8 @@ jobs:
echo "$DOCKERHUB_PASS" | docker login -u "$DOCKERHUB_USERNAME" --password-stdin
sanitized_branch=`echo $CIRCLE_BRANCH | sed 's:/:-:g'`
tag_suffix=$(echo "$CIRCLE_SHA1" | cut -c 1-8)
make docker-mainnet-push LILY_IMAGE_TAG=${sanitized_branch}-${tag_suffix}
tag_date=$(date +%Y%m%d%H%M)
make docker-mainnet-push LILY_IMAGE_TAG=${sanitized_branch}-${tag_date}-${tag_suffix}
Copy link
Contributor Author

@placer14 placer14 Nov 16, 2022

Choose a reason for hiding this comment

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

The new docker image tags that are created look like cd-staging-${tag_date}-${tag_suffix}. This is what we will filter on when deciding what to deploy into staging.

@codecov-commenter
Copy link

Codecov Report

Merging #1090 (b8ac585) into master (8d3c4b7) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #1090   +/-   ##
======================================
  Coverage    34.2%   34.2%           
======================================
  Files          44      44           
  Lines        2945    2945           
======================================
  Hits         1008    1008           
  Misses       1841    1841           
  Partials       96      96           

@frrist frrist requested a review from birdychang November 16, 2022 00:50
@placer14 placer14 merged commit d90220e into master Nov 16, 2022
@placer14 placer14 deleted the mg/chore/create-docker-image-for-dev-release branch November 16, 2022 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/chore Technical debt due to be resolved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants