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`.

Still keep it as an `After` though; *if* something pulls in
`network-online.target`, then we might as well be nicer and start our
retries after we know we're online. Also add `network.target` so that
if `network-online.target` *isn't* pulled in, we still have a reasonable
lower bound on when we start retrying.

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 6, 2023
1 parent 8068d13 commit bad17e3
Show file tree
Hide file tree
Showing 2 changed files with 17 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
3 changes: 1 addition & 2 deletions systemd/coreos-installer.service
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
[Unit]
Description=CoreOS Installer
Before=coreos-installer.target
After=network-online.target
Wants=network-online.target
After=network.target network-online.target
# 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 bad17e3

Please sign in to comment.