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)
Tree Management Graveyard
OrangeFactor
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
Details
Attachments
(1 file)
403 bytes,
patch
|
jgriffin
:
review+
|
Details | Diff | Splinter Review |
[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.
Assignee | ||
Comment 1•10 years ago
|
||
> 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)
Assignee | ||
Comment 2•10 years ago
|
||
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)
Comment 3•10 years ago
|
||
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)
Updated•10 years ago
|
Attachment #8461565 -
Flags: review?(jgriffin) → review+
Assignee | ||
Comment 4•10 years ago
|
||
(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
Updated•10 years ago
|
Product: Testing → Tree Management
Updated•4 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•