Closed
Bug 730251
Opened 13 years ago
Closed 13 years ago
Prepare for domain change of l10n dashboard, possible redirects
Categories
(Release Engineering :: General, defect, P4)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Pike, Assigned: armenzg)
References
Details
(Whiteboard: waiting on dependent bug)
Attachments
(1 file)
1.90 KB,
patch
|
Pike
:
review+
rail
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
We're going to claim the l10n.mozilla.org domain for the dashboard soon, and would ask you to verify what works and what needs to change in the release automation for that.
l10n-stage-sj.m.o is likely going to redirect once we take the new domain, so the existing links don't break immediately, but I don't know if all customers of the dashboard URLs properly follow redirects?
Also, once we have the new domain, we should switch the links over to l10n.m.o so that we can rip out the redirects of the old domain after the transition period.
Assignee | ||
Comment 1•13 years ago
|
||
We have two parts:
1) grab changesets manually from the dashboard
2) before kicking the release we have a script that compares data on the dashboard with what we grabbed from #1 (just in case)
What would a URL like this look like?
https://l10n-stage-sj.mozilla.org/shipping/l10n-changesets?ms=fx11_beta_b4
Once you have the new domain up we can switch our documentation and our scripts.
Reporter | ||
Comment 2•13 years ago
|
||
https://l10n.mozilla.org/shipping/l10n-changesets?ms=fx11_beta_b4 would be the new url. Just the host name changes.
Assignee | ||
Comment 3•13 years ago
|
||
It redirects to https://l10n.mozilla-community.org should we use that? or would the redirection from l10n.m.o to l10n.m-c.o work?
Assignee | ||
Comment 4•13 years ago
|
||
Assignee: nobody → armenzg
Status: NEW → ASSIGNED
Attachment #602026 -
Flags: review?(rail)
Attachment #602026 -
Flags: review?(l10n)
Reporter | ||
Comment 5•13 years ago
|
||
(In reply to Armen Zambrano G. [:armenzg] - Release Engineer from comment #3)
> It redirects to https://l10n.mozilla-community.org should we use that? or
> would the redirection from l10n.m.o to l10n.m-c.o work?
This is just for the transition phase. Once elmo takes over l10n.m.o, it won't redirect.
Reporter | ||
Comment 6•13 years ago
|
||
Filed bug 732092 to have a tracker of the elmo side to block this one. We shouldn't land the changes here before we did the upstream changes to elmo.
Do you guys need a particular heads-up notice, and what's the timing of landing a change like this?
Depends on: 732092
Reporter | ||
Comment 7•13 years ago
|
||
Comment on attachment 602026 [details] [diff] [review]
change L10n dahsboard url
Review of attachment 602026 [details] [diff] [review]:
-----------------------------------------------------------------
This looks good from the URL POV, but there's a chunk in this patch I find surprising.
Leaving that up for Rail's review.
Also, see my previous comments, this shouldn't land before we're through with the upstream change of elmo actually serving the content on l10n.m.o.
::: buildfarm/maintenance/manage_masters.py
@@ +69,5 @@
> masters.append(m)
> elif m['role'] in options.roles:
> masters.append(m)
> + elif len(options.roles) == 1 and m["name"].endswith(options.roles[0]):
> + masters.append(m)
is this part of this patch?
Attachment #602026 -
Flags: review?(l10n) → review+
Assignee | ||
Comment 8•13 years ago
|
||
That wasn't part of the patch.
We need this just before it goes live or soon after.
We normally trigger a release on Tuesday night/ Wednesday morning.
Comment 9•13 years ago
|
||
Comment on attachment 602026 [details] [diff] [review]
change L10n dahsboard url
r+ for the first hunk.
Attachment #602026 -
Flags: review?(rail) → review+
Assignee | ||
Updated•13 years ago
|
Priority: -- → P4
Whiteboard: waiting on dependent bug
Assignee | ||
Comment 10•13 years ago
|
||
When can we land this?
Reporter | ||
Comment 11•13 years ago
|
||
We're ready on the l10-dashboard side.
Assignee | ||
Comment 12•13 years ago
|
||
Comment on attachment 602026 [details] [diff] [review]
change L10n dahsboard url
tools repo:
589d386799d9
Attachment #602026 -
Flags: checked-in+
Assignee | ||
Comment 13•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•