Monday, 2014-03-31

*** dpamio has joined #openstack-climate00:26
*** dpamio has quit IRC00:27
*** dpamio has joined #openstack-climate01:07
*** DinaBelova has quit IRC01:13
*** DinaBelova has joined #openstack-climate01:14
*** dpamio has quit IRC01:17
*** dpamio has joined #openstack-climate01:31
*** dpamio has quit IRC01:40
*** dpamio has joined #openstack-climate02:03
*** dpamio has quit IRC02:22
*** chandan_kumar has joined #openstack-climate05:50
*** chandan_kumar has quit IRC07:06
*** tomek_adamczewsk has joined #openstack-climate07:14
*** bauzas has joined #openstack-climate07:14
openstackgerritA change was merged to stackforge/climate: Adding migrate and lockfile to test_requirements  https://review.openstack.org/8401107:20
*** tomek_adamczewsk has quit IRC07:36
*** tomek_adamczewsk has joined #openstack-climate07:38
*** tomek_adamczewsk has quit IRC07:49
*** tomek_adamczewsk has joined #openstack-climate08:40
openstackgerritMicha? Kami?ski proposed a change to stackforge/climate: Extend Reservation API with additional properties  https://review.openstack.org/8346208:48
tomek_adamczewskHi all, I've a question about ending a physical host lease when there're instances running in it. I guess such a lease will fail, won't it? does in mean that climate will still block hosts (i.e. keep them in dedicated aggregate and not allow other leases to use them)?09:03
tomek_adamczewskmy use case is - user creates a lease, starts heat stacks with appropriate scheduler hint, lease times out... but user didn't stop his stack09:04
bauzastomek_adamczewsk: the end scenario is something we need to be operator-driven09:19
bauzastomek_adamczewsk: let me check how f_rossigneux implemented the first prototype09:20
*** tomek_adamczewsk has quit IRC09:52
*** tomek_adamczewsk has joined #openstack-climate09:53
bauzasDinaBelova: mkaminsk: you there ?10:11
DinaBelovame here10:11
mkaminski'm here10:11
bauzashttp://logs.openstack.org/62/83462/5/check/gate-climate-devstack-dsvm/1a82717/logs/devstacklog.txt.gz10:11
bauzasthere is a trouble with Alembic10:12
DinaBelovayep, see it10:12
bauzaswe should safely import logger without having an exception10:12
DinaBelovaoslo.config.cfg.ArgsAlreadyParsedError: arguments already parsed: cannot register CLI option10:12
bauzasDinaBelova: yup10:12
bauzasbecause of Alembic upgrade10:13
mkaminskso i will create a bug for it10:13
bauzasoh, just thinking it breaks the db stnc10:13
bauzassync10:13
bauzasI have to change my vote to -110:13
bauzas:(10:13
mkaminskhttps://bugs.launchpad.net/climate/+bug/130013210:14
bauzasmkaminsk: could you please remove the logger in your code and just put a FIXME comment mentioning the bug ?10:14
mkaminskyes i am preparing next patch set,10:21
DinaBelovamkaminsk, cool)10:24
openstackgerritMicha? Kami?ski proposed a change to stackforge/climate: Extend Reservation API with additional properties  https://review.openstack.org/8346210:26
*** tomek_adamczewsk has quit IRC11:01
*** tomek_adamczewsk has joined #openstack-climate11:08
*** tomek_adamczewsk has quit IRC11:14
*** tomek_adamczewsk has joined #openstack-climate11:16
openstackgerritSylvain Bauza proposed a change to stackforge/climate: Add extensions for API v2 controllers  https://review.openstack.org/7830911:51
openstackgerritSylvain Bauza proposed a change to stackforge/climate: Port to Pecan/WSME for API v2  https://review.openstack.org/7101111:51
*** saju_m has joined #openstack-climate11:51
bauzasDinaBelova: that's the final patchset :)11:51
DinaBelova;)11:51
DinaBelovayay)11:51
*** saju_m has quit IRC12:00
*** saju_m has joined #openstack-climate12:06
*** saju_m has quit IRC12:08
openstackgerritMicha? Kami?ski proposed a change to stackforge/climate: Reservation with No Host Available error returns HTTP 200  https://review.openstack.org/8411412:08
*** cmart has joined #openstack-climate12:36
openstackgerritA change was merged to stackforge/climate: Extend Reservation API with additional properties  https://review.openstack.org/8346213:11
*** pafuent has joined #openstack-climate13:21
cmartHello people13:27
cmartI'm working in the https://bugs.launchpad.net/climate/+bug/1299467 and I need to ask about the "create_image" action that is being executed on the "on_end" method of the vm plugin..13:28
DinaBelovaok13:28
DinaBelovacmart, o/13:29
*** cmart_ has joined #openstack-climate13:30
DinaBelovacmart_ or cmart?13:30
DinaBelova:)13:30
cmart_every time that DinaBelova says "HI", I got disconnected hahaah13:30
DinaBelovait's karma13:30
DinaBelova:(13:30
cmart_hahaha13:31
DinaBelovai'm bad ptl, your internet connection does not like me13:31
DinaBelovahm, or probably it's you connection who's bad)13:31
cmart_hahaha:)13:31
cmart_my chrome doesn't like irc rooms..13:32
cmart_that's why I'm using IE :p13:32
cmart_(just kidding)13:32
*** cmart has quit IRC13:32
DinaBelovao_013:32
*** cmart_ is now known as cmart13:32
DinaBelovawell, you may install irc client13:33
DinaBelovasomeone13:33
cmartI'm using Chatzilla.. and it works just fine :)13:33
DinaBelovaok)13:33
cmartso.. I was analyzing the issue related to https://bugs.launchpad.net/climate/+bug/1299467 with pafuent..13:34
DinaBelovayep13:34
cmartand I don't understand why are we calling nova to create an image when we are deleting (or ending) a lease reservation..13:34
DinaBelovathat are actions defined for end virtual lease13:35
DinaBelovaby default when lease ends13:35
DinaBelovavm might be not only simply deleted13:35
DinaBelovabut some other actions might be done13:35
DinaBelovalike image creation or suspending13:35
DinaBelovahttps://github.com/stackforge/climate/blob/master/climate/plugins/instances/vm_plugin.py#L2913:36
DinaBelovaby default these actions are creating image and instance deletion13:36
DinaBelovathe thing is that currently things that are done with resources on lease end and lease deletion13:36
DinaBelovaare the same13:36
DinaBelovaand that's why we have this situation13:36
DinaBelovait13:37
DinaBelovais interesting question what should be done with it13:37
cmartso we are creating images and then deleting them?13:37
DinaBelovano, we're creating image and delete vm)13:37
DinaBelovabecause vm is not more needed13:38
DinaBelovabut ii user wants to store it or some reasons after lease end13:38
DinaBelovaimage might be created13:38
cmartlike a snapshot..13:38
DinaBelovathen to start the same vm it was to the lease end time13:38
DinaBelovayrs13:38
DinaBelovait's basically snapshot13:38
DinaBelovabut there is no such word in nova dictionary13:38
DinaBelovait13:38
DinaBelovais called image13:38
DinaBelovaas it'll be used as image to create vm from later13:39
DinaBelovathat's terms thing13:39
DinaBelovajust words - basically it's snapshot thing13:39
cmartohh.. now I get it.. I thought that we were creating an image (glance style) :S13:40
cmartSo.. related to the bug.. the fix will be just to add another except block to capture the exception retrieved by nova client, right?13:41
DinaBelovainteresting thing really13:42
DinaBelovabecause13:42
DinaBelovasnapshot cannot be created in shelved state13:42
DinaBelovaas there is already snapshot there13:42
DinaBelovacalled shelved_<instance_name>13:43
DinaBelovawell, image13:43
DinaBelovanot snapshot13:43
cmart;)13:43
DinaBelovaso probably it should be warning raised like "vm is in shelved state so there is already snapshot, doing nothing for create_image stuff"13:43
DinaBelovaI guess smth like that13:44
DinaBelovaand you'll capture that nova exception13:44
DinaBelovaI guess it's enough her13:44
DinaBelovahere*13:44
cmartOK.. I'll add that and let you (and all the guys) know.. thanks!13:44
DinaBelovaok)13:44
pafuentHi. I'm doing the last tweaks to the trust in PR plugin, but I found some bugs doing it and I'm not sure if I tested properly. I will upload a patch as a Work in }13:48
pafuentWork in progress to let you put a lot of -113:48
pafuentBTW, would be great if swann patch is on master to avoid the dependency on the patch13:49
openstackgerritMicha? Kami?ski proposed a change to stackforge/climate: Reservation with No Host Available error returns HTTP 200  https://review.openstack.org/8411413:53
DinaBelovapafuent, sorry, will check Swann's change one more time now14:19
DinaBelovaand will merge it14:19
DinaBelovacompletely forgot about it on Friday14:19
DinaBelovasorry14:19
pafuentDinaBelova: No problem, Thanks14:19
*** cmart_ has joined #openstack-climate14:37
*** cmart has quit IRC14:38
*** cmart_ is now known as cmart14:39
*** cmart has quit IRC14:46
openstackgerritA change was merged to stackforge/climate: Use nova client wrapper in all plugins modules  https://review.openstack.org/7683015:01
*** cmart has joined #openstack-climate15:09
*** bauzas has quit IRC15:13
openstackgerritPablo Andres Fuente proposed a change to stackforge/climate: Added use of trusts to Physical Host plugin  https://review.openstack.org/8415815:41
*** tomek_adamczewsk has quit IRC15:54
openstackgerritPablo Andres Fuente proposed a change to stackforge/climate: Added use of trusts to Physical Host plugin  https://review.openstack.org/8415816:15
openstackgerritPablo Andres Fuente proposed a change to stackforge/climate: PR plugin uses host name to query Nova inventory  https://review.openstack.org/8418216:41
*** pafuent has left #openstack-climate16:42
*** pafuent has joined #openstack-climate16:47
pafuentI proposed two options to fix this bug https://bugs.launchpad.net/climate/+bug/1299122.17:11
pafuentI know the ugly one is really ugly, but I want to check if the 'good one' is really a 'good one'17:12
*** cmart has quit IRC17:30
*** cmart has joined #openstack-climate18:01
openstackgerritChristian Martinez proposed a change to stackforge/climate: Fixing issue with no started leases deletion  https://review.openstack.org/8419418:07
openstackgerritPablo Andres Fuente proposed a change to stackforge/climate: PR plugin uses host name to query Nova inventory  https://review.openstack.org/8418220:42
openstackgerritDiego Pamio proposed a change to stackforge/climate-nova: Unit tests for the ClimateFilter class  https://review.openstack.org/7939920:45
*** cmart has quit IRC20:54
*** pafuent has left #openstack-climate21:13

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!