*** hshiina2 has joined #openstack-nova | 00:00 | |
*** hshiina|afk has quit IRC | 00:04 | |
*** mingyu has quit IRC | 00:05 | |
*** bkopilov__ has joined #openstack-nova | 00:06 | |
*** bkopilov_ has joined #openstack-nova | 00:06 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Service version check for NUMA live migration https://review.openstack.org/566723 | 00:08 |
---|---|---|
*** mingyu has joined #openstack-nova | 00:09 | |
*** mingyu has quit IRC | 00:11 | |
*** fragatina has quit IRC | 00:12 | |
*** takashin has joined #openstack-nova | 00:17 | |
*** MasterofJOKers has quit IRC | 00:17 | |
*** dgonzalez has quit IRC | 00:17 | |
*** dgonzalez has joined #openstack-nova | 00:17 | |
*** MasterofJOKers has joined #openstack-nova | 00:18 | |
*** ebbex has quit IRC | 00:18 | |
*** ebbex has joined #openstack-nova | 00:20 | |
*** Swami_ has quit IRC | 00:21 | |
*** Swami has quit IRC | 00:21 | |
*** yingjun has joined #openstack-nova | 00:23 | |
*** tuanla____ has joined #openstack-nova | 00:36 | |
*** gyee has quit IRC | 00:36 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Service version check for NUMA live migration https://review.openstack.org/566723 | 00:38 |
*** Nel1x has joined #openstack-nova | 00:40 | |
*** Dinesh_Bhor has joined #openstack-nova | 00:42 | |
*** mriedem has quit IRC | 00:44 | |
*** harlowja has quit IRC | 00:51 | |
openstackgerrit | fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181 | 00:52 |
*** andreas_s has joined #openstack-nova | 00:54 | |
*** tiendc has joined #openstack-nova | 00:57 | |
*** nicolasbock has quit IRC | 00:57 | |
*** andreas_s has quit IRC | 00:58 | |
*** phuongnh has joined #openstack-nova | 01:01 | |
*** felipemonteiro__ has joined #openstack-nova | 01:08 | |
*** gjayavelu has quit IRC | 01:10 | |
*** tuanla____ has quit IRC | 01:12 | |
*** trungnv has quit IRC | 01:12 | |
*** tuanla____ has joined #openstack-nova | 01:12 | |
*** trungnv has joined #openstack-nova | 01:13 | |
*** hshiina has joined #openstack-nova | 01:13 | |
*** armaan has quit IRC | 01:15 | |
*** hshiina2 has quit IRC | 01:16 | |
*** armaan has joined #openstack-nova | 01:16 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Service version check for NUMA live migration https://review.openstack.org/566723 | 01:18 |
*** username_ has joined #openstack-nova | 01:22 | |
openstackgerrit | Merged openstack/nova master: Convert fping API to processutils.execute() https://review.openstack.org/564144 | 01:22 |
*** username_ is now known as username__ | 01:23 | |
*** felipemonteiro__ has quit IRC | 01:27 | |
*** zhaochao has joined #openstack-nova | 01:28 | |
*** hongbin has joined #openstack-nova | 01:37 | |
*** gongysh has joined #openstack-nova | 01:37 | |
*** zcorneli has quit IRC | 01:40 | |
*** hongbin_ has joined #openstack-nova | 01:44 | |
*** hongbin has quit IRC | 01:45 | |
*** lei-zh has joined #openstack-nova | 01:46 | |
*** psachin` has joined #openstack-nova | 02:10 | |
*** yamamoto has joined #openstack-nova | 02:20 | |
*** tiendc has quit IRC | 02:20 | |
*** tiendc has joined #openstack-nova | 02:21 | |
*** yamamoto has quit IRC | 02:24 | |
*** psachin` has quit IRC | 02:26 | |
takashin | 02:35 | |
*** username__ has quit IRC | 02:35 | |
*** yamamoto has joined #openstack-nova | 02:37 | |
*** psachin has joined #openstack-nova | 02:42 | |
openstackgerrit | fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181 | 02:44 |
*** vladikr has quit IRC | 02:44 | |
*** vladikr has joined #openstack-nova | 02:45 | |
*** fragatina has joined #openstack-nova | 02:55 | |
*** fragatina has quit IRC | 02:55 | |
*** fragatina has joined #openstack-nova | 02:56 | |
*** fragatina has quit IRC | 02:57 | |
*** fragatina has joined #openstack-nova | 02:57 | |
*** lei-zh has quit IRC | 03:03 | |
*** lei-zh has joined #openstack-nova | 03:03 | |
*** rcernin has quit IRC | 03:14 | |
*** slaweq has joined #openstack-nova | 03:16 | |
*** slaweq has quit IRC | 03:20 | |
*** vivsoni has quit IRC | 03:25 | |
*** vivsoni has joined #openstack-nova | 03:25 | |
*** andreas_s has joined #openstack-nova | 03:28 | |
*** tuanla____ has quit IRC | 03:29 | |
*** takashin has quit IRC | 03:29 | |
*** tuanla____ has joined #openstack-nova | 03:30 | |
*** yamamoto has quit IRC | 03:31 | |
*** andreas_s has quit IRC | 03:32 | |
*** udesale has joined #openstack-nova | 03:36 | |
*** threestrands has joined #openstack-nova | 03:38 | |
*** hiro-kobayashi has joined #openstack-nova | 03:41 | |
*** links has joined #openstack-nova | 03:42 | |
*** yamamoto has joined #openstack-nova | 03:44 | |
*** gyan_ has joined #openstack-nova | 03:44 | |
*** hongbin_ has quit IRC | 03:46 | |
*** dave-mccowan has quit IRC | 03:48 | |
*** wolverineav has joined #openstack-nova | 03:58 | |
*** udesale has quit IRC | 04:00 | |
*** udesale has joined #openstack-nova | 04:01 | |
*** Nel1x has quit IRC | 04:08 | |
*** janki has joined #openstack-nova | 04:10 | |
openstackgerrit | Merged openstack/nova master: Add docs for hw_video:ram_max_mb flavor extra spec https://review.openstack.org/567070 | 04:11 |
*** yingjun has quit IRC | 04:15 | |
*** cburgess_ has quit IRC | 04:16 | |
*** vladikr has quit IRC | 04:16 | |
*** vladikr has joined #openstack-nova | 04:16 | |
*** amotoki has joined #openstack-nova | 04:19 | |
*** vladikr has quit IRC | 04:24 | |
*** vladikr has joined #openstack-nova | 04:25 | |
*** cburgess has joined #openstack-nova | 04:26 | |
*** udesale_ has joined #openstack-nova | 04:36 | |
*** gongysh has quit IRC | 04:36 | |
*** sridharg has joined #openstack-nova | 04:38 | |
*** udesale has quit IRC | 04:39 | |
*** ratailor has joined #openstack-nova | 04:42 | |
*** Shilpa has joined #openstack-nova | 04:44 | |
openstackgerrit | Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751 | 04:44 |
*** Shilpa has quit IRC | 04:45 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | 04:46 |
openstackgerrit | Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437 | 04:50 |
openstackgerrit | Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439 | 04:50 |
openstackgerrit | Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438 | 04:50 |
*** takashin has joined #openstack-nova | 04:50 | |
openstackgerrit | Merged openstack/nova master: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/564964 | 04:51 |
openstackgerrit | Michael Still proposed openstack/nova master: Move IVS VIF plugging into the new utilities module. https://review.openstack.org/557552 | 04:54 |
openstackgerrit | Michael Still proposed openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835 | 04:54 |
openstackgerrit | Michael Still proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487 | 04:54 |
*** abhishekk has joined #openstack-nova | 04:55 | |
openstackgerrit | Merged openstack/nova master: Convert xenapi's xvp console to processutils. https://review.openstack.org/564145 | 04:57 |
*** Dinesh_Bhor has quit IRC | 05:00 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:03 | |
openstackgerrit | Takahito Hirose proposed openstack/python-novaclient master: api_version decorator becomes an error in Python 3.5.0. https://review.openstack.org/564702 | 05:04 |
*** tomtom001 has quit IRC | 05:05 | |
*** tomtom001 has joined #openstack-nova | 05:06 | |
*** lei-zh has quit IRC | 05:07 | |
*** slaweq has joined #openstack-nova | 05:11 | |
*** yamamoto has quit IRC | 05:15 | |
*** slaweq has quit IRC | 05:16 | |
*** gongysh has joined #openstack-nova | 05:18 | |
*** yamamoto has joined #openstack-nova | 05:20 | |
*** udesale_ has quit IRC | 05:22 | |
*** abhishekk has quit IRC | 05:31 | |
*** Swami has joined #openstack-nova | 05:32 | |
*** Swami_ has joined #openstack-nova | 05:32 | |
*** hoonetorg has quit IRC | 05:33 | |
*** yikun__ has quit IRC | 05:33 | |
*** yikun__ has joined #openstack-nova | 05:34 | |
*** moshele has joined #openstack-nova | 05:34 | |
*** Eran_Kuris has joined #openstack-nova | 05:35 | |
*** gongysh has quit IRC | 05:37 | |
*** rajinir has quit IRC | 05:38 | |
*** fnordahl has quit IRC | 05:40 | |
*** lei-zh has joined #openstack-nova | 05:41 | |
*** ratailor has quit IRC | 05:41 | |
*** fnordahl has joined #openstack-nova | 05:42 | |
*** hoonetorg has joined #openstack-nova | 05:44 | |
*** tssurya has joined #openstack-nova | 05:46 | |
*** evin has quit IRC | 05:47 | |
*** yingjun has joined #openstack-nova | 05:48 | |
*** udesale_ has joined #openstack-nova | 05:51 | |
*** udesale_ has quit IRC | 05:53 | |
*** udesale has joined #openstack-nova | 05:53 | |
*** threestrands has quit IRC | 05:54 | |
*** ratailor has joined #openstack-nova | 05:55 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | 05:56 |
*** pcaruana has joined #openstack-nova | 05:57 | |
*** bhujay has joined #openstack-nova | 05:59 | |
*** vladikr has quit IRC | 06:02 | |
*** vladikr has joined #openstack-nova | 06:02 | |
*** ratailor has quit IRC | 06:05 | |
*** ratailor_ has joined #openstack-nova | 06:05 | |
*** armaan has quit IRC | 06:05 | |
*** armaan has joined #openstack-nova | 06:05 | |
*** vladikr has quit IRC | 06:08 | |
*** vladikr has joined #openstack-nova | 06:08 | |
*** Swami has quit IRC | 06:13 | |
*** Swami_ has quit IRC | 06:13 | |
*** armaan_ has joined #openstack-nova | 06:13 | |
*** evin has joined #openstack-nova | 06:15 | |
*** armaan has quit IRC | 06:16 | |
*** vladikr has quit IRC | 06:16 | |
*** Dinesh_Bhor has quit IRC | 06:16 | |
*** vladikr has joined #openstack-nova | 06:16 | |
*** Dinesh_Bhor has joined #openstack-nova | 06:18 | |
*** annp has joined #openstack-nova | 06:22 | |
*** gbarros has joined #openstack-nova | 06:22 | |
*** tianhui_ has joined #openstack-nova | 06:24 | |
*** andreas_s has joined #openstack-nova | 06:25 | |
*** andreas_s has quit IRC | 06:25 | |
*** andreas_s has joined #openstack-nova | 06:25 | |
*** tianhui has quit IRC | 06:26 | |
*** lei-zh has quit IRC | 06:28 | |
*** lei-zh1 has joined #openstack-nova | 06:28 | |
*** MasterofJOKers_ has joined #openstack-nova | 06:33 | |
*** armaan_ has quit IRC | 06:33 | |
*** armaan has joined #openstack-nova | 06:34 | |
*** MasterofJOKers has quit IRC | 06:34 | |
*** MasterofJOKers_ is now known as MasterofJOKers | 06:34 | |
*** jaosorior has joined #openstack-nova | 06:34 | |
*** ccamacho has quit IRC | 06:38 | |
*** brault has joined #openstack-nova | 06:38 | |
*** ccamacho has joined #openstack-nova | 06:39 | |
*** gbarros has quit IRC | 06:49 | |
*** slaweq has joined #openstack-nova | 06:51 | |
*** yikun__ has quit IRC | 06:56 | |
*** yikun__ has joined #openstack-nova | 06:56 | |
*** yingjun has quit IRC | 06:56 | |
*** wolverineav has quit IRC | 06:58 | |
*** wolverineav has joined #openstack-nova | 06:58 | |
openstackgerrit | Merged openstack/nova master: Remove IronicHostManager and baremetal scheduling options https://review.openstack.org/565805 | 07:01 |
*** wolverineav has quit IRC | 07:03 | |
*** yingjun has joined #openstack-nova | 07:03 | |
*** wolverineav has joined #openstack-nova | 07:04 | |
*** ratailor__ has joined #openstack-nova | 07:08 | |
*** wolverineav has quit IRC | 07:10 | |
*** yamamoto_ has joined #openstack-nova | 07:10 | |
*** ratailor_ has quit IRC | 07:10 | |
*** yamamoto has quit IRC | 07:10 | |
*** lei-zh1 has quit IRC | 07:10 | |
*** lei-zh1 has joined #openstack-nova | 07:11 | |
*** ratailor_ has joined #openstack-nova | 07:15 | |
*** alexchadin has joined #openstack-nova | 07:16 | |
*** ratailor__ has quit IRC | 07:17 | |
*** gjayavelu has joined #openstack-nova | 07:19 | |
*** moshele has quit IRC | 07:20 | |
*** lei-zh1 has quit IRC | 07:21 | |
*** tbachman has quit IRC | 07:21 | |
*** lei-zh1 has joined #openstack-nova | 07:22 | |
*** tesseract has joined #openstack-nova | 07:22 | |
*** wolverineav has joined #openstack-nova | 07:22 | |
openstackgerrit | fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181 | 07:23 |
*** tbachman has joined #openstack-nova | 07:24 | |
*** moshele has joined #openstack-nova | 07:24 | |
*** moshele has quit IRC | 07:25 | |
*** yamamoto_ has quit IRC | 07:29 | |
*** yamamoto has joined #openstack-nova | 07:30 | |
*** bertys____ has joined #openstack-nova | 07:31 | |
*** masber has quit IRC | 07:36 | |
*** amoralej|off is now known as amoralej | 07:36 | |
*** mgoddard has joined #openstack-nova | 07:36 | |
*** ragiman has joined #openstack-nova | 07:39 | |
*** abhishekk has joined #openstack-nova | 07:40 | |
*** alexchadin has quit IRC | 07:41 | |
*** mdnadeem has joined #openstack-nova | 07:41 | |
*** mdnadeem has quit IRC | 07:41 | |
*** alexchadin has joined #openstack-nova | 07:42 | |
*** jangutter has joined #openstack-nova | 07:48 | |
*** armaan has quit IRC | 07:50 | |
*** armaan has joined #openstack-nova | 07:51 | |
*** jpena|off is now known as jpena | 07:51 | |
openstackgerrit | sahid proposed openstack/nova master: compute: introduce cpu_shared_set option https://review.openstack.org/561912 | 07:52 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: always pass emulator threads policy https://review.openstack.org/561913 | 07:52 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: place emulator threads on CONF.cpu_shared_set https://review.openstack.org/510897 | 07:52 |
*** dtantsur|afk is now known as dtantsur | 07:53 | |
*** gjayavelu has quit IRC | 07:57 | |
*** hiro-kobayashi has quit IRC | 08:00 | |
*** namnh has joined #openstack-nova | 08:02 | |
*** takashin has left #openstack-nova | 08:03 | |
*** armaan has quit IRC | 08:05 | |
*** armaan has joined #openstack-nova | 08:06 | |
*** yingjun has quit IRC | 08:07 | |
*** liuzz_ has quit IRC | 08:07 | |
*** mdbooth has joined #openstack-nova | 08:11 | |
*** mdnadeem has joined #openstack-nova | 08:12 | |
*** liuzz has joined #openstack-nova | 08:13 | |
*** yamamoto has quit IRC | 08:15 | |
*** yamamoto has joined #openstack-nova | 08:17 | |
*** dougsz has joined #openstack-nova | 08:17 | |
*** wolverineav has quit IRC | 08:18 | |
openstackgerrit | Jan Gutter proposed openstack/nova master: Add plumbing for Contrail/Tungsten Fabric offloads https://review.openstack.org/567147 | 08:20 |
openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148 | 08:22 |
stephenfin | jaypipes: By mechanisms, I was referring to the various ways of a network can be configured and segmented, either L2 or L3. Local means connectivity is only on a single host https://gist.github.com/bonclay7/458b353f997e05db374d | 08:26 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | 08:26 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider cadidates https://review.openstack.org/567150 | 08:26 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix broken allocation candidates with traits https://review.openstack.org/567151 | 08:26 |
stephenfin | jaypipes: Also https://ask.openstack.org/en/question/51388/ | 08:28 |
tetsuro | ^ jauypipes, efried, bauzas: started nested allocation candidates branch in a different way. | 08:28 |
*** udesale_ has joined #openstack-nova | 08:28 | |
*** udesale has quit IRC | 08:28 | |
*** sridharg has quit IRC | 08:29 | |
*** sridharg has joined #openstack-nova | 08:31 | |
kashyap | stephenfin: Hey, when you can, see what's wrong with my test here: https://review.openstack.org/#/c/565242/5/nova/tests/unit/virt/libvirt/test_driver.py | 08:31 |
*** lucas-bbl is now known as lucasagomes | 08:31 | |
*** vladikr has quit IRC | 08:32 | |
*** vladikr has joined #openstack-nova | 08:33 | |
openstackgerrit | Merged openstack/nova master: Remove unused methods in nova/compute/utils.py https://review.openstack.org/566855 | 08:33 |
*** lpetrut has joined #openstack-nova | 08:33 | |
* stephenfin looks | 08:33 | |
*** jmccarthy has joined #openstack-nova | 08:35 | |
*** edmondsw has joined #openstack-nova | 08:35 | |
*** bkopilov__ has quit IRC | 08:35 | |
kashyap | Merci! | 08:36 |
*** bkopilov_ has quit IRC | 08:36 | |
kashyap | stephenfin: It is mostly just me not being good with the unit tests. The change is quite simple, if you see the commit message. | 08:36 |
kashyap | (And the driver.py code) | 08:36 |
*** jaosorior has quit IRC | 08:36 | |
*** andreas_s has quit IRC | 08:38 | |
*** liuzz_ has joined #openstack-nova | 08:39 | |
*** derekh has joined #openstack-nova | 08:40 | |
*** edmondsw has quit IRC | 08:40 | |
*** andreas_s has joined #openstack-nova | 08:40 | |
*** jaosorior has joined #openstack-nova | 08:41 | |
*** liuzz has quit IRC | 08:42 | |
*** yingjun has joined #openstack-nova | 08:43 | |
*** hshiina is now known as hshiina|afk | 08:49 | |
openstackgerrit | Merged openstack/nova master: Convert certificate generation to processutils. https://review.openstack.org/564146 | 08:49 |
*** bkopilov has joined #openstack-nova | 08:50 | |
*** wolverineav has joined #openstack-nova | 08:50 | |
*** bkopilov_ has joined #openstack-nova | 08:50 | |
*** tomtom001 has quit IRC | 08:51 | |
*** tomtom001 has joined #openstack-nova | 08:52 | |
*** yamamoto has quit IRC | 08:55 | |
*** gyankum has joined #openstack-nova | 08:55 | |
*** alexchadin has quit IRC | 08:55 | |
*** nicolasbock has joined #openstack-nova | 08:56 | |
*** udesale_ has quit IRC | 08:59 | |
*** udesale_ has joined #openstack-nova | 08:59 | |
*** udesale_ has quit IRC | 09:00 | |
*** udesale has joined #openstack-nova | 09:00 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add tests for sharing rps in test_common_rc() https://review.openstack.org/566842 | 09:09 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add tests for alloc cands with poor local disk https://review.openstack.org/566842 | 09:11 |
*** tetsuro has left #openstack-nova | 09:11 | |
stephenfin | kashyap: Done | 09:11 |
kashyap | stephenfin: Thanks; will look | 09:11 |
*** yamamoto has joined #openstack-nova | 09:11 | |
kashyap | stephenfin: About actually being supported by the processor or not change. I first thought to remove it completely because, regardless of it, it's useless | 09:12 |
kashyap | But ... | 09:12 |
kashyap | Since the first change is to deprecate the support, I should retain it | 09:13 |
stephenfin | kashyap: No, you can remove it but call that out and fix the test accordingly | 09:15 |
*** wolverineav has quit IRC | 09:15 | |
stephenfin | No point keeping dead code around | 09:15 |
kashyap | stephenfin: Right. So the plan is: Deprecate it in Rocky; delete support for it in Stein. | 09:16 |
*** wolverineav has joined #openstack-nova | 09:16 | |
kashyap | I have it locally as you indicated | 09:16 |
* kashyap goes to fix... | 09:19 | |
*** wolverineav has quit IRC | 09:20 | |
openstackgerrit | lei zhang proposed openstack/python-novaclient master: Fix the incorrect cirros default password https://review.openstack.org/567170 | 09:24 |
openstackgerrit | jichenjc proposed openstack/nova master: [Doc]Link policies file into api https://review.openstack.org/561503 | 09:28 |
kashyap | stephenfin: With the 'assert_has_calls', you meant: | 09:30 |
kashyap | mock_warn.assert_has_calls([ | 09:30 |
kashyap | mock.call(...), | 09:30 |
kashyap | mock.call(...), | 09:30 |
kashyap | ]) | 09:30 |
kashyap | ? | 09:30 |
stephenfin | yup | 09:30 |
kashyap | Maybe I should consider that. | 09:31 |
openstackgerrit | jichenjc proposed openstack/nova master: Enhance api-guide general info some updates https://review.openstack.org/561773 | 09:31 |
*** Dinesh_Bhor has quit IRC | 09:32 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 09:37 |
openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 09:37 |
openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 09:37 |
openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067 | 09:37 |
*** yingjun has quit IRC | 09:42 | |
*** ratailor__ has joined #openstack-nova | 09:44 | |
openstackgerrit | jichenjc proposed openstack/nova master: [doc] Add soft_deleted flag https://review.openstack.org/562897 | 09:44 |
*** ratailor_ has quit IRC | 09:47 | |
openstackgerrit | Jan Gutter proposed openstack/nova stable/queens: Add plumbing for Contrail/Tungsten Fabric offloads https://review.openstack.org/567177 | 09:53 |
liuzz_ | hello everyone, Who can help me? i want to create a new role and only allow it to manage keystone resource, can't manager other reources(instances, volumes, ...),What should I do? | 09:59 |
*** tbachman has quit IRC | 10:00 | |
kashyap | stephenfin: How about I just assert that the warning has happened N number of times for each event? | 10:01 |
kashyap | Like: self.assertEquals(N, mock_warn.call_count) | 10:01 |
*** tbachman has joined #openstack-nova | 10:03 | |
stephenfin | I'm not sure. I'd just do what you've done there | 10:03 |
kashyap | No problem; I'll tinker a bit | 10:03 |
kashyap | (Calling the exact warning message isn't nice.) | 10:04 |
kashyap | Anyhow, disregard me for now | 10:04 |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Deprecate support for monitoring Intel CMT `perf` events https://review.openstack.org/565242 | 10:06 |
*** lei-zh1 has quit IRC | 10:06 | |
*** ragiman has quit IRC | 10:14 | |
*** ragiman has joined #openstack-nova | 10:14 | |
*** cdent has joined #openstack-nova | 10:26 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Simplify instance name generation https://review.openstack.org/516573 | 10:26 |
*** namnh has quit IRC | 10:28 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Add mypy target https://review.openstack.org/538221 | 10:29 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Store list of converted files https://review.openstack.org/538222 | 10:29 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to 'nova.pci' https://review.openstack.org/538224 | 10:29 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Add 'mypy' job https://review.openstack.org/539168 | 10:29 |
openstackgerrit | Chris Dent proposed openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116 | 10:31 |
cdent | johnthetubaguy: there's a spec and real ci behind that ^ now | 10:32 |
cdent | (it's got your -2 on it) | 10:32 |
*** annp has quit IRC | 10:33 | |
*** wolverineav has joined #openstack-nova | 10:34 | |
*** abhishekk has quit IRC | 10:36 | |
andreas_s | melwitt: Hi, I'm facing an issue with the following devstack patch you commited a few days ago: https://github.com/openstack-dev/devstack/commit/12579c3db7b28381c8ec97945aa23ee02d54d22b | 10:37 |
stephenfin | bauzas: If you're about today, could you take a look at https://review.openstack.org/#/c/541290/ ? | 10:38 |
andreas_s | melwitt: It seems like this patch is moving the console configurtion (vnc, spice, serial) from nova.conf into the nova-cell config | 10:38 |
andreas_s | melwitt: but the n-cpu systemd service is not consuming the nova_cell config - with the result, that the console setting is being ignored | 10:39 |
stephenfin | andreas_s: I was thinking the exact same thing but chose to ignore my intuition https://review.openstack.org/#/c/550141/1/ | 10:40 |
stephenfin | (vs. https://review.openstack.org/#/c/550141/2/) | 10:41 |
andreas_s | stephenfin: ah good to know | 10:41 |
andreas_s | stephenfin: but the challenge is still that devstack seems to specify the cell_config file as input parameter to configure_console_proxies, doesn't it? | 10:43 |
andreas_s | at least in our thirdparty CI the config landed in the cell config file: http://sng01.objectstorage.softlayer.net/v1/AUTH_1940ea10-6e82-4501-b2f9-eb236510e575/ibmzkvmci/production/510897/13/check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x/3e67649/logs/etc/nova/ | 10:43 |
stephenfin | andreas_s: Yup. I'm still trying to figure out which specific setting the conductors (which I think are the only services to use those files) need configured | 10:44 |
*** priteau has joined #openstack-nova | 10:45 | |
stephenfin | because I thought most/all of them are only used by nova-compute | 10:45 |
*** hjensas|afk is now known as hjensas | 10:45 | |
andreas_s | yeah, nova-compute needs this config in definitively! | 10:45 |
*** tbachman has quit IRC | 10:47 | |
stephenfin | andreas_s: I'd file a bug, personally. I'll take a look at it after lunch and see if we can come up with something | 10:47 |
jmccarthy | mriedem: Thanks for the update on https://review.openstack.org/#/c/566367 ! I tested this out, and patchest2 worked great for me | 10:49 |
andreas_s | stephenfin: that would be great. thanks. Can you subscribe me to this bug once opened (andreas scheuring)? thx. If you need any more input, let know... | 10:49 |
*** yamamoto has quit IRC | 10:49 | |
stephenfin | andreas_s: "I would file a bug, personally", e.g. I suggest _you_ file a bug :) | 10:50 |
stephenfin | I could do it, but you probably have the most context | 10:50 |
stephenfin | and logs | 10:50 |
andreas_s | ok, will do | 10:50 |
andreas_s | stephenfin: just was not sure how to interpret "I would" ;) | 10:50 |
stephenfin | That's an Irish'ism. Sorry :) | 10:51 |
*** yamamoto has joined #openstack-nova | 10:51 | |
andreas_s | hehe, enjoy your lunch! | 10:51 |
openstackgerrit | Merged openstack/nova master: Remove [scheduler]/host_manager config option https://review.openstack.org/565806 | 10:53 |
*** mgoddard_ has joined #openstack-nova | 10:54 | |
*** derekh_ has joined #openstack-nova | 10:54 | |
*** derekh has quit IRC | 10:54 | |
*** slaweq has quit IRC | 10:54 | |
*** slaweq_ has joined #openstack-nova | 10:56 | |
*** andreas_s_ has joined #openstack-nova | 10:56 | |
*** mgoddard has quit IRC | 10:56 | |
*** mgoddard_ is now known as mgoddard | 10:56 | |
*** psachin` has joined #openstack-nova | 10:56 | |
*** andreas_s has quit IRC | 10:57 | |
*** psachin has quit IRC | 10:57 | |
*** mchlumsky_ has joined #openstack-nova | 10:59 | |
*** RaoulHC has joined #openstack-nova | 10:59 | |
*** slaweq has joined #openstack-nova | 10:59 | |
*** andreas_s has joined #openstack-nova | 10:59 | |
*** slaweq_ has quit IRC | 10:59 | |
*** mchlumsky has quit IRC | 10:59 | |
*** lyarwood has quit IRC | 11:00 | |
*** lyarwood has joined #openstack-nova | 11:01 | |
*** andreas_s_ has quit IRC | 11:01 | |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Deprecate support for monitoring Intel CMT `perf` events https://review.openstack.org/565242 | 11:02 |
openstackgerrit | Rajat Sharma proposed openstack/nova master: Imposing restrictions on resource providers create uuid https://review.openstack.org/567191 | 11:04 |
andreas_s | stephenfin, melwitt: opened the bug against devstack: https://bugs.launchpad.net/devstack/+bug/1770143 | 11:06 |
openstack | Launchpad bug 1770143 in devstack "Nova Console setup broken" [Undecided,New] | 11:06 |
*** lucasagomes is now known as lucas-hungry | 11:06 | |
ShilpaSD | dansmith: hi | 11:11 |
*** yamamoto has quit IRC | 11:25 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201 | 11:29 |
*** sridharg has quit IRC | 11:30 | |
*** yamamoto has joined #openstack-nova | 11:31 | |
*** yamamoto has quit IRC | 11:32 | |
*** sridharg has joined #openstack-nova | 11:34 | |
*** Nel1x has joined #openstack-nova | 11:37 | |
*** armaan has quit IRC | 11:41 | |
*** armaan has joined #openstack-nova | 11:41 | |
*** phuongnh has quit IRC | 11:42 | |
*** jpena is now known as jpena|lunch | 11:44 | |
*** ratailor__ has quit IRC | 11:47 | |
*** tiendc has quit IRC | 11:52 | |
*** hemna_ has quit IRC | 11:53 | |
*** blkart has quit IRC | 11:54 | |
*** bhujay has quit IRC | 11:55 | |
*** yamamoto has joined #openstack-nova | 11:58 | |
*** bertys____ has quit IRC | 12:02 | |
*** pchavva has joined #openstack-nova | 12:06 | |
*** lucas-hungry is now known as lucasagomes | 12:07 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 12:09 |
openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 12:09 |
openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 12:09 |
openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067 | 12:10 |
*** edmondsw has joined #openstack-nova | 12:10 | |
*** tbachman has joined #openstack-nova | 12:16 | |
*** armaan has quit IRC | 12:16 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Followup for multiple member_of qparams support https://review.openstack.org/567209 | 12:19 |
*** jmlowe has quit IRC | 12:20 | |
*** pcaruana has quit IRC | 12:21 | |
*** Nel1x has quit IRC | 12:28 | |
*** armaan has joined #openstack-nova | 12:30 | |
openstackgerrit | Radoslav Gerganov proposed openstack/nova stable/queens: Handle PortNotFoundClient exception when getting ports https://review.openstack.org/567211 | 12:30 |
*** janki has quit IRC | 12:30 | |
*** janki has joined #openstack-nova | 12:31 | |
*** yingjun has joined #openstack-nova | 12:36 | |
*** yamamoto has quit IRC | 12:39 | |
openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148 | 12:41 |
*** lyan has joined #openstack-nova | 12:42 | |
*** lyan is now known as Guest53916 | 12:42 | |
*** yamamoto has joined #openstack-nova | 12:44 | |
*** dave-mccowan has joined #openstack-nova | 12:44 | |
*** yamamoto has quit IRC | 12:45 | |
*** vladikr has quit IRC | 12:48 | |
*** vladikr has joined #openstack-nova | 12:48 | |
*** jpena|lunch is now known as jpena | 12:49 | |
*** gyan_ has quit IRC | 12:51 | |
*** gyankum has quit IRC | 12:51 | |
*** tuanla____ has quit IRC | 12:52 | |
*** lpetrut has quit IRC | 12:53 | |
*** armaan has quit IRC | 12:54 | |
*** amoralej is now known as amoralej|lunch | 12:58 | |
*** andreas_s has quit IRC | 13:00 | |
*** andreas_s has joined #openstack-nova | 13:00 | |
mdbooth | efried: Morning! https://review.openstack.org/#/c/564017/ | 13:01 |
mdbooth | That's causing me a problem, and I considering proposing a revert. However, I think it only exists due to bad naming. | 13:01 |
efried | mdbooth: stand by. | 13:02 |
*** jmlowe has joined #openstack-nova | 13:02 | |
efried | mdbooth: Okay, do tell. | 13:02 |
mdbooth | efried: I see you proposed it because of the loop in the ironic driver | 13:02 |
efried | mdbooth: Well, actually because of [2] (in the commit message), but [1] was a good excuse. | 13:03 |
mdbooth | efried: Sec, lemme check 2 | 13:03 |
openstackgerrit | Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | 13:03 |
*** andreas_s has quit IRC | 13:03 | |
efried | mdbooth: TL;DR, we were checking for volume-ness in a different (and apparently wrong | 13:03 |
efried | ) way in the powervm driver. | 13:03 |
*** andreas_s has joined #openstack-nova | 13:04 | |
mdbooth | efried: Thing is, the ironic driver doesn't need that check | 13:04 |
efried | mdbooth: mriedem pointed out that we should be using is_volume... which led us to find out that it didn't exist. | 13:04 |
efried | mdbooth: Well, is the new version of the check doing something the old one wasn't? | 13:04 |
mdbooth | Because, confusingly, block_device_info_get_mapping() should actually be called block_device_info_get_volumes() | 13:04 |
*** psachin` has quit IRC | 13:04 | |
efried | mdbooth: I.e. did we break something afresh, or just leave something that (we didn't know) was broken before? | 13:04 |
*** sapd has quit IRC | 13:05 | |
mdbooth | If it was called *_get_volumes(), it would be obvious that an is_volume call was redundant subsequently | 13:05 |
jaypipes | stephenfin: yes, sorry, I understand the differences between gre, vlan, vxlan, local, etc. what I'm wondering about is why you used the term "mechanism". Is that because of the use of "mechanism driver" to describe the implementation module that Neutron uses under the hood for constructing a particular network? | 13:05 |
mdbooth | Where's _vol_drv_iter() called in your patch? | 13:06 |
stephenfin | jaypipes: I think I saw it used in documentation somewhere and couldn't think of a better term :) | 13:06 |
mdbooth | efried: Where I'm going with that is... are you sure they aren't all volumes anyway? And if they're not, why not just get the volumes in the first place? | 13:07 |
*** liuzz_ has quit IRC | 13:08 | |
jaypipes | stephenfin: the reason I asked is because neither of those pages mentions the term "mechanism", and it seems to me (I may very well be wrong) that you are conflating two things (segmentation protocol and whether overlapping IP address ranges can occur on the network) | 13:08 |
*** lpetrut has joined #openstack-nova | 13:09 | |
mdbooth | efried: Ah, yes. You're doing the same thing. | 13:09 |
mdbooth | efried: So, they're all volumes. | 13:09 |
efried | mdbooth: So let's back up just for a sec. | 13:09 |
mdbooth | Have a look in driver.get_block_device_info(), which is what generates that data | 13:09 |
*** cdent has quit IRC | 13:09 | |
mdbooth | See: | 13:09 |
mdbooth | 'block_device_mapping': | 13:09 |
mdbooth | virt_block_device.convert_all_volumes(*block_device_mapping) | 13:09 |
efried | mdbooth: 1) Is there actually a problem here, or just some redundancy? and 2) was anything actually broken (or changed at all) by the patch you referenced? | 13:09 |
mdbooth | 'block_device_mapping' should really be called 'volumes' | 13:10 |
*** liverpooler has joined #openstack-nova | 13:10 | |
stephenfin | jaypipes: Hmm, protocol would probably have done. I definitely saw 'mechanism' used somewhere but of course I can't find where now | 13:10 |
mdbooth | Well, the change is redundant, but due to bad naming that's not at all obvious. | 13:11 |
efried | mdbooth: It is my understanding that there is quite a lot of horribleness in the bdm code that could be improved upon. | 13:11 |
jaypipes | stephenfin: no worries, it's not a huge deal really. | 13:11 |
mdbooth | The issue is that you added is_volume, which is a property, to the inherited list | 13:11 |
efried | mdbooth: The change itself is (was intended to be) a no-op. It was only supposed to get rid of the need to ask that @property through a private member of the outer bdm. | 13:11 |
stephenfin | jaypipes: I don't think I'm conflating those things. The main difference is whether it's an L2 network (uses physnets) or L3 network (uses a tunnel endpoint). tbh, the whole section on tenant networks and provider networks could be dropped | 13:12 |
mdbooth | So it breaks when something attempts to write to it | 13:12 |
efried | mdbooth: Write to what, .is_volume ? | 13:12 |
mdbooth | Yeah | 13:12 |
stephenfin | I'll probably do that as a follow-up. At this point I just want to get the spec in so I can grab a runway :) | 13:12 |
efried | mdbooth: Who was doing that, and where? | 13:12 |
efried | and why? | 13:12 |
efried | and based on what source info, if not _bdm_obj.is_volume itself? | 13:13 |
efried | mdbooth: sounds like we need a fup to remove... whatever that was. | 13:13 |
*** mriedem has joined #openstack-nova | 13:13 | |
efried | mriedem: see email | 13:13 |
mdbooth | efried: Let me reload that context, but it broke my patch https://review.openstack.org/#/c/528362/ | 13:13 |
mdbooth | efried: I believe the issue is due to something iterating over properties to initialise the object | 13:13 |
mdbooth | Let me just remind myself... | 13:14 |
mriedem | nothing like an immediate pounce | 13:14 |
efried | mriedem: No hurry, just responding to your question about the granular bp | 13:14 |
*** derekh_ has quit IRC | 13:14 | |
efried | I suppose I didn't need to pounce for that. Sorry. | 13:14 |
ShilpaSD | stephenfin: Hi | 13:14 |
efried | mdbooth: Okay, so it kicked your patch out of the gate, didn't like melt CERNs supercollider, that's a relief anyway. | 13:14 |
mdbooth | efried: My patch is more important than somebody's magnet set. | 13:15 |
efried | clearly | 13:15 |
*** andreas_s has quit IRC | 13:16 | |
jaypipes | stephenfin: understood. | 13:17 |
*** tbachman has quit IRC | 13:18 | |
mdbooth | efried: Ok, so I think I can unpick this from my end. | 13:25 |
mdbooth | efried: However, the point about is_volume being redundant in that context still stands | 13:25 |
efried | mdbooth: Okay. From what I've been able to tell so far, it's not clear that we should be setting things on _bdm_obj rather than on the outer object itself. | 13:25 |
mdbooth | Which, as I say, would be obvious if we called 'block_device_mappings' 'volumes' | 13:25 |
mriedem | lyarwood: can you poke this regression fix for queens? https://review.openstack.org/#/c/564461/ we're going to do a release soon for another regression fix | 13:26 |
efried | mdbooth: I'll be on the lookout for that review :) | 13:26 |
mriedem | this one https://review.openstack.org/#/q/If9e1581db9c1ae14340b787d03c815d243d5a50c | 13:26 |
mdbooth | efried: Meh, I can't be arsed renaming things I'm not actively hitting. It's painful and nobody cares. | 13:26 |
lyarwood | mriedem: yeah np, looking now | 13:27 |
efried | mdbooth: Just so. Also, that's a slippery slope, as I mentioned before. Once we start "fixing" the block device code, there'll be no stopping. | 13:27 |
*** andreas_s_ has joined #openstack-nova | 13:28 | |
*** andreas_s_ has quit IRC | 13:28 | |
jmccarthy | mriedem: Morning ! I checked this out and it worked out great https://review.openstack.org/#/c/566367 ! | 13:28 |
*** andreas_s_ has joined #openstack-nova | 13:29 | |
*** mingyu has joined #openstack-nova | 13:29 | |
*** andreas_s_ has quit IRC | 13:31 | |
*** andreas_s_ has joined #openstack-nova | 13:31 | |
*** amoralej|lunch is now known as amoralej | 13:31 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: add consumer.generation/create missing consumers https://review.openstack.org/557958 | 13:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add Project, User and Consumer objects https://review.openstack.org/565403 | 13:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404 | 13:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405 | 13:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | 13:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 13:32 |
mdbooth | efried: Added a comment to the PowerVM patch. Not really worth a -1, tbh, but I figure somebody's going to have to touch it anyway. | 13:33 |
lyarwood | mriedem: right, so this is a stable/queens cherry-pick of a revert of a bugfix that only ever landed on master, fun. | 13:34 |
*** felipemonteiro__ has joined #openstack-nova | 13:34 | |
*** armaan has joined #openstack-nova | 13:34 | |
lyarwood | mriedem: it's valid as bugfix on stable/queens AFAICT, do we have enough time to drop the `This reverts commit` line? | 13:35 |
mriedem | lyarwood: which one? https://review.openstack.org/#/c/564461/ ? | 13:36 |
mriedem | it's not really a revert, it's a redo | 13:36 |
mriedem | jmccarthy: great, thanks | 13:37 |
lyarwood | mriedem: yeah the revert line just confused me | 13:37 |
mriedem | jaypipes: i think you have some context on this https://review.openstack.org/#/c/566367 | 13:37 |
mriedem | would be good to get that fixed since we backported the regression to ocata | 13:37 |
mriedem | lyarwood: right, so we revert the one workaround hack to remove the instance dir after the by_name init thing | 13:37 |
mriedem | because if we don't do that hack, we don't need to remove the dir, and that was the original bug anyway | 13:38 |
mriedem | lyarwood: https://review.openstack.org/#/c/524208/ landed when queens was master | 13:39 |
lyarwood | mriedem: right, ignore me, the fscking gerrit UI didn't list it as in Queens | 13:39 |
*** cdent has joined #openstack-nova | 13:39 | |
mriedem | "included in" dropdown | 13:39 |
mriedem | is what i check | 13:39 |
mriedem | lyarwood: you and mdbooth might also be interested in https://review.openstack.org/#/c/566367 | 13:39 |
lyarwood | mriedem: yeah it didn't display correctly, checked it again and there it is | 13:39 |
mriedem | dealing with libvirt imagebackend fun | 13:39 |
* lyarwood should really double check with git directly next time | 13:40 | |
*** pcaruana has joined #openstack-nova | 13:40 | |
efried | mdbooth: ack, and thanks. | 13:42 |
*** jaosorior has quit IRC | 13:43 | |
*** mriedem1 has joined #openstack-nova | 13:43 | |
*** mriedem has quit IRC | 13:46 | |
*** yamamoto has joined #openstack-nova | 13:46 | |
*** mriedem1 is now known as mriedem | 13:46 | |
mdbooth | mriedem: Somebody should clean that thing up ;) | 13:46 |
*** jaosorior has joined #openstack-nova | 13:50 | |
*** hongbin has joined #openstack-nova | 13:52 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix detach_volume calls when rolling back a failed attach https://review.openstack.org/563213 | 13:52 |
mriedem | efried: gibi: test fixed, ^ - nice catch gibi | 13:52 |
mriedem | that's one that also needs to go back to queens as a regression fix | 13:52 |
*** andreas_s_ has quit IRC | 13:53 | |
*** felipemonteiro_ has joined #openstack-nova | 13:53 | |
mriedem | dansmith: can you hit this queens backport? https://review.openstack.org/#/c/564461/ | 13:53 |
*** andreas_s has joined #openstack-nova | 13:53 | |
dansmith | mriedem: yeah | 13:54 |
mriedem | thanks | 13:54 |
*** evin has quit IRC | 13:55 | |
kashyap | Matt, when you can: https://review.openstack.org/#/c/565242/ ("libvirt: Deprecate support for monitoring Intel CMT `perf` events") | 13:55 |
kashyap | The unit test is "fixed", but in the most robust way. | 13:55 |
kashyap | (But that's dead code anyway, and we will remove it in Stein.) | 13:55 |
kashyap | If there are any better ways I can rewrite the new unit test, welcome. | 13:55 |
*** yamamoto has quit IRC | 13:56 | |
*** felipemonteiro__ has quit IRC | 13:56 | |
mriedem | lyarwood: this would also be good for cern https://review.openstack.org/#/c/566288/ | 13:57 |
*** esberglu has joined #openstack-nova | 13:58 | |
jaypipes | mriedem: sure thing. gimme a few, though. trying to rebase a nasty. | 14:00 |
gibi | mriedem: +2+A on https://review.openstack.org/563213 | 14:00 |
*** awaugama has joined #openstack-nova | 14:01 | |
*** andreas_s has quit IRC | 14:02 | |
*** lpetrut has quit IRC | 14:02 | |
*** mlavalle has joined #openstack-nova | 14:02 | |
*** pcaruana has quit IRC | 14:05 | |
mriedem | gibi: thanks | 14:06 |
*** andreas_s has joined #openstack-nova | 14:07 | |
wolverineav | mriedem: i've updated the patch with changes as suggested (https://review.openstack.org/#/c/534371/) please take a look when you have a moment :) | 14:08 |
*** andreas_s has quit IRC | 14:08 | |
*** pcaruana has joined #openstack-nova | 14:08 | |
*** lpetrut has joined #openstack-nova | 14:08 | |
*** andreas_s has joined #openstack-nova | 14:08 | |
mriedem | dansmith: pretty sure this is going to be a no-no backport given the object version changes yeah? https://review.openstack.org/#/c/563797/ | 14:08 |
*** burt has joined #openstack-nova | 14:10 | |
mriedem | wolverineav: ack, lgtm, just waiting on zuul | 14:10 |
mriedem | actually +2; stephenfin - mind hitting https://review.openstack.org/#/c/534371/ again? | 14:11 |
wolverineav | mriedem: thanks! i wanted to proactively get that looked at. its been in review quite some time. | 14:11 |
* stephenfin clicks | 14:11 | |
mriedem | wolverineav: yup, i know, thanks for your patience | 14:11 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Fix detach_volume calls when rolling back a failed attach https://review.openstack.org/567232 | 14:11 |
mriedem | lyarwood: dansmith: ^ we'll want to get that in the queens release we do this week | 14:12 |
dansmith | mriedem: yeah, if it had been done non-remotable it would have been okay, but.. | 14:12 |
dansmith | mriedem: okay doesn't look like the master one has landed yet | 14:13 |
mriedem | dansmith: re that pike list_hosts one, -2 since they still have the nova-manage host list command in pike | 14:15 |
mriedem | so they can just run that with --config-file pointed at the cell db | 14:15 |
*** mingyu has quit IRC | 14:15 | |
dansmith | ack | 14:16 |
*** lpetrut has quit IRC | 14:18 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Expose driver_block_device fields consistently https://review.openstack.org/528362 | 14:19 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347 | 14:19 |
*** mingyu has joined #openstack-nova | 14:19 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553829 | 14:21 |
dansmith | mriedem: melwitt tssurya: Do we have content for a cells meeting today? I got feedback on my patches to work on, which was my only thing. tssurya said maybe belmiro would have some things to discuss this week I think? | 14:22 |
mriedem | lyarwood: are you still working on this pike backport? https://review.openstack.org/#/c/561015/ | 14:23 |
*** eharney has quit IRC | 14:24 | |
mriedem | dansmith: i don't really have anything; i created a todo for myself to write some kind of functional regression test for https://bugs.launchpad.net/nova/+bug/1746863 - separate from the one i think mel was going to write | 14:24 |
openstack | Launchpad bug 1746863 in OpenStack Compute (nova) "scheduler affinity doesn't work with multiple cells" [High,In progress] - Assigned to melanie witt (melwitt) | 14:24 |
mriedem | for a separate but related bug we talked about the other day | 14:24 |
dansmith | ack | 14:24 |
lyarwood | mriedem: nope that dropped off my list, I can take another look at it in the next hour or so | 14:24 |
mriedem | "TODO: We need a new bug and test for the multi-cell affinity scenario where two instances are scheduled at the same time in the same affinity group. We need 2 cells with 1 host each, with exactly enough capacity to fit just one instance so that placement will fail the first request and throw it into the other host in the other cell. The late affinity check in the compute won't fail because it can't see the other member | 14:24 |
mriedem | he other cell, so it will think it's fine. " | 14:24 |
*** pcaruana has quit IRC | 14:25 | |
mriedem | getting ^ to not be racy will probably be the hard part | 14:25 |
mriedem | dansmith: https://review.openstack.org/#/c/553829/ is pike and cells related if you want to put that through | 14:26 |
mriedem | melwitt: i've gone through open queens, pike and ocata changes and added what i think we should target for a release this week on all 3 branches https://etherpad.openstack.org/p/nova-stable-branch-status | 14:26 |
*** pcaruana has joined #openstack-nova | 14:26 | |
mdbooth | jaypipes: https://review.openstack.org/#/c/529029/11/nova/virt/driver.py | 14:28 |
efried | mdbooth: Did you reverse the order of those patches? | 14:29 |
mdbooth | jaypipes: So, I agree with the confusing naming. How about I rename *_get_root_device() to *_get_root_device_name() ? | 14:29 |
mdbooth | efried: Yep | 14:29 |
efried | mdbooth: Then your commit message is no longer correct :( | 14:29 |
*** andreas_s has quit IRC | 14:30 | |
mdbooth | efried: Which one? | 14:30 |
jaypipes | mdbooth: ++ | 14:30 |
efried | mdbooth: https://review.openstack.org/#/c/526347/ says "a subsequent change will..." | 14:31 |
*** andreas_s has joined #openstack-nova | 14:31 | |
mdbooth | jaypipes: K. Separate patch to come before. | 14:31 |
openstackgerrit | Chris Dent proposed openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116 | 14:31 |
*** derekh has joined #openstack-nova | 14:31 | |
efried | mdbooth: when in fact it is the prior | 14:31 |
efried | mdbooth: I don't care, just happened to notice. | 14:31 |
mdbooth | efried: Ah, I didn't reorder that patch | 14:31 |
mdbooth | I believe that's still correct. | 14:31 |
efried | okay | 14:31 |
efried | I didn't dig | 14:31 |
tssurya | dansmith: nah, we can skip :) I don't think belmiro is joining today | 14:34 |
dansmith | okay cool | 14:34 |
*** andreas_s_ has joined #openstack-nova | 14:34 | |
*** felipemonteiro__ has joined #openstack-nova | 14:34 | |
*** andreas_s has quit IRC | 14:36 | |
mriedem | efried: melwitt: mtreinish: f yeah e-r is commenting again https://review.openstack.org/#/c/567211/ | 14:36 |
efried | mriedem: woot | 14:37 |
*** felipemonteiro_ has quit IRC | 14:37 | |
*** germs has joined #openstack-nova | 14:37 | |
*** germs has quit IRC | 14:37 | |
*** germs has joined #openstack-nova | 14:37 | |
efried | mriedem: what was the problem/solution? | 14:37 |
*** r-daneel has joined #openstack-nova | 14:37 | |
*** evin has joined #openstack-nova | 14:37 | |
mriedem | efried: https://review.openstack.org/#/c/567058/ and https://review.openstack.org/#/c/567068/ | 14:38 |
*** andreas_s_ has quit IRC | 14:38 | |
*** reshift has joined #openstack-nova | 14:39 | |
efried | heh, so no CI for elastic recheck? | 14:39 |
mtreinish | mriedem: cool | 14:39 |
mriedem | efried: there is, but that part of the code is kind of tricky | 14:40 |
mriedem | the function takes a specific change, patchset, and build name/id, | 14:40 |
mriedem | and the logs are gone after 10 days | 14:40 |
mriedem | so not sure how you're going to reliably test for that | 14:40 |
mtreinish | efried: the ci there doesn't check the bot actually sending messages. (it's complex) Mostly we validate the individual pieces work and that the queries are valid (although that looks like it's failing) | 14:40 |
*** r-daneel_ has joined #openstack-nova | 14:41 | |
reshift | Hi, does the nova python client have any methods to search servers for a metadata key or value? | 14:41 |
openstackgerrit | Eric Fried proposed openstack/nova master: [doc] Add soft_deleted flag https://review.openstack.org/562897 | 14:41 |
*** jaosorior has quit IRC | 14:41 | |
*** r-daneel has quit IRC | 14:42 | |
*** r-daneel_ is now known as r-daneel | 14:42 | |
mriedem | reshift: the REST API doesn't support that so i don't think the client does either | 14:42 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398 | 14:43 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Service version check for NUMA live migration https://review.openstack.org/566723 | 14:43 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: NUMA-aware live migration https://review.openstack.org/567242 | 14:43 |
reshift | ok, thanks | 14:43 |
mriedem | reshift: see https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L301 | 14:43 |
mriedem | there is https://developer.openstack.org/api-ref/compute/#server-metadata-servers-metadata but (1) you'd need to hit that for all servers and (2) it doesn't have filter params | 14:43 |
reshift | I think I might try using tags instead | 14:44 |
mriedem | reshift: i was just going to say, you can filter servers on tags | 14:44 |
mriedem | if that works for you | 14:44 |
*** gbarros has joined #openstack-nova | 14:44 | |
*** andreas_s has joined #openstack-nova | 14:44 | |
*** germs has quit IRC | 14:46 | |
*** udesale has quit IRC | 14:46 | |
*** germs has joined #openstack-nova | 14:46 | |
*** germs has quit IRC | 14:46 | |
*** germs has joined #openstack-nova | 14:46 | |
*** janki has quit IRC | 14:46 | |
*** jdillaman has quit IRC | 14:48 | |
*** gbarros has quit IRC | 14:49 | |
*** andreas_s has quit IRC | 14:49 | |
*** gbarros has joined #openstack-nova | 14:49 | |
*** hemna_ has joined #openstack-nova | 14:53 | |
efried | mdbooth: I don't see how convert_all_volumes filters down to only volumes. | 14:53 |
efried | rather, how it filters anything. | 14:53 |
*** felipemonteiro__ has quit IRC | 14:55 | |
*** felipemonteiro_ has joined #openstack-nova | 14:55 | |
mdbooth | efried: Those functions are weird | 14:56 |
openstackgerrit | Michael Bayer proposed openstack/nova master: Add connection_parameters to list of items copied from database https://review.openstack.org/567248 | 14:57 |
mdbooth | efried: So, it's a concatenation of volume, snapshot, image, and blank, all of which are volume types | 14:57 |
mdbooth | efried: Each of those functions individually works by calling <class>._transform on everything in the list, and those of the wrong type raise an error | 14:58 |
mdbooth | It's obtuse, but the result is that they're all volumes. | 14:58 |
efried | mdbooth: Noted. The obtuseness, and the cheapness of the is_volume check, would have me preferring to leave it in even if redundant. | 14:59 |
mdbooth | efried: The *implementation* is obtuse, however the intent is very clear | 14:59 |
mdbooth | It's called 'convert_all_volumes', and it returns only volumes | 15:00 |
mdbooth | In my view, the is_volume test adds confusion, because it sets an expectation that the input might be something other than volumes | 15:00 |
mdbooth | I'd rename 'bdms' to 'volumes' | 15:01 |
*** andreas_s has joined #openstack-nova | 15:01 | |
efried | esberglu: If you're following this, it'd be your action ^ | 15:01 |
*** jdillaman has joined #openstack-nova | 15:02 | |
*** gbarros has quit IRC | 15:03 | |
mdbooth | I wonder, however, if it's possible to have a volume bdm with no connection_info. I guess you'd have that at least briefly before attach? | 15:03 |
*** edmondsw has quit IRC | 15:04 | |
*** lpetrut has joined #openstack-nova | 15:05 | |
zzzeek | jaypipes melwitt hurry i need you to drop everything and attend to my personal nova issue | 15:05 |
*** andreas_s has quit IRC | 15:06 | |
*** gyankum has joined #openstack-nova | 15:08 | |
*** gyan_ has joined #openstack-nova | 15:08 | |
*** yingjun has quit IRC | 15:10 | |
openstackgerrit | Merged openstack/python-novaclient master: Fix the incorrect cirros default password https://review.openstack.org/567170 | 15:13 |
jaypipes | zzzeek: how may I assist you, fine sir? | 15:15 |
*** felipemonteiro_ has quit IRC | 15:16 | |
zzzeek | jaypipes: more "I added an option to oslo.db" pain: https://review.openstack.org/#/c/567248/ | 15:16 |
*** felipemonteiro_ has joined #openstack-nova | 15:16 | |
zzzeek | jaypipes: i want to put up the backports for this | 15:17 |
*** mingyu has quit IRC | 15:18 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Skip ServerActionsTestJSON.test_rebuild_server for cells v1 job https://review.openstack.org/567256 | 15:18 |
*** fragatina has quit IRC | 15:20 | |
*** zhaochao has quit IRC | 15:21 | |
*** eharney has joined #openstack-nova | 15:22 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Simplify instance name generation https://review.openstack.org/516573 | 15:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove '_apply_instance_name_template' https://review.openstack.org/567257 | 15:22 |
*** eharney has quit IRC | 15:22 | |
*** tbachman has joined #openstack-nova | 15:23 | |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove usage of migrateToURI2() API https://review.openstack.org/567258 | 15:23 |
*** jmccarthy has quit IRC | 15:24 | |
kashyap | mdbooth: ^ You might be interested in the above change. As I've see you fiddle with migration parameters of libvirt migrate API :-) | 15:24 |
kashyap | No urgency at all. This can wait. | 15:24 |
*** yingjun has joined #openstack-nova | 15:25 | |
kashyap | s/see/seen/ | 15:25 |
*** links has quit IRC | 15:25 | |
*** armaan has quit IRC | 15:25 | |
*** jmccarthy has joined #openstack-nova | 15:25 | |
*** armaan has joined #openstack-nova | 15:26 | |
*** derekh has quit IRC | 15:26 | |
*** tbachman_ has joined #openstack-nova | 15:26 | |
*** rajinir has joined #openstack-nova | 15:26 | |
*** tbachman has quit IRC | 15:27 | |
*** tbachman_ is now known as tbachman | 15:27 | |
*** sridharg has quit IRC | 15:30 | |
mriedem | stephenfin: why do we need this? https://review.openstack.org/#/c/567257/ - looks like you're just taking a separate method and munging it back into where it's called from | 15:30 |
mriedem | well, not where it's called from, but close | 15:30 |
stephenfin | It makes the follow up way easier to parse, in my mind at least https://review.openstack.org/#/c/516573/ | 15:31 |
stephenfin | (follow up to https://review.openstack.org/#/c/499612/) | 15:31 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add CellMapping.get_by_project_id() query method https://review.openstack.org/509002 | 15:33 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | 15:33 |
mriedem | stephenfin: left some comments, the commit message clearly needs more context | 15:34 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Skip placement on rebuild in same host https://review.openstack.org/561015 | 15:34 |
*** germs has quit IRC | 15:36 | |
*** germs has joined #openstack-nova | 15:37 | |
*** dpawlik_ has quit IRC | 15:37 | |
*** gyee has joined #openstack-nova | 15:38 | |
*** evin has quit IRC | 15:42 | |
openstackgerrit | Merged openstack/nova stable/queens: Fix dropped check for boot_index 0 in _validate_bdm https://review.openstack.org/564461 | 15:42 |
*** yingjun has quit IRC | 15:43 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove '_apply_instance_name_template' https://review.openstack.org/567257 | 15:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Simplify instance name generation https://review.openstack.org/516573 | 15:43 |
*** dpawlik has joined #openstack-nova | 15:44 | |
*** ragiman has quit IRC | 15:44 | |
*** reshift has quit IRC | 15:49 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: add resource_requests field to RequestSpec https://review.openstack.org/567267 | 15:52 |
stephenfin | Anyone have any idea why this change could be causing Tempest to fail? https://review.openstack.org/#/c/564444/ http://logs.openstack.org/44/564444/2/check/tempest-full/762ada1/testr_results.html.gz | 15:52 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: transfer port.resource_request to the scheduler https://review.openstack.org/567268 | 15:52 |
stephenfin | The only functional thing I've changed is that we now fetch physnet information for any network IDs we passed in network requests - not just port IDs. Can't reproduce locally though :/ | 15:53 |
openstackgerrit | Artom Lifshitz proposed openstack/nova stable/pike: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/567271 | 15:55 |
openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/567273 | 15:55 |
openstackgerrit | Artom Lifshitz proposed openstack/nova stable/pike: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/567271 | 15:56 |
gibi | stephenfin: now you include the meta network 'auto' to the query http://logs.openstack.org/44/564444/2/check/tempest-full/762ada1/controller/logs/screen-n-api.txt.gz?level=ERROR#_May_07_16_24_34_423219 | 15:57 |
stephenfin | gibi: Dang it. Thanks! I've been trying to suss that all day and couldn't find the required log 😅 | 15:59 |
lbragstad | jaypipes: re: the scope type patches for nova, i reworked the ksm (gating) and the oslo.context patches (still in review) - hopefully they're a bit more clear from a consuming service perspective | 15:59 |
lbragstad | oslo.context patch - https://review.openstack.org/#/c/530509/ | 15:59 |
mdbooth | dansmith: I think the explanation you requested is better in a class docstring rather than the commit message. That ok with you? | 16:00 |
mdbooth | Means we can find it later without git spelunking. | 16:00 |
dansmith | sure | 16:00 |
mriedem | stephenfin: the NetworkRequest(List) has helper methods to determine if it's an auto-allocate request | 16:01 |
*** mingyu has joined #openstack-nova | 16:03 | |
artom | mriedem, to be honest I just mechanically did the backport since our downstream bug was against 12 (https://bugzilla.redhat.com/show_bug.cgi?id=1572836 FWIW) | 16:05 |
openstackgerrit | Merged openstack/nova master: Followup for multiple member_of qparams support https://review.openstack.org/567209 | 16:05 |
openstack | bugzilla.redhat.com bug 1572836 in openstack-nova "nova-compute should log messages about stale resource allocations at warning priority" [Medium,Post] - Assigned to alifshit | 16:05 |
mriedem | artom: see my comments when i merged the change on master | 16:05 |
mriedem | artom: queens and rocky shouldn't be hitting that code so warning is ok, and we can't rely on ocata/pike computes to heal the allocations, | 16:06 |
mriedem | but that gets fuzzy when you're in pike and could have ocata computes, and we also fixed a bunch of places post-pike GA that didn't cleanup allocations on failure, | 16:06 |
mriedem | so from that bz, it's possible they hit one of those places that are now fixed | 16:06 |
artom | mriedem, dammit, I was hoping I wouldn't have to use my brain for this one ;) | 16:07 |
mriedem | defer your brain to dansmith | 16:07 |
mriedem | he knows that code well | 16:07 |
artom | I... I can't tell if you're serious or just messing with Dan :/ | 16:08 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Do not use SameHostFilter in API sample tests https://review.openstack.org/563037 | 16:11 |
mriedem | artom: i'm phoning a friend | 16:12 |
dansmith | mriedem: I think warning on pike is reasonable, and I think that's where we hit it downstream.. they didn't realize things had gotten legit out of sync because they had debug off | 16:13 |
dansmith | when they turned debug on, we realized some instances had allocations on three computes because of legit leaks | 16:13 |
mriedem | was this pike 16.0.0 GA? | 16:13 |
artom | dansmith, wait, was that the juicy backported you mentioned in internal IRC? | 16:13 |
dansmith | I think we made it debug initially because we were pretty sure we'd log that a lot | 16:14 |
dansmith | artom: no | 16:14 |
dansmith | but since we've seen it in the wild, it probably needs to be more visible | 16:14 |
dansmith | mriedem: it was whatever our build is based on | 16:14 |
dansmith | mriedem: are you asking if it was something past GA because of backports? | 16:14 |
mriedem | we fixed a lot of leaky allocation stuff post pike GA, | 16:15 |
dansmith | mriedem: the actual leakage happened some time in the past, they don't know when, but noticed it when computes were refusing to schedule the last bit of resource, which turned out to be because of some stale allocations | 16:15 |
dansmith | right | 16:15 |
*** phuongnh has joined #openstack-nova | 16:15 | |
mriedem | so i'm wondering if the customer hit this on 16.0.0 GA before we fixed the leaks | 16:15 |
dansmith | entirely possible | 16:15 |
dansmith | could also have been from ocata, or during/around the upgrade | 16:15 |
mriedem | i realize the leaks could have been from GA, even if we stopped the leaks later | 16:15 |
dansmith | yeah | 16:16 |
dansmith | it wasn't very acute, so I don't think they're just leaking like crazy right now | 16:16 |
dansmith | it was just "hmm, this should fit, help us figure out why not" | 16:16 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347 | 16:16 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029 | 16:16 |
mriedem | given we should have fixed those leaks now in pike, if we do hit this, my comment from the change on master probably applies | 16:16 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Rename block_device_info_get_root_device https://review.openstack.org/567277 | 16:16 |
mriedem | i.e. we shouldn't hit this, so if we are leaking, we f'ed up and warning is probably ok | 16:16 |
dansmith | yes | 16:16 |
*** r-daneel has quit IRC | 16:18 | |
*** phuongnh has quit IRC | 16:19 | |
*** Phuongnh has joined #openstack-nova | 16:20 | |
*** lpetrut has quit IRC | 16:25 | |
mriedem | artom: ok +2 on queens, +1 on pike | 16:27 |
*** hamzy has quit IRC | 16:28 | |
artom | mriedem, dansmith, thank you gentlemen | 16:32 |
*** udesale has joined #openstack-nova | 16:32 | |
*** gyankum has quit IRC | 16:33 | |
*** imacdonn has quit IRC | 16:34 | |
*** imacdonn has joined #openstack-nova | 16:34 | |
*** gyan_ has quit IRC | 16:34 | |
*** lpetrut has joined #openstack-nova | 16:34 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Add CellMapping.get_by_project_id() query method https://review.openstack.org/509002 | 16:39 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | 16:39 |
*** lucasagomes is now known as lucas-afk | 16:39 | |
*** masber has joined #openstack-nova | 16:40 | |
*** tssurya has quit IRC | 16:40 | |
*** Phuongnh has quit IRC | 16:42 | |
*** mingyu has quit IRC | 16:52 | |
*** r-daneel has joined #openstack-nova | 16:54 | |
*** gbarros has joined #openstack-nova | 16:55 | |
*** Swami has joined #openstack-nova | 16:55 | |
*** armaan has quit IRC | 16:55 | |
*** armaan has joined #openstack-nova | 16:55 | |
*** dougsz has quit IRC | 16:57 | |
*** mingyu has joined #openstack-nova | 16:57 | |
*** evin has joined #openstack-nova | 16:58 | |
*** trozet has joined #openstack-nova | 16:59 | |
*** lpetrut has quit IRC | 17:00 | |
mdbooth | mriedem: If you get a sec could you check I haven't misrepresented you in my response to jaypipes here: https://review.openstack.org/#/c/528363/15/nova/virt/block_device.py ? | 17:01 |
*** lpetrut has joined #openstack-nova | 17:01 | |
*** mgoddard has quit IRC | 17:04 | |
mriedem | replied | 17:04 |
mriedem | i really need to be reviewing that series given my somewhat grossly intimate relationship with that code now | 17:05 |
mriedem | which i plan on making my afternoon | 17:06 |
mriedem | mdbooth: the entire goal of that series is to get a serial value in the non-volume block devices right? since we only have serial == volume id today, so for the non-volumes we'll use the bdm.uuid | 17:08 |
mriedem | and the 20 patch refactor series is because that's how you roll :) | 17:08 |
mriedem | no offense, i know this code is horrible | 17:08 |
*** mingyu has quit IRC | 17:09 | |
*** fragatina has joined #openstack-nova | 17:09 | |
melwitt | mriedem: cool, thanks for the etherpad for the stable release needs | 17:11 |
*** RaoulHC has quit IRC | 17:14 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: rework how we pass candidate request information https://review.openstack.org/566166 | 17:14 |
*** tesseract has quit IRC | 17:15 | |
*** mdnadeem has quit IRC | 17:16 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: rework how we pass candidate request information https://review.openstack.org/566166 | 17:16 |
*** jpena is now known as jpena|off | 17:17 | |
*** trozet has quit IRC | 17:17 | |
*** aloga has quit IRC | 17:19 | |
openstackgerrit | Merged openstack/nova master: Fix detach_volume calls when rolling back a failed attach https://review.openstack.org/563213 | 17:22 |
*** mgoddard has joined #openstack-nova | 17:23 | |
*** mingyu has joined #openstack-nova | 17:24 | |
*** hamzy has joined #openstack-nova | 17:27 | |
*** gouthamr has quit IRC | 17:27 | |
*** mchlumsky_ has quit IRC | 17:35 | |
*** gbarros has quit IRC | 17:36 | |
*** mchlumsky has joined #openstack-nova | 17:37 | |
*** germs has quit IRC | 17:37 | |
*** eharney has joined #openstack-nova | 17:38 | |
*** germs has joined #openstack-nova | 17:38 | |
*** germs has quit IRC | 17:38 | |
*** germs has joined #openstack-nova | 17:38 | |
*** elod has quit IRC | 17:39 | |
*** mchlumsky has quit IRC | 17:41 | |
*** mchlumsky has joined #openstack-nova | 17:43 | |
mriedem | dansmith: lyarwood: https://review.openstack.org/#/c/567232/ should be good to go now - change on master just merged | 17:47 |
*** gjayavelu has joined #openstack-nova | 17:47 | |
*** mingyu has quit IRC | 17:47 | |
*** eharney has quit IRC | 17:48 | |
*** udesale has quit IRC | 17:50 | |
*** janki has joined #openstack-nova | 17:52 | |
jmccarthy | Hmm trying to setup an encrypted volume, anyone know where does it come up with this device name ? /dev/disk/by-id/scsi-360014057e4c08a456174ed99b53aa5d6 (paste.openstack.org/show/720695/) | 18:00 |
*** armaan has quit IRC | 18:00 | |
jmccarthy | It all goes wrong after this, and I can't find that device anywhere .. | 18:00 |
mriedem | jmccarthy: os-brick library | 18:00 |
*** armaan has joined #openstack-nova | 18:00 | |
*** bkopilov has quit IRC | 18:01 | |
*** eharney has joined #openstack-nova | 18:01 | |
*** bkopilov_ has quit IRC | 18:02 | |
jmccarthy | mriedem: Hmm ok and it's making it up ? Should I be able to 'see' it someplace ? | 18:02 |
mriedem | lyarwood: is your guy for encrypted volume stuff in os-brick | 18:02 |
jmccarthy | mriedem: kk ! | 18:03 |
lyarwood | jmccarthy: what are you trying to find, the original volume on the compute host? | 18:03 |
jmccarthy | lyarwood: I'm just trying to find where it's going wrong, I can attach unencrpyted, but not encrypted volumes paste.openstack.org/show/720695/ | 18:04 |
jmccarthy | It goes wrong after here - as this device can't be found anywhere ? Or I'm looking in wrong places | 18:04 |
* lyarwood looks | 18:04 | |
lyarwood | jmccarthy: paste.openstack.org isn't working for me, can you use https://paste.fedoraproject.org/ | 18:05 |
jmccarthy | lyarwood: https://paste.fedoraproject.org/paste/nL0L92U4ArsPfyVkgQlX-Q | 18:06 |
jmccarthy | It's brief .. but in the case where it works, I'm pretty sure I find that device under /dev/disk/by-id on the compute host | 18:07 |
*** r-daneel has quit IRC | 18:07 | |
lyarwood | jmccarthy: yeah that's expected the first time you attach the volume | 18:07 |
*** r-daneel has joined #openstack-nova | 18:07 | |
lyarwood | jmccarthy: well, with the older flow where os-brick formats the volume | 18:07 |
jmccarthy | That's expected to return 1 is it ? | 18:07 |
jmccarthy | lyarwood: It's expected to return 1 I mean is it ? | 18:08 |
jmccarthy | lyarwood: Ok let me try and attach it again | 18:08 |
lyarwood | jmccarthy: yeah I think so, 0 is the device is encrypted already, 1 if it isn't | 18:08 |
mriedem | mdbooth: https://review.openstack.org/#/c/528362/14/nova/tests/unit/virt/test_block_device.py@283 | 18:09 |
mriedem | am i missing something? | 18:09 |
jmccarthy | lyarwood: Ok my bad, I assumed that was the start of where my attach failed | 18:09 |
*** germs has quit IRC | 18:09 | |
jmccarthy | lyarwood: I'll look at logs more, it's going wrong but I'm doing a bad job at figuring why lol | 18:10 |
*** germs has joined #openstack-nova | 18:10 | |
*** germs has quit IRC | 18:10 | |
*** germs has joined #openstack-nova | 18:10 | |
*** mdbooth has quit IRC | 18:10 | |
lyarwood | jmccarthy: np, it's an odd flow tbh, feel free to throw another pastebin my way if you want a hand :) | 18:10 |
jmccarthy | lyarwood: Ok thanks ! Will do :) | 18:11 |
*** germs has quit IRC | 18:12 | |
*** eharney has quit IRC | 18:13 | |
*** germs has joined #openstack-nova | 18:13 | |
*** germs has quit IRC | 18:13 | |
*** germs has joined #openstack-nova | 18:13 | |
*** janki has quit IRC | 18:13 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add missing encryption_secret_uuid tests https://review.openstack.org/540679 | 18:13 |
*** bkopilov has joined #openstack-nova | 18:14 | |
jmccarthy | lyarwood: Maybe this is where I'm going wrong ? https://paste.fedoraproject.org/paste/oSRFKMBJiuCEifxfLXUxhg the volume says it's attaching for a short time, and then just goes back to available | 18:15 |
lyarwood | jhesketh: yeah it's disconnecting after the libvirt failure to attach the disk to the domain | 18:16 |
*** tssurya has joined #openstack-nova | 18:16 | |
*** germs has quit IRC | 18:17 | |
*** armaan has quit IRC | 18:17 | |
*** germs has joined #openstack-nova | 18:17 | |
*** germs has quit IRC | 18:17 | |
*** germs has joined #openstack-nova | 18:17 | |
*** armaan has joined #openstack-nova | 18:17 | |
*** armaan has quit IRC | 18:18 | |
*** eharney has joined #openstack-nova | 18:18 | |
lyarwood | jmccarthy: can you pastebin the entire req-138f35c8-ed21-4430-bf68-525574657926 flow ? | 18:18 |
*** armaan has joined #openstack-nova | 18:18 | |
jmccarthy | lyarwood: Sure - let me get that together | 18:19 |
*** dklyle has joined #openstack-nova | 18:19 | |
*** mvenesio has quit IRC | 18:20 | |
*** mdnadeem has joined #openstack-nova | 18:20 | |
*** dklyle has quit IRC | 18:21 | |
*** mvenesio has joined #openstack-nova | 18:21 | |
*** germs has quit IRC | 18:21 | |
jmccarthy | lyarwood: I think this is all of it ? https://paste.fedoraproject.org/paste/6sdkA90XTlkpftZL5j5D~w | 18:24 |
*** bkopilov has quit IRC | 18:24 | |
*** dklyle has joined #openstack-nova | 18:25 | |
*** mvenesio has quit IRC | 18:25 | |
lyarwood | jmccarthy: odd, which version of Nova is this? | 18:28 |
jmccarthy | lyarwood: This is a kolla deploy, queens | 18:29 |
lyarwood | attach device xml: <disk type="block" device="disk"> is wrong | 18:29 |
*** bkopilov_ has joined #openstack-nova | 18:29 | |
jmccarthy | lyarwood: Ok let me check that a bit more - I do have one setup that is working and one that isn't - I just can't see wtf is going on | 18:30 |
jmccarthy | lyarwood: Great - Ok, I'll go try and see what is the deal with that - thanks ! | 18:30 |
lyarwood | jmccarthy: can you just double check that line and make sure it wasn't cut off in your terminal etc | 18:32 |
jmccarthy | lyarwood: One sec | 18:32 |
*** mgoddard has quit IRC | 18:32 | |
jmccarthy | lyarwood: Ok sorry - part of that bit was missed - this ? https://paste.fedoraproject.org/paste/jR3KP6NlhvTjQZrTl~yK~w | 18:34 |
lyarwood | jmccarthy: yeah that's better | 18:34 |
lyarwood | jmccarthy: okay so the XML looks good | 18:34 |
lyarwood | jmccarthy: are you using fixed_key in nova.conf? | 18:35 |
jmccarthy | lyarwood: Nope - should I be ? I'm going by this here: https://docs.openstack.org/cinder/queens/configuration/block-storage/volume-encryption.html | 18:36 |
jmccarthy | lyarwood: I'm pretty sure they are both configured the same, but setup1 works and setup2 doesn't - I have the logs I'm just going crosseyed with where it's going wrong | 18:36 |
lyarwood | jmccarthy: is it the same key in both envs? | 18:37 |
lyarwood | jmccarthy: and is the key the same in nova.conf and cinder.conf? | 18:37 |
lyarwood | ah sorry that doc is using barbican | 18:37 |
jmccarthy | lyarwood: the setups are not connected to each other at all as such, just in the same lab | 18:38 |
*** bkopilov has joined #openstack-nova | 18:38 | |
jmccarthy | lyarwood: nothing is shared | 18:38 |
lyarwood | jmccarthy: kk can you do a cinder show of the volume? | 18:38 |
*** mdnadeem has quit IRC | 18:39 | |
jmccarthy | lyarwood: Should be this here: https://paste.fedoraproject.org/paste/SQ5quYSt8wKOV~-eTqLNMA | 18:40 |
jmccarthy | lyarwood: I have gone through this a few times, I think the logs aligh with where it's at atm | 18:40 |
jmccarthy | lyarwood: * s/aligh/align | 18:40 |
jmccarthy | lyarwood: maybe the log I sent earlier doesn't make senses .. it was grepped on req-138f35c8-ed21-4430-bf68-525574657926 | 18:42 |
lyarwood | jmccarthy: yeah there's an underlying libvirt issue here with the attach, I'm just trying to rule out some older known issues with passphrase length etc, for example https://bugzilla.redhat.com/show_bug.cgi?id=1447297 | 18:43 |
openstack | bugzilla.redhat.com bug 1447297 in libvirt "Unable to use LUKS passphrase that is exactly 16 bytes long" [Unspecified,Closed: errata] - Assigned to pkrempa | 18:43 |
*** gjayavelu has quit IRC | 18:43 | |
lyarwood | jmccarthy: final ask, `openstack volume type show luks` just to confirm the key_size | 18:44 |
lyarwood | ^ I think that's the correct command at least | 18:44 |
lyarwood | https://docs.openstack.org/python-openstackclient/pike/cli/command-objects/volume-type.html#volume-type-show yeah it is | 18:45 |
lyarwood | openstack volume type show --encryption-type luks | 18:45 |
jmccarthy | lyarwood: https://paste.fedoraproject.org/paste/34qbpIwDx867wt66JCnUlA | 18:47 |
lyarwood | jmccarthy: yeah 256 should be fine, odd. | 18:48 |
jmccarthy | lyarwood: I should be lioadm for the scsi_helper I think it is, right ? | 18:48 |
lyarwood | jmccarthy: is the same version of libvirt installed in both envs? | 18:48 |
lyarwood | jmccarthy: that really depends on the underlying env but that isn't the issue here | 18:49 |
*** dtantsur is now known as dtantsur|afk | 18:49 | |
lyarwood | jmccarthy: we can see the raw volume, crypsetup is able to format it etc | 18:49 |
lyarwood | jmccarthy: libvirt just borks when you try to attach it to the instance | 18:49 |
jmccarthy | lyarwood: I should have maybe mentioned earlier :/ one is tgdadm I think ? vs lioadm | 18:50 |
jmccarthy | lyarwood: Should be same version, have to confirm | 18:50 |
lyarwood | jmccarthy: I still can't see how that could cause this failure tbh | 18:51 |
*** mvenesio has joined #openstack-nova | 18:51 | |
jmccarthy | lyarwood: virsh --version says 3.9.0 on both | 18:52 |
jmccarthy | lyarwood: I know ? | 18:54 |
lyarwood | jmccarthy: kk, I'm out of ideas on the OpenStack / Nova side tbh, do you have libvirtd logs? | 18:54 |
lyarwood | jmccarthy: I need to drop in 5 btw before my wife kills me | 18:54 |
jmccarthy | lyarwood: Oh I did earlier lemme check again - they didn't say much - it's all good I'm fading also - I'll step through them tomorrow | 18:55 |
jmccarthy | lyarwood: Just more of this here `error : qemuMonitorJSONCheckError:389 : internal error: unable to execute QEMU command 'device_add': Property 'virtio-blk-device.drive' can't find value 'drive-virtio-disk1'` | 18:56 |
jmccarthy | lyarwood: Ok maybe I can get some more debug log out of that | 18:56 |
jmccarthy | lyarwood: Thanks for the help ! I thought maybe it was something obvious I was just missing, but I guess not | 18:56 |
lyarwood | jmccarthy: yeah increase the logging there and it might shed some light, iirc I've seen this before when the passphrase wouldn't unlock the volume, I just can't see how that's possible with the current flow tbh. | 18:57 |
* lyarwood drops \o_ | 18:58 | |
jmccarthy | lyarwood: o/ | 18:58 |
jmccarthy | lyarwood: \o lol | 18:58 |
jmccarthy | Zzz | 18:58 |
*** gouthamr has joined #openstack-nova | 18:59 | |
*** jmccarthy has left #openstack-nova | 18:59 | |
*** mvenesio has quit IRC | 19:00 | |
*** hamzy has quit IRC | 19:01 | |
*** wolverineav has quit IRC | 19:01 | |
*** elod has joined #openstack-nova | 19:02 | |
*** wolverineav has joined #openstack-nova | 19:02 | |
*** pchavva has quit IRC | 19:03 | |
*** wolverineav has quit IRC | 19:06 | |
*** r-daneel_ has joined #openstack-nova | 19:11 | |
*** r-daneel has quit IRC | 19:11 | |
*** r-daneel_ is now known as r-daneel | 19:11 | |
*** felipemonteiro__ has joined #openstack-nova | 19:11 | |
*** mvk has quit IRC | 19:13 | |
*** gouthamr has quit IRC | 19:13 | |
*** felipemonteiro_ has quit IRC | 19:15 | |
*** ociuhandu has joined #openstack-nova | 19:16 | |
*** gouthamr has joined #openstack-nova | 19:17 | |
*** hamzy has joined #openstack-nova | 19:18 | |
*** fragatina has quit IRC | 19:19 | |
*** links has joined #openstack-nova | 19:22 | |
*** ociuhandu has quit IRC | 19:27 | |
*** dklyle has quit IRC | 19:40 | |
*** links has quit IRC | 19:45 | |
*** cdent has quit IRC | 19:45 | |
*** awaugama has quit IRC | 19:47 | |
*** wolverineav has joined #openstack-nova | 19:50 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | 19:53 |
*** wolverineav has quit IRC | 19:54 | |
*** edmondsw has joined #openstack-nova | 19:54 | |
mriedem | dansmith: on this heal allocations CLI, i'm currently pulling instaces per cell and filtering by task_state=None since we don't want to mess with instances that are going through some task state transition, | 19:55 |
dansmith | yeah | 19:55 |
mriedem | however, for the purpose of reporting why we didn't create allocations for a given instance (like we do if it didn't have a host, e.g. shelved offloaded), | 19:55 |
mriedem | i'm thinking about removing that filter so we can just check instance.task_state in code and if not None, we log something and continue | 19:55 |
mriedem | because it would be weird to say we healed all instances when we don't know that we actually got all instances in a cell | 19:56 |
dansmith | okay | 19:56 |
dansmith | in many cases the instances with a task state outstanding will be a fraction of the total | 19:57 |
dansmith | not all cases, but probably reasonable as long as you're batching | 19:57 |
*** jmlowe has quit IRC | 19:58 | |
mriedem | by default we pull 50 instances per iteration | 19:58 |
dansmith | yeah, I mean I was stating that you were batching, not asking | 19:59 |
esberglu | efried: melwitt: mriedem: I've enabled run_validation and added scenario testing in PowerVM CI. This should unblock these three in the runway queue | 20:01 |
esberglu | Snapshot: https://review.openstack.org/#/c/543023/ | 20:01 |
esberglu | Localdisk: https://review.openstack.org/#/c/549053/ & https://review.openstack.org/#/c/549300/ | 20:01 |
esberglu | vSCSI is still blocked due to lack of CI, I have noted that on the etherpad | 20:01 |
dansmith | I thought it was going to make the runs take a long time? | 20:02 |
mriedem | ++lib/tempest:configure_tempest:429 [m iniset /opt/stack/tempest/etc/tempest.conf validation run_validation True | 20:02 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add consumer.generation/create missing consumers https://review.openstack.org/557958 | 20:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add Project, User and Consumer objects https://review.openstack.org/565403 | 20:04 |
mriedem | looks like most of the scenario tests are skipped which is what we talked about http://184.172.12.213/23/543023/20/check/nova-in-tree-pvm/ab0b59a/powervm_os_ci.html | 20:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404 | 20:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405 | 20:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | 20:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 20:04 |
esberglu | mriedem: Most of the scenario tests require floating ips and/or volume support which we don't have. I've enabled test_server_basic_ops which is the test specifically called out as blocking | 20:05 |
mriedem | esberglu: dansmith: melwitt: yeah tempest.scenario.test_server_basic_ops.TestServerBasicOps.test_server_basic_ops is what we wanted | 20:05 |
mriedem | yup | 20:05 |
mriedem | lgtm | 20:05 |
mriedem | test_server_basic_ops does ssh | 20:05 |
esberglu | mriedem: Yep | 20:05 |
mriedem | https://github.com/openstack/tempest/blob/8d3f0bec6e5e7f396ca1ae9c89978e88ae4f6aeb/tempest/scenario/test_server_basic_ops.py#L30 | 20:05 |
*** EranBD has joined #openstack-nova | 20:06 | |
esberglu | dansmith: test_server_basic_ops is takes a long time, but with tempest concurrency it doesn't actually do much to overall run time | 20:06 |
mriedem | 2018-05-09 12:32:43.192 23165 DEBUG tempest [-] compute-feature-enabled.config_drive = True log_opt_values /usr/local/lib/python2.7/dist-packages/oslo_config/cfg.py:2908 | 20:06 |
dansmith | well, right, it takes longer on other platforms too, but I thought that was the reason given for not enabling it | 20:06 |
mriedem | the volume stuff takes 10 minutes for them | 20:07 |
mriedem | and they don't have a good way to cleanup volume things in their backing storage | 20:07 |
dansmith | I don't understand what that means | 20:07 |
melwitt | which part of the log shows the pass/fail status of each test? I see in the console.txt it shows "will run" or "skip" | 20:08 |
dansmith | nor how it would be different for someone real trying to do this | 20:08 |
mriedem | melwitt: http://184.172.12.213/23/543023/20/check/nova-in-tree-pvm/ab0b59a/powervm_os_ci.html | 20:08 |
esberglu | melwitt: powervm_os_ci.html in the top level dir | 20:08 |
melwitt | ah, thanks | 20:08 |
esberglu | dansmith: test_server_basic_ops doesn't do much to overall runtime. But running all of the volume scenario tests will | 20:09 |
mriedem | volume storage isn't on a local disk, i think is the main difference | 20:09 |
esberglu | Plus the cleanup issue mriedem mentioned | 20:09 |
esberglu | I'm working on a vSCSI CI job that can be run on demand | 20:09 |
dansmith | right, I don't understand the cleanup issue specifically | 20:09 |
mriedem | it can't handle the number of concurrent volume creates and deletes is my understanding | 20:10 |
mriedem | because it's enterprise storage :) | 20:10 |
dansmith | lol | 20:10 |
mriedem | i remember when i was running powerkvm ci and we had a similar setup, | 20:11 |
mriedem | it was bricked at least once per week, and that was serial runs twice per day | 20:11 |
dansmith | the original powervm ci was similar IIRC | 20:11 |
mriedem | *powerkvm ci for the ibm distro thing, not pkvm that ibm ltc runs | 20:11 |
esberglu | ouch | 20:11 |
mriedem | granted our cleanup bash scripts probably sucked | 20:12 |
mriedem | and i didn't know enough VIOS CLI to do much good, | 20:12 |
mriedem | but i remember having to sametime the lab admin once a week who eventually gave me creds to the IMM to just reboot the thing | 20:12 |
mriedem | i think he eventually pre-empted my sametimes because he knew we were breaking it | 20:13 |
*** lpetrut has quit IRC | 20:14 | |
mriedem | looking at these times, it's not much difference | 20:14 |
mriedem | IBM PowerVM CI (1 rechecks)1:33 PMnova-out-of-tree-pvmSUCCESS in 1h 27m 36snova-in-tree-pvmSUCCESS in 1h 31m 07s | 20:14 |
mriedem | efried: i think you can drop the -2 on this now https://review.openstack.org/#/c/543023/ | 20:15 |
*** zcorneli has joined #openstack-nova | 20:15 | |
efried | mriedem: Done. | 20:15 |
*** zcorneli has quit IRC | 20:17 | |
*** germs has joined #openstack-nova | 20:18 | |
*** zcorneli has joined #openstack-nova | 20:18 | |
*** zcorneli has quit IRC | 20:21 | |
*** zcorneli has joined #openstack-nova | 20:21 | |
*** awaugama has joined #openstack-nova | 20:22 | |
*** germs has quit IRC | 20:22 | |
*** germs has joined #openstack-nova | 20:27 | |
*** germs has quit IRC | 20:27 | |
*** germs has joined #openstack-nova | 20:27 | |
*** germs has quit IRC | 20:27 | |
*** vladikr has quit IRC | 20:28 | |
openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Localdisk https://review.openstack.org/549300 | 20:36 |
*** vladikr has joined #openstack-nova | 20:40 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add nova-manage placement heal_allocations CLI https://review.openstack.org/565886 | 20:40 |
*** dklyle has joined #openstack-nova | 20:48 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Deduplicate config/policy reference docs from main index https://review.openstack.org/564846 | 20:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement granular policy rules for placement https://review.openstack.org/524425 | 20:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578 | 20:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup placement policy generator docs https://review.openstack.org/565225 | 20:53 |
*** jmlowe has joined #openstack-nova | 20:53 | |
*** tbachman has quit IRC | 20:53 | |
*** pcaruana has quit IRC | 20:59 | |
*** hamzy has quit IRC | 21:01 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement granular policy rules for placement https://review.openstack.org/524425 | 21:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578 | 21:01 |
*** mvenesio has joined #openstack-nova | 21:10 | |
*** gabor_antal has quit IRC | 21:10 | |
*** gabor_antal has joined #openstack-nova | 21:10 | |
*** hamzy has joined #openstack-nova | 21:11 | |
*** Guest53916 has quit IRC | 21:13 | |
*** vladikr has quit IRC | 21:13 | |
*** hamzy has quit IRC | 21:15 | |
*** gouthamr has quit IRC | 21:16 | |
*** dave-mccowan has quit IRC | 21:17 | |
*** gabor_antal has quit IRC | 21:20 | |
*** dave-mccowan has joined #openstack-nova | 21:22 | |
*** gabor_antal has joined #openstack-nova | 21:22 | |
*** mvenesio has quit IRC | 21:24 | |
*** fragatina has joined #openstack-nova | 21:26 | |
*** fragatina has quit IRC | 21:26 | |
*** vladikr has joined #openstack-nova | 21:27 | |
*** fragatina has joined #openstack-nova | 21:27 | |
efried | mriedem: Is there any exception to the rule of proposing a thing in master and then backporting it? I.e. any exception process by which we can propose directly to stable without a corresponding master patch? | 21:27 |
edmondsw | if a bug only exists in a stable branch, then it'd be silly to merge something in master just to backport it to stable | 21:27 |
edmondsw | and then revert master | 21:28 |
edmondsw | so there must be cases where we make changes directly in a stable branch | 21:28 |
*** hamzy_ has joined #openstack-nova | 21:29 | |
*** hamzy_ is now known as hamzy | 21:30 | |
mriedem | efried: yes if the bug doesn't exist in master for the most part | 21:39 |
efried | mriedem: Okay, cool. edmondsw esberglu ^ | 21:39 |
*** jmlowe has quit IRC | 21:39 | |
mriedem | https://docs.openstack.org/project-team-guide/stable-branches.html#appropriate-fixes | 21:39 |
efried | mriedem: We fixed it in pypowervm 1.1.15 which is now minimum in master nova, but we can't backport the req bump to stable, so we have to do a workaround in nova. | 21:39 |
*** gouthamr has joined #openstack-nova | 21:42 | |
*** awaugama has quit IRC | 21:42 | |
mriedem | melwitt: you can post backports for this now https://review.openstack.org/#/c/555945/ | 21:48 |
mriedem | efried: i thought the breaking oslo lib change was reverted and blacklisted on stable? | 21:49 |
efried | mriedem: It wasn't reverted exactly; and the way in which it was "fixed" doesn't actually fix us. esberglu is posting the workaround; that commit message will have more details. | 21:50 |
efried | gory, gruesome details. | 21:50 |
mriedem | awesome | 21:53 |
mriedem | i'm going to delegate to tonyb and bauzas | 21:53 |
efried | mriedem: No worries, I was tapping you as the resident process nazi^Wguru to see if it was even legal to skip master. | 21:54 |
* mriedem wonders how efried knew i had that new stylish stache | 21:55 | |
efried | drone cameras, baby. | 21:55 |
*** gouthamr has quit IRC | 21:59 | |
mriedem | zzzeek: ever seen "2018-05-09 16:29:33.278 11031 ERROR nova.api.openstack.extensions DBError: (pymysql.err.InternalError) (1, u"Can't create/write to file '/tmp/#sql_4038_1.MAI' (Errcode: 2)")" ? | 21:59 |
mriedem | in RHOSP 11 | 21:59 |
zzzeek | mriedem: there's a lot of files in /tmp/ and /tmp/ can run out of space or have fs problems | 21:59 |
zzzeek | mriedem: it also does this trick where it unlinks the file before using it | 22:00 |
zzzeek | so you can't see them with ls | 22:00 |
openstackgerrit | Merged openstack/nova stable/pike: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553829 | 22:01 |
openstackgerrit | Merged openstack/nova master: Expose driver_block_device fields consistently https://review.openstack.org/528362 | 22:01 |
mriedem | ok; was just doing bug triage and someone reported that when listing instances | 22:01 |
openstackgerrit | Merged openstack/nova stable/queens: Fix detach_volume calls when rolling back a failed attach https://review.openstack.org/567232 | 22:01 |
openstackgerrit | Merged openstack/nova stable/queens: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/567273 | 22:01 |
mriedem | i'll leave that info, thanks | 22:01 |
zzzeek | mriedem: i should find the command we used to see them, it was someting unpleasant | 22:04 |
zzzeek | oh it was lsof | 22:04 |
zzzeek | but , you see the same files over and over again | 22:04 |
openstackgerrit | Moshe Levi proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | 22:04 |
zzzeek | mriedem: https://dev.mysql.com/doc/refman/8.0/en/temporary-files.html "MySQL arranges that temporary files are removed if mysqld is terminated. On platforms that support it (such as Unix), this is done by unlinking the file after opening it. The disadvantage of this is that the name does not appear in directory listings and you do not see a big temporary file that fills up the file system in which the temporary file directory is | 22:06 |
zzzeek | located. (In such cases, lsof +L1 may be helpful in identifying large files associated with mysqld.) " | 22:06 |
*** felipemonteiro__ has quit IRC | 22:06 | |
mriedem | jroll: https://bugs.launchpad.net/nova/+bug/1769817 if you have any insight into that | 22:07 |
openstack | Launchpad bug 1769817 in OpenStack Compute (nova) "Ironic serial console doesn't get disabled when instance is deleted" [Undecided,Confirmed] | 22:07 |
*** gbarros has joined #openstack-nova | 22:10 | |
*** slaweq has quit IRC | 22:10 | |
*** esberglu has quit IRC | 22:10 | |
*** slaweq has joined #openstack-nova | 22:10 | |
*** zigo has quit IRC | 22:11 | |
*** zigo has joined #openstack-nova | 22:11 | |
*** gouthamr has joined #openstack-nova | 22:14 | |
*** liverpooler has quit IRC | 22:15 | |
*** mlavalle has quit IRC | 22:19 | |
*** tbachman has joined #openstack-nova | 22:19 | |
mriedem | jesus just triaged 2 separate duplicate bugs for this https://review.openstack.org/#/c/564336/ - people must really be getting screwed on that; dansmith - probably good to ^ get in | 22:20 |
mriedem | err https://review.openstack.org/#/c/564336/ | 22:20 |
*** gbarros has quit IRC | 22:21 | |
*** priteau has quit IRC | 22:22 | |
*** tbachman has quit IRC | 22:25 | |
*** vladikr has quit IRC | 22:26 | |
mriedem | does this ring a bell? https://bugs.launchpad.net/nova/+bug/1769025 | 22:29 |
openstack | Launchpad bug 1769025 in OpenStack Compute (nova) "OrphanedObjectError: Cannot call obj_load_attr on orphaned Instance object error is thrown for ironic deploy in Pike" [Undecided,New] | 22:29 |
mriedem | that's on pike | 22:29 |
*** oomichi has joined #openstack-nova | 22:30 | |
*** mvk has joined #openstack-nova | 22:32 | |
*** threestrands has joined #openstack-nova | 22:33 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 22:35 |
*** threestrands_ has joined #openstack-nova | 22:36 | |
*** threestrands_ has joined #openstack-nova | 22:36 | |
*** vladikr has joined #openstack-nova | 22:38 | |
*** threestrands has quit IRC | 22:38 | |
*** hamzy has quit IRC | 22:39 | |
mriedem | ooo upcalls https://bugs.launchpad.net/nova/+bug/1768876 | 22:39 |
openstack | Launchpad bug 1768876 in OpenStack Compute (nova) "Old instances can get AVZ from metadata" [Undecided,New] - Assigned to Surya Seetharaman (tssurya) | 22:39 |
mriedem | so apparently the metadata-api used to do an up-call to the API DB to get the AZ for an instance, but in pike we started storing the AZ directly on the instance rather than the dumb default 'nova' AZ, | 22:41 |
mriedem | but for any instances created before pike, they aren't going to have that, so they needed the up-call | 22:41 |
*** r-daneel has quit IRC | 22:43 | |
*** hongbin has quit IRC | 22:43 | |
openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Granular requests to get_allocation_candidates https://review.openstack.org/515811 | 22:43 |
*** vladikr has quit IRC | 22:45 | |
*** EranBD has quit IRC | 22:47 | |
openstackgerrit | Merged openstack/nova stable/queens: Make association_refresh configurable https://review.openstack.org/566288 | 22:48 |
*** gouthamr has quit IRC | 22:52 | |
*** gouthamr has joined #openstack-nova | 22:55 | |
*** dklyle has quit IRC | 22:57 | |
*** andreas_s has joined #openstack-nova | 22:57 | |
*** vladikr has joined #openstack-nova | 22:58 | |
mriedem | melwitt: everything is ready for queens except https://review.openstack.org/#/c/566367/ which is still on master | 23:00 |
mriedem | will redouble my bugging efforts in the wee morn | 23:00 |
mriedem | same for pike and ocata actually | 23:00 |
*** andreas_s has quit IRC | 23:01 | |
*** mriedem has quit IRC | 23:02 | |
tonyb | efried, esberglu: Add me as a reviewer on the patch and I'll review it with my stable hat | 23:02 |
*** edmondsw has quit IRC | 23:10 | |
*** edmondsw has joined #openstack-nova | 23:11 | |
*** edmondsw has quit IRC | 23:15 | |
openstackgerrit | Merged openstack/nova master: Remove mox in test_xenapi.py (1) https://review.openstack.org/564639 | 23:25 |
*** moshele has joined #openstack-nova | 23:35 | |
*** takashin has joined #openstack-nova | 23:42 | |
*** hshiina|afk has quit IRC | 23:45 | |
*** stakeda has joined #openstack-nova | 23:45 | |
*** lpetrut has joined #openstack-nova | 23:47 | |
*** Swami has quit IRC | 23:49 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (2) https://review.openstack.org/564643 | 23:51 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (3) https://review.openstack.org/564645 | 23:51 |
*** lpetrut has quit IRC | 23:52 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!