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

Split -dx and base Containerfiles and Workflows #1134

Closed
m2Giles opened this issue Apr 14, 2024 · 6 comments
Closed

Split -dx and base Containerfiles and Workflows #1134

m2Giles opened this issue Apr 14, 2024 · 6 comments
Labels
dx Developer Experience Image specific github_actions Pull requests that update GitHub Actions code

Comments

@m2Giles
Copy link
Member

m2Giles commented Apr 14, 2024

Currently our -dx images build the base image in the process of building them. With the reusable build workflow we can possible have -dx built directly from the just built base image.

@m2Giles m2Giles added dx Developer Experience Image specific github_actions Pull requests that update GitHub Actions code labels Apr 14, 2024
@p5
Copy link
Member

p5 commented Apr 14, 2024

I'm in two minds about this...

It would increase the feedback loop in PRs and make us wait around double the time before merging the PRs, however it would reduce the overall time spent on builders since dx would not need to build everything in the base again.

@castrojo
Copy link
Member

I bet dx continues to rev fast so it might end up freeing up builders if they were split? I'm curious now lol.

@m2Giles
Copy link
Member Author

m2Giles commented Apr 14, 2024

#1136

@m2Giles
Copy link
Member Author

m2Giles commented Apr 16, 2024

There is a blocker for this. -dx builds are now dependent on the underlying image to build and be available. On pull requests we do not publish those images meaning that we will be unable to ensure that nothing in the underlying base has broken -dx.

@m2Giles
Copy link
Member Author

m2Giles commented Apr 21, 2024

Unless we allow PRs to be published, this is blocked.

@m2Giles m2Giles closed this as completed Apr 21, 2024
@p5
Copy link
Member

p5 commented Apr 21, 2024

An alternative could be to split the Containerfiles, but build both in a single workflow/job.

This reduces the total build runner minutes, but adds some separation.

E.g.

  1. step to build base
  2. optional push to registry
  3. step to build DX from local base
  4. optional push to registry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dx Developer Experience Image specific github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

No branches or pull requests

3 participants