search

taskcluster / ec2-manager

Mozilla Public License 2.0
2 stars 14 forks source link

August updates #69

Closed jhford closed 6 years ago

jhford commented 6 years ago

This should be everything needed to support root url in ec2-manager. I'm not totally certain about the lib-monitor stuff, so I'd appreciate a once-over on that part of the code.

jhford commented 6 years ago

Note that I'm using this PR to debug some travis issues

imbstack commented 6 years ago

new tasklcluster-lib-testing should have the fix you need!

imbstack commented 6 years ago

The lib-monitor stuff looks correct so far from what I've seen.

jhford commented 6 years ago

Brian, I'm still sorting out the travis tests (their postgres setup is... complicated), but the tests pass locally. Could you take another look when you have a chance?

jhford commented 6 years ago

I have no clue why these tests are failing in travis... the exact same tests work perfectly locally.

jhford commented 6 years ago

I've filed #70 for the authorize() calls, so that we don't forget about it, but not block a release of ec2-manager.

jhford commented 6 years ago

a60d226..a42d707 master -> master