The default bug view has changed. See this FAQ.

need to add comm-2.0 and releases/comm-{aurora,beta} to MXR

RESOLVED FIXED

Status

mozilla.org Graveyard
Server Operations
--
major
RESOLVED FIXED
6 years ago
2 years ago

People

(Reporter: InvisibleSmiley, Assigned: jakem)

Tracking

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
+++ This bug was initially created as a clone of Bug #446355 +++

comm-central, comm-1.9.1 and comm-1.9.2 already exist on MXR, so I guess the setup should need no further explanation. Please add comm-2.0 which branch SeaMonkey (SM2.1) is built from.

While we're at it, we may also clarify the descriptions:
- comm-central: used for trunk SM, TB and Calendar
- comm-2.0: used for branch SM
- comm-1.9.2: used for branch TB and Calendar
- comm-1.9.1: used for branch SM, TB and Calendar
ping?
This is a configuration change moreso than a code change, i.e. needs to be done by ops, not by the app dev.
Assignee: nobody → server-ops
Component: MXR → Server Operations: Web Content Push
Product: Webtools → mozilla.org
QA Contact: mxr → mrz
Version: Trunk → other
Blocks: 646804
Depends on: 649855
(In reply to comment #2)
> This is a configuration change moreso than a code change, i.e. needs to be
> done by ops, not by the app dev.

eh, config should be in version control, really... which makes it app-related.

Comment 4

6 years ago
Not sure if this belongs in ops or app dev. Dave can you make the config change if it is for ops, or re-assign if it is for app dev?
Assignee: server-ops → justdave
Assigning to myself, since it was pointed out that mxr lives in hg, and needs code changes.

Since I'm doing the work, and easier to do all at once, I'm morphing this to track comm-aurora/beta as well.
Assignee: justdave → bugspam.Callek
Blocks: 656838
Summary: need to add comm-2.0 to MXR → need to add comm-2.0 and releases/comm-{aurora,beta} to MXR

Updated

6 years ago
Component: Server Operations: Web Content Push → MXR
Product: mozilla.org → Webtools
QA Contact: mrz → mxr
Created attachment 546390 [details] [diff] [review]
do it v1

timeless, dave, whoever reviews first is fine. Also do I need a special deploy date or does mxr pull from hg periodically automatically?

Mark, is this mxr index format ok for you from a useability standpoint for TB use as well?
Attachment #546390 - Flags: review?(timeless)
Attachment #546390 - Flags: review?(justdave)
Attachment #546390 - Flags: feedback?(mbanner)
Comment on attachment 546390 [details] [diff] [review]
do it v1

...sorry for tab insertion, fixed locally.

Comment 8

6 years ago
(In reply to comment #6)
> Created attachment 546390 [details] [diff] [review] [review]
> do it v1
> 
> timeless, dave, whoever reviews first is fine. Also do I need a special
> deploy date or does mxr pull from hg periodically automatically?
> 
> Mark, is this mxr index format ok for you from a useability standpoint for
> TB use as well?

Will comm-2.0 be in another patch?
Status: NEW → ASSIGNED
(In reply to comment #8)
> Will comm-2.0 be in another patch?

Whops, yes. I'll correct that tonight.
Comment on attachment 546390 [details] [diff] [review]
do it v1

Looks fine to me, though you may want to clean up tabs versus spaces ;-)
Attachment #546390 - Flags: feedback?(mbanner) → feedback+
Callek, update the patch, and then we'll try and hopefully it'll get reviewed this time?
Created attachment 560811 [details] [diff] [review]
v1.1

This should do it. Now if I can only get official review.
Attachment #546390 - Attachment is obsolete: true
Attachment #546390 - Flags: review?(timeless)
Attachment #546390 - Flags: review?(justdave)
Attachment #560811 - Flags: review?(timeless)
Attachment #560811 - Flags: review?(justdave)

Updated

6 years ago
Attachment #560811 - Attachment is patch: true
Comment on attachment 560811 [details] [diff] [review]
v1.1

Cory, can we find someone to own this service/review this. Dave (over IRC) insists it is not him, Timeless hasn't been seen here in a while, etc.

This is basically blocking both SeaMonkey and Thunderbird from doing our rapid-release work as efficiently as possible. And making it 10x harder to diagnose risk on our approvals. I had the prior patch on here awaiting review for a long time previous as well. And even before that the bug itself was around longer.
Attachment #560811 - Flags: review?(cshields)
hm, AFAICT mxr isn't owned (no match for the string in https://wiki.mozilla.org/Modules/All ). Justin, if this goes on, maybe we should try "Figuring out what to do if a module isn't getting enough attention", see https://wiki.mozilla.org/Modules/All#Module_Ownership_System
applied the patch to stage..  More work needs to be done to make it all work though.  Cron script didn't seem happy either when I tested it.

I'm out of time on this for now, will come back to it when I have some time.
Assignee: bugspam.Callek → cshields
Component: MXR → Server Operations
Product: Webtools → mozilla.org
QA Contact: mxr → cshields
(In reply to Corey Shields [:cshields] from comment #15)
> applied the patch to stage..  More work needs to be done to make it all work
> though.  Cron script didn't seem happy either when I tested it.
> 
> I'm out of time on this for now, will come back to it when I have some time.

Corey, any updates on this - that comment was more than a month ago.

I'm bumping the severity as this has been on-going for more than 6 months now and this keeps hampering us when we want to check things on branches.
Severity: normal → major
(In reply to Mark Banner (:standard8) from comment #16)
> Corey, any updates on this - that comment was more than a month ago.

nope.. haven't had time for this, it has been preempted.

> I'm bumping the severity as this has been on-going for more than 6 months
> now and this keeps hampering us when we want to check things on branches.

I'll reassign to jakem in hopes that he can fit it in.  He's in the data center through the end of this week so it might be next week before he can look at it.
Assignee: cshields → nmaul
Status: ASSIGNED → NEW
(Assignee)

Comment 18

5 years ago
I'll see what I can do on this. As stated before, MXR doesn't *have* a maintainer, really, which causes lots of problems like this (new trees needing tracked, bug fixing, performance tuning, etc).

I do not have Hg commit access at the moment. On top of that, there have been local changes to the MXR configs over the last few months, mostly due to this type of confusion (IT making changes b/c there is no dev maintainer). Those will need to get merged into the repo as well.

I would not expect to have it completed this week, but I should be able to at least look it over. I suspect adding these should be pretty straightforward.
(Reporter)

Comment 19

5 years ago
(In reply to Jake Maul [:jakem] from comment #18)
> I suspect adding these should be pretty straightforward.

I guess so, too, simply because similar setups already exist (comm-1.9.[12]).

Also note that comm-2.0 is not used anymore (it was only used for the SeaMonkey 2.1 branch, and SeaMonkey 2.1 is already discontinued and unsupported) so it only needs to be processed once and can then be left alone. Not sure about comm-1.9.[12]; I'll leave that to Callek and Mark.
comm-1.9.[12] are useful to have around - 1.9.2 is still active in any case. However they are useful in just the same way that it is useful to have the mozilla cvs repository and the mozilla-1.8 repos around - sometimes it is useful to be able to go back to those and search for code to get the historical perspective.

They could be updated less frequently if that helps, I believe older repos are already updated less frequently.
(Assignee)

Comment 21

5 years ago
comm-2.0 is done processing, but per this bug will not be scheduled for regular processing... one-time-only, for reference.

comm-beta and comm-aurora have just finished as well. They will be scheduled up for daily processing. This matches mozilla-beta and mozilla-aurora.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Attachment #560811 - Flags: review?(timeless)
Attachment #560811 - Flags: review?(justdave)
Attachment #560811 - Flags: review?(cshields)
Product: mozilla.org → mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.