Re: [ACP Doctor] What is it?

2016-04-05 Thread Ian Rae
I don't believe this is freely available, rather is a tool Citrix developed for helping troubleshoot CCP customer deployments. I would imagine that Accelerite owns this tool now and it is likely available if you are an ACP customer, but not necessarily for ACS users. Probably best for Accelerite

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

2016-04-05 Thread alexandrelimassantana
Github user alexandrelimassantana commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1464#discussion_r58633962 --- Diff: tools/marvin/marvin/lib/vcenter.py --- @@ -183,7 +192,157 @@ def get_clusters(self, dc, clus=None): """

Re: [LOGISTICS] Move 'apache/cloudstack' to 'apache-cloudstack' org

2016-04-05 Thread Will Stevens
Thank you. Will do. On Apr 5, 2016 6:38 PM, "Chris Mattmann" wrote: > Hey Will, I’m not on the infra@ team, but I am a board member > and have provided by +1 and support (and I think there is general > support there informally). > > However, I would point you at recent

Re: [LOGISTICS] Move 'apache/cloudstack' to 'apache-cloudstack' org

2016-04-05 Thread Chris Mattmann
Hey Will, I’m not on the infra@ team, but I am a board member and have provided by +1 and support (and I think there is general support there informally). However, I would point you at recent threads related to expanding the MATT experiment that have surfaced on board@ in recent days and I would

Re: [LOGISTICS] Move 'apache/cloudstack' to 'apache-cloudstack' org

2016-04-05 Thread Will Stevens
So this topic has been very slow moving, so I will keep pushing this along. @Chris, are you on the ASF Infra team? From your perspective are we missing anything to move forward? @Sam & David, is there anything else I need to do from the ASF side other than working out the logistics with the

Re: [ACP Doctor] What is it?

2016-04-05 Thread Will Stevens
It used to be CCP Doctor and it is not in ACS from my understanding. It is a set of scripts that will do basic validation of a CloudStack setup. It does things like verify the system VMs are running and the connectivity is working between all of the systems. It also does some checking to make

[ACP Doctor] What is it?

2016-04-05 Thread ilya
Saw ACP Doctor in CCP release notes from Accelerite. Curious what it is, is it integrated into cloudstack or collection of shell scripts? Thanks ilya

[GitHub] cloudstack pull request: Add ability to download templates in Swif...

2016-04-05 Thread syed
Github user syed commented on the pull request: https://github.com/apache/cloudstack/pull/1332#issuecomment-205981012 Sure @swill . I've sqashed them --- 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-9336 surround the execution of...

2016-04-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1463#issuecomment-205968247 I replayed the functionality tested in the second test and it is working. Will have to look into the test code to see what is happening. retesting the

[GitHub] cloudstack pull request: CLOUDSTACK-8611:Handle SSH if server "for...

2016-04-05 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-205968363 Well, if this PR gets merged, I do not know much how we can close the PR #561. Maybe @swill or @DaanHoogland can help with that. Now the code

[GitHub] cloudstack pull request: CLOUDSTACK-9297 - Reworked logic in Stora...

2016-04-05 Thread mike-tutkowski
Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/1441#issuecomment-205949475 Thanks, Will! On Apr 5, 2016, at 12:35 PM, Will Stevens > wrote: I will get

[GitHub] cloudstack pull request: CLOUDSTACK-9297 - Reworked logic in Stora...

2016-04-05 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1441#issuecomment-205937640 I will get CI run against this PR as soon as I have my CI reinstalled. Thanks @mike-tutkowski --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-8611:Handle SSH if server "for...

2016-04-05 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-205932840 @rafaelweingartner now this commit is not cherry-picked from the commit (b9181c689e0e7b5f1e28c81d73710196dfabd0ba). When this PR be merged (or closed),

[GitHub] cloudstack pull request: Add ability to download templates in Swif...

2016-04-05 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1332#issuecomment-205926253 @syed can you squash the commits. This is best applied as a single commit. Thx. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: Add ability to download templates in Swif...

2016-04-05 Thread syed
Github user syed commented on the pull request: https://github.com/apache/cloudstack/pull/1332#issuecomment-205922970 @bvbharatk it looks to be a problem with the test. Volume creation should work irrespective of my changes. --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: CLOUDSTACK-9333: Exclude clusters from OV...

2016-04-05 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1457#issuecomment-205917619 thanks @rafaelweingartner and @swill ! --- 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 pull request: CLOUDSTACK-9336 surround the execution of...

2016-04-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1463#issuecomment-205909585 two failures. they seem unrelated but double ckecking

[GitHub] cloudstack pull request: CLOUDSTACK-9333: Exclude clusters from OV...

2016-04-05 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1457#issuecomment-205892175 Perfect. Thank you both for verifying this. I probably won't be able to get to testing this till tomorrow evening because I am about to reinstall my CI to get my

[GitHub] cloudstack pull request: CLOUDSTACK-9323: Fix cancel host maintena...

2016-04-05 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1454#issuecomment-205892559 @abhinandanprateek slight delay. I am going to blow away my CI setup and reinstall it now to get it running on SSD drives so I can better parallelize my tests.

[GitHub] cloudstack pull request: CLOUDSTACK-9333: Exclude clusters from OV...

2016-04-05 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1457#issuecomment-205885858 Good question, those changes affect every single hypervisor. I have reviewed the code and after that I can say that if those changes work for KVM, I

[GitHub] cloudstack pull request: CLOUDSTACK-9323: Fix cancel host maintena...

2016-04-05 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1454#issuecomment-205885703 @abhinandanprateek: Thank you. I will run the whole set of tests against it again tonight. I have to let the current tests finish and run this on its own because I

[GitHub] cloudstack pull request: CLOUDSTACK-9333: Exclude clusters from OV...

2016-04-05 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1457#issuecomment-205883002 @rafaelweingartner I can run tests via KVM, but I don't have the ability to run tests against VMware hardware. Are tests against KVM good enough in this case? ---

[GitHub] cloudstack pull request: CLOUDSTACK-9333: Exclude clusters from OV...

2016-04-05 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1457#issuecomment-205882042 For me everything (with the code) is ok now, LGTM for the PR. Could you please run functional tests? --- If your project is set up for it, you can

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-04-05 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-205867477 @DaanHoogland feel free to split this however you feel, I've already moved the Travis fix to a separate PR --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: CLOUDSTACK-9333: Exclude clusters from OV...

2016-04-05 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1457#issuecomment-205857772 @rafaelweingartner cool! I added it. --- 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 pull request: CLOUDSTACK-9287 - Fix unique mac address ...

2016-04-05 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1413#issuecomment-205831656 Thank you. I've raised CLOUDSTACK-9339 on Jira about the non-VPC RvR issues. As my company needs those issues solved in the short term I'll begin working on a fix

[GitHub] cloudstack pull request: CLOUDSTACK-9287 - Fix unique mac address ...

2016-04-05 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1413#issuecomment-205820963 @dsclose This bug exists as far back as 4.6 I believe, as that was when RvR for VPC was introduced. So you might want base your patch on 4.7 and then it can be

RE: [GitHub] cloudstack pull request: CLOUDSTACK-9287 - Fix unique mac address ...

2016-04-05 Thread Dean Close
Thank you, I'm going to raise a separate bug report about the issues I've found with RvR setups. If I work on a patch I'll fork from the master branch unless I hear otherwise. Dean -Original Message- From: kiwiflyer [mailto:g...@git.apache.org] Sent: 05 April 2016 14:28 To:

[GitHub] cloudstack pull request: speedup iptables setup

2016-04-05 Thread kollyma
Github user kollyma commented on the pull request: https://github.com/apache/cloudstack/pull/1449#issuecomment-205809122 @Swill: tried to squash the commits shortly from holidays. Did not worked as expected, messed up my PR. I need to debug this when I am back next week... --- If

[GitHub] cloudstack pull request: CLOUDSTACK-9287 - Fix unique mac address ...

2016-04-05 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1413#issuecomment-205804779 @dsclose - just FYI. As far as I'm aware, this PR has not been committed yet. @ustcweizhou has requested a new PR/rebase. I just pulled these into our

jenkins restart

2016-04-05 Thread Daan Hoogland
sorry for not giving prior notice. I am updating plugins on jenkins.buildacloud.org. It will restart. -- Daan

[GitHub] cloudstack pull request: CLOUDSTACK-9323: Fix cancel host maintena...

2016-04-05 Thread abhinandanprateek
Github user abhinandanprateek commented on the pull request: https://github.com/apache/cloudstack/pull/1454#issuecomment-205780575 Updated marvin test to use the builtin template. @swill --- 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-9337]Enhance vcenter.py to cr...

2016-04-05 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1464 [CLOUDSTACK-9337]Enhance vcenter.py to created data center in vcenter server automatically These changes have been made to support vmware deployments in CI. For CI to create cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9336 surround the execution of...

2016-04-05 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/1463 CLOUDSTACK-9336 surround the execution of baremetal-vr.py with condition You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: CLOUDSTACK-9336 surround the execution of...

2016-04-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1463#issuecomment-205763867 integration tests pending --- 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-8611:Handle SSH if server "for...

2016-04-05 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-205755861 Thanks @DaanHoogland, I will check on that. --- 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: maven: Upgrade dependency versions

2016-04-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-205755506 @bhaisaab I'm kind of confused, you write about three parts but I count four. I would agree that the bouncy castle is a completely seperate one. I think mvn

Re: [QUESTION] baremetal router

2016-04-05 Thread Daan Hoogland
thakns Hari, I'll make a small pr for putting the baremetal-vr.py under a condition. On Tue, Apr 5, 2016 at 11:10 AM, Harikrishna Patnala < harikrishna.patn...@accelerite.com> wrote: > Hi Daan, > > Yes, this should run only in Virtual Router. Instead of moving it to > existing “if” block, better

Re: monday morning reading

2016-04-05 Thread Wido den Hollander
> Op 4 april 2016 om 9:28 schreef Daan Hoogland : > > > H, for those interested in how we get to improve, here is something I > stumbled upon on reddit [1] > > I must admit I think most of us are already working as described but > still,... it touches on some of the

Re: [QUESTION] baremetal router

2016-04-05 Thread Harikrishna Patnala
Hi Daan, Yes, this should run only in Virtual Router. Instead of moving it to existing “if” block, better to have a seperate check only for normal router, since we don’t support baremetal deployment in VPC. Regarding the later question, with the current implementation I don’t think we can

RE: Redundant Router Interfaces

2016-04-05 Thread Dean Close
Hi Wei, Have you noticed the PR #1413 at https://github.com/apache/cloudstack/pull/1413 ? The commit at https://github.com/apache/cloudstack/commit/8bbea5eeb6598b213b78a324c184841dbba69280 changes the check_is_up method considerably such that your patch would no-longer apply. That PR #1413

[GitHub] cloudstack pull request: CLOUDSTACK-9287 - Fix unique mac address ...

2016-04-05 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1413#issuecomment-205716220 @wilderrodrigues @ustcweizhou Sorry to arrive late to the party but this appears to break/ignore some RvR functionality. 4 things in particular stand out:

[GitHub] cloudstack pull request: CLOUDSTACK-8611:Handle SSH if server "for...

2016-04-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-205715741 the other one is persistant i'm afraid: [1459.results.vpc_redundant.txt](https://github.com/apache/cloudstack/files/204056/1459.results.vpc_redundant.txt)

[GitHub] cloudstack pull request: CLOUDSTACK-9334: Support jenv and pyenv t...

2016-04-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1460#issuecomment-205705013 @jburwell the failure is not in rat but in the unit tests for the secondary storage controller project --- If your project is set up for it, you can reply to

Re: Hooking into the SecurityGroups

2016-04-05 Thread Nux!
Thanks Jayapal! I won't propose this change as a pull request since this is a pretty custom job. Myipset (with a different name) will include all our data centre (AS) subnets, the end result being that with a simple "iptables-save -c" I can now know what traffic was done against our data

[GitHub] cloudstack pull request: CLOUDSTACK-8611:Handle SSH if server "for...

2016-04-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-205684941 the first one wwas a false positive: [1459.results.password.txt](https://github.com/apache/cloudstack/files/203940/1459.results.password.txt) --- If

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

2016-04-05 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-205684992 LGTM based on the code review !! Please squash the commits into single commit. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: CLOUDSTACK-8611:Handle SSH if server "for...

2016-04-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1459#issuecomment-205679998 @GabrielBrascher I ran the regression suite as defined by SBP and found two failures. I am rerunning them individually to see if I can find out some more.

[GitHub] cloudstack pull request: CLOUDSTACK-9323: Fix cancel host maintena...

2016-04-05 Thread abhinandanprateek
Github user abhinandanprateek commented on the pull request: https://github.com/apache/cloudstack/pull/1454#issuecomment-205672352 On comparing test execution times with macchinina Vs Centos builtin template I did not find much of a difference. So modifying the test to go with the

[GitHub] cloudstack pull request: CLOUDSTACK-9335: fix typo in dashboard's ...

2016-04-05 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1462#issuecomment-205671313 Thanks @terbolous, fixed --- 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