Skip to content

Refactor podcvd setup procedure#2524

Open
jemoreira wants to merge 6 commits intogoogle:mainfrom
jemoreira:podcvdsetup
Open

Refactor podcvd setup procedure#2524
jemoreira wants to merge 6 commits intogoogle:mainfrom
jemoreira:podcvdsetup

Conversation

@jemoreira
Copy link
Copy Markdown
Member

@jemoreira jemoreira commented May 6, 2026

Implements go/podcvd-setup

Introduces podcvd-setup script, /etc/podcvd.users config file for users
with podcvd access and IP range allocation, and the podcvd-acl-restore
system service.

Podcvd also ensures podman.socket is running before attempting to contact it.

Assisted-by: Gemini:Next

@jemoreira jemoreira requested a review from 0405ysj May 6, 2026 23:59
Comment thread container/debian/podcvd-setup Outdated
Comment thread container/debian/cuttlefish-podcvd.cuttlefish-podcvd.init Outdated
Comment thread container/debian/cuttlefish-podcvd.install Outdated
Comment thread container/debian/cuttlefish-podcvd.install
Comment thread container/debian/cuttlefish-podcvd.postinst Outdated
Comment thread container/debian/podcvd-setup
Comment thread container/debian/podcvd-setup Outdated
Comment thread container/debian/podcvd-setup Outdated
Comment thread container/src/podcvd/internal/check.go Outdated
Comment thread container/src/podcvd/internal/check.go Outdated
Comment thread container/src/podcvd/internal/main.go
jemoreira added 5 commits May 7, 2026 14:31
Implements go/podcvd-setup

Introduces podcvd-setup script, /etc/podcvd.users config file for users
with podcvd access and IP range allocation, and the podcvd-acl-restore
system service.
podcvd-setup should be used instead. The warning is added as a comment
in the config file itself, which is created by the post-install script.
With the transition to a /20 subnetwork, the old range (192.168.80.0/20)
would conflict with the statically allocated networks used by
cuttlefish-base.
@jemoreira jemoreira requested a review from 0405ysj May 7, 2026 21:32
Copy link
Copy Markdown
Collaborator

@0405ysj 0405ysj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for improving podcvd!

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could see /usr/lib/cuttlefish-podcvd/bin/podcvd-setup: Permission denied error from E2E test. This file should be executable.

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