add_basic_images for liberty and above needs correction to pick correct glance command

Bug #1639210 reported by sundarkh
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.1
Fix Committed
High
sundarkh
R3.2
Fix Committed
High
sundarkh
Trunk
Fix Committed
High
sundarkh

Bug Description

add_basic_images for liberty and above needs correction to pick correct glance command

for glance commands

--is-public True should be changed to --visibility public, for Liberty or above SKUs.

glance image-create --name "tiny_trans_fw" --is-public True --container-format bare --disk-format vmdk --file /tmp/tinycore-net-transparent-fw.qcow2
usage: glance [--version] [-d] [-v] [--get-schema] [--no-ssl-compression] [-f]
              [--os-image-url OS_IMAGE_URL]
              [--os-image-api-version OS_IMAGE_API_VERSION]
              [--profile HMAC_KEY] [--insecure] [--os-cacert <ca-certificate>]
              [--os-cert <certificate>] [--os-key <key>] [--timeout <seconds>]
              [--os-auth-url OS_AUTH_URL] [--os-domain-id OS_DOMAIN_ID]
              [--os-domain-name OS_DOMAIN_NAME]
              [--os-project-id OS_PROJECT_ID]
              [--os-project-name OS_PROJECT_NAME]
              [--os-project-domain-id OS_PROJECT_DOMAIN_ID]
              [--os-project-domain-name OS_PROJECT_DOMAIN_NAME]
              [--os-trust-id OS_TRUST_ID] [--os-user-id OS_USER_ID]
              [--os-username OS_USERNAME]
              [--os-user-domain-id OS_USER_DOMAIN_ID]
              [--os-user-domain-name OS_USER_DOMAIN_NAME]
              [--os-password OS_PASSWORD] [--key-file OS_KEY]
              [--ca-file OS_CACERT] [--cert-file OS_CERT]
              [--os-tenant-id OS_TENANT_ID] [--os-tenant-name OS_TENANT_NAME]
              [--os-region-name OS_REGION_NAME]
              [--os-auth-token OS_AUTH_TOKEN]
              [--os-service-type OS_SERVICE_TYPE]
              [--os-endpoint-type OS_ENDPOINT_TYPE]
              <subcommand> ...
glance: error: unrecognized arguments: --is-public True

right now, it works only for Ubuntu and not for centos, which needs to be changed

Tags: automation
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/25710
Submitter: sundarkh (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.1

Review in progress for https://review.opencontrail.org/25712
Submitter: sundarkh (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.2

Review in progress for https://review.opencontrail.org/25713
Submitter: sundarkh (<email address hidden>)

Jeba Paulaiyan (jebap)
tags: added: automation
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/25712
Committed: http://github.org/Juniper/contrail-fabric-utils/commit/50e8d40b0a2701d8269fef5b8119eb9f686299af
Submitter: Zuul
Branch: R3.1

commit 50e8d40b0a2701d8269fef5b8119eb9f686299af
Author: Sundar K H <email address hidden>
Date: Fri Nov 4 17:47:14 2016 +0530

correcting the check in add basic images to work for liberty or above

Change-ID: I3b9180c2ac5aa5aa1cc086f4e31d65d50a82f0b1
Closes-Bug: #1639210

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/25710
Committed: http://github.org/Juniper/contrail-fabric-utils/commit/82b3519f15b66f4c91ba8f0633265afaa0160e29
Submitter: Zuul
Branch: master

commit 82b3519f15b66f4c91ba8f0633265afaa0160e29
Author: Sundar K H <email address hidden>
Date: Fri Nov 4 15:34:57 2016 +0530

correcting the check in add basic images to work for liberty or above

Change-Id: I459a1e049ff853c2b42329218ffcae2af5cfa4a3
Closes-Bug: #1639210

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/25713
Committed: http://github.org/Juniper/contrail-fabric-utils/commit/fb7c5b14d0bd8cc18f692f221e6c8d2271943638
Submitter: Zuul
Branch: R3.2

commit fb7c5b14d0bd8cc18f692f221e6c8d2271943638
Author: Sundar K H <email address hidden>
Date: Fri Nov 4 17:58:15 2016 +0530

correcting the check in add basic images to work for liberty or above

Change-ID: Ib1a3ce3260c02810fafb442bee586a95fb777a45
Closes-Bug: #1639210

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.