Skip to content

Updated step 6 in 4.1 Creating an Ethernet connection to fix DOCTEAM 2148#700

Open
harneshalaka wants to merge 3 commits into
mainfrom
sharne/DOCTEAM2148
Open

Updated step 6 in 4.1 Creating an Ethernet connection to fix DOCTEAM 2148#700
harneshalaka wants to merge 3 commits into
mainfrom
sharne/DOCTEAM2148

Conversation

@harneshalaka

Copy link
Copy Markdown
Contributor

PR creator: Description

Updated step 6 in 4.1 Creating an Ethernet connection to fix DOCTEAM 2148

PR creator: Are there any relevant issues/feature requests?

PR reviewer: Checklist for editorial review

Apart from the usual checks, please double-check also the following:

@grisu48 grisu48 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good overall work! The wording is IMHO clear and concise. I think there is one minor content error, which needs to be addressed before merge.

Restart=always</screen>
</listitem>
<listitem>
<para>Centralized Non-Local Binding (Alternative): Instead of managing overrides per service, you can allow applications to safely bind to IPv4 and IPv6 addresses that are not yet fully initialized on the system. Create a custom sysctl file at <filename>/etc/sysctl.d/90-nonlocal_bind.conf</filename> containing the following lines:</para>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This method does not require NetworkManager-wait-online.service but acts as an alternative method one step higher.

So this means, there are two methods to achieve the goal to prevent services from failing or experiencing race conditions:

  1. NetworkManager-wait-online.service + Individual systemd unit overrides
  2. The nonlocal_bind setting in sysctl.

Currently I read the document as it would suggest that NetworkManager-wait-online.service is required for both methods. It is not required for the nonlocal_bind setting.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Resolved in the current state.

@grisu48 grisu48 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@lvicoun lvicoun requested a review from dariavladykina June 11, 2026 14:33

@dariavladykina dariavladykina left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hi, a suggestion for this one. Thanks!

<para>Configure the connection profile.</para>
<para>The generic command syntax is the following:</para>
<screen>&prompt.sudo;nmcli connection modify <replaceable>CONNECTION_NAME</replaceable> <replaceable>SETTING</replaceable> <replaceable>VALUE</replaceable></screen>
<para>For example, automatic IP addresses assignment (DHCP or SLAAC) is enabled by default, and you may want to configure a static IP address:</para>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
<para>For example, automatic IP addresses assignment (DHCP or SLAAC) is enabled by default, and you may want to configure a static IP address:</para>
<para>For example, automatic IP address assignment through DHCP or SLAAC is enabled by default. You may want to configure a static IP address:</para>

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.

4 participants