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

RFC: add Kube unit support #473

Closed
wants to merge 1 commit into from

Conversation

dabelenda
Copy link

Pull Request (PR) description

Podman integration brings support for multiple Unit types, including Kube unit that allows to start a pod with a YAML file declaration.

This PR in its current form is an RFC to discuss if these Unit Types are in the scope of this module. Or if they should be added to another module.

@@ -4,7 +4,7 @@
#
# @see systemd.unit(5)
#
# @param name [Pattern['^[^/]+\.(service|socket|device|mount|automount|swap|target|path|timer|slice|scope)$']]
# @param name [Pattern['^[^/]+\.(service|socket|device|mount|automount|swap|target|path|timer|slice|scope|kube)$']]
Copy link
Member

Choose a reason for hiding this comment

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

I think we shouldn't document it here, that's already in the type alias definition.

@dabelenda dabelenda closed this Jul 4, 2024
@dabelenda
Copy link
Author

Sorry I made a mistake, apparently there is a utility to run that will generate a service unit from the definition of the .kube file.

systemd will not directly read kube units so it makes no sense to make this PR here...

@traylenator
Copy link
Contributor

traylenator commented Jul 4, 2024

I tried adding quadlet support southalc/podman#83 which ,kube is an example of.

Having said that that MR seems to be going nowhere

@dabelenda dabelenda deleted the add_kube_unit branch July 4, 2024 12:28
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.

3 participants