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 Viaq installation guide to the oVirt metrics store repo #1551
Conversation
@richm Whould appreciate your review of my change. |
------------------------ | ||
|
||
These instructions and config files are for an all-in-one, single machine, run | ||
ansible on the same machine you are installing ViaQ on. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the same as the next paragraph. You can remove it.
e.g. `chgrp 65534 /var/lib/elasticsearch` | ||
- make this directory writable by the group `chmod -R g+w /var/lib/elasticsearch` | ||
|
||
# chgrp 65534 /var/lib/elasticsearch` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you need a blank line after this line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sradco
I think you are missing the headers, like these for example:
title: oVirt Metrics Store - Installation Guide
category: feature
authors: sradco
feature_name: oVirt Metrics Store Installation
feature_modules: engine
feature_status: In Development
Please add them, and we will see if the CI pass
Add Viaq installation guide to the oVirt metrics store repo
This adds the viaq installation to the oVirt repo, So we can remove the steps that were already automated.
I confirm that this pull request was submitted according to the contribution guidelines: @sradco
This pull request needs review by: @mykaul