[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1579 @blueorangutan package --- 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

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @blueorangutan test matrix --- 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

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @blueorangutan test matrix --- 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

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @blueorangutan package --- 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

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @blueorangutan package --- 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

[GitHub] cloudstack issue #1772: CLOUDSTACK-9627 Fix template sync for region store.

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1772 @syed can you amend the git commit message with this format to include the JIRA id: `CLOUDSTACK-xxx: one line summary`? --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1793 Thanks @sateesh-chodapuneedi I'll merge this now. --- 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

[GitHub] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1793 @sateesh-chodapuneedi is the guest_os_name `windows7Server64Guest` known by vmware versions 4.x, 5.x? The sql update will change guest_os_name for vmware versions 4.0, 4.1, 5.0, 5.1, 5.5 and 6.0

[GitHub] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1793 Test LGTM. One of the travis jobs took time while uploading the cache, but it passed all component tests: https://travis-ci.org/apache/cloudstack/jobs/179401939 Since, it's a simpl

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @blueorangutan test matrix --- 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

[GitHub] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1793 @blueorangutan test centos7 vmware-55u3 --- 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

[GitHub] cloudstack issue #1794: added more guest os

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1794 @karuturi this looks useful, can you get a JIRA id for this and amend the git commit summary with the ID. Can you also consider changing the PR's base branch to 4.9? --- If your project i

[GitHub] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1793 Thanks @sateesh-chodapuneedi @blueorangutan package --- 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

[GitHub] cloudstack issue #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1710 Tests 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 enabled and wishes so, or if

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @blueorangutan package --- 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

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @blueorangutan package --- 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

[GitHub] cloudstack issue #1752: [4.8] Latest health test run

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1752 @blueorangutan package --- 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

[GitHub] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1793 @sateesh-chodapuneedi thanks, this looks useful. Can you rebase against 4.9, and change the PR's base branch to 4.9? --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1755: CLOUDSTACK-9584: run component tests in Travis run

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1755 Thanks @borisstoyanov I'll proceed with merging this as Travis is all green now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1357: CLOUDSTACK-8958: release dedicated ip range in domai...

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1357 @ustcweizhou can you rebase against 4.8/4.9, change the base branch to 4.8/4.9? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack issue #1755: CLOUDSTACK-9584: run component tests in Travis run

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1755 @borisstoyanov okay, though changes in smoketests have been confirmed by Travis test results as well. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack issue #1755: CLOUDSTACK-9584: run component tests in Travis run

2016-11-28 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1755 @borisstoyanov thanks, but a Trillian test is not necessary as all changes are related to Travis. Since Travis is green, I'll need a couple of some lgtms to proceed with merging this. -

[GitHub] cloudstack issue #1755: CLOUDSTACK-9584: run component tests in Travis run

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1755 @abhinandanprateek @murali-reddy @jburwell @serg38 @nvazquez @syed @koushik-das @karuturi @milamberspace @wido @JayapalUradi @K0zka @rafaelweingartner @yvsubhash -- ping, please help review this

[GitHub] cloudstack issue #1623: CLOUDSTACK-9317: Enable/disable static NAT associate...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1623 @abhinandanprateek @murali-reddy ping --- 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

[GitHub] cloudstack issue #1637: Command route not available on CentOS 7

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1637 @lmrv ping --- 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

[GitHub] cloudstack issue #1655: Fix ajaxviewer.js to solve console on Firefox

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1655 @lygutas ping, can you get a JIRA id for this and rebase the branch to 4.8 or 4.9, and change the PR's base branch to 4.8 or 4.9? Thanks. --- If your project is set up for it, you can rep

[GitHub] cloudstack issue #1722: CLOUDSTACK-9558 Cleanup the snapshots on the primary...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1722 @blueorangutan package --- 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

[GitHub] cloudstack issue #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1706 @murali-reddy can you help review this? thanks. --- 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

[GitHub] cloudstack issue #1781: CLOUDSTACK-9612: Fixed issue in restarting redundant...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1781 @jayapalu this looks useful, can you change PR's base branch to 4.9? Thanks. @blueorangutan package --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1782: CLOUDSTACK-9617: Fixed enabling remote access after ...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1782 @murali-reddy can you help review this? @jayapalu this looks useful, can you rebase against 4.9 and change PR's base branch to 4.9? --- If your project is set up for it, you can rep

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1783 @jayapalu can you rebase against 4.9 and change PR's base branch to 4.9? Thanks. @murali-reddy can you help review this? --- If your project is set up for it, you can reply to this emai

[GitHub] cloudstack issue #1784: CS-505: Marvin test to check VR internal DNS Service

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1784 @blueorangutan test centos7 vmware-55u3 --- 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

[GitHub] cloudstack issue #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1710 @blueorangutan test centos7 xenserver-65sp1 --- 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

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Migrate master to Spring 4.x

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1638 @blueorangutan test --- 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

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @murali-reddy can you help investigate the failure with `test_router_dhcp_opts` --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @serg38 I looked across PRs merged since 23rd Nov, and found an issue with fwd-merging a PR that caused an issue in CsDhcp.py. I've fixed the issue now, and kicked test runs. So, far it

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan test --- 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

[GitHub] cloudstack issue #1791: CLOUDSTACK-9622 Localisation for 'Project' label on ...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1791 LGTM, tested manually similar changes as seen in screenshots from @milamberspace I'll proceed with merging this now. --- If your project is set up for it, you can reply to this emai

[GitHub] cloudstack issue #1789: Update L10N files from Transifex (2016-11-27) for th...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1789 Thanks @milamberspace I'll proceed with the merge now. --- 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 doe

[GitHub] cloudstack issue #1789: Update L10N files from Transifex (2016-11-27) for th...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1789 LGTM, did manual tests. Cycled between various lang options, don't see any failures. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] cloudstack issue #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1710 Failure wrt debian pkg was due to an env issue. @blueorangutan test --- 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

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @blueorangutan test matrix --- 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

[GitHub] cloudstack issue #1791: CLOUDSTACK-9622 Localisation for 'Project' label on ...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1791 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 enabled and wishes so, or if the

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Migrate master to Spring 4.x

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1638 @blueorangutan package --- 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

[GitHub] cloudstack issue #1772: Fix template sync for region store.

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1772 @syed can you add a JIRA id for this and use that in the git commit summary. LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1790: CLOUDSTACK-9622 Localisation for 'Project' label on ...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1790 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 enabled and wishes so, or if the

[GitHub] cloudstack issue #1790: CLOUDSTACK-9622 Localisation for 'Project' label on ...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1790 @milamberspace can you change the base branch of the PR to 4.9, this looks useful for 4.9 as well. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @blueorangutan test --- 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

[GitHub] cloudstack issue #1784: CS-505: Marvin test to check VR internal DNS Service

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1784 @abhinandanprateek can you check if `test_router_dns.py` is similar to your test, you may also move your test case in that test file? Thanks. --- If your project is set up for it, you can reply

[GitHub] cloudstack issue #1784: CS-505: Marvin test to check VR internal DNS Service

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1784 @blueorangutan package --- 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

[GitHub] cloudstack issue #1789: Update L10N files from Transifex (2016-11-27) for th...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1789 Thanks @milamberspace can you also use that JIRA id in the git commit summary such as: `CLOUDSTACK-: one liner...` --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1787: fix syntax error introduced in CsDhcp.py

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1787 Thanks @murali-reddy I've already fixed this with #1788 and kicking tests on PR #1753 #1754 --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack issue #1788: systemvm: Fix regression from fwd-merging PR #1766

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1788 Notes: centos6 build failed due to an intermittent unit test that fails sometimes when clocks are skewed: ` HypervisorUtilsTest.checkVolumeFileForActivityTest:68 Didn't block long e

[GitHub] cloudstack issue #1789: Update L10N files from Transifex (2016-11-27) for th...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1789 @milamberspace thanks for the PR, can you get a JIRA id for this. I'll kick one manual test to see that UI is broken. @blueorangutan package --- If your project is set up for it

[GitHub] cloudstack issue #1778: CLOUDSTACK-9613: Unable to set NAT rules on any inte...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1778 Thanks @nitin-maharana a similar fix was merged with #1785 since it was for 4.9. Please close this PR. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1785: CLOUDSTACK-9416 : Enabling Static NAT on an associat...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1785 LGTM, this fixes the same issue as #1778. Since this is a pure UI change, manual testing with screenshot confirm the fix. I'll move with the merge now. The failure at Travis is not due to

[GitHub] cloudstack issue #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1710 @blueorangutan package --- 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

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan package --- 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

[GitHub] cloudstack issue #1752: [4.8] Latest health test run

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1752 @murali-reddy can you check why `test_router_dhcp_opts (test_router_dhcphosts.py)` failed, this test is failing intermittently on 4.9, master as well. --- If your project is set up for it, you

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @blueorangutan package --- 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

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @blueorangutan package --- 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

[GitHub] cloudstack issue #1788: systemvm: Fix regression from fwd-merging PR #1766

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1788 Merging this based on internal tests, the issue was tuple did not include `lease`. Merging this now. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request #1788: systemvm: Fix regression from fwd-merging PR ...

2016-11-27 Thread rhtyd
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1788 systemvm: Fix regression from fwd-merging PR #1766 Fixes merge conflict issue incorrectly fixed during a fwd-merge in 825935 from PR #1766. You can merge this pull request into a Git

[GitHub] cloudstack issue #1788: systemvm: Fix regression from fwd-merging PR #1766

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1788 @blueorangutan package --- 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

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @serg38 thanks, I'll bisect them today. --- 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 fe

[GitHub] cloudstack issue #1748: KVM enhancements for managed storage

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1748 @mike-tutkowski can you use the JIRA id in the commit summary as: `CLOUDSTACK-: one-line summary ... rest of the commit message... ` --- If your project is set up for it, you

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 Thanks @mike-tutkowski can you add a JIRA id for this, and use that id in the commit summary? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @blueorangutan test matrix --- 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

[GitHub] cloudstack issue #1752: [4.8] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1752 @blueorangutan test --- 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

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @serg38 thanks, that's my opinion as well, I'll kick one more round just to be sure, and will bisect recently merged changes and revert any changes if necessary. @blueorangutan t

[GitHub] cloudstack issue #1752: [4.8] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1752 @blueorangutan package --- 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

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @serg38 Likely environment related, I'll check that /cc @PaulAngus Meanwhile I'll kick another round. @blueorangutan package --- If your project is set up for it, you can rep

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @blueorangutan package --- 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

[GitHub] cloudstack issue #1765: Cloudstack 9586: When using local storage with Xense...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1765 @blueorangutan test centos7 xenserver-65sp1 --- 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

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan test --- 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

[GitHub] cloudstack issue #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1710 @blueorangutan test --- 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

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan package --- 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

[GitHub] cloudstack issue #1765: Cloudstack 9586: When using local storage with Xense...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1765 @blueorangutan package --- 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

[GitHub] cloudstack pull request #1765: Cloudstack 9586: When using local storage wit...

2016-11-25 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1765#discussion_r89576730 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/Xenserver625StorageProcessor.java --- @@ -100,36 +100,25 @@ protected

[GitHub] cloudstack issue #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1710 There were some intermittent errors seen, I'll re-kick tests. @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] cloudstack pull request #1710: CLOUDSTACK-9538: FIX failure in Deleting Snap...

2016-11-25 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1710#discussion_r89576049 --- Diff: engine/storage/snapshot/src/org/apache/cloudstack/storage/snapshot/XenserverSnapshotStrategy.java --- @@ -268,7 +268,9 @@ public boolean

[GitHub] cloudstack issue #1664: CLOUDSTACK-8676 Deploy user instance from vm snapsho...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1664 Thanks for sharing @sateesh-chodapuneedi --- 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

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1579 @nlivens @prashanthvarma @singalrahul please squash your changes, fix conflicts, rebase against latest master. Pending lgtm/review is requested. /cc @jburwell --- If your project is set up for

[GitHub] cloudstack issue #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1577 LGTM. Few failures are related to env, merging this now. --- 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

[GitHub] cloudstack issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multipl...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1760 @marcaurele an upgrade path such as `Upgrade4910to4920` would only work if your destination/final version is 4.9.2.0, have a look at the db upgrade checker workflow and you'll see how to tr

[GitHub] cloudstack issue #1733: CLOUDSTACK-9563 ExtractTemplate returns malformed UR...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1733 @blueorangutan test --- 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

[GitHub] cloudstack issue #1470: Make the generated json files unique to prevent conc...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1470 @remibergsma can you address comments from @jburwell, especially remove the `new String()`. I can help with further reviewing and testing once code changes are finalized, thanks. --- If your

[GitHub] cloudstack issue #1519: Cloudstack 9339: Virtual Routers do not handle Multi...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1519 @murali-reddy @abhinandanprateek ping, please review this one as well, thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack issue #1573: CLOUDSTACK-9396:fixing issue related to script test_...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1573 @nitt10prashant can you rebase against 4.9, change the base branch for the PR to 4.9. Thanks. /cc @jburwell --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1606: Allow CGN (RFC6598) to be used within a VPC

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1606 LGTM on tests and code review. @leprechau @kiwiflyer can we have a JIRA id for this and use that in the commit summary. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1623: CLOUDSTACK-9317: Enable/disable static NAT associate...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1623 @abhinandanprateek @murali-reddy and others -- can we have review on this, thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1727 @nvazquez can you look at comments from @koushik-das @serg38 do we have lgtms on this one? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1748: KVM enhancements for managed storage

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1748 @mike-tutkowski please get a JIRA id for this and use it in the commit summary, thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @mike-tutkowski please get a JIRA if for this and use that in the commit summary, thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1759: CLOUDSTACK-9589 vmName entries from host_details tab...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1759 Move this to 4.9 or master branch. --- 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

[GitHub] cloudstack issue #1765: Cloudstack 9586: When using local storage with Xense...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1765 @murali-reddy @abhinandanprateek @jburwell pinging for review continuation. --- 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

[GitHub] cloudstack issue #1738: CLOUDSTACK-9566 instance-id metadata for baremetal V...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1738 Thanks all, I'll proceed with the merging as build, travis and packaging works, without an explicit Trillian test or BVT as both don't run tests against baremetal and the change is a s

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1635 Test LGTM, the two failures are intermittent and env related. --- 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

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @blueorangutan test matrix --- 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

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @blueorangutan test matrix --- 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

<    4   5   6   7   8   9   10   11   12   13   >