1657199 - Generate the same docker-image digests locally as in automation.

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, defect)

Description

[Tom Prince [:tomprince]]

In bug 1626058, I changed how the docker image digest was generated:

• I used the same directory structure to generate the digest as was used for generating the context
• I moved context generation to the decision task, and used the hash of that as part of the digest.

Unfortunately, it turns out the file name in the gzip part of the context .tar.gz differed between when we are creating a context to write out, and when were just generating the hash.

Comment 1

[Tom Prince [:tomprince]]

Attachment: Bug 1657199: Generate the same docker-image digests locally as in automation; r?Callek

In bug 1626058, I changed how the docker image digest was generated:

• I used the same directory structure to generate the digest as was used for generating the context
• I moved context generation to the decision task, and used the hash of that as part of the digest.

Unfortunately, it turns out the file name in the gzip header of the context
.tar.gz differed between when we are creating a context to write out, and when
were just generating the hash.

This adjust the name used in the gzip header to be consistent.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7f0f8c22d1cc
Generate the same docker-image digests locally as in automation; r=Callek

Comment 3

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/7f0f8c22d1cc

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1626058

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

The patch landed in nightly and beta is affected.
:tomprince, is this bug important enough to require an uplift?
If not please set status_beta to wontfix.

For more information, please visit auto_nag documentation.

Comment 6

[(Out Sep 17-23) Ryan VanderMeulen]

https://hg.mozilla.org/releases/mozilla-beta/rev/6fa6624abc9c

Comment 7

[Tom Prince [:tomprince]]

It isn't needed, but it doesn't hurt.