1563680 - Move network monitor actors into network-monitor directory

Status: RESOLVED FIXED

(DevTools :: Netmonitor, task, P3)

Description

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Network monitor actor files:

• network-monitor.js
• network-event.js

... should be moved into devtools/server/actors/network-monitor directory. This way we'll have all the Network monitor back-end nicely at one place.

Honza

Comment 1

[J]

Hi Jan,

Im pretty new to patching bugs and I’ve been looking into this bug. I think this might be a good next bug for me to work on. I found the files on my local build:

C:\mozilla-source\mozilla-central\devtools\server\actors\network-monitor.js
C:\mozilla-source\mozilla-central\devtools\server\actors\network-event.js

but I’m not sure how you would create a patch to have them moved to a different folder? Any guidance would be appreciated.

Thanks,
J

Comment 2

[J]

I assume that when I move the files locally to devtools/server/actors/network-monitor, commit those changes, and then create a differential revision, it will recognize the differences in the path and that is all that is needed for the patch? Let me know if this is correct and I'll make the changes.

Thanks,
J

Comment 3

[Jan Honza Odvarko [:Honza] (always need-info? me)]

(In reply to J from comment #2)

I assume that when I move the files locally to devtools/server/actors/network-monitor, commit those changes, and then create a differential revision, it will recognize the differences in the path and that is all that is needed for the patch? Let me know if this is correct and I'll make the changes.

Correct, but use hg mv for moving the files, so we keep history.
You also need to update moz.build files presented in related directories (this file lists all the files in dir for mozilla build system)

Honza

Comment 4

[J]

Thanks for the info Honza. I'll create a revision when I get a chance. Could you please assign me this bug then?

Thanks,
J

Comment 5

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Done
Honza

Comment 6

[J]

Hi Honza,

I was able to move the files for a commit, but I'm having an on going issue that I can't figure out. Every time I create a new revision, all of the previous patches I worked on attach themselves to this newest revision. You can see it happening in my last revision here:

https://phabricator.services.mozilla.com/D36550

My last reviewer, gl, thought that using bookmarks might solve my issue, but I tried creating one for this patch but it didn't solve the issue. Any Idea on why this merging of my previous commits might be happening? Here is my current workflow:

From /c/mozilla-source/mozilla-central:

hg bookmark bug1563680

hg mv devtools/server/actors/network-monitor.js devtools/server/actors/network-monitor/network-monitor.js
hg mv devtools/server/actors/network-event.js devtools/server/actors/network-monitor/network-event.js

./mach build
./mach run

hg add devtools/server/actors/network-monitor/network-monitor.js devtools/server/actors/network-monitor/network-event.js

hg commit -m "Bug 1563680 - Move network monitor actor files into the network-monitor directory r=Honza"

arc diff

I feel like I'm missing a step either at the beginning of the process or at the end, do I need to clean up or clear out something after creating a revision in Differential using arc diff? I noticed when I do run arc diff and my editor opens to have me fill in info, the text file template is already filled with data from the old commit messages. So it seems to be automatically attaching these old patches for some reason.

Any guidance would be greatly appreciated,
J

Comment 7

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Not sure when the problem is, but couple of comments:

1. Make sure that your patches are not committed (especially when you used HG queues, which are broken)
   Use hg log to check the history (or clone the source again)

2. When you want to update an existing phab revision use --update
   Example: arc diff .^ --update D<yourrev#> this should send only the last commit to phab
   You might also use: arc diff .^ --update D<yourrev#> --trace to get more output info

Honza

Comment 8

[J]

Hi Honza,

Is it possible to just delete all of my local commits safely and start from scratch? Since I’ve only done a couple of patches, I’m thinking that working with a clean slate might be the easiest and best way forward.

Thanks,
J

Comment 9

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Yes, you can use hg strip tip to remove the last commit (tip)
Or, if you have trouble to get to the clean state, you can clone the repo again
hg clone https://hg.mozilla.org/mozilla-central/

(don't forget to backup your changes)

Honza

Comment 10

[J]

Hi Honza,

Thank you for the info. So I should backup my changes for either option, using hg strip tip or cloning the repo again? How do I go about backing up my changes?

Thank you,
J

Comment 11

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Since this bug is only about moving two files to different dir, I don't think that there is so much changes.

I don't know in what state your repo is, so cloning it again sounds to me like the way to go.

Honza

Comment 12

[J]

Attachment: Bug 1563680 - Move network monitor actor files into the network-monitor directory r=Honza

Moved network monitor actor files:

network-monitor.js
network-event.js

into devtools/server/actors/network-monitor directory.

And updated moz.build files presented in related directories.

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/209c91065bfb
Move network monitor actor files into the network-monitor directory r=Honza

Comment 14

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a2cd0c66786a
Backed out changeset 209c91065bfb for causing mochitest failures CLOSED TREE

Comment 15

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Comments in Phab
Honza

Comment 16

[Alexandru Michis [:malexandru]]

Backed out changeset 209c91065bfb for causing mochitest failures

Backout link: https://hg.mozilla.org/integration/autoland/rev/a2cd0c66786a56d6982908a7ca7084806594a53c

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cusercancel%2Crunnable&tochange=a2cd0c66786a56d6982908a7ca7084806594a53c&fromchange=209c91065bfb9c86dc2dc9613fa1c209c26fa521&selectedJob=256881562

Failure logs;

• A promise chain failed to handle a rejection: this.threadFront is undefined - stack: _stopThreadFrontListeners@resource://devtools/client/framework/toolbox.js:572:5 & Uncaught exception - at resource://devtools/client/framework/toolbox.js:2451 - Error: Can't select tool, wait for toolbox 'ready' event
  https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=256881571&repo=autoland&lineNumber=1568

• Timeouts
  https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=256881237&repo=autoland&lineNumber=10976
  https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=256881221&repo=autoland&lineNumber=1183
  https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=256880907&repo=autoland&lineNumber=7957
  https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=256881569&repo=autoland&lineNumber=49436

Comment 17

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/47a9a289491e
Move network monitor actor files into the network-monitor directory r=Honza

Comment 18

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/47a9a289491e