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

WIP: Add support for configmaps #100

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cgwalters
Copy link
Collaborator

This is part of #22

@openshift-ci
Copy link

openshift-ci bot commented May 2, 2023

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@cgwalters
Copy link
Collaborator Author

I was typing this up, but thinking about it...perhaps it should land in ostree-ext so it can be shared with rpm-ostree?

Or...maybe we publish a crate for bootc and pull that into rpm-ostree too...

@cgwalters cgwalters force-pushed the configmap branch 3 times, most recently from d040dfe to 84f3edf Compare June 11, 2023 16:35
@cgwalters cgwalters force-pushed the configmap branch 2 times, most recently from 9d4fa2b to 6b0a900 Compare June 18, 2023 18:39
@cgwalters
Copy link
Collaborator Author

OK what I'm getting hung up on a bit here is exactly what the API is to upload objects outside of kubernetes. It looks like podman play kube basically punts on this and you point the command at an arbitrary directory...which is rather un-opinionated - and also has some annoying ergonomics because then you typically want to delete the files after they've been created.

The typical systemd-style pattern of supporting e.g. /run/bootc/api and /etc/bootc/api (but probably not /usr/lib/bootc/api since that gets weird around mutability) is perhaps a bit better than an arbitrary directory.

We'd want to have bootc api get configmap to dump all the configmap objects that bootc knows about. (Or maybe just bootc get?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant