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

Automatically create entries for all workloads and all relations? #3

Open
sed-i opened this issue Jun 30, 2022 · 2 comments
Open

Automatically create entries for all workloads and all relations? #3

sed-i opened this issue Jun 30, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@sed-i
Copy link

sed-i commented Jun 30, 2022

All workloads would probably have a Waiting for pebble-ready and all relations should probably have a Blocked: invalid relation data, so might as well offer them by default?

  • all workloads
  • all relations
  • one entry for config (e.g. "Waiting for K8s resource limit patch to apply" which needs to persist across pod churn)
@PietroPasotti PietroPasotti added the enhancement New feature or request label Jul 1, 2022
@PietroPasotti
Copy link
Owner

This feels to me a bit too specific.
What if your charm has no relations? What if your charm is not a k8s charm?
What if you don't care about the status of all relations you have, but only some?

I'm not sure the cost of figuring this out and writing the machinery to implement it offsets the one-time investment the user has to make to set up the StatusPool base class for the charm.

We could write a script to generate for you that code by looking at the metadata.yaml, sure, but that feels like something on top of this lib, not something that this lib should be concerned about.

@jnsgruk
Copy link

jnsgruk commented Jul 8, 2022

Yeh, while I think I can see where @sed-i is coming from, this feels like premature optimisation! I think any such functionality would probably be making too many assumptions to be really useful in every case?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants