Skip to content

Problems with containers started with --detach=True#182

Open
TheRealBecks wants to merge 1 commit intomainfrom
issue174
Open

Problems with containers started with --detach=True#182
TheRealBecks wants to merge 1 commit intomainfrom
issue174

Conversation

@TheRealBecks
Copy link
Collaborator

Fixes #174

@TheRealBecks TheRealBecks linked an issue Feb 9, 2026 that may be closed by this pull request
@TheRealBecks
Copy link
Collaborator Author

@k9withabone Is it correct to remove #[serde(skip_serializing)]? I checked other code examples in the repository and they don't have that line in combination with the now more specific #[arg... line.

@k9withabone
Copy link
Member

No, #[serde(skip_serializing)] is necessary. --detach is a special case because Quadlet always puts -d in the output .service file's ExecStart= option. So if Podlet does not skip it when setting PodmanArgs= in the .container file, you'll get --detach -d in the output.

@k9withabone
Copy link
Member

Also, please try to use conventional commits with a Fixes: footer linking the issue. See the commit message of 79cd688 for an example.

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.

Problems with containers started with --detach=True

2 participants