Required options should be listed as required and error when not provided.
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Glance Client |
Fix Released
|
Undecided
|
Gorka Eguileor | ||
| Kilo |
New
|
Undecided
|
Unassigned | ||
Bug Description
I recently did a `glance image-create --name 'foo' --file 'foo.img' --disk-format 'qcow2'` and neglected to provide --container-format. Instead of failing immediately and telling me I must provide a container format glance client happily spent a few minutes copying bytes to a glance server then after all bytes had been copied I was told that None is not a valid container-format.
Glance client should make --container-format a required argument to image-create. I would fix this myself but after looking at the code I have realized that many other args and options are probably required too and I don't have a good way of building then correcting that list. So I am just going to add a bug.
If someone does have a list I am happy to take a stab at fixing this more properly.
| vikash irrinki (vikash-irrinki) wrote | #1 |
| Changed in python-glanceclient: | |
| assignee: | nobody → Gorka Eguileor (gorka) |
| OpenStack Infra (hudson-openstack) wrote Fix proposed to python-glanceclient (master) | #2 |
| Changed in python-glanceclient: | |
| status: | New → In Progress |
| OpenStack Infra (hudson-openstack) wrote Fix merged to python-glanceclient (master) | #3 |
| Changed in python-glanceclient: | |
| status: | In Progress → Fix Committed |
| Changed in python-glanceclient: | |
| milestone: | none → 1.0.0 |
| status: | Fix Committed → Fix Released |
is it ok if we take container format as 'bare' if not mentioned?