← Back to issue list

lxd(remotes): fix deprecation warnings

View original Github issue

Metadata

Project
craft-providers
Number
#318
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #318 corrected the configure_buildd_image_remote deprecation warning so its replacement advice was accurate, resolving #260.

Resolution: closed resolved

Reason: PR #318 was merged into main and corrected the stale deprecation advice for configure_buildd_image_remote. The PR explicitly states that it resolves #260.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [X] Have you followed the guidelines for contributing? - [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [X] Have you successfully run `tox`? ----- The deprecation warning for `configure_buildd_image_remote` had the wrong advice! In the original PR, I wrote the deprecation, then changed the function name and forgot to update the deprecation. Resolves #260.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #318 corrected the configure_buildd_image_remote deprecation warning so its replacement advice was accurate, resolving #260.
qwen/qwen3.6-35b-a3b — closed resolved Fixed incorrect deprecation warning advice in configure_buildd_image_remote. Merged as #318 (commit 1f320c2), resolving #260.

Update history

Date Change
created

Related work

Related issues

Issue Project State Summary Similarity
#219 lxd: add support for interim Ubuntu releases craft-providers merged Merged PR #219 as commit c428f4b, adding RemoteImage/get_remote_image() support for interim and devel Ubuntu releases while retaining deprecated APIs.
74%
#1916 lxd: initialize remote lazily snapcraft merged Merged PR #1916 as dd4404843, deferring LXD remote initialization until needed so clean operations no longer probe or start an unnecessary remote.
73%
#220 lxd(remotes): rename BUILDD_REMOTE to BUILDD_RELEASE_REMOTE craft-providers closed BUILDD_REMOTE was renamed to BUILDD_RELEASE_REMOTE through stacked PR #219, merged as commit c428f4b after #220 was closed unmerged.
72%
#55 lxd: add buildd remote support (CRAFT-260) craft-providers merged PR #55 merged (commit ea7ed71 + f596edc): added configure_buildd_image_remote() in lxd/remotes.py, renaming the remote to craft-com.ubuntu.cloud-buildd, plus buildd launcher integration tests. Depended on merged #53 and #54.
72%
#96 Avoid race conditions when adding remotes in `configure_buildd_image_remote` craft-providers closed Fixed in merged PR #158 (commit 3069492): remote creation now rechecks after an add failure, avoiding concurrent setup races. The deprecated function was later removed in b9c60b6.
72%
#817 lxd: use built-in image streams. snapcraft merged Merged PR #817 (commit 7d502379e) by sergiusens, dropping cleanbuild's per-invocation custom LXD remotes in favour of LXD's built-in image: streams; fixes LP#1625787.
72%
#1516 lxd: LXD not installed when using remote snapcraft merged Merged as commit 5f8b122e4, fixing incorrect lxc installation checks for LXD remotes and completing related test-scenario handling.
72%
#962 fix(lxd/remotes): use release images for plucky craft-providers merged Merged PR #962 switched the LXD Plucky remote from discontinued daily buildd images to release images in commit 92a8243.
71%