Re: Python Question (with regards to Marvin)

2016-04-30 Thread Tutkowski, Mike
Here are my tests (run from http://ideone.com/). The short story is that having multiple methods with the same name (even if one is an instance method and one is a class method) should probably not be done. If you try to invoke the instance method (ex. test.run()), the last method by that name

Re: Python Question (with regards to Marvin)

2016-04-30 Thread Tutkowski, Mike
I can play around with it later tonight. I'm not home at the moment. When I did invoke it as Test.run(), it invoked the class method (the class method was listed after the instance method for that test, so I wasn't surprised that the class method did, in fact, get executed there). What I did

Re: Python Question (with regards to Marvin)

2016-04-30 Thread Will Stevens
That's strange. That means the @classmethod decorator is not working. You should have gotten the instance method in both cases. What if you don't instantiate Test and only do the following. Test.run() In both cases. On Apr 30, 2016 6:04 PM, "Tutkowski, Mike" wrote:

Re: Python Question (with regards to Marvin)

2016-04-30 Thread Tutkowski, Mike
I ran this with an online Python tool and it calls the class method: 1 class Test: 2 def run(self): 3 print 'instance hi' 4 5 @classmethod 6 def run(cls): 7 print 'class hi' 8 9 test = Test() 10 11 test.run() If

[GitHub] cloudstack pull request: CLOUDSTACK-9365 : updateVirtualMachine wi...

2016-04-30 Thread cristofolini
Github user cristofolini commented on the pull request: https://github.com/apache/cloudstack/pull/1523#issuecomment-215996953 @nlivens May I suggest extracting lines 2501-2514 to their own separate method? That would also allow you to create separate test cases for each possibility

[GitHub] cloudstack pull request: KVM: Fix plugging ip addresses to the wro...

2016-04-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1524#issuecomment-215991817 Jenkins error says: `Unable to instrument project`, seems like a workspace issue or such. --- If your project is set up for it, you can reply to this email

Re: Python Question (with regards to Marvin)

2016-04-30 Thread Will Stevens
I am on my phone so I have not been able to research this for you. I think you are right for the most part. Instead of multiple methods, python kind of fakes overloading by being to have named function arguments which can have default values, so you can call the method with a dynamic number of

Re: Rafael Weingartner joining the PMC

2016-04-30 Thread Marco Sinhoreli
Congrats Rafael! Marco Sinhoreli Consultant Manager marco.sinhor...@shapeblue.com mobile: +55 21 98276 3636 Praia de Botafogo 501, bloco 1 - sala 101 – Botafogo Rio de Janeiro, RJ - Brazil - CEP 22250-040 office: + 55 21 2586 6390 | fax: +55 21 2586 6002 http://www.shapeblue.com/ | twitter:

Re: Python Question (with regards to Marvin)

2016-04-30 Thread Tutkowski, Mike
Will - You can override a method in Python, but can you overload it? http://stackoverflow.com/questions/10202938/how-do-i-use-method-overloading-in-python > On Apr 30, 2016, at 6:23 AM, Will Stevens wrote: > > Here is a pretty good explanation. > >

Re: [ANNOUNCE] New committer: Simon Weller

2016-04-30 Thread ilya
thanks Simon and congrats! On 4/28/16 2:30 PM, Simon Weller wrote: > Thanks everyone! I look forward to continuing to help out where I can as we > march towards 4.9. > > - Si > > From: Pierre-Luc Dion > Sent: Thursday, April 28,

[GitHub] cloudstack pull request: zip the processed json files so we save d...

2016-04-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1421#issuecomment-215988060 @swill force pushed --- 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: Reimplement router.redundant.vrrp.interva...

2016-04-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1486#issuecomment-215988010 @swill force pushed --- 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: Restore iptables at once using iptables-r...

2016-04-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1482#issuecomment-215987980 @swill conflict resolved --- 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: Bump ssh retries to prevent false positiv...

2016-04-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1473#issuecomment-215987371 @swill Force pushed. --- 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: KVM: Fix plugging ip addresses to the wro...

2016-04-30 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1524#issuecomment-215986875 Strangely enough, I just ran into this problem yesterday.I'll pull it in. Thanks Remi! --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: KVM: Fix plugging ip addresses to the wro...

2016-04-30 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1524 KVM: Fix plugging ip addresses to the wrong interface Fix for KVM, where the public ip would be added to the guest network. Reason is because it used 'broadcastUrl' as an index, which

[GitHub] cloudstack pull request: Followup fix for #1162: Add support for n...

2016-04-30 Thread ProjectMoon
Github user ProjectMoon commented on the pull request: https://github.com/apache/cloudstack/pull/1330#issuecomment-215979893 Right now I suppose I still need to make a unit test. And I guess also re-open against 4.7. --- If your project is set up for it, you can reply to this email

Re: Python Question (with regards to Marvin)

2016-04-30 Thread Tutkowski, Mike
Allow me to clarify. :-) host.enableMaintenance(apiclient) is how I was invoking it. Just like a regular instance method. For some reason, it wanted to invoke the class method when I did that and was complaining about the lack of a parameter. That being the case, I switched my code to the

Re: Python Question (with regards to Marvin)

2016-04-30 Thread Will Stevens
You are probably getting this error because you are trying to call: Host.enableMaintenance(client) Check my examples above for how to call it. Sorry I am on my phone, so I am not very efficient with my phone and cant civet you better details. :) On Apr 30, 2016 8:23 AM, "Will Stevens"

Re: Python Question (with regards to Marvin)

2016-04-30 Thread Will Stevens
Here is a pretty good explanation. http://stackoverflow.com/questions/136097/what-is-the-difference-between-staticmethod-and-classmethod-in-python I am guessing that both exist because the function is called both with a host instance and with the class itself. Class instance example:

Re: Python Question (with regards to Marvin)

2016-04-30 Thread Prashanth Manthena
Hi Mike, classmethod's first argument is always its class as it is always bound to the class, which is not the case with normal methods. Thus, both of the above methods can in essence still co-exist in base.py as they serve different purposes. In the latter classmethod, you need to pass your

Re: Access to Docker Hub repo

2016-04-30 Thread Wido den Hollander
Thanks! That helps a lot. Wido > Op 30 april 2016 om 9:05 schreef Sebastien Goasguen : > > > done > > > On Apr 30, 2016, at 8:58 AM, Wido den Hollander wrote: > > > > Hi, > > > > I am working on the Ubuntu 14.04 and 16.04 packaging. Using Docker I want >

Re: Access to Docker Hub repo

2016-04-30 Thread Sebastien Goasguen
done > On Apr 30, 2016, at 8:58 AM, Wido den Hollander wrote: > > Hi, > > I am working on the Ubuntu 14.04 and 16.04 packaging. Using Docker I want to > start building the packages for all distros. > > I'd like to publish the Docker images to Docker Hub: >

Access to Docker Hub repo

2016-04-30 Thread Wido den Hollander
Hi, I am working on the Ubuntu 14.04 and 16.04 packaging. Using Docker I want to start building the packages for all distros. I'd like to publish the Docker images to Docker Hub: https://hub.docker.com/r/cloudstack/ Can somebody grant 'widodh' access to the registry? Wido