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

Beta docs restructuring c tl p lane services pt3 #402

Conversation

klgill
Copy link
Contributor

@klgill klgill commented Apr 10, 2024

Last batch of control plane services files.

Peer reviews, please use the checklist to stay in scope: https://docs.google.com/document/d/1hxKJdhGNqSOTg6WrbYjJaQIWDu7x5cl2QRYJ7G0HFt8/edit#heading=h.o8ezt27iw7kl

@jistr
Copy link
Contributor

jistr commented Apr 11, 2024

LGTM

@klgill
Copy link
Contributor Author

klgill commented Apr 11, 2024

LGTM

FYI I'm going to push more changes today, and then the whole "control plane" chapter will be ready for review.

@klgill klgill force-pushed the BetaDocs-RestructuringCTlPLaneServicesPt3 branch from 0b33826 to f115653 Compare April 11, 2024 17:45
@klgill klgill marked this pull request as ready for review April 11, 2024 17:51
@klgill klgill requested review from jistr, fao89, matbu and hjensas April 11, 2024 17:51
@klgill
Copy link
Contributor Author

klgill commented Apr 11, 2024

@matbu Can you please review the new structure of the following os-diff files?

  • docs_user/assemblies/assembly_reviewing-the-openstack-control-plane-configuration.adoc
  • docs_user/modules/proc_pulling-configuration-from-a-tripleo-deployment.adoc
  • docs_user/modules/proc_retrieving-services-topology-specific-configuration.adoc

@hjensas Can you please review the new structure if the following ironic files?

  • docs_user/assemblies/assembly_adopting-the-bare-metal-provisioning-service.adoc
  • docs_user/modules/con_bare-metal-provisioning-service-configurations.adoc
  • docs_user/modules/deploying-the-bare-metal-provisioning-service.adoc

@klgill klgill requested a review from fmount April 11, 2024 17:55
@klgill
Copy link
Contributor Author

klgill commented Apr 11, 2024

@fmount Can you please review the new structure of the "Shared File System" files?

  • docs_user/assemblies/assembly_adopting-the-shared-file-systems-service.adoc
  • docs_user/modules/con_changes-to-cephFS-via-NFS.adoc
  • docs_user/modules/proc_deploying-file-systems-service-control-plane.adoc
  • docs_user/modules/proc_decommissioning-rhosp-standalone-ceph-NFS-service.adoc

@fmount
Copy link
Contributor

fmount commented Apr 11, 2024

@klgill +2 on the Manila front, feel free to merge when you get all the reviews in place!

Before starting the adoption workflow, pull the configuration from the {rhos_prev_long} services and {OpenStackPreviousInstaller} on your file system to back up the configuration files. You can then use the files later, during the configuration of the adopted services, and for the record to compare and make sure nothing has been missed or misconfigured.

Make sure you have pull the os-diff repository and configure according to your environment:
link:planning.md#Configuration tooling[Configure os-diff]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this need to be commented out?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matbu Should this "planning.md" link be included in the downstream docs?
And should this link be in the downstream docs? https://github.com/openstack-k8s-operators/os-diff/blob/main/config.yaml

CC: @jistr

@klgill klgill force-pushed the BetaDocs-RestructuringCTlPLaneServicesPt3 branch from f115653 to 88a8a7d Compare April 15, 2024 19:14
Copy link
Contributor

@hjensas hjensas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. +1

I have one comment on removing one sentance, but that was not added by this change. So I am fine doing that separately.

@klgill klgill force-pushed the BetaDocs-RestructuringCTlPLaneServicesPt3 branch from 88a8a7d to 68e2a10 Compare April 17, 2024 19:22
@klgill klgill merged commit fac4803 into openstack-k8s-operators:main Apr 17, 2024
3 checks passed
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.

6 participants