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
RHCOS4: Enhance instructions to better reflect how to work with the platform #6796
RHCOS4: Enhance instructions to better reflect how to work with the platform #6796
Conversation
Changes identified: Show detailsMacro systemd_ocil_service_disabled: |
@yuumasato @ggbecker could you take a look at this? I'm starting to change instructions for RHCOS4 to better guide users reading the content. this is a bit bigger change since I'm modifying the macros, and before going forward, I want to know what you folks think of this approach. |
/test all |
setting, create a <tt>MachineConfig</tt> similar to the following: | ||
<pre> | ||
apiVersion: machineconfiguration.openshift.io/v1 | ||
kind: MachineConfig | ||
metadata: | ||
labels: | ||
machineconfiguration.openshift.io/role: master | ||
name: 75-master-disable-ctrlaltdel-burstaction | ||
spec: | ||
config: | ||
ignition: | ||
version: 3.1.0 | ||
storage: | ||
files: | ||
- contents: | ||
source: data:,CtrlAltDelBurstAction%3Dnone | ||
mode: 0644 | ||
path: /etc/systemd/system.conf.d/disable_ctrlaltdelete_burstaction.conf | ||
overwrite: true | ||
EOF |
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'll want to add similar text to the OCIL as well.
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.
Yeah, that makes sense. I'll get to that.
{{% if product != "rhcos4" %}} | ||
setting, add or modify the following to <tt>/etc/systemd/system.conf</tt>: | ||
<pre>CtrlAltDelBurstAction=none</pre> | ||
{{% else %}} | ||
setting, create a <tt>MachineConfig</tt> similar to the following: | ||
<pre> | ||
apiVersion: machineconfiguration.openshift.io/v1 | ||
kind: MachineConfig | ||
metadata: | ||
labels: | ||
machineconfiguration.openshift.io/role: master | ||
name: 75-master-disable-ctrlaltdel-burstaction | ||
spec: | ||
config: | ||
ignition: | ||
version: 3.1.0 | ||
storage: | ||
files: | ||
- contents: | ||
source: data:,CtrlAltDelBurstAction%3Dnone | ||
mode: 0644 | ||
path: /etc/systemd/system.conf.d/disable_ctrlaltdelete_burstaction.conf | ||
overwrite: true | ||
EOF | ||
</pre> | ||
<p> | ||
This will add the relevant configuration to <tt>/etc/systemd/system.conf.d/</tt>, | ||
thus configuring Systemd apropriately. | ||
</p> | ||
{{{ machineconfig_description_footer() | indent(4) }}} | ||
{{% endif %}} |
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.
Side comment: The Jinja conditionals in between the text make things tricky to read...
One way to increase readability, would be to make it possible to split the description in two parts, one which pertains to description of the setting and its behavior, and another about product specific configuration steps. For example, description
and configuration
.
Another approach would be to allow entire product specific descriptions, e.g. description@rhcos4
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.
maybe the former approach is better, (having description@<product>
@yuumasato, thanks for the review! I'll continue adding changes then! I'll target the ocil macros and service_disabled next. |
Pull-request updated, HEAD is now 8636b28 |
3bdbd44
to
8636b28
Compare
@yuumasato added changes to both descriptions and ocil, WDYT? |
/retest |
This makes it easier for folks to remediate themselves. Signed-off-by: Juan Antonio Osorio Robles <jaosorior@redhat.com>
This makes it easier for folks to remediate themselves. Signed-off-by: Juan Antonio Osorio Robles <jaosorior@redhat.com>
This makes it easier for folks to remediate themselves. Signed-off-by: Juan Antonio Osorio Robles <jaosorior@redhat.com>
This reduces the duplication in the content. Signed-off-by: Juan Antonio Osorio Robles <jaosorior@redhat.com>
This adds an alternative description for this macro in case of rhcos4 to better reflect what an admin would actually need to do. Signed-off-by: Juan Antonio Osorio Robles <jaosorior@redhat.com>
This adds an alternative description for this macro in case of rhcos4 to better reflect what an admin would actually need to do. Signed-off-by: Juan Antonio Osorio Robles <jaosorior@redhat.com>
…d) macros This ensures admins have a better notion of what to do. Signed-off-by: Juan Antonio Osorio Robles <jaosorior@redhat.com>
8636b28
to
4ffa281
Compare
/retest |
one nit, looks good to me otherwise |
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.
LGTM
Signed-off-by: Juan Antonio Osorio Robles <jaosorior@redhat.com>
d6e9255
to
99b8f66
Compare
/retest |
This changes various descriptions and macros in the case of rhcos4 to better
reflect what an admin would really need to do to check or remediate.