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

Add Ceph module #401

Closed
wants to merge 1 commit into from
Closed

Add Ceph module #401

wants to merge 1 commit into from

Conversation

pbrit
Copy link
Contributor

@pbrit pbrit commented Sep 5, 2016

A simple bunch of tests is added as well.

@raphink
Copy link
Member

raphink commented Sep 5, 2016

That looks good. Could you just add NaturalDocs documentation to the lens please? (see keepalived.aug for example).

@pbrit
Copy link
Contributor Author

pbrit commented Sep 5, 2016

Good point! I'll add the documentation soon. Thanks!

openstack-gerrit pushed a commit to openstack-archive/fuel-octane that referenced this pull request Sep 8, 2016
 - Graphs and puppet manifests part
 - Delete lib from .gitignore
 - Augeas lens for Ceph is copied and pasted, should be switched to the
   version from the upstream when [0] is merged and published.

[0] hercules-team/augeas#401

Change-Id: I639cbf786971fea8c56b4da6b2661477b3b12c41
@pspacek
Copy link

pspacek commented Mar 25, 2017

@PaulChe Could you add the documented lens to this PR? Thanks!

@lutter
Copy link
Member

lutter commented Sep 27, 2017

@raphink any objections to merging this as-is ? I'd rather have this in without docs if it works

@lutter
Copy link
Member

lutter commented Sep 29, 2017

Merged as 1a3b488

@lutter lutter closed this Sep 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants