Skip to content

Add systemd unit file for cuttlefish-podcvd.service#2572

Merged
0405ysj merged 1 commit into
google:mainfrom
0405ysj:systemd_service
May 18, 2026
Merged

Add systemd unit file for cuttlefish-podcvd.service#2572
0405ysj merged 1 commit into
google:mainfrom
0405ysj:systemd_service

Conversation

@0405ysj
Copy link
Copy Markdown
Collaborator

@0405ysj 0405ysj commented May 15, 2026

Context: b/473836983, b/510167998

@0405ysj 0405ysj requested review from ikicha and k311093 May 15, 2026 08:32
@0405ysj 0405ysj added the kokoro:force-run Trigger a presubmit build unconditionally. label May 15, 2026
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:force-run Trigger a presubmit build unconditionally. label May 15, 2026
@0405ysj 0405ysj enabled auto-merge May 15, 2026 10:25
@0405ysj 0405ysj added this pull request to the merge queue May 17, 2026
@rmuthiah rmuthiah removed this pull request from the merge queue due to a manual request May 17, 2026
@0405ysj 0405ysj added this pull request to the merge queue May 18, 2026
@0405ysj
Copy link
Copy Markdown
Collaborator Author

0405ysj commented May 18, 2026

Thanks for the review!

@@ -0,0 +1,21 @@
[Unit]
SourcePath=/etc/init.d/cuttlefish-podcvd
Description=Setup initial environment to use 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.

QQ: why do we need systemd service for 'initial' env setup? can't it be done in postinstall script in deb pkg?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Because of network interface(named podcvd) configuration for linking and routing. It wasn't even triggered during post-installation, when SysV init is not compatible on the machine.

Merged via the queue into google:main with commit 04809ba May 18, 2026
52 checks passed
@0405ysj 0405ysj deleted the systemd_service branch May 18, 2026 01:14
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.

4 participants