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

Unable to push bundle with kubernetes-service-annotations YAML entry to charm store #869

Open
knkski opened this issue Feb 13, 2019 · 3 comments

Comments

@knkski
Copy link

knkski commented Feb 13, 2019

I have a bundle that contains this bit of YAML:

options:
  kubernetes-service-annotations: {...} # Some k8s-specific YAML

It works great when deploying locally with juju deploy ./bundle.yaml, but fails when pushing up to the charm store:

$ charm push . cs:~kubeflow-charmers/kubeflow
ERROR cannot post archive: bundle verification failed: ["cannot validate application \"kubeflow-jupyterhub\": configuration option \"kubernetes-service-annotations\" not found in charm \"cs:~kubeflow-charmers/kubeflow-jupyterhub\"","cannot validate application \"kubeflow-tf-job-dashboard\": configuration option \"kubernetes-service-annotations\" not found in charm \"cs:~kubeflow-charmers/kubeflow-tf-job-dashboard\""]
@rogpeppe
Copy link
Contributor

i suspect it's about the fact that cs:~kubeflow-charmers/kubeflow-jupyterhub is a gated charm, and that bundle checking verifies based on the latest public revision when no revision is explicitly specified.

I'm not sure it could reasonably do otherwise, because it can't necessarily assume that the user pushing the bundle has the same permissions as the user that will use it.

Could you try specifying exact charm revisions in the bundle and see if that fixes the issue?

@knkski
Copy link
Author

knkski commented Feb 14, 2019

I've tried making the charms public (both stable and edge for kubeflow-jupyterhub), and specifying particular revisions, neither of which seemed to work.

@knkski
Copy link
Author

knkski commented Feb 26, 2019

@rogpeppe: I've removed the annotations in bundle.yaml in favor of setting up the annotations in pod-set-spec that @wallyworld just added support for, and the push works great.

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

No branches or pull requests

3 participants