Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support multi doc yaml for generate/play kube #9759

Merged

Conversation

EduardoVega
Copy link
Contributor

Fix: #9129

This PR adds the following for podman play / generate kube

  • podman play kube
    • Reads every document in a multi doc yaml file
    • Fails if any of the documents is invalid
    • Creates a podman pod if document is a valid kubernetes pod or deployment
    • Skips any other valid kubernetes kind
  • podman generate kube
    • Accepts multiple podman pods
    • Generates a multi doc yaml file

Signed-off-by: Eduardo Vega [email protected]

@EduardoVega EduardoVega force-pushed the 9129-multi-docs-kube branch 2 times, most recently from ee077f9 to 5f786be Compare March 20, 2021 01:58
@zhangguanzhang
Copy link
Collaborator

should also handle the configmap and volumes

@EduardoVega
Copy link
Contributor Author

@zhangguanzhang I can add configmaps. For volumes (k8s persistent volume claims) I think there is some work in progress for that. I can track all the issues/prs that are related to it and check their status because that should be a dependency for this change.

@rhatdan
Copy link
Member

rhatdan commented Mar 24, 2021

Nice work @EduardoVega LGTM

@rhatdan
Copy link
Member

rhatdan commented Mar 25, 2021

@containers/podman-maintainers PTAL

@rhatdan
Copy link
Member

rhatdan commented Mar 25, 2021

@vrothberg @giuseppe @saschagrunert PTAL

go.mod Outdated Show resolved Hide resolved
@EduardoVega EduardoVega force-pushed the 9129-multi-docs-kube branch from 5f786be to cbf53c1 Compare March 25, 2021 12:54
Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@vrothberg
Copy link
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Mar 25, 2021
@rhatdan
Copy link
Member

rhatdan commented Mar 25, 2021

/approve

@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: EduardoVega, rhatdan

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-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 25, 2021
@openshift-merge-robot openshift-merge-robot merged commit 24581d8 into containers:master Mar 25, 2021
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 23, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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. 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.

"podman play kube" yaml multiple resources
6 participants