-
Notifications
You must be signed in to change notification settings - Fork 20
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
Doc from openstack admin guide #1083
base: stackhpc/2023.1
Are you sure you want to change the base?
Conversation
90a9b23
to
ac2cafb
Compare
- ``cephadm-keys.yml`` - defines Ceph users/keys | ||
- ``cephadm-pools.yml`` - defines Ceph pools\ | ||
|
||
Running Ceph commands |
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 we should get someone with more Ceph experience to review the rest of this file to check it for accuracy.
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.
Still required
ba4f72f
to
6b2e8d6
Compare
6b2e8d6
to
34f41b6
Compare
bac5a3a
to
ee1ca6c
Compare
71b9425
to
5eb89ce
Compare
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.
Nice work, quite close now. Mainly need to update the vgpu page and get someone with more ceph experince to review the ceph page
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.
Also, the docs job is failing
8f58fa8
to
ab64a55
Compare
ab64a55
to
68818b5
Compare
doc/source/operations/bifrost-hardware-inventory-management.rst
Outdated
Show resolved
Hide resolved
doc/source/operations/bifrost-hardware-inventory-management.rst
Outdated
Show resolved
Hide resolved
doc/source/operations/bifrost-hardware-inventory-management.rst
Outdated
Show resolved
Hide resolved
- ``cephadm-keys.yml`` - defines Ceph users/keys | ||
- ``cephadm-pools.yml`` - defines Ceph pools\ | ||
|
||
Running Ceph commands |
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.
Still required
Too many commits at this point. Let's squash this when merging |
LGTM but someone with ceph knowledge should review the ceph page before we merge |
Moved general operation documentations from Openstack admin guide.
Removed customer specific templatings and tried to generalise.
Updated old contents
Please let me know if there are more contents that need to be updated.
Related PR on Openstack Admin Guide stackhpc/openstack-admin-guide#53