[GitHub] cloudstack-docs-admin pull request: Update provisioning.rst

2014-02-13 Thread sedukull
GitHub user sedukull opened a pull request: https://github.com/apache/cloudstack-docs-admin/pull/1 Update provisioning.rst Edited a link You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/cloudstack-docs-admin patch-1

[GitHub] cloudstack-docs-rn pull request: Update rnotes.rst

2014-02-13 Thread sedukull
GitHub user sedukull opened a pull request: https://github.com/apache/cloudstack-docs-rn/pull/1 Update rnotes.rst Added few changes to alignments. You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/cloudstack-docs-rn patch-1

[GitHub] cloudstack-docs-install pull request: Update optional_installation...

2014-02-13 Thread sedukull
GitHub user sedukull opened a pull request: https://github.com/apache/cloudstack-docs-install/pull/3 Update optional_installation.rst Fixed Few alignment issues You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/cloudstack

[GitHub] cloudstack-docs-install pull request: Update optional_installation...

2014-02-13 Thread sedukull
Github user sedukull commented on the pull request: https://github.com/apache/cloudstack-docs-install/pull/3#issuecomment-35002947 Fixed Few alignment issues.

[GitHub] cloudstack-docs-install pull request: Update optional_installation...

2014-02-13 Thread sedukull
Github user sedukull closed the pull request at: https://github.com/apache/cloudstack-docs-install/pull/3

[GitHub] cloudstack-docs-install pull request: Update optional_installation...

2014-02-13 Thread sedukull
Github user sedukull commented on the pull request: https://github.com/apache/cloudstack-docs-install/pull/3#issuecomment-35003054 Not yet applied

[GitHub] cloudstack-docs-install pull request: Update optional_installation...

2014-02-13 Thread sedukull
GitHub user sedukull reopened a pull request: https://github.com/apache/cloudstack-docs-install/pull/3 Update optional_installation.rst Fixed Few alignment issues You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/cloudstack

[GitHub] cloudstack-docs-rn pull request: Update rnotes.rst

2014-02-13 Thread sedukull
Github user sedukull closed the pull request at: https://github.com/apache/cloudstack-docs-rn/pull/1

[GitHub] cloudstack-docs-install pull request: Update hypervisor_installati...

2014-02-13 Thread sedukull
GitHub user sedukull opened a pull request: https://github.com/apache/cloudstack-docs-install/pull/4 Update hypervisor_installation.rst Fixed few alignment issues You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35534192 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,473 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35534267 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,473 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35605705 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,487 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35605790 --- Diff: reporter/usage-report-collector.py --- @@ -0,0 +1,64 @@ +#!/usr/bin/env python +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35605766 --- Diff: reporter/usage-report-collector.py --- @@ -0,0 +1,64 @@ +#!/usr/bin/env python +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35605842 --- Diff: reporter/usage-report-collector.py --- @@ -0,0 +1,64 @@ +#!/usr/bin/env python +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35606107 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,487 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8677: Call-home functionality ...

2015-07-28 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/625#discussion_r35627322 --- Diff: server/src/org/apache/cloudstack/report/UsageReporter.java --- @@ -0,0 +1,487 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36608900 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtCopyFileInVmCommandWrapper.java --- @@ -0,0 +1,59

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36609019 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -1279,6 +1285,27 @@ private static DiskTO

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36609071 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -1279,6 +1285,27 @@ private static DiskTO

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36609212 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtCopyFileInVmCommandWrapper.java --- @@ -0,0 +1,59

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36609308 --- Diff: plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java --- @@ -595,6 +597,25 @@ private

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36611175 --- Diff: server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java --- @@ -1425,19 +1423,39 @@ public boolean finalizeCommandsOnStart(Commands

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36611238 --- Diff: server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java --- @@ -1425,19 +1423,39 @@ public boolean finalizeCommandsOnStart(Commands

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36611512 --- Diff: plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java --- @@ -595,6 +597,25 @@ private

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36611661 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtCopyFileInVmCommandWrapper.java --- @@ -0,0 +1,59

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36704745 --- Diff: server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java --- @@ -1425,19 +1423,39 @@ public boolean finalizeCommandsOnStart(Commands

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36705033 --- Diff: systemvm/js/ajaxviewer.js --- @@ -184,7 +205,7 @@ KeyboardMapper.prototype = { if(eventType == AjaxViewer.KEY_UP

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36705202 --- Diff: plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java --- @@ -595,6 +597,25 @@ private

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36705320 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtCopyFileInVmCommandWrapper.java --- @@ -0,0 +1,59

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-10 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36705867 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtCopyFileInVmCommandWrapper.java --- @@ -0,0 +1,59

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-12 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36832023 --- Diff: client/pom.xml --- @@ -478,6 +478,12 @@ + --- End

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-12 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36839282 --- Diff: client/pom.xml --- @@ -478,6 +478,12 @@ + --- End

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

2015-08-12 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/685#discussion_r36855234 --- Diff: setup/db/db/schema-452to460.sql --- @@ -353,6 +353,8 @@ CREATE VIEW `cloud`.`user_vm_view` AS `cloud`.`user_vm_details

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

2015-08-12 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/685#discussion_r36855422 --- Diff: setup/db/db/schema-452to460.sql --- @@ -353,6 +353,8 @@ CREATE VIEW `cloud`.`user_vm_view` AS `cloud`.`user_vm_details

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

2015-08-12 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/685#discussion_r36855515 --- Diff: setup/db/db/schema-452to460.sql --- @@ -353,6 +353,8 @@ CREATE VIEW `cloud`.`user_vm_view` AS `cloud`.`user_vm_details

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

2015-08-12 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/685#discussion_r36855563 --- Diff: setup/db/db/schema-452to460.sql --- @@ -353,6 +353,8 @@ CREATE VIEW `cloud`.`user_vm_view` AS `cloud`.`user_vm_details

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

2015-08-12 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/685#discussion_r36857039 --- Diff: setup/db/db/schema-452to460.sql --- @@ -353,6 +353,8 @@ CREATE VIEW `cloud`.`user_vm_view` AS `cloud`.`user_vm_details

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2015-08-13 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r36952117 --- Diff: client/pom.xml --- @@ -478,6 +478,12 @@ + --- End

[GitHub] cloudstack-www pull request: Python script to generate pmc and com...

2015-08-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack-www/pull/14#discussion_r38068341 --- Diff: makewholist.py --- @@ -0,0 +1,68 @@ +#!/usr/bin/env python +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] cloudstack-www pull request: Python script to generate pmc and com...

2015-08-27 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack-www/pull/14#discussion_r38068366 --- Diff: makewholist.py --- @@ -0,0 +1,68 @@ +#!/usr/bin/env python +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] cloudstack-www pull request: Python script to generate pmc and com...

2015-08-27 Thread sedukull
Github user sedukull commented on the pull request: https://github.com/apache/cloudstack-www/pull/14#issuecomment-135375292 Was looking to be consistent when referring to global variables, but otherwise LGTM. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: CLOUDSTACK-8727: API call listVirtualMach...

2015-08-28 Thread sedukull
Github user sedukull commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/685#discussion_r38192659 --- Diff: setup/db/db/schema-452to460.sql --- @@ -354,6 +354,10 @@ CREATE VIEW `cloud`.`user_vm_view` AS left join `cloud

[GitHub] cloudstack pull request: Fixing findbugs error due to PR #755 merg...

2015-09-06 Thread sedukull
Github user sedukull commented on the pull request: https://github.com/apache/cloudstack/pull/779#issuecomment-138193843 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