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

2014-12-04 Thread Niels de Vos


> 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

2014-12-03 Thread daan Hoogland

---
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

2014-02-25 Thread Wido den Hollander

---
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

2014-02-19 Thread Niels de Vos

---
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

2014-02-19 Thread Niels de Vos

---
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

2014-02-19 Thread Niels de Vos

---
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



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

2014-01-10 Thread Amogh Vasekar

---
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
> 
>



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

2013-12-01 Thread Niels de Vos

---
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