Closed
Bug 767117
Opened 10 years ago
Closed 10 years ago
jetperf needs config files
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: k0scist, Unassigned)
References
Details
(Whiteboard: [mozharness])
Attachments
(1 file, 2 obsolete files)
2.43 KB,
patch
|
aki
:
review+
|
Details | Diff | Splinter Review |
In order to be run in production, jetperf will need a production config file. It should probably also have a more developer friendly config.
Reporter | ||
Updated•10 years ago
|
Whiteboard: [mozharness]
Reporter | ||
Comment 1•10 years ago
|
||
By no means finished, just wanted to put up what I have before I get pulled off to something else
Reporter | ||
Comment 2•10 years ago
|
||
These seem to work for me. I'm not sure if this is adequate for production or not, but its my best first guess. I'm also proposing a test_config.py for local testing, but I don't really care if this makes it into the mozharness repo or not
Attachment #637703 -
Attachment is obsolete: true
Attachment #642615 -
Flags: feedback?(aki)
Comment 3•10 years ago
|
||
Comment on attachment 642615 [details] [diff] [review] initial implementation That works. Are we going to be doing any production work on windows? If so, we need windows-specific config files as well; hopefully the existing jetperf stuff will help.
Attachment #642615 -
Flags: feedback?(aki) → feedback+
Reporter | ||
Comment 4•10 years ago
|
||
(In reply to Aki Sasaki [:aki] from comment #3) > Comment on attachment 642615 [details] [diff] [review] > initial implementation > > That works. > Are we going to be doing any production work on windows? > If so, we need windows-specific config files as well; hopefully the existing > jetperf stuff will help. AFAIK, we do want to run on windows. Alex, is that correct?
Comment 5•10 years ago
|
||
(In reply to Jeff Hammel [:jhammel] from comment #4) > AFAIK, we do want to run on windows. Alex, is that correct? Yes. While not being mandatory for a first step, it sounds quite important to test on the most used platform.
Reporter | ||
Comment 6•10 years ago
|
||
This includes a windows config with parameters taken from peptest
Attachment #642615 -
Attachment is obsolete: true
Attachment #642715 -
Flags: review?(aki)
Comment 7•10 years ago
|
||
Comment on attachment 642715 [details] [diff] [review] proposed implementation We may have to put in a pypi_url to avoid hitting pypi proper, and other buildbot/prod-related things, but this should allow us to have something to work off of.
Attachment #642715 -
Flags: review?(aki) → review+
Reporter | ||
Comment 8•10 years ago
|
||
pushed: http://hg.mozilla.org/build/mozharness/rev/e326a6a377c2
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•9 years ago
|
Product: mozilla.org → Release Engineering
Assignee | ||
Updated•4 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•