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

ci: build on-demand docker images for testing on devnets #4667

Merged
merged 41 commits into from
Apr 3, 2022
Merged

ci: build on-demand docker images for testing on devnets #4667

merged 41 commits into from
Apr 3, 2022

Conversation

strophy
Copy link

@strophy strophy commented Jan 18, 2022

This PR:

  • Implements a new Dockerfile designed to quickly build and release a multi-arch docker image from any branch based on a semver tag provided by developers, as requested by @PastaPastaPasta here. Example built images are here. Note that these are not yet gitian builds, but I could modify this to do gitian builds instead (probably with a performance impact?)
  • Implements CI that integrates with this new AWS infrastructure repository, allowing builds to execute quickly on powerful ephemeral self-hosted runners. Build time is currently around 25 minutes.
  • Modifies existing CI to use an improved caching mechanism, and renames the existing release Dockerfile to live alongside the new Dockerfile for building alpha releases

@strophy strophy marked this pull request as ready for review March 8, 2022 05:01
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

Also two linters fail lint-whitespace and lint-shell

.github/workflows/release_alpha.yml Outdated Show resolved Hide resolved
contrib/gitian-build.py Outdated Show resolved Hide resolved
@PastaPastaPasta PastaPastaPasta requested a review from UdjinM6 March 11, 2022 19:34
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

🤷 LGTM, still need to change strophy/dash

.github/workflows/release_alpha.yml Outdated Show resolved Hide resolved
.github/workflows/release_alpha.yml Outdated Show resolved Hide resolved
.github/workflows/release_alpha.yml Outdated Show resolved Hide resolved
.github/workflows/release_alpha.yml Outdated Show resolved Hide resolved
.github/workflows/release_alpha.yml Outdated Show resolved Hide resolved
.github/workflows/release_alpha.yml Outdated Show resolved Hide resolved
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

LGTM, I think it's good. utACK for squash merge

PastaPastaPasta
PastaPastaPasta previously approved these changes Apr 1, 2022
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

re-utACK

.github/workflows/release_alpha.yml Outdated Show resolved Hide resolved
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK

@UdjinM6 UdjinM6 added this to the 18 milestone Apr 3, 2022
@UdjinM6 UdjinM6 requested a review from PastaPastaPasta April 3, 2022 11:25
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

re-utACK for squash merge ci is passing

@PastaPastaPasta PastaPastaPasta merged commit 1fddc53 into dashpay:develop Apr 3, 2022
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.

4 participants