Closed Bug 1313428 Opened 4 years ago Closed 3 years ago

Autophone - ap-assignments.py - autophone_path thinko

Categories

(Testing :: Autophone, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bc, Assigned: bc)

References

Details

Attachments

(2 files)

ap-assignments does not take into account the [builds] buildtypes in test config files which results in all tests being treated as if they have both opt and debug set. This is due to a thinko when I added the utils.autophone_path() function:

--- a/ap-assignments.py
+++ b/ap-assignments.py
@@ -219,7 +219,7 @@ for test_name in test_sections:
         if test_option == 'config':
             # config is required and is the first option in the section.
             test_config = ConfigParser.RawConfigParser()
-            test_config.read("%s/tests/%s" % (autophone_path,
+            test_config.read("%s/tests/%s" % (autophone_path(),
                                               test_value))
             try:
                 test_build_types = test_config.get('builds', 'buildtypes').split()
Attachment #8805206 - Flags: review?(gbrown)
Attachment #8805206 - Flags: review?(gbrown) → review+
updates ap-assignments.py to add distinct opt,debug timing, update run times and load based on recent history and https://treeherder.mozilla.org/#/jobs?repo=try&revision=0af35e37244027de0a91b4514c20654d561e4144&group_state=expanded and removes the fx-team repo since it is being obsoleted in bug 1296046.
Attachment #8805600 - Flags: review?(gbrown)
Comment on attachment 8805600 [details] [diff] [review]
bug-1313428-2-v1.patch

Why no more fx-team?
Attachment #8805600 - Flags: review?(gbrown) → review+
Oh, I see - bug 1296046.
deployed ~2016-11-13 22:20 PST.
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.