Closed
Bug 1293263
Opened 8 years ago
Closed 4 years ago
build docker images (but don't push) as part of CI
Categories
(Release Engineering Graveyard :: Applications: Balrog (backend), defect, P3)
Release Engineering Graveyard
Applications: Balrog (backend)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Unassigned)
Details
While making a change to the Dockerfile today, I realized that we have no verification that it will actually build (beyond doing a local build) until we merge to master. It would be good to have a job that runs as part of CI that builds the Docker image, but doesn't push it.
Reporter | ||
Updated•7 years ago
|
Priority: -- → P3
Whiteboard: [ready]
Comment 1•7 years ago
|
||
I would like to work on this issue
Comment 2•7 years ago
|
||
Does this task mean: 1. The CI should build a docker image on PRs made to master for the verification before merging and not push the image 2. Only on merge will it push the image to Dockerhub
Flags: needinfo?(bhearsum)
Reporter | ||
Comment 3•7 years ago
|
||
(In reply to Mercy Muchai from comment #2) > Does this task mean: > 1. The CI should build a docker image on PRs made to master for the > verification before merging and not push the image > 2. Only on merge will it push the image to Dockerhub Hi Mercy, I actually don't think this is a great task for you -- it's almost exclusively tied to our CI system, and requires the Dockerhub credentials to test properly.
Assignee: mercy.muchai → nobody
Flags: needinfo?(bhearsum)
Comment 4•7 years ago
|
||
Cool. Thank you for the clarification. Let me look into a new issue.
Reporter | ||
Updated•7 years ago
|
Whiteboard: [ready]
Reporter | ||
Comment 5•4 years ago
|
||
We do this now!
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Updated•4 years ago
|
Product: Release Engineering → Release Engineering Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•