Skip to content

Commit

Permalink
install: Don't require the network by default
Browse files Browse the repository at this point in the history
The `Wants=network-online.target` predates the osmet work which enabled
the now default fully offline install flow.

It also predates the addition of fetch retries. So even in an online
flow, now that we retry HTTP requests indefinitely, we don't really
need this.

Let's follow best practices and stop pulling in `network-online.target`.

We still need to keep pulling in `NetworkManager.service`
though. It's enabled by default in `multi-user.target` but not
`coreos-installer-post.target` (which is what we boot to in an automated
install).

NetworkManager is capable of handling offline environments just fine and
won't block the install just because a connection isn't available (yet
or ever).

Related: coreos/fedora-coreos-config#1088
Related: coreos#565 (comment)
Related: https://www.freedesktop.org/wiki/Software/systemd/NetworkTarget/
Closes: https://github.com/coreos/coreos-installer/issues/1334
  • Loading branch information
jlebon committed Nov 9, 2023
1 parent 8068d13 commit e6bb2e1
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 2 deletions.
16 changes: 16 additions & 0 deletions docs/release-notes.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,22 @@ nav_order: 8

## Upcoming coreos-installer 0.19.0 (unreleased)

Major changes:



Minor changes:

- install: Don't require the network by default

Internal changes:



Packaging changes:




## coreos-installer 0.18.0 (2023-08-24)

Expand Down
4 changes: 2 additions & 2 deletions systemd/coreos-installer.service
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
[Unit]
Description=CoreOS Installer
Before=coreos-installer.target
After=network-online.target
Wants=network-online.target
After=NetworkManager.service
Wants=NetworkManager.service
# Until we retry HTTP requests let's wait here until
# systemd-resolved comes up if enabled.
# https://github.com/coreos/coreos-installer/issues/283
Expand Down

0 comments on commit e6bb2e1

Please sign in to comment.