Generate docker images automatically when pushing

RESOLVED FIXED in 3.27

Status

NSS
Test
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: ttaubert, Assigned: ttaubert)

Tracking

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Updated

a year ago
Summary: Move nss-ci image from ttaubert/ to mozilla/ on DockerHub → Generate docker images automatically when pushing
(Assignee)

Updated

a year ago
Assignee: ttaubert → nobody
Status: ASSIGNED → NEW
(Assignee)

Updated

a year ago
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
(Assignee)

Updated

a year ago
Blocks: 1299128
(Assignee)

Updated

a year ago
Blocks: 1299130
(Assignee)

Comment 1

a year ago
https://treeherder.mozilla.org/#/jobs?repo=nss-try&revision=6939bde44b1c29b6273dcd4ca31a66a3f6fb1abd
(Assignee)

Comment 2

a year ago
Created attachment 8786342 [details] [diff] [review]
0001-Bug-1275501-Generate-docker-images-automatically-whe.patch

The "image: automation/taskcluster/docker" notation currently is only compared to a static string in the .js file but it's very easy to pick this up and support multiple image-builder tasks, e.g. one for x86 and one for arm later.
Attachment #8786342 - Flags: review?(franziskuskiefer)
(Assignee)

Comment 3

a year ago
Created attachment 8786348 [details] [diff] [review]
0001-Bug-1275501-Generate-docker-images-automatically-whe.patch, v2

https://treeherder.mozilla.org/#/jobs?repo=nss-try&revision=0ecfc939933f7d235ee21e7657fc4ac5bd7b230d
Attachment #8786342 - Attachment is obsolete: true
Attachment #8786342 - Flags: review?(franziskuskiefer)
Attachment #8786348 - Flags: review?(franziskuskiefer)
Comment on attachment 8786348 [details] [diff] [review]
0001-Bug-1275501-Generate-docker-images-automatically-whe.patch, v2

Review of attachment 8786348 [details] [diff] [review]:
-----------------------------------------------------------------

I'd really like to see some sort of caching, but I guess this is fine for the beginning.
Attachment #8786348 - Flags: review?(franziskuskiefer) → review+
(Assignee)

Comment 5

a year ago
(In reply to Franziskus Kiefer [:fkiefer or :franziskus] from comment #4)
> I'd really like to see some sort of caching, but I guess this is fine for
> the beginning.

Agreed, will work on a follow-up asap.
(Assignee)

Comment 6

a year ago
https://hg.mozilla.org/projects/nss/rev/8139abd6fa83
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → 3.27
You need to log in before you can comment on or make changes to this bug.