Skip to content

Fix URL scheme in deprecation message#6565

Merged
blackboxsw merged 1 commit into
canonical:mainfrom
kikitux:fix-cocloud-http
Nov 11, 2025
Merged

Fix URL scheme in deprecation message#6565
blackboxsw merged 1 commit into
canonical:mainfrom
kikitux:fix-cocloud-http

Conversation

@kikitux
Copy link
Copy Markdown
Contributor

@kikitux kikitux commented Nov 8, 2025

  • I have signed the CLA: https://ubuntu.com/legal/contributors
  • I have included a comprehensive commit message using the guide below
  • I have added unit tests to cover the new behavior under tests/unittests/
    • Test files should map to source files i.e. a source file cloudinit/example.py should be tested by tests/unittests/test_example.py
    • Run unit tests with tox -e py3
  • I have kept the change small, avoiding unnecessary whitespace or non-functional changes.
  • I have added a reference to issues that this PR relates to in the PR message (Refs integration: do not LXD bind mount /etc/cloud/cloud.cfg.d #1234, Fixes integration: do not LXD bind mount /etc/cloud/cloud.cfg.d #1234)
  • I have updated the documentation with the changed behavior.
    • If the change doesn't change the user interface and is trivial, this step may be skipped.
    • Cloud-config documentation is generated from the jsonschema.
    • Generate docs with tox -e doc.

Proposed Commit Message

Updates a missed colon : on deprecation notice for nocloud-net

# as now
2025-11-08 08:59:02,803 - lifecycle.py[DEPRECATED]: The 'nocloud-net' datasource name is deprecated in 24.1 and scheduled to be removed in 29.1. Use 'nocloud' instead, which uses the seedfrom protocolscheme (http// or file://) to decide how to run.

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>)

Copy link
Copy Markdown
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 contribution @kikitux and making cloud-init better.

@blackboxsw
Copy link
Copy Markdown
Collaborator

Tests currently failing are unrelated to this changeset.

@blackboxsw blackboxsw merged commit f3975e4 into canonical:main Nov 11, 2025
13 of 21 checks passed
blackboxsw pushed a commit to blackboxsw/cloud-init that referenced this pull request Dec 12, 2025
holmanb pushed a commit that referenced this pull request Dec 18, 2025
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.

2 participants