-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move more Travis jobs to TaskCluster #15724
Conversation
db2df52
to
ff1ce2d
Compare
e254dae
to
f624961
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have one or two concerns related to how familiar people are with TC's UI, and it being non-obvious in places, versus familiarity with Travis, but in almost every other way this seems like a great move!
I think the TC UI problems will be largely fixed when we are able to turn on the checks api. As far as I know that's still blocked on wpt.fyi, but @Hexcles may know more. Also they are redesigning the UI and I opened an issue about making the logs more prominent. @foolip Any concerns about merging this? |
I intend to merge this tomorrow in the absence of feedback to the contrary. |
That's OK with me! |
@jgraham is there an open issue or PR about enabling the checks integration for Taskcluster again? web-platform-tests/wpt.fyi#1182 is resolved now. @Hexcles, has that been deployed to prod as well? Can we confirm it working in prod before flipping the switch again? |
#15946 to reenable the checks API. |
Affirmative. And according to web-platform-tests/wpt.fyi#1183 (comment) , it's working. |
Without the comma, the string literals are concatenated. Spotted via `sudo earlyoom -p -r 60--prefer=(chrome|firefox) --avoid=python` with a missing space in Taskcluster logs. This was introduced in #15724.
def start_userspace_oom_killer(): | ||
# Start userspace OOM killer: https://github.com/rfjakob/earlyoom | ||
# It will report memory usage every minute and prefer to kill browsers. | ||
start(["sudo", "earlyoom", "-p", "-r", "60" "--prefer=(chrome|firefox)", "--avoid=python"]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a missing comma here, sent #18630 to fix that.
Without the comma, the string literals are concatenated. Spotted via `sudo earlyoom -p -r 60--prefer=(chrome|firefox) --avoid=python` with a missing space in Taskcluster logs. This was introduced in #15724.
Without the comma, the string literals are concatenated. Spotted via `sudo earlyoom -p -r 60--prefer=(chrome|firefox) --avoid=python` with a missing space in Taskcluster logs. This was introduced in web-platform-tests#15724.
…ist, a=testonly Automatic update from web-platform-tests Add missing comma in earlyoom argument list (#18630) Without the comma, the string literals are concatenated. Spotted via `sudo earlyoom -p -r 60--prefer=(chrome|firefox) --avoid=python` with a missing space in Taskcluster logs. This was introduced in web-platform-tests/wpt#15724. -- wpt-commits: b3b48132272f9a92414ce879bd8a5ab468879b8f wpt-pr: 18630
…ist, a=testonly Automatic update from web-platform-tests Add missing comma in earlyoom argument list (#18630) Without the comma, the string literals are concatenated. Spotted via `sudo earlyoom -p -r 60--prefer=(chrome|firefox) --avoid=python` with a missing space in Taskcluster logs. This was introduced in web-platform-tests/wpt#15724. -- wpt-commits: b3b48132272f9a92414ce879bd8a5ab468879b8f wpt-pr: 18630
…ist, a=testonly Automatic update from web-platform-tests Add missing comma in earlyoom argument list (#18630) Without the comma, the string literals are concatenated. Spotted via `sudo earlyoom -p -r 60--prefer=(chrome|firefox) --avoid=python` with a missing space in Taskcluster logs. This was introduced in web-platform-tests/wpt#15724. -- wpt-commits: b3b48132272f9a92414ce879bd8a5ab468879b8f wpt-pr: 18630 UltraBlame original commit: 3842950c1e45d64e4ba8c131a589d209e1619dad
…ist, a=testonly Automatic update from web-platform-tests Add missing comma in earlyoom argument list (#18630) Without the comma, the string literals are concatenated. Spotted via `sudo earlyoom -p -r 60--prefer=(chrome|firefox) --avoid=python` with a missing space in Taskcluster logs. This was introduced in web-platform-tests/wpt#15724. -- wpt-commits: b3b48132272f9a92414ce879bd8a5ab468879b8f wpt-pr: 18630 UltraBlame original commit: 3842950c1e45d64e4ba8c131a589d209e1619dad
…ist, a=testonly Automatic update from web-platform-tests Add missing comma in earlyoom argument list (#18630) Without the comma, the string literals are concatenated. Spotted via `sudo earlyoom -p -r 60--prefer=(chrome|firefox) --avoid=python` with a missing space in Taskcluster logs. This was introduced in web-platform-tests/wpt#15724. -- wpt-commits: b3b48132272f9a92414ce879bd8a5ab468879b8f wpt-pr: 18630 UltraBlame original commit: 3842950c1e45d64e4ba8c131a589d209e1619dad
tc-jobs: all