Closed Bug 1381770 Opened 2 years ago Closed 2 years ago

Miscellaneous tooltool-download.sh changes

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set

Tracking

(firefox56 fixed)

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(5 files)

No description provided.
Comment on attachment 8887387 [details]
Bug 1381770 - Use $WORKSPACE instead of $HOME/workspace in tooltool-download.sh.

https://reviewboard.mozilla.org/r/158232/#review164252
Attachment #8887387 - Flags: review?(mshal) → review+
Comment on attachment 8887388 [details]
Bug 1381770 - Enable multiple download retries in tooltool-download.sh.

https://reviewboard.mozilla.org/r/158234/#review164254
Attachment #8887388 - Flags: review?(mshal) → review+
Comment on attachment 8887389 [details]
Bug 1381770 - In tooltool-download.sh, only use the relengapi proxy when it's configured.

https://reviewboard.mozilla.org/r/158236/#review164258

::: taskcluster/scripts/misc/tooltool-download.sh:7
(Diff revision 1)
>  
>  cd $WORKSPACE/build/src
>  
> +TOOLTOOL_DL_FLAGS=
> +
> +if [ -n "$RELENGAPI_PORT" ]; then

Where does RELENGAPI_PORT get set? Would it make sense to pass in a TOOLTOOL_URL instead and set the --tooltool-url flag based on that?
Attachment #8887389 - Flags: review?(mshal) → review+
Comment on attachment 8887390 [details]
Bug 1381770 - In tooltool-download.sh, automatically setup the relengapi authentication file.

https://reviewboard.mozilla.org/r/158238/#review164260
Attachment #8887390 - Flags: review?(mshal) → review+
Comment on attachment 8887391 [details]
Bug 1381770 - Don't use the relengapi proxy for Linux toolchain jobs.

https://reviewboard.mozilla.org/r/158240/#review164262
Attachment #8887391 - Flags: review?(mshal) → review+
(In reply to Michael Shal [:mshal] from comment #8)
> Comment on attachment 8887389 [details]
> Bug 1381770 - In tooltool-download.sh, only use the relengapi proxy when
> it's configured.
> 
> https://reviewboard.mozilla.org/r/158236/#review164258
> 
> ::: taskcluster/scripts/misc/tooltool-download.sh:7
> (Diff revision 1)
> >  
> >  cd $WORKSPACE/build/src
> >  
> > +TOOLTOOL_DL_FLAGS=
> > +
> > +if [ -n "$RELENGAPI_PORT" ]; then
> 
> Where does RELENGAPI_PORT get set? Would it make sense to pass in a
> TOOLTOOL_URL instead and set the --tooltool-url flag based on that?

It's set $somewhere when relengapi-proxy is enabled.
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/5dbf26e7d92d
Use $WORKSPACE instead of $HOME/workspace in tooltool-download.sh. r=mshal
https://hg.mozilla.org/integration/autoland/rev/688c2808b32c
Enable multiple download retries in tooltool-download.sh. r=mshal
https://hg.mozilla.org/integration/autoland/rev/ba2a0e45421b
In tooltool-download.sh, only use the relengapi proxy when it's configured. r=mshal
https://hg.mozilla.org/integration/autoland/rev/66bb1f3a35f9
In tooltool-download.sh, automatically setup the relengapi authentication file. r=mshal
https://hg.mozilla.org/integration/autoland/rev/a2ae5410cfd3
Don't use the relengapi proxy for Linux toolchain jobs. r=mshal
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.