Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ansible installer docs #1836

Merged
merged 3 commits into from
Aug 19, 2015
Merged

Ansible installer docs #1836

merged 3 commits into from
Aug 19, 2015

Conversation

arm4b
Copy link
Member

@arm4b arm4b commented Aug 18, 2015

Initial docs showing how to install StackStorm with Ansible, from ansible-st2 repo.

English syntax-check is welcome :)


The bad thing about .rst table, have no idea how to make multi-line description column, to avoid scrolling here:

@lakshmi-kannan
Copy link
Contributor

So is this tested? Better question, how are we going to validate this doesn't break after every release.

I don't mean to block your PR but we have a policy that anything that goes in the docs needs to be tested before every release and without automation it's hard to scale this.

@arm4b
Copy link
Member Author

arm4b commented Aug 19, 2015

I see.

Of course I've tested it manually many times (but still no automation here).

In that case I need further work on improvements to make it more stable, provide automatic tests. It's part of big checklist: StackStorm/ansible-st2#2 on which I'll work further.

But as I understand @jfryman wanted to see some initial docs ASAP.
So you guys decide if it's mergeable or Ansible installer needs more stability and tests. I'm ok with any decision and in any way will continue to work on ansible-st2.

@jfryman
Copy link
Contributor

jfryman commented Aug 19, 2015

@lakshmi-kannan I did give both @dennybaa and @armab direction to get some docs out there.

Can we exclude Config Management from this requirement for the time being? I want to give visibility to these installation methods, but we have some work to do before they'll be at this higher bar of quality and it's being worked on right now.

In terms of the actual plan, @armab has it outlined in his checklist.

@lakshmi-kannan
Copy link
Contributor

Yeah, if you already made a call to include it, I am fine. I just wanted to make sure we are thinking about testing etc. LGTM.

jfryman added a commit that referenced this pull request Aug 19, 2015
@jfryman jfryman merged commit a0d7d67 into StackStorm:master Aug 19, 2015
@arm4b arm4b deleted the ansible-installation-docs branch August 22, 2015 10:41
@cognifloyd cognifloyd removed the RFR label Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants