Skip to content

Conversation

@ygalblum
Copy link
Contributor

@ygalblum ygalblum commented Jan 5, 2026

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
Copy link
Member

baude commented Jan 12, 2026

LGTM

1 similar comment
@TomSweeneyRedHat
Copy link
Member

LGTM

@TomSweeneyRedHat
Copy link
Member

/lgtm

@jankaluza
Copy link
Member

LGTM.

I spent time thinking if we should search for @ or @.. The thing is that strictly speaking, template is just [email protected]. If there is a [email protected], 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: containers#27844

Signed-off-by: Ygal Blum <[email protected]>
@ygalblum ygalblum force-pushed the quadlet-pod-template branch from c0c77fe to 0d3c438 Compare January 15, 2026 14:23
@ygalblum
Copy link
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
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
Member

@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 containers:main Jan 16, 2026
78 checks passed
@ygalblum ygalblum deleted the quadlet-pod-template branch January 16, 2026 18:23
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.

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

4 participants