You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by rhtyd <gi...@git.apache.org> on 2016/06/23 07:00:54 UTC

[GitHub] cloudstack pull request #1595: ui: show resize volume button to all users

GitHub user rhtyd opened a pull request:

    https://github.com/apache/cloudstack/pull/1595

    ui: show resize volume button to all users

    ![screenshot from 2016-06-23 12-29-56](https://cloud.githubusercontent.com/assets/95203/16294438/436f6dbc-393e-11e6-91b5-cb2e49a01cc6.png)
    
    The resize volume is support on all major hypervisors (Xen, VMware, KVM).
    The hypervisor key is returned by the list volumes response only for admins
    but not for users or domain admin users. This removes the check, as the operation
    is supported on all major hypervisors that CloudStack supports.
    
    With this bug fix all users would see resize volume button in the UI.
    
    /cc @swill 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/shapeblue/cloudstack ui-resize-show-allusers

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1595.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1595
    
----
commit b68ae1949e9fa5ae376a0ae7b67c3349b9e83418
Author: Rohit Yadav <ro...@shapeblue.com>
Date:   2016-06-23T06:57:51Z

    ui: show resize volume button to all users
    
    The resize volume is support on all major hypervisors (Xen, VMware, KVM).
    The hypervisor key is returned by the list volumes response only for admins
    but not for users or domain admin users. This removes the check, as the operation
    is supported on all major hypervisors that CloudStack supports.
    
    With this changes all users would see resize volume button in the UI.
    
    Signed-off-by: Rohit Yadav <ro...@shapeblue.com>

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1595: ui: show resize volume button to all users

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1595
  
    @swill this is a simple UI bugfix, please consider merging for 4.9.0 rc2 as well.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1595: ui: show resize volume button to all users

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1595
  
    Thanks @remibergsma /cc @swill ?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1595: ui: show resize volume button to all users

Posted by syed <gi...@git.apache.org>.
Github user syed commented on the issue:

    https://github.com/apache/cloudstack/pull/1595
  
    LGTM :+1: 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1595: ui: show resize volume button to all users

Posted by wido <gi...@git.apache.org>.
Github user wido commented on the issue:

    https://github.com/apache/cloudstack/pull/1595
  
    LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1595: ui: show resize volume button to all users

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/cloudstack/pull/1595


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1595: ui: show resize volume button to all users

Posted by remibergsma <gi...@git.apache.org>.
Github user remibergsma commented on the issue:

    https://github.com/apache/cloudstack/pull/1595
  
    Thanks @rhtyd nice fix. Tested it and LGTM.
    
    Before:
    ![screen shot 2016-06-23 at 21 45 41 pm](https://cloud.githubusercontent.com/assets/1630096/16317719/c5662704-398c-11e6-92e4-e08addd892f4.png)
    
    After:
    ![screen shot 2016-06-23 at 21 47 08 pm](https://cloud.githubusercontent.com/assets/1630096/16317715/c184037c-398c-11e6-8b32-6a8f2760bde8.png)



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---