Skip to content
This repository has been archived by the owner on Jul 7, 2020. It is now read-only.

Add a streamlined deploy option through ansible #155

Closed
wants to merge 27 commits into from

Conversation

ravishivt
Copy link
Contributor

@ravishivt ravishivt commented Jan 14, 2017

This implements #149. It creates a new ansible/ directory with ansible playbooks and roles to deploy the project to a pre-existing cluster. The user needs to still specify the ansible inventory file and the topology.json file. The new ansible option does not supersede the existing manual deployment procedure or documentation. An initial README is added with details on what the playbooks and roles do and how to use them.


This change is Reviewable

… expected to create the deploy/files/topology.json file and create an inventory file for ansible. The deploy playbook will upgrade OS packages, install glusterfs-client, build heketi, run gk-deploy, and create the StorageClass.
…le. Ansible now has a GK_DIR variable that points to the project root deploy directory.
…onsistency. Removed unnecessary ansible groups in inventory.ini.sample. Added the inventory and topology to .gitignore.
…hould just copy both template directories and let gk-deploy do the auto-detection. Renamed destroy_vg.yaml to destroy_vgs.yaml.
…d on. Updated code references to use .yaml files instead of .yml.
@jarrpa
Copy link
Contributor

jarrpa commented Jan 18, 2017

I want to do a careful review of this and test it out myself, so it may take me a bit. But I like it thus far!

@jarrpa
Copy link
Contributor

jarrpa commented Jan 24, 2017

@ravishivt A bit of a non-update, since I don't want you to think I'm not thinking about this:

I've been prepping for a cross-continental trip to two conferences, so my apologies for the continued delay. I'll have plenty of time on planes and a week between the conferences to try and check this out, so I hope to get some comments back to you soon.

@ravishivt
Copy link
Contributor Author

@jarrpa No worries. Take your time and safe travels. One thing to note is that your ansible version must be < 2.2.x. Ansible 2.x and after has a bug that our playbook will hit.

@jarrpa
Copy link
Contributor

jarrpa commented Feb 21, 2017

@ravishivt Heyo. Coming in almost a month later to say that i still haven't been able to take this for a proper test run. :( My apologies. I've been hit with deadlines downstream, but we're hoping to clear them by next week.

I have, however, been able to at least read through most of it and offhand it looks pretty solid on the Debian side. I'm currently not planning to run the Debian side through its paces, mostly because I'd rather save the time of figuring out how to bring up an Ubuntu cluster (it's been long enough as it is!).

One thing to note is that with the release of heketi version 4 I plan on going off of that version for this project rather than tracking heketi master. That work will go in to the repo before this PR does, which means I'd like this playbook to build heketi v4 rather than master. I'll update this PR when that happens, and if I'm a decent human being maybe even provide patches. :)

Again, my apologies, we really do appreciate your contributions and don't like how long we've been letting this sit idle. Hope to bring some good news soon.

@jarrpa
Copy link
Contributor

jarrpa commented Mar 6, 2017

@ravishivt All right, I've taken a crack at this and have some questions/comments:

  1. Is there a reason I'm not seeing as to why the bootstrap and gk-prep roles couldn't be merged?
  2. We should be able to specify cluster nodes and devices in the inventory.ini file and just generate a topology file.
  3. I know that for yum at least if you specify a package that is already installed it will still succeed, just with a warning. Is apt-get similar? If so we don't need to check for python already existing.
  4. heketi 4 has been released, so we should build that version

@goern
Copy link

goern commented Mar 18, 2017

I wonder if you could provide a heketi-cli container as I cant build it on an Fedora Atomic Host

@obnoxxx
Copy link
Contributor

obnoxxx commented Mar 26, 2017

@goern how does that request/question relate to this PR?

It would certainly be possible. up to now we've never needed to build a specific cli container.

@jarrpa
Copy link
Contributor

jarrpa commented Apr 26, 2017

Closing this PR. It can be reopened if needed.

@jarrpa jarrpa closed this Apr 26, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants