HomeMesosphereNo notifications. 4 unresolved issues.

Remove Ruby dependencies from Jenkins node.
ClosedAll Users

Authored by jeschkies on Apr 24 2017, 4:42 PM.

Diff Detail

Repository
rMARATHON marathon
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
Changes from before your most recent comment are hidden. Show Older Changes
jeschkies updated this revision to Diff 2885.Apr 25 2017, 1:09 PM
jeschkies marked an inline comment as done.
  • Install jq and Ammonite.
jenkins requested changes to this revision.Apr 25 2017, 1:13 PM
This revision now requires changes to proceed.Apr 25 2017, 1:13 PM
aquamatthias accepted this revision.Apr 25 2017, 4:15 PM

Did you use c4.2xlarge on this new node type?

Yes. I configured Jenkins to use the new instance type and update the docs on the wiki.

Did you happen to add the instance labels too? Makes it way easier to see...

Also, if you could click "done" (I'm bad about it too) it'd help the review process.

ami/install.bash
61–67

The original code was just the curl line and didn't follow the redirect, etc. also, I did confirm that jq-1.5 is in the jessie backports so you could get it via apt-get.

This revision is now accepted and ready to land.Apr 25 2017, 4:25 PM
This revision was automatically updated to reflect the committed changes.
jeschkies marked an inline comment as done.Apr 25 2017, 4:45 PM