| *** Nikolay_St has joined #openstack-climate | 04:13 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 05:22 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 06:29 | |
| *** YorikSar has quit IRC | 07:08 | |
| *** DinaBelova_ is now known as DinaBelova | 07:18 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 07:22 | |
| *** Nikolay_St has quit IRC | 07:38 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_a | 07:39 | |
| *** SergeyLukjanov_a is now known as SergeyLukjanov_ | 07:40 | |
| *** YorikSar has joined #openstack-climate | 07:46 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 07:49 | |
| *** bauzas has joined #openstack-climate | 08:01 | |
| *** Nikolay_St has joined #openstack-climate | 08:32 | |
| bauzas | morning all | 08:39 |
|---|---|---|
| bauzas | Nikolay_St: https://review.openstack.org/68609 | 08:39 |
| DinaBelova | o/ | 08:41 |
| bauzas | Nikolay_St: when running keystone CLI, it provides me --service, not --service_id | 08:41 |
| bauzas | Nikolay_St: for some reason, my comment wasn't published :D | 08:41 |
| bauzas | so, I'm just passing --service climate | 08:42 |
| bauzas | I don't need to get the service_id, so the step 2 is unnecessary for me | 08:42 |
| bauzas | Nikolay_St: here is why I putted this comment | 08:42 |
| bauzas | Nikolay_St: apologies again, it was unclear :( | 08:42 |
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Add notes about Keystone v3 endpoint to README.rst https://review.openstack.org/68609 | 08:43 |
| Nikolay_St | bauzas: I see | 08:55 |
| Nikolay_St | bauzas: I'll fix it | 08:56 |
| Nikolay_St | bauzas: :D | 08:56 |
| Nikolay_St | bauzas: but wait | 08:57 |
| Nikolay_St | bauzas: we create another service with name keystone so, the only difference will be service-id | 08:57 |
| Nikolay_St | so we need it | 08:58 |
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Add notes about Keystone v3 endpoint to README.rst https://review.openstack.org/68609 | 08:59 |
| DinaBelova | Nikolay_St, as I got Sylvain's point me may create service with name not keystone | 09:01 |
| DinaBelova | but keystone v3 | 09:01 |
| DinaBelova | keystonev3, sorry | 09:01 |
| DinaBelova | and that will allow to use --service flag | 09:02 |
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Implement basic plugin for VM management https://review.openstack.org/49142 | 09:04 |
| bauzas | FYI, scroiset is off today | 09:09 |
| bauzas | so, I'm the only guy for reviewing your code :) | 09:09 |
| DinaBelova | bauzas, :D | 09:10 |
| DinaBelova | good luck :D | 09:10 |
| bauzas | I'm about testing the plugin without the nova extension | 09:11 |
| DinaBelova | bauzas, that will be hard... | 09:11 |
| bauzas | as the nova extension is only responsible for shelving the instance | 09:11 |
| DinaBelova | although you may boot vm | 09:11 |
| DinaBelova | shelve it | 09:11 |
| bauzas | I just need to boot a VM, shelve it | 09:11 |
| DinaBelova | and then create lease | 09:12 |
| bauzas | and manually create a lease | 09:12 |
| DinaBelova | yep | 09:12 |
| bauzas | :) | 09:12 |
| DinaBelova | :D | 09:12 |
| *** Nikolay_St has quit IRC | 09:12 | |
| bauzas | the only issue is that I'll need to shelve the instance thanks to the PAI | 09:12 |
| bauzas | API | 09:12 |
| bauzas | don't think the CLI supports it | 09:13 |
| DinaBelova | it supprots, as i remember | 09:13 |
| bauzas | oh cool | 09:13 |
| DinaBelova | Nick was using that | 09:13 |
| DinaBelova | while testing | 09:13 |
| bauzas | ok, will check | 09:13 |
| *** Nikolay_St has joined #openstack-climate | 09:14 | |
| DinaBelova | stack@ubuntu-12:~/devstack$ nova help | grep shelve | 09:14 |
| DinaBelova | shelve Shelve a server. | 09:14 |
| DinaBelova | shelve-offload Remove a shelved server from the compute node. | 09:14 |
| DinaBelova | unshelve Unshelve a server. | 09:14 |
| bauzas | btw, about the extension, how could I run it ? | 09:14 |
| DinaBelova | it is :) | 09:14 |
| bauzas | how can I install it ? | 09:14 |
| DinaBelova | there is example of nova conf file | 09:15 |
| DinaBelova | like always | 09:15 |
| DinaBelova | install climatenova | 09:15 |
| DinaBelova | use it | 09:15 |
| DinaBelova | :) | 09:15 |
| bauzas | I guess that's a stevedore plugin, so in setup.cfg no ? | 09:15 |
| DinaBelova | that's nova extension - it has its own format of using | 09:15 |
| bauzas | oh damn | 09:15 |
| DinaBelova | it's set in nova.cong file | 09:15 |
| DinaBelova | conf, sorry | 09:15 |
| bauzas | didn't see the example conf | 09:15 |
| bauzas | my bad :) | 09:16 |
| DinaBelova | it's used like your filter really | 09:16 |
| DinaBelova | :) | 09:16 |
| bauzas | ok, I know what to do today | 09:16 |
| DinaBelova | :D | 09:16 |
| *** SergeyLukjanov is now known as SergeyLukjanov_a | 09:31 | |
| *** SergeyLukjanov_a is now known as SergeyLukjanov_ | 09:32 | |
| *** DinaBelova is now known as DinaBelova_ | 09:42 | |
| *** chandankumar_ has quit IRC | 09:56 | |
| *** DinaBelova_ is now known as DinaBelova | 10:10 | |
| Nikolay_St | DinaBelova: I've tested our plugin on latest master | 10:20 |
| Nikolay_St | DinaBelova: look like it works | 10:20 |
| DinaBelova | Nikolay_St, great :) | 10:21 |
| DinaBelova | I feel great :) | 10:22 |
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Add notes about Keystone v3 endpoint to README.rst https://review.openstack.org/68609 | 10:27 |
| Nikolay_St | DinaBelova: Now I'll try it with snapshot and suspend | 10:28 |
| DinaBelova | ok | 10:28 |
| Nikolay_St | DinaBelova: one moment I want to discuss about Climate 0.2 | 10:32 |
| Nikolay_St | DinaBelova: I suppose that we will support Neutron resource reservation belore volumes | 10:33 |
| Nikolay_St | DinaBelova: Am I wrong? | 10:33 |
| DinaBelova | that's your research, Nick :) | 10:34 |
| DinaBelova | I dunno for sure | 10:34 |
| DinaBelova | I will be pleased if you'll add Neutron's resources to that etherpad | 10:34 |
| bauzas | Nikolay_St: DinaBelova: As said in the etherpad, I want to propose we abandon the idea of subteams | 10:34 |
| bauzas | DinaBelova: if you run out of resources, we can help you | 10:34 |
| DinaBelova | sure, I only mean that Nick has already taken a look on that | 10:35 |
| bauzas | DinaBelova: Nikolay_St: and in parellel, if you want to contribute to some PR aspects, I'm fine with it | 10:35 |
| DinaBelova | in august - speaking about POC | 10:35 |
| DinaBelova | bauzas, sure | 10:35 |
| bauzas | DinaBelova: yep, I know :) | 10:35 |
| bauzas | DinaBelova: that's only matter of prioritization about blueprints and people | 10:35 |
| DinaBelova | bauzas, I believe that too | 10:36 |
| DinaBelova | for example I LOVE ambitious core HA functionality :D | 10:36 |
| bauzas | :D | 10:36 |
| DinaBelova | and I believe you guys will allow me to implement it :D | 10:37 |
| bauzas | and I LOVE having some energy efficiency in Climate :D | 10:37 |
| DinaBelova | :D | 10:37 |
| DinaBelova | ;) | 10:37 |
| DinaBelova | sure | 10:37 |
| bauzas | that's because it's called Climate btw. :D | 10:37 |
| DinaBelova | still can't get point about naming, really | 10:37 |
| DinaBelova | climate - energy efficiency are not really connected in my mind | 10:38 |
| bauzas | because originally, the idea was about energy efficiency | 10:38 |
| bauzas | and if you're efficient with energy, you save the world | 10:38 |
| DinaBelova | :D | 10:38 |
| DinaBelova | wow | 10:38 |
| DinaBelova | really :D | 10:39 |
| DinaBelova | nice | 10:39 |
| bauzas | global warming and its consequences, you know | 10:39 |
| DinaBelova | yep | 10:39 |
| bauzas | *climate* changes | 10:39 |
| bauzas | etc. | 10:39 |
| DinaBelova | am i right energy efficiency is somehow idealized/implemented by Ironic+Ceilometer? | 10:40 |
| DinaBelova | i just remember some talks/conversations | 10:40 |
| bauzas | well, energy efficiency has various concerns | 10:44 |
| bauzas | and some can be handled by either Ironic or Ceilometer | 10:44 |
| bauzas | Ironic for IPMI/PDU usage | 10:44 |
| bauzas | in order to get metrics about Watt consumption | 10:44 |
| bauzas | because Ironic already has interfaces with IPMI interfaces | 10:45 |
| bauzas | and Ceilometer as well | 10:45 |
| bauzas | for PDU usage, see the kwapi project from Francosi | 10:45 |
| DinaBelova | ok | 10:50 |
| DinaBelova | thanks | 10:50 |
| *** DinaBelova is now known as DinaBelova_ | 10:54 | |
| bauzas | DinaBelova_: Nikolay_St: can't unshelve the instance | 11:15 |
| bauzas | oh | 11:16 |
| bauzas | dammit | 11:16 |
| bauzas | forget to amend my climate.cofn | 11:16 |
| bauzas | btw., the climate.conf.example should be amended accordingly | 11:18 |
| bauzas | Nikolay_St: DinaBelova_: I'm finishing to test but I will leave a -1 | 11:18 |
| bauzas | we need to update the conf file and remove the dummy vm plugin | 11:18 |
| bauzas | because it does have the same name, and could lead to confusioon | 11:18 |
| Nikolay_St | bauzas: I'll fix it in few minutes | 11:35 |
| bauzas | hold on | 11:35 |
| bauzas | I'm finishing to leave other comments | 11:35 |
| bauzas | there is a merge issue | 11:35 |
| openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Implement basic plugin for VM management https://review.openstack.org/49142 | 11:38 |
| bauzas | Nikolay_St: ... | 11:40 |
| bauzas | I'm finishing to test | 11:41 |
| *** Nikolay_St has quit IRC | 11:42 | |
| *** Nikolay_St has joined #openstack-climate | 11:43 | |
| *** Nikolay_St has quit IRC | 11:48 | |
| bauzas | DinaBelova_: I'm about to deliver a patch for showing errors to the client | 11:53 |
| bauzas | atm, we only provide an ugly and raw error | 11:53 |
| bauzas | either we include it in 0.1 or we wait for later | 11:53 |
| bauzas | I should expect to deliver it by end of today | 11:54 |
| *** Nikolay_St has joined #openstack-climate | 11:55 | |
| Nikolay_St | bauzas: can't get this http://paste.openstack.org/show/61810/ | 12:07 |
| Nikolay_St | how did you get this error? | 12:07 |
| Nikolay_St | bauzas: about nova client and auth token | 12:12 |
| Nikolay_St | bauzas: how old is your nova client? | 12:12 |
| Nikolay_St | bauzas: https://github.com/openstack/python-novaclient/blob/master/novaclient/v1_1/client.py | 12:14 |
| Nikolay_St | bauzas: looks like novaclient can get auth_token | 12:14 |
| *** Nikolay_St has quit IRC | 12:24 | |
| *** chandankumar_ has joined #openstack-climate | 12:31 | |
| *** chandankumar_ has quit IRC | 12:34 | |
| bauzas | I'm having python-novaclient-2.15.0 on my venv | 12:50 |
| bauzas | which is the latest one | 12:50 |
| *** DinaBelova_ is now known as DinaBelova | 13:00 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 13:01 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 13:02 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 13:03 | |
| DinaBelova | Ш куегктув руку Ж) | 13:04 |
| DinaBelova | wow | 13:04 |
| DinaBelova | I returned here | 13:04 |
| DinaBelova | needed to go afk | 13:04 |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 13:11 | |
| DinaBelova | bauzas, will you be ok if i'll fix Nick's CR? and you'll take one more look on it? | 13:22 |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 13:24 | |
| bauzas | DinaBelova: ok | 13:41 |
| bauzas | no pb, the main thing is that we can give +2 on it | 13:41 |
| *** openstackgerrit has quit IRC | 13:42 | |
| *** joel_c has quit IRC | 13:43 | |
| *** joel_c has joined #openstack-climate | 13:43 | |
| DinaBelova | my irc client does not show I have new PS | 13:48 |
| DinaBelova | https://review.openstack.org/#/c/49142/ | 13:48 |
| DinaBelova | that's it | 13:48 |
| *** DinaBelova is now known as DinaBelova_ | 13:50 | |
| *** DinaBelova_ is now known as DinaBelova | 13:52 | |
| SergeyLukjanov | https://review.openstack.org/68040 << merged | 13:55 |
| SergeyLukjanov | (Allow use of oslo.messaging 1.3.0a4 from pypi) | 13:55 |
| SergeyLukjanov | DinaBelova | 13:55 |
| DinaBelova | i see | 13:56 |
| DinaBelova | bauzas, may you check, btw, https://review.openstack.org/#/c/49142/ ? | 13:56 |
| bauzas | sorry, was on a different subject | 14:02 |
| bauzas | DinaBelova: what do you say ? | 14:02 |
| DinaBelova | i've loaded new patch set for vm plugin | 14:03 |
| DinaBelova | it would e nice if you'll take a look on it | 14:03 |
| DinaBelova | i mean its working | 14:03 |
| DinaBelova | in case of not using extension | 14:03 |
| DinaBelova | wow :) | 14:04 |
| DinaBelova | Jenkins failed | 14:04 |
| bauzas | Jenkins -1 ;:( | 14:04 |
| DinaBelova | interesting, btw | 14:05 |
| DinaBelova | it could not load all needed plugins | 14:05 |
| DinaBelova | that's quite strange | 14:05 |
| DinaBelova | i've checked namings in setup.cfg | 14:05 |
| DinaBelova | it seems to be nice | 14:05 |
| DinaBelova | and path too | 14:05 |
| bauzas | mmm | 14:06 |
| bauzas | I had to run a new setup.py develop for doing that work | 14:06 |
| bauzas | but here, Jenkins is fully reinstalling | 14:06 |
| bauzas | so, there should not be an issue | 14:07 |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 14:07 | |
| DinaBelova | sometimes there are some problems with testing nodes, but it's rare for them to have problems twice | 14:07 |
| bauzas | can you reproduce the bug ? | 14:09 |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 14:09 | |
| DinaBelova | not yet | 14:09 |
| DinaBelova | reinstalling venv | 14:11 |
| DinaBelova | let's see if it will happen | 14:11 |
| bauzas | yup :( | 14:12 |
| DinaBelova | yes, i see that | 14:13 |
| DinaBelova | quite interesting | 14:13 |
| bauzas | sorry, I know we're running out of time, but I still need to make sure everything is working fine :( | 14:13 |
| bauzas | next release should be less stressy, as we'll have the core features :) | 14:13 |
| DinaBelova | i know issue | 14:14 |
| DinaBelova | moment | 14:14 |
| DinaBelova | i did not see that | 14:14 |
| DinaBelova | nick did smth really bad ЖВ | 14:14 |
| DinaBelova | :D | 14:14 |
| DinaBelova | checking it now | 14:15 |
| DinaBelova | gotcha | 14:16 |
| DinaBelova | uploading new ps | 14:16 |
| DinaBelova | smth whent wrong with out gerrit bot :( | 14:17 |
| DinaBelova | it does not publish info about new ps's :( | 14:17 |
| DinaBelova | https://review.openstack.org/#/c/49142 | 14:17 |
| DinaBelova | bauzas, that is new one | 14:18 |
| DinaBelova | you may check it | 14:18 |
| bauzas | ok, just finishing to fix the bug about the exceptions on the client | 14:19 |
| DinaBelova | great | 14:19 |
| bauzas | that was a very little fix | 14:19 |
| DinaBelova | :) | 14:20 |
| DinaBelova | i love such :) | 14:20 |
| bauzas | you were trying to extract resp.body while it doesn't exist | 14:20 |
| bauzas | I'm on it | 14:20 |
| bauzas | creating the bugfile | 14:20 |
| bauzas | I'll let you categorize it | 14:20 |
| DinaBelova | :) | 14:20 |
| bauzas | https://bugs.launchpad.net/climate/+bug/1272352 | 14:22 |
| DinaBelova | done | 14:23 |
| DinaBelova | bauzas, Jenkins approved that last patchset :D | 14:37 |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 14:41 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 14:41 | |
| bauzas | yup, testing it | 14:42 |
| bauzas | ok, so you enabled again the dummy plugin | 14:46 |
| bauzas | DinaBelova: I think it would be worth removing it | 14:46 |
| bauzas | as it provides confusion | 14:46 |
| bauzas | but that can be done on a later commit | 14:46 |
| bauzas | DinaBelova: cross your fingers | 14:49 |
| DinaBelova | it's good for testing | 14:49 |
| DinaBelova | :D | 14:49 |
| bauzas | dammit | 14:50 |
| DinaBelova | :( | 14:50 |
| bauzas | you didn't fixed the issue with ClimateException | 14:50 |
| bauzas | File "/home/bauzas/climate/climate/manager/service.py", line 352, in _basic_action | 14:50 |
| bauzas | except exceptions.ClimateException: | 14:50 |
| bauzas | should be common_ex.ClimateException | 14:50 |
| DinaBelova | ммм | 14:50 |
| bauzas | but still | 14:50 |
| DinaBelova | sorry, missed it | 14:51 |
| bauzas | no pb | 14:51 |
| DinaBelova | will fix now | 14:51 |
| bauzas | we still have an issue even with the fix | 14:51 |
| bauzas | oh | 14:51 |
| bauzas | still the climate.conf | 14:51 |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 14:51 | |
| DinaBelova | sorry? | 14:51 |
| DinaBelova | I fixed cong example? | 14:51 |
| bauzas | yep | 14:51 |
| DinaBelova | conf* | 14:51 |
| bauzas | but mine not :p | 14:51 |
| DinaBelova | :D | 14:51 |
| bauzas | anyway, you still need to fix the ClimateException | 14:52 |
| DinaBelova | i suppose that might be another cr :? | 14:52 |
| DinaBelova | will you be ok with having it in this one? | 14:52 |
| bauzas | 2014-01-24 15:52:41.866 30242 CRITICAL climate [-] Plugin Configuration error : You have provided several plugins for one resource type in configuration file. Please set one plugin per resource type. | 14:52 |
| bauzas | dammit | 14:52 |
| DinaBelova | mmm | 14:53 |
| DinaBelova | set one please | 14:53 |
| bauzas | this f*** dummy_vm plugin is making me crazy | 14:53 |
| DinaBelova | :D:D:D | 14:53 |
| DinaBelova | i'll add this change to current VM change | 14:55 |
| DinaBelova | ok with that? | 14:55 |
| DinaBelova | bauzas? | 14:55 |
| *** ppetit has joined #openstack-climate | 14:55 | |
| bauzas | DinaBelova: sorry, can't get it | 14:56 |
| bauzas | what do you want ? | 14:56 |
| DinaBelova | sorry, i'm about exceptions exceptions->common_ex | 14:56 |
| bauzas | yay | 14:56 |
| bauzas | got it | 14:56 |
| DinaBelova | File "/home/bauzas/climate/climate/manager/service.py", line 352, in _basic_action | 14:56 |
| bauzas | but what do you want about it ? | 14:56 |
| DinaBelova | this one problem | 14:56 |
| bauzas | yep please, another patchset | 14:57 |
| bauzas | but hold on | 14:57 |
| DinaBelova | another change? or patch set? | 14:57 |
| bauzas | http://paste.openstack.org/show/61827/ | 14:58 |
| bauzas | wtf ? | 14:58 |
| bauzas | I would prefer another patchset for the common_ex thing | 14:59 |
| DinaBelova | em | 14:59 |
| bauzas | yup | 14:59 |
| bauzas | pretty crazy | 14:59 |
| bauzas | about the unshelve thing | 14:59 |
| DinaBelova | i'm quite shocked | 14:59 |
| bauzas | I'll pdf | 14:59 |
| bauzas | I'll pdb | 14:59 |
| bauzas | give me a min | 14:59 |
| DinaBelova | ok | 14:59 |
| DinaBelova | you have one till our meeting :) | 15:00 |
| DinaBelova | :D | 15:00 |
| DinaBelova | go to meeting :) | 15:01 |
| bauzas | oups | 15:01 |
| bauzas | dammit, introspecting the nova objecty | 15:03 |
| *** Nikolay_St has joined #openstack-climate | 15:07 | |
| bauzas | (Pdb) nova_client.servers.shelve | 15:10 |
| bauzas | *** AttributeError: 'ServerManager' object has no attribute 'shelve' | 15:10 |
| bauzas | (Pdb) nova_client.servers.suspend | 15:10 |
| bauzas | <bound method ServerManager.suspend of <novaclient.v1_1.servers.ServerManager object at 0x2627450>> | 15:10 |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 15:19 | |
| DinaBelova | Nikolay_St, please help bauzas to fix all this moment | 15:43 |
| bauzas | well, there is not much to do now | 15:43 |
| bauzas | :( | 15:43 |
| bauzas | I shelved the instance because I was not on my venv but devstack | 15:43 |
| bauzas | on my venv, I don't have the action | 15:43 |
| bauzas | :) | 15:43 |
| bauzas | the main frustrating point is that we just discover the issue now | 15:44 |
| bauzas | so we're running out of time and depending on a new release | 15:44 |
| bauzas | which can last 1 week or 2 | 15:44 |
| bauzas | because we also need to update the oslo requirements... | 15:44 |
| bauzas | so | 15:45 |
| bauzas | here is the problem : | 15:45 |
| bauzas | either we wait for 2.16 which can arrive in more than one week | 15:45 |
| bauzas | or, we ask for master in the requirements | 15:45 |
| DinaBelova | bauzas, last is not a working variant | 15:46 |
| DinaBelova | that will ruin our gate | 15:46 |
| bauzas | correct | 15:46 |
| DinaBelova | Nikolay_St, write letter right now please | 15:46 |
| DinaBelova | include there novaclient ptl | 15:46 |
| bauzas | that said, are you sure the requirements check would fail ? | 15:46 |
| DinaBelova | bauzas, yep, pretty sure | 15:47 |
| DinaBelova | SergeyLukjanov, am I right? | 15:47 |
| SergeyLukjanov | I'm here | 15:47 |
| SergeyLukjanov | which question? | 15:47 |
| DinaBelova | we've got problem | 15:47 |
| DinaBelova | we need new novaclient release to be done :) | 15:48 |
| bauzas | is also the Nova extension patch potentially having the same issue ? | 15:48 |
| DinaBelova | to have working Climate code | 15:48 |
| SergeyLukjanov | oh | 15:48 |
| SergeyLukjanov | that's a problem | 15:48 |
| bauzas | that's a big one indeed | 15:48 |
| DinaBelova | bauzas, not really... it uses only nova extension mechanism | 15:48 |
| DinaBelova | it should work ok | 15:48 |
| bauzas | ok, I'll test it anyway | 15:49 |
| DinaBelova | SergeyLukjanov, last novaclient release is Sep 2013 | 15:49 |
| bauzas | SergeyLukjanov: how can we make sure the reqs are followed if by using devstack ? | 15:49 |
| bauzas | SergeyLukjanov: that's a terrible error-prone thing | 15:49 |
| SergeyLukjanov | you could try to ping Russel directly | 15:49 |
| DinaBelova | SergeyLukjanov we have this problem because of devstack testing lab :( | 15:50 |
| DinaBelova | reqs are missing there | 15:50 |
| DinaBelova | because of masters | 15:50 |
| bauzas | reqs are not missing | 15:50 |
| bauzas | they are followed | 15:50 |
| bauzas | master is > 2.15.0 :) | 15:50 |
| DinaBelova | bauzas, I used wrong word, sorry | 15:50 |
| DinaBelova | :) | 15:50 |
| bauzas | luckly I am to be using a venv | 15:50 |
| DinaBelova | Nikolay_St, you here? | 15:51 |
| bauzas | I think the lib/climate stuff would be installing climate on a venv | 15:51 |
| SergeyLukjanov | which kind of problem you have with client? | 15:51 |
| SergeyLukjanov | nova client* | 15:51 |
| SergeyLukjanov | lack of features? | 15:51 |
| bauzas | that's potentially the only option I see | 15:51 |
| bauzas | SergeyLukjanov: yup | 15:51 |
| DinaBelova | SergeyLukjanov surrent version cannot shelve/unshelve | 15:51 |
| bauzas | shelve/unshelve | 15:51 |
| SergeyLukjanov | the easiest option is too backport it to your code | 15:51 |
| SergeyLukjanov | and use | 15:51 |
| DinaBelova | I see Nick has written letter | 15:51 |
| SergeyLukjanov | I think it's something like 20-50 lines of code | 15:52 |
| SergeyLukjanov | I've do the same in savanna | 15:52 |
| bauzas | good idea | 15:52 |
| DinaBelova | Nikolay_St, forward this letter directly to Russel please | 15:52 |
| Nikolay_St | SergeyLukjanov: really not bad | 15:52 |
| bauzas | DinaBelova: I think SergeyLukjanov's idea is brilliant | 15:52 |
| Nikolay_St | DinaBelova: I'm working on it | 15:52 |
| SergeyLukjanov | we find the lack of features in nova client, I've implemented it in savanna and then contributed it to the nova client | 15:52 |
| bauzas | DinaBelova: because we already override the nova client | 15:52 |
| DinaBelova | SergeyLukjanov I'm proud of my nice fiance :) | 15:53 |
| bauzas | we only need to backport the shelve/unshelve stuff | 15:53 |
| SergeyLukjanov | let me find the code in savanna | 15:53 |
| DinaBelova | agreed | 15:53 |
| DinaBelova | +2/A | 15:53 |
| bauzas | we already have the __getattr_ stuff | 15:53 |
| SergeyLukjanov | https://github.com/openstack/savanna/blob/master/savanna/utils/openstack/nova.py#L39 | 15:54 |
| SergeyLukjanov | here is how we're doing it | 15:54 |
| SergeyLukjanov | https://github.com/openstack/savanna/blob/master/savanna/utils/openstack/keypairs.py | 15:54 |
| DinaBelova | SergeyLukjanov, we even do not need this - we have to write only two methods | 15:54 |
| bauzas | https://github.com/stackforge/climate/blob/master/climate/utils/openstack/nova.py#L95 | 15:54 |
| bauzas | we only need to create a shelve method | 15:55 |
| SergeyLukjanov | the best approach is to inherit from the manager and add methods you need | 15:55 |
| bauzas | sorry, an unshelve method | 15:55 |
| SergeyLukjanov | personally I dislike __getattr__ approach due to the tricky behavior of this devil feature | 15:56 |
| bauzas | about inheritance ? | 15:56 |
| bauzas | that's something manageable IMHO | 15:56 |
| bauzas | anyway | 15:57 |
| DinaBelova | ok, we'll do it somehow - way does not really matter how now :D | 15:57 |
| DinaBelova | I'll add this | 15:57 |
| SergeyLukjanov | anyway, you have several workarounds for the problem you've faced | 15:57 |
| bauzas | cool | 15:57 |
| Nikolay_St | DinaBelova: I'll try to fix it on weekend, or early Monday | 16:00 |
| Nikolay_St | DinaBelova: I think it'll be okay | 16:00 |
| DinaBelova | Nikolay_St, I'll do it now | 16:00 |
| DinaBelova | it will be quicker | 16:00 |
| Nikolay_St | DinaBelova: as you wish | 16:01 |
| bauzas | and test it :) | 16:01 |
| bauzas | :D | 16:01 |
| *** Nikolay_St has quit IRC | 16:11 | |
| *** openstackgerrit has joined #openstack-climate | 16:12 | |
| openstackgerrit | Dina Belova proposed a change to stackforge/climate: Implement basic plugin for VM management https://review.openstack.org/49142 | 16:26 |
| DinaBelova | bauzas, I've proposed change | 16:27 |
| DinaBelova | it need to be tested | 16:27 |
| DinaBelova | if you still have your venv | 16:28 |
| DinaBelova | you may try :D | 16:28 |
| bauzas | your patch sounds good to me | 16:29 |
| bauzas | I'm running out of time, but I'm testing it | 16:29 |
| bauzas | I'll have to go in 10 mins | 16:29 |
| bauzas | :( | 16:29 |
| bauzas | daughter etc. | 16:29 |
| DinaBelova | bauzas, sure :) | 16:30 |
| bauzas | maybe one remark about the import | 16:30 |
| DinaBelova | v11? | 16:30 |
| bauzas | from novaclient.v1_1 import servers | 16:30 |
| bauzas | yup, | 16:30 |
| bauzas | I'm good with it | 16:30 |
| bauzas | provided it's only a quick and dirty backport that we don't keep | 16:31 |
| DinaBelova | bauzas, I also though about it | 16:31 |
| DinaBelova | as soon 2.16 will be release | 16:31 |
| DinaBelova | I'll just drop all these lines | 16:31 |
| bauzas | exactly | 16:31 |
| bauzas | that was what I thought | 16:31 |
| bauzas | just the create_image stuff needs to be managed too | 16:32 |
| DinaBelova | sure | 16:32 |
| bauzas | because that one won't be part of 2.16.0 | 16:32 |
| DinaBelova | I'll leave ServerManager and will replace resource type by usual one | 16:32 |
| DinaBelova | and it'll look nice I think | 16:32 |
| DinaBelova | only moment I'm not sure now | 16:33 |
| DinaBelova | without testing | 16:33 |
| DinaBelova | is | 16:33 |
| DinaBelova | self.manager.unshelve(self) | 16:33 |
| DinaBelova | I'm not sure ClimateServer will understand what manager to use | 16:33 |
| DinaBelova | still it has to | 16:33 |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 16:37 | |
| bauzas | dammit, the common_ex stuff | 16:37 |
| bauzas | I have to go | 16:38 |
| DinaBelova | ok | 16:38 |
| DinaBelova | i'll provide new change request for it | 16:38 |
| DinaBelova | not new PS, but CR | 16:39 |
| DinaBelova | and will put VM staff on it | 16:39 |
| DinaBelova | oh | 16:39 |
| DinaBelova | no | 16:39 |
| DinaBelova | new PS | 16:39 |
| DinaBelova | it will be better | 16:39 |
| DinaBelova | dunno | 16:39 |
| DinaBelova | will see))) | 16:39 |
| *** DinaBelova is now known as DinaBelova_ | 16:41 | |
| *** YorikSar has quit IRC | 16:42 | |
| *** bauzas has quit IRC | 16:44 | |
| *** ppetit has quit IRC | 17:00 | |
| *** DinaBelova_ is now known as DinaBelova | 17:16 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 17:17 | |
| *** DinaBelova is now known as DinaBelova_ | 17:41 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_a | 17:42 | |
| *** DinaBelova_ is now known as DinaBelova | 17:42 | |
| *** SergeyLukjanov_a is now known as SergeyLukjanov_ | 17:43 | |
| *** DinaBelova is now known as DinaBelova_ | 17:57 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 18:12 | |
| *** YorikSar has joined #openstack-climate | 18:12 | |
| *** DinaBelova_ is now known as DinaBelova | 18:26 | |
| *** DinaBelova is now known as DinaBelova_ | 20:42 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 21:39 | |
| openstackgerrit | Jenkins proposed a change to stackforge/climate: Updated from global requirements https://review.openstack.org/69048 | 22:42 |
| openstackgerrit | Jenkins proposed a change to stackforge/python-climateclient: Updated from global requirements https://review.openstack.org/69050 | 22:42 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!