Open
Bug 1255476
Opened 8 years ago
Updated 2 years ago
Make a task for running JS gdb integration test
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
ASSIGNED
People
(Reporter: sfink, Assigned: sfink)
References
Details
Attachments
(4 files, 2 obsolete files)
4.81 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
10.91 KB,
patch
|
garndt
:
review+
|
Details | Diff | Splinter Review |
2.54 KB,
patch
|
garndt
:
review+
|
Details | Diff | Splinter Review |
4.98 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
The JS tree contains some prettyprinters for various types, and a test suite for when we inevitably break them. It would be good to have a task that runs these tests. I'm not sure when to schedule these jobs, though. I'll worry about that later.
Assignee | ||
Comment 1•8 years ago
|
||
Some refactoring to make it easier to create sm-* jobs that share some but not all things with existing sm-* jobs.
Assignee | ||
Comment 2•8 years ago
|
||
First step is to teach autospider.sh how to run the gdb tests.
Attachment #8729063 -
Flags: review?(terrence)
Assignee | ||
Comment 3•8 years ago
|
||
Create the gdb test task. It is a little arguable whether we actually need a separate job for this. But up until now, breaking the gdb integration has not been considered a tier 1 failure, so I am reluctant to make it one without getting buy-in first. And it seems like it's pretty harmless to have a job that only runs on try if you explicitly request it. I'll need to figure out when and how to get it scheduled for integration trees eventually.
Assignee | ||
Comment 4•8 years ago
|
||
This is almost entirely just refactoring, but one part needs review: I removed the buildbot routes from sm_base.yml because (1) they didn't seem like they were likely to be used or useful; and (2) they were named "sm-plain" for *all* sm-* jobs, which clearly doesn't seem right. But I may be wrong and they may have been used for something?
Attachment #8729068 -
Flags: review?(garndt)
Assignee | ||
Updated•8 years ago
|
Attachment #8729061 -
Attachment is obsolete: true
Assignee | ||
Comment 5•8 years ago
|
||
Minor update to move the sm_base.yml changes to the earlier patch, since they make more sense there.
Attachment #8729070 -
Flags: review?(garndt)
Assignee | ||
Updated•8 years ago
|
Attachment #8729064 -
Attachment is obsolete: true
Comment 6•8 years ago
|
||
Comment on attachment 8729063 [details] [diff] [review] Add --test, --skip-test, and gdb test to autospider.sh Review of attachment 8729063 [details] [diff] [review]: ----------------------------------------------------------------- Very neat!
Attachment #8729063 -
Flags: review?(terrence) → review+
Updated•8 years ago
|
Attachment #8729068 -
Flags: review?(garndt) → review+
Updated•8 years ago
|
Attachment #8729070 -
Flags: review?(garndt) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/5315d961f36e https://hg.mozilla.org/integration/mozilla-inbound/rev/f2e2b775624f https://hg.mozilla.org/integration/mozilla-inbound/rev/9485ee35cb1d
Comment 8•8 years ago
|
||
Backed out for breaking Windows SM jobs. https://hg.mozilla.org/integration/mozilla-inbound/rev/38508f1f08d1 https://treeherder.mozilla.org/logviewer.html#?job_id=23468897&repo=mozilla-inbound#L3130 /c/builds/moz2_slave/m-in_w32_sm-plain-000000000000/src/js/src/devtools/automation/autospider.sh: line 189: declare: -A: invalid option
Assignee | ||
Comment 9•8 years ago
|
||
$ bash --version GNU bash, version 3.1.23(1)-release (i686-pc-msys) :(
Assignee | ||
Comment 10•8 years ago
|
||
Ok, we'll have to drop back to a somewhat more Real Programmer-y way of doing this.
Attachment #8729278 -
Flags: review?(terrence)
Comment 11•8 years ago
|
||
I'll bet Python would be vastly more portable than Bash, and work better on Windows to boot. AND ACTUALLY BE A REAL LANGUAGE WITH, YOU KNOW, TYPES
Comment 12•8 years ago
|
||
Comment on attachment 8729278 [details] [diff] [review] Add --test, --skip-test, and gdb test to autospider.sh, Review of attachment 8729278 [details] [diff] [review]: ----------------------------------------------------------------- I'll accept this for now so we can move on, but it is time to rewrite this in python: most of the logic here is just logic and not the sort of thing that needs to be intimate with the command line.
Attachment #8729278 -
Flags: review?(terrence) → review+
Assignee | ||
Comment 13•8 years ago
|
||
Yeah, autospider.sh is well over the threshold of complexity. Although on the other hand, there are two things that argue for using bash instead of python: (1) it loads in some existing shell scripts (mozconfig files, actually) in order to get settings that work in automation; and (2) at least with my weak knowledge of python, it's actually easier to use shell script to implement the timeout logic. (Which is sad, because it's pretty darn ugly in the shell script version.) Still, the rest of it would be way easier in python, so I may as well switch it over before I sink too much more time into freshly-learned shell scripting craziness. There's a distinct lack of urgency here, because in order to land this stack it turns out I'm going to have to compile my own gdb for the automation system and get it installed/downloaded. So for now, I'll probably just put this on ice.
Assignee | ||
Comment 14•6 years ago
|
||
Lots of outdated cruft here. autospider has been rewritten in Python by now, and I think most of the functionality here probably already exists in it? It would require some archaeology to see what else is still needed.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•