Closed
Bug 1154796
Opened 10 years ago
Closed 9 years ago
make a copy of mozharness and put it in gecko tree
Categories
(Release Engineering :: Applications: MozharnessCore, defect)
Tracking
(firefox40 fixed, firefox41 fixed, firefox42 fixed, firefox-esr38 fixed)
People
(Reporter: jlund, Assigned: jlund)
References
Details
Attachments
(4 files, 1 obsolete file)
(deleted),
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
RyanVM
:
review+
jlund
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
RyanVM
:
review+
|
Details | Diff | Splinter Review |
this will initially be for internal testing (so it does not have to be mirrored)
context: http://jordan-lund.ghost.io/mozharness-steps-into-the-forest/
Assignee | ||
Comment 1•9 years ago
|
||
this ticket tracks:
part 1 - putting a copy of mozharness on ash project repo
part 2 - putting a copy of mozharness on trunk (m-c) related repos
part 3 - putting a copy of mozharness across all stabilizing/release repos
Assignee | ||
Comment 2•9 years ago
|
||
Assignee | ||
Comment 3•9 years ago
|
||
as discussed over irc yesterday. This is a straight up copy of hg.m.o/build/mozharness with two differences:
1) there is no README.md
2) I modified the existing README.txt to point out that this is a copy and based off of: http://hg.mozilla.org/build/mozharness/rev/50903ae436e5
I left mozharness.json in there for now so that as we transition, I don't break existing jobs that are still using the pin.
this patch will be a no-op until we turn it on everywhere.
Attachment #8629158 -
Flags: review?(mshal)
Comment 4•9 years ago
|
||
Comment on attachment 8629158 [details] [diff] [review]
150702_bug_1154796_in_gecko_tree_mozharness_copy-m-c.patch
Looks good! Sorry for already bitrotting you in bug 1179382 - feel free to bring in the recent production branch and carry r+ forward.
Attachment #8629158 -
Flags: review?(mshal) → review+
Comment 6•9 years ago
|
||
Assignee: nobody → jlund
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Assignee | ||
Comment 7•9 years ago
|
||
this is asking for two things:
for a review of an update to the in-tree copy of mozharness to sync with latest prod mh repo rev: 43f7600b8c80
and to be able to uplift this across all the branches with r=mshal, a=testing
again, it will be a no-op until I land: https://bugzilla.mozilla.org/attachment.cgi?id=8630722
Attachment #8630741 -
Flags: review?(mshal)
Comment 8•9 years ago
|
||
Comment on attachment 8630741 [details] [diff] [review]
150702_bug_1154796_in_gecko_tree_mozharness_sync_43f7600b8c80.patch
It looks like a few files were removed in the latest production rev of mozharness, but are still in-tree with this patch:
Only in ./configs: update_tests
Only in ./configs/vcs_sync: staging_build-repos.py
Only in ./configs/vcs_sync: staging_l10n.py
Attachment #8630741 -
Flags: review?(mshal) → feedback+
Assignee | ||
Comment 9•9 years ago
|
||
thanks for the catch. verified this time by diffing deleted files.
Attachment #8630741 -
Attachment is obsolete: true
Attachment #8631113 -
Flags: review?(mshal)
Comment 10•9 years ago
|
||
Comment on attachment 8631113 [details] [diff] [review]
150708_bug_1154796_in_gecko_tree_mozharness_sync_43f7600b8c80.patch
Looks good!
Attachment #8631113 -
Flags: review?(mshal) → review+
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Comment 12•9 years ago
|
||
verified locally with (only dotfiles differ):
jlund@Hastings163:~/devel/mozilla/cleanRepos/mozilla-central/testing/mozharness
> diff -r . $DOWNLOADS/mozharness-4a31e6739409 > /tmp/diff
Attachment #8631685 -
Flags: review?(mshal)
Updated•9 years ago
|
Attachment #8631685 -
Flags: review?(mshal) → review+
Assignee | ||
Comment 14•9 years ago
|
||
Comment on attachment 8631685 [details] [diff] [review]
150709_1154796_sync_to_4a31e6739409.patch
thanks
https://hg.mozilla.org/mozilla-central/rev/95a40a305e1b
Attachment #8631685 -
Flags: checked-in+
Assignee | ||
Comment 15•9 years ago
|
||
Attachment #8634813 -
Flags: review?(ryanvm)
Updated•9 years ago
|
Attachment #8634813 -
Flags: review?(ryanvm) → review+
Comment 16•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
Comment 17•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/43020b934bba
https://hg.mozilla.org/releases/mozilla-aurora/rev/0e2b4433dfcd
status-firefox41:
--- → fixed
Updated•9 years ago
|
Keywords: leave-open
Comment 18•9 years ago
|
||
On second thought, we decided on IRC to go with mozharness rev cd537fcb6ff0 as the final base rev for the release branches.
https://hg.mozilla.org/releases/mozilla-aurora/rev/91c53b279514
Comment 19•9 years ago
|
||
status-firefox40:
--- → fixed
Comment 20•9 years ago
|
||
status-firefox-esr38:
--- → fixed
Comment 21•9 years ago
|
||
Jordan, what is the canonical location of mozharness development at the moment? Is it still https://hg.mozilla.org/build/mozharness/ or https://hg.mozilla.org/mozilla-central/testing/mozharness?
I ask now because I got the information that I should land all my patches to mozilla-central and not in the other repository given that it is not used anymore. But yesterday I have seen that several test harnesses report a script_revlink of mozharness to Treeherder which still points to /build/mozharness. Als checking this repository I see active development! I'm kinda confused now about where to land changes.
Flags: needinfo?(jlund)
Comment 22•9 years ago
|
||
That script_revlink should be dropped.
The Firefox OS team was landing there since they had not yet moved to the in-tree mozharness.
We should make the original place read/only.
Assignee | ||
Comment 23•9 years ago
|
||
(In reply to Henrik Skupin (:whimboo) from comment #21)
> Jordan, what is the canonical location of mozharness development at the
> moment? Is it still https://hg.mozilla.org/build/mozharness/ or
> https://hg.mozilla.org/mozilla-central/testing/mozharness?
https://hg.mozilla.org/mozilla-central/testing/mozharness
there is still a small tail of things that need cleaning up: https://bugzil.la/1180060 and I just dep'd a small bug specifically to remove script_revlink: bug 1208117
all mh automation scripts bar some releng services (b2g_bumper, vcs_sync) should be using the gecko-tree based mh. please land there. we can't make the original read/only as b2g_bumper/vcs_sync still need to push there. It is forked. We could rename the build/mozharness to clear up confusion.
Flags: needinfo?(jlund)
Comment 24•9 years ago
|
||
Thanks Jordan. Good to know that all of our code landed correctly then.
You need to log in
before you can comment on or make changes to this bug.
Description
•