Re: Question about JavaScript validators in UI

2016-10-26 Thread Marc-Aurèle Brothier
>From a quick look at the code you've forgotten one "|" for the or condition: $.validator.addMethod("ipv46cidr", function(value, element) { if (!$.validator.methods.ipv4cidr.call(this, value, element) || !$.validator.methods.ipv6cidr.call(this, value, element)) return false;

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-26 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1615 reran the test for delete account and it passes. `Test for delete account ... === TestName: test_delete_account | Status : SUCCESS === ok` merging this PR now --- If your project

[GitHub] cloudstack pull request #1737: CLOUDSTACK-9561 Unable to delete domain/Accou...

2016-10-26 Thread SudharmaJain
GitHub user SudharmaJain opened a pull request: https://github.com/apache/cloudstack/pull/1737 CLOUDSTACK-9561 Unable to delete domain/Account While deleting the UserAccount Cleanup for the removed VMs/volumes are not happening. For the removed VMs, snapshots doesn't get cleaned.

[GitHub] cloudstack pull request #1736: CLOUDSTACK-9562 Linux Guest VM get wrong defa...

2016-10-26 Thread SudharmaJain
GitHub user SudharmaJain opened a pull request: https://github.com/apache/cloudstack/pull/1736 CLOUDSTACK-9562 Linux Guest VM get wrong default route when there are… … multiple Nic REPRO STEPS == 1. Log as admin, create a VM CentOSx64 integrate

[GitHub] cloudstack issue #1718: [RC][Testing] Dummy PR for kicking tests on 4.9.1.0

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1718 Packaging result: ✔centos6 ✖centos7 ✖debian. JID-104 --- 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 #1719: [RC][Testing] Dummy PR for kicking tests on 4.10.0.0

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1719 Packaging result: ✔centos6 ✖centos7 ✖debian. JID-105 --- 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 #1717: [RC][Testing] Dummy PR for kicking tests on 4.8.2.1

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1717 Packaging result: ✔centos6 ✖centos7 ✔debian. JID-103 --- 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 #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-26 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1615 ### ACS CI BVT Run **Sumarry:** Build Number 125 Hypervisor xenserver NetworkType Advanced Passed=102 Failed=3 Skipped=6 _Link to logs Folder

[GitHub] cloudstack issue #1719: [RC][Testing] Dummy PR for kicking tests on 4.10.0.0

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1719 @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 #1717: [RC][Testing] Dummy PR for kicking tests on 4.8.2.1

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1717 @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 #1718: [RC][Testing] Dummy PR for kicking tests on 4.9.1.0

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1718 @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 #1717: [RC][Testing] Dummy PR for kicking tests on 4.8.2.1

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1717 @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 #1718: [RC][Testing] Dummy PR for kicking tests on 4.9.1.0

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1718 @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 #1719: [RC][Testing] Dummy PR for kicking tests on 4.10.0.0

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1719 @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

[ANNOUNCE] Apache CloudStack Security Releases 4.8.1.1, 4.9.0.1

2016-10-26 Thread Rohit Yadav
# Apache CloudStack Security Releases 4.8.1.1, 4.9.0.1 The Apache CloudStack project announces security releases 4.8.1.1, 4.9.0.1 that fixes the bug causing vulnerability over previously released minor versions 4.8.1 and 4.9.0 respectively. As a security release, no new features are included but

[GitHub] cloudstack issue #1718: [RC][Testing] Dummy PR for kicking tests on 4.9.1.0

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1718 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-101 --- 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 #1719: [RC][Testing] Dummy PR for kicking tests on 4.10.0.0

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1719 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-102 --- 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 #1717: [RC][Testing] Dummy PR for kicking tests on 4.8.2.1

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1717 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-100 --- 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 #1717: [RC][Testing] Dummy PR for kicking tests on 4.8.2.1

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1717 @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 #1717: [RC][Testing] Dummy PR for kicking tests on 4.8.2.1

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1717 @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 #1719: [RC][Testing] Dummy PR for kicking tests on 4.10.0.0

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1719 @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 #1718: [RC][Testing] Dummy PR for kicking tests on 4.9.1.0

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1718 @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 #1718: [RC][Testing] Dummy PR for kicking tests on 4.9.1.0

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1718 @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 #1719: [RC][Testing] Dummy PR for kicking tests on 4.10.0.0

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1719 @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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1729 Trillian test result (tid-194) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 6 Total time taken: 35338 seconds Marvin logs:

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

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

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-10-26 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/872 Tomorrow I will pick a recommended S2S VPN configuration and verify the rest of the different possible options with that single configuration to give a better global picture of the state of this

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-10-26 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/872 I wrote a small testing setup which allows me to automate the build up and tear down different configurations in test environment. Here are the results so far of my branch (hopefully soon

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-10-26 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @karuturi Anything else @syed needs to do for this one? 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

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-26 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1615 ### ACS CI BVT Run **Sumarry:** Build Number 124 Hypervisor xenserver NetworkType Advanced Passed=98 Failed=4 Skipped=6 _Link to logs Folder

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-10-26 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/872 @jayapalu I am not sure why, but I can't seem to do a pull request against your branch. Any ideas why? Maybe you have some specific permissions on your repo to stop this? If you are not sure, I

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed +1 -- I would like to see a full component test run for this PR on XS 7 and smoke tests for XS 6.5. Also, we might want to consider a PR for 4.9 and then another PR for master like we

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @jburwell Got it. I want to stabilize the tests and once everything looks green here. I will open a PR against 4.9 --- If your project is set up for it, you can reply to this email and have your

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

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 @murali-reddy 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

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

2016-10-26 Thread murali-reddy
Github user murali-reddy 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

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

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-99 --- 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 #1711: Xenserver7 Support

2016-10-26 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed given that XenServer 6.5 doesn't run on newer hardware, XenServer users don't have a realistic LTS release. Therefore, I propose that this PR be targeted to the 4.9 branch and forward

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

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 @murali-reddy 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

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

2016-10-26 Thread murali-reddy
Github user murali-reddy 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

[GitHub] cloudstack pull request #1735: CLOUDSTACK-9570: Bug in listSnapshots for sna...

2016-10-26 Thread nvazquez
GitHub user nvazquez opened a pull request: https://github.com/apache/cloudstack/pull/1735 CLOUDSTACK-9570: Bug in listSnapshots for snapshots with deleted data stores ## Actual behaviour If there is snapshot on a data store that is removed, `listSnapshots` still tries to

[GitHub] cloudstack issue #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1729 Trillian test result (tid-188) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 6 Total time taken: 36442 seconds Marvin logs:

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @rhtyd @PaulAngus I will verify the failed tests (except VPC and private gateway) and update this. It would be nice if we can get this into 4.10 --- If your project is set up for it, you can

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 Trillian test result (tid-192) Environment: CUSTOM-XenServer (x2), Advanced Networking with Mgmt server 6 Total time taken: 35929 seconds Marvin logs:

PR Backlog

2016-10-26 Thread Paul Angus
All, Now that we're getting some testing momentum going, I was thinking that we need a way to categorise and prioritise the PR queue. There are a limited number of labels available for use to apply to the PRs within Github as it is, but we could go through and highlight bug fixes and

[GitHub] cloudstack issue #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1729 @rhtyd a Trillian-Jenkins test job (centos6 mgmt + vmware-55u3) 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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1729 @blueorangutan test centos6 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 #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

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

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1615 Trillian test result (tid-191) Environment: vmware-60u2 (x2), Advanced Networking with Mgmt server 7 Total time taken: 30139 seconds Marvin logs:

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-26 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1615 ### ACS CI BVT Run **Sumarry:** Build Number 123 Hypervisor xenserver NetworkType Advanced Passed=102 Failed=3 Skipped=6 _Link to logs Folder

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

2016-10-26 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1727 @ustcweizhou @koushik-das thanks for your review! I refactored based on your comments and tested everything is working as expected. To test new marvin test introduced in

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1732 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

Re: Question about JavaScript validators in UI

2016-10-26 Thread Wido den Hollander
So my JS skills are way to low, but I tried this, but it doesn't seem to work: $.validator.addMethod("ipv46cidr", function(value, element) { if (!$.validator.methods.ipv4cidr.call(this, value, element) | !$.validator.methods.ipv6cidr.call(this, value, element)) return false;

[GitHub] cloudstack pull request #1734: CLOUDSTACK-9567 Difference in the api call ou...

2016-10-26 Thread SudharmaJain
GitHub user SudharmaJain opened a pull request: https://github.com/apache/cloudstack/pull/1734 CLOUDSTACK-9567 Difference in the api call outputs for CAPACITY_TYPE_… …CPU = 1 The total capacity reported listPods and listCapacity is different, ideally they have to be

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 Trillian test result (tid-183) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 32182 seconds Marvin logs:

[GitHub] cloudstack pull request #1726: CLOUDSTACK-9560 Root volume of deleted VM lef...

2016-10-26 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1726#discussion_r85108024 --- Diff: server/src/com/cloud/storage/StorageManagerImpl.java --- @@ -2199,15 +2199,20 @@ public void cleanupDownloadUrls(){

[GitHub] cloudstack pull request #1733: CLOUDSTACK-9563 ExtractTemplate returns malfo...

2016-10-26 Thread SudharmaJain
GitHub user SudharmaJain opened a pull request: https://github.com/apache/cloudstack/pull/1733 CLOUDSTACK-9563 ExtractTemplate returns malformed URL after migrating… … NFS to s3 While downloading the template for the first time install path was not available. During

Re: patchviasocket seems to be broken with qemu 2.3(+?)

2016-10-26 Thread Linas Žilinskas
So after some investigation I've found out that qemu 2.3.0 is indeed broken, at least the way CS uses the qemu chardev/socket. Not sure in which specific version it happened, but it was fixed in 2.4.0-rc3, specifically noting that CloudStack 4.2 was not working. qemu git commit:

[GitHub] cloudstack issue #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1729 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-98 --- 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 #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

2016-10-26 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1586 ### ACS CI BVT Run **Sumarry:** Build Number 122 Hypervisor xenserver NetworkType Advanced Passed=101 Failed=3 Skipped=5 _Link to logs Folder

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1615 @karuturi 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

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-26 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1615 @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

[GitHub] cloudstack pull request #1650: fix macchinina template specificied for vmwar...

2016-10-26 Thread PaulAngus
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1650 --- 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 pull request #1688: Marvin Tests: Fix VPC network offering select...

2016-10-26 Thread PaulAngus
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1688 --- 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 #1728: CLOUDSTACK-9551: Move java tmp dir to cloudstack-age...

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1728 LGTM both test results (failures were intermittent issues and can be ignored) and code changes. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request #1731: CLOUDSTACK-9565: Fix intermittent failure in ...

2016-10-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1731 --- 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 #1731: CLOUDSTACK-9565: Fix intermittent failure in test_oo...

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1731 On basis of Travis result, manual test run and code reviews, I'm going ahead and merging this to avoid intermittent failures in upcoming tests. --- If your project is set up for it, you can

[GitHub] cloudstack issue #1731: CLOUDSTACK-9565: Fix intermittent failure in test_oo...

2016-10-26 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1731 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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1729 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-97 --- 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 #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1732 @rhtyd a Trillian-Jenkins test job ((this mgmt + is) 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 appear on

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1732 @blueorangutan test (this is to run a regression testing round) --- 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 #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1732 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-96 --- 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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1729 @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 #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1729 @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 #1604: improvement: systemvm slow operations

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1604 @murali-reddy can you help review this one? @romain-dartigues thanks, can you also squash your commits to a single commit, and amend the git commit message to reflect the JIRA ID in the

[GitHub] cloudstack issue #1731: CLOUDSTACK-9565: Fix intermittent failure in test_oo...

2016-10-26 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1731 @LGTM on code review @rhtyd --- 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 #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1732 @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 #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1732 All changes are restricted to the solidfire plugin and currently we don't have infra/tests to run them in a general env so considering Mike's tests, LGTM. @blueorangutan package --- If your

[GitHub] cloudstack issue #1731: CLOUDSTACK-9565: Fix intermittent failure in test_oo...

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1731 Test seems to be passing now: === TestName: test_oobm_zchange_password | Status : SUCCESS === --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1731: CLOUDSTACK-9565: Fix intermittent failure in test_oo...

2016-10-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1731 Jenkins failure was due to git pull issue, not the PR changes. Pinging for quick review/merge -- @abhinandanprateek @murali-reddy @karuturi @koushik-das --- If your project is set up for it,

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

2016-10-26 Thread romain-dartigues
Github user romain-dartigues commented on the issue: https://github.com/apache/cloudstack/pull/1604 @rhtyd < JIRA issue [#9568](https://issues.apache.org/jira/browse/CLOUDSTACK-9568) I'll rebase as soon as possible. --- If your project is set up for it, you can reply to

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestVolumes.py (no resigning) test_00_check_template_cache (TestVolumes.TestVolumes) ... === TestName: test_00_check_template_cache | Status : SUCCESS === ok Attach a

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestVolumes (resigning) test_00_check_template_cache (TestVolumes.TestVolumes) ... === TestName: test_00_check_template_cache | Status : SUCCESS === ok Attach a volume to

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestVMSnapshots.py test_01_take_VM_snapshot (TestVMSnapshots.TestVMSnapshots) ... === TestName: test_01_take_VM_snapshot | Status : SUCCESS === ok

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestVMMigrationWithStorage test_01_storage_migrate_root_and_data_disks (TestVMMigrationWithStorage.TestVMMigrationWithStorage) ... === TestName:

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestSnapshots.py: The failure in test 04 makes sense as that test case is actually dependent upon #1600, which has not yet been merged in.

[GitHub] cloudstack issue #1732: Switched to the official SolidFire SDK for Java

2016-10-26 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1732 TestAddRemoveHosts.py test_add_remove_host_with_solidfire_plugin_1 (TestAddRemoveHosts.TestAddRemoveHosts) ... === TestName: test_add_remove_host_with_solidfire_plugin_1 |

[GitHub] cloudstack pull request #1732: Switched to the official SolidFire SDK for Ja...

2016-10-26 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request: https://github.com/apache/cloudstack/pull/1732 Switched to the official SolidFire SDK for Java SolidFire has recently released an official SDK for Java. I have converted the SolidFire storage plug-ins to make use of this new