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

Workflow where knative-serving/eventing spawns new namespaces feels unusual #96

Open
ca-scribner opened this issue Apr 6, 2023 · 0 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@ca-scribner
Copy link
Contributor

At present, charmed knative is designed1 such that:

  • the knative-operator charm deploys the upstream knative operator. This operator manages all knative serving/eventing deployments for us
  • knative-serving (/knative-eventing) charms deploy/manage an instance of a KnativeServing (/KnativeEventing) CR which is acted upon by the upstream knative operator

The way someone might use these then is by:

juju deploy knative-operators
juju deploy knative-serving --config namespace=some-namespace [...other config]

This would result in the knative-serving charm creating a KnativeServing CR that requests for knative serving to be deployed in the namespace some-namespace, and the upstream knative operator will create this namespace/deploy things.

This pattern feels odd compared to other juju things. Typically, we'd expect applications to work within their models. Would it make more sense to design the knative-serving (/knative-eventing) charms to always ask for an instance within their own namespace, and then deploy them to a separate model? The workflow would then be:

# from kubeflow model
juju deploy knative-operators

juju add-model some-namespace
juju switch some-namespace
juju deploy knative-serving [...other config]  # Note that `--config namespace` is removed

where the result is a knative serving instance in the namespace some-namespace?

This feels more Juju, but might be less practical. We package knative in the kubeflow bundle - can a bundle have multiple models?

Footnotes

  1. Charmed Knative design spec

@i-chvets i-chvets added bug Something isn't working enhancement New feature or request labels May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants