bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

upload release files to candidates instead of nightly

VERIFIED FIXED

Status

Cloud Services
General
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: bhearsum, Assigned: oremj)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

3 years ago
Although we still support candidates dirs in /pub/mozilla.org/firefox/nightly on FTP, /pub/mozilla.org/firefox/candidates is the official location of them. The new post upload script is only uploading to nightly, and returning URLs pointing there as well. Now is a good time to cut over and only support release builds in /pub/mozilla.org/firefox/candidates, so we may as well do it.
Yes please.
Assignee: nobody → oremj
(Assignee)

Comment 2

3 years ago
What change needs to be made?
(Assignee)

Updated

3 years ago
Flags: needinfo?(nthomas)
(Reporter)

Comment 3

3 years ago
(In reply to Jeremy Orem [:oremj] from comment #2)
> What change needs to be made?

Files should go into /pub/$product/candidates instead of /pub/$product/nightly when --release-to-candidates is used. I think https://github.com/mozilla-services/product-delivery-tools/blob/master/post_upload/postupload/release.go#L150 and https://github.com/mozilla-services/product-delivery-tools/blob/master/post_upload/postupload/release.go#L170 are returning the wrong thing (r.nightlyPath(), in particular)
(Reporter)

Comment 5

3 years ago
(In reply to Jeremy Orem [:oremj] from comment #4)
> https://github.com/mozilla-services/product-delivery-tools/pull/28 r?

Looks good to me. I can veriy once it's deployed.
Flags: needinfo?(bhearsum)
(Assignee)

Comment 6

3 years ago
The new version has been deployed.
Flags: needinfo?(nthomas)
(Assignee)

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Reporter)

Comment 7

3 years ago
Looks to be working, thanks for the quick fix :)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.