You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Thomas O'Dowd <tp...@cloudian.com> on 2013/08/06 08:07:06 UTC

Review Request 13312: Add help docs for S3 secondary storage configuration.

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

Review request for cloudstack.


Repository: cloudstack-git


Description
-------

Add help docs to S3 secondary storage configuration.
    
When adding an S3 secondary storage, the administrator will now be shown help for each input field making it friendlier to use.


Diffs
-----

  ui/scripts/docs.js 03318b8 
  ui/scripts/system.js 92fa867 

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


Testing
-------

Clean compile of 4.2. Logged in as admin. Clicked Infrastrucure. Add Secondary Storage, select S3 as the provider and used the UI while viewing help.


Thanks,

Thomas O'Dowd


Re: Review Request 13312: CLOUDSTACK-3211 - Add help docs for S3 secondary storage configuration.

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13312/#review24705
-----------------------------------------------------------


Commit a764cb91b30cb998aa391a1dc897cbafec26f34a in branch refs/heads/master from Thomas O'Dowd
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=a764cb9 ]

CLOUDSTACK-3211: Add help docs to S3 secondary storage configuration.

When adding an S3 secondary storage, the administrator will
now be shown help for each input field making it friendlier
to use.

Signed-off-by: Prasanna Santhanam <ts...@apache.org>
(cherry picked from commit 74643c9561b1ea564f180c3780ef762e97e7c10b)


- ASF Subversion and Git Services


On Aug. 6, 2013, 7:10 a.m., Thomas O'Dowd wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13312/
> -----------------------------------------------------------
> 
> (Updated Aug. 6, 2013, 7:10 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-3211
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Add help docs to S3 secondary storage configuration.
>     
> When adding an S3 secondary storage, the administrator will now be shown help for each input field making it friendlier to use.
> 
> 
> Diffs
> -----
> 
>   ui/scripts/docs.js 03318b8 
>   ui/scripts/system.js 92fa867 
> 
> Diff: https://reviews.apache.org/r/13312/diff/
> 
> 
> Testing
> -------
> 
> Clean compile of 4.2. Logged in as admin. Clicked Infrastrucure. Add Secondary Storage, select S3 as the provider and used the UI while viewing help.
> 
> 
> Thanks,
> 
> Thomas O'Dowd
> 
>


Re: Review Request 13312: CLOUDSTACK-3211 - Add help docs for S3 secondary storage configuration.

Posted by Prasanna Santhanam <ts...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13312/#review24707
-----------------------------------------------------------

Ship it!


Applied on 4.2 and master

- Prasanna Santhanam


On Aug. 6, 2013, 7:10 a.m., Thomas O'Dowd wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13312/
> -----------------------------------------------------------
> 
> (Updated Aug. 6, 2013, 7:10 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-3211
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Add help docs to S3 secondary storage configuration.
>     
> When adding an S3 secondary storage, the administrator will now be shown help for each input field making it friendlier to use.
> 
> 
> Diffs
> -----
> 
>   ui/scripts/docs.js 03318b8 
>   ui/scripts/system.js 92fa867 
> 
> Diff: https://reviews.apache.org/r/13312/diff/
> 
> 
> Testing
> -------
> 
> Clean compile of 4.2. Logged in as admin. Clicked Infrastrucure. Add Secondary Storage, select S3 as the provider and used the UI while viewing help.
> 
> 
> Thanks,
> 
> Thomas O'Dowd
> 
>


Re: Review Request 13312: CLOUDSTACK-3211 - Add help docs for S3 secondary storage configuration.

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13312/#review24706
-----------------------------------------------------------


Commit 7fd71d708a34556eb9ab168e52f4471a8c2acdc8 in branch refs/heads/4.2 from Thomas O'Dowd
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=7fd71d7 ]

CLOUDSTACK-3211: Add help docs to S3 secondary storage configuration.

When adding an S3 secondary storage, the administrator will
now be shown help for each input field making it friendlier
to use.

Signed-off-by: Prasanna Santhanam <ts...@apache.org>


- ASF Subversion and Git Services


On Aug. 6, 2013, 7:10 a.m., Thomas O'Dowd wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13312/
> -----------------------------------------------------------
> 
> (Updated Aug. 6, 2013, 7:10 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-3211
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Add help docs to S3 secondary storage configuration.
>     
> When adding an S3 secondary storage, the administrator will now be shown help for each input field making it friendlier to use.
> 
> 
> Diffs
> -----
> 
>   ui/scripts/docs.js 03318b8 
>   ui/scripts/system.js 92fa867 
> 
> Diff: https://reviews.apache.org/r/13312/diff/
> 
> 
> Testing
> -------
> 
> Clean compile of 4.2. Logged in as admin. Clicked Infrastrucure. Add Secondary Storage, select S3 as the provider and used the UI while viewing help.
> 
> 
> Thanks,
> 
> Thomas O'Dowd
> 
>


Re: Review Request 13312: CLOUDSTACK-3211 - Add help docs for S3 secondary storage configuration.

Posted by Thomas O'Dowd <tp...@cloudian.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13312/
-----------------------------------------------------------

(Updated Aug. 6, 2013, 4:10 p.m.)


Review request for cloudstack.


Changes
-------

This patch addresses CLOUDSTACK-3211. Added to bugs section and prefixed to summary.


Summary (updated)
-----------------

CLOUDSTACK-3211 - Add help docs for S3 secondary storage configuration.


Bugs: CLOUDSTACK-3211


Repository: cloudstack-git


Description
-------

Add help docs to S3 secondary storage configuration.
    
When adding an S3 secondary storage, the administrator will now be shown help for each input field making it friendlier to use.


Diffs
-----

  ui/scripts/docs.js 03318b8 
  ui/scripts/system.js 92fa867 

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


Testing
-------

Clean compile of 4.2. Logged in as admin. Clicked Infrastrucure. Add Secondary Storage, select S3 as the provider and used the UI while viewing help.


Thanks,

Thomas O'Dowd