Skip to content

(vpnd) Add polkitd ad nym-vpnd dependency#5470

Open
agentpietrucha wants to merge 1 commit into
developfrom
agentpietrucha/polkit-dep-update
Open

(vpnd) Add polkitd ad nym-vpnd dependency#5470
agentpietrucha wants to merge 1 commit into
developfrom
agentpietrucha/polkit-dep-update

Conversation

@agentpietrucha
Copy link
Copy Markdown
Contributor

@agentpietrucha agentpietrucha commented Jun 1, 2026

Ticket

JIRA-NYM-XXXX

Description

Latest Debian 13 and Ubuntu 25 don't provide either policykit-1 or polkit. polkitd is added as 3rd alternative to support these latest distros

This PR solves the following errors that users will experience:

sudo apt install --only-upgrade nym-vpnd
Solving dependencies... Error!
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

Unsatisfied dependencies:
 nym-vpnd : Depends: policykit-1 but it is not installable or
                     polkit but it is not installable
Error: Unable to correct problems, you have held broken packages.
Error: The following information from --solver 3.0 may provide additional context:
   Unable to satisfy dependencies. Reached two conflicting decisions:
   1. nym-vpnd:amd64=1.30.0 is selected as an upgrade
   2. nym-vpnd:amd64=1.30.0 Depends policykit-1 | polkit
      but none of the choices are installable:
      [no choices]

Checklist:

  • Changelog

Screenshots (optional, if UI related)


This change is Reviewable

Summary by CodeRabbit

  • Chores
    • Updated package dependencies to support additional system policykit providers, improving compatibility with different Linux distributions.

@agentpietrucha agentpietrucha added this to the v2026.10 - Tatry milestone Jun 1, 2026
@agentpietrucha agentpietrucha self-assigned this Jun 1, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 1, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 624277b6-f06f-45ec-848c-673fd0a0811d

📥 Commits

Reviewing files that changed from the base of the PR and between b22701f and abc31f1.

📒 Files selected for processing (1)
  • nym-vpn-core/crates/nym-vpnd/Cargo.toml

📝 Walkthrough

Walkthrough

The Debian package metadata for nym-vpnd is updated to include polkitd as an additional policykit provider alternative. The depends field now accepts policykit-1 | polkit | polkitd instead of only policykit-1 | polkit, broadening the set of compatible policykit implementations.

Changes

Debian Package Metadata Update

Layer / File(s) Summary
Expand policykit dependency alternatives
nym-vpn-core/crates/nym-vpnd/Cargo.toml
The Debian package depends field is updated to include polkitd as an additional policykit provider alternative alongside policykit-1 and polkit.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

Suggested reviewers

  • trojanfoe
  • neacsu

Poem

🐰 A hop and a skip through deps so fine,
Polkitd joins the provider line,
Where policykit dances with its crew,
More choices bloom—the pkg is new! 🔐

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change—adding polkitd as an alternative dependency for nym-vpnd—matching the file-level changes and PR objectives.
Description check ✅ Passed The description includes the ticket reference, clear explanation of the issue with supporting evidence (error logs), and the solution, though the Changelog item remains unchecked as required.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agentpietrucha/polkit-dep-update

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 1, 2026

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.

1 participant