[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-220595800 Im not sure what to think of those failures. I will run again in clean setup to see if I can get rid of those. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-220595991 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 2 Errors: 0 Duration: 4h 12m 50s ``` **Summary of the

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-220596270 I am getting similar errors in other PRs, so I don't think they are related to this PR. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: CLOUDSTACK-9368: Fix for Support configur...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1518#issuecomment-220619741 For some reason Travis is unhappy. Would you mind trying again. Sorry for the inconvenience... --- If your project is set up for it, you can reply to this email

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

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-220621430 @DaanHoogland yes, I will retest now. Thanks guys... --- 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: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-220622074 Can I get some code review on this one. It is looking to be in pretty good shape otherwise. Thanks... --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-220624878 I have seen similar issues in a couple other PRs. Since this is a PR related to connectivity, I do want to try to get a cleaner CI run if I can. Also, travis is

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220626559 @rhtyd thank you. I will review what happened there. This is something that has started showing up intermittently, but only started happening in the last few weeks

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1251#issuecomment-220627549 Thanks @koushik-das, I need to track down what changed to cause that intermittent problem of the `template.status`. I need some code review on this one. Thanks

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-220627907 ### CI RESULTS ``` Tests Run: 83 Skipped: 0 Failed: 0 Errors: 2 Duration: 8h 27m 27s ``` **Summary of the

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-220628087 Two clean runs in a row. I think this one is ready to put back in now... --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-220628402 BTW, thank you for your quick response time to work through issues on your PRs and get things stabilized. I really do appreciate it. --- If your project is set up

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1251#issuecomment-220628656 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 0 Errors: 4 Duration: 8h 31m 38s ``` **Summary of the

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1251#issuecomment-220628965 Similar issues on this run. They are not related to this PR. Still need some review on this one... --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CLOUDSTACK-6975: Prevent dnsmasq from sta...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1514#issuecomment-220629958 This one is coming back clean. If we can get Jenkins and Travis green I think this one is ready... --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CLOUDSTACK-6975: Prevent dnsmasq from sta...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1514#issuecomment-220629640 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 8h 44m 23s

[GitHub] cloudstack pull request: CLOUDSTACK-9289:Automation for feature de...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-220631924 Ok, thanks @nitt10prashant. 👍 --- 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 pull request: CLOUDSTACK-9377: Fix metrics pagesize iss...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1540#issuecomment-220632778 Ya, sorry, I had a backlog of commits that were ready while I sorted out some merge conflicts doing a forward merge from 4.8 and then testing the resulting master to

[GitHub] cloudstack pull request: Convert patchviasocket to python (removes...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1533#issuecomment-220633650 @sverrirab thank you. Would you mind adding the revert of #1495 into this commit in order to clean up unnecessary packages once this PR is accepted? I think we are

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-220640589 Ok. Travis has been pretty consistently green for PRs recently, so I have not seen this problem elsewhere. I am not sure why there are sometimes more than once

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1360#issuecomment-220656462 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 1 Errors: 4 Duration: 10h 22m 36s ``` **Summary of the

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1360#issuecomment-220656774 Ran again because the error was not something I had seen often. The errors in the new run are not related to this PR. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-220715003 And of course the first build I do after I merge it into master I get the `addHost` error and I did not get it once after I pulled it out. Haha... I will build

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-220715486 @rhtyd can you specify the logs you need if I run into this issue so you can track down why it is happening? I will leave it in for now but I would like to send you

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220715917 I am getting it on almost every PR right now (if you notice the last 6 or 7 CI runs), so I will have to review this as it seems to have been introduced recently

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220716022 I think since it is an intermittent issue, we did not catch it on the actual PR it was introduced in. --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-220735448 @rhtyd uggg... The first two builds on master after merging this back in have both resulted in the `addHost` error. :( Let me know what you need to troubleshoot

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-220737825 This is one of the only PRs I have seen consistently failing on the scale vm test. I am not sure what to think about that. Can you rebase and force push or close

[GitHub] cloudstack pull request: Restore iptables at once using iptables-r...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1482#issuecomment-220738113 @remibergsma can you give me a status on this PR. I know you reverted it because of missing licenses. Should I be trying to wait for this one to get back in for

[GitHub] cloudstack pull request: CLOUDSTACK-6975: Prevent dnsmasq from sta...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1514#issuecomment-220738431 I would like to get this one re-pushed or closed and reopened so we can kick the CI again to see if we can get things green. I am getting very close to freezing and

[GitHub] cloudstack pull request: CLOUDSTACK-6928: fix issue disk I/O throt...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1410#issuecomment-220738747 Getting close to freeze, please repush to see if we can get travis green. Thx... --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-6928: fix issue disk I/O throt...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1410#issuecomment-220738889 @ustcweizhou ^ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack pull request: [CLOUDSTACK-8973] Fix create template fro...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1424#issuecomment-220738801 @syed: Getting close to freeze, please repush to see if we can get jenkins green. Thx... --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1423#issuecomment-220739003 @syed please close and reopen or repush. we are close to freeze and I want to get some of these PRs in... --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: [CLOUDSTACK-9337]Enhance vcenter.py to cr...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1464#issuecomment-220739360 @sanju1010 I don't know what to do with this one. I don't have any verification that this PR works, so I am not sure what information I am supposed to b

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-220739661 @DaanHoogland you think the issue that travis is having is a genuine issue, is that what you mean? --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-220741573 OK. I don't really have the bandwidth to troubleshoot this (unfortunately). This is one of the many PRs that are on the bubble which I am trying to figure out

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-220745686 Have a nice weekend. :) --- 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 pull request: [CLOUDSTACK-8973] Fix create template fro...

2016-05-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1424#issuecomment-220755087 the sweet sweet color green... 👍 --- 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 pull request: Convert patchviasocket to python (removes...

2016-05-21 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1533#issuecomment-220810829 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 8h 55m 29s

[GitHub] cloudstack pull request: Convert patchviasocket to python (removes...

2016-05-21 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1533#issuecomment-220810862 This is coming back clean. Would you mind trying to rebase and re-push or close and reopen to see if we can kick off travis one more time. Thanks... --- If your

[GitHub] cloudstack pull request: Hyper-V communication broken by change in...

2016-05-21 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1556#issuecomment-220811376 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 1 Errors: 1 Duration: 11h 00m 25s ``` **Summary of the

[GitHub] cloudstack pull request: CLOUDSTACK-9280: Allow system VM volumes ...

2016-05-21 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1406#issuecomment-220813294 @ProjectMoon can you reopen this against at least 4.7, maybe even master. Thanks... --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: add DHCP lease folders for Ubuntu

2016-05-21 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1552#issuecomment-220813347 Does this need CI? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

2016-05-21 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1423#issuecomment-220813537 @syed can you provide details for how to add `ike-scan` in order to run the test and verify via CI? Not sure how I feel about adding a test that can't be

[GitHub] cloudstack pull request: Restore iptables at once using iptables-r...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1482#issuecomment-221151970 @remibergsma ok, no problem. Thanks for the heads up. :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: add DHCP lease folders for Ubuntu

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1552#issuecomment-221155776 Ok, so I have two LGTM both with validation that the code behaves as expected. I think that is enough for this to get in. Thanks... --- If your project is set up

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

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-221158751 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 4h 25m 18s

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

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-221158910 This is coming back clean. Can I get a second LGTM? Maybe @rhtyd? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-221159015 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 1 Duration: 4h 20m 05s ``` **Summary of the

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-221159200 @DaanHoogland can you rebase and push again to try to kick off Travis and see if we can clear this one up? Thanks... --- If your project is set up for it, you can

[GitHub] cloudstack pull request: Hyper-V communication broken by change in...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1556#issuecomment-221159616 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 1 Errors: 1 Duration: 10h 51m 43s ``` **Summary of the

[GitHub] cloudstack pull request: Hyper-V communication broken by change in...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1556#issuecomment-221159792 I ran CI again hoping to clear up those errors and I got the same ones again. I don't think they are related to this PR, but twice in a row concerns me a bi

[GitHub] cloudstack pull request: Add Java Default Certificat Authorities i...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1555#issuecomment-221159912 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 8h 55m 13s

[GitHub] cloudstack pull request: Add Java Default Certificat Authorities i...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1555#issuecomment-221159974 I think this one is ready... --- 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 pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-221160227 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 0 Errors: 3 Duration: 8h 35m 03s ``` **Summary of the

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-221160368 This is coming back clean now. Those failures are not related to this PR. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: CLOUDSTACK-9380: fix NPE in listDomains A...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1550#issuecomment-221160504 ### CI RESULTS ``` Tests Run: 78 Skipped: 0 Failed: 0 Errors: 4 Duration: 7h 14m 41s ``` **Summary of the

[GitHub] cloudstack pull request: CLOUDSTACK-9380: fix NPE in listDomains A...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1550#issuecomment-221160650 This is a clean CI run. The failures are not related to this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-221162055 @rhtyd fingers crossed, things seem to be in pretty good shape. The first two runs after the merge had `addHost` errors, but I have done about 6-7 CI runs since and

[GitHub] cloudstack pull request: CLOUDSTACK-6975: Prevent dnsmasq from sta...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1514#issuecomment-221162753 This one is green. I think this one is ready??? Please verify if there is anything outstanding on this one. Thanks... :) --- If your project is set up for it

[GitHub] cloudstack pull request: Convert patchviasocket to python (removes...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1533#issuecomment-221162872 Yes, ready now. Thank you. :) --- 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 pull request: CLOUDSTACK-8715: Add channel to Instances...

2016-05-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-221166274 I missed that this one was going to need a new systemvmtemplate. Given that I am already over a week late on this release and I have not done any testing or work

[GitHub] cloudstack pull request: Hyper-V communication broken by change in...

2016-05-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1556#issuecomment-221277292 This is a clean run. The errors here are not related to this PR. I will merge this. Thanks... --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-221278053 Apparently `status` is missing from the `listTemplates` response in some cases. I will investigate after we freeze and before we cut an RC to see if we can resolve

[GitHub] cloudstack pull request: Hyper-V communication broken by change in...

2016-05-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1556#issuecomment-221277026 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 0 Errors: 3 Duration: 8h 11m 34s ``` **Summary of the

[GitHub] cloudstack pull request: Vmdk findbugs

2016-05-24 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1530#discussion_r64395993 --- Diff: vmware-base/src/com/cloud/hypervisor/vmware/mo/VmdkFileDescriptor.java --- @@ -27,55 +27,53 @@ import org.apache.log4j.Logger

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221278559 @dsclose if you can fix the merge conflicts I can run this again and see what is outstanding. Thanks... --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: Compabitility fix for Docker >= 1.11 (doc...

2016-05-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1548#issuecomment-221284250 @moul can you close and reopen the PR or rebase and force push in order to kick off Jenkins again so we can get this all green? @pdion891 I think you have

[GitHub] cloudstack pull request: Add Java Default Certificat Authorities i...

2016-05-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1555#issuecomment-221324840 Thanks for the update. :) --- 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 pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-221409302 Was the only change since the last CI run the updating of the marvin test (which I can't run anyway)? --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-221409650 Oh, I think you were just resolving conflicts. I will run CI again just to verify... Otherwise this is looking good. Thanks everyone... --- If your project is

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1423#issuecomment-221631046 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 1 Duration: 4h 14m 49s ``` **Summary of the

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1423#issuecomment-221631305 @syed can you review this error? It seems to be related to your test. Thanks... --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8870: Skip external de...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/846#issuecomment-221633655 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 2 Errors: 1 Duration: 10h 41m 55s ``` **Summary of the

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-221631430 ### CI RESULTS ``` Tests Run: 84 Skipped: 0 Failed: 0 Errors: 1 Duration: 5h 26m 59s ``` **Summary of the

[GitHub] cloudstack pull request: CLOUDSTACK-9280: System VM volumes can be...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1559#issuecomment-221633251 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 1 Errors: 1 Duration: 10h 41m 20s ``` **Summary of the

[GitHub] cloudstack pull request: Dynamically load drivers before creating ...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1553#issuecomment-221634279 ### CI RESULTS ``` Tests Run: 83 Skipped: 0 Failed: 1 Errors: 2 Duration: 8h 32m 15s ``` **Summary of the

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-221631597 This error is unrelated to this PR. This is ready to merge... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221637516 @dsclose can you kick it off again. It is just a timeout which caused it to fail, so kicking it off again could fix it. `No output has been received in the last

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8870: Skip external de...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/846#issuecomment-221757982 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 0 Errors: 3 Duration: 8h 33m 42s ``` **Summary of the

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8870: Skip external de...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/846#issuecomment-221758314 This is a cleaner run because this issue is a known issue. I think this one is ready now... --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: Dynamically load drivers before creating ...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1553#issuecomment-221758690 ### CI RESULTS ``` Tests Run: 83 Skipped: 0 Failed: 0 Errors: 2 Duration: 8h 39m 49s ``` **Summary of the

[GitHub] cloudstack pull request: Dynamically load drivers before creating ...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1553#issuecomment-221758855 This is a known issue, so this one should be ready now. Thx... --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Add Java Default Certificat Authorities i...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1555#issuecomment-221759081 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 0 Errors: 3 Duration: 8h 37m 16s ``` **Summary of the

[GitHub] cloudstack pull request: Add Java Default Certificat Authorities i...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1555#issuecomment-221759242 This is ready to merge now. The issue we are seeing is a known issue not related to this PR. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: CLOUDSTACK-9317: Enable/disable static NA...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1450#issuecomment-221762730 CI is clean and everything is green. I need some code review on this one. Thanks... --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: Lower the time we wait for interfaces to ...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1471#issuecomment-221762946 Can I get some code review on 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

[GitHub] cloudstack pull request: Fixes regarding VOLUME_DELETE events resu...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1491#issuecomment-221763412 This one needs review 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

[GitHub] cloudstack pull request: Undetected bug correct and refactor of th...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1499#issuecomment-221763535 Bump... @alexandrelimassantana this one has merge conflicts. Can you rebase and fix the conflicts? Thanks... --- If your project is set up for it, you can

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/669#issuecomment-221763988 This one needs some work. We have merge conflicts and we need another code review. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1251#issuecomment-221764160 Need code review on this one. Thx... --- 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 pull request: Made the adding new keyboard language sup...

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/669#issuecomment-221861017 Can I get one more review on this one? I think we are basically set otherwise. Thx... --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: CLOUDSTACK-8751 minimise downtime of netw...

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/866#issuecomment-221862006 @bvbharatk can you close and reopen this PR to kick off jenkins again to see if we can get this green? I will rerun CI on it to make sure we are still good

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221862724 @dsclose sorry to do this to you. Can you close and reopen again to kick off the jobs again? Thanks... --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-221863141 this one just does not want to turn green. :( wanna give it another shot @rhtyd? --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-9388: Remove string conversion...

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1561#issuecomment-221863597 @sanju1010 thanks for posting the test results with this change. This one should be good now. Thanks everyone... --- If your project is set up for it, you can

[GitHub] cloudstack pull request: removed unused HypervDummyResourceBase cl...

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1437#issuecomment-221871305 I think the only thing we are missing with this one is verification that the change works and does not break anything. @pedro-martins is there anything you can

[GitHub] cloudstack pull request: Add lsb-release dependency to mgmt server...

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1565#issuecomment-221872868 @ProjectMoon please close and reopen to kick off travis again. Thanks... --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: Lower the time we wait for interfaces to ...

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1471#issuecomment-221873120 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 8h 23m 38s

<    1   2   3   4   5   6   7   8   9   10   >