Closed
Bug 764284
Opened 13 years ago
Closed 13 years ago
hgtool.py isn't working correctly for releases (fails to pull release branch)
Categories
(Release Engineering :: Release Automation, defect)
Release Engineering
Release Automation
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: standard8, Assigned: bhearsum)
References
Details
Attachments
(1 file)
875 bytes,
patch
|
catlee
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
hgtool.py is currently used on the source step. However it isn't working correctly, and will always clone the repository rather than using the share.
I've see two versions of logs.
1) If the hg share for the repository doesn't exist, hgtool will clone the whole repository to the share location and do a share off of that.
This works fine as cloning the whole repository gets everything including all branches.
2) If the hg share for the repository does exist, the following happens:
command: START
command: hg pull -b default http://hg.mozilla.org/releases/mozilla-beta
command: cwd: /builds/hg-shared/releases/mozilla-beta
command: output:
pulling from http://hg.mozilla.org/releases/mozilla-beta
searching for changes
no changes found
command: END (1.86s elapsed)
Trying to share /builds/hg-shared/releases/mozilla-beta to /builds/slave/rel-m-beta-firefox-source/mozilla-beta
command: START
command: hg share -U /builds/hg-shared/releases/mozilla-beta /builds/slave/rel-m-beta-firefox-source/mozilla-beta
command: cwd: /builds/slave/rel-m-beta-firefox-source
command: output:
command: END (0.11s elapsed)
command: START
command: hg update -C -r FIREFOX_14_0b7_RELEASE
command: cwd: /builds/slave/rel-m-beta-firefox-source/mozilla-beta
command: output:
abort: unknown revision 'FIREFOX_14_0b7_RELEASE'!
command: END (27.90s elapsed)
Error updating /builds/slave/rel-m-beta-firefox-source/mozilla-beta from sharedRepo (/builds/hg-shared/releases/mozilla-beta):
command: START
command: hg clone http://hg.mozilla.org/releases/mozilla-beta /builds/slave/rel-m-beta-firefox-source/mozilla-beta
command: cwd: /builds/slave/rel-m-beta-firefox-source
command: output:
requesting all changes
adding changesets
adding manifests
adding file changes
added 96053 changesets with 453822 changes to 93742 files (+207 heads)
updating to branch default
58052 files updated, 0 files merged, 0 files removed, 0 files unresolved
command: END (389.90s elapsed)
Hence the share has "hg pull -b default" performed on it - because of the branch specification, this doesn't necessarily pull in the revisions from the branches. So the update to the revision that is on the branch then fails, so it pulls the whole repository from scratch.
Reporter | ||
Comment 1•13 years ago
|
||
I think the solution here is if no branch argument is specified to hgtool, then don't put in "default", just leave the branch passed to the mercurial function as None, xref:
http://hg.mozilla.org/build/tools/file/cf15fda270bf/buildfarm/utils/hgtool.py#l22
http://hg.mozilla.org/build/tools/file/cf15fda270bf/buildfarm/utils/hgtool.py#l85
If the branch isn't specified, then hg pull is free to get all branches and hence hg update can then do the right thing.
If we're concerned about that pulling all branches, another option might be to enable clone_by_revision, but I don't think that's quite necessary, and I'm not sure it would work with tags.
Assignee | ||
Comment 2•13 years ago
|
||
That seems reasonable to me. Any thoughts, Rail or Chris?
Comment 3•13 years ago
|
||
I think that hgtool.py shouldn't decide instead of you what branch to pull from. If you want to pull from a branch, just use -b! :)
Setting branch=None as a default value sounds OK to me.
Assignee | ||
Comment 4•13 years ago
|
||
Attachment #634049 -
Flags: review?(catlee)
Assignee | ||
Comment 5•13 years ago
|
||
I tried to verify whether or not we always pull by revision or branch, but it's almost impossible to because they're read from a properties file in most cases. Based on the build logs I've looked at it seems like we are in all of the most common cases....
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → bhearsum
Updated•13 years ago
|
Attachment #634049 -
Flags: review?(catlee) → review+
Assignee | ||
Updated•13 years ago
|
Attachment #634049 -
Flags: checked-in+
Assignee | ||
Updated•13 years ago
|
Status: NEW → 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
•