Skip to content
This repository has been archived by the owner on Apr 12, 2021. It is now read-only.

conjure-up uses petname for model name #595

Closed
adam-stokes opened this issue Jan 18, 2017 · 1 comment
Closed

conjure-up uses petname for model name #595

adam-stokes opened this issue Jan 18, 2017 · 1 comment
Assignees
Labels

Comments

@adam-stokes
Copy link
Contributor

adam-stokes commented Jan 18, 2017

conjure-up apparently uses single-word petnames for Juju models it
creates, e.g.:

| james@malefic:~$ juju models
| Controller: lxd
|
| Model Cloud/Region Status Machines Cores Access Last connection
| controller localhost/localhost available 1 - admin just now
| default localhost/localhost available 0 - admin 2017-01-02
| dane* localhost/localhost available 4 - admin 6 minutes ago
|
| james@malefic:~$

A bare pet name is pretty confusing and would only get more so if you
conjured up multiple spells. At a minimum, I think the spell should
be indicated in some way in the model name (perhaps combined with a
petname, if you need to avoid clashes). It might also be useful to
indicate in some way that this is a conjure-up model?

@mikemccracken
Copy link
Contributor

I like this idea. Honestly I'd prefer $spellname-$UUID
where UUID=uuidgen -t | cut -d '-' -f 1 instead of using petnames at all

@adam-stokes adam-stokes self-assigned this Jan 19, 2017
adam-stokes pushed a commit that referenced this issue Jan 23, 2017
Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants