search

StackStorm / showcase-ansible-chatops

Vagrant Demo showing ChatOps with Ansible
https://stackstorm.com/2015/06/24/ansible-chatops-get-started-%f0%9f%9a%80/
Apache License 2.0
77 stars 22 forks source link

Update to new packages #5

Closed arm4b closed 8 years ago

arm4b commented 8 years ago

Since old installing method is deprecated, update demo to use new bundled st2 package from https://docs.stackstorm.com/install/deb.html

LindsayHill commented 8 years ago

I've started making some changes for this, similar to these changes https://github.com/StackStorm/st2-ansible-vagrant/commit/f8d62e7e92f43822408c1c78efe7027085e90fd3

Should get it tidied up soon. Will probably include changes required for #6

arm4b commented 8 years ago

Thanks for saying @LindsayHill, since I'm currently playing with this repo locally too.

But my thinking was to avoid using new install script https://stackstorm.com/packages/install.sh, because it's not idempotent and I'm not fan of installing additional st2mistral + postgresql dependencies when it's not needed (for this showcase).

However I'm OK with your approach, go for it :+1: It would be probably simpler for user and easier to support in future.

I'll play with #6 ChatOps Aliases then (not in this repo), because it was something I totally missed to try when we worked on new packages.

WDYT?

LindsayHill commented 8 years ago

@armab Take a look at this branch https://github.com/StackStorm/showcase-ansible-chatops/tree/migrate_to_packages

This uses the new packages method. I've done basic testing, and it looks OK so far. I haven't looked at the aliases at all. Can you give it a quick test?

arm4b commented 8 years ago

@LindsayHill Nice! Just open PR.

So far old aliases won't break anything, but I'll work on improvements and include them in next PR.