command "cinder cgsnapshot-show" doesn't list snapshot-volumes and snapshot volume-type

Bug #1572993 reported by bhavik bhavsar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Confirmed
Wishlist
Unassigned

Bug Description

The current implementation doesn't list following :

snapshot-volumes and snapshot volume-type for command "cinder cgsnapshot-show"

User can get a better visibility of snapshots volumes and snapshot volume types for cgsnapshot-show.

Revision history for this message
Sheel Rana (ranasheel2000) wrote :

This is not an issue rather improvement.

Moving this to wishlist...

Changed in cinder:
importance: Undecided → Wishlist
Changed in cinder:
assignee: nobody → gundarapu kalyan reddy (gundarapu-reddy)
Revision history for this message
gundarapu kalyan reddy (gundarapu-reddy) wrote :

I think it is already implemented in openstack Mitaka version. Now user can verify the volume-types from cinder consisgroup-show command.

Changed in cinder:
assignee: gundarapu kalyan reddy (gundarapu-reddy) → nobody
Revision history for this message
Sheel Rana (ranasheel2000) wrote :

Right, cinder code specify that cinder consisgroup-show displays volume_type in output

        if is_detail:
            consistencygroups = self._view_builder.detail_list(
                req, consistencygroups)
        else:
            consistencygroups = self._view_builder.summary_list(
                req, consistencygroups)

view_buider->

    def detail_list(self, request, consistencygroups):
        """Detailed view of a list of consistency groups ."""
        return self._list_view(self.detail, request, consistencygroups)

    def detail(self, request, consistencygroup):
        """Detailed view of a single consistency group."""
        if consistencygroup.volume_type_id:
            volume_types = consistencygroup.volume_type_id.split(",")
            volume_types = [type_id for type_id in volume_types if type_id]
        else:
            volume_types = []

        return {
            'consistencygroup': {
                'id': consistencygroup.id,
                'status': consistencygroup.status,
                'availability_zone': consistencygroup.availability_zone,
                'created_at': consistencygroup.created_at,
                'name': consistencygroup.name,
                'description': consistencygroup.description,
                'volume_types': volume_types,
            }
        }

But "cinder cgsnapshot-show" does not show snapshot-volumes and snapshot-volume-types:

    def detail(self, request, cgsnapshot):
        """Detailed view of a single cgsnapshot."""
        return {
            'cgsnapshot': {
                'id': cgsnapshot.id,
                'consistencygroup_id': cgsnapshot.consistencygroup_id,
                'status': cgsnapshot.status,
                'created_at': cgsnapshot.created_at,
                'name': cgsnapshot.name,
                'description': cgsnapshot.description
            }
        }

summary: - command "cinder consisgroup-show" and "cinder cgsnapshot-show" doesn't
- list volume-type, snapshot-volumes and snapshot volume-type
+ command "cinder cgsnapshot-show" doesn't list snapshot-volumes and
+ snapshot volume-type
description: updated
Revision history for this message
Sheel Rana (ranasheel2000) wrote :

Changed title and details of bug accordingly.

Changed in cinder:
status: New → Confirmed
Changed in cinder:
assignee: nobody → venkatamahesh (venkatamaheshkotha)
Changed in cinder:
assignee: venkatamahesh (venkatamaheshkotha) → nobody
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.