Closed Bug 405185 Opened 12 years ago Closed 12 years ago
Talos fails on 10
.5/Leopard: user does not exist
It looks like the arguments to ps have changed which causes the GetProcessData function in cmanger_mac.py to fail on 10.5. This function uses: ps -Acup+pid (e.g. ps -Acup303) If you run this in the terminal you'll get "ps: No user named 'p303'. This causes Talos to hang when it hits this spot. The semantics of -u has changed betwen 10.4 and 10.5 it seems. Simple fix is to tell ps what you want it to print 'ps -o pid,vsize,rss -p+pid' Which also simplifies the parsing. Tried this on 10.4 and works great there as well. Patch attached.
Comment on attachment 289993 [details] [diff] [review] Patch to make talos run fine on 10.5 First attachment was wrong - this one should do the trick. I know Talos isn't meant for 10.5 right away - but this may make it easier for folks doing stand-alone testing.
Attachment #289993 - Flags: review?(anodelman)
Summary: Talos does fails on 10.5/Leopard: user does not exist → Talos fails on 10.5/Leopard: user does not exist
looks good to me.
Comment on attachment 289993 [details] [diff] [review] Patch to make talos run fine on 10.5 Looks good to me too.
Attachment #289993 - Flags: review?(anodelman) → review+
Checking in cmanager_mac.py; /cvsroot/mozilla/testing/performance/talos/cmanager_mac.py,v <-- cmanager_mac.py new revision: 1.2; previous revision: 1.1 done
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Mass move of Core:Testing bugs to mozilla.org:Release Engineering:Talos. Filter on RelEngTalosMassMove to ignore.
Component: Testing → Release Engineering: Talos
Product: Core → mozilla.org
QA Contact: testing → release
Version: Trunk → other
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.