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

Allow custom deploy command in charm header #1664

Open
sed-i opened this issue Sep 12, 2023 · 5 comments
Open

Allow custom deploy command in charm header #1664

sed-i opened this issue Sep 12, 2023 · 5 comments

Comments

@sed-i
Copy link
Contributor

sed-i commented Sep 12, 2023

Currently, all charms have the same deploy command,
juju deploy <charm name>:

image

In some cases this would result in "broken" deployment.
For example, deploying grafana (and others) requires a --trust flag:
juju deploy grafana-k8s --trust


  <div class="p-charm-header__code">
    <div class="p-tooltip--information">
      <div>
        <code>juju deploy grafana-k8s</code>  <!--- This should be customizable -->
@Lukewh
Copy link
Contributor

Lukewh commented Sep 29, 2023

This seems like an easy feature, but has large consequences and adds a lot more complexity. I would suggest the Grafana charm should be updated with proper documentation that informs users that they may need to run juju trust grafana-k8s after deployment, while we look into this further.

@taurus-forever
Copy link

This is could be nice feature!
Meanwhile we are blocking the charm hinting user to call juju trust if K8s charm deployed without --trust:

Unit               Workload  Agent      Address       Ports  Message
postgresql-k8s/0*  blocked   executing  10.1.110.75          Insufficient permissions, try: `juju trust postgresql-k8s --scope=cluster`

@sayantan1410
Copy link

Hi, I would like to work on this,

What I have understood is that we need to send the deploy commands to html file from python or may be from some config file.

Is my understanding correct and is this issue still up for development ?

Thank you.

Copy link

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/WD-13797.

This message was autogenerated

@sed-i
Copy link
Contributor Author

sed-i commented Jul 31, 2024

A good example is huggingface. You click "use this model" and you get exactly what you need.

Screenshot

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

4 participants