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

Use containerized heketi with non-containerized GlusterFS nodes #281

Merged
merged 3 commits into from Aug 11, 2017

Conversation

Projects
None yet
2 participants
@jarrpa
Contributor

jarrpa commented Jul 19, 2017

This PR intends to allow gk-deploy to deploy a solution where heketi remains containerized and manages a set of non-containerized GlusterFS nodes. In its most basic form, this new functionality can be achieved by running:

gk-deploy -s <keyfile>

Where <keyfile> is a path to an SSH private key that can be used to get acces to either root or a sudoable account on the GlusterFS nodes.


This change is Reviewable

@obnoxxx

This comment has been minimized.

Show comment
Hide comment
@obnoxxx

obnoxxx Jul 26, 2017

Contributor

@jarrpa travis-ci failed. Could you please investigate?

Contributor

obnoxxx commented Jul 26, 2017

@jarrpa travis-ci failed. Could you please investigate?

@obnoxxx

This comment has been minimized.

Show comment
Hide comment
@obnoxxx

obnoxxx Aug 1, 2017

Contributor

@jarrpa , I am planning to do final review tomorrow

Contributor

obnoxxx commented Aug 1, 2017

@jarrpa , I am planning to do final review tomorrow

jarrpa added some commits Jul 19, 2017

gk-deploy: add eval command to debug output.
Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
gk-deploy: apply config parameters to kube templates.
Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
gk-deploy: allow for external (SSH) GlusterFS nodes.
Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
@obnoxxx

This comment has been minimized.

Show comment
Hide comment
@obnoxxx

obnoxxx Aug 11, 2017

Contributor

LGTM

Contributor

obnoxxx commented Aug 11, 2017

LGTM

@obnoxxx obnoxxx merged commit 638b1ab into gluster:master Aug 11, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jarrpa jarrpa deleted the jarrpa:heketi-ha branch Aug 13, 2017

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