[api-ref]Added 'bootable volume' filter for non-admin user

Bug #1536845 reported by OpenStack Infra
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cinder
Invalid
Medium
Unassigned
openstack-manuals
Fix Released
Low
Unassigned

Bug Description

https://review.openstack.org/268322
Dear bug triager. This bug was created since a commit was marked with DOCIMPACT.

commit 7ac7413ae31f1298af3d50f5b3e017f2af572ecd
Author: Sheel Rana <email address hidden>
Date: Sat Jan 16 01:53:49 2016 +0530

    Added 'bootable volume' filter for non-admin user

    During launch instance from Horizon, if non-admin
    user selects volume as source for launching
    instance, drop down list for volumes will also
    show non-bootable volumes. It is wrong behaviour.

    To fix this, 'bootable volume' search filter is
    enabled so that only bootable volumes get
    displayed in drop down list of volume.

    APIImpact

    DocImpact:Need to add bootable against
    query_volume_filters in OpenStack Configuration
    Reference

    Closes-Bug: #1524450

    Change-Id: If5bfbd73bbe02b13b76d7169ea16424493ac5fca

Changed in openstack-manuals:
status: New → Confirmed
importance: Undecided → Low
milestone: none → mitaka
tags: added: autogenerate-config-docs config-reference
Changed in openstack-manuals:
assignee: nobody → Sheel Rana (ranasheel2000)
Changed in openstack-manuals:
status: Confirmed → In Progress
Revision history for this message
Sheel Rana (ranasheel2000) wrote :
Revision history for this message
Sheel Rana (ranasheel2000) wrote :

Hi Venkatamahesh,

I am planning to add this parameter in doc/config-reference/source/tables/cinder-api.rst so that same can be reflected in
http://docs.openstack.org/liberty/config-reference/content/section_volume-misc.html
( /openstack-manuals/doc/config-reference/source/block-storage/volume-misc.rst-> include:: ../tables/cinder-api.rst)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on openstack-manuals (master)

Change abandoned by Sheel Rana (<email address hidden>) on branch: master
Review: https://review.openstack.org/271717
Reason: This change is abandoned because it seems everything gets updated through doc-tools and no manual change required for same..

Revision history for this message
Sheel Rana (ranasheel2000) wrote : Re: Added 'bootable volume' filter for non-admin user

Marking incomplete as requirements are not very much clear... everything seems automatically updated through doc-tools.
May be no change required for http://docs.openstack.org/liberty/config-reference/content/section_volume-misc.html through patchset.

Changed in openstack-manuals:
status: In Progress → Incomplete
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-manuals (master)

Fix proposed to branch: master
Review: https://review.openstack.org/271753

Revision history for this message
Gauvain Pocentek (gpocentek) wrote : Re: Added 'bootable volume' filter for non-admin user

https://review.openstack.org/271753 should be OK to handle the config-ref part. This change still has to be documented in the API site.

Changed in openstack-manuals:
assignee: Sheel Rana (ranasheel2000) → venkatamahesh (venkatamaheshkotha)
status: Incomplete → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on openstack-manuals (master)

Change abandoned by Sheel Rana (<email address hidden>) on branch: master
Review: https://review.openstack.org/271717
Reason: fix is ok, but in manuals they do it once for all changes required in different doc impacts.
So, have to abandon due to this.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-manuals (master)

Reviewed: https://review.openstack.org/271753
Committed: https://git.openstack.org/cgit/openstack/openstack-manuals/commit/?id=513a06f8a92c9d4d5da074fab19aed124cff7669
Submitter: Jenkins
Branch: master

commit 513a06f8a92c9d4d5da074fab19aed124cff7669
Author: Gauvain Pocentek <email address hidden>
Date: Sun Jan 24 12:04:15 2016 +0100

    [config-ref] Update the cinder tables

    Several patches have landed in cinder and an update is required.

    Note: the cinder.overrides is a temporary workaround for oslo.messaging.
    See https://review.openstack.org/#/c/263818/. This file can be removed
    once a new oslo.messaging release has landed.

    Change-Id: I8ebcc489c2c84ebc19c51e4712301d4f71660f06
    Partial-Bug: #1528332
    Partial-Bug: #1537321
    Partial-Bug: #1536845
    Partial-Bug: #1529720
    Partial-Bug: #1529669
    Partial-Bug: #1512198
    Partial-Bug: #1489231
    Partial-Bug: #1488415
    Partial-Bug: #1478650

Changed in openstack-manuals:
assignee: venkatamahesh (venkatamaheshkotha) → nobody
status: In Progress → Confirmed
Anne Gentle (annegentle)
Changed in openstack-api-site:
status: New → Confirmed
importance: Undecided → Medium
Chason Chan (chen-xing)
Changed in openstack-manuals:
status: Confirmed → Fix Released
Atsushi SAKAI (sakaia)
affects: openstack-api-site → cinder
summary: - Added 'bootable volume' filter for non-admin user
+ [api-ref]Added 'bootable volume' filter for non-admin user
tags: added: api-ref
Changed in cinder:
assignee: nobody → Sheel Rana (ranasheel2000)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/316587

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Sean McGinnis (<email address hidden>) on branch: master
Review: https://review.openstack.org/316587
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Changed in cinder:
assignee: Sheel Rana (ranasheel2000) → nobody
Revision history for this message
Jay Bryant (jsbryant) wrote :

This bug seems to have wandered horribly out of control. It was originally a Doc bug and has a bunch of other incomplete changes pushed against it. The config ref aspect seems to be addressed so I think we can close this.

Changed in cinder:
status: In Progress → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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