Skip to content

ec2 instance enumeration#3

Open
mike-bailey wants to merge 7 commits into
slack-ruby:masterfrom
mike-bailey:master
Open

ec2 instance enumeration#3
mike-bailey wants to merge 7 commits into
slack-ruby:masterfrom
mike-bailey:master

Conversation

@mike-bailey

Copy link
Copy Markdown
Contributor

Unfortunately literally the only proper command we use at work that's standardized. The rest are either hacky or dependent on things (i.e. @AWSbot ec2 run has an AMI hardcoded), etc.

@mike-bailey

mike-bailey commented Jul 29, 2017

Copy link
Copy Markdown
Contributor Author

yeah I kinda figured as such - please ignore me until cases passed

@mike-bailey

Copy link
Copy Markdown
Contributor Author

Please let me test this live again too before you merge - I noticed a major F up.

@mike-bailey

Copy link
Copy Markdown
Contributor Author

image
It works, verified, the faraday_middleware install is just buggy on build

@mike-bailey

Copy link
Copy Markdown
Contributor Author

It'll fail - it's a Travis CI problem, not a me problem. See my slack-google-bot PR for more.

@dblock

dblock commented Jul 29, 2017

Copy link
Copy Markdown
Collaborator

Thanks. I can fix Travis separately, this project hasn't been updated for a while. This needs tests, README and CHANGELOG updates either way, please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants