| *** haomaiwang has quit IRC | 00:01 | |
| *** thorst has quit IRC | 00:01 | |
| *** ducttape_ has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-nova | 00:01 | |
| *** ZZelle_ has quit IRC | 00:02 | |
| *** browne has quit IRC | 00:07 | |
| *** zz_dimtruck is now known as dimtruck | 00:08 | |
| *** tbachman has quit IRC | 00:17 | |
| *** tbachman has joined #openstack-nova | 00:17 | |
| *** Pistachio has quit IRC | 00:18 | |
| *** abitha has quit IRC | 00:18 | |
| *** abitha has joined #openstack-nova | 00:19 | |
| *** shaohe_feng_ has quit IRC | 00:20 | |
| *** mylu has quit IRC | 00:21 | |
| *** Sukhdev has quit IRC | 00:22 | |
| *** cdent has joined #openstack-nova | 00:22 | |
| cdent | dansmith: you still around? was there a decision about not needing the uniq constraint (cuz it's gone now)? | 00:23 |
|---|---|---|
| dansmith | cdent: um, what? | 00:23 |
| dansmith | I thought we were doing it | 00:23 |
| *** abitha has quit IRC | 00:24 | |
| cdent | dansmith: you pushed a patchset 24 that took it back out | 00:24 |
| cdent | https://review.openstack.org/#/c/281837/24 | 00:24 |
| dansmith | oh fsck | 00:24 |
| dansmith | I did not mean to do that | 00:24 |
| dansmith | I must've messed up the giant N-way merge | 00:24 |
| cdent | It seems to have made gerrit all kinds of confused | 00:25 |
| *** davideagle has joined #openstack-nova | 00:25 | |
| dansmith | okay, let's do this | 00:26 |
| dansmith | cdent: if you can refresh it with the right stuff, we should try to get it merged, and then the two db patches on top | 00:26 |
| dansmith | and I won't re-push until that stuff lands | 00:26 |
| cdent | dansmith: I was just getting in to bed (and doing a stupid last email check) and mostly asleep so probably not save to operate heavy machinery. If you could get it that would be awesome, otherwise I'll do it in the morning. | 00:27 |
| dansmith | cdent: I just don't want to mess it up again.. I can grab the hash of PS23 and resubmit .. is that all -- just that one? | 00:27 |
| cdent | yeah, that should be fine | 00:28 |
| dansmith | I guess the next one is jays | 00:28 |
| dansmith | okay | 00:28 |
| dansmith | hmm, I don't have that hash | 00:29 |
| dansmith | wtf | 00:29 |
| cdent | I'll look through that stack in the morning and see if it looks kosher, sorry for not being available to help out now, my head's dropping | 00:29 |
| cdent | 'night | 00:30 |
| *** cdent has quit IRC | 00:30 | |
| dansmith | ah, here we go | 00:30 |
| *** suresh12 has quit IRC | 00:31 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Adjust resource-providers models for resource-pools https://review.openstack.org/281837 | 00:31 |
| *** ccarmack has quit IRC | 00:32 | |
| *** mspreitz has joined #openstack-nova | 00:32 | |
| *** jschlueter has quit IRC | 00:32 | |
| oomichi | alex_xu: gmann: hi, can you provide the gerrit dashboard link for searching api-related patches ? | 00:33 |
| *** ducttape_ has joined #openstack-nova | 00:33 | |
| oomichi | that was given by sdague but I forgot the link due to a new PC. | 00:33 |
| *** boris-42 has quit IRC | 00:34 | |
| gmann | oomichi: https://review.openstack.org/#/dashboard/?foreach=(project:openstack/nova OR project:openstack/python-novaclient OR project:openstack/nova-specs OR project:openstack/api-site) ((file:^.*nova/api.* OR file:^.*api_samples.* OR file:^.*compute-api.* OR file:^.*api-guide.*) OR message:apiimpact) status:open NOT owner:self NOT label:Workflow<=-1 label:Verified>=1,jenkins NOT label:Code-Review>=-2,self&title=Nova API&Proposed AP | 00:35 |
| gmann | I changing Specs=project:openstack/nova-specs NOT label:Code-Review>=-2,self&Needs Feedback (Changes older than 5 days that have not been reviewed by anyone)=NOT label:Code-Review<=2 age:5d&Your are a reviewer, but haven't voted in the current revision=reviewer:self&Needs final +2=label:Code-Review>=2&Passed Jenkins, No Negative Feedback=NOT label:Code-Review>=2 NOT label:Code-Review<=-1 limit:50&Down voted changes=label:Code-Revie | 00:35 |
| gmann | w<=-1 | 00:35 |
| gmann | oomichi: oops | 00:35 |
| gmann | oomichi: http://paste.openstack.org/show/488663/ | 00:35 |
| oomichi | gmann: thanks! | 00:35 |
| *** markvoelker has quit IRC | 00:38 | |
| *** Sukhdev has joined #openstack-nova | 00:38 | |
| *** mylu has joined #openstack-nova | 00:39 | |
| *** davideagle has quit IRC | 00:39 | |
| *** tangchen has quit IRC | 00:39 | |
| *** hoangcx has joined #openstack-nova | 00:42 | |
| *** claudiub has quit IRC | 00:42 | |
| *** cdelatte has quit IRC | 00:43 | |
| *** ducttape_ has quit IRC | 00:45 | |
| alex_xu | oomichi: there also have one in the line 56 https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking | 00:50 |
| *** MVenesio has joined #openstack-nova | 00:51 | |
| *** baoli has joined #openstack-nova | 00:52 | |
| *** Sukhdev has quit IRC | 00:52 | |
| oomichi | alex_xu: thanks:-) | 00:53 |
| alex_xu | oomichi: np :) | 00:53 |
| *** MVenesio has quit IRC | 00:57 | |
| *** jschlueter has joined #openstack-nova | 00:58 | |
| *** baoli has quit IRC | 00:58 | |
| *** thorst_ has quit IRC | 01:00 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** amotoki has joined #openstack-nova | 01:01 | |
| *** thorst has joined #openstack-nova | 01:01 | |
| *** haomaiwang has joined #openstack-nova | 01:01 | |
| *** mylu has quit IRC | 01:02 | |
| *** apoorvad has quit IRC | 01:06 | |
| *** EinstCrazy has joined #openstack-nova | 01:06 | |
| *** penick has quit IRC | 01:08 | |
| *** mylu has joined #openstack-nova | 01:08 | |
| *** thorst has quit IRC | 01:09 | |
| *** suresh12 has joined #openstack-nova | 01:10 | |
| *** suresh12 has quit IRC | 01:10 | |
| *** suresh12_ has joined #openstack-nova | 01:10 | |
| *** ducttape_ has joined #openstack-nova | 01:11 | |
| *** suresh12_ has quit IRC | 01:16 | |
| *** apoorvad has joined #openstack-nova | 01:17 | |
| *** suresh12 has joined #openstack-nova | 01:17 | |
| *** angdraug has quit IRC | 01:20 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Decode injected content when checking length limit https://review.openstack.org/285649 | 01:21 |
| *** Daisy has joined #openstack-nova | 01:23 | |
| *** haukebruno has joined #openstack-nova | 01:25 | |
| *** haomaiwang has quit IRC | 01:26 | |
| *** Daisy has quit IRC | 01:27 | |
| *** vilobhmm11 has quit IRC | 01:33 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Decode injected content when checking length limit https://review.openstack.org/285649 | 01:35 |
| *** sheel has joined #openstack-nova | 01:36 | |
| *** markvoelker has joined #openstack-nova | 01:38 | |
| *** apoorvad has quit IRC | 01:44 | |
| *** thorst has joined #openstack-nova | 01:45 | |
| *** jamielennox is now known as jamielennox|away | 01:45 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Improve assert statements in iptables and virt tests https://review.openstack.org/282992 | 01:45 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Pass BDM in swap_volume method of LibvirtDriver https://review.openstack.org/257135 | 01:46 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add some validations for BDM https://review.openstack.org/258788 | 01:46 |
| *** Jeffrey4l has joined #openstack-nova | 01:47 | |
| *** mylu has quit IRC | 01:49 | |
| *** andreykurilin__ has quit IRC | 01:53 | |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 01:53 |
| *** thorst has quit IRC | 01:53 | |
| *** mylu has joined #openstack-nova | 01:54 | |
| *** mtanino has quit IRC | 01:55 | |
| *** mylu has quit IRC | 01:56 | |
| *** ducttape_ has quit IRC | 01:57 | |
| *** doug-fish has quit IRC | 01:58 | |
| *** irenab_ has joined #openstack-nova | 02:03 | |
| *** irenab has quit IRC | 02:04 | |
| *** irenab_ is now known as irenab | 02:04 | |
| *** dims has quit IRC | 02:04 | |
| *** mylu has joined #openstack-nova | 02:05 | |
| *** haomaiwa_ has joined #openstack-nova | 02:07 | |
| *** markvoelker has quit IRC | 02:08 | |
| *** markvoelker has joined #openstack-nova | 02:08 | |
| *** Sukhdev has joined #openstack-nova | 02:08 | |
| *** dims has joined #openstack-nova | 02:08 | |
| *** baoli has joined #openstack-nova | 02:11 | |
| *** dims has quit IRC | 02:13 | |
| *** Sukhdev has quit IRC | 02:14 | |
| *** mylu has quit IRC | 02:15 | |
| *** Marga__ has joined #openstack-nova | 02:16 | |
| *** dims has joined #openstack-nova | 02:17 | |
| *** baoli has quit IRC | 02:18 | |
| *** mylu has joined #openstack-nova | 02:18 | |
| *** baoli has joined #openstack-nova | 02:18 | |
| *** Marga_ has quit IRC | 02:19 | |
| *** hemna_ has joined #openstack-nova | 02:19 | |
| *** mylu_ has joined #openstack-nova | 02:19 | |
| *** mylu has quit IRC | 02:20 | |
| *** ducttape_ has joined #openstack-nova | 02:20 | |
| *** mylu_ has quit IRC | 02:22 | |
| *** baoli has quit IRC | 02:22 | |
| *** baoli has joined #openstack-nova | 02:23 | |
| *** mylu has joined #openstack-nova | 02:25 | |
| *** mylu has quit IRC | 02:27 | |
| *** fawadkhaliq has quit IRC | 02:27 | |
| *** suresh12 has quit IRC | 02:28 | |
| *** suresh12 has joined #openstack-nova | 02:30 | |
| *** nic has quit IRC | 02:32 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Decode injected content when checking length limit https://review.openstack.org/285649 | 02:34 |
| *** andymaier has quit IRC | 02:34 | |
| *** armax has quit IRC | 02:35 | |
| *** gcb has joined #openstack-nova | 02:36 | |
| *** houming has joined #openstack-nova | 02:38 | |
| *** mylu has joined #openstack-nova | 02:38 | |
| eliqiao | andreykurilin: hi, there? | 02:42 |
| *** ijw has quit IRC | 02:43 | |
| *** thorst has joined #openstack-nova | 02:51 | |
| *** MVenesio has joined #openstack-nova | 02:52 | |
| *** hemna_ has quit IRC | 02:57 | |
| *** MVenesio has quit IRC | 02:58 | |
| *** thorst has quit IRC | 02:58 | |
| *** baoli has quit IRC | 03:00 | |
| *** haomaiwa_ has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** dimtruck is now known as zz_dimtruck | 03:01 | |
| *** mylu has quit IRC | 03:04 | |
| *** zz_dimtruck is now known as dimtruck | 03:12 | |
| *** suresh12 has quit IRC | 03:16 | |
| *** dims has quit IRC | 03:19 | |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 03:21 |
| *** mylu has joined #openstack-nova | 03:22 | |
| alex_xu | shaohe_feng: ping | 03:23 |
| *** baoli has joined #openstack-nova | 03:25 | |
| *** Marga__ has quit IRC | 03:26 | |
| shaohe_feng | alex_xu: pong | 03:27 |
| alex_xu | shaohe_feng: I have comment on https://review.openstack.org/#/c/281335/15, i guess you miss it due to update patch, would you like take a look at? | 03:28 |
| *** vilobhmm11 has joined #openstack-nova | 03:28 | |
| shaohe_feng | alex_xu: Sorry. I'm busy on to improve the testcase for the patch. | 03:29 |
| alex_xu | shaohe_feng: no worries, just a reminder | 03:29 |
| shaohe_feng | alex_xu: yesterday, we find the novaclient testcase is not good enough. so andreykurilin and I want this patch to be an example for the following similar patches. This takes us a long time. | 03:30 |
| alex_xu | shaohe_feng: ah, that is cool | 03:30 |
| *** Marga_ has joined #openstack-nova | 03:30 | |
| shaohe_feng | alex_xu: thanks. and thanks for review. | 03:31 |
| alex_xu | shaohe_feng: np | 03:31 |
| *** Nakato has quit IRC | 03:31 | |
| *** ccard_ has quit IRC | 03:34 | |
| *** Marga_ has quit IRC | 03:35 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Decode injected content when checking length limit https://review.openstack.org/285649 | 03:40 |
| *** sasukeh has joined #openstack-nova | 03:42 | |
| *** Marga_ has joined #openstack-nova | 03:43 | |
| *** dave-mccowan has quit IRC | 03:45 | |
| *** Nakato has joined #openstack-nova | 03:45 | |
| *** Marga_ has quit IRC | 03:47 | |
| *** ccard_ has joined #openstack-nova | 03:47 | |
| *** baoli has quit IRC | 03:49 | |
| *** links has joined #openstack-nova | 03:49 | |
| *** sasukeh has quit IRC | 03:50 | |
| *** vilobhmm11 has quit IRC | 03:52 | |
| *** vilobhmm11 has joined #openstack-nova | 03:54 | |
| *** rhagarty_ has joined #openstack-nova | 03:55 | |
| *** thorst has joined #openstack-nova | 03:56 | |
| *** rhagarty has quit IRC | 03:58 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-nova | 04:01 | |
| *** rhagarty_ has quit IRC | 04:02 | |
| *** thorst has quit IRC | 04:03 | |
| *** chhavi has joined #openstack-nova | 04:07 | |
| *** armax has joined #openstack-nova | 04:15 | |
| *** yamahata has joined #openstack-nova | 04:16 | |
| *** Marga_ has joined #openstack-nova | 04:16 | |
| *** chhavi has quit IRC | 04:16 | |
| *** Marga_ has quit IRC | 04:16 | |
| *** chhavi has joined #openstack-nova | 04:16 | |
| *** Marga_ has joined #openstack-nova | 04:17 | |
| *** fawadkhaliq has joined #openstack-nova | 04:18 | |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 04:19 |
| *** haomaiwang has quit IRC | 04:21 | |
| *** dimtruck is now known as zz_dimtruck | 04:22 | |
| *** ducttape_ has quit IRC | 04:22 | |
| *** coreywright has quit IRC | 04:22 | |
| *** haomaiwang has joined #openstack-nova | 04:26 | |
| *** sheel has quit IRC | 04:27 | |
| *** coreywright has joined #openstack-nova | 04:36 | |
| *** haukebruno has quit IRC | 04:49 | |
| *** haukebruno has joined #openstack-nova | 04:51 | |
| *** MVenesio has joined #openstack-nova | 04:53 | |
| *** edtubill has joined #openstack-nova | 04:54 | |
| *** phuongnh has joined #openstack-nova | 04:56 | |
| *** MVenesio has quit IRC | 04:58 | |
| *** haukebruno has quit IRC | 05:00 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-nova | 05:01 | |
| *** thorst has joined #openstack-nova | 05:01 | |
| *** suresh12 has joined #openstack-nova | 05:02 | |
| *** RuiChen has quit IRC | 05:02 | |
| *** RuiChen has joined #openstack-nova | 05:03 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: introduce unit test for XenAPI plugins https://review.openstack.org/283962 | 05:03 |
| *** edtubill has quit IRC | 05:06 | |
| *** edtubill has joined #openstack-nova | 05:07 | |
| *** thorst has quit IRC | 05:08 | |
| *** sasukeh_ has joined #openstack-nova | 05:08 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: add unit test for plugin test_pluginlib_nova.py https://review.openstack.org/286383 | 05:09 |
| *** sasukeh_ has quit IRC | 05:18 | |
| *** mspreitz has quit IRC | 05:18 | |
| *** salv-orl_ has joined #openstack-nova | 05:19 | |
| *** nkrinner has joined #openstack-nova | 05:19 | |
| *** Marga_ has quit IRC | 05:20 | |
| *** salv-orlando has quit IRC | 05:22 | |
| *** ekuris_ has joined #openstack-nova | 05:27 | |
| *** edtubill has quit IRC | 05:28 | |
| *** mylu has quit IRC | 05:30 | |
| *** Marga_ has joined #openstack-nova | 05:30 | |
| eliqiao | hi andreykurilin shaohe_feng, thanks for you comments on https://review.openstack.org/#/c/260383/6/novaclient/tests/unit/v2/fakes.py, but I have a question: the method check_server_actions() is a classmethod, we can not get api_version before __init__ a class. | 05:33 |
| *** mylu has joined #openstack-nova | 05:34 | |
| eliqiao | pkoniszewski: hi, good morning, thanks for helping rebase patch while I am in sleep :) | 05:34 |
| openstackgerrit | Eli Qiao proposed openstack/python-novaclient: Support for abort an ongoing live migration https://review.openstack.org/284296 | 05:36 |
| *** jogo has quit IRC | 05:38 | |
| *** oomichi_ has joined #openstack-nova | 05:40 | |
| *** oomichi_ has quit IRC | 05:40 | |
| *** jogo has joined #openstack-nova | 05:41 | |
| *** sudipto has joined #openstack-nova | 05:42 | |
| *** fawadkhaliq has quit IRC | 05:42 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: add unit test for plugin test_pluginlib_nova.py https://review.openstack.org/286383 | 05:44 |
| *** haomaiwang has quit IRC | 05:45 | |
| *** sridharg has joined #openstack-nova | 05:45 | |
| *** haomaiwang has joined #openstack-nova | 05:45 | |
| *** takashin has joined #openstack-nova | 05:46 | |
| *** mylu has quit IRC | 05:46 | |
| *** mylu has joined #openstack-nova | 05:47 | |
| *** jaosorior has joined #openstack-nova | 05:49 | |
| *** irenab has quit IRC | 05:50 | |
| openstackgerrit | Srushti Gadadare proposed openstack/nova: Use check_string_length from oslo_utils https://review.openstack.org/286390 | 05:52 |
| *** Sukhdev has joined #openstack-nova | 05:55 | |
| *** fawadkhaliq has joined #openstack-nova | 05:58 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-nova | 06:01 | |
| *** Daisy_ has joined #openstack-nova | 06:02 | |
| *** lucasagomes has quit IRC | 06:02 | |
| *** sheel has joined #openstack-nova | 06:02 | |
| *** lucasagomes has joined #openstack-nova | 06:02 | |
| *** Daisy_ has quit IRC | 06:03 | |
| *** thorst has joined #openstack-nova | 06:06 | |
| *** rcernin has joined #openstack-nova | 06:09 | |
| *** thorst has quit IRC | 06:13 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Decode injected content when checking length limit https://review.openstack.org/285649 | 06:14 |
| eliqiao | oh my, why python-novaclient v2 api testing case still using V1's fixture_data? https://github.com/openstack/python-novaclient/blob/master/novaclient/tests/unit/v2/test_servers.py#L36, this won't work if we changes something return data in microversion | 06:16 |
| openstackgerrit | Feng Xi Yan proposed openstack/nova: <VMWARE> Add power state cache for host init and power state sync. https://review.openstack.org/242006 | 06:17 |
| openstackgerrit | Eli Qiao proposed openstack/python-novaclient: [microversion] Bump to 2.25 https://review.openstack.org/260383 | 06:18 |
| *** e0ne has joined #openstack-nova | 06:23 | |
| *** hoangcx has quit IRC | 06:24 | |
| *** lucasagomes has quit IRC | 06:24 | |
| *** lucasagomes has joined #openstack-nova | 06:27 | |
| *** e0ne has quit IRC | 06:31 | |
| *** esracelik has joined #openstack-nova | 06:33 | |
| *** e0ne has joined #openstack-nova | 06:33 | |
| *** e0ne has quit IRC | 06:34 | |
| openstackgerrit | Tang Chen proposed openstack/nova: Implement state machine for migration. https://review.openstack.org/241479 | 06:34 |
| openstackgerrit | Tang Chen proposed openstack/nova: Implement state machine for evacuation. https://review.openstack.org/241478 | 06:34 |
| openstackgerrit | Tang Chen proposed openstack/nova: Implement state machine for resize. https://review.openstack.org/241477 | 06:34 |
| *** dmellado has quit IRC | 06:34 | |
| *** whenry_ has quit IRC | 06:34 | |
| *** mariusv has quit IRC | 06:35 | |
| *** mariusv has joined #openstack-nova | 06:36 | |
| *** dmellado has joined #openstack-nova | 06:39 | |
| *** irenab has joined #openstack-nova | 06:39 | |
| *** lpetrut has joined #openstack-nova | 06:41 | |
| *** mylu has quit IRC | 06:45 | |
| *** Sukhdev has quit IRC | 06:45 | |
| *** deepakcs has joined #openstack-nova | 06:46 | |
| *** zenoway has joined #openstack-nova | 06:47 | |
| *** suresh12 has quit IRC | 06:47 | |
| *** suresh12 has joined #openstack-nova | 06:48 | |
| *** harlowja_at_home has quit IRC | 06:49 | |
| openstackgerrit | Angus Lees proposed openstack/nova: Add os-brick rootwrap filter for privsep https://review.openstack.org/277670 | 06:51 |
| openstackgerrit | Angus Lees proposed openstack/nova: Require privsep-based os-brick https://review.openstack.org/277671 | 06:51 |
| *** suresh12 has quit IRC | 06:53 | |
| *** ishant has joined #openstack-nova | 06:53 | |
| *** MVenesio has joined #openstack-nova | 06:54 | |
| *** Daisy has joined #openstack-nova | 06:56 | |
| *** lunarlamp has joined #openstack-nova | 06:56 | |
| *** Marga_ has quit IRC | 06:57 | |
| *** Daisy has quit IRC | 06:58 | |
| *** achanda has quit IRC | 06:58 | |
| *** MVenesio has quit IRC | 06:59 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-nova | 07:01 | |
| *** vilobhmm11 has quit IRC | 07:02 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Decode injected content when checking length limit https://review.openstack.org/285649 | 07:04 |
| *** scheuran has joined #openstack-nova | 07:05 | |
| *** Daisy_ has joined #openstack-nova | 07:06 | |
| *** vilobhmm11 has joined #openstack-nova | 07:07 | |
| *** thorst has joined #openstack-nova | 07:11 | |
| *** Daisy_ has quit IRC | 07:11 | |
| *** hoangcx has joined #openstack-nova | 07:11 | |
| *** liverpooler has joined #openstack-nova | 07:13 | |
| *** jwcroppe has quit IRC | 07:13 | |
| *** jwcroppe has joined #openstack-nova | 07:14 | |
| *** paul-carlton2 has joined #openstack-nova | 07:15 | |
| *** achanda has joined #openstack-nova | 07:15 | |
| *** mjura has joined #openstack-nova | 07:17 | |
| *** lpetrut has quit IRC | 07:17 | |
| *** thorst has quit IRC | 07:18 | |
| *** vilobhmm11 has quit IRC | 07:21 | |
| *** hoangcx has quit IRC | 07:26 | |
| eliqiao | ' | 07:26 |
| *** moshele has joined #openstack-nova | 07:28 | |
| *** suresh12 has joined #openstack-nova | 07:28 | |
| *** bnemec has quit IRC | 07:36 | |
| *** haukebruno has joined #openstack-nova | 07:37 | |
| *** bnemec has joined #openstack-nova | 07:39 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: WIP add changes-since support when list servers https://review.openstack.org/286423 | 07:43 |
| openstackgerrit | huan proposed openstack/nova: XenAPI:Resolve Nova/Neutron race condition https://review.openstack.org/241127 | 07:47 |
| *** suresh12 has quit IRC | 07:47 | |
| *** suresh12 has joined #openstack-nova | 07:48 | |
| *** suresh12 has quit IRC | 07:52 | |
| *** ankit_ag has joined #openstack-nova | 07:56 | |
| *** mpavone has joined #openstack-nova | 07:59 | |
| *** haomaiwang has quit IRC | 07:59 | |
| *** haomaiwang has joined #openstack-nova | 07:59 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** belmoreira has joined #openstack-nova | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** achanda_ has joined #openstack-nova | 08:02 | |
| *** bsv has joined #openstack-nova | 08:02 | |
| openstackgerrit | Fawad Khaliq proposed openstack/nova: remove network information from IOvisor vif https://review.openstack.org/259176 | 08:03 |
| *** achanda has quit IRC | 08:04 | |
| *** achanda_ has quit IRC | 08:04 | |
| *** jwcroppe_ has joined #openstack-nova | 08:04 | |
| *** slagle has quit IRC | 08:06 | |
| *** jwcroppe has quit IRC | 08:07 | |
| *** achanda has joined #openstack-nova | 08:08 | |
| *** fawadkhaliq has quit IRC | 08:08 | |
| *** armax has quit IRC | 08:08 | |
| *** pcaruana has joined #openstack-nova | 08:09 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add changes-since support when list servers https://review.openstack.org/286423 | 08:09 |
| *** ccesario has quit IRC | 08:09 | |
| *** jichen has joined #openstack-nova | 08:11 | |
| *** yamahata has quit IRC | 08:11 | |
| bsv | hi guys, im getting errors when trying to create volumes at instance creation time (in Horizon). After talking with mrunge on #openstack-horizon, he nailed down the api call to Nova that gives the error. | 08:15 |
| bsv | https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/instances/workflows/create_instance.py#L974 | 08:15 |
| *** PaulMurray has joined #openstack-nova | 08:15 | |
| bsv | Is this a know issue, I have found several old posts (1-2 years), about it. | 08:15 |
| bsv | Also, should there be any issues running Nova inside a LXC container? | 08:16 |
| *** thorst has joined #openstack-nova | 08:16 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add changes-since support when list servers https://review.openstack.org/286423 | 08:16 |
| *** amotoki has quit IRC | 08:18 | |
| *** amotoki has joined #openstack-nova | 08:19 | |
| *** andymaier has joined #openstack-nova | 08:19 | |
| *** rdopiera has joined #openstack-nova | 08:21 | |
| *** thorst has quit IRC | 08:23 | |
| *** mingyu has joined #openstack-nova | 08:25 | |
| *** ekuris_ has quit IRC | 08:25 | |
| *** achanda has quit IRC | 08:27 | |
| bauzas | good morning Nova | 08:27 |
| *** takashin has left #openstack-nova | 08:30 | |
| *** ociuhandu has quit IRC | 08:31 | |
| *** jlanoux has joined #openstack-nova | 08:32 | |
| *** tdurakov has joined #openstack-nova | 08:32 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient: Remove unused code in tests/unit/v2/fakes.py https://review.openstack.org/286435 | 08:33 |
| PaulMurray | eliqiao, shaohe_feng I think you're debugging the wrong patchset - https://review.openstack.org/#/c/284296/7 is broken | 08:34 |
| PaulMurray | eliqiao, shaohe_feng https://review.openstack.org/#/c/260383/7 is based on it | 08:34 |
| PaulMurray | eliqiao, BTW - why did you make trivial changes to a patch that had +W and passed jenkins ? A follow on would have been better at this stage | 08:35 |
| *** sridharg has quit IRC | 08:38 | |
| *** ccesario has joined #openstack-nova | 08:38 | |
| *** daemontool__ has joined #openstack-nova | 08:38 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common https://review.openstack.org/286442 | 08:39 |
| *** ekuris_ has joined #openstack-nova | 08:39 | |
| *** rk4n has joined #openstack-nova | 08:39 | |
| openstackgerrit | yuntongjin proposed openstack/nova: Add members in InstanceGroup object members field https://review.openstack.org/286443 | 08:40 |
| *** daemontool_ has quit IRC | 08:41 | |
| *** mpavone has quit IRC | 08:42 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: add unit test for plugin bandwidth https://review.openstack.org/286445 | 08:42 |
| *** mpavone has joined #openstack-nova | 08:43 | |
| eliqiao | hi PaulMurray , sorry about that. I don't notice that will break the process. | 08:43 |
| *** ndipanov has joined #openstack-nova | 08:44 | |
| PaulMurray | eliqiao, do you have any idea why that test is breaking - it does seem unrelated | 08:44 |
| openstackgerrit | Rui Chen proposed openstack/nova-specs: Task states improvement https://review.openstack.org/216578 | 08:44 |
| eliqiao | PaulMurray: I was thinking that v2.23 still need to be refine and my patch v2.25 will need based on v2.24, so I did a rebase. | 08:44 |
| openstackgerrit | Rui Chen proposed openstack/nova: Create image for suspended instance booted from volume https://review.openstack.org/223382 | 08:44 |
| eliqiao | PaulMurray: let me check the log. | 08:44 |
| *** matrohon has joined #openstack-nova | 08:45 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Only return latest fault of each instance in db api https://review.openstack.org/223392 | 08:45 |
| bauzas | Yingxin: thanks for your figures, really interesting | 08:46 |
| bauzas | Yingxin: I'll take time to look thru that before replying, but probably after M-3 (and FF) | 08:47 |
| openstackgerrit | Rui Chen proposed openstack/python-novaclient: Fix Resource.__eq__ mismatch semantics of object equal https://review.openstack.org/251020 | 08:47 |
| *** xiexs has joined #openstack-nova | 08:47 | |
| *** salv-orl_ has quit IRC | 08:47 | |
| *** openstackgerrit has quit IRC | 08:48 | |
| *** salv-orlando has joined #openstack-nova | 08:48 | |
| *** openstackgerrit has joined #openstack-nova | 08:48 | |
| *** sridharg has joined #openstack-nova | 08:49 | |
| eliqiao | PaulMurray: that's the gate's issue, an unrelated case test_trigger_crash_dump_in_resized_state faild, I will do a recheck. | 08:49 |
| PaulMurray | eliqiao, it happened in both that patch and the following one | 08:50 |
| shaohe_feng | eliqiao: PaulMurray : our patches do not touch the test_trigger_crash_dump_in_resized_state code. let's try recheck. | 08:51 |
| *** mingyu_ has joined #openstack-nova | 08:51 | |
| shaohe_feng | PaulMurray: eliqiao: alex_xu: does the migration-list should also display the migration id? | 08:52 |
| *** pav0 has joined #openstack-nova | 08:52 | |
| *** mingyu_ has quit IRC | 08:52 | |
| *** mingyu_ has joined #openstack-nova | 08:52 | |
| *** sridharg has quit IRC | 08:53 | |
| *** sridharg has joined #openstack-nova | 08:53 | |
| *** MVenesio has joined #openstack-nova | 08:54 | |
| alex_xu | yea, I suggest that, as we have server-migration-show, which accept args server id and migration id. but migration-list didn't show that id even the server-migration-list. that means nobody can use server-migration-show.... | 08:54 |
| *** mingyu has quit IRC | 08:54 | |
| alex_xu | shaohe_feng: is there anywhere people can get the migration id from the cli? | 08:55 |
| alex_xu | if not, that means server-migration-show nobody can use it... | 08:55 |
| shaohe_feng | PaulMurray: eliqiao: alex_xu: yes. migration-list command display the migration id. Then the admin can check the migrations in the list is in-progress live migration. Then he can use the server-migration-show server-uuid migration-id to get progress details | 08:55 |
| eliqiao | PaulMurray: yes, I see. sorry about that, but we don't touch that's case at all, let see the recheck result. | 08:55 |
| shaohe_feng | alex_xu: seems not. let me check it. | 08:56 |
| *** haukebruno has quit IRC | 08:57 | |
| * PaulMurray I'm in a call for a while, but still lurking | 08:58 | |
| andreykurilin | eliqiao: hi! Can we just remove classmethod decorator? | 08:59 |
| eliqiao | andreykurilin: seems no | 08:59 |
| andreykurilin | :( | 08:59 |
| eliqiao | I will show you andreykurilin | 08:59 |
| eliqiao | andreykurilin: https://review.openstack.org/#/c/260383/6/novaclient/tests/unit/v2/fakes.py | 09:00 |
| *** MVenesio has quit IRC | 09:00 | |
| *** paul-carlton2 has quit IRC | 09:00 | |
| *** haomaiwang has quit IRC | 09:01 | |
| andreykurilin | eliqiao: about v1 fixtures: v1_1 was the same as v2. While deprecation of v1_1, we forgot to rename the fixture, but it should be ok:) | 09:01 |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| Yingxin | bauzas: thank you, I'll start to write the specs because the resource provider design is relatively stable I think. | 09:01 |
| *** andymaier has quit IRC | 09:01 | |
| andreykurilin | eliqiao: ok, what about transmitting api_version as an argument of method? | 09:02 |
| bauzas | Yingxin: yup, that's one of my priorities for Mitaka => merging the resource-providers branch starting here https://review.openstack.org/#/c/281837/ | 09:02 |
| eliqiao | andreykurilin: yeah, but that v1 fixtures is not versioned, so we can not check the version inside of it. | 09:02 |
| bauzas | Yingxin: the last patch of the branch being https://review.openstack.org/#/c/279313/ | 09:02 |
| bauzas | Yingxin: that's what we agreed to land for this release | 09:03 |
| bauzas | (so basically, the DB tables and those populated) | 09:03 |
| andreykurilin | eliqiao: so we need to modify it:) | 09:03 |
| bauzas | for the RT | 09:03 |
| eliqiao | andreykurilin: hmm.. it seems that we need to redesign, because of it is a static data which is initiald while setup, we don't have a way to pass api_version to it | 09:03 |
| Yingxin | bauzas: I have more free time to do investigations since there are no works assigned to me by the scheduler subteam currently. I'll focus on reviewing in my free time :) | 09:04 |
| andreykurilin | eliqiao: ok let's skip it now and fix it in next patch | 09:04 |
| bauzas | Yingxin: feel free to discuss with cdent too, both of you are interested in that topic | 09:04 |
| Yingxin | bauzas: ok | 09:05 |
| eliqiao | andreykurilin: okay, thanks, I think we need more adoption to achieve that. | 09:05 |
| bauzas | Yingxin: you know the release process, right? | 09:05 |
| Yingxin | yes, there's a wiki | 09:05 |
| bauzas | Yingxin: so, we'll be cutting Mitaka-3 by Thursday EU morning, then we'll be on RC1 | 09:06 |
| bauzas | once RC1 is cut, master branch becomes Newton, and possible other RCs are on the mitaka stable branch | 09:06 |
| *** rk4n has quit IRC | 09:06 | |
| bauzas | Yingxin: which means that we'll focus on the specs by RC1 | 09:07 |
| bauzas | I mean, not by RC1, but once RC1 is shipped | 09:07 |
| bauzas | Yingxin: in the meantime, you can either help on bugs, or sharpen your shared-state-scheduler branch | 09:08 |
| Yingxin | bauzas: thanks for reminding, I have two specs to ship, one is eventually-consistent bp, another is scheduler functional test framework. | 09:08 |
| bauzas | yeah I remember | 09:08 |
| *** ildikov has quit IRC | 09:08 | |
| Yingxin | bauzas: The latter one has a lower priority to me, because I also have two sessions to prepare maybe. | 09:08 |
| *** rk4n has joined #openstack-nova | 09:08 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Improvements for BaseCoreFilter https://review.openstack.org/285427 | 09:09 |
| bauzas | Yingxin: honestly, I leave johnthetubaguy answer but I think that scheduler functional test framework doesn't require a spec | 09:09 |
| bauzas | Yingxin: it would be thrown under the bus by the fact that we have very little review bandwidth until we cut RC1 (and even the release), but that's not technically needing some spec approval | 09:10 |
| Yingxin | bauzas: that's good | 09:10 |
| * bauzas walks away from keyboard, have an appointment | 09:11 | |
| bauzas | bbiab | 09:11 |
| *** salv-orlando has quit IRC | 09:11 | |
| * Yingxin gonna have supper | 09:11 | |
| *** ndipanov has quit IRC | 09:11 | |
| *** mingyu_ has quit IRC | 09:13 | |
| *** ihrachys has joined #openstack-nova | 09:13 | |
| *** amotoki has quit IRC | 09:14 | |
| *** jistr has joined #openstack-nova | 09:14 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Decode injected content when checking length limit https://review.openstack.org/285649 | 09:15 |
| *** scheuran has quit IRC | 09:15 | |
| *** andymaier has joined #openstack-nova | 09:18 | |
| *** amotoki has joined #openstack-nova | 09:19 | |
| *** thorst has joined #openstack-nova | 09:22 | |
| *** shaohe_feng has quit IRC | 09:23 | |
| *** shh_away has quit IRC | 09:25 | |
| *** shaohe_feng has joined #openstack-nova | 09:25 | |
| *** scheuran has joined #openstack-nova | 09:27 | |
| *** paul-carlton2 has joined #openstack-nova | 09:28 | |
| *** danpb has joined #openstack-nova | 09:28 | |
| *** e0ne has joined #openstack-nova | 09:28 | |
| johnthetubaguy | Yingxin yes, agreed with bauzas, that doesn't need a spec, although I would reach out to sdague when he is online, as he has a good idea of how our tests should be structured in the future, and it would be good to double check your approach fits in with all that | 09:28 |
| *** thorst has quit IRC | 09:28 | |
| johnthetubaguy | Yingxin: its not something I would block for Feature Freeze, but its certainly not a priority till newton at this point, we can get it a specless blueprint for newton | 09:29 |
| *** aviw has joined #openstack-nova | 09:29 | |
| *** haukebruno has joined #openstack-nova | 09:29 | |
| aviw | Hi, I wanted to ask what libvirt version currently get deployed with nova when installed from nova master ? | 09:30 |
| *** yassine__ has joined #openstack-nova | 09:32 | |
| johnthetubaguy | aviw: non, thats a system package you need to install, now devstack in our test infrastructure installs a specific version, but that not really the same thing. For the python bindings its libvirt-python===1.3.1 due to: https://github.com/openstack/requirements/blob/master/upper-constraints.txt | 09:32 |
| johnthetubaguy | s/non/none/ | 09:33 |
| sudipto | johnthetubaguy, Hi, could you pls have a look at this patch when you can: https://review.openstack.org/#/c/245765/ This stops powerpc from having NUMA support at all. I think i have addressed all the review comments... | 09:33 |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Use novaclient/utils directly and remove openstack/common https://review.openstack.org/286442 | 09:34 |
| *** mpavone has quit IRC | 09:34 | |
| *** davideagle has joined #openstack-nova | 09:34 | |
| johnthetubaguy | sudipto: hopefully get to those things today, but focusing on cutting mitaka-3 for the next day or so | 09:35 |
| *** ildikov has joined #openstack-nova | 09:35 | |
| *** ociuhandu has joined #openstack-nova | 09:35 | |
| *** mpavone has joined #openstack-nova | 09:36 | |
| *** mpavone has quit IRC | 09:36 | |
| sudipto | johnthetubaguy, ok sure. It's not a complicated patchset... | 09:36 |
| aviw | johnthetubaguy: I see, when will 1.3.2 version be used? I need that specific version to integrate a fix related to ceilometer. | 09:36 |
| *** mpavone has joined #openstack-nova | 09:36 | |
| *** lpetrut has joined #openstack-nova | 09:36 | |
| pkoniszewski | aviw: i think that it can be used actually, problem is that 1.3.2 is not part of any linux repo yet | 09:38 |
| *** e0ne has quit IRC | 09:38 | |
| *** mpavone has quit IRC | 09:38 | |
| aviw | I see, I heard that 1.3.2 got released by end of Feb. | 09:38 |
| *** derekh has joined #openstack-nova | 09:39 | |
| pkoniszewski | Xenial will support 1.3.1 - http://packages.ubuntu.com/xenial/libvirt-bin | 09:39 |
| *** hoangcx has joined #openstack-nova | 09:40 | |
| *** mpavone has joined #openstack-nova | 09:40 | |
| pkoniszewski | so it will take some time until 1.3.2 (or newer) is supported | 09:40 |
| *** rha has joined #openstack-nova | 09:40 | |
| *** rha has joined #openstack-nova | 09:40 | |
| markus_z | nova-bugs-team meeting in 20 minutes in #openstack-meeting-4 see https://wiki.openstack.org/wiki/Meetings/Nova/BugsTeam for details | 09:41 |
| aviw | pkoniszewski: so install (e.g. devstack), takes libvirt from linux repo ? not from git hub? | 09:41 |
| pkoniszewski | aviw: exactly, that's why we still use libvirt 1.2.2 in gate | 09:41 |
| pkoniszewski | aviw: because it is based on ubuntu 14.04 | 09:41 |
| aviw | pkoniszewski: I see. And python binding also taken from linux repo? | 09:42 |
| johnthetubaguy | aviw: 1.3.2 will not be required by Nova for a long time, I think our current minimum is 0.10.2 see: https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | 09:42 |
| johnthetubaguy | aviw: python binding comes from pip | 09:43 |
| pkoniszewski | aviw: i think it's part of requirements so it's taken from pypi | 09:43 |
| aviw | pkoniszewski: i see | 09:43 |
| pkoniszewski | aviw: also note what John said, we support 0.10.2 libvirt right now - https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L406 | 09:43 |
| pkoniszewski | aviw: and the next min version supported by nova will be 1.2.1 | 09:44 |
| *** mylu has joined #openstack-nova | 09:45 | |
| aviw | pkoniszewski: so it will take quite of a time when it reaches 1.3.2. Can you assess how long? couple of weeks? months.. ? | 09:46 |
| aviw | johnthetubaguy: thanks for the info | 09:47 |
| pkoniszewski | aviw: i'd say months, or even more until min libvirt version in nova will be >= 1.3.2 | 09:47 |
| pkoniszewski | aviw: but there is still a chance to get your fix earlier | 09:48 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Console log copy for block live migratiob https://review.openstack.org/284674 | 09:48 |
| johnthetubaguy | so we allow features for newer versions a bit sooner than that, but thats a tricky discussion, we are really waiting on a viable CI for a "new" libvirt at this point | 09:48 |
| pkoniszewski | aviw: we have a lot of features that require newer libvirt than actually supported | 09:48 |
| *** aarefiev has quit IRC | 09:48 | |
| pkoniszewski | aviw: you can find list, starting from here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L416 | 09:48 |
| *** Cao__ has joined #openstack-nova | 09:49 | |
| pkoniszewski | aviw: so we do allow checks, whether libvirt installed on compute node supports specific feature, e.g., https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5231 | 09:49 |
| johnthetubaguy | yeah, we need the CI in place before we get any more features we can't test, at least thats what we seemed to roughly agreed at the midcycle | 09:49 |
| *** aarefiev has joined #openstack-nova | 09:49 | |
| pkoniszewski | and that's another point | 09:50 |
| *** mylu has quit IRC | 09:50 | |
| Cao__ | Can anyone help me to review this, thanks in advance: | 09:51 |
| markus_z | johnthetubaguy: I try to get in touch with tonyb this week. I agreed to take over his efforts there but couldn't get anything done yet. | 09:51 |
| Cao__ | https://review.openstack.org/#/c/276639/ | 09:51 |
| *** mnestratov has joined #openstack-nova | 09:51 | |
| tonyb | markus_z: que? | 09:52 |
| markus_z | tonyb: Huh, still online. It's about the bleeding libvirt gate job | 09:53 |
| tonyb | markus_z: Oh right | 09:53 |
| johnthetubaguy | cool, I meant to catch up with you both about that | 09:53 |
| tonyb | markus_z, johnthetubaguy: So I have the prototype | 09:53 |
| *** EinstCrazy has quit IRC | 09:54 | |
| tonyb | I'me just working the how do we get it in the gate process | 09:54 |
| markus_z | tonyb: I have the nova bugs team meeting in a few minutes. How long are you online today? | 09:54 |
| aviw | pkoniszewski: thanks.. your info is | 09:54 |
| tonyb | markus_z, johnthetubaguy: https://github.com/tbreeds/tbreeds-devstack-plugin-additional-pkg-repos is the plugin | 09:54 |
| aviw | pkoniszewski: thanks for the info. | 09:54 |
| *** EinstCrazy has joined #openstack-nova | 09:54 | |
| *** andrearosa_web has joined #openstack-nova | 09:54 | |
| tonyb | markus_z: I was about to go to be but I can stay up for a bit | 09:54 |
| markus_z | tonyb: I think it's getting to late. tomorrow morning? 8am UTC? | 09:55 |
| markus_z | 1-2 hours earlier is fine too | 09:56 |
| tonyb | markus_z: 0900UTC would be better | 09:57 |
| tonyb | 6-8UTC is prime kids dinner and bath time | 09:57 |
| markus_z | tonyb: perfect. I'll look at your github repo until then. | 09:57 |
| tonyb | markus_z: cool. | 09:58 |
| tonyb | markus_z: there isn't that much there as it turns out | 09:58 |
| markus_z | tonyb: we'll see :) good night, see you tomorrow | 09:59 |
| tonyb | markus_z: talk tomorrow | 09:59 |
| *** ociuhandu has quit IRC | 09:59 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** ptm_away has joined #openstack-nova | 10:01 | |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| *** PaulMurray has quit IRC | 10:04 | |
| BobBall | johnthetubaguy: Unit tests for XAPI plugins. Kevin has requested a new env + job to test these in py24 but I think they could be tested in the py27 env; what are your thoughts? https://review.openstack.org/#/c/283962/ | 10:05 |
| johnthetubaguy | BobBall: seems fine to do them in py27, as a starting point | 10:06 |
| BobBall | johnthetubaguy: Would you mind commenting on that change with your view? | 10:07 |
| BobBall | alternatively I can start a ML discussion if that would be the easier way? | 10:07 |
| markus_z | Who's our focal point for notifications again? I forgot the name. | 10:08 |
| _gryf | markus_z, I think gibi | 10:09 |
| markus_z | _gryf: yes, right, thanks | 10:09 |
| *** davideagle has quit IRC | 10:10 | |
| markus_z | gibi: Could you have a look at https://bugs.launchpad.net/nova/+bug/1551641 please? I doubt that this is a bug, but I'd like to have your opinion there. | 10:10 |
| openstack | Launchpad bug 1551641 in OpenStack Compute (nova) "unable to config topic for versioned notifier" [Low,Incomplete] - Assigned to Liyingjun (liyingjun) | 10:10 |
| johnthetubaguy | BobBall: I can comment on the patch | 10:10 |
| BobBall | Awesome. Thanks. One other option may be to get the py27 job to run a new subdir which the py34 test completely ignores? | 10:11 |
| *** e0ne has joined #openstack-nova | 10:11 | |
| *** chhavi has quit IRC | 10:13 | |
| johnthetubaguy | BobBall: possibly, but isn't that tests/virt/xenapi/plugins at this point? | 10:13 |
| johnthetubaguy | BobBall: I would really like us to focus on the ones that change a lot first, like the glance one | 10:13 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/268932 | 10:13 |
| BobBall | johnthetubaguy: yes - but py34 job will include _everything_ under nova/tests/unit by default | 10:14 |
| BobBall | If we put them into a differnt subdir (e.g. nova/test/plugins/xenapi?) then py34 job wouldn't try and parse. | 10:14 |
| johnthetubaguy | BobBall: the important bit is to get them in there, we can work this all out as time goes one | 10:14 |
| BobBall | which would make the implementation much cleaner since the plugins aren't py34 parsable, so we have to currently have nasty hacks in the way we import them to avoid breaking the py34 job | 10:15 |
| *** EinstCrazy has quit IRC | 10:15 | |
| *** EinstCrazy has joined #openstack-nova | 10:16 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/nova: Keep py3.X compatibility for urllib https://review.openstack.org/261034 | 10:16 |
| johnthetubaguy | BobBall: whats wrong with just adding those tests on the exclude list? I am confused? | 10:17 |
| BobBall | py34 will parse all files first, to find all tests, then exclude the ones it can't run. The plugins are not all py34 compatible, and do not parse in a py34 environment, therefore having a unit test that imports the plugins in the normal way would cause the py34 job to fail to parse. | 10:19 |
| johnthetubaguy | BobBall: but we don't want those plugin files in our standard nova tree anyways | 10:19 |
| johnthetubaguy | BobBall: regardless of py34 | 10:19 |
| johnthetubaguy | BobBall: that looks like the same magic I used in my poc patches for the tests | 10:20 |
| BobBall | johnthetubaguy: We're not _moving_ the plugin files, just importing them | 10:20 |
| johnthetubaguy | yes | 10:20 |
| johnthetubaguy | I am saying thats what I want to have happen | 10:20 |
| johnthetubaguy | you couldn't import them normally if they are outside the tree, anyway | 10:20 |
| johnthetubaguy | well, I guess you could mod the python path, but that seems stranger | 10:21 |
| *** EinstCrazy has quit IRC | 10:21 | |
| gibi | markus_z: looking... | 10:22 |
| markus_z | gibi: thanks! | 10:22 |
| BobBall | johnthetubaguy: I mean you can't invoke plugins.load_plugin globally; you have to do it within the test in the setup method | 10:22 |
| BobBall | johnthetubaguy: which is a little strange :) | 10:22 |
| BobBall | See https://review.openstack.org/#/c/283962/1/nova/tests/unit/virt/xenapi/plugins/test_pluginlib_nova.py which failed in py34 because it was importing pluginlib_nova.py globally | 10:22 |
| gibi | markus_z: this was an easy one.:) No it is not a bug. The spec was approved with the hard coded topic. So if somebody needs a settable topic then that is a new feature | 10:23 |
| gibi | markus_z: commenting to the bug... | 10:23 |
| johnthetubaguy | BobBall: the test setup still feels like the correct place to me for this stuff, but maybe thats just me | 10:23 |
| markus_z | gibi: cool, thanks! I wondered why we made the topics configurable for the other stuff. I didn't see any reason for that. | 10:24 |
| gibi | markus_z: agree | 10:24 |
| *** Pistachio has joined #openstack-nova | 10:24 | |
| BobBall | johnthetubaguy: Fair enough! | 10:24 |
| *** andymaier has quit IRC | 10:25 | |
| *** Pistachio33 has joined #openstack-nova | 10:25 | |
| *** ptm_away is now known as PaulMurray | 10:26 | |
| *** ildikov has quit IRC | 10:26 | |
| * bauzas is back | 10:26 | |
| *** thorst has joined #openstack-nova | 10:26 | |
| BobBall | johnthetubaguy: What are your thoughts of having the plugin unit tests in another subdir (such as nova/tests/plugins) vs including them in the py34 exclusion list? | 10:27 |
| johnthetubaguy | BobBall: I don't really mind, I would rather we get them merged | 10:27 |
| *** ishant has quit IRC | 10:28 | |
| *** Pistachio has quit IRC | 10:29 | |
| *** ndipanov has joined #openstack-nova | 10:29 | |
| *** lucasagomes has quit IRC | 10:30 | |
| *** lucasagomes has joined #openstack-nova | 10:31 | |
| BobBall | understood | 10:31 |
| *** lucasagomes has quit IRC | 10:32 | |
| *** lucasagomes has joined #openstack-nova | 10:32 | |
| *** thorst has quit IRC | 10:33 | |
| *** Cao__ has quit IRC | 10:36 | |
| *** daemontool has joined #openstack-nova | 10:36 | |
| *** lucasagomes has quit IRC | 10:37 | |
| -openstackstatus- NOTICE: Gerrit is going to be restarted due to poor performance | 10:37 | |
| *** ChanServ changes topic to "Gerrit is going to be restarted due to poor performance" | 10:37 | |
| *** lucasagomes has joined #openstack-nova | 10:37 | |
| *** chhavi has joined #openstack-nova | 10:38 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs: Xenapi: a new VDI store via streaming https://review.openstack.org/274045 | 10:39 |
| *** ildikov has joined #openstack-nova | 10:40 | |
| *** daemontool__ has quit IRC | 10:40 | |
| *** lucasagomes has quit IRC | 10:41 | |
| *** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Mitaka_Release_Schedule" | 10:43 | |
| -openstackstatus- NOTICE: gerrit finished restartign | 10:43 | |
| *** lucasagomes has joined #openstack-nova | 10:43 | |
| *** phuongnh has quit IRC | 10:45 | |
| *** Pistachio33 has quit IRC | 10:47 | |
| *** ihrachys has quit IRC | 10:48 | |
| markus_z | ndipanov: Could you have a short look at https://bugs.launchpad.net/nova/+bug/1550250 please? The "delete_on_termination" flag for a volume gets changed during a volume migration. | 10:49 |
| openstack | Launchpad bug 1550250 in OpenStack Compute (nova) "migrate in-use status volume, the volume's "delete_on_termination" flag lost" [Undecided,New] - Assigned to YaoZheng_ZTE (zheng-yao1) | 10:49 |
| *** davideagle has joined #openstack-nova | 10:54 | |
| *** MVenesio has joined #openstack-nova | 10:55 | |
| *** vladikr__ has quit IRC | 10:58 | |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 10:59 |
| *** ptm_away has joined #openstack-nova | 11:00 | |
| *** vladikr__ has joined #openstack-nova | 11:00 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** MVenesio has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-nova | 11:01 | |
| *** cdent has joined #openstack-nova | 11:01 | |
| openstackgerrit | jichenjc proposed openstack/nova: Make test cases test_crypto.py from NoDBTestCase https://review.openstack.org/233786 | 11:02 |
| *** PaulMurray has quit IRC | 11:03 | |
| *** mvk has quit IRC | 11:03 | |
| *** dims has joined #openstack-nova | 11:03 | |
| *** hoangcx has quit IRC | 11:06 | |
| *** amotoki has quit IRC | 11:06 | |
| *** ptm_away is now known as PaulMurray | 11:09 | |
| *** davideagle has quit IRC | 11:11 | |
| Yingxin | johnthetubaguy: ok | 11:11 |
| *** mingyu has joined #openstack-nova | 11:13 | |
| *** mingyu has quit IRC | 11:16 | |
| *** dedery has joined #openstack-nova | 11:17 | |
| *** mvk has joined #openstack-nova | 11:20 | |
| ndipanov | markus_z, hmmm | 11:22 |
| *** tbachman has quit IRC | 11:22 | |
| ndipanov | markus_z, so it might be a nova bug | 11:22 |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 11:24 |
| *** claudiub has joined #openstack-nova | 11:25 | |
| *** shaohe_feng_ has joined #openstack-nova | 11:26 | |
| *** rodrigods has quit IRC | 11:29 | |
| *** rodrigods has joined #openstack-nova | 11:29 | |
| *** jlanoux_ has joined #openstack-nova | 11:30 | |
| *** jlanoux has quit IRC | 11:30 | |
| *** thorst has joined #openstack-nova | 11:31 | |
| *** dedery has quit IRC | 11:33 | |
| *** PaulMurray has quit IRC | 11:34 | |
| *** PaulMurray has joined #openstack-nova | 11:35 | |
| *** vladikr__ has quit IRC | 11:35 | |
| *** sudipto has quit IRC | 11:36 | |
| *** ildikov has quit IRC | 11:36 | |
| *** thorst has quit IRC | 11:38 | |
| *** aysyd has joined #openstack-nova | 11:39 | |
| *** ihrachys has joined #openstack-nova | 11:39 | |
| *** ociuhandu has joined #openstack-nova | 11:40 | |
| *** salv-orlando has joined #openstack-nova | 11:41 | |
| *** PaulMurray has quit IRC | 11:43 | |
| *** houming has quit IRC | 11:45 | |
| *** tbachman has joined #openstack-nova | 11:45 | |
| *** smatzek has joined #openstack-nova | 11:45 | |
| *** wxy has left #openstack-nova | 11:45 | |
| *** aysyd has quit IRC | 11:46 | |
| *** aysyd has joined #openstack-nova | 11:46 | |
| *** mylu has joined #openstack-nova | 11:46 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Repropose expose-host-capabilities to Newton https://review.openstack.org/286520 | 11:50 |
| *** jlanoux_ has quit IRC | 11:50 | |
| *** mylu has quit IRC | 11:51 | |
| *** Jeffrey4l has quit IRC | 11:51 | |
| *** sdague has joined #openstack-nova | 11:51 | |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 11:51 |
| *** jlanoux has joined #openstack-nova | 11:51 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Deprecate nova.hooks https://review.openstack.org/286276 | 11:52 |
| markus_z | ndipanov: So, no need to poke Cinder folks? Do you maybe have a pointer for me where this happens in the code? I want to get a picture of it too. | 11:54 |
| *** salv-orlando has quit IRC | 11:58 | |
| *** amotoki has joined #openstack-nova | 11:58 | |
| *** mylu has joined #openstack-nova | 11:58 | |
| *** gmann_ has joined #openstack-nova | 11:59 | |
| *** davideagle has joined #openstack-nova | 12:00 | |
| * alex_xu reminder nova api meeting is running | 12:00 | |
| *** haomaiwang has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-nova | 12:01 | |
| *** Hazelesque_ is now known as Hazelesque | 12:03 | |
| *** bjornar__ has quit IRC | 12:07 | |
| *** daemontool_ has joined #openstack-nova | 12:07 | |
| *** andrei_ has quit IRC | 12:07 | |
| *** rfolco_ has joined #openstack-nova | 12:07 | |
| *** amotoki has quit IRC | 12:07 | |
| *** andrei_ has joined #openstack-nova | 12:08 | |
| *** bjornar__ has joined #openstack-nova | 12:09 | |
| *** Jeffrey4l has joined #openstack-nova | 12:09 | |
| *** daemontool has quit IRC | 12:10 | |
| *** MVenesio has joined #openstack-nova | 12:11 | |
| *** haukebruno has quit IRC | 12:12 | |
| claudiub | johnthetubaguy: hello. If you have one free minute, can you take a look at one small bugfix? https://review.openstack.org/#/c/285370/ Basically, it will fix the live-migration issue on hyper-v. Thanks! | 12:13 |
| claudiub | it already has a +2. | 12:13 |
| *** davideagle has quit IRC | 12:14 | |
| *** rtheis has joined #openstack-nova | 12:15 | |
| cdent | alaski: In case you missed it, the cross-project quota stuff that we talked about a while ago has become it's own meeting, outside of the regular cross-project meeting: http://lists.openstack.org/pipermail/openstack-dev/2016-February/087684.html | 12:15 |
| *** raildo-afk is now known as raildo | 12:18 | |
| *** PaulMurray has joined #openstack-nova | 12:18 | |
| PaulMurray | /msg NickServ identify mmharc | 12:19 |
| PaulMurray | interesting | 12:19 |
| *** gszasz has joined #openstack-nova | 12:22 | |
| *** ccesario has quit IRC | 12:24 | |
| *** irenab has quit IRC | 12:24 | |
| ndipanov | markus_z, I think it's swap volume | 12:25 |
| kashyap | PaulMurray: You're not going to change the passwd, are you? :-) | 12:25 |
| PaulMurray | kashyap, already done :) | 12:26 |
| PaulMurray | I will watch out for those leading spaces in future | 12:27 |
| kashyap | PaulMurray: FWIW, to avoid this, I go the 'status' bar and do the auth | 12:27 |
| markus_z | ndipanov: ok, thanks | 12:27 |
| PaulMurray | kashyap, yeah, I keep losing the connection and just forgot | 12:27 |
| *** dave-mccowan has joined #openstack-nova | 12:27 | |
| * kashyap nods | 12:28 | |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 12:28 |
| *** ankit_ag has quit IRC | 12:29 | |
| *** davideagle has joined #openstack-nova | 12:29 | |
| *** andymaier has joined #openstack-nova | 12:30 | |
| *** daemontool__ has joined #openstack-nova | 12:34 | |
| *** jaosorior has quit IRC | 12:36 | |
| *** jaosorior has joined #openstack-nova | 12:37 | |
| *** daemontool_ has quit IRC | 12:37 | |
| *** moshele has quit IRC | 12:38 | |
| *** ildikov has joined #openstack-nova | 12:38 | |
| *** moshele has joined #openstack-nova | 12:38 | |
| alaski | cdent: thanks. I dropped in on it yesterday right at the end. I just need to add it to my calendar so I can catch it at the beginning. | 12:43 |
| cdent | alaski: cool, assumed you probably had seen it, but wanted to make sure I dispensed with my official cpl duties or whatever. Now I can stop thinking about that. | 12:44 |
| *** davideagle has quit IRC | 12:45 | |
| * PaulMurray goes to lunch | 12:45 | |
| alaski | cdent: I have officially taken the baton for this | 12:45 |
| cdent | \o/ | 12:45 |
| *** thorst has joined #openstack-nova | 12:46 | |
| *** thorst_ has joined #openstack-nova | 12:47 | |
| *** davideagle has joined #openstack-nova | 12:48 | |
| *** jaypipes has joined #openstack-nova | 12:48 | |
| markus_z | dansmith: You're the assignee of https://bugs.launchpad.net/nova/+bug/1449062 which is the last "liberty-rc-potential" (not a typo). Comment 41 said it is fixed but the reporter reopened. | 12:48 |
| openstack | Launchpad bug 1449062 in Glance "qemu-img calls need to be restricted by ulimit (CVE-2015-5162)" [High,In progress] - Assigned to nikhil komawar (nikhil-komawar) | 12:48 |
| markus_z | dansmith: Do you know more about this? | 12:49 |
| jaypipes | morning folks. | 12:49 |
| markus_z | morning jay | 12:49 |
| _gryf | afternoon, jaypipes | 12:50 |
| *** PaulMurray has quit IRC | 12:50 | |
| jaypipes | dansmith, bauzas, cdent: hey, sorry I missed the scheduler meeting yesterday morning. unfortunately, I was somewhere over Tennessee stuck in a middle seat with no room to open a laptop :( | 12:50 |
| *** PaulMurray has joined #openstack-nova | 12:50 | |
| cdent | jaypipes: I sent you some links to relevant conversation (at least I think I did) | 12:51 |
| *** thorst has quit IRC | 12:51 | |
| *** amotoki has joined #openstack-nova | 12:52 | |
| *** lbeliveau has joined #openstack-nova | 12:52 | |
| *** vladikr__ has joined #openstack-nova | 12:53 | |
| jaypipes | cdent: cool. trying to get through my email before meetings begin... | 12:54 |
| *** baoli has joined #openstack-nova | 12:54 | |
| *** salv-orlando has joined #openstack-nova | 12:55 | |
| *** andymaier has quit IRC | 12:57 | |
| *** mpavone has quit IRC | 12:58 | |
| *** gmann_ has quit IRC | 13:00 | |
| alex_xu | sdague: sorry, I need go home now, and when I reach home probably late. if not hurry, I will take a look at the regression tests tomorrow | 13:00 |
| *** haomaiwang has quit IRC | 13:01 | |
| sdague | alex_xu: yep, no worries | 13:01 |
| *** haomaiwang has joined #openstack-nova | 13:01 | |
| *** vladikr__ is now known as vladikr | 13:01 | |
| alex_xu | sdague: ok, will check with you tomorrow morning, if I find something | 13:02 |
| alex_xu | s/morning/evening/... | 13:02 |
| *** dims has quit IRC | 13:02 | |
| sdague | alex_xu: yeh it did the right thing - http://logs.openstack.org/43/281143/16/check/gate-nova-tox-functional/70d60c9/console.html#_2016-02-29_17_34_59_392 | 13:02 |
| sdague | however os-testr didn't count that as a fail | 13:02 |
| sdague | I'll get on mtreinish about it | 13:03 |
| *** andrearosa_web has left #openstack-nova | 13:04 | |
| *** mgoddard_ has joined #openstack-nova | 13:04 | |
| sdague | johnthetubaguy: ok, can you see if the reno works for you now on - https://review.openstack.org/#/c/286276/ ? | 13:04 |
| *** ducttape_ has joined #openstack-nova | 13:05 | |
| *** ndipanov_ has joined #openstack-nova | 13:05 | |
| *** ndipanov has quit IRC | 13:06 | |
| *** mgoddard has quit IRC | 13:07 | |
| *** davideagle has quit IRC | 13:07 | |
| *** links has quit IRC | 13:07 | |
| *** sudipto has joined #openstack-nova | 13:08 | |
| *** dims has joined #openstack-nova | 13:08 | |
| *** irenab has joined #openstack-nova | 13:09 | |
| *** davideagle has joined #openstack-nova | 13:10 | |
| *** mpavone has joined #openstack-nova | 13:11 | |
| *** deepakcs has quit IRC | 13:13 | |
| *** pradk has joined #openstack-nova | 13:14 | |
| *** lucasagomes is now known as lucas-hungry | 13:15 | |
| *** haomaiwang has quit IRC | 13:15 | |
| *** haomaiwang has joined #openstack-nova | 13:17 | |
| *** mingyu has joined #openstack-nova | 13:17 | |
| *** ccesario has joined #openstack-nova | 13:18 | |
| *** Daisy has joined #openstack-nova | 13:19 | |
| *** andymaier has joined #openstack-nova | 13:20 | |
| *** tbachman has quit IRC | 13:20 | |
| *** haomaiwang has quit IRC | 13:22 | |
| *** mingyu has quit IRC | 13:22 | |
| bauzas | jaypipes: no worries, we just agreed to try to merge the resource-providers changes up to https://review.openstack.org/#/c/279313/ | 13:23 |
| *** davideagle has quit IRC | 13:23 | |
| bauzas | jaypipes: that said, I had a few concerns during reviewing those | 13:23 |
| bauzas | jaypipes: the main blocker for me being https://review.openstack.org/#/c/284484/3 | 13:23 |
| *** mylu has quit IRC | 13:24 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Fix wrong watermark state during live migration https://review.openstack.org/279616 | 13:25 |
| claudiub | bauzas: hello. mind if you take a look at a very small bugfix? it is a importance bug and it already has a +2: https://review.openstack.org/#/c/285370/ | 13:26 |
| *** mgoddard_ has quit IRC | 13:26 | |
| bauzas | claudiub: sure, that said if this is a bugfix, you can be sure it would reviewed by after FF ;) | 13:26 |
| *** mgoddard_ has joined #openstack-nova | 13:27 | |
| *** mingyu has joined #openstack-nova | 13:27 | |
| claudiub | bauzas: sure, but the hyper-v ci will keep failing until the bugfix merges. :) | 13:28 |
| bauzas | a-ha, good point, thanks ! | 13:28 |
| bauzas | claudiub: I was just seeing "As a result, this issue breaks Hyper-V live migration." in the bug report, not that the Hyper-V CI being sad | 13:29 |
| claudiub | bauzas: :)) true, but live migration is included as a tempest test. :) Plus, green is the color of happiness and red is not. :D | 13:30 |
| *** ducttape_ has quit IRC | 13:31 | |
| bauzas | claudiub: fair enough | 13:31 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Console log copy for block live migration https://review.openstack.org/284674 | 13:31 |
| eliqiao | hello, does anyone know why nova-manage db sync doesn't create request_specs table? | 13:31 |
| *** davideagle has joined #openstack-nova | 13:32 | |
| *** slagle has joined #openstack-nova | 13:33 | |
| alaski | eliqiao: it's in a different db and has a separate command 'nova-manage api_db sync'. https://review.openstack.org/#/c/274885/ is working to fix that | 13:34 |
| markus_z | johnthetubaguy: Regarding review focus, these are the open reviews for high|critical bug reports: http://bit.ly/1L1XlAE | 13:34 |
| jaypipes | bauzas: k, looking at your reviews in a bit. | 13:36 |
| *** mylu has joined #openstack-nova | 13:36 | |
| bauzas | claudiub: just adding a comment for maybe asking to provide an UT | 13:36 |
| markus_z | johnthetubaguy: I generate that query with: https://github.com/markuszoeller/openstack/blob/master/scripts/launchpad/query_high_prio_bug_fixes.py | 13:36 |
| bauzas | claudiub: that said, it can be done in a latter patch, hence me +1ign | 13:37 |
| bauzas | claudiub: so, see, I'm reviewing the test class for the hyper-v live migrations and I don't see any RPC related call | 13:37 |
| *** ndipanov_ is now known as ndipanov | 13:37 | |
| bauzas | claudiub: like for example what we do with test_conductor using RPCAPI | 13:38 |
| *** davideagle has quit IRC | 13:39 | |
| bauzas | claudiub: so, I think we had this bug because we are not checking that | 13:39 |
| *** chhavi has quit IRC | 13:39 | |
| bauzas | hence me wondering how we could test that | 13:39 |
| *** haomaiwang has joined #openstack-nova | 13:40 | |
| *** baoli has quit IRC | 13:40 | |
| *** aviw has quit IRC | 13:40 | |
| *** haomaiwang has quit IRC | 13:40 | |
| *** baoli has joined #openstack-nova | 13:40 | |
| *** haomaiwa_ has joined #openstack-nova | 13:41 | |
| *** ekuris_ has quit IRC | 13:41 | |
| eliqiao | alaski: ah, it's first time I know this. thanks :) | 13:41 |
| *** baoli has quit IRC | 13:43 | |
| *** liverpooler has quit IRC | 13:43 | |
| claudiub | bauzas: so, check_can_live_migrate_destination is called by the source host, right? The call is fine, until the migration_data_object is returned. previously, it was returning a LiveMigrateData object, which is not registered. Because of this, that object couldn't get deserialized at the source host. | 13:43 |
| *** liverpooler has joined #openstack-nova | 13:43 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Support for abort an ongoing live migration https://review.openstack.org/284296 | 13:43 |
| *** baoli has joined #openstack-nova | 13:43 | |
| bauzas | claudiub: I understand the problem, I wonder how to cover it in an UT | 13:43 |
| bauzas | UT = unittest, sorrty | 13:43 |
| bauzas | claudiub: like passing the object to the fake oslo.messaging driver, and see it's okay | 13:44 |
| *** ivase has joined #openstack-nova | 13:46 | |
| claudiub | bauzas: oh, I see now. | 13:46 |
| *** Marga_ has joined #openstack-nova | 13:46 | |
| *** cdelatte has joined #openstack-nova | 13:46 | |
| claudiub | i'll check to see what I can do regarding the UT. | 13:46 |
| *** ivase has quit IRC | 13:48 | |
| *** claudiub|2 has joined #openstack-nova | 13:48 | |
| *** ivase has joined #openstack-nova | 13:48 | |
| johnthetubaguy | markus_z: can we get that in the usual etherpad? maybe its there already? | 13:49 |
| *** Marga_ has quit IRC | 13:50 | |
| markus_z | johnthetubaguy: yeah, right, I'll do that | 13:50 |
| *** claudiub has quit IRC | 13:51 | |
| markus_z | sdague: Do you have a minute? The bug report https://bugs.launchpad.net/nova/+bug/1551660 is API related and looks odd. | 13:51 |
| openstack | Launchpad bug 1551660 in OpenStack Compute (nova) "Server/ServerGroup creation on especified tenant doesn't works as expected" [Undecided,New] | 13:51 |
| sdague | markus_z: sure | 13:52 |
| sdague | markus_z: I think this is a docs bug honestly | 13:53 |
| *** tbachman has joined #openstack-nova | 13:53 | |
| sdague | I feel like I've seen this recently | 13:53 |
| markus_z | sdague: One is not supposed to create a server(group) on a different tenant, right? | 13:53 |
| markus_z | s/tenant/project | 13:53 |
| *** mylu has quit IRC | 13:54 | |
| *** mrkz has joined #openstack-nova | 13:54 | |
| sdague | the person is reading the docs wrong | 13:54 |
| sdague | tenant_id is a url part | 13:54 |
| sdague | but, yes, you can't change it | 13:54 |
| sdague | so we should fix the docs | 13:55 |
| markus_z | Ah, that's what the "style = URI" tries to say? | 13:55 |
| *** rmart04 has joined #openstack-nova | 13:56 | |
| PaulMurray | live migration meeting in a few minutes: PaulMurray paul-carlton2 eliqiao shaohe_feng pkoniszewski jlanoux yuntongjin mdbooth kashyap kmartin claudiub tobasco | 13:57 |
| PaulMurray | on #openstack-meeting-3 | 13:58 |
| *** mylu has joined #openstack-nova | 14:00 | |
| bauzas | claudiub|2: so, honestly, I think it could need some new TestCase for that since we're not checking that already in the hyper-v LiveMigrationOpsTestCase | 14:00 |
| bauzas | claudiub|2: as it could be difficult, I could be fine with merging first the bugfix (since it's an High prio) but leave you thinking about adding a later UT, okay? | 14:00 |
| *** mingyu has quit IRC | 14:01 | |
| *** rlrossit has joined #openstack-nova | 14:01 | |
| *** haomaiwa_ has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Console log copy for block live migration https://review.openstack.org/284674 | 14:01 |
| claudiub|2 | bauzas: sure, will do. currently looking for a way to do a unit test for this. | 14:02 |
| bauzas | claudiub|2: oh, johnthetubaguy already +W'd which is cool, I just added my own +2 while I was adding this above comment | 14:04 |
| *** derekh is now known as ndipanov_ | 14:04 | |
| *** ndipanov_ is now known as derekh | 14:04 | |
| bauzas | claudiub|2: so feel free to take time to see how we could do this, but in a later patch:) | 14:04 |
| johnthetubaguy | bauzas: yeah, a good follow test would be cool, but yeah, it looks good | 14:04 |
| bauzas | johnthetubaguy: yeah, I was just adding a comment when you called Jenkins atm | 14:05 |
| bauzas | johnthetubaguy: saying that given the priority, I'm fine with merging the bugfix and claim for a test later on | 14:05 |
| *** baoli has quit IRC | 14:08 | |
| *** lucas-hungry is now known as lucasagomes | 14:09 | |
| johnthetubaguy | bauzas: ack, +1 | 14:10 |
| *** eglynn has quit IRC | 14:11 | |
| *** mdrabe has joined #openstack-nova | 14:11 | |
| claudiub|2 | johnthetubaguy: bauzas: thanks! | 14:12 |
| *** baoli has joined #openstack-nova | 14:12 | |
| *** roeyc has joined #openstack-nova | 14:13 | |
| *** moshele has quit IRC | 14:14 | |
| *** mylu has quit IRC | 14:14 | |
| *** ociuhandu has quit IRC | 14:15 | |
| *** jamie_h has joined #openstack-nova | 14:15 | |
| *** baoli has quit IRC | 14:15 | |
| *** claudiub has joined #openstack-nova | 14:15 | |
| *** ociuhandu has joined #openstack-nova | 14:16 | |
| *** Daisy has quit IRC | 14:17 | |
| *** claudiub|2 has quit IRC | 14:18 | |
| *** edtubill has joined #openstack-nova | 14:21 | |
| *** ociuhandu has quit IRC | 14:21 | |
| *** baoli has joined #openstack-nova | 14:22 | |
| *** baoli has quit IRC | 14:23 | |
| *** jichen has quit IRC | 14:25 | |
| *** ctrath has joined #openstack-nova | 14:25 | |
| *** Daisy has joined #openstack-nova | 14:25 | |
| *** Daisy has quit IRC | 14:25 | |
| *** baoli has joined #openstack-nova | 14:25 | |
| *** Daisy has joined #openstack-nova | 14:26 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Go away from using PciDeviceStats consume_requests https://review.openstack.org/243131 | 14:26 |
| edleafe | johnthetubaguy: thanks for your review of https://review.openstack.org/284833. I'll address your comments, but I have a ton of other stuff to handle today, so it might not be for a while. | 14:27 |
| *** roeyc has quit IRC | 14:27 | |
| *** ccarmack has joined #openstack-nova | 14:27 | |
| *** aysyd has quit IRC | 14:27 | |
| johnthetubaguy | edleafe: so that is mitaka-3 critical at this point, can we get someone else to work this if you are busy? | 14:28 |
| *** aysyd has joined #openstack-nova | 14:28 | |
| johnthetubaguy | cdent: ^ | 14:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Added supports_requests method to PciDeviceList https://review.openstack.org/242087 | 14:28 |
| johnthetubaguy | edleafe: i.e. I am currently blocking the release on patches above that | 14:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Fix _test_pci() to NOT actually claim device https://review.openstack.org/285865 | 14:29 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Go away from using PciDeviceStats consume_requests https://review.openstack.org/243131 | 14:29 |
| *** jroll has quit IRC | 14:29 | |
| *** jroll has joined #openstack-nova | 14:30 | |
| *** jroll has quit IRC | 14:30 | |
| *** jroll has joined #openstack-nova | 14:30 | |
| cdent | johnthetubaguy, edleafe: I can look at that if that seems like a good idea | 14:30 |
| PaulMurray | shaohe_feng, you were just too late then - is there something specific that's not working for some users? | 14:30 |
| *** sfinucan has joined #openstack-nova | 14:30 | |
| johnthetubaguy | cdent: it might be easier if I take a quick peak at that, thinking about it | 14:30 |
| *** nkrinner has quit IRC | 14:31 | |
| cdent | go for johnthetubaguy | 14:31 |
| *** roeyc has joined #openstack-nova | 14:31 | |
| *** andymaier has quit IRC | 14:31 | |
| *** edmondsw has joined #openstack-nova | 14:31 | |
| *** mpavone has quit IRC | 14:31 | |
| cdent | there seesms to be a lot of understanding of the existing db api required | 14:31 |
| dansmith | jaypipes: so we have a problem | 14:32 |
| dansmith | jaypipes: this never got converted to objects: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3669 | 14:32 |
| dansmith | jaypipes: I think we might be able to make the tests pass if you do your magic to that method, but without it, it starts reporting zero for all the compute node fields we're zero-ing | 14:33 |
| *** eharney has joined #openstack-nova | 14:33 | |
| jaypipes | dansmith: k. wil look at that shortly. right now, cleaning up some pci-stats-generate patches with dstepanenko | 14:33 |
| *** ociuhandu has joined #openstack-nova | 14:34 | |
| *** erno has quit IRC | 14:34 | |
| dansmith | jaypipes: okay | 14:35 |
| *** McNinja has joined #openstack-nova | 14:38 | |
| *** baoli has quit IRC | 14:39 | |
| *** ociuhandu has quit IRC | 14:39 | |
| *** EinstCrazy has joined #openstack-nova | 14:39 | |
| *** mylu has joined #openstack-nova | 14:41 | |
| *** baoli has joined #openstack-nova | 14:41 | |
| *** Daisy has quit IRC | 14:44 | |
| *** cfcp1 has joined #openstack-nova | 14:44 | |
| sdague | there are 2 ceph failures reseting the gate atm | 14:44 |
| jaypipes | dansmith: k, Dmitry is off working on those reviews now. So, I agree with you about that being a problem. Let me have 30 minutes to fix it up in the original patch. Or... we can fix compute_node_statistics() in edleafe's followup patch to mine? preference there? | 14:44 |
| dansmith | jaypipes: I don't much care | 14:45 |
| mtreinish | sdague: more xfail woes? | 14:45 |
| dansmith | jaypipes: just has to be somewhere under mine :) | 14:45 |
| sdague | mtreinish: don't know | 14:45 |
| sdague | http://tinyurl.com/z5qcbch | 14:45 |
| sdague | it's 25% fail | 14:45 |
| *** ric has joined #openstack-nova | 14:46 | |
| bauzas | dansmith: good catch, I remember myself refraining to push my laptop out of my window when trying to fix https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3669 | 14:46 |
| mtreinish | oh, i was talking about your ping before. But, that was for a different thing | 14:47 |
| jaypipes | dansmith: I'll put it in the original in order to not have that "bug" exist in the codebase for any amount of time. | 14:47 |
| *** ociuhandu has joined #openstack-nova | 14:47 | |
| jaypipes | edleafe: FYI ^ | 14:47 |
| dansmith | jaypipes: it doesn't exist until my patch, but sure | 14:47 |
| mriedem | sdague: did you ever have a patch up for this? https://bugs.launchpad.net/nova/+bug/1522454 | 14:48 |
| openstack | Launchpad bug 1522454 in OpenStack Compute (nova) "Nova is leaking libvirt internal ids on some Instance Not Found errors" [Medium,In progress] - Assigned to idzzy (idzzy) | 14:48 |
| jaypipes | dansmith: ah, yes, I suppose you are right about that :) | 14:48 |
| *** xyang1 has joined #openstack-nova | 14:48 | |
| mriedem | mtreinish: sdague: the ceph failures have bugs | 14:48 |
| mriedem | there was a package update in ubuntu in the last week | 14:48 |
| jaypipes | dansmith: then actually probably better to do it in edleafe's patch, since it is ostensibly trying to apply the new code to all the "other" compute_node_get stuff | 14:48 |
| mriedem | jamespage was working on getting a fix backported for ceph | 14:48 |
| jaypipes | edleafe: mind if I push a rev to your patch? | 14:48 |
| sdague | mriedem: ok, so that's kind of destroying our gate & capacity | 14:49 |
| sdague | mriedem: if it's a known issue, we should make that non voting | 14:49 |
| dansmith | jaypipes: sure | 14:49 |
| *** EinstCrazy has quit IRC | 14:50 | |
| bauzas | jaypipes: so, what about https://review.openstack.org/#/c/284484/3/nova/db/sqlalchemy/api.py@756 ? | 14:50 |
| *** EinstCrazy has joined #openstack-nova | 14:50 | |
| mriedem | sdague: this is the bug fwiw https://bugs.launchpad.net/cinder/+bug/1551305 | 14:51 |
| openstack | Launchpad bug 1551305 in Cinder "backup service crashes in ceph job with "pure virtual method called"" [Medium,Confirmed] | 14:51 |
| bauzas | jaypipes: even if the word "reserved" seems the same for two different usages, I think we should add the disk reserved unit | 14:51 |
| sdague | mriedem: ok, so you pushing up a non voting patch? | 14:51 |
| mriedem | i can | 14:51 |
| sdague | thanks | 14:51 |
| *** knikolla has joined #openstack-nova | 14:51 | |
| *** vladikr has quit IRC | 14:52 | |
| *** drankis has joined #openstack-nova | 14:52 | |
| jaypipes | bauzas: agreed. will add. | 14:52 |
| dansmith | sdague: mriedem so what's the outlook on making it voting again? bugs are out of our control, or waiting for new packages or what? | 14:53 |
| bauzas | jaypipes: nice, ping me when you're done with that | 14:53 |
| bauzas | jaypipes: so I could +2 this | 14:53 |
| sdague | dansmith: don't know, but it's 25% fail rate right now | 14:53 |
| sdague | and none of the ceph folks are getting ahead of things like this | 14:53 |
| mriedem | dansmith: well, getting it fixed in 14.04.1 don't look good right now https://bugs.launchpad.net/cinder/+bug/1551305/comments/5 | 14:53 |
| openstack | Launchpad bug 1551305 in Cinder "backup service crashes in ceph job with "pure virtual method called"" [Medium,Confirmed] | 14:53 |
| *** daemontool_ has joined #openstack-nova | 14:53 | |
| mriedem | the backport to firefly was rejected http://tracker.ceph.com/issues/13757 | 14:53 |
| mriedem | b/c firefly is EOL | 14:54 |
| mriedem | and that's what ubuntu 14.04.1 LTS is using | 14:54 |
| dansmith | jbernard: jdurgin: any comments? | 14:54 |
| mriedem | so i guess ubuntu would have to patch the package themselves? | 14:54 |
| mriedem | i'll talk to jamespage in -dev | 14:54 |
| sdague | regardless, a 25% fail rate in this job is just not a thing we can have. | 14:55 |
| *** doug-fish has joined #openstack-nova | 14:55 | |
| dansmith | sdague: we wouldn't be making the main dsvm job nonvoting because of a failure like this, we'd be working the failure | 14:55 |
| mriedem | btw, is it even worthwhile to have the ceph plugin nv job running? it's been failing for weeks | 14:55 |
| dansmith | sdague: and given the like 90% usage of ceph, I don't really think we should turn it off | 14:55 |
| dansmith | just MHO | 14:55 |
| sdague | dansmith: right, and when this job was made voting it was with the promiss the people would maintain it | 14:56 |
| mriedem | dansmith: working the failure in this case is getting ubuntu to patch their librados package | 14:56 |
| sdague | and they are not | 14:56 |
| sdague | if I've got to raise the issue we're at 25% fail, then it's not being maintained | 14:56 |
| mriedem | i'm asking jamespage in -dev if ubuntu will patch their package | 14:56 |
| *** daemontool__ has quit IRC | 14:56 | |
| *** e0ne has quit IRC | 14:56 | |
| *** esracelik has quit IRC | 14:56 | |
| dansmith | sdague: but given that it's 90% of our user base, I feel like it's all of our responsibility | 14:56 |
| sdague | dansmith: well, it clearly isn't | 14:57 |
| dansmith | sdague: haven't we made a lot of "but it's critical for the users" arguments lately? | 14:57 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Use new inventory schema in compute_node_get_all() https://review.openstack.org/284484 | 14:57 |
| *** edmondsw has quit IRC | 14:57 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Use new inventory schema in compute_node_get_all() https://review.openstack.org/284484 | 14:57 |
| jaypipes | bauzas: done. | 14:57 |
| mriedem | btw, there is another ceph related bug hitting the gate http://status.openstack.org/elastic-recheck/index.html#1549854 | 14:57 |
| bauzas | jaypipes: coolness | 14:58 |
| sdague | and right now all it's doing is preventing people from landing unrelated code | 14:58 |
| mriedem | so there are 2, and they both started around 2/24 when that package update was released | 14:58 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Use new inventory schema in all compute_node gets https://review.openstack.org/284833 | 14:58 |
| * tbachman always thinks he sees “huzzah” each time he sees bauzas | 14:58 | |
| *** ducttape_ has joined #openstack-nova | 14:58 | |
| openstackgerrit | Paul Murray proposed openstack/python-novaclient: [microversion] Bump to 2.25 https://review.openstack.org/260383 | 14:59 |
| sdague | we are trying to build a test system that validates if a new patch is good, not a vengence system that breaks people's stuff randomly because the ceph platform we have access to is buggy | 14:59 |
| *** mylu has quit IRC | 14:59 | |
| dansmith | sdague: mriedem: if libvirt had a bug like this, and it was preventing unrelated cinder code from landing, we'd not be disabling the main job. And nearly as many people use ceph as libvirt | 14:59 |
| mriedem | dansmith: live snapshot in libvirt was doing this and we disabled that | 14:59 |
| sdague | I disagree | 14:59 |
| sdague | exactly | 14:59 |
| dansmith | lol | 15:00 |
| mriedem | even though danpb couldn't reproduce the failure outside the gate | 15:00 |
| danpb | what? when ? where ? who ? why ? | 15:00 |
| sdague | it is massively irresponsible to fail everyone elses code for stuff like this | 15:00 |
| dansmith | is there a test we can disable for this or does it affect the whole thing? | 15:00 |
| mriedem | it's random | 15:00 |
| mriedem | cinder backup service crashes randomly, nova scheduling fails with NoValidHost | 15:00 |
| dansmith | cool, then the relationship to the live snapshot bug is invalid :) | 15:00 |
| sdague | and this shouldn't even be a conversation | 15:00 |
| *** haomaiwang has quit IRC | 15:01 | |
| sdague | a test job is above 20% failure | 15:01 |
| sdague | it gets turned off | 15:01 |
| sdague | if people want it back on, then fix it | 15:01 |
| mriedem | ubuntu is patching the package | 15:01 |
| mriedem | jamespage just confirmed in -dev | 15:01 |
| mriedem | so it should be non-voting until that's released | 15:01 |
| dansmith | sdague: cool, next time dsvm is failing over 20% then we'll disable without discussion, right? | 15:01 |
| *** busterswt has joined #openstack-nova | 15:01 | |
| sdague | dansmith: go for it | 15:01 |
| *** claudiub|2 has joined #openstack-nova | 15:01 | |
| *** haomaiwa_ has joined #openstack-nova | 15:01 | |
| *** burt has joined #openstack-nova | 15:01 | |
| *** fawadkhaliq has joined #openstack-nova | 15:02 | |
| *** mylu has joined #openstack-nova | 15:02 | |
| dansmith | mriedem: thanks, if there's a plan, then nonvoting until it happens seems like a reasonable compromise | 15:02 |
| mriedem | yar | 15:03 |
| *** claudiub has quit IRC | 15:04 | |
| *** Daisy has joined #openstack-nova | 15:04 | |
| *** mylu has quit IRC | 15:05 | |
| *** yamahata has joined #openstack-nova | 15:07 | |
| *** sneti has joined #openstack-nova | 15:07 | |
| *** Daisy has quit IRC | 15:09 | |
| * edleafe is back | 15:12 | |
| mriedem | alaski: i just saw this again https://review.openstack.org/#/c/260864/ - the guy didn't seem to care to update it, so i can clean it up unless you want to https://review.openstack.org/#/c/260864/ | 15:12 |
| *** zz_dimtruck is now known as dimtruck | 15:12 | |
| *** EinstCrazy has quit IRC | 15:12 | |
| *** sridharg has quit IRC | 15:13 | |
| alaski | mriedem: I'll take a look and try to clean it up. I agree that it shouldn't be done in compute/api | 15:15 |
| mriedem | ok, it looks like an easy enough fix, so ping me to review it when it's up | 15:15 |
| mriedem | feel free to push a separate change and we can abandon this one | 15:15 |
| *** mpavone has joined #openstack-nova | 15:17 | |
| *** rhagarty has joined #openstack-nova | 15:18 | |
| *** lpetrut has quit IRC | 15:19 | |
| *** rhagarty_ has joined #openstack-nova | 15:21 | |
| *** mylu has joined #openstack-nova | 15:21 | |
| *** lbeliveau_ has joined #openstack-nova | 15:22 | |
| *** pradk has quit IRC | 15:22 | |
| bauzas | jaypipes: still around for a question before I +2 https://review.openstack.org/#/c/284484/5/nova/db/sqlalchemy/api.py ? | 15:22 |
| bauzas | jaypipes: I wonder why you join on rp.uuid for the compute_nodes table while you join against rp.id for the allocations and inventories table | 15:23 |
| bauzas | jaypipes: I can surely review the model, but you could shed me some light here | 15:23 |
| bauzas | jaypipes: oh wait, nvm | 15:24 |
| *** daemontool__ has joined #openstack-nova | 15:24 | |
| *** mylu has quit IRC | 15:24 | |
| *** rhagarty has quit IRC | 15:24 | |
| bauzas | jaypipes: gotcha, the resource_provider uuid is for the resource provider relationship while the id is FK on the others | 15:25 |
| bauzas | I missed that somehow | 15:25 |
| *** rhagarty_ has quit IRC | 15:25 | |
| bauzas | I was thinking that we were FKing on uuid | 15:25 |
| BobBall | johnthetubaguy / others: What's your view on minor code refactors when adding a UT? Do we have to separate them out into two changes? or can we just do the minor refactor in the same change that adds the UT? | 15:26 |
| *** mtanino has joined #openstack-nova | 15:27 | |
| *** lbeliveau has quit IRC | 15:27 | |
| *** haplo37 has joined #openstack-nova | 15:27 | |
| *** jdurgin1 has joined #openstack-nova | 15:28 | |
| *** daemontool_ has quit IRC | 15:28 | |
| *** mpavone has quit IRC | 15:28 | |
| *** sneti_ has joined #openstack-nova | 15:28 | |
| *** sneti has quit IRC | 15:28 | |
| *** gszasz has quit IRC | 15:29 | |
| *** gszasz has joined #openstack-nova | 15:32 | |
| *** mdrabe has quit IRC | 15:34 | |
| edleafe | dansmith: so what's your feeling on keeping nova.db.api.compute_node_get_model? Will it be needed for the objects, or can I delete it? | 15:36 |
| *** mylu has joined #openstack-nova | 15:37 | |
| dansmith | edleafe: I don't think it'll be needed, but I'm not sure it's really worth removing since it's already there, so I'd just leave it | 15:37 |
| edleafe | dansmith: ok, just wanted to ask before adding the unit test for it. | 15:38 |
| dansmith | if the test is hard, then you could drop it.. if not, then maybe just keep it | 15:38 |
| dansmith | I dunno | 15:38 |
| dansmith | can you tell I don't have a strong opinion (for once) ? | 15:38 |
| *** mylu has quit IRC | 15:39 | |
| edleafe | Aha! I must have won some kind of prize for that! | 15:39 |
| dansmith | heh | 15:39 |
| *** ric has quit IRC | 15:39 | |
| edleafe | the test will mock out the sqla backend, so it's really not terribly difficult | 15:40 |
| *** mpavone has joined #openstack-nova | 15:40 | |
| *** andymaier has joined #openstack-nova | 15:40 | |
| edleafe | since it's basically passthrough | 15:40 |
| dansmith | yeah | 15:40 |
| jaypipes | bauzas: right | 15:40 |
| *** mylu has joined #openstack-nova | 15:42 | |
| mriedem | kevinbenton: did you see this? https://review.openstack.org/#/c/271444/ | 15:43 |
| *** Sam-I-Am has joined #openstack-nova | 15:46 | |
| *** mpavone has quit IRC | 15:46 | |
| *** amotoki has quit IRC | 15:46 | |
| Sam-I-Am | danpb: hello | 15:46 |
| danpb | Sam-I-Am: ? | 15:46 |
| Sam-I-Am | danpb: mriedem sent me your way about libvirt questions | 15:47 |
| * danpb curses mriedem ;-P | 15:47 | |
| Sam-I-Am | haha | 15:47 |
| mriedem | danpb: forums and ML threads failed me | 15:48 |
| danpb | Sam-I-Am: anyway, just ask your q | 15:49 |
| Sam-I-Am | danpb: what would cause virsh capabilities to return xml without any <model> ? | 15:49 |
| danpb | you mean without a cpu model ? | 15:49 |
| Sam-I-Am | http://paste.openstack.org/show/488775/ | 15:49 |
| Sam-I-Am | yeah | 15:50 |
| Sam-I-Am | its nonexistant | 15:50 |
| *** jaypipes has quit IRC | 15:50 | |
| Sam-I-Am | in which case cpu_mode/cpu_model in nova.conf dont have any impact | 15:50 |
| *** jlanoux has quit IRC | 15:50 | |
| danpb | Sam-I-Am: hmm, check in the logs from libvirtd - i'd expect to see an error message if it couldn't do that | 15:50 |
| Sam-I-Am | there is one | 15:51 |
| *** yamahata has quit IRC | 15:51 | |
| Sam-I-Am | http://paste.openstack.org/show/488777/ | 15:51 |
| Sam-I-Am | oops, that didnt get the whole thing | 15:52 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds HostCapabilities object model https://review.openstack.org/285856 | 15:52 |
| Sam-I-Am | http://paste.openstack.org/show/488778/ | 15:52 |
| Sam-I-Am | mo better | 15:52 |
| danpb | Sam-I-Am: are you using nested-virt by chance ? | 15:52 |
| Sam-I-Am | this is a softlayer vm | 15:53 |
| danpb | what do you mean by 'softlayer vm' ? | 15:53 |
| Sam-I-Am | deploying openstack on a cloud server somewhere | 15:54 |
| Sam-I-Am | that provider is softlayer | 15:54 |
| Sam-I-Am | usually qemu this just works | 15:54 |
| Sam-I-Am | with qemu, rather | 15:54 |
| Sam-I-Am | since nested kvm is rarely supported in this case | 15:54 |
| danpb | oh right, i see | 15:54 |
| *** gokrokve has joined #openstack-nova | 15:54 | |
| danpb | well i guess the cpu model exposed to the guest OS can't be mapped to any of the pre-defined models that libvirt knows about | 15:55 |
| *** delattec has joined #openstack-nova | 15:55 | |
| *** cdelatte has quit IRC | 15:55 | |
| Sam-I-Am | makes sense | 15:56 |
| danpb | i'm not sure of any easy fix for this really | 15:57 |
| *** mpavone has joined #openstack-nova | 15:57 | |
| Sam-I-Am | i was afraid of that | 15:57 |
| Sam-I-Am | didnt see any obvious way to tell libvirt which cpu to use anyway | 15:57 |
| Sam-I-Am | wonder if SL does this to break vms on their cloud boxes :/ | 15:57 |
| *** tbachman has quit IRC | 15:58 | |
| danpb | if you're using qemu rather than kvm, the choice of cpu model is pretty much irrelevant | 15:58 |
| danpb | as your guest will have crap performance regardless of choice | 15:58 |
| Sam-I-Am | thats also what i was thinking | 15:58 |
| danpb | so you should jsut ignore the cpu_mode/cpu_model config options - leave them unset, or set cpu_mode=none iirc | 15:59 |
| danpb | i think the latter actually | 15:59 |
| Sam-I-Am | i tried that. nova wont start. | 15:59 |
| Sam-I-Am | claims it cant find any <model> in the xml to use | 15:59 |
| danpb | can you show me the full trace from nova | 16:00 |
| *** haomaiwa_ has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-nova | 16:01 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Versioned notification transformation https://review.openstack.org/286675 | 16:02 |
| Sam-I-Am | danpb: http://paste.openstack.org/show/488781/ | 16:03 |
| knikolla | Hey guys, what would be the best way to save a second auth object in the context? | 16:04 |
| *** haplo37 has quit IRC | 16:04 | |
| *** kevinbenton has quit IRC | 16:04 | |
| knikolla | Assuming a second token to a second openstack | 16:04 |
| danpb | Sam-I-Am: oh that's totally bogus and unrelated to the cpu_mode/cpu_model config options | 16:05 |
| danpb | Sam-I-Am: in the host.py get_capabilities() method before it does [self._caps.host.cpu.to_xml()], | 16:05 |
| *** jaosorior is now known as jaosorior_away | 16:06 | |
| danpb | Sam-I-Am: it should check whether self._cps.host.cpu has a non-None model | 16:06 |
| danpb | Sam-I-Am: if the model is None, it should skip the baselineCPU() api call | 16:06 |
| *** vilobhmm11 has joined #openstack-nova | 16:06 | |
| Sam-I-Am | danpb: interesting | 16:06 |
| *** Bjoern__ has joined #openstack-nova | 16:07 | |
| markus_z | johnthetubaguy: I've seen in one of the last meeting that we paused the config options work due to focusing the review effort, which is perfectly reasonable. How do I handle this in the blueprint? | 16:07 |
| *** ric has joined #openstack-nova | 16:07 | |
| *** tbachman has joined #openstack-nova | 16:08 | |
| johnthetubaguy | markus_z: good questions, I am doing that today, going through the blueprints, at least thats been my aim, I will probably just mark that one as partially complete, and we open a new blueprint for next cycle | 16:08 |
| *** Bjoern__ is now known as bjoernt | 16:09 | |
| markus_z | johnthetubaguy: OK, I have no strong feelings here, just tell what fits best for you. | 16:09 |
| danpb | Sam-I-Am: you want to submit a patch to make that change... | 16:10 |
| danpb | or at the very least submit a bug report with that stack trace & capabilities XML included | 16:11 |
| *** shawnmaten has joined #openstack-nova | 16:11 | |
| *** shawnmaten has quit IRC | 16:11 | |
| *** shawnmaten has joined #openstack-nova | 16:11 | |
| mriedem | ccarmack: add the history section to this spec and i'm +2 on it https://review.openstack.org/#/c/286302/ | 16:11 |
| *** efried has joined #openstack-nova | 16:11 | |
| Sam-I-Am | danpb: ok, i can do a bug. in the middle of too many other things. | 16:12 |
| *** gszasz has quit IRC | 16:12 | |
| Sam-I-Am | danpb: thx for helping figure out what it is | 16:12 |
| ccarmack | ok mriedem | 16:12 |
| *** e0ne has joined #openstack-nova | 16:13 | |
| *** jaypipes has joined #openstack-nova | 16:13 | |
| mriedem | Sam-I-Am: danpb: the capabilities are in the paste with the stack trace, so that should be enough. i can also take a crack at a patch if i get time today | 16:14 |
| mriedem | ndipanov: what are your feelings on this one now? https://review.openstack.org/#/c/266095/ | 16:15 |
| *** liverpooler has quit IRC | 16:15 | |
| *** haplo37 has joined #openstack-nova | 16:16 | |
| *** ptm_away has joined #openstack-nova | 16:19 | |
| *** jlanoux has joined #openstack-nova | 16:20 | |
| *** coreywright has quit IRC | 16:20 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove unused libvirt _get_all_block_devices and _get_interfaces https://review.openstack.org/286686 | 16:21 |
| openstackgerrit | Chuck Carmack proposed openstack/nova-specs: Re-propose spec for refresh-quotas-usage https://review.openstack.org/286302 | 16:22 |
| *** PaulMurray has quit IRC | 16:22 | |
| ccarmack | mriedem ^ | 16:22 |
| tdurakov | mdbooth, are you around? | 16:25 |
| *** yamahata has joined #openstack-nova | 16:25 | |
| edleafe | johnthetubaguy: arrrgghhh! I didn't realize you were going to make the changes to https://review.openstack.org/#/c/284833 | 16:27 |
| ndipanov | mriedem, will look at it in a sec | 16:27 |
| edleafe | johnthetubaguy: I had them ready to upload when I noticed. | 16:27 |
| *** mjura has quit IRC | 16:28 | |
| johnthetubaguy | edleafe: sorry, thought I pinged you earlier, it failed to upload first time | 16:28 |
| *** belmoreira has quit IRC | 16:28 | |
| johnthetubaguy | edleafe: its missing the unit test though | 16:28 |
| edleafe | johnthetubaguy: I was on the road | 16:29 |
| edleafe | johnthetubaguy: I see it now in the scrollback | 16:29 |
| johnthetubaguy | edleafe: OK, sorry about that | 16:29 |
| edleafe | johnthetubaguy: I'll see about adding the unit tests to your patch | 16:30 |
| edleafe | johnthetubaguy: unless you're already working on them :) | 16:30 |
| johnthetubaguy | edleafe: cool, no I haven't started those yet | 16:30 |
| *** kevinbenton has joined #openstack-nova | 16:30 | |
| *** armax has joined #openstack-nova | 16:31 | |
| mdbooth | tdurakov: Yup | 16:32 |
| *** READ10 has joined #openstack-nova | 16:32 | |
| tdurakov | mdbooth, left comment on my patch to you. ok to split this patch, but not clear why destroy_disks is redundant | 16:33 |
| tdurakov | mdbooth, could you clarify | 16:33 |
| mdbooth | tdurakov: Sorry, thought it was obvious :) | 16:33 |
| mdbooth | Look at the block immediately above. | 16:33 |
| *** fawadkhaliq has quit IRC | 16:33 | |
| tdurakov | mdbooth, ok, then will split it only | 16:34 |
| mdbooth | You can literally just delete that line. | 16:34 |
| mdbooth | It's only a nit, tbh. | 16:34 |
| *** ptm_away is now known as PaulMurray | 16:34 | |
| *** rhagarty has joined #openstack-nova | 16:34 | |
| tdurakov | mdbooth, which line?:) | 16:34 |
| mdbooth | 'if destroy_disks' | 16:35 |
| *** roeyc has quit IRC | 16:35 | |
| mdbooth | You've refactored a bunch of code, and the resulting guard is identical to the guard on the block immediately above | 16:35 |
| *** mylu has quit IRC | 16:35 | |
| mdbooth | You've got: | 16:35 |
| mdbooth | if destroy_disks: | 16:35 |
| mdbooth | ... | 16:35 |
| mdbooth | if destroy_disks: | 16:35 |
| mdbooth | more ... | 16:35 |
| claudiub|2 | jaypipes: hello. If you have one free second, could you take a look at the HostCapabilities object? I'd like some suggestions / feedback on it. :D https://review.openstack.org/#/c/285856/ | 16:36 |
| mdbooth | Might as well just delete the redundant one. | 16:36 |
| tdurakov | mdbooth, got it | 16:36 |
| tdurakov | thanks! | 16:36 |
| mdbooth | tdurakov: Nothing substantive :) | 16:36 |
| *** yamahata has quit IRC | 16:36 | |
| *** paul-carlton2 has quit IRC | 16:37 | |
| *** ric has quit IRC | 16:38 | |
| *** e0ne has quit IRC | 16:38 | |
| *** eharney has quit IRC | 16:39 | |
| *** coreywright has joined #openstack-nova | 16:39 | |
| *** apoorvad has joined #openstack-nova | 16:39 | |
| *** e0ne has joined #openstack-nova | 16:40 | |
| *** mgoddard_ has quit IRC | 16:40 | |
| mriedem | ndipanov: i -1'ed just b/c of the confusing logic in the compute manager, feel free to weigh in | 16:40 |
| *** mgoddard has joined #openstack-nova | 16:40 | |
| *** eharney has joined #openstack-nova | 16:41 | |
| mriedem | ccarmack: looks good, i'll score after the docs job is done | 16:41 |
| ccarmack | thanks | 16:42 |
| *** josecastroleon has quit IRC | 16:43 | |
| *** mpavone has quit IRC | 16:45 | |
| mriedem | tdurakov: are you still using this? https://review.openstack.org/#/c/272950/ | 16:45 |
| ndipanov | mriedem, I don't like that patch tbh | 16:46 |
| mriedem | tell me more | 16:47 |
| ndipanov | but I guess we should merge it since it fixes a bug | 16:47 |
| *** thorst_ is now known as thorst_afk | 16:47 | |
| ndipanov | but I really don't like it | 16:47 |
| mriedem | ndipanov: i'm not crazy about it either | 16:48 |
| mriedem | do you have something specific that bothers you? | 16:48 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add Allocation and AllocationList objects https://review.openstack.org/282442 | 16:48 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP Add ResourcePool object https://review.openstack.org/284963 | 16:48 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a get_by_uuid for aggregates https://review.openstack.org/284895 | 16:48 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add name to ResourceProvider object https://review.openstack.org/281945 | 16:48 |
| *** scheuran has quit IRC | 16:49 | |
| mriedem | ndipanov: ideally cinder would store the connector info with the volume attachment in their db | 16:49 |
| tdurakov | mriedem, no, tested locally. | 16:49 |
| mriedem | tdurakov: ok i'm going to abandon the tempest and devstack changes | 16:49 |
| tdurakov | mriedem, what about marking tempest tests for l-m as smoke and trying to merge your changes instead? | 16:50 |
| mriedem | tdurakov: it's probably an issue of stability | 16:51 |
| *** rk4n has quit IRC | 16:51 | |
| mriedem | tdurakov: as in the gate-tempest-dsvm-multinode-full job is failing and non-voting right now | 16:51 |
| *** sneti_ has quit IRC | 16:51 | |
| mriedem | presumably b/c of live migration test failures | 16:51 |
| *** gokrokve has quit IRC | 16:52 | |
| mriedem | like this one http://status.openstack.org/elastic-recheck/index.html#1539271 | 16:52 |
| *** vilobhmm111 has joined #openstack-nova | 16:52 | |
| *** sneti has joined #openstack-nova | 16:53 | |
| *** toabctl has quit IRC | 16:53 | |
| *** josecastroleon has joined #openstack-nova | 16:53 | |
| ndipanov | mriedem, commented | 16:54 |
| mriedem | ccarmack: you have to fix somethign with the history section in your spec | 16:55 |
| *** rcernin has quit IRC | 16:55 | |
| mriedem | ndipanov: thanks | 16:55 |
| *** vilobhmm11 has quit IRC | 16:55 | |
| PaulMurray | eliqiao, those two patches keep failing on test_trigger_crash_dump_in_resized_state | 16:55 |
| *** fawadkhaliq has joined #openstack-nova | 16:56 | |
| *** toabctl has joined #openstack-nova | 16:57 | |
| *** mylu has joined #openstack-nova | 16:57 | |
| ccarmack | mriedem: looking | 16:57 |
| *** tbachman has quit IRC | 16:57 | |
| *** mgoddard_ has joined #openstack-nova | 16:58 | |
| mriedem | tdurakov: http://tinyurl.com/gvt5h56 | 16:59 |
| mriedem | the multinode job that runs live migration tests is at 75% failure rate | 16:59 |
| mriedem | sdague: ^ | 16:59 |
| tdurakov | mriedem, acked | 16:59 |
| mriedem | looks like it's test_volume_backed_live_migration that's failing | 17:00 |
| sdague | mriedem: right, I think that hasn't been fully sorted yet | 17:00 |
| johnthetubaguy | is tdurakov is around about that ^ | 17:00 |
| *** haomaiwang has quit IRC | 17:01 | |
| mriedem | http://logs.openstack.org/20/283820/2/check/gate-tempest-dsvm-multinode-full/5427c27/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-02-25_15_38_04_593 | 17:01 |
| openstackgerrit | Chuck Carmack proposed openstack/nova-specs: Re-propose spec for refresh-quotas-usage https://review.openstack.org/286302 | 17:01 |
| tdurakov | johnthetubaguy, m? | 17:01 |
| *** haomaiwa_ has joined #openstack-nova | 17:01 | |
| *** mgoddard has quit IRC | 17:01 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Use new inventory schema in all compute_node gets https://review.openstack.org/284833 | 17:01 |
| *** nic has joined #openstack-nova | 17:02 | |
| edleafe | johnthetubaguy: ^^ tests added, code fixed | 17:02 |
| *** matrohon has quit IRC | 17:02 | |
| johnthetubaguy | edleafe: sweet, looking at that in a sec, thank you! | 17:02 |
| *** ptm_away has joined #openstack-nova | 17:03 | |
| mriedem | tdurakov: here is the volume-backed live migration failure http://logs.openstack.org/20/283820/2/check/gate-tempest-dsvm-multinode-full/5427c27/logs/subnode-2/screen-n-cpu.txt.gz?level=TRACE#_2016-02-25_15_37_36_052 | 17:04 |
| jaypipes | claudiub|2: you got it. | 17:04 |
| claudiub|2 | jaypipes: cool, thanks. :) | 17:04 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Aggregate object fixups https://review.openstack.org/268712 | 17:05 |
| *** jistr has quit IRC | 17:05 | |
| tdurakov | mriedem, is it the same? https://bugs.launchpad.net/nova/+bug/1535232 | 17:05 |
| openstack | Launchpad bug 1535232 in OpenStack Compute (nova) "live-migration ci failure on nfs shared storage" [Medium,Confirmed] | 17:05 |
| mriedem | tdurakov: hmm, the live migration data object has is_shared_block_storage=True | 17:05 |
| mriedem | is that true here? i didn't think this was shared storage | 17:05 |
| mriedem | tdurakov: no not the same as that bug which is using block storage, this isn't using block storage | 17:06 |
| mriedem | should be using an attached volume | 17:06 |
| *** PaulMurray has quit IRC | 17:06 | |
| *** mylu has quit IRC | 17:06 | |
| *** pcaruana has quit IRC | 17:07 | |
| mriedem | oh right, this is the random abort volume-backed live migration bug that dansmith hit | 17:07 |
| mriedem | http://logs.openstack.org/20/283820/2/check/gate-tempest-dsvm-multinode-full/5427c27/logs/subnode-2/screen-n-cpu.txt.gz#_2016-02-25_15_38_04_615 | 17:07 |
| *** lpetrut has joined #openstack-nova | 17:07 | |
| mriedem | which is https://bugs.launchpad.net/nova/+bug/1539271 | 17:08 |
| openstack | Launchpad bug 1539271 in OpenStack Compute (nova) "Libvirt live migration stalls" [High,Confirmed] | 17:08 |
| *** ptm_away is now known as PaulMurray | 17:08 | |
| sdague | mriedem: that's the full on stall ? | 17:08 |
| sdague | that only happens on volume backed migrations? | 17:08 |
| mriedem | actually, since we never posted the links to dan's failure, i don't know | 17:08 |
| sdague | mriedem: we're tracking it in es | 17:09 |
| sdague | http://status.openstack.org//elastic-recheck/#1539271 | 17:09 |
| sdague | so you could figure out | 17:09 |
| mriedem | yeah | 17:09 |
| mriedem | this might actually be another one that someone else hit | 17:09 |
| sdague | could be | 17:10 |
| mriedem | ah it's this https://bugs.launchpad.net/nova/+bug/1524898 | 17:10 |
| openstack | Launchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,Confirmed] | 17:10 |
| mriedem | that's the one i was looking for | 17:10 |
| *** Swami has joined #openstack-nova | 17:10 | |
| *** achanda has joined #openstack-nova | 17:10 | |
| *** ptm_away has joined #openstack-nova | 17:11 | |
| jdurgin1 | mriedem: re: ceph gate issues, jamespage confirms he's backporting the fix for the precise packages | 17:11 |
| mriedem | jdurgin1: we need it on 14.04.1 trusty | 17:12 |
| *** vilobhmm111 has quit IRC | 17:12 | |
| jdurgin1 | mriedem: my mistake, it is trusty | 17:13 |
| *** shawnmaten has quit IRC | 17:14 | |
| mriedem | sdague: confirmed that the stall one is block migration | 17:14 |
| mriedem | the abort one is volume-backed | 17:14 |
| sdague | mriedem: the stall one is only block migration? | 17:14 |
| *** PaulMurray has quit IRC | 17:15 | |
| sdague | the stall one probably still needs a libvirt bug report on it, iirc danpb thought it was something in libvirt locking up | 17:15 |
| *** mylu has joined #openstack-nova | 17:15 | |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add manage command for cell0. https://review.openstack.org/275266 | 17:17 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Map instances to cell0 on failed schedule https://review.openstack.org/267827 | 17:17 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Manage db sync command for cell0 https://review.openstack.org/274885 | 17:17 |
| *** dims_ has joined #openstack-nova | 17:17 | |
| *** hemna has quit IRC | 17:18 | |
| *** dims has quit IRC | 17:19 | |
| *** salv-orl_ has joined #openstack-nova | 17:19 | |
| *** hemna has joined #openstack-nova | 17:19 | |
| mriedem | sdague: from my random sampling of 2 failures it's block storage :) | 17:19 |
| *** salv-orlando has quit IRC | 17:22 | |
| sdague | dims_: why did you add nova to this - https://bugs.launchpad.net/nova/+bug/1510234 | 17:23 |
| openstack | Launchpad bug 1510234 in oslo.service "Heartbeats stop when time is changed" [Undecided,Confirmed] | 17:23 |
| *** josecastroleon has quit IRC | 17:23 | |
| *** jdurgin1 has quit IRC | 17:24 | |
| *** ric has joined #openstack-nova | 17:24 | |
| *** shawnmaten has joined #openstack-nova | 17:25 | |
| *** erlon has joined #openstack-nova | 17:27 | |
| *** mylu has quit IRC | 17:27 | |
| *** paul-carlton2 has joined #openstack-nova | 17:29 | |
| *** rk4n has joined #openstack-nova | 17:30 | |
| *** sfinucan has quit IRC | 17:34 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: DNM: see where instance.pci_devices is getting lazy-loaded https://review.openstack.org/284839 | 17:35 |
| *** thorst_afk has quit IRC | 17:35 | |
| *** thorst_afk has joined #openstack-nova | 17:36 | |
| *** sneti_ has joined #openstack-nova | 17:36 | |
| *** thorst_afk is now known as thorst_ | 17:36 | |
| openstackgerrit | Sean Dague proposed openstack/nova: libvirt: Add ppc to supported arch for NUMA https://review.openstack.org/245765 | 17:38 |
| *** sneti has quit IRC | 17:39 | |
| *** ptm__ has joined #openstack-nova | 17:40 | |
| *** jlanoux has quit IRC | 17:41 | |
| *** vilobhmm11 has joined #openstack-nova | 17:42 | |
| *** ptm__ has quit IRC | 17:42 | |
| *** ihrachys has quit IRC | 17:42 | |
| *** Jeffrey4l has quit IRC | 17:43 | |
| *** ihrachys has joined #openstack-nova | 17:43 | |
| *** derekh has quit IRC | 17:43 | |
| *** PaulMurray has joined #openstack-nova | 17:43 | |
| *** ptm_away has quit IRC | 17:44 | |
| *** harlowja_at_home has joined #openstack-nova | 17:45 | |
| *** shawnmaten has quit IRC | 17:45 | |
| *** tbachman has joined #openstack-nova | 17:45 | |
| *** ihrachys has quit IRC | 17:47 | |
| *** Bjoern_ has joined #openstack-nova | 17:48 | |
| *** claudiub|2 has quit IRC | 17:49 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 17:49 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: libvirt: Refactor _update_xml https://review.openstack.org/286742 | 17:49 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: trivial: remove unused argument from a method https://review.openstack.org/286743 | 17:49 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML https://review.openstack.org/286744 | 17:49 |
| *** bjoernt has quit IRC | 17:50 | |
| *** rk4n has quit IRC | 17:50 | |
| *** vilobhmm111 has joined #openstack-nova | 17:50 | |
| *** rk4n has joined #openstack-nova | 17:50 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Cleanup for post/rollback of live-migrations https://review.openstack.org/280653 | 17:51 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: New field for LibvirtLiveMigrateData https://review.openstack.org/286745 | 17:51 |
| *** vilobhmm11 has quit IRC | 17:52 | |
| *** vilobhmm112 has joined #openstack-nova | 17:52 | |
| *** sneti_ has quit IRC | 17:53 | |
| *** Bjoern_ has quit IRC | 17:53 | |
| *** vilobhmm111 has quit IRC | 17:54 | |
| openstackgerrit | sean mooney proposed openstack/nova: stop setting mtu when pluging vhostuser port https://review.openstack.org/271444 | 17:54 |
| *** ptm_away has joined #openstack-nova | 17:58 | |
| *** PaulMurray has quit IRC | 17:58 | |
| *** baoli has quit IRC | 17:58 | |
| *** krtaylor has quit IRC | 18:00 | |
| *** sc68cal has joined #openstack-nova | 18:00 | |
| *** haomaiwa_ has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-nova | 18:01 | |
| *** efried has quit IRC | 18:02 | |
| *** ptm_away is now known as PaulMurray | 18:04 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: nova-manage: Declare a PciDevice online migration script https://review.openstack.org/278079 | 18:04 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Release notes: online_data_migrations nova-manage command https://review.openstack.org/281803 | 18:04 |
| *** BobBall is now known as BobBall_AWOL | 18:04 | |
| *** penick has joined #openstack-nova | 18:05 | |
| *** rdopiera has quit IRC | 18:05 | |
| *** mriedem has quit IRC | 18:05 | |
| *** ndipanov has quit IRC | 18:05 | |
| *** mriedem has joined #openstack-nova | 18:06 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/nova: Implement setup_networks_on_host for Neutron networks https://review.openstack.org/275073 | 18:06 |
| *** busterswt has quit IRC | 18:07 | |
| *** baoli has joined #openstack-nova | 18:07 | |
| *** busterswt has joined #openstack-nova | 18:07 | |
| *** sudipto has quit IRC | 18:08 | |
| *** e0ne has quit IRC | 18:08 | |
| openstackgerrit | Dmitry Stepanenko proposed openstack/nova: Move logic of pci consume_requests to manager https://review.openstack.org/243131 | 18:12 |
| *** vilobhmm112 has quit IRC | 18:12 | |
| *** jed56 has quit IRC | 18:13 | |
| *** baoli has quit IRC | 18:14 | |
| sdague | ok, the nova.hooks deprecation has a jenkins +1 now - https://review.openstack.org/#/c/286276/ | 18:14 |
| *** rmart04 has quit IRC | 18:15 | |
| *** tongli has joined #openstack-nova | 18:15 | |
| *** suresh12 has joined #openstack-nova | 18:17 | |
| *** openstackgerrit has quit IRC | 18:18 | |
| *** openstackgerrit has joined #openstack-nova | 18:18 | |
| Sam-I-Am | mriedem: hey. not yet, been slammed. | 18:22 |
| mriedem | Sam-I-Am: i can do it, i still have the paste | 18:22 |
| *** lucasagomes is now known as lucas-dinner | 18:22 | |
| *** whenry_ has joined #openstack-nova | 18:25 | |
| Sam-I-Am | mriedem: that would be amazing | 18:25 |
| *** jichen has joined #openstack-nova | 18:25 | |
| *** jschlueter is now known as jschlueter|food | 18:26 | |
| *** serverascode has quit IRC | 18:26 | |
| *** ccarmack has quit IRC | 18:27 | |
| *** zhiyan has quit IRC | 18:27 | |
| *** xiexs has quit IRC | 18:27 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 18:28 |
| *** zhiyan has joined #openstack-nova | 18:28 | |
| *** serverascode has joined #openstack-nova | 18:29 | |
| *** permalac has quit IRC | 18:31 | |
| *** jaosorior_away is now known as jaosorior | 18:31 | |
| *** permalac has joined #openstack-nova | 18:34 | |
| *** permalac has quit IRC | 18:34 | |
| *** paul-carlton2 has quit IRC | 18:34 | |
| *** cfcp1 has quit IRC | 18:35 | |
| *** moshele has joined #openstack-nova | 18:35 | |
| *** danpb has quit IRC | 18:38 | |
| *** rlrossit has quit IRC | 18:39 | |
| openstackgerrit | Thomas Maddox proposed openstack/nova: [WIP - DO NOT MERGE] Testing LXC Gate https://review.openstack.org/274792 | 18:39 |
| *** ccarmack has joined #openstack-nova | 18:41 | |
| *** zenoway has quit IRC | 18:42 | |
| mriedem | Sam-I-Am: this was a master or liberty vm? | 18:42 |
| *** ptm_away has joined #openstack-nova | 18:42 | |
| *** krtaylor has joined #openstack-nova | 18:42 | |
| *** ijw has joined #openstack-nova | 18:42 | |
| *** BobBall_1WOL has joined #openstack-nova | 18:44 | |
| *** lpetrut has quit IRC | 18:44 | |
| *** dims has joined #openstack-nova | 18:46 | |
| *** PaulMurray has quit IRC | 18:46 | |
| *** haplo37 has quit IRC | 18:46 | |
| *** dmellado has quit IRC | 18:48 | |
| Sam-I-Am | mriedem: master | 18:48 |
| *** BobBall_AWOL has quit IRC | 18:50 | |
| *** tbachman_ has joined #openstack-nova | 18:50 | |
| mriedem | ok, i'll re-use https://bugs.launchpad.net/nova/+bug/1447342 | 18:50 |
| *** wolsen has quit IRC | 18:50 | |
| openstack | Launchpad bug 1447342 in OpenStack Compute (nova) "libvirtError: XML error: Missing CPU model name lead to compute service fail to start" [High,Confirmed] | 18:50 |
| *** Yingxin has quit IRC | 18:52 | |
| *** kashyap has quit IRC | 18:52 | |
| *** tbachman has quit IRC | 18:52 | |
| *** dims_ has quit IRC | 18:52 | |
| *** ducttape_ has quit IRC | 18:52 | |
| *** tbachman_ is now known as tbachman | 18:52 | |
| *** knikolla has quit IRC | 18:52 | |
| *** kashyap has joined #openstack-nova | 18:52 | |
| *** ociuhandu has quit IRC | 18:52 | |
| *** Yingxin has joined #openstack-nova | 18:52 | |
| *** acabot has quit IRC | 18:52 | |
| *** afazekas has quit IRC | 18:52 | |
| *** john5223 has quit IRC | 18:52 | |
| *** SpamapS has quit IRC | 18:54 | |
| *** openstack has joined #openstack-nova | 19:16 | |
| *** RuiChen has quit IRC | 19:16 | |
| mriedem | dansmith: ^ interested in your thoughts on adding more serialzed dicts to bdm.connection_info | 19:16 |
| mriedem | vs adding a new column for that | 19:16 |
| *** SpamapS has joined #openstack-nova | 19:16 | |
| *** READ10 has quit IRC | 19:16 | |
| hemna | so that will eventually save the connector into the BDM ? | 19:16 |
| mriedem | yes, it would | 19:16 |
| mriedem | it's a good thing that bdm.connection_info is 2^24 characters in the db | 19:17 |
| hemna | probably large enough | 19:17 |
| mriedem | heh | 19:17 |
| *** openstackstatus has joined #openstack-nova | 19:17 | |
| *** ChanServ sets mode: +v openstackstatus | 19:17 | |
| mriedem | i'm not sure rpc can even pass something that big | 19:17 |
| hemna | so I think that works for evacuate | 19:17 |
| hemna | but doesn't fix force-detach | 19:17 |
| hemna | when nova doesn't know anything about the attachment or the instance | 19:17 |
| mriedem | hemna: sure, it fixes evacuate, but the bigger point is nova isn't crazy about stashing this information | 19:18 |
| hemna | maybe we don't care about solving both of them right away. | 19:18 |
| mriedem | and i was wondering if it would be better stored in the cinder volume attachment table | 19:18 |
| edleafe | dansmith: what revision? | 19:18 |
| hemna | mriedem, that's what I wanted | 19:18 |
| dansmith | edleafe: adding compute node stats to it | 19:18 |
| hemna | to store the connector into the volume attachment table as a new DB column for each attachment. | 19:18 |
| hemna | the only problem with that | 19:18 |
| mriedem | hemna: i guess we could also migrate that data if/when the cinder api is available | 19:18 |
| hemna | is updating it, during live migration. | 19:18 |
| edleafe | dansmith: do you know if he's already started on it? | 19:19 |
| dansmith | edleafe: I thought so yeah | 19:19 |
| hemna | mriedem, jgriffith has a WIP patch in cinder that tries to add the connector at initialize_connection time, and it would probably work, but only for single attachments. multi-attach would break. | 19:19 |
| hemna | fwiw | 19:19 |
| *** openstackgerrit has joined #openstack-nova | 19:19 | |
| dansmith | mriedem: I hate it, as you'd exoect | 19:20 |
| dansmith | *expect | 19:20 |
| edleafe | dansmith: ok. I nearly stepped on johnthetubaguy's toes earlier when I was offline for a little bit | 19:20 |
| mriedem | dansmith: but you like json blobs :) | 19:20 |
| *** zenoway has joined #openstack-nova | 19:20 | |
| dansmith | mriedem: not unversioned ones | 19:20 |
| mriedem | oh right | 19:20 |
| mriedem | which reminds me that i have an os-brick patch to make that connector dict a versioned object | 19:21 |
| mriedem | but won't revisit it until newton probably | 19:21 |
| hemna | mriedem, yah, I have some other work to refactor the way we build the connector as well. it's a WIP in gerrit | 19:21 |
| mriedem | yeah, i haven't read through that one yet | 19:22 |
| hemna | mriedem, https://review.openstack.org/#/c/275943/ | 19:22 |
| mriedem | it's building up in my gerrit email folder | 19:22 |
| hemna | that guy | 19:22 |
| jgriffith | hemna: actually it shouldn't break multi-attach | 19:22 |
| hemna | I haven't touched it in a bit | 19:22 |
| hemna | anyway, i'd like to get some alignment on the connector changes at some point after M | 19:23 |
| jgriffith | hemna: I just don't know how I'm going to get all the unit tests fixed | 19:23 |
| mriedem | one by one :) | 19:23 |
| *** ptm__ has quit IRC | 19:23 | |
| jgriffith | mriedem: :( | 19:23 |
| jgriffith | mriedem: sadly yes... and perhaps rewrite one by one | 19:23 |
| *** mylu has joined #openstack-nova | 19:23 | |
| *** dedery has joined #openstack-nova | 19:24 | |
| hemna | heh that's always the worst of it....making unit tests work. | 19:25 |
| hemna | :( | 19:25 |
| *** markmcclain has quit IRC | 19:25 | |
| *** Savemech has quit IRC | 19:25 | |
| *** briancurtin has quit IRC | 19:25 | |
| *** sulo has quit IRC | 19:25 | |
| *** StevenK has quit IRC | 19:25 | |
| *** alaski has quit IRC | 19:25 | |
| *** rushiagr has quit IRC | 19:25 | |
| *** bauzas has quit IRC | 19:25 | |
| *** johnthetubaguy has quit IRC | 19:25 | |
| openstackgerrit | jichenjc proposed openstack/python-novaclient: move migrations from extension https://review.openstack.org/285213 | 19:26 |
| *** sc68cal has quit IRC | 19:27 | |
| *** lpetrut has quit IRC | 19:30 | |
| *** PaulMurray has joined #openstack-nova | 19:31 | |
| *** markmcclain has joined #openstack-nova | 19:31 | |
| *** Savemech has joined #openstack-nova | 19:31 | |
| *** briancurtin has joined #openstack-nova | 19:31 | |
| *** sulo has joined #openstack-nova | 19:31 | |
| *** StevenK has joined #openstack-nova | 19:31 | |
| *** alaski has joined #openstack-nova | 19:31 | |
| *** rushiagr has joined #openstack-nova | 19:31 | |
| *** bauzas has joined #openstack-nova | 19:31 | |
| *** johnthetubaguy has joined #openstack-nova | 19:31 | |
| *** Savemech has quit IRC | 19:31 | |
| *** efried has joined #openstack-nova | 19:33 | |
| *** geekinutah has joined #openstack-nova | 19:35 | |
| *** ediardo has joined #openstack-nova | 19:37 | |
| *** Savemech has joined #openstack-nova | 19:37 | |
| *** tbachman has joined #openstack-nova | 19:40 | |
| *** Bulacco has joined #openstack-nova | 19:40 | |
| Bulacco | hi, i'm a new developer, or i hope to be, i need only one simple info at ones. For a toy accademic project, I need to modify openstack api allowing nova to retrive a new custom parameter. I need that parameter in the scheduler. My question is what is the class that i need to modify to reach the goal? | 19:41 |
| *** efried has quit IRC | 19:41 | |
| sdague | doffm: though think through an encoding change in terms of upgrade, and you get a little dizzy | 19:41 |
| *** ediardo has left #openstack-nova | 19:42 | |
| mriedem | dansmith: wanna see something fun? | 19:42 |
| cdent | sdague, doffm: how many actual database rows are we talking about, in real life? | 19:42 |
| mriedem | http://logs.openstack.org/59/284959/5/gate/gate-nova-python27/8ab5da7/console.html#_2016-03-01_14_54_26_207 | 19:42 |
| mriedem | https://github.com/openstack/nova/blob/9ab6840af64d6af58baf51040afe86a3324221e5/nova/tests/unit/virt/test_virt_drivers.py#L659 | 19:42 |
| *** dedery has quit IRC | 19:43 | |
| mriedem | the NotImplementedError that objects can raise for attributes not being there can be swallowed up in our tests | 19:43 |
| *** Sukhdev has joined #openstack-nova | 19:43 | |
| *** ctrath has quit IRC | 19:43 | |
| *** delattec has quit IRC | 19:44 | |
| openstackgerrit | Mark Doffman proposed openstack/nova: Map instances to cell0 on failed schedule https://review.openstack.org/267827 | 19:44 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Manage db sync command for cell0 https://review.openstack.org/274885 | 19:44 |
| doffm | sdague: Well this would be when moving tables to the api database. So we would have a chance to check encoding when moving the data. Not sure what we would do if we found an encoding error though. Probably not worth it. | 19:46 |
| *** moshele has joined #openstack-nova | 19:46 | |
| *** dedery has joined #openstack-nova | 19:47 | |
| sdague | cdent: if you touch instances ... potentially a lot | 19:47 |
| *** delattec has joined #openstack-nova | 19:47 | |
| doffm | cdent: Not sure, haven't checked all the tables to which this might apply. | 19:47 |
| sdague | we used to have data migrations that would take hours on rax dataset, hence why turbo hipster exists | 19:47 |
| Bulacco | i don't know if i'm in the correct channel | 19:48 |
| doffm | cdent: in the aggregatehost case the number of unique aggregate/host commbinations. | 19:48 |
| sdague | mriedem: does https://review.openstack.org/#/c/286776/1 expose anywhere else | 19:49 |
| jaypipes | dansmith: doh, shit, sorry... coming soon. | 19:49 |
| *** ctrath1 has joined #openstack-nova | 19:49 | |
| mriedem | sdague: i can check, the libvirt driver tests were mocking that out in other cases, i'll run it through after fixing something else quick | 19:50 |
| *** ctrath1 has quit IRC | 19:51 | |
| knikolla | Bulacco: what api call are you trying to modify? | 19:52 |
| *** sc68cal has joined #openstack-nova | 19:53 | |
| mriedem | alaski: i get to use the SpawnIsSynchronous in another place that was silently failing | 19:54 |
| alaski | \o/ | 19:54 |
| mriedem | good thing yo ufixed it | 19:54 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Fix _test_pci() to NOT actually claim device https://review.openstack.org/285865 | 19:54 |
| *** efried has joined #openstack-nova | 19:55 | |
| Bulacco | knikolla: i need to modify nova boot adding a for example --newparam [integer] | 19:55 |
| *** dedery has quit IRC | 19:56 | |
| sdague | mriedem: how did you trip over these issues? Going through the stack traces on success? | 19:57 |
| *** shaohe_feng_ has quit IRC | 19:57 | |
| mriedem | sdague: our py26 jobs in internal juno failed :) | 19:57 |
| mriedem | b/c that /etc/machine-id file was empty | 19:58 |
| sdague | heh | 19:58 |
| *** josecastroleon has joined #openstack-nova | 19:58 | |
| mriedem | but i'm just finding a string of issues from nova.tests.virt.test_virt_drivers | 19:58 |
| mriedem | like https://bugs.launchpad.net/nova/+bug/1551925 | 19:58 |
| openstack | Launchpad bug 1551925 in OpenStack Compute (nova) "NotImplementedError: Cannot load 'serial_listen_addr' in the base class" [Medium,Triaged] - Assigned to Matt Riedemann (mriedem) | 19:58 |
| knikolla | Bulacco: The json object passed from novaclient to the API is this one https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/servers.py#L527 | 19:58 |
| *** david-lyle has quit IRC | 19:59 | |
| knikolla | Bulacco: If you add a new attribute there, you also need to add it to the API schema of Nova | 19:59 |
| *** moshele has quit IRC | 19:59 | |
| dansmith | mriedem: ah, interesting | 19:59 |
| mriedem | anyway, i have a fix | 19:59 |
| *** ctrath has joined #openstack-nova | 19:59 | |
| mriedem | plus cleaning up some of the turd that is test_virt_drivers | 20:00 |
| mriedem | which is uber biased to libvirt btw :) | 20:00 |
| knikolla | Bulacco: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py | 20:00 |
| *** haomaiwang has quit IRC | 20:01 | |
| *** zengine has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-nova | 20:01 | |
| knikolla | Bulacco: That body you change corresponds to the body here https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L536 | 20:02 |
| knikolla | Bulacco: Did that help? | 20:04 |
| Bulacco | knikolla: ok perfect, thanks. After i added the parameter in the schema and in the python-client ( and i think also in the parser) i need to have this parameter at that level: https://github.com/openstack/nova/blob/master/nova/scheduler/chance.py . How can i retrieve it? | 20:04 |
| *** jdillaman has quit IRC | 20:04 | |
| Bulacco | knikolla: help a lot! :) | 20:04 |
| knikolla | Bulacco: You need to add it to the context object and pass it through rpc in rpcapi.py | 20:05 |
| *** daemontool__ has joined #openstack-nova | 20:05 | |
| knikolla | Bulacco: Then you can get it through the context object. | 20:05 |
| *** ijw has quit IRC | 20:07 | |
| Bulacco | knikolla: ok perfect. Tomorrow i will implement this API modifing the file you say! You are very polite!!! thank you so much! | 20:08 |
| *** ric has quit IRC | 20:08 | |
| *** daemontool_ has quit IRC | 20:09 | |
| *** david-lyle has joined #openstack-nova | 20:09 | |
| *** Bulacco has quit IRC | 20:09 | |
| knikolla | Bulacco: You're welcome. Give it a try! Good luck. | 20:09 |
| *** Bulacco has joined #openstack-nova | 20:10 | |
| *** gokrokve has joined #openstack-nova | 20:12 | |
| *** jaosorior has quit IRC | 20:12 | |
| *** Bulacco has left #openstack-nova | 20:13 | |
| jaypipes | omg, who wrote the test_compute_node_statistics() tests? :( http://paste.openstack.org/show/488837/ | 20:14 |
| *** vilobhmm111 has joined #openstack-nova | 20:14 | |
| mriedem | looks great | 20:15 |
| cdent | jaypipes: that is awesome | 20:15 |
| mriedem | well, what's in self.item ? | 20:15 |
| *** vilobhmm111 has quit IRC | 20:15 | |
| *** mylu has quit IRC | 20:15 | |
| cdent | of course given the overuse of mocks in some of the tests, its not that far off from others | 20:15 |
| jaypipes | mriedem: the compute node object :) | 20:15 |
| *** vilobhmm111 has joined #openstack-nova | 20:16 | |
| *** ssurana has joined #openstack-nova | 20:16 | |
| jaypipes | mriedem: it's a test that says "go and do an aggregate query against a table with one record in it and then test that the only thing returned is that record." | 20:16 |
| jaypipes | mriedem: awesome. | 20:16 |
| *** mylu has joined #openstack-nova | 20:17 | |
| *** vilobhmm11 has quit IRC | 20:18 | |
| *** ihrachys has quit IRC | 20:18 | |
| *** bjornar has joined #openstack-nova | 20:19 | |
| bjornar | Does it exist such thing as availability zones composed of availability zones? | 20:20 |
| *** ctrath has quit IRC | 20:20 | |
| *** yamahata has quit IRC | 20:21 | |
| bjornar | I have the case where I have DC1,DC2,DC2, and I would like some tenants to be able to see these datacenters as AZ's, but other tenants should just se the ALLDCS AZ.. | 20:21 |
| *** xyang1 has quit IRC | 20:21 | |
| *** ctrath has joined #openstack-nova | 20:22 | |
| openstackgerrit | jichenjc proposed openstack/nova: Change wording of microversion bump about 503 https://review.openstack.org/286819 | 20:22 |
| jaypipes | bauzas, mriedem, dansmith: quick question for you on this compute_node_statistics() thing... so, currently, the compute_node_statistics() method returns a SUM(compute_nodes.free_ram_mb) and SUM(compute_nodes.free_disk_gb). These fields, as you may know, have dubious value in terms of correctness (we actually don't use them in the scheduler for determining available resource amounts; they are only updated in the periodic interval job on the RT and | 20:24 |
| jaypipes | exposed via this compute REST API call that does the compute_node_statistics() call. My question to you is this: do you care if I return the *actual* available amount of free RAM and disk space in these output fields as calculated from the inventories and allocations tables? | 20:24 |
| bauzas | jaypipes: so, AFAIK, the statistics() is only for the host API | 20:25 |
| *** vilobhmm111 has quit IRC | 20:25 | |
| *** dedery has joined #openstack-nova | 20:26 | |
| bauzas | jaypipes: that's not something for the RT | 20:26 |
| bauzas | jaypipes: just for operators knowing the statistics | 20:26 |
| *** josecastroleon has quit IRC | 20:27 | |
| bauzas | jaypipes: so, sorry os-hypervisors | 20:28 |
| jaypipes | bauzas: yes, I know. what I'm asking is: is it OK if I completely ignore the commpute_nodes.free_ram_mb and compute_nodes.free_disk_gb fields. | 20:28 |
| bauzas | jaypipes: since it's an API stuff, you want to not provide them for the resource? | 20:29 |
| bauzas | jaypipes: not sure I understand your question | 20:29 |
| jaypipes | bauzas: I'm asking for permission to essentially deprecate the free_ram_mb and free_disk_gb fields. | 20:30 |
| bauzas | jaypipes: you mean to remove those fields in the response here ? http://developer.openstack.org/api-ref-compute-v2.1.html#showHypervisorStatistics | 20:30 |
| bauzas | jaypipes: if so, that would need an API version I think | 20:31 |
| *** zengine has joined #openstack-nova | 20:32 | |
| dansmith | jaypipes: I dunno.. they're reporting zero now, which makes me fail api_samples of course | 20:32 |
| bauzas | jaypipes: unless you're still providing those fields in the API but using the inventories vs. allocations | 20:32 |
| dansmith | I don't really know the impact of lying more or less with that output | 20:32 |
| *** mdrabe has joined #openstack-nova | 20:34 | |
| *** tongli has quit IRC | 20:35 | |
| jaypipes | bauzas: I'm not proposing changing the REST API, no. I'm proposing not using those fields in the compute_nodes table from my patch onward. | 20:35 |
| bauzas | a-ah by reviewed your above message, I understand now | 20:37 |
| bauzas | sec, I think we use free_ram_mb in the filters, don't we ? | 20:38 |
| *** alejandrito has joined #openstack-nova | 20:38 | |
| *** penick has quit IRC | 20:38 | |
| *** e0ne has joined #openstack-nova | 20:38 | |
| *** mylu has quit IRC | 20:39 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: check for optional LibvirtLiveMigrateData attrs before loading https://review.openstack.org/286828 | 20:39 |
| *** yamahata has joined #openstack-nova | 20:39 | |
| bauzas | jaypipes: so, at least the RAMWeigher uses it now | 20:40 |
| *** jdillaman has joined #openstack-nova | 20:40 | |
| jaypipes | bauzas: RAMWeigher must be borked.. :) | 20:41 |
| bauzas | jaypipes: and indeed the RAMFilter | 20:41 |
| bauzas | jaypipes: I know your opinion about that, but they're still in use | 20:41 |
| bauzas | and while you could say that the RAMFilter could be deprecated if we agree to have the scheduler using a where conditional for the RAM resources, but that's still not the case, and I feel that the RAMWeigher is heavily in use for most ops | 20:42 |
| openstackgerrit | jichenjc proposed openstack/python-novaclient: move migrations from extension https://review.openstack.org/285213 | 20:42 |
| *** pcaruana has joined #openstack-nova | 20:44 | |
| *** jamielennox|away is now known as jamielennox | 20:45 | |
| jaypipes | bauzas: Sylvain, I'm not arguing about the filter vs. DB WHERE condition. | 20:46 |
| jaypipes | bauzas: I am asking for permission to generate that information instead of return the free_ram_mb and free_disk_gb fieds. | 20:46 |
| bauzas | jaypipes: so, Jay, I probably again misunderstood your question :) | 20:46 |
| *** jamie_h has quit IRC | 20:46 | |
| openstackgerrit | jichenjc proposed openstack/python-novaclient: move migrations from extension https://review.openstack.org/285213 | 20:46 |
| *** e0ne has quit IRC | 20:47 | |
| bauzas | jaypipes: you're asking for permission to use the inventories vs. allocation (like you did for compute_node_get()) for returning the free RAM and disk ? | 20:47 |
| *** yamahata has quit IRC | 20:47 | |
| bauzas | if so, yes | 20:47 |
| mriedem | jaypipes: you were +2 on this before it was rebased, can you hit it again? https://review.openstack.org/#/c/284257/ | 20:48 |
| bauzas | jaypipes: for returning the free RAM and disk for the API call said here http://developer.openstack.org/api-ref-compute-v2.1.html#showHypervisorStatistics ? If so, yes | 20:48 |
| jaypipes | mriedem: done | 20:48 |
| *** MVenesio has quit IRC | 20:48 | |
| mriedem | <3 | 20:48 |
| * bauzas feels his english comprehension skills to be in the trashbin tonight :( | 20:49 | |
| *** dedery has quit IRC | 20:50 | |
| *** gyee has joined #openstack-nova | 20:50 | |
| jaypipes | bauzas, dansmith: so the issue with the compute_node_statistics() code is that once dansmith's patch to migrate inventory lands, the values for the memory_mb, local_gb, and vcpus fields will be 0, right? | 20:52 |
| dansmith | yes | 20:52 |
| bauzas | that's only a problem for the statistics because it doesn't use the object facade, right? | 20:53 |
| jaypipes | bauzas, dansmith: nvm then, gimme a little while to work this sqlalchemy-fu out. apologies. | 20:53 |
| mriedem | sdague: on https://review.openstack.org/#/c/286776/ i tried the rest of the libvirt unit tests with an empty /etc/machine-id file and it was ok | 20:53 |
| bauzas | jaypipes: if the problem is that because statistics are not using the object facade, then there are two possibilities : | 20:55 |
| *** e0ne has joined #openstack-nova | 20:55 | |
| *** ctrath has quit IRC | 20:55 | |
| bauzas | #1 either use the object facade for the statistics by adding a new remotable method for the ComputeNode object that would propose statistics new-way (using the inventories table) | 20:55 |
| jaypipes | bauzas: no, please accept my apologies. I just need to get this code done. problem is on my end. | 20:56 |
| bauzas | #2 fix the API caller to instead use another DB API that could use the inventories table | 20:56 |
| *** ctrath has joined #openstack-nova | 20:57 | |
| bauzas | jaypipes: not sure I understood your problem, but I accept your apologies and I truly ask permission for accepting my apologies about me just not understanding your problem :D | 20:57 |
| dansmith | lol | 20:57 |
| bauzas | see, French people can be gentle ? | 20:57 |
| bauzas | yo | 20:57 |
| bauzas | http://static.lexpress.fr/medias_10133/w_1520,h_855,c_fill,g_north/v1421314965/fox-news-nolan-peterson_5188209.jpeg evn | 20:58 |
| *** e0ne has quit IRC | 20:59 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-nova | 21:01 | |
| *** cdent has quit IRC | 21:03 | |
| *** sneti has quit IRC | 21:04 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add a hacking check for test method closures https://review.openstack.org/284970 | 21:05 |
| *** MVenesio has joined #openstack-nova | 21:06 | |
| *** raildo is now known as raildo-afk | 21:07 | |
| *** david-lyle has quit IRC | 21:08 | |
| *** suresh12 has quit IRC | 21:11 | |
| *** tbachman has quit IRC | 21:13 | |
| *** suresh12 has joined #openstack-nova | 21:15 | |
| *** zengine_ has joined #openstack-nova | 21:15 | |
| *** jdurgin has quit IRC | 21:15 | |
| *** efried has quit IRC | 21:16 | |
| *** zengine has quit IRC | 21:17 | |
| *** zengine_ is now known as zengine | 21:17 | |
| *** david-lyle has joined #openstack-nova | 21:17 | |
| *** mylu has joined #openstack-nova | 21:20 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add a hacking check for test method closures https://review.openstack.org/284970 | 21:26 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Ensure there are no unreferenced closures in tests https://review.openstack.org/286846 | 21:26 |
| *** jichen has quit IRC | 21:28 | |
| *** efried has joined #openstack-nova | 21:29 | |
| mriedem | tonyb: nova is busted on kilo now too http://logs.openstack.org/72/286772/1/check/gate-nova-python27/46350f4/console.html#_2016-03-01_20_20_52_585 | 21:29 |
| mriedem | :( | 21:29 |
| mriedem | lifeless: ^ can we talk about https://bugs.launchpad.net/python-fixtures/+bug/1542984 ? | 21:29 |
| openstack | Launchpad bug 1542984 in Python Fixtures "fixtures 1.2.0 isn't compatible with testtools 2.0.0" [Undecided,New] | 21:29 |
| tonyb | mriedem: *sigh* | 21:29 |
| tonyb | mriedem: neutron timedout and added my work around ... | 21:30 |
| mriedem | something is pulling in fixtures 1.2.0 or testtools 2.0.0, not sure what that is yet | 21:30 |
| tonyb | mriedem: oslo.concurrency | 21:31 |
| *** dims_ has joined #openstack-nova | 21:31 | |
| mriedem | so kilo caps fixtures<1.3.0 so we get fixtures===1.2.0 which has uncapped testtools | 21:31 |
| mriedem | which pulls in 2.0.0, which requires fixtures>=1.3.0 | 21:31 |
| mriedem | right? | 21:31 |
| *** yamahata has joined #openstack-nova | 21:32 | |
| mriedem | doesn't nova just need to install testtools before fixtures in test-requirements? | 21:32 |
| *** dims has quit IRC | 21:33 | |
| sdague | mriedem: yes | 21:33 |
| tonyb | mriedem: Yeah that works | 21:33 |
| sdague | order in the file is important, then it will be fine | 21:34 |
| sdague | we've fixed things like that before | 21:34 |
| mriedem | sdague: yeah i know | 21:34 |
| mriedem | tonyb: i see https://review.openstack.org/#/c/277701/1/requirements.txt | 21:34 |
| tonyb | mriedem: I resisted doign that until lifeless's issue is closed or rejected | 21:34 |
| mriedem | let me verify nova is failing by oslo.concurrency pulling in testtools | 21:34 |
| tonyb | mriedem: look at: http://logs.openstack.org/72/286772/1/check/gate-nova-python27/46350f4/tox/py27-2.log.txt | 21:34 |
| tonyb | it's there pretty clear | 21:35 |
| tonyb | mriedem: Yeah you can see -W'd that. | 21:35 |
| mriedem | http://logs.openstack.org/72/286772/1/check/gate-nova-python27/46350f4/tox/py27-2.log.txt | 21:35 |
| mriedem | yeah | 21:35 |
| mriedem | Collecting testtools>=0.9.22 (from fixtures<1.3.0,>=0.3.14->oslo.concurrency<1.9.0,>=1.8.2->nova==2015.1.4.dev10) Downloading http://mirror.iad.rax.openstack.org/pypi/packages/2.7/t/testtools/testtools-2.0.0-py2.py3-none-any.whl (176kB) | 21:36 |
| tonyb | Yup that's the one | 21:36 |
| * jaypipes predicts dansmith is going to vomit when he sees this sqlalchemy code. | 21:36 | |
| tonyb | jaypipes: good job | 21:36 |
| tonyb | mriedem: I need to take the kids to school, I'll be back in 30 | 21:37 |
| tonyb | mriedem: sorry. | 21:37 |
| mriedem | np | 21:37 |
| tonyb | mriedem: I think we shoudl just start landing work arounds. | 21:37 |
| mriedem | jaypipes: but his abs will look awesome afterward! | 21:37 |
| mriedem | tonyb: agreed | 21:37 |
| *** jdurgin has joined #openstack-nova | 21:38 | |
| tonyb | mriedem: okay I'll remove my blocks and work on that today. | 21:38 |
| tonyb | mriedem: There are still parts I don | 21:38 |
| *** smatzek has quit IRC | 21:38 | |
| tonyb | t understand but we can work through that later ..... | 21:38 |
| tonyb | be back in 30 | 21:38 |
| *** cfcp1 has joined #openstack-nova | 21:40 | |
| lifeless | mriedem: sorry, been a crazy period for me | 21:40 |
| lifeless | mriedem: tony has talked me around | 21:40 |
| mriedem | lifeless: yeah i think we just need a fixtures 1.2.1 | 21:40 |
| mriedem | that caps testtools | 21:40 |
| mriedem | fixtures 1.2.1 would require testtools<2.0.0 | 21:40 |
| *** derekh has joined #openstack-nova | 21:41 | |
| mriedem | ah the real nut is that oslo.concurrency 1.8.2 has fixtures in requirements.txt | 21:42 |
| mriedem | that should probably have been in extras or something | 21:42 |
| lifeless | mriedem: fixtures is for more than tests, but thats a different discussion | 21:43 |
| *** Sukhdev has quit IRC | 21:43 | |
| lifeless | mriedem: the real problem is pip | 21:43 |
| *** baoli has joined #openstack-nova | 21:43 | |
| mriedem | yeah, but | 21:44 |
| *** lpetrut has joined #openstack-nova | 21:44 | |
| mriedem | lifeless: so is it going to be possible to get a fixtures 1.2.1 release with testtools>=0.9.22,<2.0.0? | 21:44 |
| lifeless | mriedem: yes, as I said tony has talked me around | 21:44 |
| *** baoli_ has joined #openstack-nova | 21:44 | |
| *** efried has quit IRC | 21:44 | |
| lifeless | mriedem: we have no branches of it though, so I need to poke around and see what the impact of making one will be | 21:44 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add a hacking check for test method closures https://review.openstack.org/284970 | 21:45 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Ensure there are no unreferenced closures in tests https://review.openstack.org/286846 | 21:45 |
| mriedem | lifeless: ok | 21:45 |
| *** cfcp1 has quit IRC | 21:46 | |
| mriedem | lifeless: couldn't you just branch from the 1.2.0 tag? | 21:47 |
| openstackgerrit | Mark Sturdevant proposed openstack/nova: Provide correct connector for evacuate terminate https://review.openstack.org/266095 | 21:48 |
| lifeless | mriedem: yes, making the branch in git is not hard | 21:48 |
| *** baoli has quit IRC | 21:48 | |
| *** vilobhmm11 has joined #openstack-nova | 21:49 | |
| *** vilobhmm11 has quit IRC | 21:49 | |
| *** vilobhmm11 has joined #openstack-nova | 21:49 | |
| *** sdague has quit IRC | 21:50 | |
| *** fawadkhaliq has quit IRC | 21:54 | |
| *** tbachman has joined #openstack-nova | 21:55 | |
| *** Bjoern_ has joined #openstack-nova | 21:58 | |
| openstackgerrit | Mark Sturdevant proposed openstack/nova: Add instance host to evacuate terminate connector https://review.openstack.org/274318 | 22:00 |
| *** haomaiwang has quit IRC | 22:01 | |
| *** pcaruana has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-nova | 22:01 | |
| *** mingyu has joined #openstack-nova | 22:02 | |
| *** dims_ has quit IRC | 22:02 | |
| *** efried has joined #openstack-nova | 22:03 | |
| *** fawadkhaliq has joined #openstack-nova | 22:04 | |
| *** Sukhdev has joined #openstack-nova | 22:04 | |
| *** ivase has quit IRC | 22:04 | |
| *** knikolla has quit IRC | 22:04 | |
| mriedem | lifeless: alright, i have a workaround for nova for now | 22:05 |
| mriedem | will push that up | 22:05 |
| *** mingyu has quit IRC | 22:06 | |
| *** MVenesio has quit IRC | 22:07 | |
| *** ivase has joined #openstack-nova | 22:08 | |
| dansmith | jaypipes: oops, missed that invitation to comment :P | 22:08 |
| *** rtheis has quit IRC | 22:09 | |
| mriedem | damn, sean silently left | 22:09 |
| *** thorst_ has quit IRC | 22:09 | |
| *** ijw has joined #openstack-nova | 22:09 | |
| *** dims has joined #openstack-nova | 22:10 | |
| *** rk4n has quit IRC | 22:11 | |
| mtreinish | mriedem: yeah he does that now | 22:11 |
| mriedem | silent and deadly | 22:11 |
| mriedem | mtreinish: maybe you'll like this https://review.openstack.org/#/c/286863/ | 22:12 |
| *** ivase has quit IRC | 22:12 | |
| mtreinish | mriedem: I don't like it, but +2 | 22:14 |
| mriedem | you love it | 22:14 |
| *** efried has quit IRC | 22:15 | |
| *** zenoway has quit IRC | 22:16 | |
| *** penick has joined #openstack-nova | 22:16 | |
| *** tbachman has quit IRC | 22:16 | |
| dims | haha mriedem | 22:17 |
| *** alejandrito has quit IRC | 22:18 | |
| *** Sukhdev has quit IRC | 22:19 | |
| *** gcb has quit IRC | 22:19 | |
| dims | diana_clarke : ping | 22:19 |
| *** gcb has joined #openstack-nova | 22:19 | |
| dims | diana_clarke : can you please try this? https://review.openstack.org/#/c/286838/ | 22:20 |
| *** dimtruck is now known as zz_dimtruck | 22:20 | |
| *** rlrossit has quit IRC | 22:20 | |
| dims | _diana_ : ^^ | 22:22 |
| *** ric has joined #openstack-nova | 22:22 | |
| *** achanda has quit IRC | 22:23 | |
| *** josecastroleon has joined #openstack-nova | 22:24 | |
| *** mvk has joined #openstack-nova | 22:26 | |
| *** sneti has joined #openstack-nova | 22:27 | |
| *** krtaylor has quit IRC | 22:27 | |
| *** david-lyle has quit IRC | 22:29 | |
| *** yamahata has quit IRC | 22:30 | |
| *** suresh12 has quit IRC | 22:32 | |
| *** ccarmack has quit IRC | 22:34 | |
| *** efried has joined #openstack-nova | 22:36 | |
| *** achanda has joined #openstack-nova | 22:36 | |
| *** angdraug has quit IRC | 22:36 | |
| *** aysyd has quit IRC | 22:38 | |
| *** zenoway has joined #openstack-nova | 22:38 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: don't attempt to get baseline cpu features if host cpu model is None https://review.openstack.org/286868 | 22:39 |
| mriedem | Sam-I-Am: ^ can you test that out and see if it fixes your issue? | 22:39 |
| *** ctrath has quit IRC | 22:42 | |
| *** zenoway has quit IRC | 22:43 | |
| *** zz_dimtruck is now known as dimtruck | 22:45 | |
| Sam-I-Am | mriedem: looking | 22:49 |
| *** mylu has quit IRC | 22:50 | |
| *** ctrath has joined #openstack-nova | 22:51 | |
| *** PaulMurray has quit IRC | 22:52 | |
| *** mriedem has quit IRC | 22:52 | |
| tonyb | dims: are you around? | 22:53 |
| *** baoli_ has quit IRC | 22:54 | |
| *** ccarmack has joined #openstack-nova | 22:54 | |
| *** josecastroleon has quit IRC | 22:54 | |
| dims | tonyb : pong | 22:54 |
| *** baoli has joined #openstack-nova | 22:54 | |
| *** edtubill has quit IRC | 22:54 | |
| *** andymaier has quit IRC | 22:55 | |
| *** dims has quit IRC | 22:55 | |
| tonyb | dims: anycahnce you can look at: https://review.openstack.org/#/c/277701/1 and give an opinion? | 22:56 |
| tonyb | Gah he just left :( | 22:56 |
| *** delattec has quit IRC | 22:57 | |
| *** mylu has joined #openstack-nova | 22:58 | |
| *** baoli has quit IRC | 22:58 | |
| *** ccarmack has quit IRC | 22:58 | |
| *** baoli has joined #openstack-nova | 22:59 | |
| *** josecastroleon has joined #openstack-nova | 22:59 | |
| *** whenry_ has quit IRC | 23:00 | |
| *** sc68cal has quit IRC | 23:00 | |
| *** harlowja_at_home has quit IRC | 23:00 | |
| *** delattec has joined #openstack-nova | 23:00 | |
| *** baoli has quit IRC | 23:00 | |
| *** david-lyle has joined #openstack-nova | 23:00 | |
| *** dims has joined #openstack-nova | 23:00 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** yassine__ has quit IRC | 23:01 | |
| *** baoli has joined #openstack-nova | 23:01 | |
| dims | tonyb : pong 2 | 23:01 |
| *** haomaiwang has joined #openstack-nova | 23:01 | |
| tonyb | dims: dims: anycahnce you can look at: https://review.openstack.org/#/c/277701/1 and give an opinion? | 23:01 |
| tonyb | sorry to poke you late in your day | 23:01 |
| dims | tonyb : if it works, more power to you :) i don't mind at all | 23:02 |
| tonyb | dims: :) | 23:03 |
| dims | no worries about timing tonyb | 23:03 |
| *** lpetrut has quit IRC | 23:06 | |
| *** ianw has quit IRC | 23:07 | |
| *** busterswt has quit IRC | 23:08 | |
| *** burt has quit IRC | 23:08 | |
| *** busterswt has joined #openstack-nova | 23:08 | |
| *** busterswt has quit IRC | 23:08 | |
| *** doug-fis_ has joined #openstack-nova | 23:09 | |
| *** doug-fis_ has quit IRC | 23:09 | |
| *** doug-fis_ has joined #openstack-nova | 23:09 | |
| *** rfolco_ has quit IRC | 23:11 | |
| *** baoli has quit IRC | 23:11 | |
| *** mdrabe has quit IRC | 23:11 | |
| *** doug-fish has quit IRC | 23:13 | |
| *** zengine has quit IRC | 23:13 | |
| *** haplo37 has quit IRC | 23:14 | |
| *** doug-fis_ has quit IRC | 23:14 | |
| *** krtaylor has joined #openstack-nova | 23:16 | |
| *** suresh12 has joined #openstack-nova | 23:16 | |
| *** sheel has quit IRC | 23:17 | |
| *** Sukhdev has joined #openstack-nova | 23:18 | |
| *** tbachman has joined #openstack-nova | 23:19 | |
| *** salv-orl_ has joined #openstack-nova | 23:19 | |
| *** Jeffrey4l has joined #openstack-nova | 23:21 | |
| *** rk4n has joined #openstack-nova | 23:21 | |
| *** salv-orlando has quit IRC | 23:22 | |
| *** sneti has quit IRC | 23:23 | |
| *** Bjoern_ has quit IRC | 23:24 | |
| *** josecastroleon has quit IRC | 23:29 | |
| *** Jeffrey4l has quit IRC | 23:31 | |
| *** haukebruno has joined #openstack-nova | 23:31 | |
| *** zenoway has joined #openstack-nova | 23:33 | |
| *** darrenc is now known as darrenc_afk | 23:33 | |
| *** ric has quit IRC | 23:35 | |
| *** mrkz has quit IRC | 23:36 | |
| *** zenoway has quit IRC | 23:37 | |
| *** ducttape_ has quit IRC | 23:38 | |
| *** david-lyle has quit IRC | 23:41 | |
| *** mriedem has joined #openstack-nova | 23:42 | |
| *** dimtruck is now known as zz_dimtruck | 23:44 | |
| *** doug-fish has joined #openstack-nova | 23:46 | |
| *** doug-fish has quit IRC | 23:47 | |
| *** mylu has quit IRC | 23:49 | |
| *** salv-orlando has joined #openstack-nova | 23:50 | |
| *** salv-orlando has quit IRC | 23:50 | |
| *** salv-orl_ has quit IRC | 23:50 | |
| *** darrenc_afk is now known as darrenc | 23:50 | |
| *** salv-orlando has joined #openstack-nova | 23:50 | |
| *** mylu has joined #openstack-nova | 23:51 | |
| *** mylu has quit IRC | 23:55 | |
| *** unicell has joined #openstack-nova | 23:55 | |
| *** delattec has quit IRC | 23:59 | |
| *** mylu has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
