On Thu, Oct 17, 2013 at 6:55 AM, Chiradeep Vittal < chiradeep.vit...@citrix.com> wrote:
> Fixed pep8 and other issues (with flake8). > > Does this warrant a new release? > The idea of fast iteration and release process was to get even smallest feature that is tested and works out in the public, so yes let's test it, fix/add docs and whatnot, and release it. If you insist we can have weekly/monthly release windows (but no longer than that please), so we won't end up releasing for every single feature. Advise? Cheers. > > On 10/16/13 11:13 AM, "Chiradeep Vittal" <chiradeep.vit...@citrix.com> > wrote: > > >Passwords can be changed too. No difference in security, IMO. > > > >Plus the api key option is always there. > > > >In fact I first wrote it as an option ( --populate-api-keys > >USERNAME:PASSWORD), but decided that it didn't buy anything. > > > >The original code didn't pass pep8 (imports not being used etc), that's > >probably a separate patch. > > > > > >-----Original Message----- > >From: rohityada...@gmail.com [mailto:rohityada...@gmail.com] On Behalf Of > >Rohit Yadav > >Sent: Wednesday, October 16, 2013 10:50 AM > >To: dev@cloudstack.apache.org > >Cc: us...@cloudstack.apache.org > >Subject: Re: [cloudmonkey] username / password support > > > >I never intended to have support for password as I thought people will > >end up using and storing plain text username/password, keys (specific for > >cloudmonkey) are revokable :) > >Maybe we can use usernames/password initially to create keys or maybe let > >users decided what they want. If this could be refactored as a reusable > >module Prasanna can use this for Marvin. > > > >On Wed, Oct 16, 2013 at 10:21 AM, Chiradeep Vittal < > >chiradeep.vit...@citrix.com> wrote: > > > >> Sorry, urllib2 and cookiejar looked just too painful. > >> > > > >Totally agree, we should throw away painful things for better ones. With > >requests as dependency in setup.py so no one has to install it manually. > >You just do pip install --upgrade stuff and pip would get all the deps > >from setup.py. Thanks Prasanna for adding that, maybe specify a minimum > >or maximum version? > > > >Initially my idea was to use least possible dependency, like have > >cloudmonkey pure python 2.6 program that just uses standard libs but > >eventually added prettytable and pygments. We should have more good > >stuff, remove painful libs. > > > >Lastly, could the code be pep8'd, tested and released :) > > > >Cheers, > >bhaisaab > > > > > >> > >> On 10/15/13 9:26 PM, "Prasanna Santhanam" <t...@apache.org> wrote: > >> > >> > > >> >I fixed this temporarily by adding the requests as a dependency. But > >> >may be we could do the login using urllib2 itself to avoid the > >> >requests dependency. > >> > > >> >+1 to username,password login. I need to add that to marvin too. > >> > > >> >On Wed, Oct 16, 2013 at 11:38:40AM +0800, Ryan Lei wrote: > >> >> I was unable to run cloudmonkey after applying your update. > >> >> My commands were: > >> >> > >> >> $ git checkout username_password_support $ git pull $ python > >> >> setup.py build $ python setup.py install $ cloudmonkey > >> >> > >> >> Then I got this import error: > >> >> Import error in cloudmonkey.requester : No module named requests > >> >> > >> >> Switching back to master branch runs fine, however. > >> >> > >> >> > >> >> > >> >>-------------------------------------------------------------------- > >> >>----- > >> >>------------------ > >> >> Yu-Heng (Ryan) Lei, Associate Researcher Chunghwa Telecom > >> >>Laboratories / Cloud Computing Laboratory > >> >> > >> >>ryan...@cht.com.tw< > >> https://email.cht.com.tw/owa/redir.aspx?C=-wE1FEC3G0SW > >> >>YpVkiWo8SsDdf3ZqO9AIuAPTzRnFYCUi-z4YljtI_hyVKkNHfn9F1Bn-vUWJnQ4.&URL > >> >>=mail > >> >>to%3aryanlei%40cht.com.tw> > >> >> or > >> >> ryanlei750...@gmail.com > >> >> > >> >> > >> >> > >> >> On Wed, Oct 16, 2013 at 11:23 AM, Chiradeep Vittal < > >> >> chiradeep.vit...@citrix.com> wrote: > >> >> > >> >> > Hi folks, > >> >> > > >> >> > I modified cloudmonkey to work off of username and password. > >> >> > Feedback > >> >>and > >> >> > testing required. > >> >> > The code is in the branch username_password_support > >> >> > http://goo.gl/5xTgo5 > >> >> > > >> >> > Thanks > >> >> > -- > >> >> > Chiradeep > >> >> > > >> > > >> >-- > >> >Prasanna., > >> > > >> >------------------------ > >> >Powered by BigRock.com > >> > > >> > >> > >
