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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1733 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply

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

[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 pull request #1738: CLOUDSTACK-9566 instance-id metadata for bare...

2016-11-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1738 --- 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

[GitHub] cloudstack pull request #1635: CLOUDSTACK-9451

2016-11-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1635 --- 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

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

2016-11-24 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1519 @LGTM on code review. --- 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 #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 pull request #1765: Cloudstack 9586: When using local storage wit...

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

[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 #1754: [4.10] Latest health test run

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 Trillian test result (tid-433) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 45986 seconds Marvin logs:

[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 #1760: CLOUDSTACK-9593: userdata: enforce data is a multipl...

2016-11-24 Thread marcaurele
Github user marcaurele commented on the issue: https://github.com/apache/cloudstack/pull/1760 @rhtyd The DB update class for the DatabaseUpgradeChecker doesn't work on Travis. Is there something I'm doing wrong, or missed something? --- If your project is set up for it, you can

[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 single line

[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 #1738: CLOUDSTACK-9566 instance-id metadata for baremetal V...

2016-11-24 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1738 LGTM as well --- 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

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

2016-11-24 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1781 @jayapalu we had the same issue before. we changed the code in line 634 from setIsReduntant to setIsRedundant to fix it. ``` diff --git

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

2016-11-24 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1778 @nitin-maharana so I suggest to use "networkId" instead of nicId, it is simpler change. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #827: BUG-ID: CLOUDSTACK-8849: Usage job should stop usage ...

2016-11-24 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/827 @yvsubhash yes, we will miss some records if we skip the exception, but we will miss all others records if we stop the usage server. For me, it is important that usage server will still be

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

2016-11-24 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1738 Code changes 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

[GitHub] cloudstack pull request #1781: CLOUDSTACK-9612: Fixed issue in restarting re...

2016-11-24 Thread jayapalu
GitHub user jayapalu opened a pull request: https://github.com/apache/cloudstack/pull/1781 CLOUDSTACK-9612: Fixed issue in restarting redundant network with cle… Rvr Network which is updated from the isolated network is failed to restart with cleanup set. This is

[GitHub] cloudstack issue #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-24 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1762 @rafaelweingartner Tried tracing where deadlock 5 originated. It seems both transactions are part of the same method fetchNewPublicIp in IpAddressManagerImpl . Transactions are executed on

[GitHub] cloudstack issue #827: BUG-ID: CLOUDSTACK-8849: Usage job should stop usage ...

2016-11-24 Thread yvsubhash
Github user yvsubhash commented on the issue: https://github.com/apache/cloudstack/pull/827 @ustcweizhou In case we continue with usage generation, we may miss out some of the records which actually resulted in exception and would in turn lead to wrong usage numbers. stopping after

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

2016-11-24 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1778 @ustcweizhou : Yes, we can directly pass networkid instead nicId. They handled the same way in case of multiple vm. --- If your project is set up for it, you can reply to this email and

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 Trillian test result (tid-432) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 33969 seconds Marvin logs:

[GitHub] cloudstack issue #1769: tools: fix Dockerfile

2016-11-24 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1769 Changes look good to me, but waiting for @pdion891 to tell us more :) --- 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 #1780: Fix the KVM hypervisor's handling of not-really-SCSI...

2016-11-24 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1780 These small fixes *can* be very dangerous. So wondering how we can test this easily. --- 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 #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-24 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1762 @rafaelweingartner Looks like the deadlocks 2 and 3 are the same. I scanned our production log and since last December we had 6400 deadlocks. Out of them close to 6000 were Deadlock 1 20

[GitHub] cloudstack issue #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-24 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1580 @rhtyd I have resolved all your comments and updated the PR. Moreover, all the tests that are run on this PR are now fully successful (blueorangutan tests + added Marvin tests in this

[GitHub] cloudstack issue #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1762 Thanks, @serg38. Looking at the SQLs you posted. We could start to discuss whether or not some SQLs statements need locking transactions. Ignoring Deadlocks 3 and 4 for

[GitHub] cloudstack pull request #1580: CLOUDSTACK-9402 : Support for underlay featur...

2016-11-24 Thread prashanthvarma
Github user prashanthvarma commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1580#discussion_r89540443 --- Diff: tools/marvin/marvin/lib/base.py --- @@ -3377,6 +3377,40 @@ def list(cls, apiclient, **kwargs): cmd.listall = True

[GitHub] cloudstack issue #1735: CLOUDSTACK-9570: Bug in listSnapshots for snapshots ...

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1735 Trillian test result (tid-426) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36018 seconds Marvin logs:

[GitHub] cloudstack issue #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-24 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1580 @rhtyd Here are the latest test results of the added Marvin tests that test the changes in this PR. 1) Marvin test run: nosetests --with-marvin --marvin-config=nuage.cfg

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests --- If your project is set up for

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests --- If your project is set up for

[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

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-251 --- 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 issue #1753: [4.9] Latest health test run

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-250 --- 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 issue #1752: [4.8] Latest health test run

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1752 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-249 --- 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 issue #1635: CLOUDSTACK-9451

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 Trillian test result (tid-427) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28223 seconds Marvin logs:

[GitHub] cloudstack issue #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-24 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1762 Here it is few samples of deadlocks we observe in high transaction volume environment with multiple management servers. As you can see most of them are concurrent operations from different

[GitHub] cloudstack issue #1604: improvement: systemvm slow operations

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1604 Trillian test result (tid-425) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 35008 seconds Marvin logs:

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1752 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[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 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-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 Thanks @nvazquez @serg38 -- they could be env related. I'll check again tomorrow, kick some tests now. Thanks @mike-tutkowski - happy thanksgiving to you and everyone in the community

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

2016-11-24 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 #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1762 @serg38 I have just now started reading this PR (excuse me if I overlooked some information). > If we are to try to implement a general way of dealing with deadlocks in ACS

[GitHub] cloudstack pull request #1580: CLOUDSTACK-9402 : Support for underlay featur...

2016-11-24 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1580#discussion_r89534499 --- Diff: tools/marvin/marvin/lib/base.py --- @@ -3377,6 +3377,40 @@ def list(cls, apiclient, **kwargs): cmd.listall = True

[GitHub] cloudstack pull request #1545: CLOUDSTACK-8715: Add channel to Instances for...

2016-11-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1545 --- 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

[GitHub] cloudstack issue #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1545 LGTM, the tests are all good as one (one failure was due to cleanup in the create template test case and was env related). Based on this I'm merging this. --- If your project is set up for it,

[GitHub] cloudstack pull request #1545: CLOUDSTACK-8715: Add channel to Instances for...

2016-11-24 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1545#discussion_r89533059 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java --- @@ -175,6 +180,26 @@ public boolean

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

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1710 @ustcweizhou @ozhanrk can you check snapshot and volume related errors in the last test runs above. --- If your project is set up for it, you can reply to this email and have your reply appear

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

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

[GitHub] cloudstack pull request #1737: CLOUDSTACK-9561 After domain/account deletion...

2016-11-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1737 --- 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

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1710 Trillian test result (tid-418) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36313 seconds Marvin logs:

[GitHub] cloudstack issue #1737: CLOUDSTACK-9561 After domain/account deletion, snaps...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1737 Tests LGTM, some failures are known intermittent issues. --- 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 #1765: Cloudstack 9586: When using local storage with Xense...

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1765 Trillian test result (tid-417) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36680 seconds Marvin logs:

[GitHub] cloudstack issue #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-24 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1762 @rafaelweingartner @swill @wido @koushik-das @karuturi @rhtyd @jburwell Let's ask a different question. If we are to try to implement a general way of dealing with deadlocks in ACS how could it

[GitHub] cloudstack issue #1737: CLOUDSTACK-9561 After domain/account deletion, snaps...

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1737 Trillian test result (tid-424) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 29573 seconds Marvin logs:

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1710 Trillian test result (tid-419) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 32311 seconds Marvin logs:

[GitHub] cloudstack issue #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1545 Trillian test result (tid-421) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 29503 seconds Marvin logs:

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

2016-11-24 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1754 Hi Rohit, Today is a holiday in the US and many people are also off work tomorrow. I have a bunch of holiday-related activities going on today and tomorrow, but should

[GitHub] cloudstack issue #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1580 Trillian test result (tid-423) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 27940 seconds Marvin logs:

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1577 Trillian test result (tid-422) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 27764 seconds Marvin logs:

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1623 Trillian test result (tid-420) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28011 seconds Marvin logs:

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

2016-11-24 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1754 @rhtyd adding to @serg38's comment, in test_06_download_detached_volume extractVolumeCmd fails with this error: `Invalid state of the volume with specified ID. It should be either detached

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

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1765 Trillian test result (tid-416) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 27983 seconds Marvin logs:

[GitHub] cloudstack issue #1735: CLOUDSTACK-9570: Bug in listSnapshots for snapshots ...

2016-11-24 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1735 @jburwell @rhtyd sorry I've been off yesterday, do I push force to kick tests again? --- 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 #1774: CLOUDSTACK-9608: Errored State and Abandoned state T...

2016-11-24 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1774 @parthj If the templates are displayed on UI, is that actions we can do on them? eg, delete template. --- If your project is set up for it, you can reply to this email and have your reply

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

2016-11-24 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1778 @nitin-maharana Hi Nitin, can we use "networkId: network.id" instead nicId ? --- 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 #1769: tools: fix Dockerfile

2016-11-24 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1769 Could you look if it conflict with #1435 ? Because I made a pull request to automatically increment version in docker file when changing release of master via

[GitHub] cloudstack pull request #1780: Fix the KVM hypervisor's handling of not-real...

2016-11-24 Thread ppenchev-storpool
GitHub user ppenchev-storpool opened a pull request: https://github.com/apache/cloudstack/pull/1780 Fix the KVM hypervisor's handling of not-really-SCSI devices. Hi, Thanks for developing and maintaining CloudStack! What do you think about the attached trivial

[GitHub] cloudstack issue #1320: CLOUDSTACK-8968: UI icon over VM snapshot to deploy ...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1320 Thanks @nitin-maharana --- 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 #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-24 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1577 @rhtyd I am afraid, the newly added integration test takes well beyond 20 mins (Ran 4 tests in 6729.698s in the previous run, refer earlier comments for more details) to run all its

[GitHub] cloudstack pull request #1779: CLOUDSTACK-9610: Disabled Host Keeps Being up...

2016-11-24 Thread priyankparihar
GitHub user priyankparihar opened a pull request: https://github.com/apache/cloudstack/pull/1779 CLOUDSTACK-9610: Disabled Host Keeps Being up status after unmanaging … Disabled Host Keeps Being up status after unmanaging cluster. You can merge this pull request into a Git

[GitHub] cloudstack pull request #1778: CLOUDSTACK-9613: Unable to set NAT rules on a...

2016-11-24 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1778 CLOUDSTACK-9613: Unable to set NAT rules on any interfaces except first interface of VM. Description: = VMs have multiple network interfaces. It is not possible to

[GitHub] cloudstack pull request #1777: clearCLOUDSTACK-9133: Two volume.delete usage...

2016-11-24 Thread priyankparihar
GitHub user priyankparihar opened a pull request: https://github.com/apache/cloudstack/pull/1777 clearCLOUDSTACK-9133: Two volume.delete usage events are getting generated… … for destoy vm. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request #1776: CLOUDSTACK-9603: concurrent.snapshots.thresho...

2016-11-24 Thread priyankparihar
GitHub user priyankparihar opened a pull request: https://github.com/apache/cloudstack/pull/1776 CLOUDSTACK-9603: concurrent.snapshots.threshold.perhost does not vali… concurrent.snapshots.threshold.perhost does not validate value given You can merge this pull request into a Git

[GitHub] cloudstack pull request #1775: CLOUDSTACK-9606: While IP address is released...

2016-11-24 Thread priyankparihar
GitHub user priyankparihar opened a pull request: https://github.com/apache/cloudstack/pull/1775 CLOUDSTACK-9606: While IP address is released, tag are not deleted. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request #1774: CLOUDSTACK-9608: Errored State and Abandoned ...

2016-11-24 Thread priyankparihar
GitHub user priyankparihar opened a pull request: https://github.com/apache/cloudstack/pull/1774 CLOUDSTACK-9608: Errored State and Abandoned state Templates are not … Errored State and Abandoned state Templates are not displayed on UI. You can merge this pull request into a Git

[GitHub] cloudstack pull request #1580: CLOUDSTACK-9402 : Support for underlay featur...

2016-11-24 Thread prashanthvarma
Github user prashanthvarma commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1580#discussion_r89472040 --- Diff: client/tomcatconf/commands.properties.in --- @@ -0,0 +1,804 @@ +# Licensed to the Apache Software Foundation (ASF) under one ---

[GitHub] cloudstack pull request #1580: CLOUDSTACK-9402 : Support for underlay featur...

2016-11-24 Thread prashanthvarma
Github user prashanthvarma commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1580#discussion_r89471555 --- Diff: tools/marvin/marvin/lib/base.py --- @@ -3377,6 +3377,40 @@ def list(cls, apiclient, **kwargs): cmd.listall = True

[GitHub] cloudstack pull request #1773: CLOUDSTACK-9607: Preventing template deletion...

2016-11-24 Thread priyankparihar
GitHub user priyankparihar opened a pull request: https://github.com/apache/cloudstack/pull/1773 CLOUDSTACK-9607: Preventing template deletion when template is in use. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack issue #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-24 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1762 Even trying the full transaction again could be problematic as there might be checks done before firing the transaction that may not be valid now. The thing is it may mostly work,

[GitHub] cloudstack issue #1320: CLOUDSTACK-8968: UI icon over VM snapshot to deploy ...

2016-11-24 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1320 @rhtyd : I discussed with @sateesh-chodapuneedi regarding this. I will give my changes to him. He will include my commit in his PR. Thanks. --- If your project is set up for it, you can

[GitHub] cloudstack issue #1320: CLOUDSTACK-8968: UI icon over VM snapshot to deploy ...

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1320 @nitin-maharana the feature is related, and exclusive; in that case please move your changes to PR #1664 --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply

[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 Checked centos6 failed due to an intermittent unit test failure, we can ignore that for now. @blueorangutan test --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1320: CLOUDSTACK-8968: UI icon over VM snapshot to deploy ...

2016-11-24 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1320 @rhtyd: This is dependent on #1664, Once #1664 gets merged, I will make a new PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1734: CLOUDSTACK-9567 Difference in the api call outputs f...

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1734 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-247 --- 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 issue #1733: CLOUDSTACK-9563 ExtractTemplate returns malformed UR...

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1733 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-246 --- 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

  1   2   >