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

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

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

2016-11-24 Thread sateesh-chodapuneedi
Github user sateesh-chodapuneedi commented on the issue: https://github.com/apache/cloudstack/pull/1664 Hi @rhtyd, Working on this PR this week, should be able to update and push soon. --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[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 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-243 --- 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 d

[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 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-244 --- 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 d

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

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1604 @murali-reddy @abhinandanprateek can you help review this, thanks @blueorangutan test --- 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 #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 @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 ap

[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 @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 so

[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 @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 ap

[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 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-245 --- 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 d

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

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1735 @jburwell the Travis failure is ignorable caused by a oobm corner case (exception is ignorable). I'll work on a fix soon. @blueorangutan test --- If your project is set up for it, you can rep

[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 @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 ap

Re: Fault percentage value of CPU usage in Cloud Platform

2016-11-24 Thread anil lakineni
Dear Will, Good Afternoon, i hope everything is fine at your end. Please find my comments for your questions, - do you have VMs allocated, but turned off? They will count towards the provisioned CPU even though they are not running because they could be started at any time and are expecting to h

[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 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-248 --- 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 d

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

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

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

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

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

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

[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 #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, b

[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 https://github.com/priyankpariha

[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 #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 #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 r

[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 https://github.com/priyankparihar/c

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

[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 https:

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

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

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

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

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

[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 ```tools/build/setnextversion.sh`

[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. I

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

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

[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: https://github.com/blueoranguta

[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 #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: https://github.com/blueoranguta

[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: https://github.com/blueoranguta

[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: https://github.com/blueoranguta

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

[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: https://github.com/blueoranguta

[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: https://github.com/blueoranguta

[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: https://github.com/blueoranguta

[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 #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: https://github.com/blueoran

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

[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: https://github.com/blueoran

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

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

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

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

[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, yo

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

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

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

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

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

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

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

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

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

[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: https://github.com/blueoranguta

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

[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: https://github.com/blueoranguta

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

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

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

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

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

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

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

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

[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: https://github.com/blueoranguta

[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 #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 now

[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 PR)

[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 #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 well.

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

[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: https://github.com/blueoranguta

[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 #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 t

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

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

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

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

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

[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 a/engine/schema/src/com/cloud/network/dao/

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

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

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

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

[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 #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 enable

[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: https://github.com/blueoranguta

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

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

  1   2   >