Closed Bug 1043388 Opened 10 years ago Closed 10 years ago

Logparser directory on orangefactor1.dmz.phx1 contains local changes

Categories

(Tree Management Graveyard :: OrangeFactor, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

[webtools@orangefactor1.dmz.phx1 logparser]$ hg status M logparser/config.py ? logparser.egg-info/PKG-INFO ? logparser.egg-info/SOURCES.txt ? logparser.egg-info/dependency_links.txt ? logparser.egg-info/entry_points.txt ? logparser.egg-info/not-zip-safe ? logparser.egg-info/requires.txt ? logparser.egg-info/top_level.txt ? logparser/__init__.pyc ? logparser/buildlog.pyc ? logparser/config.py.orig ? logparser/config.pyc ? logparser/logparser.py.orig ? logparser/logparser.pyc ? logparser/savelogs.pyc ? logparser/taloslogparser.pyc ? logparser/testsuiteparser.pyc ? logparser/tinderboxparser.py.orig ? logparser/tinderboxparser.pyc [webtools@orangefactor1.dmz.phx1 logparser]$ hg diff diff -r 4b0fd863cdb7 logparser/config.py --- a/logparser/config.py Fri Jul 18 17:05:40 2014 -0700 +++ b/logparser/config.py Thu Jul 24 07:03:16 2014 -0700 @@ -11,15 +11,12 @@ test_list = ['reftest', 'jsreftest', 'xpcshell', 'mochitests-1', 'mochitests-2', 'mochitests-3', 'mochitests-4', 'mochitests-5', 'mochitest-browser-chrome', 'mochitest-other', 'crashtest', 'crashtest-ipc', - 'reftest-ipc', 'paint', 'svg', 'chrome', 'chrome_mac', 'tp', - 'nochrome', 'dromaeo', 'dirty', 'v8', 'xperf', + 'reftest-ipc', 'mochitest-1', 'mochitest-2', 'mochitest-3', 'mochitest-4', 'mochitest-5', 'mochitest-6', 'mochitest-7', 'mochitest-8', 'browser-chrome', 'crashtest-1', 'crashtest-2', 'jsreftest-1', 'jsreftest-2', 'jsreftest-3', 'reftest-1', - 'reftest-2', 'remote-tdhtml', 'remote-tpan', 'remote-tsspider', - 'remote-tsvg', 'remote-tp4m', 'remote-tp4m_nochrome', - 'remote-ts', 'remote-twinopen', 'remote-tzoom' + 'reftest-2' ] We need to: 1) Add *.pyc and logparser.egg-info/ to .hgignore (one isn't created yet). 2) Remove the .orig backups (presume from use of hg revert seen in bash history). 3) Work out why the changes to config.py were made, and if we need to preserve them.
> We need to: > 1) Add *.pyc and logparser.egg-info/ to .hgignore (one isn't created yet). > 2) Remove the .orig backups (presume from use of hg revert seen in bash > history). [webtools@orangefactor1.dmz.phx1 logparser]$ rm logparser/config.py.orig logparser/logparser.py.orig logparser/tinderbo xparser.py.orig > 3) Work out why the changes to config.py were made, and if we need to > preserve them. Jonathan?
Flags: needinfo?(jgriffin)
Attached patch Add an .hgignoreSplinter Review
Avoids |hg status| on orangefactor1.dmz.phx returning: ? logparser.egg-info/PKG-INFO ? logparser.egg-info/SOURCES.txt ? logparser.egg-info/dependency_links.txt ? logparser.egg-info/entry_points.txt ? logparser.egg-info/not-zip-safe ? logparser.egg-info/requires.txt ? logparser.egg-info/top_level.txt ? logparser/__init__.pyc ? logparser/buildlog.pyc ? logparser/config.pyc ? logparser/logparser.pyc ? logparser/savelogs.pyc ? logparser/taloslogparser.pyc ? logparser/testsuiteparser.pyc ? logparser/tinderboxparser.pyc
Attachment #8461565 - Flags: review?(jgriffin)
The local changes were made to skip parsing Talos tests, since we're not using Talos test data anywhere; we were only using it temporarily while investigating Talos numbers a couple of years ago.
Flags: needinfo?(jgriffin)
Attachment #8461565 - Flags: review?(jgriffin) → review+
(In reply to Ed Morley [:edmorley] from comment #2) > Created attachment 8461565 [details] [diff] [review] > Add an .hgignore https://hg.mozilla.org/automation/logparser/rev/1c1603afbfb6 (In reply to Jonathan Griffin (:jgriffin) from comment #3) > The local changes were made to skip parsing Talos tests, since we're not > using Talos test data anywhere; we were only using it temporarily while > investigating Talos numbers a couple of years ago. Ok - I think I'll just revert the changes for now, since they'll need to be rebased on top of bug 817269 anyway, if we decide to delete them again. [webtools@orangefactor1.dmz.phx1 ~]$ cd ~/apps/logparser/src/logparser [webtools@orangefactor1.dmz.phx1 logparser]$ hg revert logparser/config.py --no-backup [webtools@orangefactor1.dmz.phx1 logparser]$ hg pull -u -v pulling from http://hg.mozilla.org/automation/logparser searching for changes all local heads known remotely adding changesets adding manifests adding file changes added 6 changesets with 6 changes to 2 files resolving manifests getting .hgignore getting logparser/config.py 2 files updated, 0 files merged, 0 files removed, 0 files unresolved [webtools@orangefactor1.dmz.phx1 logparser]$ hg status <empty>
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Product: Testing → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: