Skip to content

Quadlet - do not link pod service units to quadlet templates - #27860

Merged
TomSweeneyRedHat merged 1 commit into
podman-container-tools:mainfrom
ygalblum:quadlet-pod-template
Jan 16, 2026
Merged

Quadlet - do not link pod service units to quadlet templates#27860
TomSweeneyRedHat merged 1 commit into
podman-container-tools:mainfrom
ygalblum:quadlet-pod-template

Conversation

@ygalblum

@ygalblum ygalblum commented Jan 5, 2026

Copy link
Copy Markdown
Contributor

When a template container unit is linked to a pod do not add its service to the pod units dependency

Resolves: #27844

Checklist

Ensure you have completed the following checklist for your pull request to be reviewed:

  • Certify you wrote the patch or otherwise have the right to pass it on as an open-source patch by signing all
    commits. (git commit -s). (If needed, use git commit -s --amend). The author email must match
    the sign-off email address. See CONTRIBUTING.md
    for more information.
  • Referenced issues using Fixes: #00000 in commit message (if applicable)
  • Tests have been added/updated (or no tests are needed)
  • Documentation has been updated (or no documentation changes are needed)
  • All commits pass make validatepr (format/lint checks)
  • Release note entered in the section below (or None if no user-facing changes)

Does this PR introduce a user-facing change?

Quadlet - Do not link pod service units to template container service units

@baude

baude commented Jan 12, 2026

Copy link
Copy Markdown
Contributor

LGTM

1 similar comment
@TomSweeneyRedHat

Copy link
Copy Markdown
Contributor

LGTM

@TomSweeneyRedHat

Copy link
Copy Markdown
Contributor

/lgtm

@jankaluza

Copy link
Copy Markdown
Contributor

LGTM.

I spent time thinking if we should search for @ or @.. The thing is that strictly speaking, template is just foo@.service. If there is a foo@bar.service, it is an instance of that template afaik. The isTemplateUnit currently matches both templates and instances, but I think it is fine in this context.

Just saying that in case someone with more experience with this code would find this an issue.

@ygalblum
ygalblum force-pushed the quadlet-pod-template branch from 47b88ef to c0c77fe Compare January 15, 2026 14:07
When a template container unit is linked to a pod do not add
its service to the pod units dependency

Fixes: podman-container-tools#27844

Signed-off-by: Ygal Blum <ygal.blum@gmail.com>
@ygalblum
ygalblum force-pushed the quadlet-pod-template branch from c0c77fe to 0d3c438 Compare January 15, 2026 14:23
@ygalblum

Copy link
Copy Markdown
Contributor Author

@jankaluza good catch. I've fixed the code. Note that it required reverting part of the change since in the other cases, the check is in fact for "template or an instance of a template".
I've updated the test as well. PTAL

@ygalblum

Copy link
Copy Markdown
Contributor Author

BTW is there an issue with the bot? @TomSweeneyRedHat's LGTM did not trigger any labeling (and as a result the PR did not merge already)

@TomSweeneyRedHat

Copy link
Copy Markdown
Contributor

@ygalblum this is the second time this week the bot has ignored my merge request. @bbaude do you know what's going on?

@TomSweeneyRedHat
TomSweeneyRedHat merged commit 8a33866 into podman-container-tools:main Jan 16, 2026
78 checks passed
@ygalblum
ygalblum deleted the quadlet-pod-template branch January 16, 2026 18:23
@stale-locking-app stale-locking-app Bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Apr 17, 2026
@stale-locking-app stale-locking-app Bot locked as resolved and limited conversation to collaborators Apr 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Quadlet container which sets Pod causes unwanted Pod dependency on untemplated container

4 participants