Beginning on October 25th, 2016, Persona will no longer be an option for authentication on BMO. For more details see Persona Deprecated.
Last Comment Bug 620865 - Upgrade cvs/cvs-mirror to cvs 1.12.13
: Upgrade cvs/cvs-mirror to cvs 1.12.13
Product: Developer Services
Classification: Other
Component: General (show other bugs)
: other
: All Other
: -- enhancement
: ---
Assigned To: server-ops-webops
: Jake Maul [:jakem]
Depends on:
Blocks: 628913
  Show dependency treegraph
Reported: 2010-12-21 20:30 PST by Dave Miller [:justdave] (
Modified: 2014-10-10 13:50 PDT (History)
11 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Description Dave Miller [:justdave] ( 2010-12-21 20:30:00 PST
cvs 1.12 has a few useful features 1.11 doesn't have that the few remaining people using cvs would like to be able to use (being able to diff on a date range on a branch is one of them).

We've been running a custom-compiled version of 1.11 that was patched to not do any read locking on cvs-mirror since it was a read-only repository.  1.12 already contains this feature (though with a different command-line switch) and so will not require any patches to work (but we will have to recompile our pseudo-shell to use the new command-line switch to call it).
Comment 1 Dave Miller [:justdave] ( 2010-12-21 20:40:45 PST
Turns out the pseudo-shell was included in the custom package we built, so I rebuilt with that change to it included.

Wrote: /usr/src/redhat/SRPMS/cvs-1.12.13-0mozilla1.el5.src.rpm
Wrote: /usr/src/redhat/RPMS/i386/cvs-1.12.13-0mozilla1.el5.i386.rpm

RPM is on im-pkgdev02.

Probably need to schedule an outage window for this just as a heads-up to people in case anything happens.

Should be no visible downtime with 10 minutes or less to revert if anything ends up broken.

*MAY* require a tree closure, but that depends on what's actually still using CVS at all.
Comment 2 Reed Loden [:reed] (use needinfo?) 2010-12-21 20:45:00 PST
Nothing production should be using the mirror... If there is anything left that uses CVS, I would think it would be using directly, no?
Comment 3 Nelson Bolyard (seldom reads bugmail) 2010-12-21 20:52:21 PST
NSS and NSPR still use Mozilla's CVS repository as their upstream repository.
The copies in Mozilla's Hg repo are considered downstream.
Comment 4 Dave Miller [:justdave] ( 2010-12-22 01:01:43 PST
This would cover as well, not just the mirrors.
Comment 5 matthew zeier [:mrz] 2011-01-12 21:31:29 PST
Can someone confirm if this is a tree closure event?
Comment 6 Ben Hearsum (:bhearsum) 2011-01-13 09:35:09 PST
I don't think we need downtime for this, as far as I can tell. Besides release automation, I can't find any part of a production releng system that depends on CVS. The next scheduled releases start on Monday/Tuesday, so doing this then would be bad, but any other time should be fine from our standpoint.
Comment 7 Wan-Teh Chang 2011-01-13 10:31:16 PST
You just need to announce in the and newsgroup to notify the NSPR and
NSS developers and users of the scheduled downtime of and cvs-mirror.
Comment 8 Dave Miller [:justdave] ( 2011-01-25 21:21:47 PST
This is done. was updated within 5 minutes of the begining of the downtime window.  cvs-mirror just completed just now.  Wasn't so worried about that because of the redundancy (swapped them out on the load balancer while playing).  I can't get the new -R mode to work in the new CVS.  It spews lots of nasty warnings that the clients can't deal with, or if you shut it up with -Q it just doesn't work?  Not many people are using cvs anymore so we probably don't really need the perf boost it would buy us anyway, so I just disabled the read-only stuff, and it seems to work great without it, as far as I can tell.

All three servers are now reporting:

Server: Concurrent Versions System (CVS) 1.12.13 (client/server)
Comment 9 Dave Miller [:justdave] ( 2011-01-25 21:56:59 PST
OK, I reverted cvs-mirror back to 1.11.22.  We're getting permission errors trying to create locks and I don't have the energy to chase that down right now (I suspect anything we can do on that front will get screwed by the rsync anyway).

This needs some more thought, preferably by someone that doesn't mind hacking on CVS.
Comment 10 Dave Miller [:justdave] ( 2011-01-25 23:30:25 PST had issues, as well (see bug 628913) and has been reverted.
Comment 11 Aki Sasaki [:aki] 2011-02-02 11:32:29 PST
Are we still planning on doing this?
Removing the needs-treeclosure flag until we need to schedule this.
Comment 12 Dave Miller [:justdave] ( 2011-02-02 14:53:06 PST
It's going to need some debugging work done with it that I don't have time to do in the immediate future.  Might revisit in a week or so if I get caught up on everything else on my plate.
Comment 13 Kendall Libby [:fubar] 2014-05-29 12:04:22 PDT
CVS is slated to die, so wontfixing.

Note You need to log in before you can comment on or make changes to this bug.