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

2016-11-07 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1748 Thanks @mike-tutkowski. Will take a look at #1749 --- 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

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

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

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

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

Re: ACS - Some VMs unable to get DHCP IP from VR

2016-11-07 Thread ilya
Can you move the routerVM on the same hypervisor as guest VM (or guest VM to same hypervisor as routerVM)? If it works, then move the routerVM out to another hypervisor within the same cluster (but same switch). Are you running vSphere? I've seen similar problem where ARPs would not make it thru

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

2016-11-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 @jburwell @karuturi Although I won't be able to run regression tests on #1749 anytime soon (due to being out of the country for a while), @syed has volunteered to do so. --- If your

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

2016-11-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 @jburwell @karuturi I went ahead and opened #1749. That one just has the fixes for #1600. That should go into 4.10 because it fixes pretty important issues and is low risk. --- If your

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

2016-11-07 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request: https://github.com/apache/cloudstack/pull/1749 Updates for SAN-assisted snapshots This PR is to address a few issues in #1600 (which was recently merged to master for 4.10). In

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

2016-11-07 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1735 @rhtyd @jburwell The tests didn't kick in last time. Can we fire matrix again? --- 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 #1676: CLOUDSTACK-9502: DS template copies don’t get dele...

2016-11-07 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1676 @jburwell Non-VPC tests on VmWare passed this time. This one seems to be ready for merging. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1676: CLOUDSTACK-9502: DS template copies don’t get dele...

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

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

2016-11-07 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1748 @mike-tutkowski I will slot this PR into the 4.11.0.0 release. For the next release cycle, I would like to expand testing scenarios to include multi-zone, as well as, zone-wide storage which

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-11-07 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @PaulAngus @jburwell I've added more changes and rebased. now My tests are now running without errors. I did add the ` --config global{metadata_read_only=0} ` as suggested by @PaulAngus and did

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

2016-11-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 @karuturi @jburwell Oh, and just to point out, the reason I embedded those changes in this PR at the time was because I couldn't finish running my regression tests on this PR until that

[GitHub] cloudstack pull request #1711: Xenserver7 Support

2016-11-07 Thread syed
Github user syed commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r86864517 --- Diff: setup/db/db/schema-4910to41000.sql --- @@ -35,4 +35,192 @@ INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervi

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

2016-11-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 @karuturi @jburwell Hi I was recently working on some enhancements to 4.5 for a prospect. Once I completed that work in their custom branch, I wanted to forward port the

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

2016-11-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 All tests for managed storage have been run and passed: [results.txt](https://github.com/apache/cloudstack/files/576566/results.txt)

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

2016-11-07 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request: https://github.com/apache/cloudstack/pull/1748 KVM enhancements for managed storage This PR contains changes mainly related to enhancing managed storage on KVM. I recently added these features to a custom build of 4.5 for a prospect

Re: ACS - Some VMs unable to get DHCP IP from VR

2016-11-07 Thread Wei ZHOU
can you paste the result of following command? cat /etc/dnsmasq.conf |grep -v "^#" |grep -v "^$" -Wei 2016-11-07 20:27 GMT+01:00 Cloud List : > Hi Wei, > > In addition, > > The VR is serving a shared not isolated network, meaning the IP it serves > is 'guest' not 'public'

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

2016-11-07 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1710 @jburwell John, this is a small bug fix for issue: failed to delete snapshot on RBD if the original volume is removed. It will not impact other scenario and bring other issues. I

Re: ACS - Some VMs unable to get DHCP IP from VR

2016-11-07 Thread Chiradeep Vittal
I wonder if there are more VMs than the range allowed in the guest network. The excess VMs may not be getting IPs. Check /etc/dnsmasq.d/multiple_ranges.conf /etc/dnsmasq.conf Check this http://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/2008q3/002301.html You can also turn on logging on

Re: ACS - Some VMs unable to get DHCP IP from VR

2016-11-07 Thread Cloud List
Hi Wei, In addition, The VR is serving a shared not isolated network, meaning the IP it serves is 'guest' not 'public' IP. Will that make a difference on the iptables command we need to execute? Looking forward to your reply, thank you. Cheers. On Tue, Nov 8, 2016 at 3:19 AM, Cloud List

Re: ACS - Some VMs unable to get DHCP IP from VR

2016-11-07 Thread Cloud List
Hi Wei and Ozhan, Thanks for your reply. The problem doesn't affect only Debian-based guest VMs, but also affected some Windows and Ubuntu-based VMs as well. I have executed the command on the VR and reset the NIC of the guest VM, but unfortunately the issue still persists. iptables -t mangle

[GitHub] cloudstack issue #1730: VMTemplateZone needs some love

2016-11-07 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1730 @marcaurele thank you for the cleanup. Since this change does not fix a defect, we will hold off merging this PR until after after the 4.82.0/4.9.1.0/4.10.0.0 release set is completed. ---

Re: ACS - Some VMs unable to get DHCP IP from VR

2016-11-07 Thread Wei ZHOU
GOOD point. @CloudList, can you try again after executing the following command in VR ? iptables -t mangle -A POSTROUTING -p udp -m udp --dport 68 -j CHECKSUM --checksum-fill -Wei 2016-11-07 14:46 GMT+01:00 Özhan Rüzgar Karaman : > Hi; > Whats your problematic vm's

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

2016-11-07 Thread ozhanrk
Github user ozhanrk commented on the issue: https://github.com/apache/cloudstack/pull/1710 Hi @jburwell , i am so sorry but i do not understand, what you mean by combining fixes under a single pr because this is a only single fix with a 4 lines of code change which only effects rbd

[GitHub] cloudstack issue #1676: CLOUDSTACK-9502: DS template copies don’t get dele...

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

[GitHub] cloudstack issue #1676: CLOUDSTACK-9502: DS template copies don’t get dele...

2016-11-07 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1676 @murali-reddy only the following failures are expected: * `test_01_vpc_site2site_vpn` * `test_01_redundant_vpc_site2site_vpn` The test failures, particularly around

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

2016-11-07 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1710 @ozhanrk I recommend combining the fixes into a single PR for review cohesion and to ensure that the merge maintains the stability of the release branches. You could either amend this PR with

[GitHub] cloudstack issue #846: Bug-ID: CLOUDSTACK-8870: Skip external device usage c...

2016-11-07 Thread rajesh-battala
Github user rajesh-battala commented on the issue: https://github.com/apache/cloudstack/pull/846 code/patch 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 issue #1673: CLOUDSTACK-9071: Properly parse stats.output.uri in ...

2016-11-07 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1673 @wido there are two errors related to template download in the blueorganutan run that we need to explain before merging this PR. We are seeing a similar issue on #1587. @abhinandanprateek

Re: ACS - Some VMs unable to get DHCP IP from VR

2016-11-07 Thread Özhan Rüzgar Karaman
Hi; Whats your problematic vm's type is it Debian? Maybe you are affected from the bug CLOUDSTACK-8326? I do not know if this bug has effected on ACS 4.2 release but i know that it effects release 4.8.x 4.9.x Thanks Özhan On Mon, Nov 7, 2016 at 4:36 PM, Cloud List wrote: >

Re: ACS - Some VMs unable to get DHCP IP from VR

2016-11-07 Thread Cloud List
Hi Wei, Thanks for your reply. I checked and I can confirm that the mac address is listed on /etc/dhcphosts.txt in VR. For example: Nov 7 13:30:19 dnsmasq-dhcp[18986]: DHCPDISCOVER(eth0) 06:31:ac:01:13:YY ignored Nov 7 13:30:24 dnsmasq-dhcp[18986]: DHCPDISCOVER(eth0) 06:31:ac:01:13:YY

Re: ACS - Some VMs unable to get DHCP IP from VR

2016-11-07 Thread Wei ZHOU
If the mac address is not listed in /etc/dhcphosts.txt in VR, the request will be ignored. Can you give more details so we can reproduce it and fix it ? -Wei 2016-11-07 13:44 GMT+01:00 Cloud List : > Hi, > > After our upgrade from CloudStack 4.2 to 4.8.1.1, we noted that

Re: Long delay in listDomains API response

2016-11-07 Thread Cloud List
Thanks Sergey, appreciate your reply. Points noted. Cheers. On Mon, Nov 7, 2016 at 10:44 AM, Sergey Levitskiy < sergey.levits...@autodesk.com> wrote: > What essentially happens here that for each domain, you have access with > your key, ACS executes 12 queries to get domain limits. Even if

ACS - Some VMs unable to get DHCP IP from VR

2016-11-07 Thread Cloud List
Hi, After our upgrade from CloudStack 4.2 to 4.8.1.1, we noted that some (but not all) of our VMs are not able to get DHCP from the VR. This gives problem when the VM is restarted and cannot get up and running because unable to get IP. I logged in to the VR and found below messages showing that

Re: ConfigDepot vs the Config enum

2016-11-07 Thread Jeff Hair
Hi, That's what I thought was the case. Is there any particular process for moving configs to the depot besides just moving it out of the enum and into a manager bean somewhere? For example, would the pull request(s) need to have a database migration of some kind? Jeff *Jeff Hair* Technical

Re: ConfigDepot vs the Config enum

2016-11-07 Thread Koushik Das
Config enum is legacy code. For any new configuration parameters ConfigDepot should be used. Some of the parameters got moved from Config enum to the appropriate manager classes (where it is getting used). But there are still lot of them left to be moved. Feel free to submit a PR for moving

[GitHub] cloudstack issue #1673: CLOUDSTACK-9071: Properly parse stats.output.uri in ...

2016-11-07 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1673 This one has two LGTMs, I think we can merge it in? --- 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

ConfigDepot vs the Config enum

2016-11-07 Thread Jeff Hair
Hi, What's the difference between the Config enum and the ConfigDepot? It seems like the configuration options are somwhat arbitrarily split between the two. ConfigDepot entries get created automatically, while ones in the Config enum don't (and thus require a database migration). ConfigDepot

[GitHub] cloudstack issue #1713: CLOUDSTACK-9552: Allow egress TCP/53 implicitly in B...

2016-11-07 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1713 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