OpenStack Image Registry and Delivery Service (Glance)

bin/glance should allow nullable Name to match API spec

Reported by justinsb on 2012-02-17
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Low
Jay Pipes

Bug Description

On an image upload through the API, the name need not be specified, and the image then has a null name.

The CLI tool requires a name (but does so client-side)

Should name be nullable?

Brian Waldon (bcwaldon) wrote :

We don't use name for anything, as we have the unique uuid. I don't want to force users to provide a name if they really don't want to provide one. How about we make the name nullable through the cli?

Jay Pipes (jaypipes) wrote :

I don't mind making the name nullable through the CLI to match the API spec.

Changed in glance:
status: New → Triaged
importance: Undecided → Low
milestone: none → essex-4
summary: - Name is not required (can be null) on image upload
+ bin/glance should allow nullable Name to match API spec
Jay Pipes (jaypipes) on 2012-02-29
Changed in glance:
milestone: essex-4 → essex-rc1
Jay Pipes (jaypipes) on 2012-03-13
Changed in glance:
status: Triaged → In Progress
assignee: nobody → Jay Pipes (jaypipes)

Reviewed: https://review.openstack.org/5283
Committed: http://github.com/openstack/glance/commit/1615a1fc57da85caae147064c45967d90a9b73f6
Submitter: Jenkins
Branch: master

commit 1615a1fc57da85caae147064c45967d90a9b73f6
Author: Jay Pipes <email address hidden>
Date: Tue Mar 13 15:21:40 2012 -0400

    Fixes LP#934492 - Allow Null Name

    * Modify bin/glance to allow nullable name property,
      in accordance with API spec

    This changeset also adds a fix for Swift functional
    tests where HTTPS locations weren't being properly
    constructed.

    Change-Id: Ibad2d3ab150ee347449b4e9b3b9e941ce7a0fc6f

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2012-03-21
Changed in glance:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2012-04-05
Changed in glance:
milestone: essex-rc1 → 2012.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers