update docker-worker:image:taskclusterprivate/upload_symbols:0.0.3

RESOLVED FIXED in mozilla52

Status

Taskcluster
Docker Images
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: kmoir, Assigned: kmoir)

Tracking

unspecified
mozilla52

Details

Attachments

(3 attachments)

(Assignee)

Description

2 years ago
It uploads target.crashreporter-symbols.zip when it should upload target.crashreporter-symbols-full.zip 

See https://bugzilla.mozilla.org/show_bug.cgi?id=1314345#c9

root@f1402077dc31:/tmp# cat bin/upload.sh 
#! /bin/bash
set -e
# checkout the script
source $(dirname $0)/checkout-script.sh
# Check that we have a taskid to checkout
if [ -z ${ARTIFACT_TASKID} ]; then
    echo "Please set ARTIFACT_TASKID. Exiting"
    exit 0
fi
# grab the symbols from an arbitrary task
symbol_url=https://queue.taskcluster.net/v1/task/${ARTIFACT_TASKID}/artifacts/public/build/target.crashreporter-symbols.zip
wget ${symbol_url}
# run
symbol_zip=$(basename ${symbol_url})
script_name=$(basename ${SCRIPT_PATH})
python -u ${script_name} ${symbol_zip}

I couldn't find where the dockerfile for docker-worker:image:taskclusterprivate/upload_symbols:0.0.3 was stored, perhaps in a private repo since it's a private image
That image was created as part of bug 1168979, the Dockerfile is in this repo: https://github.com/acmiyaguchi/upload-symbols
(I would have linked that on your other bug yesterday but I could not recall it offhand and I lost track of it!)
Moving that to testing/docker in the process would probably be a good idea.
(Assignee)

Comment 4

2 years ago
Created attachment 8807642 [details] [diff] [review]
bug1315287.patch

patch to move upload-symbols dockerfile in tree
Comment on attachment 8807642 [details] [diff] [review]
bug1315287.patch

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

Thanks!  There's some trailing whitespace in the Dockerfile that could be cleaned up.
Attachment #8807642 - Flags: review+

Comment 6

2 years ago
Pushed by kmoir@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/18a98f6dabd9
update docker-worker:image:taskclusterprivate/upload_symbols:0.0.3 r=dustin DONTBUILD
(Assignee)

Updated

2 years ago
Assignee: nobody → kmoir
(Assignee)

Comment 7

2 years ago
Created attachment 8807659 [details] [diff] [review]
patch with whitespace fixed
(Assignee)

Comment 8

2 years ago
Do I have to do anything to create the new docker image, according to this is it all automatic?

https://docs.taskcluster.net/tutorial/gecko-docker-images
Flags: needinfo?(dustin)
Unfortunately you'll have to manually build this and publish it because it needs to include the Socorro authentication token. bug 1253620 covers adding a proxy to taskcluster to avoid the need to bake the token into the image.
(Assignee)

Comment 10

2 years ago
So to rebuild the docker image for the signing upload I need the socorro token which I don't have.  And presumably I need to publish it to a private place too. Anyone have any pointers to this data?
Flags: needinfo?(dustin)
I think I have the rights to push to taskclusterprivate.  Which means I can also pull from it and find the socorro token..
I dunno where it's stored, but there's a puppet module to add it to buildbot builders:
https://dxr.mozilla.org/build-central/source/puppet/modules/slave_secrets/manifests/crash_stats_api_token.pp

You can always have coop generate a new one by logging into crash-stats. That's all we did originally...
OK, pushed the updated version from attachment 8807659 [details] [diff] [review] as 0.0.4.
0.0.4: digest: sha256:c3a5cce3e37aba59971462a6a9ddd54af7253e982115776b310c189128b5458f size: 2612

if anyone cares :)
(Assignee)

Comment 15

2 years ago
thanks dustin!

Comment 16

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/18a98f6dabd9
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
(Assignee)

Comment 17

2 years ago
Created attachment 8808201 [details] [diff] [review]
bug1315287-2.patch

patch to use the new image

Comment 18

2 years ago
Pushed by kmoir@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c1e5b5c23216
update docker-worker:image:taskclusterprivate/upload_symbols:0.0.3 r=kmoir DONTBUILD
You need to log in before you can comment on or make changes to this bug.