Skip to content

Conversation

@axel7083
Copy link
Contributor

fixes #26274

Does this PR introduce a user-facing change?

Following #27292, the podman kube play/down command supports multiple files as argument. (Thanks again @Honny1 for your amazing work getting this fixed and merged).

However, the issue #26274 originally targeted the Kube Quadlet, and I explained that this is a two steps process

  1. Making the podman cli supporting multiple arguments
  2. Updating the quadlet parser & generator

The first step has been completed by #27292 and this is the second.

kube quadlet support multiple Yaml entries

@ygalblum
Copy link
Contributor

I'm OK with this change.
I wonder if we should look for a way for setting the working directory for multiple yaml files? Users can manually set it in the Service section, but that breaks portability.

Copy link
Member

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

Thanks, the code LGTM at first glance.

@axel7083 axel7083 force-pushed the feature/quadlet/kube-support-multiple-yaml branch from cd39793 to 8e013c0 Compare October 22, 2025 14:10
@axel7083 axel7083 requested a review from Honny1 October 22, 2025 14:16
@@ -0,0 +1,4 @@
[Kube]
Copy link
Contributor

Choose a reason for hiding this comment

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

This file is missing:

## assert-failed
## assert-stderr-contains <The expected error message>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! I was kinda lost 😿 !

Signed-off-by: axel7083 <[email protected]>
Copy link
Member

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM.

@Honny1
Copy link
Member

Honny1 commented Oct 22, 2025

It looks like the CI failure was just a flaky test.

Copy link
Contributor

@ygalblum ygalblum left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 22, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 22, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: axel7083, Honny1, ygalblum

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 22, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit bc3b910 into containers:main Oct 22, 2025
81 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release-note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Kube Quadlet should support multiple YAML

3 participants