Skip to content

Conversation

@AndreiCherniaev
Copy link
Contributor

@AndreiCherniaev AndreiCherniaev commented Dec 9, 2025

Proposed Commit Message

wget Removed from Ubuntu Server 25.10 Default Install

Additional Context

Test Steps

Merge type

  • Squash merge using "Proposed Commit Message"
  • Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

@github-actions github-actions bot added the documentation This Pull Request changes documentation label Dec 9, 2025
Copy link
Collaborator

@blackboxsw blackboxsw left a comment

Choose a reason for hiding this comment

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

Thank you for this submission and updating docs.

Note that most of the curl replacements suggested in docs are not valid curl calls, please update and test them on a running system to make sure we are documenting functional equivalents without errors before we land it.

I'm curious which images streams you are familiar with where wget does not exist:
Looking at desktop manifest it still has wget in the image, same with ubuntu server images and ubuntu minimal images because it's pulled in by other image dependencies like pciutils which exist in most images.

# Early boot environments can race systemd-tmpfiles-clean LP: #1707222.
- mkdir /run/mydir
- [ wget, "http://slashdot.org", -O, /run/mydir/index.html ]
- [ curl, "http://slashdot.org", -O, /run/mydir/index.html ]
Copy link
Collaborator

Choose a reason for hiding this comment

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

These are not valid curl commands. We should update them to commands that can run without error if we want to land this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My fail, see update please

@blackboxsw blackboxsw added the incomplete Action required by submitter label Dec 10, 2025
wget Removed from Ubuntu Server 25.10 Default Install
wget Removed from Ubuntu Server 25.10 Default Install
'test_nocloud.py' fixed by instructions from https://github.com/FiloSottile/mkcert
@AndreiCherniaev
Copy link
Contributor Author

AndreiCherniaev commented Dec 11, 2025

I'm curious which images streams you are familiar with where wget does not exist:
Looking at desktop manifest it still has wget in the image, same with ubuntu server images and ubuntu minimal images because it's pulled in by other image dependencies like pciutils which exist in most images.

"Ubuntu developers have decided to remove wget from the default install of Ubuntu Server 25.10, in favour of wcurl, which offers similar functionality." - from link
So I decide to help Canonical to drop wget totally.

@github-actions
Copy link

Hello! Thank you for this proposed change to cloud-init. This pull request is now marked as stale as it has not seen any activity in 14 days. If no activity occurs within the next 7 days, this pull request will automatically close.

If you are waiting for code review and you are seeing this message, apologies! Please reply, tagging blackboxsw, and he will ensure that someone takes a look soon.

(If the pull request is closed and you would like to continue working on it, please do tag blackboxsw to reopen it.)

@github-actions github-actions bot added the stale-pr Pull request is stale; will be auto-closed soon label Dec 26, 2025
@AndreiCherniaev
Copy link
Contributor Author

@blackboxsw I don't know why the bot plans to close PR soon... But let's use curl instead of wget

@github-actions github-actions bot removed the stale-pr Pull request is stale; will be auto-closed soon label Dec 27, 2025
Copy link
Member

@holmanb holmanb left a comment

Choose a reason for hiding this comment

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

Thanks @AndreiCherniaev, this looks good to me.

@holmanb holmanb merged commit 85c4638 into canonical:main Jan 5, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation This Pull Request changes documentation incomplete Action required by submitter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants