[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-29 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/206#issuecomment-97313324 since travis and jenkins are happy, I am merging this pull request. @bhaisaab the js messaging/i18n changes will be handled through separate pull requests. ---

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/206 --- 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

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/199#issuecomment-97306176 closing this. created a new one #206 --- 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

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread karuturi
Github user karuturi closed the pull request at: https://github.com/apache/cloudstack/pull/199 --- 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 i

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/206 [MERGE] Browser based template or volume upload (resolved conflicts with master) Proposal discussion: http://markmail.org/message/ne6ucmnfqhkddnhu FS and testplan: https://cwiki.apache.org

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29309160 --- Diff: engine/storage/image/src/org/apache/cloudstack/storage/image/store/TemplateObject.java --- @@ -28,7 +28,6 @@ import org.apache.cloudstac

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29279216 --- Diff: engine/storage/image/src/org/apache/cloudstack/storage/image/store/TemplateObject.java --- @@ -28,7 +28,6 @@ import org.apache.cloudstac

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread rags22489664
Github user rags22489664 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29233570 --- Diff: ui/scripts/templates.js --- @@ -906,7 +1229,7 @@ } },

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread rags22489664
Github user rags22489664 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29233545 --- Diff: ui/scripts/templates.js --- @@ -97,17 +100,24 @@ reorder: cloudStack.api.actions.sort('updateTemplate', 'templates

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread rags22489664
Github user rags22489664 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29233530 --- Diff: ui/scripts/storage.js --- @@ -68,7 +68,7 @@ actions: { // Add volume

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29224917 --- Diff: systemvm/patches/debian/config/etc/init.d/cloud-early-config --- @@ -1156,6 +1156,36 @@ setup_secstorage() { fi setup_apache2 $ETH

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29224874 --- Diff: engine/storage/image/src/org/apache/cloudstack/storage/image/store/TemplateObject.java --- @@ -28,7 +28,6 @@ import org.apache.cloudstac

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29224805 --- Diff: services/secondary-storage/server/src/org/apache/cloudstack/storage/resource/HttpUploadServerHandler.java --- @@ -0,0 +1,295 @@ +// Licens

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/199#issuecomment-96971442 Overall looks good, I'm yet to test it. Can you share any test reports against latest master with this branch? --- If your project is set up for it, you can reply t

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29222391 --- Diff: ui/scripts/templates.js --- @@ -906,7 +1229,7 @@ } },

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29222317 --- Diff: ui/scripts/storage.js --- @@ -68,7 +68,7 @@ actions: { // Add volume

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29222349 --- Diff: ui/scripts/templates.js --- @@ -97,17 +100,24 @@ reorder: cloudStack.api.actions.sort('updateTemplate', 'templates'),

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r2911 --- Diff: systemvm/patches/debian/config/etc/init.d/cloud-early-config --- @@ -1156,6 +1156,36 @@ setup_secstorage() { fi setup_apache2 $ETH

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29222011 --- Diff: services/secondary-storage/server/src/org/apache/cloudstack/storage/resource/HttpUploadServerHandler.java --- @@ -0,0 +1,295 @@ +// Licens

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-28 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/199#discussion_r29221712 --- Diff: engine/storage/image/src/org/apache/cloudstack/storage/image/store/TemplateObject.java --- @@ -28,7 +28,6 @@ import org.apache.cloudstac

[GitHub] cloudstack pull request: [MERGE] Browser based template or volume ...

2015-04-27 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/199 [MERGE] Browser based template or volume upload We worked on below proposal for Browser based template or volume upload. development effort from @bvbharat, Koushik Das and @karuturi UI h