Add commando.io service #945

Merged
merged 2 commits into from Oct 25, 2014

Projects

None yet

2 participants

@aw
Contributor
aw commented Oct 9, 2014

Hi,

This PR is for adding the official commando.io service integration ๐Ÿ’ก

โ„น๏ธ Summary:

Commando.io is a simpler way to manage servers online. Commando.io makes it simple to execute commands on groups of servers from a beautiful web interface.

With this service integration, GitHub/Commando.io users can run executions from a simple git push to their :octocat: repository.

I'm the official maintainer ๐Ÿ’‚ so let me know if you have any โ‰๏ธ or require any changes ๐Ÿ˜ธ

Thanks!

@kdaigle
Member
kdaigle commented Oct 9, 2014

@aw thanks! Is it possible that you'd be willing to have a specific endpoint for GitHub related webhooks? We're really trying to limit services to only be those that are unique communication protocols that simply cannot work via an HTTP POST. It looks like this service is attempting to map to an existing API.

Would it be possible to have your service install a webhook via an OAuth application? ๐Ÿฐ

@aw
Contributor
aw commented Oct 9, 2014

Ahh yeah I know, and that was the initial plan, but we realized it requires a pretty big rewrite of the API.

  • Executions based on a :octocat: push have a unique URL for each recipe, ex: https://api.commando.io/v1/recipes/#{data['recipe']}/execute
  • The API doesn't accept JSON payloads, but rather www-form requests
  • Form POST parameters must be sent a specific way (such as groups which is a comma-separated string, but must arrive as separate groups[] values -- similar to html multi select form)
  • Certain values must be boolean and others string
  • There's thousands of users on the service, so we can't break the API. Perhaps in v2 API we can remove the service from GH but for now it's the only way to let the users push directly to GitHub and run their executions.
@aw
Contributor
aw commented Oct 25, 2014

Hi,

Can I have an update on this? Thanks! :octocat:

@kdaigle
Member
kdaigle commented Oct 25, 2014

@aw sorry for the delay on this! This will get pushed to production in the next few days. ๐Ÿฐ

@kdaigle kdaigle merged commit faa5859 into github:master Oct 25, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@aw
Contributor
aw commented Oct 25, 2014

โœจ Thank You!! ๐Ÿ‘

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