You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Niels de Vos <nd...@redhat.com> on 2013/12/01 15:07:41 UTC

Review Request 15933: Add Gluster to the list of protocols in the Management Server

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15933/
-----------------------------------------------------------

Review request for cloudstack.


Repository: cloudstack-git


Description
-------

Gluster can now be used for Primary Storage just like NFS. This change adds the
Gluster protocol to the Management Server:

    Infrastructure -> Primary Storage -> Add Primary Storage

And also add the option to create Primary Storage on Gluster when
following the 'Add Zone' wizard from:

    Infrastructure -> Zones -> Add Zone


Diffs
-----

  client/WEB-INF/classes/resources/messages.properties eb11fe4 
  ui/dictionary.jsp ab37781 
  ui/scripts/sharedFunctions.js 87255cb 
  ui/scripts/system.js f632887 
  ui/scripts/zoneWizard.js 617c6e6 

Diff: https://reviews.apache.org/r/15933/diff/


Testing
-------

Some screenshots and verification:
- http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html


Thanks,

Niels de Vos


Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

Posted by Amogh Vasekar <am...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15933/#review31527
-----------------------------------------------------------


Reminder - 
Hi,
This review has been pending for long. Please add relevant reviewers, or seek volunteers from the community to review.
Thanks!

- Amogh Vasekar


On Dec. 1, 2013, 2:07 p.m., Niels de Vos wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15933/
> -----------------------------------------------------------
> 
> (Updated Dec. 1, 2013, 2:07 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Gluster can now be used for Primary Storage just like NFS. This change adds the
> Gluster protocol to the Management Server:
> 
>     Infrastructure -> Primary Storage -> Add Primary Storage
> 
> And also add the option to create Primary Storage on Gluster when
> following the 'Add Zone' wizard from:
> 
>     Infrastructure -> Zones -> Add Zone
> 
> 
> Diffs
> -----
> 
>   client/WEB-INF/classes/resources/messages.properties eb11fe4 
>   ui/dictionary.jsp ab37781 
>   ui/scripts/sharedFunctions.js 87255cb 
>   ui/scripts/system.js f632887 
>   ui/scripts/zoneWizard.js 617c6e6 
> 
> Diff: https://reviews.apache.org/r/15933/diff/
> 
> 
> Testing
> -------
> 
> Some screenshots and verification:
> - http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html
> 
> 
> Thanks,
> 
> Niels de Vos
> 
>


Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

Posted by Niels de Vos <nd...@redhat.com>.

> On Dec. 3, 2014, 1:31 p.m., daan Hoogland wrote:
> > Niels, I see several Gluster fixes of your hand in the (4.4) code. Is this one still needed? please close if no, thanks

I think this has been merged already (confirmed with 4.4.2).


- Niels


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15933/#review63663
-----------------------------------------------------------


On Feb. 19, 2014, 9:27 a.m., Niels de Vos wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15933/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2014, 9:27 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Gluster can now be used for Primary Storage just like NFS. This change adds the
> Gluster protocol to the Management Server:
> 
>     Infrastructure -> Primary Storage -> Add Primary Storage
> 
> And also add the option to create Primary Storage on Gluster when
> following the 'Add Zone' wizard from:
> 
>     Infrastructure -> Zones -> Add Zone
> 
> 
> Diffs
> -----
> 
>   client/WEB-INF/classes/resources/messages.properties bd4a27d 
>   ui/dictionary.jsp 7ccb466 
>   ui/scripts/sharedFunctions.js 2a15967 
>   ui/scripts/system.js 8159124 
>   ui/scripts/zoneWizard.js fd5705b 
> 
> Diff: https://reviews.apache.org/r/15933/diff/
> 
> 
> Testing
> -------
> 
> Some screenshots and verification:
> - http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html
> 
> 
> Thanks,
> 
> Niels de Vos
> 
>


Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

Posted by daan Hoogland <da...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15933/#review63663
-----------------------------------------------------------


Niels, I see several Gluster fixes of your hand in the (4.4) code. Is this one still needed? please close if no, thanks

- daan Hoogland


On Feb. 19, 2014, 8:27 a.m., Niels de Vos wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15933/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2014, 8:27 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Gluster can now be used for Primary Storage just like NFS. This change adds the
> Gluster protocol to the Management Server:
> 
>     Infrastructure -> Primary Storage -> Add Primary Storage
> 
> And also add the option to create Primary Storage on Gluster when
> following the 'Add Zone' wizard from:
> 
>     Infrastructure -> Zones -> Add Zone
> 
> 
> Diffs
> -----
> 
>   client/WEB-INF/classes/resources/messages.properties bd4a27d 
>   ui/dictionary.jsp 7ccb466 
>   ui/scripts/sharedFunctions.js 2a15967 
>   ui/scripts/system.js 8159124 
>   ui/scripts/zoneWizard.js fd5705b 
> 
> Diff: https://reviews.apache.org/r/15933/diff/
> 
> 
> Testing
> -------
> 
> Some screenshots and verification:
> - http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html
> 
> 
> Thanks,
> 
> Niels de Vos
> 
>


Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

Posted by Wido den Hollander <wi...@widodh.nl>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15933/#review35382
-----------------------------------------------------------

Ship it!


Ship It!

- Wido den Hollander


On Feb. 19, 2014, 8:27 a.m., Niels de Vos wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15933/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2014, 8:27 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Gluster can now be used for Primary Storage just like NFS. This change adds the
> Gluster protocol to the Management Server:
> 
>     Infrastructure -> Primary Storage -> Add Primary Storage
> 
> And also add the option to create Primary Storage on Gluster when
> following the 'Add Zone' wizard from:
> 
>     Infrastructure -> Zones -> Add Zone
> 
> 
> Diffs
> -----
> 
>   client/WEB-INF/classes/resources/messages.properties bd4a27d 
>   ui/dictionary.jsp 7ccb466 
>   ui/scripts/sharedFunctions.js 2a15967 
>   ui/scripts/system.js 8159124 
>   ui/scripts/zoneWizard.js fd5705b 
> 
> Diff: https://reviews.apache.org/r/15933/diff/
> 
> 
> Testing
> -------
> 
> Some screenshots and verification:
> - http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html
> 
> 
> Thanks,
> 
> Niels de Vos
> 
>


Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

Posted by Niels de Vos <nd...@redhat.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15933/
-----------------------------------------------------------

(Updated Feb. 19, 2014, 9:27 a.m.)


Review request for cloudstack.


Repository: cloudstack-git


Description
-------

Gluster can now be used for Primary Storage just like NFS. This change adds the
Gluster protocol to the Management Server:

    Infrastructure -> Primary Storage -> Add Primary Storage

And also add the option to create Primary Storage on Gluster when
following the 'Add Zone' wizard from:

    Infrastructure -> Zones -> Add Zone


Diffs
-----

  client/WEB-INF/classes/resources/messages.properties bd4a27d 
  ui/dictionary.jsp 7ccb466 
  ui/scripts/sharedFunctions.js 2a15967 
  ui/scripts/system.js 8159124 
  ui/scripts/zoneWizard.js fd5705b 

Diff: https://reviews.apache.org/r/15933/diff/


Testing
-------

Some screenshots and verification:
- http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html


Thanks,

Niels de Vos


Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

Posted by Niels de Vos <nd...@redhat.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15933/
-----------------------------------------------------------

(Updated Feb. 19, 2014, 9:26 a.m.)


Review request for cloudstack.


Repository: cloudstack-git


Description
-------

Gluster can now be used for Primary Storage just like NFS. This change adds the
Gluster protocol to the Management Server:

    Infrastructure -> Primary Storage -> Add Primary Storage

And also add the option to create Primary Storage on Gluster when
following the 'Add Zone' wizard from:

    Infrastructure -> Zones -> Add Zone


Diffs
-----

  client/WEB-INF/classes/resources/messages.properties bd4a27d 
  ui/dictionary.jsp 7ccb466 
  ui/scripts/sharedFunctions.js 2a15967 
  ui/scripts/system.js 8159124 
  ui/scripts/zoneWizard.js fd5705b 

Diff: https://reviews.apache.org/r/15933/diff/


Testing
-------

Some screenshots and verification:
- http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html


Thanks,

Niels de Vos


Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

Posted by Niels de Vos <nd...@redhat.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15933/
-----------------------------------------------------------

(Updated Feb. 19, 2014, 9:26 a.m.)


Review request for cloudstack.


Changes
-------

Rebased to recent master (4.4).


Repository: cloudstack-git


Description
-------

Gluster can now be used for Primary Storage just like NFS. This change adds the
Gluster protocol to the Management Server:

    Infrastructure -> Primary Storage -> Add Primary Storage

And also add the option to create Primary Storage on Gluster when
following the 'Add Zone' wizard from:

    Infrastructure -> Zones -> Add Zone


Diffs (updated)
-----

  client/WEB-INF/classes/resources/messages.properties bd4a27d 
  ui/dictionary.jsp 7ccb466 
  ui/scripts/sharedFunctions.js 2a15967 
  ui/scripts/system.js 8159124 
  ui/scripts/zoneWizard.js fd5705b 

Diff: https://reviews.apache.org/r/15933/diff/


Testing
-------

Some screenshots and verification:
- http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html


Thanks,

Niels de Vos