| *** claudiub has quit IRC | 00:00 | |
| *** Guest20076 has joined #openstack-nova | 00:02 | |
| *** tetsuro has joined #openstack-nova | 00:02 | |
| *** r-daneel has quit IRC | 00:05 | |
| *** jobewan has joined #openstack-nova | 00:05 | |
| *** david-lyle has quit IRC | 00:07 | |
| *** dklyle has joined #openstack-nova | 00:07 | |
| *** dklyle has quit IRC | 00:07 | |
| *** jmlowe has joined #openstack-nova | 00:09 | |
| *** kumarmn has quit IRC | 00:10 | |
| openstackgerrit | melanie witt proposed openstack/nova stable/pike: libvirt: Re-initialise volumes, encryptors, and vifs on hard reboot https://review.openstack.org/531407 | 00:13 |
|---|---|---|
| *** jobewan has quit IRC | 00:16 | |
| *** itlinux has joined #openstack-nova | 00:18 | |
| *** kumarmn has joined #openstack-nova | 00:28 | |
| *** sdague has quit IRC | 00:29 | |
| *** jmlowe has quit IRC | 00:31 | |
| *** jobewan has joined #openstack-nova | 00:35 | |
| *** edmondsw has joined #openstack-nova | 00:36 | |
| *** esberglu has quit IRC | 00:37 | |
| *** jobewan has quit IRC | 00:38 | |
| *** chyka_ has quit IRC | 00:38 | |
| *** jmlowe has joined #openstack-nova | 00:38 | |
| *** hiro-kobayashi has joined #openstack-nova | 00:38 | |
| *** moshele has quit IRC | 00:42 | |
| *** felipemonteiro_ has joined #openstack-nova | 00:44 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Add CLI to show instance usage audit logs https://review.openstack.org/409413 | 00:45 |
| *** jichen has joined #openstack-nova | 00:45 | |
| *** tuanla____ has joined #openstack-nova | 00:47 | |
| *** kumarmn_ has joined #openstack-nova | 00:49 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 00:50 |
| *** kumarmn has quit IRC | 00:52 | |
| *** kumarmn_ has quit IRC | 00:56 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:57 | |
| *** kumarmn has joined #openstack-nova | 01:02 | |
| *** tidwellr has joined #openstack-nova | 01:06 | |
| *** phuongnh has joined #openstack-nova | 01:11 | |
| openstackgerrit | melanie witt proposed openstack/nova stable/ocata: libvirt: Re-initialise volumes, encryptors, and vifs on hard reboot https://review.openstack.org/531422 | 01:11 |
| *** felipemonteiro_ has quit IRC | 01:12 | |
| *** moshele has joined #openstack-nova | 01:13 | |
| *** markvoelker has joined #openstack-nova | 01:17 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.59 - List/Show all server migration types https://review.openstack.org/430839 | 01:20 |
| *** zhurong has joined #openstack-nova | 01:21 | |
| *** hoangcx has joined #openstack-nova | 01:23 | |
| *** kumarmn has quit IRC | 01:24 | |
| *** phuongnh has quit IRC | 01:24 | |
| *** phuongnh has joined #openstack-nova | 01:25 | |
| *** armax has joined #openstack-nova | 01:25 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505 | 01:26 |
| *** amodi has joined #openstack-nova | 01:31 | |
| *** mriedem1 has joined #openstack-nova | 01:40 | |
| *** mriedem has quit IRC | 01:42 | |
| *** liverpooler has joined #openstack-nova | 01:43 | |
| *** Tom-Tom has quit IRC | 01:43 | |
| *** Tom-Tom has joined #openstack-nova | 01:43 | |
| *** Dinesh_Bhor has quit IRC | 01:45 | |
| *** liverpooler has quit IRC | 01:46 | |
| *** jmlowe has quit IRC | 01:52 | |
| *** Tom-Tom has quit IRC | 01:53 | |
| *** mriedem has joined #openstack-nova | 01:56 | |
| *** mriedem1 has quit IRC | 01:58 | |
| *** felipemonteiro_ has joined #openstack-nova | 02:02 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:05 | |
| *** jmlowe has joined #openstack-nova | 02:07 | |
| *** Dinesh_Bhor has quit IRC | 02:07 | |
| *** Tom-Tom has joined #openstack-nova | 02:09 | |
| *** Tom-Tom has quit IRC | 02:09 | |
| *** Tom-Tom has joined #openstack-nova | 02:09 | |
| *** annp has joined #openstack-nova | 02:09 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Qualify the Placement 1.15 release note https://review.openstack.org/531963 | 02:13 |
| *** namnh has joined #openstack-nova | 02:17 | |
| *** vivsoni has quit IRC | 02:19 | |
| *** vivsoni has joined #openstack-nova | 02:19 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/531737 | 02:22 |
| *** jmlowe has quit IRC | 02:23 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:26 | |
| *** edmondsw has quit IRC | 02:30 | |
| *** Apoorva has quit IRC | 02:32 | |
| *** harlowja has quit IRC | 02:32 | |
| *** Dinesh_Bhor has quit IRC | 02:33 | |
| *** moshele has quit IRC | 02:34 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117 | 02:34 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429 | 02:34 |
| *** jmlowe has joined #openstack-nova | 02:37 | |
| *** xinliang has quit IRC | 02:43 | |
| *** udesale has joined #openstack-nova | 02:46 | |
| openstackgerrit | lei zhang proposed openstack/nova master: Fix the incorrect git.openstack.org source URL https://review.openstack.org/531979 | 02:46 |
| *** tidwellr has quit IRC | 02:49 | |
| *** jmlowe has quit IRC | 02:53 | |
| *** xinliang has joined #openstack-nova | 02:55 | |
| *** hongbin has joined #openstack-nova | 02:57 | |
| *** nicolasbock has quit IRC | 02:58 | |
| *** bkopilov has quit IRC | 03:04 | |
| *** markvoelker has quit IRC | 03:06 | |
| *** adreznec has quit IRC | 03:07 | |
| *** adreznec has joined #openstack-nova | 03:08 | |
| *** Tom-Tom has quit IRC | 03:09 | |
| *** Tom-Tom has joined #openstack-nova | 03:09 | |
| *** hoonetorg has quit IRC | 03:12 | |
| *** yamahata has quit IRC | 03:15 | |
| *** jmlowe has joined #openstack-nova | 03:16 | |
| *** adreznec has quit IRC | 03:16 | |
| *** adreznec has joined #openstack-nova | 03:21 | |
| *** hongbin has quit IRC | 03:24 | |
| *** hongbin has joined #openstack-nova | 03:25 | |
| *** takashin has quit IRC | 03:25 | |
| *** hoonetorg has joined #openstack-nova | 03:25 | |
| *** vladikr has quit IRC | 03:25 | |
| *** vladikr has joined #openstack-nova | 03:28 | |
| *** amodi has quit IRC | 03:31 | |
| openstackgerrit | Merged openstack/nova master: Fix OpenStack capitalization https://review.openstack.org/531593 | 03:36 |
| *** jmlowe has quit IRC | 03:39 | |
| *** yamamoto has joined #openstack-nova | 03:40 | |
| *** Tom-Tom has quit IRC | 03:42 | |
| *** Tom-Tom has joined #openstack-nova | 03:42 | |
| *** jmlowe has joined #openstack-nova | 03:44 | |
| *** abhishekk has joined #openstack-nova | 03:44 | |
| *** Tom-Tom has quit IRC | 03:47 | |
| *** gyee has quit IRC | 03:48 | |
| *** zhurong has quit IRC | 03:55 | |
| *** takashin has joined #openstack-nova | 03:56 | |
| *** shaohe_feng has quit IRC | 03:58 | |
| *** shaohe_feng has joined #openstack-nova | 03:58 | |
| *** sridharg has joined #openstack-nova | 03:58 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:01 | |
| *** Apoorva has joined #openstack-nova | 04:15 | |
| *** esberglu has joined #openstack-nova | 04:16 | |
| *** esberglu has quit IRC | 04:17 | |
| *** jobewan has joined #openstack-nova | 04:20 | |
| *** jobewan has quit IRC | 04:21 | |
| openstackgerrit | Merged openstack/nova stable/ocata: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530981 | 04:22 |
| *** hiro-kobayashi has quit IRC | 04:22 | |
| *** bkopilov has joined #openstack-nova | 04:23 | |
| openstackgerrit | Merged openstack/nova master: Use ksa adapter for neutron client https://review.openstack.org/509892 | 04:24 |
| *** hongbin has quit IRC | 04:26 | |
| *** yangyapeng has quit IRC | 04:31 | |
| *** yangyapeng has joined #openstack-nova | 04:31 | |
| *** liuyulong has joined #openstack-nova | 04:32 | |
| *** felipemonteiro_ has quit IRC | 04:34 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: [libvirt] Allow multiple volume attachments https://review.openstack.org/267587 | 04:44 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Pass multiattach flag to reserve_block_device_name https://review.openstack.org/531244 | 04:44 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: [api] Allow multi-attach in compute api https://review.openstack.org/271047 | 04:44 |
| mriedem | ildikov: ^ the api support should be plumbed in now; required quite a bit of refactor which touched a billion tests, but not complicated. just needs the new paths covered with unit tests. | 04:45 |
| *** mriedem has quit IRC | 04:47 | |
| *** janki has joined #openstack-nova | 04:47 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix missing rps in allocation candidates https://review.openstack.org/522409 | 04:49 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: placement: func tests for multiple shared RPs https://review.openstack.org/498737 | 04:49 |
| *** Dinesh_Bhor has quit IRC | 04:50 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:51 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add an additional description for 'token_ttl' https://review.openstack.org/532007 | 04:52 |
| *** Dinesh_Bhor has quit IRC | 04:52 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:55 | |
| *** markvoelker has joined #openstack-nova | 04:56 | |
| *** felipemonteiro_ has joined #openstack-nova | 05:01 | |
| *** vladikr has quit IRC | 05:03 | |
| *** jichen_ has joined #openstack-nova | 05:03 | |
| *** jichen has quit IRC | 05:04 | |
| *** psachin has joined #openstack-nova | 05:04 | |
| *** jichen_ is now known as jichen | 05:04 | |
| *** links has joined #openstack-nova | 05:07 | |
| openstackgerrit | Sriharsha Basavapatna proposed openstack/os-vif master: Fix VF-rep lookup routine to use parent PF number https://review.openstack.org/531674 | 05:08 |
| *** hiro-kobayashi has joined #openstack-nova | 05:18 | |
| *** ratailor has joined #openstack-nova | 05:19 | |
| *** Tom-Tom has joined #openstack-nova | 05:28 | |
| *** claudiub has joined #openstack-nova | 05:29 | |
| *** sree has joined #openstack-nova | 05:31 | |
| *** sree has quit IRC | 05:32 | |
| *** sree has joined #openstack-nova | 05:32 | |
| takashin | 05:33 | |
| *** rabel_b1 has joined #openstack-nova | 05:36 | |
| *** rabel_b1 has quit IRC | 05:49 | |
| *** Apoorva has quit IRC | 05:49 | |
| *** lajoskatona has joined #openstack-nova | 05:51 | |
| *** smatzek has joined #openstack-nova | 05:53 | |
| *** Tom-Tom has quit IRC | 05:53 | |
| *** hoonetorg has quit IRC | 05:55 | |
| *** xinliang has quit IRC | 05:57 | |
| *** smatzek has quit IRC | 05:57 | |
| *** hoonetorg has joined #openstack-nova | 05:59 | |
| *** tuanla____ has quit IRC | 06:00 | |
| *** Tom-Tom has joined #openstack-nova | 06:02 | |
| *** sandanar has joined #openstack-nova | 06:03 | |
| sandanar | hi.. does anyone know what is the reason behind CONF.libvirt.volume_use_path always returns the default value and not the one passed in [libvirt] section of /etc/nova/nova.conf - https://bugs.launchpad.net/nova/+bug/1723928 this is causing multipath failures on the cinder side | 06:05 |
| openstack | Launchpad bug 1723928 in os-brick "In case of volume_use_multipath=True, Nova unable to fetch CONF.libvirt.volume_use_multipath value from nova.conf" [Undecided,New] | 06:05 |
| *** tuanla____ has joined #openstack-nova | 06:05 | |
| *** Dinesh_Bhor has quit IRC | 06:07 | |
| *** felipemonteiro_ has quit IRC | 06:08 | |
| *** tuanla____ has quit IRC | 06:09 | |
| *** xinliang has joined #openstack-nova | 06:09 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:10 | |
| *** yamahata has joined #openstack-nova | 06:13 | |
| *** gouthamr has quit IRC | 06:17 | |
| *** pmannidi has joined #openstack-nova | 06:18 | |
| *** sree has quit IRC | 06:23 | |
| *** sree has joined #openstack-nova | 06:24 | |
| *** sree has quit IRC | 06:28 | |
| *** goutham has joined #openstack-nova | 06:29 | |
| *** links has quit IRC | 06:35 | |
| *** moshele has joined #openstack-nova | 06:37 | |
| *** rcernin has quit IRC | 06:38 | |
| *** jichen_ has joined #openstack-nova | 06:45 | |
| *** rabel_b1 has joined #openstack-nova | 06:46 | |
| *** ameeda has joined #openstack-nova | 06:47 | |
| *** jichen has quit IRC | 06:47 | |
| *** jichen_ is now known as jichen | 06:47 | |
| *** zhurong has joined #openstack-nova | 06:50 | |
| *** links has joined #openstack-nova | 06:51 | |
| *** Eran_Kuris has joined #openstack-nova | 06:54 | |
| *** hoonetorg has quit IRC | 06:56 | |
| *** pcaruana has joined #openstack-nova | 06:56 | |
| *** gcb has joined #openstack-nova | 06:58 | |
| *** liuyulong has quit IRC | 07:02 | |
| *** links has quit IRC | 07:03 | |
| *** Dinesh_Bhor has quit IRC | 07:05 | |
| *** hoonetorg has joined #openstack-nova | 07:12 | |
| *** links has joined #openstack-nova | 07:14 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:15 | |
| *** Dinesh_Bhor has quit IRC | 07:27 | |
| *** felipemonteiro_ has joined #openstack-nova | 07:27 | |
| *** moshele has left #openstack-nova | 07:27 | |
| *** links has quit IRC | 07:27 | |
| *** moshele has joined #openstack-nova | 07:28 | |
| *** hoonetorg has quit IRC | 07:32 | |
| *** pmannidi has quit IRC | 07:34 | |
| *** felipemonteiro_ has quit IRC | 07:39 | |
| *** sree has joined #openstack-nova | 07:40 | |
| *** sree has quit IRC | 07:44 | |
| *** links has joined #openstack-nova | 07:45 | |
| *** udesale has quit IRC | 07:46 | |
| *** slaweq has joined #openstack-nova | 07:47 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:48 | |
| *** rhagarty has quit IRC | 07:49 | |
| openstackgerrit | chenpengzi proposed openstack/nova master: Add reference to policy sample https://review.openstack.org/531713 | 07:49 |
| *** FL1SK has quit IRC | 07:50 | |
| *** slaweq has quit IRC | 07:52 | |
| *** rhagarty has joined #openstack-nova | 07:52 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:56 | |
| *** itlinux has quit IRC | 07:58 | |
| *** slaweq has joined #openstack-nova | 08:00 | |
| *** hoonetorg has joined #openstack-nova | 08:01 | |
| *** takashin has left #openstack-nova | 08:02 | |
| *** rmart04 has joined #openstack-nova | 08:05 | |
| *** armax has quit IRC | 08:08 | |
| *** hiro-kobayashi has quit IRC | 08:08 | |
| *** slaweq has quit IRC | 08:08 | |
| *** jangutter has joined #openstack-nova | 08:14 | |
| *** damien_r has joined #openstack-nova | 08:16 | |
| *** sahid has joined #openstack-nova | 08:18 | |
| *** tesseract has joined #openstack-nova | 08:20 | |
| *** alexchadin has joined #openstack-nova | 08:22 | |
| *** sree has joined #openstack-nova | 08:24 | |
| *** ralonsoh has joined #openstack-nova | 08:25 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505 | 08:26 |
| *** sree has quit IRC | 08:28 | |
| *** jafeha__ is now known as jafeha | 08:29 | |
| *** psachin has quit IRC | 08:30 | |
| *** sree has joined #openstack-nova | 08:31 | |
| *** sree has quit IRC | 08:36 | |
| *** yamamoto_ has joined #openstack-nova | 08:36 | |
| *** matrohon has joined #openstack-nova | 08:37 | |
| *** namnh has quit IRC | 08:38 | |
| *** mdnadeem has joined #openstack-nova | 08:38 | |
| *** hieulq has quit IRC | 08:38 | |
| *** phuongnh has quit IRC | 08:38 | |
| *** annp has quit IRC | 08:38 | |
| *** hoangcx has quit IRC | 08:38 | |
| *** phuongnh has joined #openstack-nova | 08:38 | |
| *** namnh has joined #openstack-nova | 08:38 | |
| *** hieulq has joined #openstack-nova | 08:38 | |
| *** hoangcx has joined #openstack-nova | 08:39 | |
| *** annp has joined #openstack-nova | 08:39 | |
| *** yamamoto has quit IRC | 08:39 | |
| *** psachin has joined #openstack-nova | 08:40 | |
| hrw | morning | 08:41 |
| hrw | any idea why https://review.openstack.org/#/c/530965 has Zuul said "Verified:+1" when 'recheck' was called after +2+2+W? | 08:43 |
| hrw | and not +2 | 08:44 |
| *** sree has joined #openstack-nova | 08:44 | |
| *** jpena|off is now known as jpena | 08:45 | |
| Roamer` | hrw, it seems to have https://review.openstack.org/#/c/521400/ as a parent, and that has not merged yet (needs workflow) | 08:47 |
| Roamer` | so Zuul cannot pass 530965 to the gate | 08:48 |
| hrw | Roamer`: ? weird. | 08:49 |
| hrw | Roamer`: I made 530965 on top of origin/master and it lacks any Depends-on fields | 08:49 |
| *** Dinesh_Bhor has quit IRC | 08:49 | |
| *** sree has quit IRC | 08:49 | |
| Roamer` | hrw, sorry, I just realized I'm a bit confused, that is, Gerrit is a bit confused | 08:50 |
| Roamer` | if I copy the "Parent" commit ID of your commit to the clipboard, then paste it in Gerrit's "search" field, it gives me 521400 back, but it has a different commit ID, so that's not true | 08:51 |
| Roamer` | of course your commit has something that has already merged as a parent, sorry | 08:52 |
| hrw | Roamer`: no problem | 08:52 |
| hrw | Roamer`: probably change-id of 521400 == commit id of parent | 08:53 |
| Roamer` | nope, that's what I thought at first (had that happen many times at work), but it's not | 08:54 |
| *** hiro-kobayashi has joined #openstack-nova | 08:54 | |
| *** priteau has joined #openstack-nova | 08:56 | |
| bauzas | good morning Novaers | 08:58 |
| hrw | elo bauzas | 09:01 |
| *** cdent has joined #openstack-nova | 09:01 | |
| bauzas | EHLO even | 09:01 |
| bauzas | (loving telnet) | 09:01 |
| hrw | ;) | 09:02 |
| hrw | bauzas: 220 2.0.0 Ready to start TLS | 09:03 |
| hrw | and then we go into binary | 09:03 |
| hrw | and no one understand | 09:03 |
| bauzas | I could speak French, I would be sure that barely only 10-20% of people here could understand me | 09:07 |
| *** Dinesh_Bhor has joined #openstack-nova | 09:08 | |
| hrw | bauzas: merci, no. | 09:08 |
| bauzas | dammit | 09:08 |
| hrw | bauzas: year ago I took my daughter to Touluise. on second day each conversation with new people started from 'excuse mua, parle anglese?' (or sth like that) | 09:09 |
| bauzas | yeah I know, things would be more easier if we could exchange public keys verbally instead of using foreign languages | 09:11 |
| *** harsha has joined #openstack-nova | 09:13 | |
| *** lucas-afk is now known as lucasagomes | 09:18 | |
| *** dtantsur|afk is now known as dtantsur | 09:20 | |
| openstackgerrit | Huan Xie proposed openstack/nova master: [WIP] XenAPI: Fix vm live migration problem with volume https://review.openstack.org/532118 | 09:21 |
| *** flwang1 has quit IRC | 09:22 | |
| *** flwang has joined #openstack-nova | 09:24 | |
| *** sree has joined #openstack-nova | 09:30 | |
| *** hoonetorg has quit IRC | 09:32 | |
| *** derekh has joined #openstack-nova | 09:33 | |
| *** sree has quit IRC | 09:34 | |
| *** jistr|off|mtg is now known as jistr | 09:36 | |
| *** goutham has quit IRC | 09:36 | |
| *** Dinesh_Bhor has quit IRC | 09:38 | |
| openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Configure privsep binary https://review.openstack.org/531358 | 09:39 |
| openstackgerrit | Stephen Finucane proposed openstack/os-vif master: zuul: Enable functional tests in gate https://review.openstack.org/530961 | 09:39 |
| stephenfin | flwang: I'm around now if you've still got questions? | 09:40 |
| *** sree has joined #openstack-nova | 09:41 | |
| *** tuanla____ has joined #openstack-nova | 09:44 | |
| *** sree has quit IRC | 09:45 | |
| stephenfin | melwitt: Ta-da! https://review.openstack.org/#/c/530950/ | 09:46 |
| * stephenfin thanks the Python 3 gods for letting his code work | 09:46 | |
| *** sree has joined #openstack-nova | 09:48 | |
| *** hoonetorg has joined #openstack-nova | 09:49 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965 | 09:49 |
| *** FL1SK has joined #openstack-nova | 09:50 | |
| *** sree has quit IRC | 09:52 | |
| *** yamamoto_ has quit IRC | 09:52 | |
| claudiub | efried: hello. The autospec patch merged in oslotest. :D | 09:55 |
| hrw | stephenfin: thx | 09:57 |
| hrw | stephenfin: I got used to openstack/kolla patches were zuul mails me "+2" and "merged" in same minute ;D | 09:57 |
| stephenfin | Yeahhhhh, our test runs take a while :) | 09:58 |
| *** hiro-kobayashi has quit IRC | 09:58 | |
| hrw | ;D | 09:58 |
| *** jaosorior has joined #openstack-nova | 09:59 | |
| *** udesale has joined #openstack-nova | 09:59 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Handle deprecation of inspect.getargspec https://review.openstack.org/530777 | 09:59 |
| bauzas | jianghuaw_: around ? | 10:01 |
| *** lpetrut has joined #openstack-nova | 10:01 | |
| bauzas | jianghuaw_: just a clarification with https://review.openstack.org/#/c/516899/34/nova/virt/xenapi/driver.py | 10:01 |
| bauzas | jianghuaw_: since you don't lookup the allocations (just verifying whether there are some VGPU ones), you don't verify how many RPs allocations are there, and just say "okay, only one", right? | 10:02 |
| bauzas | jianghuaw_: for example, say I have an allocations list having two allocations for two separate RPs, and each one having VGPU:2, then you just ask Xen to create only one (not 4), right? | 10:03 |
| jianghuaw_ | Bauzas, yes xen only supports one vGPU per VM. | 10:06 |
| bauzas | that, I know | 10:06 |
| bauzas | I was just wanting to make sure | 10:06 |
| jianghuaw_ | bauzas, why it will pass down multiple allocations? | 10:07 |
| bauzas | jianghuaw_: it won't | 10:07 |
| *** Tom-Tom has quit IRC | 10:07 | |
| bauzas | jianghuaw_: just the fact that you'll need to modify your method once we use nested RPs | 10:07 |
| bauzas | but at least, you could have one allocation for a RP, with a value more than 1, but okay | 10:08 |
| bauzas | jianghuaw_: FWIW, the nvidia driver for libvirt doesn't accept more than one vGPU per pGPU | 10:08 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505 | 10:09 |
| jianghuaw_ | bauzas, ah, Thanks. we need change that method. I think naichuan already has one patch to mapping the RP-uuid from the allocation to the proper GPU group. | 10:09 |
| bauzas | jianghuaw_: but you can ask for 2 vGPUs if you have more than 1 pGPU for your host | 10:09 |
| *** sree has joined #openstack-nova | 10:09 | |
| jianghuaw_ | no. still only one vGPU. | 10:10 |
| *** tuanla____ has quit IRC | 10:10 | |
| bauzas | jianghuaw_: WDYM ? | 10:10 |
| jianghuaw_ | It don't support >1 vGPU for one VM. | 10:10 |
| bauzas | for nvidia ? | 10:10 |
| bauzas | AFAIK, you can have 2 vGPUs with the nvidia driver *if* each of them are by a separate physical card | 10:11 |
| *** ktibi has joined #openstack-nova | 10:11 | |
| jianghuaw_ | XenServer has the restriction in xapi. | 10:11 |
| bauzas | oh ok, I misunderstood | 10:11 |
| jianghuaw_ | I guess nvidia can support but xen can't. | 10:11 |
| bauzas | no, nvidia driver can't | 10:11 |
| bauzas | jianghuaw_: http://images.nvidia.com/content/grid/pdf/GRID-vGPU-User-Guide.pdf section 1.3.2 | 10:12 |
| bauzas | it tells to use homogenous vGPU types, but it also means that you need to use separate physical devices, at least for libvirt | 10:12 |
| jianghuaw_ | can it support multiple vGPU with the same vGPUtype? | 10:13 |
| bauzas | like I said, you need to use separate physical devices | 10:14 |
| bauzas | that's a driver limitation | 10:14 |
| jianghuaw_ | bauzas, Got it. thanks for letting me know of it. | 10:15 |
| *** yamamoto has joined #openstack-nova | 10:16 | |
| *** sambetts|afk is now known as sambetts | 10:16 | |
| jianghuaw_ | bauzas, do you have a link on the *driver limitation*? | 10:16 |
| bauzas | no, unfortunately | 10:16 |
| bauzas | that's something someone told me | 10:17 |
| bauzas | awesome, heh ? | 10:17 |
| bauzas | code driven by trust | 10:17 |
| bauzas | and word of mouth | 10:17 |
| jianghuaw_ | :-) | 10:18 |
| *** mvk has joined #openstack-nova | 10:20 | |
| openstackgerrit | Peter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733 | 10:20 |
| *** sree has quit IRC | 10:23 | |
| *** ratailor_ has joined #openstack-nova | 10:24 | |
| *** jichen has quit IRC | 10:24 | |
| *** ratailor has quit IRC | 10:26 | |
| *** ratailor_ has quit IRC | 10:26 | |
| *** ratailor_ has joined #openstack-nova | 10:27 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | 10:35 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Provide an RFB security proxy implementation https://review.openstack.org/345399 | 10:35 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544 | 10:35 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Send bytes to sockets https://review.openstack.org/531834 | 10:35 |
| *** rmart04 has quit IRC | 10:36 | |
| *** janki has quit IRC | 10:38 | |
| *** Tom-Tom has joined #openstack-nova | 10:38 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505 | 10:39 |
| *** sdague has joined #openstack-nova | 10:40 | |
| *** Tom-Tom has quit IRC | 10:42 | |
| *** avolkov has joined #openstack-nova | 10:43 | |
| *** priteau has quit IRC | 10:43 | |
| *** priteau has joined #openstack-nova | 10:44 | |
| *** hoonetorg has quit IRC | 10:46 | |
| harsha | Hi folks, quick question on some python unit tests; I need to run - 'tox -e pep8'; should i run this on the test machine in the directory/path where I have my changes ? | 10:47 |
| tetsuro | stephenfin: thanks for reviewing https://review.openstack.org/#/c/531328/ :) I'm both okay with microversion or not. | 10:47 |
| *** hoonetorg has joined #openstack-nova | 10:47 | |
| *** hemna_ has quit IRC | 10:48 | |
| *** priteau has quit IRC | 10:48 | |
| *** namnh has quit IRC | 10:48 | |
| tetsuro | stephenfin: Just FYI, I happend to find this blog https://techglimpse.com/why-does-openstack-report-hypervisor-type-as-qemu-when-libvirt_type-is-kvm/ . I don't know who wrote this, though. | 10:48 |
| stephenfin | tetsuro: Oh yeah, to be fair, I honestly don't know what way it should be hence no -1 | 10:49 |
| stephenfin | cdent: Fancy weighing in on https://review.openstack.org/#/c/531328/ ? | 10:49 |
| stephenfin | I know we fix some bugs without microversions, but maybe they're just for 5xx type errors | 10:50 |
| sean-k-mooney | tetsuro: well technicall kvm is not a hypervisor | 10:50 |
| sean-k-mooney | it is kernel acclartion module that is used to acclerate qemu | 10:50 |
| *** chyka has joined #openstack-nova | 10:51 | |
| *** janki has joined #openstack-nova | 10:51 | |
| *** abhishekk has quit IRC | 10:51 | |
| tetsuro | sean-k-mooney: hi, happy new year! | 10:51 |
| sean-k-mooney | tetsuro: same to you :) i trust you had a good vaction | 10:51 |
| *** ygl has joined #openstack-nova | 10:52 | |
| *** ygl has left #openstack-nova | 10:52 | |
| *** kholkina has joined #openstack-nova | 10:52 | |
| tetsuro | sean-k-mooney: yup, I had a lot of mochi(rice cake) in new year days. | 10:52 |
| *** yamahata has quit IRC | 10:53 | |
| cdent | stephenfin: looking | 10:53 |
| *** alexchadin has quit IRC | 10:53 | |
| rabel_b1 | do you know, what happened to https://docs.openstack.org/user-guide/cli-nova-launch-instance-from-volume.html ? i cannot seem to find it anywhere in the nova docs | 10:54 |
| tetsuro | It is true that KVM is a kind of acceralator for QEMU, but in OpenStack document, it is one of the hypervisors. https://docs.openstack.org/mitaka/config-reference/compute/hypervisors.html | 10:54 |
| lyarwood | mdbooth: https://review.openstack.org/#/c/460243 btw, merged both, pretty large change to backport but meh. | 10:55 |
| *** chyka has quit IRC | 10:55 | |
| hrw | tetsuro: kvm is in-kernel hypervisor with qemu used as interface to it. sth like that iirc | 10:57 |
| mdbooth | lyarwood: Doesn't look that bad | 10:58 |
| mdbooth | Not in driver.py, anyway | 10:58 |
| *** alexchadin has joined #openstack-nova | 10:59 | |
| *** AlexeyAbashkin has quit IRC | 11:02 | |
| *** AlexeyAbashkin has joined #openstack-nova | 11:02 | |
| *** ratailor__ has joined #openstack-nova | 11:02 | |
| *** hoonetorg has quit IRC | 11:04 | |
| *** ratailor_ has quit IRC | 11:05 | |
| * cdent collides with sean-k-mooney | 11:06 | |
| cdent | you're probalby right | 11:06 |
| sean-k-mooney | cdent: well when it comes to micro version is not really someting i spend much time thinking about normally so im not sure about that but im leaning towords the bump in this instance | 11:08 |
| cdent | you've provided the critical point: a client may make decision based on the value | 11:08 |
| *** priteau has joined #openstack-nova | 11:11 | |
| *** harsha has quit IRC | 11:11 | |
| *** janki has quit IRC | 11:13 | |
| *** janki has joined #openstack-nova | 11:13 | |
| *** priteau has quit IRC | 11:15 | |
| *** jchhatbar has joined #openstack-nova | 11:19 | |
| *** moshele has quit IRC | 11:19 | |
| *** tuanla____ has joined #openstack-nova | 11:20 | |
| *** moshele has joined #openstack-nova | 11:21 | |
| *** janki has quit IRC | 11:22 | |
| *** gmann__ has joined #openstack-nova | 11:23 | |
| *** udesale has quit IRC | 11:24 | |
| openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/531737 | 11:30 |
| kashyap | tetsuro: KVM runs part of the guest instructions directly on the baremetal CPU (taking advantage of CPU Virtualization extensions - Intel-VMX / AMD-SVM). While QEMU provides rest of the PC-like components (n/w card, disk interface, etc) | 11:34 |
| *** ratailor__ has quit IRC | 11:35 | |
| *** sree has joined #openstack-nova | 11:35 | |
| *** udesale has joined #openstack-nova | 11:36 | |
| *** bkopilov has quit IRC | 11:36 | |
| openstackgerrit | Jie Li proposed openstack/nova master: Support volume-backed server rescue https://review.openstack.org/531524 | 11:36 |
| tetsuro | hrw, kashyap: good to know. thanks for the information. | 11:37 |
| tetsuro | sean-k-mooney, cdent: thank you for reviewing, let me wait for matt to leave comments here. | 11:38 |
| *** priteau has joined #openstack-nova | 11:38 | |
| sandanar | hi.. does anyone know what is the reason behind CONF.libvirt.volume_use_path always returns the default value and not the one passed in [libvirt] section of /etc/nova/nova.conf - https://bugs.launchpad.net/nova/+bug/1723928 this is causing multipath failures on the cinder side | 11:39 |
| openstack | Launchpad bug 1723928 in os-brick "In case of volume_use_multipath=True, Nova unable to fetch CONF.libvirt.volume_use_multipath value from nova.conf" [Undecided,New] | 11:39 |
| *** udesale has quit IRC | 11:39 | |
| *** udesale has joined #openstack-nova | 11:39 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination https://review.openstack.org/516168 | 11:40 |
| *** sree has quit IRC | 11:40 | |
| lyarwood | kashyap: http://paste.openstack.org/show/641098/ - just noticed this on stable/pike https://review.openstack.org/#/c/531407/ | 11:41 |
| * kashyap clicks | 11:41 | |
| lyarwood | kashyap: load of migration failed: Operation not permitted | 11:41 |
| kashyap | lyarwood: That rings a bell, I recall seeing a bug for the VQ size | 11:42 |
| * kashyap looks | 11:42 | |
| *** mvk has quit IRC | 11:45 | |
| *** kwathore has joined #openstack-nova | 11:45 | |
| kwathore | @meriedm: This is very critical bug which we have observerd releated to multipath: https://bugs.launchpad.net/nova/+bug/1723928 | 11:48 |
| openstack | Launchpad bug 1723928 in os-brick "In case of volume_use_multipath=True, Nova unable to fetch CONF.libvirt.volume_use_multipath value from nova.conf" [Undecided,New] | 11:48 |
| *** gmann__ has quit IRC | 11:49 | |
| hrw | hm. https://review.openstack.org/#/c/530965/ got rebased 2h ago and zuul still did not started | 11:50 |
| *** gmann__ has joined #openstack-nova | 11:53 | |
| openstackgerrit | Merged openstack/nova stable/newton: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530982 | 11:55 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405 | 11:55 |
| kashyap | lyarwood: From the top off my head, I vaguely recall a couple of fixes from Virtio net devices from StefanH. The bug links are eluding me | 11:56 |
| *** sree has joined #openstack-nova | 11:56 | |
| kashyap | I asked Stefan for the specific versions that fix these screwey virtio queue issues | 11:56 |
| kashyap | Will let ya know once I hear back from him | 11:57 |
| lyarwood | kashyap: cool thanks | 11:57 |
| openstackgerrit | Merged openstack/nova master: Add regression test for resize failing during retries https://review.openstack.org/531405 | 11:59 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache https://review.openstack.org/472773 | 12:00 |
| *** sree has quit IRC | 12:00 | |
| hrw | https://review.openstack.org/#/c/530965/ finally got +1 from Zuul. now just have to get second run of Zuul for +2 and merge ;d | 12:03 |
| openstackgerrit | sahid proposed openstack/nova master: hardware: only take into account small pages https://review.openstack.org/532168 | 12:03 |
| *** tuanla____ has quit IRC | 12:04 | |
| *** yamamoto has quit IRC | 12:06 | |
| *** avolkov has quit IRC | 12:06 | |
| *** yamamoto has joined #openstack-nova | 12:06 | |
| *** annp has quit IRC | 12:08 | |
| *** alexchadin has quit IRC | 12:14 | |
| *** alexchadin has joined #openstack-nova | 12:14 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination https://review.openstack.org/516168 | 12:16 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 12:19 |
| *** sree has joined #openstack-nova | 12:21 | |
| *** nicolasbock has joined #openstack-nova | 12:21 | |
| *** hareesh has joined #openstack-nova | 12:23 | |
| *** takedakn has joined #openstack-nova | 12:23 | |
| *** sree has quit IRC | 12:25 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 12:27 |
| ildikov | johnthetubaguy: gibi: hi :) this tiny patch needs a second +2 if either of you have a few minutes: https://review.openstack.org/#/c/529695/ | 12:28 |
| ildikov | johnthetubaguy: gibi: thanks in advance :) | 12:29 |
| *** moshele has quit IRC | 12:29 | |
| *** sree has joined #openstack-nova | 12:30 | |
| *** jpena is now known as jpena|lunch | 12:32 | |
| *** lucasagomes is now known as lucas-hungry | 12:33 | |
| *** hoonetorg has joined #openstack-nova | 12:34 | |
| *** hareesh has quit IRC | 12:34 | |
| *** sree has quit IRC | 12:35 | |
| *** sree has joined #openstack-nova | 12:38 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | 12:38 |
| gibi | ildikov: done :) | 12:39 |
| *** avolkov has joined #openstack-nova | 12:43 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate service status notification samples https://review.openstack.org/531381 | 12:44 |
| ildikov | gibi: thanks!! :) | 12:44 |
| *** mvk has joined #openstack-nova | 12:47 | |
| *** sree has quit IRC | 12:48 | |
| *** sree has joined #openstack-nova | 12:49 | |
| *** smatzek has joined #openstack-nova | 12:56 | |
| *** zhurong_ has joined #openstack-nova | 12:56 | |
| *** tetsuro has quit IRC | 13:02 | |
| *** hoonetorg has quit IRC | 13:03 | |
| *** udesale has quit IRC | 13:06 | |
| sean-k-mooney | stephenfin: just looking at https://review.openstack.org/#/c/531358 looks like when you change your email to stephenfin instead of sfinucan you didnt update your global git user name. not sure if thats intentional but just taught i would point it out | 13:06 |
| sean-k-mooney | * global git email | 13:06 |
| *** zhurong_ has quit IRC | 13:07 | |
| *** zhurong has quit IRC | 13:07 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | 13:08 |
| *** udesale has joined #openstack-nova | 13:08 | |
| *** dave-mccowan has joined #openstack-nova | 13:15 | |
| *** bkopilov has joined #openstack-nova | 13:17 | |
| *** rmart04 has joined #openstack-nova | 13:18 | |
| *** rabel__b1 has joined #openstack-nova | 13:21 | |
| *** dave-mccowan has quit IRC | 13:21 | |
| *** edmondsw has joined #openstack-nova | 13:21 | |
| *** alexchadin has quit IRC | 13:23 | |
| *** rabel_b1 has quit IRC | 13:24 | |
| *** alexchadin has joined #openstack-nova | 13:24 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate service status notification samples https://review.openstack.org/531381 | 13:25 |
| *** dave-mccowan has joined #openstack-nova | 13:26 | |
| mdbooth | Can anybody remind me about our stable backport window re https://review.openstack.org/#/c/530744/ | 13:26 |
| mdbooth | That's an Ocata backport. Is it going to land? | 13:27 |
| *** rabel_b1 has joined #openstack-nova | 13:27 | |
| mdbooth | s/going/eligible/ | 13:27 |
| mdbooth | stephenfin: ^^^ is your backport of my patch, btw | 13:27 |
| *** yangyapeng has quit IRC | 13:27 | |
| lyarwood | mdbooth: looking | 13:28 |
| efried | claudiub Wow, congratulations! That's been a long time coming. | 13:28 |
| sean-k-mooney | mdbooth: i think ocata is still supported for backports assuimg the patch would normally qualify. | 13:28 |
| *** rabel__b1 has quit IRC | 13:29 | |
| sean-k-mooney | mdbooth: the defult stable supprot is 12 months with the option to extend to 18 i think | 13:29 |
| stephenfin | sean-k-mooney: I've got both registered but need to configure 'user.email' on most repos | 13:29 |
| lyarwood | mdbooth / sean-k-mooney ; by the book it would need to be a >=high impact bug | 13:29 |
| stephenfin | sean-k-mooney: stephenfin is only an alias anyway so either one works | 13:29 |
| lyarwood | mdbooth: and I think this is valid still for ocata tbh | 13:29 |
| *** rabel__b1 has joined #openstack-nova | 13:30 | |
| mdbooth | lyarwood: Yeah, bug doesn't have that metadata, but I'd say it should qualify | 13:30 |
| sean-k-mooney | stephenfin: ya i remember you telling me that just was not sure if you missed it or not | 13:30 |
| lyarwood | mdbooth: yeah, had to read through the commit but yup IMHO it's fine, I'll review the pike version now | 13:30 |
| mdbooth | lyarwood: Thanks | 13:31 |
| *** lucas-hungry is now known as lucasagomes | 13:31 | |
| *** jpena|lunch is now known as jpena | 13:32 | |
| *** rabel_b1 has quit IRC | 13:32 | |
| sean-k-mooney | stephenfin: lyarwood: since we are talking about backport i would like to backport this to pike and ocata too its trival enough but its a latent bug that has been present since os-vif was first created | 13:33 |
| claudiub | efried: yeay, gonna have to celebrate. :)) | 13:33 |
| claudiub | the patches on nova still need reviews though. :D | 13:33 |
| sean-k-mooney | stephenfin: lyarwood by the book would this qualify for backporting? | 13:33 |
| stephenfin | sean-k-mooney: "this" being? | 13:33 |
| sean-k-mooney | stephenfin: https://review.openstack.org/#/c/531465/ | 13:33 |
| *** vladikr has joined #openstack-nova | 13:34 | |
| *** Tom-Tom has joined #openstack-nova | 13:35 | |
| *** rabel_b1 has joined #openstack-nova | 13:36 | |
| *** rabel__b1 has quit IRC | 13:38 | |
| lyarwood | sean-k-mooney: yeah I'd say that would be valid for ocata, it's trivial but useful. I'm not sure if there are other considerations that need to be taken into account for libs however. | 13:39 |
| sean-k-mooney | lyarwood: in normal execution i think the edgecase that causes the interface not to be present happens very infrequently however in the gate it was causing random failures hence why it has been latent for so long | 13:41 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505 | 13:45 |
| *** phuongnh has quit IRC | 13:47 | |
| *** esberglu has joined #openstack-nova | 13:48 | |
| *** gmann__ has quit IRC | 13:50 | |
| *** AlexeyAbashkin has quit IRC | 13:52 | |
| kashyap | lyarwood: So apparently there are different reasons for hitting that error you pointed out earlier. (The bug URL I was looking for was, which has a slightly different error): https://bugzilla.redhat.com/show_bug.cgi?id=1378788 (Race condition during virtio-blk dataplane stop triggers "Virtqueue size exceeded") | 13:53 |
| openstack | bugzilla.redhat.com bug 1378788 in qemu-kvm-rhev "Race condition during virtio-blk dataplane stop triggers "Virtqueue size exceeded"" [High,Closed: errata] - Assigned to stefanha | 13:53 |
| *** mriedem has joined #openstack-nova | 13:54 | |
| *** jchhatbar has quit IRC | 13:54 | |
| *** yangyapeng has joined #openstack-nova | 13:54 | |
| mriedem | dtantsur: the devstack dependency for https://review.openstack.org/#/c/527289/ is now merged | 13:55 |
| dtantsur | mriedem: thanks, approved | 13:55 |
| *** sree has quit IRC | 13:55 | |
| mriedem | thanks | 13:55 |
| lyarwood | kashyap: cool thanks, /me reads | 13:56 |
| *** sree has joined #openstack-nova | 13:56 | |
| kashyap | lyarwood: But we still don't have a root cause for the unique error you pointed out | 13:56 |
| kashyap | The above bug was fixed in version QEMU 2.6 & above, though. | 13:57 |
| lyarwood | kashyap: kk, this is with 2.8 FWIW | 13:57 |
| lyarwood | kashyap: do we need a bug open somewhere for this in that case? | 13:57 |
| kashyap | lyarwood: Hmm, okay, Stefan just responded: | 13:57 |
| kashyap | "This issue probably requires debugging. This kind of problem can occur if the vring memory is not up-to-date (e.g. requests were processed after migration paused the guest)." | 13:58 |
| kashyap | lyarwood: Do you see multiple hits for this? | 13:58 |
| kashyap | lyarwood: Yeah, an open bug helps definitely, | 13:58 |
| *** takedakn has quit IRC | 13:58 | |
| kashyap | ...as others hitting it may have more reliable reproducers. | 13:59 |
| lyarwood | kashyap: I've not looked, I'll check now, if we do then I'll create a LP for it | 13:59 |
| *** AlexeyAbashkin has joined #openstack-nova | 14:00 | |
| *** sree has quit IRC | 14:00 | |
| *** alexchadin has quit IRC | 14:06 | |
| *** kholkina has quit IRC | 14:06 | |
| edmondsw | mriedem and sdague, the powervm OVS patch is fully ready for reviews now, passing CI | 14:07 |
| edmondsw | https://review.openstack.org/#/c/422512/ | 14:07 |
| edmondsw | we're working a CI issue for the SEA patch this morning, but as stated yesterday the code is ready for review | 14:07 |
| edmondsw | will update on the CI for SEA when we have that fixed | 14:07 |
| mriedem | i would expect efried to review that first | 14:09 |
| efried | will do | 14:09 |
| lyarwood | kashyap: nvm we already have a LP from mriedem for this here - https://bugs.launchpad.net/nova/+bug/1737625 | 14:10 |
| openstack | Launchpad bug 1737625 in OpenStack Compute (nova) "live migration failed with "Failed to load virtio-blk:virtio" (libvirt/qemu)" [Undecided,Confirmed] | 14:10 |
| *** alexchadin has joined #openstack-nova | 14:10 | |
| kashyap | lyarwood: Ah, thanks. | 14:11 |
| *** rmart04 has quit IRC | 14:13 | |
| *** avolkov has quit IRC | 14:13 | |
| *** amodi has joined #openstack-nova | 14:13 | |
| *** moshele has joined #openstack-nova | 14:14 | |
| efried | mriedem Done | 14:19 |
| *** jroll has quit IRC | 14:19 | |
| efried | (OVS, that is) | 14:19 |
| *** jroll has joined #openstack-nova | 14:19 | |
| *** jmlowe has quit IRC | 14:19 | |
| *** abhishekk has joined #openstack-nova | 14:20 | |
| *** READ10 has joined #openstack-nova | 14:23 | |
| *** moshele has quit IRC | 14:24 | |
| stephenfin | mriedem: Our Python 3 woes have been resolved https://review.openstack.org/#/c/530950/ | 14:28 |
| *** psachin has quit IRC | 14:28 | |
| stephenfin | mriedem: Fancy revisiting https://review.openstack.org/#/c/345399/ at some point this week? | 14:28 |
| *** smatzek has quit IRC | 14:28 | |
| *** smatzek has joined #openstack-nova | 14:28 | |
| *** smatzek has quit IRC | 14:28 | |
| *** jmlowe has joined #openstack-nova | 14:30 | |
| *** rabel__b1 has joined #openstack-nova | 14:31 | |
| *** rabel_b1 has quit IRC | 14:32 | |
| *** cleong has joined #openstack-nova | 14:33 | |
| *** moshele has joined #openstack-nova | 14:34 | |
| *** jmlowe has quit IRC | 14:34 | |
| *** jmlowe has joined #openstack-nova | 14:35 | |
| *** jmlowe has quit IRC | 14:36 | |
| *** felipemonteiro_ has joined #openstack-nova | 14:36 | |
| *** yamamoto_ has joined #openstack-nova | 14:37 | |
| *** moshele has quit IRC | 14:38 | |
| ildikov | mriedem: are you planning to do further tweaks on the multi-attach chain in Nova? | 14:40 |
| *** yamamoto has quit IRC | 14:40 | |
| *** markvoelker has quit IRC | 14:40 | |
| *** eharney has joined #openstack-nova | 14:41 | |
| mriedem | stephenfin: maybe | 14:42 |
| mriedem | ildikov: maybe | 14:42 |
| ildikov | mriedem: ok, I started to look into the unit test failures, the first half is an easy fix, but won't touch the code for now then | 14:43 |
| stephenfin | mriedem: Good enough for me | 14:43 |
| mriedem | ildikov: i posted it at 11pm and had fixed what i knew about, but haven't looked at the results yet this morning | 14:44 |
| ildikov | mriedem: http://logs.openstack.org/47/271047/48/check/openstack-tox-py27/991a4d8/testr_results.html.gz | 14:45 |
| mriedem | i'll look later, in the middle of something | 14:45 |
| *** kaisers has quit IRC | 14:45 | |
| ildikov | mriedem: I can fix these, upload and then we can agree who does what onwards, if that helps | 14:45 |
| mriedem | sure | 14:45 |
| ildikov | mriedem: ok, will do that then, tnx | 14:46 |
| ildikov | mriedem: I just wanted to ask to avoid overwriting each others' versions... | 14:46 |
| mriedem | yup | 14:47 |
| *** kaisers has joined #openstack-nova | 14:51 | |
| mriedem | mdbooth: clearly this has to be wrong right? https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1669 | 14:53 |
| mriedem | and source_type not in ('lvm') | 14:53 |
| * mdbooth looks | 14:53 | |
| mriedem | i guess it does work as a substring match | 14:54 |
| mriedem | probably by accident | 14:54 |
| mdbooth | mriedem: I hadn't even spotted the 'strings are a list' thing... | 14:55 |
| * mdbooth was still ponding the comment above | 14:55 | |
| *** avolkov has joined #openstack-nova | 14:55 | |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | 14:56 |
| * mdbooth sent a long message: mdbooth_2018-01-09_14:56:23.txt <https://matrix.devlab.redhat.com/_matrix/media/v1/download/redhat.com/yUjvJDUFNKjpsnwVLtqVCaoN> | 14:56 | |
| *** alexchadin has quit IRC | 14:56 | |
| mdbooth | Yeah, looks like it probably works accidentally | 14:56 |
| mdbooth | >>> 'foo' in ('foobar',) | 14:57 |
| mdbooth | False | 14:57 |
| mriedem | yeah, just getting another set of eyes, just noticed it while looking at something else | 14:58 |
| *** awaugama has joined #openstack-nova | 14:59 | |
| *** gouthamr has joined #openstack-nova | 15:00 | |
| *** liverpooler has joined #openstack-nova | 15:01 | |
| *** nicolasbock has quit IRC | 15:02 | |
| alex_xu_ | sdague: as I remember we forze the '/os-migrations' API, then we want to add links in the '/os-migrations' API to the '/servers/{uuid}/migrations', is it right? https://review.openstack.org/#/c/459483/ | 15:02 |
| mriedem | alex_xu_: yes, but that was pretty short sighted | 15:03 |
| mriedem | as that change was specifically for the change to abort a running live migration for a given instance | 15:03 |
| mriedem | there is still use in the os-migrations API | 15:03 |
| *** sree has joined #openstack-nova | 15:03 | |
| *** vladikr has quit IRC | 15:03 | |
| mriedem | for example, it's the only one that you can use to list not in progress migrations | 15:03 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: rbd: flatten images when creating/unshelving an instance https://review.openstack.org/457886 | 15:06 |
| *** diga has joined #openstack-nova | 15:06 | |
| *** lyan has joined #openstack-nova | 15:09 | |
| mriedem | dansmith: i'm +2 on the migration paging/filtering series https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/add-pagination-and-change-since-for-migration-list | 15:11 |
| mriedem | that includes the cross-cell lister | 15:11 |
| *** smatzek has joined #openstack-nova | 15:11 | |
| *** smatzek has quit IRC | 15:11 | |
| dansmith | mriedem: okay | 15:11 |
| *** smatzek has joined #openstack-nova | 15:11 | |
| *** tidwellr has joined #openstack-nova | 15:12 | |
| *** markvoelker has joined #openstack-nova | 15:13 | |
| *** markvoelker has quit IRC | 15:16 | |
| *** hoonetorg has joined #openstack-nova | 15:16 | |
| *** sree has quit IRC | 15:17 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: don't attempt to live snapshot paused instances https://review.openstack.org/532214 | 15:19 |
| mriedem | sdague: does this look familiar? ^ i now remember why i added the PAUSED condition to the patch to enable libvirt live snapshot by default | 15:20 |
| mriedem | it doesn't work on older libvirt/qemu | 15:20 |
| *** jmlowe has joined #openstack-nova | 15:20 | |
| *** jmlowe has quit IRC | 15:21 | |
| *** abhishekk has quit IRC | 15:21 | |
| *** jmlowe has joined #openstack-nova | 15:21 | |
| *** armax has joined #openstack-nova | 15:21 | |
| *** jmlowe has quit IRC | 15:22 | |
| *** jmlowe has joined #openstack-nova | 15:22 | |
| *** lajoskatona has quit IRC | 15:22 | |
| *** sree has joined #openstack-nova | 15:25 | |
| openstackgerrit | sahid proposed openstack/nova master: hardware: only take into account small pages https://review.openstack.org/532168 | 15:25 |
| *** markvoelker has joined #openstack-nova | 15:25 | |
| *** rmart04 has joined #openstack-nova | 15:27 | |
| *** kumarmn has joined #openstack-nova | 15:27 | |
| *** sree has quit IRC | 15:29 | |
| mriedem | stephenfin: simple reno update https://review.openstack.org/#/c/531963/ | 15:29 |
| mriedem | i was reading the release notes last night and when you put them together, the context is lost on some of them | 15:30 |
| kashyap | mriedem: A quick aside: Follow up from your test that the 'shareable' disk flag "doesn't work when using raw images either http://paste.openstack.org/show/639185/" | 15:30 |
| kashyap | It was a real bug apparently and fixed in libvirt 3.10 and above. (All the messy dependent bugs are linked here: https://bugzilla.redhat.com/show_bug.cgi?id=1378242) (libvirt patch series: "[PATCH v2 00/11] qemu: Fix <shareable/> disks for new qemu") | 15:30 |
| openstack | bugzilla.redhat.com bug 1378242 in libvirt "QEMU image file locking (libvirt)" [Unspecified,On_qa] - Assigned to pkrempa | 15:30 |
| mriedem | kashyap: ok just reading https://bugzilla.redhat.com/show_bug.cgi?id=1378242#c21 | 15:31 |
| openstack | bugzilla.redhat.com bug 1378242 in libvirt "QEMU image file locking (libvirt)" [Unspecified,On_qa] - Assigned to pkrempa | 15:31 |
| mriedem | kashyap: but yeah, for now the change for multiattach in the libvirt driver only supports multiattach if qemu<2.10 or libvirt>=3.10 | 15:32 |
| kashyap | mriedem: BTW, about that snapshots and paused instances; I already thought it was fixed earlier, maybe I'm hallucinating | 15:32 |
| mriedem | https://review.openstack.org/#/c/267587/83/nova/virt/libvirt/driver.py | 15:32 |
| kashyap | But see the version dependancy you document in the commit message. | 15:32 |
| kashyap | mriedem: Noted (about the multi-attach change) | 15:33 |
| *** Apoorva has joined #openstack-nova | 15:33 | |
| mriedem | as for this test https://review.openstack.org/#/c/267587/83/nova/tests/unit/virt/libvirt/test_driver.py@19520 - i can't sort out the global mock nature that makes it fail when running the entire test module | 15:33 |
| * kashyap needs to take train in 10 minutes to go home; if I go radio silent or don't respond, you know why. | 15:33 | |
| mriedem | https://review.openstack.org/#/c/267587/83/nova/tests/unit/virt/libvirt/test_driver.py@19520 is a challenge for mdbooth | 15:33 |
| mdbooth | mriedem: Have you done the bisect thing? | 15:35 |
| mriedem | mdbooth: nope | 15:35 |
| * mdbooth always forgets the details | 15:35 | |
| mdbooth | I believe the instructions were originally written by sdague | 15:35 |
| mriedem | i thought that splitting the tests out into it's own class would help so they weren't running with the other tests in LibvirtConnTestCase but that didn't help | 15:35 |
| *** sree has joined #openstack-nova | 15:36 | |
| *** links has quit IRC | 15:36 | |
| mdbooth | mriedem: This one: https://wiki.openstack.org/wiki/Testr#Reproducing_Failures | 15:36 |
| mriedem | oh cool, i'll try that | 15:37 |
| mdbooth | The output of that process is a minimal test ordering which causes the failure. | 15:37 |
| mdbooth | Then you just squint at it for a bit. | 15:37 |
| mriedem | johnthetubaguy: can you drop the -2 on this? https://review.openstack.org/#/c/140733/ | 15:38 |
| mriedem | johnthetubaguy: also, i plumbed in the multiattach API functional changes last night - https://review.openstack.org/#/c/271047/ - still need to cleanup some test fallout from some refactor that was needed, but it should be ok to start reviewing the flow | 15:38 |
| johnthetubaguy | mriedem: done | 15:38 |
| mriedem | you have to use a new microversion to bfv or attach with a multiattach volume, | 15:39 |
| johnthetubaguy | mriedem: ah cool, I am chasing some ironic traits stuff today, but will try get to that soon | 15:39 |
| mriedem | and then we have min compute version checks and such | 15:39 |
| johnthetubaguy | mriedem: all sounds sensible | 15:39 |
| mriedem | it got a bit gross because i had to change a volume_id param to a volume dict in one of the lower down methods | 15:39 |
| mriedem | to avoid multiple volume GET calls | 15:40 |
| ildikov | mriedem: johnthetubaguy: the functional tests run into a KeyError on the volume which should be an easy fix if I can finally find the spot where it actually goes wrong... :) | 15:41 |
| *** ameeda has quit IRC | 15:41 | |
| *** hoonetorg has quit IRC | 15:41 | |
| mriedem | ildikov: it's probably a GET response from the CinderFixture | 15:41 |
| ildikov | mriedem: was it multiple GET calls? | 15:41 |
| *** breton has joined #openstack-nova | 15:42 | |
| ildikov | mriedem: the fakes returns a volume dict with 'multiattach' | 15:42 |
| breton | hi | 15:42 |
| ildikov | mriedem: or I found another volume dict that contained it... | 15:42 |
| *** sree has quit IRC | 15:42 | |
| mriedem | ildikov: https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L1332 | 15:42 |
| mriedem | ^ doesn't contain multiattach=False | 15:42 |
| mriedem | nor do the ones below | 15:43 |
| mriedem | same thing in CinderFixtureNewAttachFlow | 15:43 |
| mriedem | ildikov: i could have made the API code a lot of easier on tests if i just did volume.get('multiattach') but that's a cop out since the volume response from cinder's API should always have that key set | 15:43 |
| *** david-lyle has joined #openstack-nova | 15:44 | |
| breton | Is liberty affected by https://bugs.launchpad.net/nova/+bug/1664931? Is there any sense in porting https://review.openstack.org/#/c/519684/7 ? | 15:44 |
| openstack | Launchpad bug 1664931 in OpenStack Compute (nova) pike "[OSSA-2017-005] nova rebuild ignores all image properties and scheduler filters (CVE-2017-16239)" [High,Fix committed] - Assigned to Matt Riedemann (mriedem) | 15:44 |
| ildikov | mriedem: the test calls this: https://github.com/openstack/nova/blob/master/nova/tests/unit/api/openstack/fakes.py#L618 | 15:45 |
| mriedem | breton: the OSSA-2017-005 calls out the affected versions | 15:45 |
| ildikov | mriedem: which then goes here: https://github.com/openstack/nova/blob/master/nova/tests/unit/api/openstack/fakes.py#L568 | 15:45 |
| *** felipemonteiro__ has joined #openstack-nova | 15:45 | |
| mriedem | ildikov: i thought you were asking about functional tests | 15:45 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/unit/api/openstack/fakes.py#L586 should have probably never been True | 15:46 |
| breton | mriedem: so do i understand it right that all versions <=14.0.10 are affected? | 15:46 |
| ildikov | mriedem: I do | 15:46 |
| ildikov | mriedem: https://github.com/openstack/nova/blob/master/nova/tests/functional/api_sample_tests/test_volumes.py#L247 | 15:47 |
| ildikov | mriedem: and just the old flow tests fail | 15:48 |
| mriedem | breton: https://github.com/openstack/nova/commit/d6ca1cc02ab58615a6bb1b337db34dd06525112e introduced the regression, | 15:48 |
| breton | mriedem: i am unsure because in liberty request specs were not used for instance rebuild | 15:48 |
| mriedem | in 12.0.0 which was liberty | 15:48 |
| ildikov | mriedem: and I'm surely just blind and hate all the fake stuff... | 15:48 |
| *** felipemonteiro_ has quit IRC | 15:49 | |
| mriedem | ildikov: oh i didn't realize the functional tests were using unit test stubs.... | 15:49 |
| mriedem | they should be using the CinderFixture, but that's not a change to make here | 15:49 |
| ildikov | I got surprised too :) | 15:50 |
| breton | mriedem: understood, thanks | 15:50 |
| *** diga has quit IRC | 15:50 | |
| mriedem | ildikov: https://github.com/openstack/nova/blob/master/nova/tests/functional/api_sample_tests/test_volumes.py#L368 | 15:51 |
| mriedem | the 2.49 test is using the CinderFixture as well as the fakes/stubs | 15:51 |
| mriedem | and the fixture is loaded after the stubs in the parent class setup | 15:52 |
| mriedem | so the fixture overrides the GET call | 15:52 |
| mriedem | and the fixture doesn't have multiattach in the response | 15:52 |
| mriedem | so just fix the fixtures since you have to do that anyway | 15:52 |
| ildikov | the test seemed to call it explicitely | 15:52 |
| ildikov | anyway, will fix that | 15:52 |
| mriedem | the test calling fakes.stub_volume_get and the actual api runtime code calling it via the fixture are different thigns | 15:53 |
| mriedem | *things | 15:53 |
| ildikov | or in other words it's a mess :) | 15:53 |
| mriedem | yeah, the functional tests shouldn't be using stubs from the unit tests, they should use the fixture long-term | 15:54 |
| mriedem | it's ultimately trading one set of stubs for another | 15:54 |
| *** hoonetorg has joined #openstack-nova | 15:54 | |
| *** sree has joined #openstack-nova | 15:55 | |
| *** mlavalle has joined #openstack-nova | 15:56 | |
| *** slaweq has joined #openstack-nova | 15:56 | |
| *** mdnadeem has quit IRC | 15:57 | |
| smcginnis | mriedem: Are these Cinder or Nova tests you are referring to? | 15:57 |
| mriedem | nova | 15:57 |
| *** tovin07 has joined #openstack-nova | 15:57 | |
| Roamer` | johnthetubaguy, now that mriedem seems to be kinda sorta okay with the StorPool libvirt volume driver (thanks!) and it passed the Zuul checks (yay!), could you remove your procedural -2? thanks in advance! | 15:57 |
| smcginnis | OK, good. | 15:57 |
| mriedem | Roamer`: he already did | 15:57 |
| Roamer` | mriedem, johnthetubaguy, ahhhhh, right... he did indeed... thanks! | 15:58 |
| Roamer` | (today is NOT my day for reading and parsing stuff) | 15:58 |
| *** sree has quit IRC | 15:59 | |
| ildikov | smcginnis: got scared for a moment? :) | 16:02 |
| *** felipemonteiro__ has quit IRC | 16:03 | |
| *** felipemonteiro__ has joined #openstack-nova | 16:03 | |
| *** yamahata has joined #openstack-nova | 16:03 | |
| *** janki has joined #openstack-nova | 16:03 | |
| *** sandanar has quit IRC | 16:03 | |
| smcginnis | ildikov: Yep. ;) | 16:04 |
| smcginnis | ildikov: Our tests already need a lot of work. I was just hoping that it wasn't worse than I already knew. | 16:04 |
| ildikov | smcginnis: I know that part :) | 16:05 |
| ildikov | smcginnis: but you know no one is perfect, even Nova has it's flows :) | 16:05 |
| ildikov | s/it's/its/ | 16:05 |
| smcginnis | ildikov: Shh, you can't say that over here. :D | 16:05 |
| *** rmart04 has quit IRC | 16:06 | |
| ildikov | smcginnis: they all know I'm just expressing my love :) | 16:06 |
| *** markvoelker has quit IRC | 16:09 | |
| *** stvnoyes has joined #openstack-nova | 16:11 | |
| tovin07 | mriedem, can you take a quick look at this small fix (already had +2 from stephenfin): https://review.openstack.org/#/c/519664/ | 16:11 |
| *** jackie-truong has joined #openstack-nova | 16:11 | |
| stephenfin | mriedem: Done (the release note) | 16:13 |
| openstackgerrit | Merged openstack/nova master: Move aggregates from report client to ProviderTree https://review.openstack.org/521685 | 16:13 |
| *** hongbin has joined #openstack-nova | 16:17 | |
| *** lpetrut has quit IRC | 16:18 | |
| bauzas | is it me or the gate is in the weeds ? | 16:19 |
| *** itlinux has joined #openstack-nova | 16:19 | |
| bauzas | I'm still waiting a single change to be merged since this morning | 16:19 |
| *** udesale has quit IRC | 16:20 | |
| *** harsha has joined #openstack-nova | 16:24 | |
| mriedem | edleafe: comments on the alternate hosts patch for resize https://review.openstack.org/#/c/526436/21 | 16:26 |
| mriedem | bauzas: thankfully you can take this opportunity to review some patches | 16:27 |
| hrw | guys: where 'default' VM is defined? Nova decides what to include or does it just use defaults from libvirt? | 16:27 |
| efried | bauzas Yeah, https://review.openstack.org/#/c/521685/ took 8.5h to merge this morning. | 16:27 |
| hrw | I mean things like networking, graphics, usb, storage. not amount of ram/vcpu/storage | 16:28 |
| openstackgerrit | Merged openstack/nova master: Fix race condition in retrying migrations https://review.openstack.org/531022 | 16:29 |
| *** tidwellr has quit IRC | 16:29 | |
| bauzas | mriedem: yup, I'm just working on a new PS for the vGPU change, but I hope to look at reviewing changes by tomorrow | 16:30 |
| *** Tom-Tom has quit IRC | 16:30 | |
| *** tidwellr has joined #openstack-nova | 16:32 | |
| stephenfin | lyarwood: Have you got this the right way round? https://review.openstack.org/#/c/530744/ | 16:32 |
| stephenfin | Wait - O comes before P | 16:33 |
| * stephenfin considers trying coffee | 16:33 | |
| lyarwood | :D | 16:34 |
| *** markvoelker has joined #openstack-nova | 16:37 | |
| mriedem | tovin07: done | 16:37 |
| edleafe | mriedem: thx. Found a bug in my own code, so I'm working on fixing it | 16:38 |
| * edleafe loves functional tests | 16:38 | |
| cdent | functional tests love you more | 16:39 |
| *** yamamoto_ has quit IRC | 16:40 | |
| *** READ10 has quit IRC | 16:41 | |
| mriedem | efried: powervm ci is going to use SEA rather than OVS for networking right? | 16:42 |
| efried | For the immediate future, yes. esberglu ^ | 16:42 |
| mriedem | so what networking is this running against? https://review.openstack.org/#/c/422512/ | 16:42 |
| mriedem | if SEA isn't yet there | 16:42 |
| efried | I forget that answer. esberglu is on the way... | 16:43 |
| mriedem | Configured mechanism driver names: ['openvswitch', 'linuxbridge'] | 16:43 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Detach volumes on _rebuild_default_impl failure https://review.openstack.org/442105 | 16:43 |
| mriedem | so maybe just the defaults for ML2? | 16:43 |
| lyarwood | stephenfin / mdbooth; if you have any review time left today https://review.openstack.org/#/c/460243/ has been respun to include the fix from https://review.openstack.org/#/c/531233/ | 16:45 |
| *** felipemonteiro_ has joined #openstack-nova | 16:45 | |
| esberglu | mriedem: That's just running the CI we've been using since last release with no networking implementation | 16:46 |
| esberglu | About to merge a patch that fixes SEA CI | 16:46 |
| efried | esberglu IOW, neither network patch is yet running a CI that tests networking, but soon the SEA patch will run CI with SEA networking? | 16:48 |
| efried | esberglu And... if we were to run the OVS patch once that is done, it would what, fail? | 16:48 |
| *** felipemonteiro__ has quit IRC | 16:49 | |
| esberglu | efried: No it would pass, but it would be testing SEA networking | 16:49 |
| efried | esberglu How, without the SEA patch? | 16:49 |
| esberglu | efried: PowerVM is running SEA CI on all nova changes as of yesterday | 16:49 |
| efried | esberglu How, without the SEA patch? | 16:49 |
| esberglu | It is applying the SEA patch | 16:50 |
| efried | ahh | 16:50 |
| esberglu | Hence the issue applying the SEA patch to the OVS and SEA changes, which we just fixed | 16:50 |
| efried | that's what we were *just* working on, yeah :) | 16:50 |
| efried | pieces come together. | 16:50 |
| efried | mriedem Does this answer your question? | 16:51 |
| *** damien_r has quit IRC | 16:51 | |
| *** chyka has joined #openstack-nova | 16:52 | |
| mriedem | huh | 16:52 |
| mriedem | ok | 16:52 |
| mriedem | typically applying patches to the CI results for code that's not yet merged is a bit a of a no-no | 16:52 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | 16:53 |
| efried | mriedem We have a chicken/egg here | 16:54 |
| *** kumarmn has left #openstack-nova | 16:55 | |
| mriedem | well, that egg wasn't necessary in pike was it? | 16:55 |
| esberglu | mriedem: No we did this in pike as well | 16:55 |
| mriedem | or is the chicken the os-vif stuff added with the ovs change, which the SEA change relies on? | 16:55 |
| mriedem | oh fun | 16:55 |
| *** smatzek has quit IRC | 16:56 | |
| *** itlinux has quit IRC | 16:56 | |
| *** smatzek has joined #openstack-nova | 16:57 | |
| efried | mriedem Yeah, the SEA change is on top of the OVS change. Technically we would probably be able to wait to switch on the patching-and-SEA-networking part of the CI until after the OVS patch merges, but for the sake of getting both patches in the review pipeline as quickly as possible... | 16:57 |
| *** smatzek_ has joined #openstack-nova | 16:59 | |
| *** smatzek_ has quit IRC | 17:00 | |
| *** smatzek_ has joined #openstack-nova | 17:00 | |
| *** gyee has joined #openstack-nova | 17:01 | |
| *** smatzek has quit IRC | 17:02 | |
| *** slaweq has quit IRC | 17:02 | |
| *** AlexeyAbashkin has quit IRC | 17:02 | |
| *** tbachman has quit IRC | 17:03 | |
| gibi | mriedem: have you planned to join to the notification subteam meeting? | 17:03 |
| *** sridharg has quit IRC | 17:03 | |
| stephenfin | claudiub: Want to look at https://review.openstack.org/#/c/447505/ so I can sign off on it? | 17:04 |
| *** vivsoni__ has joined #openstack-nova | 17:05 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP: [api] Allow multi-attach in compute api https://review.openstack.org/271047 | 17:05 |
| *** armax has quit IRC | 17:05 | |
| *** karthiks has quit IRC | 17:05 | |
| *** janki has quit IRC | 17:06 | |
| *** mvk has quit IRC | 17:06 | |
| *** mdnadeem has joined #openstack-nova | 17:06 | |
| gibi | mriedem: as nobody joined, I've just closed the meeting, we can discuss thing separatly if neede | 17:07 |
| *** alex_xu_ has quit IRC | 17:07 | |
| mriedem | gibi: sorry missed your ping | 17:08 |
| *** hongbin has quit IRC | 17:08 | |
| *** hongbin has joined #openstack-nova | 17:08 | |
| stephenfin | lyarwood: Looked, but my head is gone. I'll hit it tomorrow morning | 17:08 |
| *** alex_xu has joined #openstack-nova | 17:08 | |
| lyarwood | stephenfin: no issues | 17:08 |
| *** jackie-truong has quit IRC | 17:12 | |
| *** felipemonteiro_ has quit IRC | 17:13 | |
| *** smatzek_ has quit IRC | 17:15 | |
| *** smatzek has joined #openstack-nova | 17:15 | |
| *** lucasagomes is now known as lucas-afk | 17:16 | |
| *** tovin07 has quit IRC | 17:16 | |
| *** moshele has joined #openstack-nova | 17:18 | |
| *** itlinux has joined #openstack-nova | 17:18 | |
| claudiub | stephenfin: done. just fyi. the patch merged in oslotest, and it is included in oslotest 3.2.0. right now, I'm still waiting for the global-requirements bump for oslotest, and then for the OpenStack Bot requirements update to be proposed to nova and be merged, Then finally, that patch is mergeable. :) | 17:18 |
| efried | bauzas You still working on series https://review.openstack.org/#/c/528832/ locally? I see "Done" comments from ~8h ago but no new patch set. | 17:19 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | 17:20 |
| *** tbachman has joined #openstack-nova | 17:21 | |
| *** smatzek has quit IRC | 17:21 | |
| *** matrohon has quit IRC | 17:22 | |
| *** rabel__b1 has quit IRC | 17:22 | |
| *** penick has joined #openstack-nova | 17:23 | |
| *** david-lyle has quit IRC | 17:23 | |
| *** jaosorior has quit IRC | 17:25 | |
| *** david-lyle has joined #openstack-nova | 17:26 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: test to make sure volume_use_multipath is properly used https://review.openstack.org/532270 | 17:27 |
| *** sree has joined #openstack-nova | 17:27 | |
| bauzas | efried: yup, I'm in there | 17:27 |
| efried | k. | 17:28 |
| *** smatzek has joined #openstack-nova | 17:29 | |
| bauzas | dansmith: around ? I have a thought | 17:29 |
| bauzas | dansmith: I'm just building a dictionary of assigned mdevs but looking up the guests by init_host | 17:29 |
| bauzas | dansmith: when I'm assigning a mdev to a guest, I'm also amending that dict | 17:30 |
| *** smatzek has quit IRC | 17:30 | |
| *** smatzek has joined #openstack-nova | 17:30 | |
| efried | bauzas Re+W https://review.openstack.org/#/c/466051/ after rebase? | 17:30 |
| bauzas | dansmith: but for example, say we migrate the instance (or just destroy it), I should call the method recreating the dict | 17:30 |
| bauzas | dansmith: or we could have a mdev said to be assigned while the guest is destroyed | 17:31 |
| *** harsha has quit IRC | 17:31 | |
| *** smatzek has quit IRC | 17:31 | |
| *** Apoorva has quit IRC | 17:31 | |
| bauzas | dansmith: do you think it would be better to call that method by each method (like migrate, rebuild and destroy) or just use a periodic method ? | 17:32 |
| *** smatzek has joined #openstack-nova | 17:32 | |
| *** tidwellr has quit IRC | 17:32 | |
| dansmith | bauzas: um, you're not persisting that dict are you? | 17:32 |
| bauzas | if you don't understand my concern, no worries, I'll upload the series soon | 17:32 |
| *** tidwellr has joined #openstack-nova | 17:32 | |
| bauzas | dansmith: by persisting, you mean what ? using an attribute ? | 17:33 |
| dansmith | I mean... persisting it.. I'm not sure what you'd need to update periodically, | 17:33 |
| dansmith | but maybe just upload and we can talk about it concretely | 17:33 |
| bauzas | nevermind, I'll upload | 17:33 |
| bauzas | sure, just working on the tests now | 17:33 |
| bauzas | efried: +Wd | 17:34 |
| efried | thx | 17:34 |
| openstackgerrit | Merged openstack/nova master: Add an additional description for 'token_ttl' https://review.openstack.org/532007 | 17:35 |
| dansmith | mriedem: did you check that this is actually using multiple cells? https://review.openstack.org/#/c/517273/39/nova/tests/functional/compute/test_migration_list.py | 17:35 |
| *** smatzek has quit IRC | 17:36 | |
| *** smatzek has joined #openstack-nova | 17:37 | |
| *** yamahata has quit IRC | 17:38 | |
| mriedem | dansmith: i think he copied that from your instance list tests, but it doesn't look like it explicitly creates multiple cells | 17:38 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/functional/compute/test_instance_list.py#L25 | 17:39 |
| mriedem | he needs that | 17:39 |
| mriedem | good point | 17:39 |
| dansmith | but I set | 17:39 |
| dansmith | yeah :) | 17:39 |
| mriedem | removed my +2 | 17:39 |
| *** smatzek has quit IRC | 17:39 | |
| *** tbachman has quit IRC | 17:39 | |
| mriedem | lay down the -1 | 17:39 |
| dansmith | I done did | 17:40 |
| *** smatzek has joined #openstack-nova | 17:40 | |
| *** moshele has quit IRC | 17:40 | |
| *** yamamoto has joined #openstack-nova | 17:41 | |
| *** mdnadeem has quit IRC | 17:41 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: add tests to check multipath in iscsi/fc volume connectors https://review.openstack.org/532272 | 17:44 |
| *** tbachman has joined #openstack-nova | 17:44 | |
| *** smatzek has quit IRC | 17:44 | |
| *** sree has quit IRC | 17:46 | |
| *** smatzek has joined #openstack-nova | 17:46 | |
| mriedem | dansmith: per https://review.openstack.org/#/c/517273/39/nova/db/sqlalchemy/api.py@4410 did you want to hold up https://review.openstack.org/#/c/514904/ ? | 17:47 |
| dansmith | no I just wanted to make the comment to sound like I'm paying attention | 17:47 |
| *** markvoelker has quit IRC | 17:47 | |
| mriedem | heh, ok | 17:47 |
| *** sree has joined #openstack-nova | 17:49 | |
| *** sahid has quit IRC | 17:49 | |
| *** yamamoto has quit IRC | 17:49 | |
| *** smatzek has quit IRC | 17:50 | |
| *** smatzek has joined #openstack-nova | 17:51 | |
| *** sree has quit IRC | 17:54 | |
| *** smatzek has quit IRC | 17:56 | |
| *** tesseract has quit IRC | 17:57 | |
| *** sree has joined #openstack-nova | 17:58 | |
| *** edmondsw has quit IRC | 17:58 | |
| *** smatzek has joined #openstack-nova | 17:58 | |
| *** derekh has quit IRC | 18:00 | |
| *** edmondsw has joined #openstack-nova | 18:01 | |
| *** Apoorva has joined #openstack-nova | 18:02 | |
| *** Apoorva has quit IRC | 18:02 | |
| *** smatzek has quit IRC | 18:03 | |
| *** smatzek has joined #openstack-nova | 18:03 | |
| *** edmondsw has quit IRC | 18:06 | |
| *** david-lyle has quit IRC | 18:06 | |
| *** penick has quit IRC | 18:10 | |
| *** satyar has joined #openstack-nova | 18:14 | |
| *** yamahata has joined #openstack-nova | 18:16 | |
| *** dtantsur is now known as dtantsur|afk | 18:17 | |
| *** ralonsoh has quit IRC | 18:21 | |
| *** avolkov has quit IRC | 18:21 | |
| *** dtruong has joined #openstack-nova | 18:24 | |
| *** satyar has quit IRC | 18:25 | |
| *** AlexeyAbashkin has joined #openstack-nova | 18:26 | |
| *** jpena is now known as jpena|off | 18:29 | |
| *** AlexeyAbashkin has quit IRC | 18:30 | |
| *** esberglu has quit IRC | 18:31 | |
| *** esberglu has joined #openstack-nova | 18:31 | |
| *** Apoorva has joined #openstack-nova | 18:32 | |
| *** Tom-Tom has joined #openstack-nova | 18:35 | |
| *** armax has joined #openstack-nova | 18:36 | |
| *** itlinux has quit IRC | 18:38 | |
| *** itlinux has joined #openstack-nova | 18:42 | |
| *** slaweq has joined #openstack-nova | 18:44 | |
| *** gouthamr has quit IRC | 18:46 | |
| *** slaweq has quit IRC | 18:46 | |
| *** itlinux has quit IRC | 18:46 | |
| *** gouthamr has joined #openstack-nova | 18:48 | |
| *** oanson has quit IRC | 18:48 | |
| *** itlinux has joined #openstack-nova | 18:49 | |
| *** moshele has joined #openstack-nova | 18:50 | |
| *** felipemonteiro has joined #openstack-nova | 18:51 | |
| *** itlinux has quit IRC | 18:52 | |
| *** itlinux has joined #openstack-nova | 18:54 | |
| *** tbachman has quit IRC | 18:55 | |
| *** itlinux has quit IRC | 18:55 | |
| *** moshele has quit IRC | 18:55 | |
| *** tbachman has joined #openstack-nova | 18:56 | |
| *** jmlowe has quit IRC | 18:56 | |
| *** amodi has quit IRC | 18:57 | |
| *** penick has joined #openstack-nova | 18:57 | |
| *** carthaca_ has joined #openstack-nova | 18:57 | |
| *** carthaca_ has quit IRC | 18:59 | |
| *** penick has quit IRC | 19:00 | |
| *** jackie-truong has joined #openstack-nova | 19:01 | |
| *** carthaca_ has joined #openstack-nova | 19:01 | |
| *** carthaca_ has quit IRC | 19:03 | |
| *** sree has quit IRC | 19:05 | |
| *** david-lyle has joined #openstack-nova | 19:10 | |
| *** moshele has joined #openstack-nova | 19:12 | |
| *** markvoelker has joined #openstack-nova | 19:13 | |
| *** markvoelker has quit IRC | 19:13 | |
| openstackgerrit | Mark Goddard proposed openstack/nova master: WIP: Send traits to ironic on server boot https://review.openstack.org/508116 | 19:15 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Add get_traits() method to ComputeDriver https://review.openstack.org/532287 | 19:15 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Implement get_traits() for the ironic virt driver https://review.openstack.org/532288 | 19:15 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Add has_any_traits() to provider tree https://review.openstack.org/532289 | 19:15 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Call get_traits() in the resource tracker https://review.openstack.org/532290 | 19:15 |
| *** fragatina has joined #openstack-nova | 19:17 | |
| *** itlinux has joined #openstack-nova | 19:18 | |
| *** fragatina has quit IRC | 19:18 | |
| *** vivsoni__ has quit IRC | 19:18 | |
| *** fragatina has joined #openstack-nova | 19:19 | |
| *** moshele has quit IRC | 19:21 | |
| *** jmlowe has joined #openstack-nova | 19:22 | |
| *** markvoelker has joined #openstack-nova | 19:23 | |
| *** sambetts is now known as sambetts|afk | 19:23 | |
| *** itlinux has quit IRC | 19:24 | |
| *** hoonetorg has quit IRC | 19:25 | |
| *** tbachman has quit IRC | 19:27 | |
| *** tbachman has joined #openstack-nova | 19:30 | |
| efried | mgoddard Let me know if you want to chat about ^ | 19:31 |
| *** priteau has quit IRC | 19:32 | |
| *** penick has joined #openstack-nova | 19:32 | |
| *** harlowja has joined #openstack-nova | 19:33 | |
| *** penick has quit IRC | 19:36 | |
| *** itlinux has joined #openstack-nova | 19:36 | |
| *** jmlowe has quit IRC | 19:40 | |
| *** harlowja_ has joined #openstack-nova | 19:41 | |
| *** itlinux has quit IRC | 19:41 | |
| *** markvoelker has quit IRC | 19:42 | |
| *** markvoelker has joined #openstack-nova | 19:43 | |
| *** harlowja has quit IRC | 19:43 | |
| *** jmlowe has joined #openstack-nova | 19:43 | |
| *** moshele has joined #openstack-nova | 19:44 | |
| *** cdent has quit IRC | 19:44 | |
| *** amodi has joined #openstack-nova | 19:46 | |
| *** matrohon has joined #openstack-nova | 19:47 | |
| *** markvoelker has quit IRC | 19:48 | |
| *** gouthamr has quit IRC | 19:53 | |
| *** harlowja_ has quit IRC | 19:55 | |
| *** harlowja has joined #openstack-nova | 19:55 | |
| *** jackie-truong has quit IRC | 19:59 | |
| *** gouthamr has joined #openstack-nova | 20:02 | |
| *** matrohon has quit IRC | 20:03 | |
| *** amodi has quit IRC | 20:03 | |
| *** damien_r has joined #openstack-nova | 20:05 | |
| *** amodi has joined #openstack-nova | 20:06 | |
| *** mvk has joined #openstack-nova | 20:11 | |
| *** yamamoto has joined #openstack-nova | 20:11 | |
| *** moshele has quit IRC | 20:12 | |
| mriedem | wait wait wait, | 20:13 |
| mriedem | johnthetubaguy has a minion? | 20:13 |
| *** moshele has joined #openstack-nova | 20:15 | |
| *** fragatina has quit IRC | 20:15 | |
| openstackgerrit | Merged openstack/nova master: libvirt: Provide VGPU inventory for a single GPU type https://review.openstack.org/514763 | 20:15 |
| *** slaweq has joined #openstack-nova | 20:17 | |
| rybridges | Hey Guys! Have a really quick question. Related to this spec: https://specs.openstack.org/openstack/nova-specs/specs/kilo/implemented/quiesced-image-snapshots-with-qemu-guest-agent.html | 20:18 |
| *** matrohon has joined #openstack-nova | 20:18 | |
| rybridges | Is that QEMU guest agent that is mentioned something that a user would install on their own instance? Or is that something that we (as operators) would install on the hypervisor itself? | 20:19 |
| *** moshele has quit IRC | 20:19 | |
| mriedem | rybridges: it's in the image | 20:20 |
| mriedem | ‘hw_qemu_guest_agent=yes’ property and ‘hw_require_fsfreeze=yes’ property is set on the image metadata, and QEMU Guest Agent is installed and enabled in the instance | 20:20 |
| mriedem | s/in the instance/in the image/ | 20:20 |
| mriedem | the image metadata is just telling nova that the image has the guest agent and requires fsfreeze | 20:21 |
| mriedem | prior to snapshot | 20:21 |
| *** hoonetorg has joined #openstack-nova | 20:21 | |
| rybridges | ok so the part i am confused about is "QEMU Guest Agent is installed and enabled in the instance" | 20:24 |
| *** eharney has quit IRC | 20:24 | |
| rybridges | so this means we would bake the QEMU Guest Agent into the image itself | 20:24 |
| rybridges | such that when we boot an instance, the QEMU Guest Agent will automatically come up and be enabled | 20:24 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP: [api] Allow multi-attach in compute api https://review.openstack.org/271047 | 20:25 |
| ildikov | mriedem: ^^ | 20:25 |
| mriedem | rybridges: yes | 20:26 |
| *** eharney has joined #openstack-nova | 20:26 | |
| mriedem | that's my understanding, nova doesn't inject something into the image | 20:26 |
| *** amodi has quit IRC | 20:26 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI https://review.openstack.org/526094 | 20:26 |
| *** imacdonn has quit IRC | 20:32 | |
| *** imacdonn has joined #openstack-nova | 20:32 | |
| *** markvoelker has joined #openstack-nova | 20:33 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | 20:33 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 20:33 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add index(updated_at) on migrations table. https://review.openstack.org/531132 | 20:33 |
| mriedem | dansmith: i addressed the 2 addressable comments in the migration multi-cell lister patch ^ | 20:34 |
| rybridges | ok thanks mriedem | 20:38 |
| mriedem | rybridges: https://www.sebastien-han.fr/blog/2015/02/09/openstack-perform-consistent-snapshots-with-qemu-guest-agent/ ? | 20:38 |
| mriedem | old blog but probably still good | 20:38 |
| *** cleong has quit IRC | 20:38 | |
| mriedem | stvnoyes: finally got that snapshot test passing http://logs.openstack.org/86/531386/7/check/tempest-full/eca51c5/job-output.txt.gz#_2018-01-09_19_34_20_731609 | 20:41 |
| mriedem | but failed in teardown | 20:41 |
| mriedem | http://logs.openstack.org/86/531386/7/check/tempest-full/eca51c5/job-output.txt.gz#_2018-01-09_20_01_02_052268 | 20:41 |
| mriedem | hmm, failed to delete the volume, | 20:41 |
| stvnoyes | i never like it when problems happen in teardown... | 20:42 |
| mriedem | but the test is explicitly deleting the volume and cascading it's snapshot delete https://review.openstack.org/#/c/266605/24/tempest/api/compute/volumes/test_attach_volume.py@417 | 20:42 |
| mriedem | smcginnis: does the volume cascading delete stuff work? | 20:42 |
| smcginnis | mriedem: Why do I feel like this is a trick question? | 20:42 |
| mriedem | maybe i don't know what's possible, but, | 20:43 |
| mriedem | i create a volume-backed instance and create a snapshot of it, which also creates a snapshot of the volume, | 20:43 |
| mriedem | at the end of the test, i do a delete volume with cascade=True so i don't have to explicitly delete the volume snapshot | 20:43 |
| mriedem | assuming that's OK? | 20:44 |
| *** jmlowe has quit IRC | 20:44 | |
| smcginnis | mriedem: So from Cinders perspective that's just a volume with a snapshot. So yeah, that should work if you pass cascade=True and it should delete any child snapshots on volume deletion. | 20:44 |
| mriedem | weird, don't even see the error in the c-api logs | 20:45 |
| mriedem | oh here we go | 20:45 |
| mriedem | damn c-api log doesn't really tell me anything | 20:46 |
| mriedem | OH | 20:47 |
| mriedem | i see | 20:47 |
| mriedem | INFO cinder.api.openstack.wsgi [None req-57d5906a-2c27-4546-8fbd-21d355fd886d tempest-AttachVolumeMultiAttachTest-336010326 tempest-AttachVolumeMultiAttachTest-336010326] https://10.0.1.221/volume/v3/1e7533c90d794cf8a7463d9a7473244e/volumes/42c444d7-4b3a-42f0-a2d6-3fe652814e05?cascade=True returned with HTTP 202 | 20:47 |
| mriedem | that's the one i'm doing and it's a 202 | 20:47 |
| mriedem | INFO cinder.api.openstack.wsgi [None req-4afc5bd6-64cd-444b-9d7f-51e87a67c0e4 tempest-AttachVolumeMultiAttachTest-336010326 tempest-AttachVolumeMultiAttachTest-336010326] https://10.0.1.221/volume/v3/1e7533c90d794cf8a7463d9a7473244e/volumes/42c444d7-4b3a-42f0-a2d6-3fe652814e05 returned with HTTP 400 | 20:47 |
| mriedem | that's the one that tempest teardown is doing becaues i'm not waiting for my call to complete | 20:48 |
| mriedem | drats | 20:48 |
| *** yamamoto has quit IRC | 20:48 | |
| smcginnis | Oh, so it's trying to delete the volume without cascade while your call is still cleaning up? | 20:48 |
| mriedem | yeah | 20:48 |
| mriedem | i just need to add a waiter | 20:48 |
| mriedem | easy peasy | 20:49 |
| mriedem | ildikov: i don't know why but your api update says it's in merge conflict | 20:49 |
| *** yamamoto has joined #openstack-nova | 20:49 | |
| ildikov | mriedem: maybe because the shared_targets patch didn't get through the gate due to zuul issues or smth? | 20:50 |
| ildikov | mriedem: I didn't want to rebase until that's merged... | 20:50 |
| mriedem | yeah that's fine, i'm going to update this tempest patch | 20:51 |
| ildikov | I only changed the top patch and downloaded the whole chain fresh before | 20:51 |
| ildikov | I will check on zuul and see where things are | 20:52 |
| *** cfriesen has joined #openstack-nova | 20:53 | |
| *** armax has quit IRC | 20:53 | |
| *** yamamoto has quit IRC | 20:54 | |
| *** priteau has joined #openstack-nova | 20:55 | |
| *** vladikr has joined #openstack-nova | 20:56 | |
| mriedem | 5: (529695,5) Use volume shared_targets to lock during attach/detach (8h17m/++++~+~++++/rem:1h56m) | 20:57 |
| *** belmoreira has joined #openstack-nova | 20:57 | |
| mriedem | #5 in the gate so should be merged soonish | 20:57 |
| mriedem | it's only been there for 8.5 hours already.. | 20:57 |
| mriedem | ildikov: ok i guess i'll start working on adding tests for the new flows when the microversion is high enough and multiattach is True | 21:00 |
| *** penick has joined #openstack-nova | 21:00 | |
| ildikov | mriedem: ok, I'm not touching the chain then | 21:02 |
| ildikov | mriedem: the unit and functional tests are fixed in the latest version | 21:03 |
| ildikov | at least it worked for me... | 21:03 |
| *** jmlowe has joined #openstack-nova | 21:06 | |
| *** liverpooler has quit IRC | 21:11 | |
| *** markvoelker has quit IRC | 21:13 | |
| *** markvoelker has joined #openstack-nova | 21:13 | |
| mriedem | mdbooth: damn, i ran that isolated test thing, 178 tests in my worker list to comb through :( | 21:14 |
| mriedem | the bisect analyzer couldn't figure it out | 21:14 |
| *** archit has joined #openstack-nova | 21:15 | |
| mriedem | although, i guess the idea is you can continue to wittle the results down with the smaller set of tests each time | 21:15 |
| *** markvoelker has quit IRC | 21:17 | |
| *** ktibi has quit IRC | 21:18 | |
| openstackgerrit | Merged openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965 | 21:23 |
| *** smatzek has quit IRC | 21:24 | |
| *** smatzek has joined #openstack-nova | 21:24 | |
| *** smatzek has quit IRC | 21:24 | |
| *** smatzek has joined #openstack-nova | 21:24 | |
| *** smatzek has quit IRC | 21:24 | |
| *** smatzek has joined #openstack-nova | 21:25 | |
| *** smatzek has quit IRC | 21:30 | |
| openstackgerrit | Merged openstack/nova master: Use volume shared_targets to lock during attach/detach https://review.openstack.org/529695 | 21:33 |
| *** armax has joined #openstack-nova | 21:34 | |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: placement: add API reference for create inventory https://review.openstack.org/511342 | 21:38 |
| *** pcaruana has quit IRC | 21:38 | |
| *** tidwellr has quit IRC | 21:38 | |
| *** tidwellr has joined #openstack-nova | 21:39 | |
| *** slaweq has quit IRC | 21:39 | |
| *** david-lyle has quit IRC | 21:40 | |
| *** dave-mccowan has quit IRC | 21:40 | |
| *** eharney has quit IRC | 21:41 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | 21:42 |
| edleafe | mriedem: ^^ addressed your concerns, and then some | 21:42 |
| *** matrohon has quit IRC | 21:44 | |
| *** threestrands has joined #openstack-nova | 21:44 | |
| *** threestrands has quit IRC | 21:44 | |
| *** threestrands has joined #openstack-nova | 21:44 | |
| *** yamamoto has joined #openstack-nova | 21:52 | |
| *** dillaman has quit IRC | 21:53 | |
| *** yamamoto has quit IRC | 21:59 | |
| *** moshele has joined #openstack-nova | 21:59 | |
| *** AlexeyAbashkin has joined #openstack-nova | 22:00 | |
| *** yamahata has quit IRC | 22:02 | |
| *** moshele has quit IRC | 22:04 | |
| *** AlexeyAbashkin has quit IRC | 22:05 | |
| *** rcernin has joined #openstack-nova | 22:06 | |
| *** belmoreira has quit IRC | 22:10 | |
| openstackgerrit | Merged openstack/nova stable/pike: doc: Add configuration index page https://review.openstack.org/531042 | 22:14 |
| *** fragatina has joined #openstack-nova | 22:15 | |
| openstackgerrit | Merged openstack/nova stable/pike: Fix NoneType error when [service_user] is misconfigured https://review.openstack.org/526468 | 22:16 |
| openstackgerrit | Merged openstack/nova stable/pike: Add missing unit tests for FilterScheduler._get_all_host_states https://review.openstack.org/529625 | 22:26 |
| *** artom_ has joined #openstack-nova | 22:26 | |
| *** markvoelker has joined #openstack-nova | 22:29 | |
| *** artom has quit IRC | 22:30 | |
| *** markvoelker has quit IRC | 22:30 | |
| *** gouthamr has quit IRC | 22:35 | |
| *** priteau has quit IRC | 22:40 | |
| *** penick has quit IRC | 22:43 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens https://review.openstack.org/325381 | 22:43 |
| openstackgerrit | melanie witt proposed openstack/nova master: Use ConsoleAuthToken object to generate authorizations https://review.openstack.org/325414 | 22:43 |
| openstackgerrit | melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | 22:43 |
| *** lyan has quit IRC | 22:44 | |
| *** damien_r has quit IRC | 22:44 | |
| *** itlinux has joined #openstack-nova | 22:46 | |
| *** yamahata has joined #openstack-nova | 22:48 | |
| *** archit has quit IRC | 22:52 | |
| *** gouthamr has joined #openstack-nova | 22:53 | |
| SamYaple | which scheduler filter is responsible for the cpu overallocation/ratio filtering? | 22:54 |
| mriedem | SamYaple: CoreFilter | 22:56 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: [libvirt] Allow multiple volume attachments https://review.openstack.org/267587 | 22:56 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Pass multiattach flag to reserve_block_device_name https://review.openstack.org/531244 | 22:56 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: [api] Allow multi-attach in compute api https://review.openstack.org/271047 | 22:56 |
| mriedem | if you're ocata+, we filter VCPUs in placement when using the FilterScheduler, thus making the CoreFilter not necessary | 22:56 |
| SamYaple | mriedem: was that aroundin mitaka? | 22:56 |
| *** _pewp_ has quit IRC | 22:56 | |
| mriedem | was what around mitaka? | 22:56 |
| mriedem | placement was ocata | 22:56 |
| SamYaple | no sorry, itc lag | 22:57 |
| mriedem | CoreFilter has been around awhile yes | 22:57 |
| SamYaple | the corefilter, thats been around since before mitaka, yes | 22:57 |
| SamYaple | cool | 22:57 |
| SamYaple | sorry just looking at https://github.com/openstack/nova/blob/mitaka-eol/nova/conf/scheduler.py#L126 | 22:57 |
| mriedem | yeah the CoreFilter was never a default filter for some reason | 22:57 |
| *** _pewp_ has joined #openstack-nova | 22:57 | |
| SamYaple | didnt see it listed and we were removing the retryfilter and now its overcommitting vcpus | 22:57 |
| *** moshele has joined #openstack-nova | 22:57 | |
| *** hongbin has quit IRC | 22:57 | |
| SamYaple | we added the corefilter intothe copied list and it started filtering likeit should | 22:58 |
| SamYaple | was just tryingto trackdown the disconnect | 22:58 |
| mriedem | ildikov: a rebase and https://review.openstack.org/#/c/271047/51/nova/tests/unit/api/openstack/compute/test_serversV21.py is as far as i got; was trying to get that libvirt test that was race failing to pass, but couldn't after over an hour | 22:58 |
| mriedem | rechecking the tempest test patch and calling it a day | 22:59 |
| *** penick has joined #openstack-nova | 23:00 | |
| *** moshele has quit IRC | 23:00 | |
| mriedem | dansmith: yes i hacked up that comment pretty bad :) | 23:00 |
| ildikov | mriedem: sounds good enough, thanks! | 23:00 |
| dansmith | mriedem: :P | 23:00 |
| mriedem | will put a fix on top tomorrow | 23:00 |
| ildikov | mriedem: I will look into it tomorrow too | 23:01 |
| *** tidwellr has quit IRC | 23:04 | |
| *** sdague has quit IRC | 23:04 | |
| openstackgerrit | Merged openstack/nova stable/pike: Fix an error in _get_host_states when deleting a compute node https://review.openstack.org/529626 | 23:09 |
| *** markvoelker has joined #openstack-nova | 23:11 | |
| *** markvoelker has quit IRC | 23:11 | |
| *** stvnoyes has quit IRC | 23:19 | |
| *** AlexeyAbashkin has joined #openstack-nova | 23:22 | |
| *** AlexeyAbashkin has quit IRC | 23:27 | |
| *** kumarmn has joined #openstack-nova | 23:30 | |
| *** takashin has joined #openstack-nova | 23:31 | |
| *** awaugama has quit IRC | 23:33 | |
| *** sdague has joined #openstack-nova | 23:34 | |
| *** gmann has joined #openstack-nova | 23:35 | |
| *** kumarmn has quit IRC | 23:35 | |
| *** oomichi_ has joined #openstack-nova | 23:39 | |
| *** oomichi_ has quit IRC | 23:42 | |
| *** moshele has joined #openstack-nova | 23:43 | |
| *** markvoelker has joined #openstack-nova | 23:43 | |
| *** markvoelker has quit IRC | 23:43 | |
| *** moshele has quit IRC | 23:49 | |
| *** ianw has quit IRC | 23:50 | |
| *** ianw has joined #openstack-nova | 23:51 | |
| *** hshiina has joined #openstack-nova | 23:52 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!