← Back to issue list

Use `pylxd` instead of `lxc exec`

View original Github issue

Metadata

Project
snapcraft
Number
#3147
Type
pull request
State
closed
Author
abitrolly
Labels
—
Created
Updated
Closed

Current evaluation

Closed unmerged as stale on 2020-07-28 after the pylxd change hit an unresolved FakeContainer.execute test failure; no replacement or resolving commit was identified.

Resolution: closed stale

Reason: Maintainer sergiusens explicitly closed the unmerged PR because it had gone stale. Four review comments and the FakeContainer test failure remained unresolved, with no merge commit or superseding PR identified.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? ----- `pylxd` ignores default LXD remote, but `lxc exec` does not. This is untested attempt to remove the todo and fix `--use-lxd` failure if default LXD remote is not local. https://forum.snapcraft.io/t/use-lxd-doesnt-work-with-non-local-remote/17775

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed stale Closed unmerged as stale on 2020-07-28 after the pylxd change hit an unresolved FakeContainer.execute test failure; no replacement or resolving commit was identified.
stealth/space-bunny-alpha — closed stale Proposed replacing lxc exec with pylxd for non-local LXD remotes; PR #3147 was closed unmerged in July 2020 after test failures and inactivity.
stealth/space-bunny-alpha — closed stale Attempt to replace lxc exec with pylxd was abandoned unmerged after test mocks lacked execute; the maintainer closed it as stale in July 2020.
stealth/space-bunny-alpha — closed stale PR #3147 proposed using pylxd to support non-local default LXD remotes, but was closed unmerged as stale after unresolved FakeContainer test failures.
qwen/qwen3.6-35b-a3b — — Abandoned after mocking-related test failures prevented progress. Maintainer closed the pull request as stale, deferring LXD remote handling improvements to future development.
qwen3.6-35b-a3b-mtp-q6 — — Closed as stale without merging. The change attempted to replace lxc exec with pylxd to fix LXD remote handling but failed due to test mocking issues. Maintainers noted future plans for LXD remote support.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#904 cleanbuild: use pylxd instead of lxd command line snapcraft closed PR #904 was closed unmerged as stale on 6 February 2017 after prolonged inactivity; the maintainer also cited substantial pylxd differences on Ubuntu Xenial.
70%