*** dpamio has joined #openstack-climate | 00:26 | |
*** dpamio has quit IRC | 00:27 | |
*** dpamio has joined #openstack-climate | 01:07 | |
*** DinaBelova has quit IRC | 01:13 | |
*** DinaBelova has joined #openstack-climate | 01:14 | |
*** dpamio has quit IRC | 01:17 | |
*** dpamio has joined #openstack-climate | 01:31 | |
*** dpamio has quit IRC | 01:40 | |
*** dpamio has joined #openstack-climate | 02:03 | |
*** dpamio has quit IRC | 02:22 | |
*** chandan_kumar has joined #openstack-climate | 05:50 | |
*** chandan_kumar has quit IRC | 07:06 | |
*** tomek_adamczewsk has joined #openstack-climate | 07:14 | |
*** bauzas has joined #openstack-climate | 07:14 | |
openstackgerrit | A change was merged to stackforge/climate: Adding migrate and lockfile to test_requirements https://review.openstack.org/84011 | 07:20 |
---|---|---|
*** tomek_adamczewsk has quit IRC | 07:36 | |
*** tomek_adamczewsk has joined #openstack-climate | 07:38 | |
*** tomek_adamczewsk has quit IRC | 07:49 | |
*** tomek_adamczewsk has joined #openstack-climate | 08:40 | |
openstackgerrit | Micha? Kami?ski proposed a change to stackforge/climate: Extend Reservation API with additional properties https://review.openstack.org/83462 | 08:48 |
tomek_adamczewsk | Hi 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_adamczewsk | my use case is - user creates a lease, starts heat stacks with appropriate scheduler hint, lease times out... but user didn't stop his stack | 09:04 |
bauzas | tomek_adamczewsk: the end scenario is something we need to be operator-driven | 09:19 |
bauzas | tomek_adamczewsk: let me check how f_rossigneux implemented the first prototype | 09:20 |
*** tomek_adamczewsk has quit IRC | 09:52 | |
*** tomek_adamczewsk has joined #openstack-climate | 09:53 | |
bauzas | DinaBelova: mkaminsk: you there ? | 10:11 |
DinaBelova | me here | 10:11 |
mkaminsk | i'm here | 10:11 |
bauzas | http://logs.openstack.org/62/83462/5/check/gate-climate-devstack-dsvm/1a82717/logs/devstacklog.txt.gz | 10:11 |
bauzas | there is a trouble with Alembic | 10:12 |
DinaBelova | yep, see it | 10:12 |
bauzas | we should safely import logger without having an exception | 10:12 |
DinaBelova | oslo.config.cfg.ArgsAlreadyParsedError: arguments already parsed: cannot register CLI option | 10:12 |
bauzas | DinaBelova: yup | 10:12 |
bauzas | because of Alembic upgrade | 10:13 |
mkaminsk | so i will create a bug for it | 10:13 |
bauzas | oh, just thinking it breaks the db stnc | 10:13 |
bauzas | sync | 10:13 |
bauzas | I have to change my vote to -1 | 10:13 |
bauzas | :( | 10:13 |
mkaminsk | https://bugs.launchpad.net/climate/+bug/1300132 | 10:14 |
bauzas | mkaminsk: could you please remove the logger in your code and just put a FIXME comment mentioning the bug ? | 10:14 |
mkaminsk | yes i am preparing next patch set, | 10:21 |
DinaBelova | mkaminsk, cool) | 10:24 |
openstackgerrit | Micha? Kami?ski proposed a change to stackforge/climate: Extend Reservation API with additional properties https://review.openstack.org/83462 | 10:26 |
*** tomek_adamczewsk has quit IRC | 11:01 | |
*** tomek_adamczewsk has joined #openstack-climate | 11:08 | |
*** tomek_adamczewsk has quit IRC | 11:14 | |
*** tomek_adamczewsk has joined #openstack-climate | 11:16 | |
openstackgerrit | Sylvain Bauza proposed a change to stackforge/climate: Add extensions for API v2 controllers https://review.openstack.org/78309 | 11:51 |
openstackgerrit | Sylvain Bauza proposed a change to stackforge/climate: Port to Pecan/WSME for API v2 https://review.openstack.org/71011 | 11:51 |
*** saju_m has joined #openstack-climate | 11:51 | |
bauzas | DinaBelova: that's the final patchset :) | 11:51 |
DinaBelova | ;) | 11:51 |
DinaBelova | yay) | 11:51 |
*** saju_m has quit IRC | 12:00 | |
*** saju_m has joined #openstack-climate | 12:06 | |
*** saju_m has quit IRC | 12:08 | |
openstackgerrit | Micha? Kami?ski proposed a change to stackforge/climate: Reservation with No Host Available error returns HTTP 200 https://review.openstack.org/84114 | 12:08 |
*** cmart has joined #openstack-climate | 12:36 | |
openstackgerrit | A change was merged to stackforge/climate: Extend Reservation API with additional properties https://review.openstack.org/83462 | 13:11 |
*** pafuent has joined #openstack-climate | 13:21 | |
cmart | Hello people | 13:27 |
cmart | I'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 |
DinaBelova | ok | 13:28 |
DinaBelova | cmart, o/ | 13:29 |
*** cmart_ has joined #openstack-climate | 13:30 | |
DinaBelova | cmart_ or cmart? | 13:30 |
DinaBelova | :) | 13:30 |
cmart_ | every time that DinaBelova says "HI", I got disconnected hahaah | 13:30 |
DinaBelova | it's karma | 13:30 |
DinaBelova | :( | 13:30 |
cmart_ | hahaha | 13:31 |
DinaBelova | i'm bad ptl, your internet connection does not like me | 13:31 |
DinaBelova | hm, 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 :p | 13:32 |
cmart_ | (just kidding) | 13:32 |
*** cmart has quit IRC | 13:32 | |
DinaBelova | o_0 | 13:32 |
*** cmart_ is now known as cmart | 13:32 | |
DinaBelova | well, you may install irc client | 13:33 |
DinaBelova | someone | 13:33 |
cmart | I'm using Chatzilla.. and it works just fine :) | 13:33 |
DinaBelova | ok) | 13:33 |
cmart | so.. I was analyzing the issue related to https://bugs.launchpad.net/climate/+bug/1299467 with pafuent.. | 13:34 |
DinaBelova | yep | 13:34 |
cmart | and I don't understand why are we calling nova to create an image when we are deleting (or ending) a lease reservation.. | 13:34 |
DinaBelova | that are actions defined for end virtual lease | 13:35 |
DinaBelova | by default when lease ends | 13:35 |
DinaBelova | vm might be not only simply deleted | 13:35 |
DinaBelova | but some other actions might be done | 13:35 |
DinaBelova | like image creation or suspending | 13:35 |
DinaBelova | https://github.com/stackforge/climate/blob/master/climate/plugins/instances/vm_plugin.py#L29 | 13:36 |
DinaBelova | by default these actions are creating image and instance deletion | 13:36 |
DinaBelova | the thing is that currently things that are done with resources on lease end and lease deletion | 13:36 |
DinaBelova | are the same | 13:36 |
DinaBelova | and that's why we have this situation | 13:36 |
DinaBelova | it | 13:37 |
DinaBelova | is interesting question what should be done with it | 13:37 |
cmart | so we are creating images and then deleting them? | 13:37 |
DinaBelova | no, we're creating image and delete vm) | 13:37 |
DinaBelova | because vm is not more needed | 13:38 |
DinaBelova | but ii user wants to store it or some reasons after lease end | 13:38 |
DinaBelova | image might be created | 13:38 |
cmart | like a snapshot.. | 13:38 |
DinaBelova | then to start the same vm it was to the lease end time | 13:38 |
DinaBelova | yrs | 13:38 |
DinaBelova | it's basically snapshot | 13:38 |
DinaBelova | but there is no such word in nova dictionary | 13:38 |
DinaBelova | it | 13:38 |
DinaBelova | is called image | 13:38 |
DinaBelova | as it'll be used as image to create vm from later | 13:39 |
DinaBelova | that's terms thing | 13:39 |
DinaBelova | just words - basically it's snapshot thing | 13:39 |
cmart | ohh.. now I get it.. I thought that we were creating an image (glance style) :S | 13:40 |
cmart | So.. 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 |
DinaBelova | interesting thing really | 13:42 |
DinaBelova | because | 13:42 |
DinaBelova | snapshot cannot be created in shelved state | 13:42 |
DinaBelova | as there is already snapshot there | 13:42 |
DinaBelova | called shelved_<instance_name> | 13:43 |
DinaBelova | well, image | 13:43 |
DinaBelova | not snapshot | 13:43 |
cmart | ;) | 13:43 |
DinaBelova | so 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 |
DinaBelova | I guess smth like that | 13:44 |
DinaBelova | and you'll capture that nova exception | 13:44 |
DinaBelova | I guess it's enough her | 13:44 |
DinaBelova | here* | 13:44 |
cmart | OK.. I'll add that and let you (and all the guys) know.. thanks! | 13:44 |
DinaBelova | ok) | 13:44 |
pafuent | Hi. 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 |
pafuent | Work in progress to let you put a lot of -1 | 13:48 |
pafuent | BTW, would be great if swann patch is on master to avoid the dependency on the patch | 13:49 |
openstackgerrit | Micha? Kami?ski proposed a change to stackforge/climate: Reservation with No Host Available error returns HTTP 200 https://review.openstack.org/84114 | 13:53 |
DinaBelova | pafuent, sorry, will check Swann's change one more time now | 14:19 |
DinaBelova | and will merge it | 14:19 |
DinaBelova | completely forgot about it on Friday | 14:19 |
DinaBelova | sorry | 14:19 |
pafuent | DinaBelova: No problem, Thanks | 14:19 |
*** cmart_ has joined #openstack-climate | 14:37 | |
*** cmart has quit IRC | 14:38 | |
*** cmart_ is now known as cmart | 14:39 | |
*** cmart has quit IRC | 14:46 | |
openstackgerrit | A change was merged to stackforge/climate: Use nova client wrapper in all plugins modules https://review.openstack.org/76830 | 15:01 |
*** cmart has joined #openstack-climate | 15:09 | |
*** bauzas has quit IRC | 15:13 | |
openstackgerrit | Pablo Andres Fuente proposed a change to stackforge/climate: Added use of trusts to Physical Host plugin https://review.openstack.org/84158 | 15:41 |
*** tomek_adamczewsk has quit IRC | 15:54 | |
openstackgerrit | Pablo Andres Fuente proposed a change to stackforge/climate: Added use of trusts to Physical Host plugin https://review.openstack.org/84158 | 16:15 |
openstackgerrit | Pablo Andres Fuente proposed a change to stackforge/climate: PR plugin uses host name to query Nova inventory https://review.openstack.org/84182 | 16:41 |
*** pafuent has left #openstack-climate | 16:42 | |
*** pafuent has joined #openstack-climate | 16:47 | |
pafuent | I proposed two options to fix this bug https://bugs.launchpad.net/climate/+bug/1299122. | 17:11 |
pafuent | I 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 IRC | 17:30 | |
*** cmart has joined #openstack-climate | 18:01 | |
openstackgerrit | Christian Martinez proposed a change to stackforge/climate: Fixing issue with no started leases deletion https://review.openstack.org/84194 | 18:07 |
openstackgerrit | Pablo Andres Fuente proposed a change to stackforge/climate: PR plugin uses host name to query Nova inventory https://review.openstack.org/84182 | 20:42 |
openstackgerrit | Diego Pamio proposed a change to stackforge/climate-nova: Unit tests for the ClimateFilter class https://review.openstack.org/79399 | 20:45 |
*** cmart has quit IRC | 20:54 | |
*** pafuent has left #openstack-climate | 21:13 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!