← Back to issue list

test: failing reproducer - no error for high UID (>65535)

View original Github issue

Metadata

Project
craft-providers
Number
#949
Type
pull request
State
open
Author
lengau
Labels
—
Created
Updated
Closed
—

Current evaluation

Failing test reproducer for craft-providers#259: _set_id_map() should raise ProviderError for UIDs >65535 before calling LXD. PR incomplete, no fix implemented.

Suggested action: needs review

Reason: This PR adds a failing test reproducer for issue #259 (still open) but does not implement the proposed fix. The test demonstrates the bug where _set_id_map() calls lxc.config_set for UIDs >65535 instead of raising a descriptive error. The PR has been inactive for 136 days with 0 reviewers. A maintainer should review whether to implement the fix, close the issue, or request changes from the author.

Impact: 50 Quick Win: 40.0 Staleness: 70 Complexity: 20 Confidence: 70

Issue body

## Reproducer for #259 Enterprise systems using NIS, SSSD, or Active Directory commonly assign UIDs above 65535. When `map_user_uid=True` with such a UID, LXD rejects it with: ``` Error: Failed instance creation: Failed creating instance record: Failed initialising instance: Host id is in the range of subids ``` This message is cryptic. craft-providers should detect this condition and raise a clear `ProviderError` **before** calling LXD. ### Failing test `test_set_id_map_raises_for_high_uid` — shows that `_set_id_map()` currently calls `lxc.config_set(raw.idmap=...)` for UIDs > 65535 rather than raising a descriptive error first. ### Fix Add a check in `_set_id_map()` before the `lxc.config_set` call: if `uid > 65535`, raise a `ProviderError` with a link to documentation. Closes #259

Evaluation history

Date Model Scores Action Summary
qwen3.6-35b-a3b-mtp-q6
Impact: 50
Quick Win: 40.0
Staleness: 70
Complexity: 20
Confidence: 70
needs review Failing test reproducer for craft-providers#259: _set_id_map() should raise ProviderError for UIDs >65535 before calling LXD. PR incomplete, no fix implemented.
qwen/qwen3.6-35b-a3b
Staleness: 65
Complexity: 10
Confidence: 75
needs review Adds a failing test reproducing LXD rejection for UIDs >65535 and proposes a check in _set_id_map() to raise a clear ProviderError before LXD invocation. Currently pending maintainer review with mixed CI results.
qwen3.6-35b-a3b-mtp-q6
Staleness: 55
Complexity: 20
Confidence: 85
needs review Adds a failing test reproducing a cryptic LXD error when using UIDs above 65535. Describes a fix to validate UIDs before LXD interaction. Currently inactive with no comments.
qwen3.6-35b-a3b-mtp-q6
Staleness: 60
Complexity: 15
Confidence: 85
needs review Adds a guard check in _set_id_map to raise a clear ProviderError for UIDs over 65535 before calling LXD. Currently a failing reproducer with no maintainer feedback or CI activity after 65 days.

Update history

No update history recorded yet.

Related work

  • Related To: charmcraft#1054 (confidence 85%)

    Charmcraft issue #1054 references craft-providers#259 as the root cause and confirms the same high-UID LXD mapping bug affects charmcraft pack.

Related issues

No related issues found above the similarity threshold.