OSDOCS CQA MCO-1: MCO Core Configuration and Customization III - #113918
Conversation
|
@mburke5678: This pull request references MCO-1 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the epic to target either version "5.0." or "openshift-5.0.", but it targets "openshift-4.15" instead. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
🤖 Thu Jul 02 16:02:47 - Prow CI generated the docs preview: https://113918--ocpdocs-pr.netlify.app/ |
lahinson
left a comment
There was a problem hiding this comment.
@mburke5678 Nice work! Just a few comments for your consideration. Feel free to merge when you are ready.
| ==== | ||
| + | ||
| Hotfixes are provided to you based on link:https://access.redhat.com/solutions/2996001[Red Hat Hotfix policy]. Apply it on top of the base image and test that new custom layered image in a non-production environment. When you are satisfied that the custom layered image is safe to use in production, you can roll it out on your own schedule to specific node pools. For any reason, you can easily roll back the custom layered image and return to using the default {op-system}. | ||
| Hotfixes are provided to you based on Red Hat Hotfix policy. Apply it on top of the base image and test that new custom layered image in a non-production environment. When you are satisfied that the custom layered image is safe to use in production, you can roll it out on your own schedule to specific node pools. For any reason, you can easily roll back the custom layered image and return to using the default {op-system}. |
There was a problem hiding this comment.
| Hotfixes are provided to you based on Red Hat Hotfix policy. Apply it on top of the base image and test that new custom layered image in a non-production environment. When you are satisfied that the custom layered image is safe to use in production, you can roll it out on your own schedule to specific node pools. For any reason, you can easily roll back the custom layered image and return to using the default {op-system}. | |
| Hotfixes are provided to you based on Red{nbsp}Hat Hotfix policy. Apply it on top of the base image and test that new custom layered image in a non-production environment. When you are satisfied that the custom layered image is safe to use in production, you can roll it out on your own schedule to specific node pools. For any reason, you can easily roll back the custom layered image and return to using the default {op-system}. |
SSG: https://redhat-documentation.github.io/supplementary-style-guide/#non-breaking-spaces
| // https://issues.redhat.com/browse/OCPBUGS-42838 | ||
|
|
||
| * *{op-system-base} packages*. You can download {op-system-base-full} packages from the link:https://access.redhat.com/downloads/content/479/ver=/rhel---9/9.1/x86_64/packages[Red Hat Customer Portal], such as chrony, firewalld, and iputils. | ||
| * *{op-system-base} packages*. You can download {op-system-base-full} packages from the Red Hat Customer Portal, such as chrony, firewalld, and iputils. |
There was a problem hiding this comment.
| * *{op-system-base} packages*. You can download {op-system-base-full} packages from the Red Hat Customer Portal, such as chrony, firewalld, and iputils. | |
| * *{op-system-base} packages*. You can download {op-system-base-full} packages from the Red{nbsp}Hat Customer Portal, such as chrony, firewalld, and iputils. |
| containerFile: # <4> | ||
| - containerfileArch: NoArch <5> | ||
| name: layered-image | ||
| containerFile: # |
There was a problem hiding this comment.
| containerFile: # | |
| containerFile: |
On this line and in a few other places in this example, I think you can remove the #.
| ---- | ||
| $ oc describe machineconfignode/<machine_config_node_name> | ||
| ---- | ||
| + |
There was a problem hiding this comment.
I'm seeing a callout in the preview. I think it's coming from the snippet on line 199.
| <4> Optional: Update the secret needed to push the newly built custom layered image to the registry. | ||
| + | ||
| where: | ||
| + |
There was a problem hiding this comment.
If you're formatting the user-replaced variable text as a definition list, the SSG says to introduce definition lists with "where:" and begin each variable description with "Specifies".
There was a problem hiding this comment.
@lahinson This rule was recently changed to drop the requirement for Specifies.
|
|
||
| If you modify and apply the `MachineOSConfig` object YAML or create a new YAML file, the YAML overwrites any changes you made directly to the `MachineOSConfig` object itself. | ||
|
|
||
| include::snippets//coreos-layering-configuring-on-pause.adoc[] |
There was a problem hiding this comment.
Check the snippet. I see a callout in the preview, and I'm guessing it's coming from there.
| + | ||
| Add the `machineconfiguration.openshift.io/rebuild: ""` annotation to trigger a rebuild of the custom layered image. | ||
|
|
||
| .Verification |
There was a problem hiding this comment.
Should the verification steps be in an ordered list? Just wondering if they should be completed in a specific order.
There was a problem hiding this comment.
No. You can do them in any order or skip one of the two.
| :_mod-docs-content-type: PROCEDURE | ||
| :_mod-docs-content-type: CONCEPT | ||
| [id="coreos-layering-configuring-on-remove_{context}"] | ||
| = Removing an on-cluster custom layered image |
There was a problem hiding this comment.
Concept module titles should be noun phrases.
| = Removing an on-cluster custom layered image | |
| = Removal of an on-cluster custom layered image |
| ---- | ||
| sh-4.4# chroot /host | ||
| ---- | ||
|
|
There was a problem hiding this comment.
If you add this plus sign, I think it'll fix the formatting issues that are in the preview.
|
@mburke5678: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
/cherrypick enterprise-4.20 |
|
/cherrypick enterprise-4.21 |
|
/cherrypick enterprise-4.22 |
|
/cherrypick enterprise-5.0 |
|
@mburke5678: #113918 failed to apply on top of branch "enterprise-4.20": DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@mburke5678: new pull request created: #114553 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@mburke5678: new pull request created: #114554 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@mburke5678: new pull request created: #114555 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
https://redhat.atlassian.net/browse/OSDOCS-16921
Previews:
Machine Confguration -> Image mode for OpenShift
Assemblies:
mco-coreos-layering.adoc