openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Report KVM hypervisor_type in hypervisor API https://review.openstack.org/554075 | 00:01 |
---|---|---|
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: disable cpu pinning with libvirt/qemu driver https://review.openstack.org/531049 | 00:02 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: enable cpu pinning with libvirt/qemu driver https://review.openstack.org/554076 | 00:02 |
*** odyssey4me has quit IRC | 00:09 | |
*** odyssey4me has joined #openstack-nova | 00:09 | |
*** fgsukhe has joined #openstack-nova | 00:10 | |
*** xinliang has quit IRC | 00:11 | |
*** hongbin has joined #openstack-nova | 00:12 | |
*** Zames has joined #openstack-nova | 00:12 | |
*** yamamoto has joined #openstack-nova | 00:13 | |
*** yamamoto has quit IRC | 00:18 | |
*** Zames has quit IRC | 00:22 | |
*** xinliang has joined #openstack-nova | 00:24 | |
*** claudiub|2 has quit IRC | 00:27 | |
*** hshiina has joined #openstack-nova | 00:28 | |
*** amodi has quit IRC | 00:29 | |
*** vladikr has quit IRC | 00:35 | |
*** vladikr has joined #openstack-nova | 00:36 | |
*** amodi has joined #openstack-nova | 00:46 | |
*** amodi has quit IRC | 00:50 | |
*** liuzz has joined #openstack-nova | 00:55 | |
*** phuongnh has joined #openstack-nova | 00:56 | |
*** gouthamr has quit IRC | 01:03 | |
*** r-daneel has joined #openstack-nova | 01:06 | |
openstackgerrit | Michael Still proposed openstack/nova master: Move configurable mkfs to privsep. https://review.openstack.org/551921 | 01:08 |
openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi xenstore_read's to privsep. https://review.openstack.org/552241 | 01:08 |
openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242 | 01:08 |
openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi partition copies to privsep. https://review.openstack.org/553605 | 01:08 |
openstackgerrit | Michael Still proposed openstack/nova master: Sync xenapi and libvirt on what flags to pass e2fsck. https://review.openstack.org/554078 | 01:08 |
*** zhaochao has joined #openstack-nova | 01:10 | |
*** yamamoto has joined #openstack-nova | 01:13 | |
*** tiendc has joined #openstack-nova | 01:14 | |
*** zhurong has joined #openstack-nova | 01:16 | |
*** yamamoto has quit IRC | 01:19 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 01:24 |
*** gouthamr has joined #openstack-nova | 01:25 | |
*** r-daneel_ has joined #openstack-nova | 01:32 | |
*** r-daneel has quit IRC | 01:33 | |
*** r-daneel_ is now known as r-daneel | 01:33 | |
*** yamamoto has joined #openstack-nova | 01:41 | |
*** OctopusZhang__ has joined #openstack-nova | 01:45 | |
*** elmaciej has quit IRC | 01:51 | |
*** hoangcx has joined #openstack-nova | 01:54 | |
*** tbachman has joined #openstack-nova | 01:55 | |
*** annp has joined #openstack-nova | 02:27 | |
*** zhurong has quit IRC | 02:36 | |
*** zhurong has joined #openstack-nova | 02:37 | |
*** tbachman has quit IRC | 02:40 | |
*** esberglu has joined #openstack-nova | 02:43 | |
*** jichen has joined #openstack-nova | 02:50 | |
*** psachin has joined #openstack-nova | 02:50 | |
*** salv-orl_ has joined #openstack-nova | 02:51 | |
*** salv-orlando has quit IRC | 02:53 | |
jichen | melwitt: hi, could you please help to review https://review.openstack.org/#/q/topic:bp/add-zvm-driver-rocky+(status:open+OR+status:merged) as it's listed https://etherpad.openstack.org/p/rocky-nova-priorities-tracking 'Non-priority approved blueprints' but no review since PTG end and I think better to request review/approval before cores get busy .... | 02:54 |
*** esberglu has quit IRC | 02:58 | |
openstackgerrit | YaoZheng proposed openstack/nova master: remove _cleanup_running_deleted_instances repeat detach volume https://review.openstack.org/554090 | 03:01 |
*** sapd has joined #openstack-nova | 03:02 | |
openstackgerrit | Hironori Shiina proposed openstack/nova master: ironic: Proxy a few attributes of block device mapping https://review.openstack.org/554093 | 03:08 |
*** germs has joined #openstack-nova | 03:17 | |
*** germs has quit IRC | 03:17 | |
*** germs has joined #openstack-nova | 03:17 | |
*** namnh has joined #openstack-nova | 03:21 | |
*** germs has quit IRC | 03:22 | |
*** rcernin_ has joined #openstack-nova | 03:23 | |
*** rcernin has quit IRC | 03:25 | |
*** rcernin_ has quit IRC | 03:29 | |
*** rcernin has joined #openstack-nova | 03:29 | |
*** rcernin has quit IRC | 03:33 | |
*** Zames has joined #openstack-nova | 03:34 | |
*** vladikr has quit IRC | 03:36 | |
*** vladikr has joined #openstack-nova | 03:36 | |
*** Zames has quit IRC | 03:37 | |
*** esberglu has joined #openstack-nova | 03:39 | |
*** sree has joined #openstack-nova | 03:40 | |
*** Kevin_Zheng has quit IRC | 03:40 | |
*** pooja_jadhav has joined #openstack-nova | 03:43 | |
*** esberglu has quit IRC | 03:43 | |
*** hongbin has quit IRC | 03:44 | |
*** udesale has joined #openstack-nova | 03:46 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_numa_memnode() https://review.openstack.org/529906 | 03:47 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_XXXpin_cpuset() https://review.openstack.org/527631 | 03:47 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add NumaTopology support for libvirt/qemu driver https://review.openstack.org/530451 | 03:47 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: trivial: omit condition evaluations https://review.openstack.org/545248 | 03:47 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: disable cpu pinning with libvirt/qemu driver https://review.openstack.org/531049 | 03:47 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: enable cpu pinning with libvirt/qemu driver https://review.openstack.org/554076 | 03:47 |
*** rcernin has joined #openstack-nova | 03:49 | |
*** janki has joined #openstack-nova | 03:50 | |
*** janki is now known as janki|office_ina | 03:51 | |
*** wxy has quit IRC | 03:56 | |
*** Dinesh_Bhor has joined #openstack-nova | 04:13 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix comments in get_all_with_shared() https://review.openstack.org/533195 | 04:17 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix hypervisor_type in /os-hypervisors/details https://review.openstack.org/531328 | 04:21 |
*** sree_ has joined #openstack-nova | 04:29 | |
*** sree_ is now known as Guest31421 | 04:29 | |
*** abhishekk has joined #openstack-nova | 04:31 | |
*** sree has quit IRC | 04:32 | |
*** sapd has quit IRC | 04:32 | |
*** bhagyashri_s is now known as bhagyashris | 04:33 | |
*** bhagyashris is now known as bhagyashri_s | 04:33 | |
*** bhagyashri_s is now known as bhagyashris | 04:33 | |
*** Zames has joined #openstack-nova | 04:35 | |
*** itlinux has quit IRC | 04:39 | |
*** sapd has joined #openstack-nova | 04:47 | |
*** suresh12 has joined #openstack-nova | 04:50 | |
*** claudiub has joined #openstack-nova | 04:52 | |
*** sridharg has joined #openstack-nova | 04:53 | |
*** gouthamr has quit IRC | 04:54 | |
*** Dinesh__Bhor has joined #openstack-nova | 04:59 | |
*** Dinesh_Bhor has quit IRC | 05:00 | |
*** lpetrut has joined #openstack-nova | 05:01 | |
*** zhurong has quit IRC | 05:04 | |
openstackgerrit | Merged openstack/nova master: Add --by-service to discover_hosts https://review.openstack.org/552691 | 05:07 |
*** suresh12 has quit IRC | 05:14 | |
*** imacdonn has quit IRC | 05:14 | |
*** imacdonn has joined #openstack-nova | 05:14 | |
*** suresh12 has joined #openstack-nova | 05:14 | |
*** germs has joined #openstack-nova | 05:18 | |
*** ratailor has joined #openstack-nova | 05:18 | |
openstackgerrit | wangqi proposed openstack/nova master: use http code constant instead of int https://review.openstack.org/554121 | 05:20 |
*** suresh12 has quit IRC | 05:22 | |
*** Zames has quit IRC | 05:22 | |
*** germs has quit IRC | 05:23 | |
*** sree_ has joined #openstack-nova | 05:33 | |
*** sree_ is now known as Guest5888 | 05:34 | |
*** Guest31421 has quit IRC | 05:37 | |
*** mdnadeem has joined #openstack-nova | 05:39 | |
*** sidx64 has joined #openstack-nova | 05:47 | |
*** openstackgerrit has quit IRC | 05:49 | |
*** lpetrut has quit IRC | 05:50 | |
*** udesale has quit IRC | 05:51 | |
*** udesale has joined #openstack-nova | 05:52 | |
*** trinaths has joined #openstack-nova | 05:54 | |
*** kholkina has joined #openstack-nova | 05:55 | |
*** kholkina has quit IRC | 05:59 | |
*** Dinesh__Bhor has quit IRC | 05:59 | |
*** Dinesh_Bhor has joined #openstack-nova | 06:01 | |
*** openstackgerrit has joined #openstack-nova | 06:03 | |
openstackgerrit | Hironori Shiina proposed openstack/nova master: ironic: Proxy a few attributes of block device mapping https://review.openstack.org/554093 | 06:03 |
*** gaurangt has quit IRC | 06:04 | |
*** sapd has quit IRC | 06:09 | |
*** cfriesen has quit IRC | 06:12 | |
*** amodi has joined #openstack-nova | 06:18 | |
*** Dinesh_Bhor has quit IRC | 06:22 | |
*** Dinesh_Bhor has joined #openstack-nova | 06:25 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772 | 06:26 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Amend the "add extra-specs to flavor" for create and update API https://review.openstack.org/554134 | 06:29 |
*** Dinesh_Bhor has quit IRC | 06:29 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Amend the "add extra-specs to flavor" for create and update API https://review.openstack.org/554134 | 06:30 |
*** Dinesh_Bhor has joined #openstack-nova | 06:31 | |
*** lajoskatona has joined #openstack-nova | 06:32 | |
*** udesale_ has joined #openstack-nova | 06:34 | |
openstackgerrit | wangqi proposed openstack/nova master: use http code constant instead of int https://review.openstack.org/554121 | 06:37 |
*** udesale has quit IRC | 06:38 | |
*** sidx64_ has joined #openstack-nova | 06:38 | |
*** alexchadin has joined #openstack-nova | 06:39 | |
*** Zames has joined #openstack-nova | 06:40 | |
*** sidx64 has quit IRC | 06:40 | |
*** fragatina has quit IRC | 06:42 | |
*** sidx64_ has quit IRC | 06:43 | |
*** sidx64 has joined #openstack-nova | 06:44 | |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791 | 06:45 |
*** udesale_ is now known as udesale | 06:47 | |
*** kholkina has joined #openstack-nova | 06:49 | |
*** Zames has quit IRC | 06:50 | |
*** zhurong has joined #openstack-nova | 07:00 | |
*** sidx64 has quit IRC | 07:04 | |
*** rcernin has quit IRC | 07:07 | |
*** sidx64 has joined #openstack-nova | 07:11 | |
*** Shilpa has joined #openstack-nova | 07:14 | |
*** suresh12 has joined #openstack-nova | 07:17 | |
*** sidx64 has quit IRC | 07:17 | |
*** udesale_ has joined #openstack-nova | 07:18 | |
*** udesale has quit IRC | 07:18 | |
*** sidx64 has joined #openstack-nova | 07:18 | |
*** germs has joined #openstack-nova | 07:18 | |
*** germs has quit IRC | 07:18 | |
*** germs has joined #openstack-nova | 07:18 | |
*** sidx64 has quit IRC | 07:20 | |
*** suresh12 has quit IRC | 07:21 | |
*** germs has quit IRC | 07:23 | |
*** sidx64 has joined #openstack-nova | 07:23 | |
*** sidx64 has quit IRC | 07:24 | |
*** sidx64 has joined #openstack-nova | 07:26 | |
*** sidx64 has quit IRC | 07:26 | |
*** amodi has quit IRC | 07:27 | |
*** sidx64 has joined #openstack-nova | 07:28 | |
*** andreas_s has joined #openstack-nova | 07:38 | |
*** Zames has joined #openstack-nova | 07:38 | |
*** Zames has quit IRC | 07:38 | |
*** pcaruana has joined #openstack-nova | 07:39 | |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791 | 07:41 |
*** claudiub has quit IRC | 07:48 | |
*** trinaths has quit IRC | 07:53 | |
*** AlexeyAbashkin has joined #openstack-nova | 07:54 | |
*** zhurong has quit IRC | 07:54 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Remove dependency to the old aggregate-based-pools https://review.openstack.org/554154 | 07:55 |
*** OctopusZhang has joined #openstack-nova | 07:55 | |
*** lpetrut has joined #openstack-nova | 07:57 | |
*** brault_ has joined #openstack-nova | 07:57 | |
*** OctopusZhang is now known as yufei | 07:59 | |
openstackgerrit | wangqi proposed openstack/nova master: use http code constant instead of int https://review.openstack.org/554121 | 07:59 |
*** brault has quit IRC | 07:59 | |
*** OctopusZhang__ has quit IRC | 07:59 | |
*** lpetrut_ has joined #openstack-nova | 08:00 | |
*** lpetrut has quit IRC | 08:00 | |
*** vivsoni_ has joined #openstack-nova | 08:01 | |
openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Update docs, API samples for noVNC 1.0.0 https://review.openstack.org/550173 | 08:08 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [WIP] Add microversion to support extra_specs in flavor API. https://review.openstack.org/554159 | 08:11 |
*** maciejjozefczyk has joined #openstack-nova | 08:14 | |
*** tesseract has joined #openstack-nova | 08:17 | |
openstackgerrit | jichenjc proposed openstack/nova master: Move test_allocation_candidates.py https://review.openstack.org/553149 | 08:22 |
*** udesale_ has quit IRC | 08:23 | |
openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | 08:23 |
*** elmaciej has joined #openstack-nova | 08:24 | |
*** aloga has quit IRC | 08:24 | |
*** janki|office_ina has quit IRC | 08:25 | |
*** udesale_ has joined #openstack-nova | 08:30 | |
*** sidx64 has quit IRC | 08:33 | |
*** sidx64 has joined #openstack-nova | 08:33 | |
*** sidx64 has quit IRC | 08:34 | |
*** Dinesh_Bhor has quit IRC | 08:34 | |
*** Tom-Tom has joined #openstack-nova | 08:35 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Remove dependency to the old aggregate-based-pools https://review.openstack.org/554154 | 08:36 |
*** ccamacho has joined #openstack-nova | 08:36 | |
*** lyarwood has joined #openstack-nova | 08:37 | |
*** tetsuro has joined #openstack-nova | 08:38 | |
*** slaweq has quit IRC | 08:38 | |
*** jaosorior has joined #openstack-nova | 08:40 | |
*** Dinesh_Bhor has joined #openstack-nova | 08:40 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 08:41 |
*** claudiub has joined #openstack-nova | 08:42 | |
openstackgerrit | YaoZheng proposed openstack/nova master: remove _cleanup_running_deleted_instances repeat detach volume https://review.openstack.org/554090 | 08:44 |
*** amoralej|off is now known as amoralej | 08:45 | |
openstackgerrit | licanwei proposed openstack/nova master: Make nova-manage capable of syncing all cell databases https://review.openstack.org/519275 | 08:48 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration https://review.openstack.org/554154 | 08:48 |
*** ragiman has joined #openstack-nova | 08:50 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451 | 08:50 |
openstackgerrit | jichenjc proposed openstack/nova master: Move placement test cases from db to placement https://review.openstack.org/553149 | 08:52 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration https://review.openstack.org/554154 | 08:53 |
*** Tom-Tom has quit IRC | 08:54 | |
*** maciejjozefczyk has quit IRC | 08:54 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration https://review.openstack.org/554154 | 08:54 |
*** maciejjozefczyk has joined #openstack-nova | 08:55 | |
*** Eran_Kuris has joined #openstack-nova | 08:56 | |
*** elod_off is now known as elod | 08:56 | |
*** yamahata has quit IRC | 08:58 | |
*** jpena|off is now known as jpena | 08:59 | |
*** Guest5888 has quit IRC | 09:00 | |
*** slaweq has joined #openstack-nova | 09:00 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451 | 09:00 |
*** sree_ has joined #openstack-nova | 09:04 | |
*** sree_ is now known as Guest45662 | 09:04 | |
*** sar has joined #openstack-nova | 09:06 | |
*** Guest45662 has quit IRC | 09:08 | |
*** salv-orl_ has quit IRC | 09:09 | |
*** salv-orlando has joined #openstack-nova | 09:09 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration https://review.openstack.org/554154 | 09:13 |
*** hshiina has quit IRC | 09:13 | |
*** salv-orlando has quit IRC | 09:14 | |
*** tssurya has joined #openstack-nova | 09:16 | |
*** psachin has quit IRC | 09:19 | |
*** hoonetorg has quit IRC | 09:19 | |
*** rmart04 has joined #openstack-nova | 09:20 | |
*** psachin has joined #openstack-nova | 09:24 | |
openstackgerrit | zhufl proposed openstack/nova master: Fix typos in release notes https://review.openstack.org/554175 | 09:25 |
*** sahid has joined #openstack-nova | 09:25 | |
*** Eran_Kuris has quit IRC | 09:28 | |
*** danpawlik has joined #openstack-nova | 09:29 | |
*** mgoddard has joined #openstack-nova | 09:29 | |
*** udesale_ has quit IRC | 09:30 | |
*** gibi_ is now known as gibi | 09:30 | |
gibi | good morning | 09:30 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Amend the "add extra-specs to flavor" for create and update API https://review.openstack.org/554134 | 09:31 |
*** ragiman has quit IRC | 09:31 | |
*** Eran_Kuris has joined #openstack-nova | 09:31 | |
*** udesale has joined #openstack-nova | 09:32 | |
*** hoonetorg has joined #openstack-nova | 09:32 | |
*** BlackDex has quit IRC | 09:34 | |
*** Dinesh_Bhor has quit IRC | 09:34 | |
*** BlackDex has joined #openstack-nova | 09:35 | |
*** yufei has quit IRC | 09:37 | |
*** cdent has joined #openstack-nova | 09:37 | |
*** trinaths has joined #openstack-nova | 09:42 | |
*** gus has quit IRC | 09:43 | |
*** yufei has joined #openstack-nova | 09:44 | |
*** sidx64 has joined #openstack-nova | 09:44 | |
*** gus has joined #openstack-nova | 09:44 | |
*** ragiman has joined #openstack-nova | 09:45 | |
*** yufei has quit IRC | 09:48 | |
openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Do not depend on jenkins user in devstack gate https://review.openstack.org/552476 | 09:49 |
gibi | bauzas: regarding the https://github.com/openstack/oslo.versionedobjects/commit/0e3526710f67b3b4ebab60864ea060fa9caf9537 and the hypens in UUIDs. My intention in that ovo patch was to fix the usage error of the warning filter in ovo | 09:50 |
cdent | mornin' | 09:51 |
openstackgerrit | Tetiana Lashchova proposed openstack/nova-specs master: Allow modification of user-data via the server update https://review.openstack.org/547964 | 09:56 |
*** diranerpi has joined #openstack-nova | 10:00 | |
diranerpi | Hi all There is a way to use instance metadata to schedule instance deployment on a specific host? | 10:02 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: docs: Disable smartquotes https://review.openstack.org/554188 | 10:05 |
*** namnh has quit IRC | 10:05 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: docs: Disable smartquotes https://review.openstack.org/554189 | 10:05 |
diranerpi | or on a specific host aggregate ? | 10:05 |
*** trinaths has quit IRC | 10:05 | |
*** hoangcx has quit IRC | 10:08 | |
*** sambetts_ is now known as sambetts | 10:08 | |
*** mvk has joined #openstack-nova | 10:13 | |
*** yamamoto has quit IRC | 10:16 | |
*** dtantsur|afk is now known as dtantsur | 10:19 | |
johnthetubaguy | diranerpi: there are almost too many options right now, have a look at most of the filters with the word "aggregate" in them: https://docs.openstack.org/nova/latest/user/filter-scheduler.html | 10:24 |
bhagyashris | johnthetubguy: Hi, | 10:25 |
bhagyashris | johnthetubguy: I just want to discuss deeply regarding "Skip instance backup image creation when rotation 0" as we discussed in Dublin PTG #link: https://etherpad.openstack.org/p/nova-ptg-rocky so as per the suggestion I am following the the idea mentioned in etherpad 1. add a microversion to disallow 0 using the schema validation while creating the backup image | 10:29 |
bhagyashris | johnthetubguy: 2. add a new API for "purge all backups" to be used instead of passing 0 so for this I have introduce new cleanup-backup api to delete the backup images of specified instance in command | 10:31 |
bhagyashris | johnthetubguy: so i am on the right patch about this idea | 10:31 |
bhagyashris | johnthetubguy: s/patch/path | 10:32 |
*** phuongnh has quit IRC | 10:33 | |
bhagyashris | johnthetubaguy: Hi, Sorry for spelling mistake in name ^ | 10:37 |
*** annp has quit IRC | 10:37 | |
diranerpi | johnthetubaguy: hi, i've read those filter but no one responds to my use case (i want to use instance metadata to choose which host aggregate will be used) | 10:37 |
*** ragiman has quit IRC | 10:38 | |
*** avolkov has joined #openstack-nova | 10:38 | |
diranerpi | So for example, if in my instance data i create a metadata like "service = database", i want this instance to be deployed on a specific aggregate host | 10:38 |
*** tiendc has quit IRC | 10:39 | |
diranerpi | johnthetubaguy: So for example, if in my instance data i create a metadata like "service = database", i want this instance to be deployed on a specific aggregate host | 10:39 |
*** tbachman has joined #openstack-nova | 10:40 | |
*** sree has joined #openstack-nova | 10:43 | |
*** sree has quit IRC | 10:48 | |
openstackgerrit | Silvan Kaiser proposed openstack/nova master: Exec systemd-run with privileges in Quobyte driver https://review.openstack.org/554195 | 10:50 |
*** ragiman has joined #openstack-nova | 10:50 | |
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 | 10:54 |
*** sdague has joined #openstack-nova | 10:55 | |
*** yamamoto has joined #openstack-nova | 11:02 | |
*** salv-orlando has joined #openstack-nova | 11:05 | |
*** abhishekk has quit IRC | 11:06 | |
johnthetubaguy | bhagyashris: hi | 11:09 |
johnthetubaguy | bhagyashris: that sounds spot on | 11:09 |
*** edmondsw has joined #openstack-nova | 11:14 | |
bhagyashris | johnthetubaguy: ok thank you :) actually cleanup-backup api will purge all the backup images of instance so is that be ok? or should i give the provision to cleanup the backup images for example cleanup last two backups etc? | 11:17 |
*** edmondsw has quit IRC | 11:18 | |
*** germs has joined #openstack-nova | 11:20 | |
*** germs has quit IRC | 11:20 | |
*** germs has joined #openstack-nova | 11:20 | |
*** alexchadin has quit IRC | 11:21 | |
*** alexchadin has joined #openstack-nova | 11:21 | |
*** alexchadin has quit IRC | 11:21 | |
*** alexchadin has joined #openstack-nova | 11:22 | |
*** alexchadin has quit IRC | 11:22 | |
*** alexchadin has joined #openstack-nova | 11:23 | |
*** alexchadin has quit IRC | 11:23 | |
*** alexchadin has joined #openstack-nova | 11:24 | |
*** alexchadin has quit IRC | 11:24 | |
bhagyashris | johnthetubaguy: and on current master at the time of createbackup the delete image backup is implementated so should i remove that implementation from create backup so should i kept it as it is? | 11:24 |
*** alexchadin has joined #openstack-nova | 11:24 | |
*** alexchadin has quit IRC | 11:24 | |
*** germs has quit IRC | 11:25 | |
gibi | melwitt: regarding the nova onboarding session in Vancouver. I can only give a tentative yes as my company haven't approved my travel yet. | 11:32 |
*** alexchadin has joined #openstack-nova | 11:35 | |
*** vladikr has quit IRC | 11:37 | |
kashyap | Is it just me, fetching from: https://git.openstack.org/openstack/nova.git fails? | 11:39 |
kashyap | It's just me | 11:42 |
cdent | works for me | 11:42 |
kashyap | Fir me, it's just hung here: | 11:44 |
kashyap | Running: `GIT_CURL_VERBOSE=1 GIT_TRACE=1 git pull` | 11:44 |
kashyap | http://paste.openstack.org/show/704380/ | 11:44 |
*** alexchadin has quit IRC | 11:49 | |
*** logan- has quit IRC | 11:52 | |
*** logan- has joined #openstack-nova | 11:52 | |
*** aloga has joined #openstack-nova | 11:53 | |
johnthetubaguy | bhagyashris: the old API has to keep working like it does today | 11:54 |
*** aloga has quit IRC | 11:54 | |
johnthetubaguy | kashyap: you tried doing http rather than https? | 11:55 |
kashyap | johnthetubaguy: I tried all three, git, https, http. | 11:55 |
kashyap | None works for me, bizarrely | 11:55 |
johnthetubaguy | bummer | 11:55 |
bhagyashris | johnthetubaguy: ok . i just need to add the functionality of roatation parameter value 0 in cleanup backup api right? | 11:56 |
johnthetubaguy | bhagyashris: yes, that sounds correct. Will be easier to review the spec with all the details, to make sure I haven't missed a bit :) | 11:57 |
bhagyashris | johnthetubaguy: ok thank you for your inputs will upload spec ASAP :) | 11:59 |
johnthetubaguy | bhagyashris: no problem, sounds good | 11:59 |
*** beagles_biab is now known as beagesl | 12:00 | |
*** beagesl is now known as beagles | 12:00 | |
*** psachin has quit IRC | 12:00 | |
*** efried has quit IRC | 12:03 | |
*** tetsuro has left #openstack-nova | 12:06 | |
*** aloga has joined #openstack-nova | 12:07 | |
*** sidx64 has quit IRC | 12:08 | |
*** psachin has joined #openstack-nova | 12:11 | |
*** sidx64 has joined #openstack-nova | 12:12 | |
*** edmondsw has joined #openstack-nova | 12:15 | |
*** sidx64 has quit IRC | 12:22 | |
*** gyankum has joined #openstack-nova | 12:22 | |
*** suresh12 has joined #openstack-nova | 12:23 | |
openstackgerrit | Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212 | 12:24 |
*** sidx64 has joined #openstack-nova | 12:25 | |
*** suresh12 has quit IRC | 12:27 | |
*** READ10 has joined #openstack-nova | 12:27 | |
openstackgerrit | Chris Dent proposed openstack/nova-specs master: Provide error codes for placement API https://review.openstack.org/418393 | 12:28 |
*** cdent has quit IRC | 12:34 | |
*** liverpooler has joined #openstack-nova | 12:39 | |
*** ratailor has quit IRC | 12:40 | |
*** liverpooler has quit IRC | 12:40 | |
*** liverpooler has joined #openstack-nova | 12:40 | |
*** odyssey4me has quit IRC | 12:42 | |
*** odyssey4me has joined #openstack-nova | 12:42 | |
*** vladikr has joined #openstack-nova | 12:42 | |
*** mvk has quit IRC | 12:44 | |
*** pchavva has joined #openstack-nova | 12:46 | |
openstackgerrit | Theodoros Tsioutsias proposed openstack/nova-specs master: Enable rebuild for instances in cell0 https://review.openstack.org/554218 | 12:55 |
*** jpena is now known as jpena|lunch | 12:56 | |
*** moshele has joined #openstack-nova | 12:56 | |
*** danpawlik has quit IRC | 12:57 | |
*** cdent has joined #openstack-nova | 12:59 | |
*** mriedem has joined #openstack-nova | 13:06 | |
*** amoralej is now known as amoralej|lunch | 13:08 | |
*** yamamoto has quit IRC | 13:08 | |
*** sidx64 has quit IRC | 13:12 | |
*** efried has joined #openstack-nova | 13:13 | |
*** ioni has left #openstack-nova | 13:14 | |
*** sidx64 has joined #openstack-nova | 13:15 | |
*** sidx64 has quit IRC | 13:15 | |
*** lyan has joined #openstack-nova | 13:18 | |
*** lyan is now known as Guest68461 | 13:18 | |
*** sidx64 has joined #openstack-nova | 13:19 | |
*** germs has joined #openstack-nova | 13:21 | |
*** germs has quit IRC | 13:21 | |
*** germs has joined #openstack-nova | 13:21 | |
*** psachin has quit IRC | 13:24 | |
*** jichen has quit IRC | 13:25 | |
*** germs has quit IRC | 13:26 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423 | 13:29 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 13:29 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 13:29 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170 | 13:29 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: compute: use port binding extended API during live migration https://review.openstack.org/551371 | 13:29 |
*** sidx64 has quit IRC | 13:29 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 13:29 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Turn on new port binding extended live migrate flow https://review.openstack.org/552173 | 13:29 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 13:30 |
*** sidx64 has joined #openstack-nova | 13:30 | |
*** sidx64 has quit IRC | 13:34 | |
*** eharney has joined #openstack-nova | 13:42 | |
*** jroll has quit IRC | 13:42 | |
*** jroll has joined #openstack-nova | 13:43 | |
*** hamzy__ is now known as hamzy | 13:43 | |
kaisers1 | efried: Hi! Regarding https://review.openstack.org/#/c/554195/ : I looked into using privsep but as the spec says true root user context will continue to use rootwrap i went with that. If privsep does offer a root execution mechanism i can go that way. Do you have a hint on docs/examples of that for me? | 13:44 |
*** sidx64 has joined #openstack-nova | 13:44 | |
efried | kaisers1: mikal is the expert on that. I believe he's in Australia, so time zones may be a bit of a challenge. I'm sure I can at least dig up the spec for the privsep work. Stand by... | 13:45 |
*** yamamoto has joined #openstack-nova | 13:45 | |
efried | kaisers1: Meanwhile, if you want to see some examples for how mikal has transitioned existing code over to privsep: https://review.openstack.org/#/q/topic:bp/hurrah-for-privsep+(status:open+OR+status:merged) | 13:45 |
dansmith | kaisers1: you should be using privsep not rootwrap for new things | 13:47 |
dansmith | kaisers1: the whole point of privsep is to replace rootwrap, and doing so does indeed get you root escalation for your code | 13:47 |
*** psachin has joined #openstack-nova | 13:47 | |
*** esberglu has joined #openstack-nova | 13:47 | |
efried | dansmith: Is there a spec (I can't find one) or documentation for how to use privsep? | 13:47 |
dansmith | efried: there's lots of code now | 13:48 |
kaisers1 | I did read the spec | 13:48 |
kaisers1 | dansmith: thanks for the inputs! | 13:48 |
*** yamamoto_ has joined #openstack-nova | 13:48 | |
efried | kaisers1: For my edification, where did you find that spec? | 13:48 |
kaisers1 | efried: https://specs.openstack.org/openstack/oslo-specs/specs/liberty/privsep.html | 13:49 |
dansmith | privsep was born of lots of conversation and design, I'm sure there are things to read, but I expect a lot of the formal documentation focuses on capability contexts, which we've short-circuited to a single root one, | 13:49 |
efried | ah, in oslo. | 13:49 |
dansmith | so it may be more reading than you need | 13:49 |
efried | So yeah, kaisers1 you may do best following the code examples I linked above. | 13:49 |
kaisers1 | efried: will do, thanks! | 13:50 |
dansmith | definitely | 13:50 |
efried | enjoy | 13:50 |
*** yamamoto has quit IRC | 13:52 | |
*** mvk has joined #openstack-nova | 13:54 | |
jaypipes | ugh, the warnings barfed out in py36 test runs is getting ridiculous. | 13:54 |
edleafe | Scheduler subteam meeting in 5 minutes in #openstack-meeting-alt | 13:55 |
cdent | jaypipes: fun, isn't it? | 13:55 |
*** dtantsur is now known as dtantsur|brb | 13:56 | |
*** amoralej|lunch is now known as amoralej | 13:56 | |
tssurya | dansmith: I am really sorry about this accidental rebase : https://review.openstack.org/#/c/553831/; could you please rebase it to the right topic before someone +W's it ? | 13:57 |
jaypipes | cdent: makes it very annoying to run tests. | 13:57 |
cdent | jaypipes: I keep meaning to find some to fix and then forget for whatever reason | 13:58 |
dansmith | tssurya: it doesn't really matter | 13:58 |
tssurya | dansmith: ok, yea still sorry | 13:59 |
dansmith | tssurya: it's not a problem, gerrit-review does that | 13:59 |
dansmith | tssurya: in the future -R will prevent you from rebasing the thing underneath you if you intentionally don't want it to move | 13:59 |
tssurya | dansmith: yep will keep that in mind next time | 14:00 |
tssurya | thank you | 14:00 |
dansmith | np | 14:00 |
*** hongbin has joined #openstack-nova | 14:00 | |
*** beekneemech is now known as bnemec | 14:02 | |
*** awaugama has joined #openstack-nova | 14:03 | |
*** Maddy has joined #openstack-nova | 14:04 | |
Maddy | Hello, I was hitting this bug https://bugs.launchpad.net/nova/+bug/1717547 and I see that the workaround for that is not using the image parameters name as 'version', 'context' and 'method'. | 14:05 |
openstack | Launchpad bug 1717547 in OpenStack Compute (nova) "Creating snapshot fails when image metadata has version field" [Low,Confirmed] | 14:05 |
Maddy | This is not good because the user needs to be aware of that. One solution is make Glance to deny use of these parameters name on it's images. Other solution is to rename the variables in Nova 'Call' function to begin with '_' (underscore), but for that we also need to enforce Glance to now allow parameters beginning with '_' (underscore). | 14:05 |
Maddy | What's your thoughts about that? Any better solution? In case not, what do you think about implementing one the above solutions? | 14:05 |
*** sree has joined #openstack-nova | 14:06 | |
*** jpena|lunch is now known as jpena | 14:06 | |
*** psachin has quit IRC | 14:06 | |
*** afaranha has joined #openstack-nova | 14:10 | |
*** sree has quit IRC | 14:10 | |
*** diga has joined #openstack-nova | 14:11 | |
*** felipemonteiro_ has joined #openstack-nova | 14:12 | |
*** gouthamr has joined #openstack-nova | 14:13 | |
*** felipemonteiro__ has joined #openstack-nova | 14:14 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: tests for alloc candidates with nested and traits https://review.openstack.org/531899 | 14:14 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add test for scenario with sum of child resources https://review.openstack.org/534339 | 14:14 |
openstackgerrit | Jay Pipes proposed openstack/nova master: _get_trees_matching_all_resources() sum children https://review.openstack.org/534866 | 14:14 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add _get_usages_from_trees() function https://review.openstack.org/534967 | 14:14 |
openstackgerrit | Jay Pipes proposed openstack/nova master: validate provider summary structure https://review.openstack.org/536516 | 14:14 |
openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - populate alloc cands for trees https://review.openstack.org/534968 | 14:14 |
*** Zames has joined #openstack-nova | 14:15 | |
*** psachin has joined #openstack-nova | 14:17 | |
*** tbachman has quit IRC | 14:17 | |
*** felipemonteiro_ has quit IRC | 14:18 | |
*** Zames has quit IRC | 14:20 | |
*** namnh has joined #openstack-nova | 14:20 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: tests for alloc candidates with nested and traits https://review.openstack.org/531899 | 14:20 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add test for scenario with sum of child resources https://review.openstack.org/534339 | 14:20 |
openstackgerrit | Jay Pipes proposed openstack/nova master: _get_trees_matching_all_resources() sum children https://review.openstack.org/534866 | 14:20 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add _get_usages_from_trees() function https://review.openstack.org/534967 | 14:20 |
openstackgerrit | Jay Pipes proposed openstack/nova master: validate provider summary structure https://review.openstack.org/536516 | 14:20 |
openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - populate alloc cands for trees https://review.openstack.org/534968 | 14:20 |
*** amodi has joined #openstack-nova | 14:23 | |
openstackgerrit | Merged openstack/nova-specs master: Add host info to instance action events https://review.openstack.org/543277 | 14:23 |
*** psachin has quit IRC | 14:23 | |
jaypipes | efried: ^^ I'm getting that nested providers in allocation candidates series back inline... | 14:27 |
efried | jaypipes: Cool dude. | 14:27 |
efried | Did you see my potentially earth-shattering dissenting opinion on the "split one resource across multiple providers" thing? | 14:28 |
jaypipes | efried: no. link? | 14:28 |
efried | jaypipes: https://review.openstack.org/#/c/534339/ -- my last comment, on PS5 before rebases | 14:29 |
*** tssurya has quit IRC | 14:29 | |
*** yamamoto_ has quit IRC | 14:30 | |
*** amodi has quit IRC | 14:30 | |
*** cfriesen has joined #openstack-nova | 14:31 | |
*** jackie-truong has joined #openstack-nova | 14:32 | |
*** moshele has quit IRC | 14:32 | |
*** salv-orlando has quit IRC | 14:32 | |
*** salv-orlando has joined #openstack-nova | 14:33 | |
*** ccamacho has quit IRC | 14:33 | |
*** ttsiouts_ has joined #openstack-nova | 14:33 | |
*** felipemonteiro__ has quit IRC | 14:34 | |
*** gyankum has quit IRC | 14:36 | |
*** takedakn has joined #openstack-nova | 14:37 | |
*** sree has joined #openstack-nova | 14:38 | |
mriedem | dansmith: replied in https://review.openstack.org/#/c/543277/ | 14:38 |
mriedem | still don't see the need for hostId in the action events API, but people at the PTG wanted it | 14:38 |
dansmith | mriedem: I'm commenting about the rst syntax, using `` instead of " | 14:38 |
mriedem | oh, heh | 14:40 |
*** mlavalle has joined #openstack-nova | 14:40 | |
mriedem | i can push a doc cleanup patch on that | 14:40 |
kashyap | mriedem: melwitt: Heya, I've read the full chat between you both, dansmith, et al (including the quip: "besides the fix, you can also configure nova for other goodies!"). | 14:41 |
kashyap | Not to discuss this to death, but 3 quick points: | 14:41 |
dansmith | mriedem: that's all I was saying :) | 14:42 |
* kashyap copy / pastes the 3 points from editor: | 14:42 | |
kashyap | (1) I want to do the Right Thing & not rush anything in. | 14:42 |
kashyap | (2) The two Nova config options thing just causes untold pain for all vendors with all their unique deployment tools; and just wastes precious time all around for little gain. | 14:42 |
kashyap | (3) On your point: "well, what I mean is if you use it and it breaks something in queens/pike/ocata then we're probably not going to fix things to accommodate the various flags they enable" — That's a good point. Yes, we have to loudly and clearly document that "NB: There's a huge combination of CPU flags possible, just by virtue of them being available in a given CPU. It's not reasonable to | 14:42 |
kashyap | test all combinations; so if an Operator wants to try certain flags, it's their responsibility to test if that works in their environment". | 14:42 |
*** felipemonteiro_ has joined #openstack-nova | 14:42 | |
* kashyap crawls back to his cave to address the technical bits | 14:43 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | 14:44 |
kashyap | melwitt: mriedem: As a last point, as dansmith also said in different words on Friday here — | 14:44 |
*** tssurya has joined #openstack-nova | 14:44 | |
kashyap | It is not sensible to create an intentionally incompatible syntax in the Nova config file when the actual Nova code to process the config options is largely identical in both cases. | 14:44 |
openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 14:45 |
*** yamamoto has joined #openstack-nova | 14:46 | |
*** salv-orlando has quit IRC | 14:46 | |
*** sidx64 has quit IRC | 14:47 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 14:47 |
*** mlavalle has quit IRC | 14:47 | |
*** salv-orlando has joined #openstack-nova | 14:47 | |
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 | 14:47 |
*** mlavalle has joined #openstack-nova | 14:48 | |
*** yamahata has joined #openstack-nova | 14:50 | |
*** udesale has quit IRC | 14:50 | |
openstackgerrit | Lenny Verkhovsky proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | 14:51 |
*** salv-orlando has quit IRC | 14:51 | |
*** yamamoto has quit IRC | 14:51 | |
*** sree_ has joined #openstack-nova | 14:52 | |
*** sree_ is now known as Guest82448 | 14:52 | |
*** ccamacho has joined #openstack-nova | 14:53 | |
*** chyka has joined #openstack-nova | 14:55 | |
*** sree has quit IRC | 14:55 | |
*** itlinux has joined #openstack-nova | 15:00 | |
*** takedakn has quit IRC | 15:01 | |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Address review comments from afdc828db3c9d0205b6ded268db24f5cdf857fa6 https://review.openstack.org/554251 | 15:01 |
*** mdnadeem has quit IRC | 15:03 | |
*** yamamoto has joined #openstack-nova | 15:04 | |
openstackgerrit | Silvan Kaiser proposed openstack/nova master: Exec systemd-run with privileges in Quobyte driver https://review.openstack.org/554195 | 15:04 |
*** felipemonteiro_ has quit IRC | 15:04 | |
*** felipemonteiro_ has joined #openstack-nova | 15:05 | |
*** takedakn has joined #openstack-nova | 15:05 | |
*** elmaciej has quit IRC | 15:05 | |
kaisers1 | efried: fyi https://review.openstack.org/#/c/554195/2 | 15:06 |
*** cdent has quit IRC | 15:06 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Report client: Remove version discovery comment https://review.openstack.org/554253 | 15:06 |
efried | jaypipes, mriedem, cdent ^ | 15:06 |
efried | kaisers1: ack | 15:06 |
efried | edleafe: also --^ | 15:06 |
cfriesen | is github.com enforcing ssh versions now? I just got a "Peer reports incompatible or unsupported protocol version." | 15:06 |
efried | Could that be what kashyap was running into earlier? | 15:07 |
kashyap | efried: Probably; it's gone now. I'm able to fetch 'https' from git.o.o. | 15:08 |
*** liverpooler has quit IRC | 15:08 | |
kashyap | cfriesen: I also had troubles this morning w/ GitHub as my 'origin' for Nova | 15:08 |
cfriesen | looks like they're "weak crypto" versions of https | 15:08 |
cfriesen | bah, they're *disabling* | 15:09 |
kashyap | Yeah, apparently | 15:09 |
efried | Cause it's crucial that the content of a public repo be strongly encrypted. | 15:09 |
efried | Is that, what, so we don't get MITMs injecting viruses into the codebase for us? | 15:10 |
*** dtantsur|brb is now known as dtantsur | 15:10 | |
efried | tox -e py27 -- trojan_horse ? | 15:11 |
jaypipes | edleafe, cdent: do we have a bug in placement that diga might be able to pick up? he's looking for some work in placement/scheduler. | 15:11 |
cfriesen | is it possible to use SSH URLs with github? | 15:12 |
*** abalutoiu has joined #openstack-nova | 15:12 | |
cfriesen | I don't see any on the nova page | 15:12 |
jroll | cfriesen: yep, git@github.com:openstack/nova | 15:12 |
jroll | need to add your public key in settings | 15:13 |
edleafe | jaypipes: diga: there's always https://bugs.launchpad.net/nova/+bugs?field.tag=placement as a starting point | 15:13 |
cfriesen | jroll: thanks | 15:13 |
jroll | np | 15:13 |
diga | edleafe: Sure, will go through it | 15:13 |
*** amodi has joined #openstack-nova | 15:14 | |
*** ameeda has quit IRC | 15:14 | |
diga | jaypipes: Thank you for your help | 15:14 |
*** takedakn has quit IRC | 15:16 | |
jaypipes | diga: if I come across something for you, I will let you know. | 15:16 |
diga | jaypipes: Sure | 15:17 |
*** cdent has joined #openstack-nova | 15:20 | |
*** eharney has quit IRC | 15:21 | |
diga | jaypipes: I also know NFV technologies like hugepages, numa, dpdk very well. If you have something on that, let me know | 15:21 |
*** germs has joined #openstack-nova | 15:22 | |
*** germs has quit IRC | 15:22 | |
*** germs has joined #openstack-nova | 15:22 | |
jaypipes | diga: cool. can you review this spec please? https://review.openstack.org/#/c/552924/ | 15:23 |
diga | jaypipes: sure | 15:24 |
*** eharney has joined #openstack-nova | 15:26 | |
*** liverpooler has joined #openstack-nova | 15:26 | |
*** germs has quit IRC | 15:26 | |
Spaz-Home | Morning folks | 15:28 |
openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | 15:29 |
openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 15:29 |
openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 15:29 |
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 | 15:29 |
*** chyka has quit IRC | 15:30 | |
*** chyka has joined #openstack-nova | 15:30 | |
Spaz-Home | Actually taking a relax weekend.. no coding.. just boiling 55 Habaneros, 2 Ghost Peppers, and trying to make a hot sauce without killing myself. Enjoy your day Cloud Creators :p | 15:32 |
*** gjayavelu has joined #openstack-nova | 15:35 | |
*** gjayavelu has joined #openstack-nova | 15:35 | |
cdent | jaypipes: what's the status of your "standardized provider descriptor file" spec/idea? Where on the scale or need<->experimental fishing is that? | 15:38 |
jaypipes | cdent: it's dead. | 15:38 |
*** Guest82448 has quit IRC | 15:38 | |
jaypipes | cdent: I'll abandon it. | 15:38 |
cdent | dead fish | 15:38 |
jaypipes | cdent: smelly dead fish. | 15:38 |
* cdent kinda liked it | 15:38 | |
*** sree has joined #openstack-nova | 15:38 | |
jaypipes | cdent: I did too :) but I understand why dansmith doesn't. | 15:39 |
*** kholkina has quit IRC | 15:40 | |
*** elmaciej has joined #openstack-nova | 15:41 | |
mriedem | sahid: there are a couple of issues in the tests for https://review.openstack.org/#/c/497457/2 | 15:42 |
mriedem | https://review.openstack.org/#/c/497457/ | 15:42 |
mriedem | sahid: specifically self.assertEqual(0, guest.migrate_configure_max_speed.called) | 15:42 |
mriedem | .called is a boolean | 15:42 |
mriedem | it's probably better to use guest.migrate_configure_max_speed.assert_not_called() | 15:43 |
*** sree has quit IRC | 15:43 | |
mriedem | if you're about done for the day i could clean those up and then just approve | 15:44 |
sahid | oh i was sure of that "called" returns a number | 15:44 |
sahid | no no, i will | 15:44 |
*** artom has joined #openstack-nova | 15:44 | |
mriedem | call_count returns a number | 15:44 |
sahid | but thanks for asking :) | 15:45 |
sahid | ack | 15:45 |
*** diga has quit IRC | 15:45 | |
mriedem | thanks for finding that issue in the multinode job config | 15:45 |
*** jmlowe_ has quit IRC | 15:46 | |
sahid | mriedem: yeah, it's that g-api service which looked very strange | 15:46 |
*** felipemonteiro_ has quit IRC | 15:46 | |
*** gjayavelu has quit IRC | 15:46 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: validate provider summary structure https://review.openstack.org/536516 | 15:47 |
openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - populate alloc cands for trees https://review.openstack.org/534968 | 15:47 |
kashyap | sahid: One more nit there: s/TAPs devices/TAP devices/ :-) | 15:47 |
kashyap | (In driver.py) | 15:47 |
*** josecastroleon has joined #openstack-nova | 15:47 | |
cfriesen | has anyone run into problems with setuptools recently? I got a report that 39.0.1 didn't work with our version of pike, wondering if it's a general problem. | 15:48 |
sahid | kashyap: :) sure i will address that too | 15:49 |
kashyap | Thx | 15:49 |
dansmith | cfriesen: I had an issue downstream last week where the parse_version was returning a tuple instead of a string | 15:50 |
dansmith | cfriesen: migi was looking at it, in case you or he know anything else | 15:50 |
*** Maddy has quit IRC | 15:52 | |
*** ttsiouts_ has quit IRC | 15:53 | |
cfriesen | dansmith: we ended up pinning it to 38.5.1 for now, which seems to work okay | 15:53 |
*** r-daneel has quit IRC | 15:54 | |
*** markvoelker_ has joined #openstack-nova | 15:56 | |
*** markvoelker has quit IRC | 15:56 | |
*** tbachman has joined #openstack-nova | 15:57 | |
*** sree has joined #openstack-nova | 15:57 | |
*** gyankum has joined #openstack-nova | 15:59 | |
*** markvoelker has joined #openstack-nova | 15:59 | |
*** gyankum has quit IRC | 16:00 | |
mriedem | jaypipes: on that vmware resource pools spec, https://review.openstack.org/#/c/549067/6/specs/rocky/approved/vmware-resource-pool-placement.rst@210 | 16:00 |
*** markvoelker_ has quit IRC | 16:01 | |
mriedem | jaypipes: finally just realized that this is proposing that the admin changes things *in vcenter*, not placement directly, and that's all proxied through the virt driver | 16:01 |
mriedem | which kind of goes against something we've always said we woudn't do, which is support out of band changes on the hypervisor | 16:01 |
*** namnh has quit IRC | 16:02 | |
*** dtruong has joined #openstack-nova | 16:02 | |
cdent | mriedem: I think those sorts of changes are pretty much why upt exists? | 16:02 |
cdent | the hardware underlaying the hypervisor changes in some fashion | 16:02 |
jaypipes | mriedem: vcenter virt driver is virtually identical (pun intended) to how ironic virt driver wants to do things. i.e. the virt driver (not the compute service worker ala the resource tracker) creates the compute node resource provider records. | 16:02 |
cdent | upt notices | 16:02 |
cdent | efried: powervm is much the same, yeah? ^ | 16:03 |
mriedem | jaypipes: this isn't creating the compute node resource provider record, | 16:03 |
mriedem | it's seeing, oh the admin created some resource pools in this cluster in vcenter directly, and now the virt driver needs to mirror that into placement, | 16:03 |
mriedem | and what happens when the admin decides to delete those pools in vcenter? nova would also need to mirror that | 16:04 |
dansmith | cdent: powervm only has one node | 16:04 |
*** salv-orlando has joined #openstack-nova | 16:05 | |
mriedem | powervm wants to manage shared storage pool aggregates from what i remember | 16:05 |
dansmith | jaypipes: I read that spec as not exposing more nodes, but exposing a cluster as a very NUMA single node | 16:05 |
*** tbachman has quit IRC | 16:05 | |
*** afaranha has quit IRC | 16:05 | |
cdent | dansmith: not talking about the node here, just what the node is aware of: efried has mentioned that the powervm driver wants to ... yeah what mriedem said | 16:05 |
*** sree has quit IRC | 16:05 | |
dansmith | mriedem: right, but not multiple nodes from one service as it stands in tree right now | 16:05 |
mriedem | these would be nested RPs | 16:05 |
dansmith | yeah, I think jaypipes is saying multiple nodes a-la ironic | 16:06 |
*** tbachman has joined #openstack-nova | 16:06 | |
dansmith | and I don't think that's on the table for either | 16:06 |
mriedem | compute node RP (created by nova) -> nested tenant-specific resource pool providers, created by the virt driver, proxied from vcenter | 16:06 |
mriedem | the 1:M compute node thing isn't in this | 16:06 |
dansmith | yep | 16:06 |
dansmith | right | 16:06 |
cfriesen | mriedem: is there anything in particular blocking https://review.openstack.org/#/c/465787/ ? I notice it hasn't really gotten any reviews other than me. | 16:06 |
dansmith | I think jaypipes was asserting that it was | 16:06 |
*** pcaruana has quit IRC | 16:07 | |
jaypipes | dansmith: sorry, on a video call for 45 mintues | 16:07 |
mriedem | i guess since we don't model numa as a nested RP for the libvirt driver yet, i'm not sure how much proxy we'll be doing to compare to the vcenter thing | 16:07 |
dansmith | the difference here I think, | 16:07 |
dansmith | is that libvirt will report the static nature of the system as nested NUMA RPs, | 16:08 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Block swapping to an encrypted volume when using QEMU to decrypt https://review.openstack.org/544238 | 16:08 |
dansmith | where vmware would be representing cluster config, which is mutable | 16:08 |
dansmith | and I agree that sucks, and I don't know what would happen if vmware swizzled things after instances are booted | 16:08 |
dansmith | but.. | 16:08 |
dansmith | what we have today is also wrong (exposing a cluster as a single uniform set of resources) | 16:08 |
elod | cfriesen: it's general and there's this patch https://review.openstack.org/#/c/554053/ (which unfortunately ends up in another bug with pip... see depends-on) | 16:09 |
*** salv-orlando has quit IRC | 16:09 | |
mriedem | this spec also bakes business logic into the driver about how those resource pools are created in vcenter, i think | 16:10 |
cdent | dansmith: since I'm not aware of the many bits of history involved with this stuff (either internal or external to vmware) why is that "wrong" instead of "different"? | 16:10 |
mriedem | basically based on the name of the resource pool, it's encoded into the pool name which tells the virt driver what to do about it | 16:10 |
*** lpetrut_ has quit IRC | 16:10 | |
dansmith | cdent: because you can't schedule instances to it that span what nova thinks are uniform resources | 16:10 |
dansmith | cdent: at least, that's my understanding of it at the moment | 16:11 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: libvirt: Block swap volume attempts with encrypted volumes prior to Queens https://review.openstack.org/543569 | 16:11 |
openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | 16:11 |
dansmith | cdent: i.e. if you have a cluster with two 1G nodes, nova thinks you should be able to boot a 2G instance there but you can't | 16:11 |
cdent | that's recently fixed via inventory handling with max_unit | 16:12 |
dansmith | s/fixed/hacked | 16:12 |
cdent | no | 16:12 |
*** tbachman has quit IRC | 16:12 | |
cdent | you're asserting that targeting a cluster is wrong, and what I'm trying to understand why that's wrong | 16:12 |
dansmith | it is a hack, because if you have memory available on one and cpu available on another, you can't represent that | 16:12 |
dansmith | no, that's not what I'm saying | 16:12 |
dansmith | I'm saying nova thinks the cluster is a single bag of resources that can be combined and that's wrong | 16:13 |
dansmith | and I'm saying that I think the spec we're discussing is trying to rectify that by using multiple providers with the proper amounts of inventory each | 16:13 |
mriedem | yeah it's trying to break the single cluster into multiple pools | 16:14 |
cdent | the spec is trying to allow resource pools (a vmware thing that enables tenanted clusters) to work | 16:14 |
mriedem | but it's also tying those pools to specific tenants | 16:14 |
mriedem | like the aggregate multitenenancy isolation filter | 16:14 |
mriedem | i think | 16:14 |
dansmith | if it's purely for the tenantization, then that's not what I think it's trying to do, and isn't really a thing the virt driver should be doing | 16:14 |
cdent | there will likely be other specs that try to turn clusters into hierarchies of nested providers so that an esxi host works much the same as targeting a particular numa setup | 16:14 |
dansmith | ^that is what I thought the actual goal was under the covers | 16:15 |
mriedem | so if the admin creates a resource pool A_foo in cluster 1, i think it means that nova is supposed to restrict all requests from tenant foo to resource pool A_foo in cluster 1 | 16:15 |
dansmith | but if not, that's disappointing | 16:15 |
cdent | okay, stepping back a bit: why do we care what a virt driver does if it follows the interface and allows things to land | 16:15 |
cdent | I'm asking that as a real question, not as a rhetorical device | 16:15 |
dansmith | because nova is an abstraction? | 16:16 |
cdent | right, if the abstraction works and continues to work, what does it matter? | 16:16 |
dansmith | the virt driver has to do more than mechanically implement the python interface to be something that a user can depend on | 16:16 |
*** fragatina has joined #openstack-nova | 16:16 | |
*** s1061123 has quit IRC | 16:17 | |
dansmith | as an example, it doesn't work properly right now but it does implement the interface | 16:17 |
cdent | and it is working to get better | 16:17 |
cdent | using the tools that the interface provides | 16:17 |
dansmith | this tenant grouping doesn't make it better | 16:17 |
dansmith | it makes it more different in behavior than the others | 16:17 |
mriedem | maybe the tenant isolation part of this should be split out | 16:18 |
*** salv-orlando has joined #openstack-nova | 16:18 | |
mriedem | so it's just a generic way to model nested resource pools in a single cluster (which is represented by the compute node provider root in nova) | 16:18 |
mriedem | and then the compute node RP for vmware doesn't have cpu/ram/disk, the nested resource pool providers do | 16:18 |
*** gjayavelu has joined #openstack-nova | 16:18 | |
*** rmart04 has quit IRC | 16:19 | |
mriedem | and then later bake tenant isolation logic into it | 16:19 |
mriedem | sorry for the bad timing but i've got a thing i've got to run to, back in awhile | 16:19 |
*** mriedem is now known as mriedem_away | 16:19 | |
dansmith | at that point, the tenant isolation becomes much the same problem as efried pointed out with ensuring that you get CPU and memory from the same numa node on a regular box, | 16:19 |
dansmith | which probably helps, if we decide that the virt drivers should draw aggregates around those groupings or whatever, | 16:19 |
dansmith | or something | 16:20 |
cdent | I suspect the tenant isolation is on the spec at all so that the resource pool stuff can be justified: it's a use case for wanting to represent resource pools | 16:20 |
cdent | if that's needed and reasource pools are groovy, that's great | 16:20 |
*** ragiman has quit IRC | 16:20 | |
efried | As I commented on https://review.openstack.org/#/c/534339/ the 1GB-from-each-of-two-children-to-make-one-2GB-instance thing shouldn't be a problem. | 16:20 |
efried | BUT the "get my CPU from one RP and my disk from another" would be. | 16:20 |
dansmith | efried: currently the former is a problem with the vmware driver AFAIK | 16:21 |
dansmith | cdent: implementing the structurally separated resources in a cluster as nested providers of the root would make progress towards fixing that ^ and then makes the tenant isolation thing a smaller delta we can probably have a more reasonable discussion over | 16:22 |
efried | To mitigate in UPT-land, you would have to do subtree lassoing like we talked about needing for NUMA; or you would have to model the cluster members as NOT being in the same tree. | 16:22 |
cdent | dansmith: that's not what the spec does | 16:22 |
cdent | resource pools are not physical | 16:22 |
dansmith | efried: that's what I'm saying would be an improvement over what it does today | 16:22 |
efried | which one dansmith? | 16:22 |
cdent | if you have a problem with the cluster presented agglomerated resources as a single thing, then you'd have a problem with resource pools too | 16:22 |
dansmith | efried: representing cluster members as children under the root | 16:23 |
*** yamamoto has quit IRC | 16:23 | |
cdent | max_unit, on either the cluster or the resource pool "fixes" that | 16:23 |
efried | dansmith: Only if we have the subtree-lassoing technology. I haven't caught up on my specs yet - did someone propose that yet? | 16:23 |
dansmith | cdent: max_unit does not prevent nova from thinking it can get cpu and memory from two different cluster members | 16:23 |
efried | Agree ^^ | 16:23 |
cdent | dansmith: yes, and? | 16:24 |
dansmith | efried: right, I'm assuming the lassoing thing as well as changing this vmware RP exposure.. the two together would be required | 16:24 |
dansmith | cdent: so max_unit has not "fixed" the vmware driver reporting a whole cluster as a single RP/node | 16:24 |
efried | In that case, yes, I agree we can do this with nested. | 16:24 |
cdent | the only way we get what you seem to want is for every esxi host to represent all its resources, which breaks the DRS, unless the virt driver can write allocations | 16:25 |
dansmith | cdent: yeah, I think we've asserted that DRS under nova is broken, for that reason exactly | 16:25 |
dansmith | broken fundamentally I mean | 16:25 |
efried | cdent: I thought we talked about the fact that you want to hide the whole cluster-ness anyway? | 16:25 |
*** jackie-truong has quit IRC | 16:26 | |
efried | Represent the whole thing as a single RP, and then vmware virt would do the individual node business under the covers. | 16:26 |
cdent | efried: yes, dan's saying that's broken | 16:26 |
dansmith | efried: that's what it does today, and it's breakable | 16:26 |
dansmith | efried: because nova will not know that one node is out of CPU but has some memory available, where another node has the opposite.. nova will think it can schedule to that, but it can't | 16:26 |
gjayavelu | @cdent @dansmith @efried I'm catching up on the discussions about the resource pools spec | 16:27 |
*** andreas_s has quit IRC | 16:27 | |
*** r-daneel has joined #openstack-nova | 16:27 | |
efried | dansmith: Okay, I agree with that. But I think the virt can probably mitigate | 16:27 |
dansmith | efried: it can't | 16:27 |
efried | ...by cleverly spoofing the inventories. | 16:27 |
dansmith | oh, it could work around it that way sure | 16:28 |
*** andreas_s has joined #openstack-nova | 16:28 | |
dansmith | that's fairly wasteful though :) | 16:28 |
efried | Basically by representing the least common denominator of the pool, as the inventory. | 16:28 |
efried | Is it? | 16:28 |
*** tbachman has joined #openstack-nova | 16:28 | |
dansmith | sure | 16:28 |
efried | Because you wouldn't be able to schedule more than that anyway, wouldja? | 16:28 |
dansmith | well, maybe you're right | 16:28 |
cdent | yeah, you either fail to get allocation candidates, or you fail later | 16:29 |
dansmith | well, no, | 16:29 |
cdent | better to fail to get allocation candidates | 16:29 |
*** lajoskatona has quit IRC | 16:29 | |
dansmith | you could have a node with small memory, small cpu, lots of disk, and another node with zero disk but lots of other resources.. you have to report yourself as fully committed at that point right? | 16:29 |
efried | Okay, I see, if you had one node with 1VCPU/1024MEM and one with 2VCPU/512MEM, you would wind up representing 1/512 even though you could technically schedule 1/1024 or 2/512. | 16:29 |
dansmith | cdent: also that | 16:29 |
efried | dansmith: Yeah, I think we're saying the same thing. | 16:30 |
dansmith | yar | 16:30 |
*** s1061123 has joined #openstack-nova | 16:30 | |
cdent | at the moment that is an accepted shortcoming | 16:30 |
cdent | mostly because the common case is for everything in the cluster to be the same | 16:30 |
efried | Yeah, was gonna say, homogeneity would make most of the issue moot. | 16:30 |
dansmith | right but your flavors have to basically fit together perfectly like puzzle pieces to avoid too much fragmentation | 16:31 |
cdent | ENOPARSE | 16:31 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | 16:31 |
efried | I think it's a pretty solid 80/20 that's worth going forward with. | 16:31 |
efried | Hypothetically the DRS will also reshuffle things to maintain/restore homogeneity, nah? That's kind of its job? | 16:32 |
cdent | yes | 16:32 |
cdent | well | 16:32 |
cdent | it will shuffle things. but it can't move hardware | 16:33 |
efried | understood. | 16:33 |
dansmith | that doesn't fix anything if your flavors are different sizes | 16:33 |
efried | It can | 16:33 |
efried | because it can consolidate VMs in ways that maximize utilization of a node. | 16:33 |
efried | If it's clever enough. | 16:33 |
dansmith | it may be able to if things fit right, yes | 16:34 |
cdent | even if its not clever enough, if vmware is being wasteful, that's vmware's problem, right? | 16:34 |
efried | But anyway, IMO it's acceptable to state that we know this solution is not 100% perfect, and move forward with it, because it will work *mostly* well, *most* of the time. | 16:34 |
openstackgerrit | Merged openstack/nova master: Revert "Refine waiting for vif plug events during _hard_reboot" https://review.openstack.org/553035 | 16:34 |
*** jpena is now known as jpena|brb | 16:43 | |
*** andreas_s has quit IRC | 16:45 | |
melwitt | cool, thanks gibi | 16:46 |
melwitt | gibi, bauzas: I'd like to bring to your attention the draft for runways during the rocky cycle, if you have any feedback about it https://etherpad.openstack.org/p/nova-runways-rocky I'd like to kick of the process later this week so we can try it out and adjust it as we go | 16:48 |
*** gyee has joined #openstack-nova | 16:49 | |
*** dikonoor has joined #openstack-nova | 16:51 | |
*** AlexeyAbashkin has quit IRC | 16:51 | |
gibi | melwitt: I opened that etherpad and I will try to check it tomorrow | 16:53 |
*** hemna_ has joined #openstack-nova | 16:53 | |
*** fragatina has quit IRC | 16:53 | |
gibi | melwitt: seem like a pretty comprehensive doc | 16:53 |
melwitt | gibi: cool, thanks. yeah, there's been a lot of feedback already last week, so np if there's nothing else you'd like to add or ask | 16:54 |
gibi | melwitt: I will try to read it anyhow :) | 16:55 |
melwitt | thanks. I'll ping bauzas again tomorrow as it looks like he's on PTO today | 16:55 |
*** cdent has quit IRC | 16:55 | |
*** sahid has quit IRC | 16:56 | |
openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | 16:57 |
*** fragatina has joined #openstack-nova | 16:58 | |
gibi | mriedem_away, alex_xu_ , mlavalle: thanks for the awesome review feedback on the bandwidth spec. I tried to answer the questions inline https://review.openstack.org/#/c/502306 | 16:58 |
mlavalle | gibi: will take a look again soon. Thanks! | 16:59 |
*** jackie-truong has joined #openstack-nova | 16:59 | |
gibi | mlavalle: thanks | 17:00 |
*** gjayavelu has quit IRC | 17:01 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Remove version/date from CLI documentation https://review.openstack.org/553903 | 17:03 |
*** weshay is now known as weshay_brb | 17:05 | |
*** yamahata has quit IRC | 17:05 | |
*** dtruong has quit IRC | 17:05 | |
*** elmaciej has quit IRC | 17:09 | |
jaypipes | sorry y'all. off phone call now. reading back.. | 17:10 |
*** diranerpi has quit IRC | 17:10 | |
*** lpetrut has joined #openstack-nova | 17:12 | |
tssurya | dansmith: I had a question about the reset cache implementation on the scheduler manager as a part of the cell disable spec, is it a good time now ? If you are busy I can come back later | 17:13 |
dansmith | tssurya: sure | 17:14 |
tssurya | So I was implementing the reset cache option on the scheduler manager , however when using the scheduler client from nova manage to call this reset on the cache in the manager, doesn’t this become an upcall which we don’t support ? | 17:14 |
tssurya | or am I not supposed to call this using the client from nova-manage ? | 17:15 |
dansmith | tssurya: I was saying this should be done in a reset() handler on scheduler manager, which gets triggered on SIGHUP | 17:15 |
dansmith | tssurya: correct, not from nova-manage or anything else, I thought I commented to that effect on the spec | 17:15 |
tssurya | yes you did, I guess I didn't understand the SIGHUP very well | 17:15 |
dansmith | tssurya: this is the example of a similar thing we have: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L543-L546 | 17:15 |
dansmith | tssurya: so you can SIGHUP compute manager now to get it to clear the service version cache and rpc pin, which is the same sort of activity you're doing | 17:16 |
tssurya | right, | 17:17 |
tssurya | I need to "SIGHUP" the scheduler manager basically | 17:17 |
dansmith | tssurya: yeah, we have a couple other signal handlers for things like guru meditation, etc | 17:20 |
*** cdent has joined #openstack-nova | 17:29 | |
*** esberglu_ has joined #openstack-nova | 17:32 | |
*** eharney has quit IRC | 17:33 | |
lennyb | Hi, my nova instance got stuck in 'deleting' when I try to delete instance that failed to be deployed via ironic n-cell-child.service.log | 17:33 |
lennyb | http://paste.openstack.org/show/704774/ | 17:33 |
*** gjayavelu has joined #openstack-nova | 17:34 | |
*** elmaciej has joined #openstack-nova | 17:35 | |
*** esberglu has quit IRC | 17:35 | |
*** jpena|brb is now known as jpena | 17:37 | |
*** mriedem_away is now known as mriedem | 17:37 | |
*** jmlowe has joined #openstack-nova | 17:38 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Mention (no) granular support for image traits https://review.openstack.org/554305 | 17:41 |
*** suresh12 has joined #openstack-nova | 17:41 | |
*** jackie-truong has quit IRC | 17:41 | |
*** oomichi has joined #openstack-nova | 17:42 | |
*** jackie-truong has joined #openstack-nova | 17:42 | |
efried | jaypipes, mriedem: ^ | 17:42 |
*** felipemonteiro_ has joined #openstack-nova | 17:43 | |
*** mvk has quit IRC | 17:44 | |
*** dikonoor has quit IRC | 17:45 | |
oomichi | melwitt: cdent: hi, do we have an approval at the PTG for https://blueprints.launchpad.net/nova/+spec/placement-extract ? | 17:46 |
jaypipes | dansmith, mriedem: would you mind looking at https://review.openstack.org/#/c/534339/ please and letting me know if you agree with efried? I'm thinking I agree with him and if so, will abandon that and the following patch to it. | 17:46 |
cdent | oomichi: the decision at ptg was to make progress in rocky, but not complete it, and not as a priority | 17:46 |
*** eharney has joined #openstack-nova | 17:47 | |
oomichi | cdent: I see, thanks. I just wanted to see the bp as approved | 17:47 |
cdent | oomichi: I didn't actually create the bp until late last week, because efried suggested it would be useful. We hadn't really declared "let's have one" but it does seem like a good idea. | 17:48 |
*** jamesdenton has joined #openstack-nova | 17:49 | |
*** suresh12 has quit IRC | 17:49 | |
melwitt | oomichi, cdent: yeah, seems useful to have it to track the work. I'll approve it | 17:50 |
oomichi | cdent: yeah, bp is good for tracking patches | 17:50 |
oomichi | melwitt: cool, thanks :) | 17:50 |
*** weshay_brb is now known as weshay | 17:50 | |
*** suresh12 has joined #openstack-nova | 17:51 | |
*** suresh12 has quit IRC | 17:52 | |
*** suresh12 has joined #openstack-nova | 17:52 | |
*** suresh12_ has joined #openstack-nova | 17:53 | |
*** suresh12 has quit IRC | 17:53 | |
efried | cdent, jaypipes: (how) does forbidden traits play into glance image traits? | 17:55 |
cdent | efried: i would guess the same as flavors? I haven't put that much thought into that side of things, just the api side | 17:57 |
jaypipes | efried: I would expect that the format of specifying a forbidden trait would be the same. I would also expect that if there was a conflict between flavor's forbidden traits and image required traits (and vice versa), that we would bomb out. | 17:57 |
efried | cdent: Oh, right (rereading the spec now), we haven't even tried to talk about how forbidden traits will be expressed in flavors, have we? | 17:57 |
efried | Is that simply not planned for R? | 17:58 |
* cdent doesn't know | 17:58 | |
*** germs has joined #openstack-nova | 17:58 | |
*** germs has quit IRC | 17:58 | |
*** germs has joined #openstack-nova | 17:58 | |
jaypipes | efried: I would have thought that s//required/forbidden in the extra spec value field would be how we'd do it. | 17:59 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | 17:59 |
*** mvk has joined #openstack-nova | 17:59 | |
*** lpetrut has quit IRC | 17:59 | |
*** jackie-truong has quit IRC | 18:00 | |
*** moshele has joined #openstack-nova | 18:02 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Fix typos in release notes https://review.openstack.org/554175 | 18:02 |
*** germs has quit IRC | 18:03 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Revert "Refine waiting for vif plug events during _hard_reboot" https://review.openstack.org/553817 | 18:06 |
*** lpetrut has joined #openstack-nova | 18:06 | |
*** elmaciej has quit IRC | 18:06 | |
*** avolkov has quit IRC | 18:07 | |
*** tesseract has quit IRC | 18:07 | |
melwitt | dansmith, mriedem: looks like we have an interesting bug here during an upgrade ocata -> pike where while reading a service record, if it doesn't have a uuid, we generate one and try to save it back to the object, but that fails with "Can't upgrade a READER transaction to a WRITE" because a read will start off with @pick_context_manager_reader and we can't upgrade to a writer in the middle of the transaction https://bugs.launchpad.net/n | 18:09 |
melwitt | ova/+bug/1746509/comments/4 | 18:09 |
melwitt | dangit | 18:09 |
melwitt | https://bugs.launchpad.net/nova/+bug/1746509/comments/4 | 18:09 |
openstack | Launchpad bug 1722404 in OpenStack Compute (nova) ocata "duplicate for #1746509 Database transactions can fail with "TypeError: Can't upgrade a READER transaction to a WRITER mid-transaction" because of scatter_gather_cells" [Undecided,In progress] - Assigned to Matt Riedemann (mriedem) | 18:09 |
*** dtantsur is now known as dtantsur|afk | 18:09 | |
dansmith | hmm, I'm not sure why it'd be different for scatter/gather | 18:10 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Revert "Refine waiting for vif plug events during _hard_reboot" https://review.openstack.org/553817 | 18:10 |
*** tbachman has quit IRC | 18:11 | |
melwitt | I don't think it's different. I think this is a separate different bug, but was highlighting the comment where someone explained what was going on | 18:11 |
*** moshele has quit IRC | 18:11 | |
dansmith | yeah, that explanation makes sense but I'm not sure why we weren't hitting that immediately is what I mean | 18:11 |
melwitt | I think we fixed the scatter gather problem but this service record thing is separate and different | 18:11 |
melwitt | oh | 18:12 |
*** eharney has quit IRC | 18:12 | |
*** sambetts is now known as sambetts|afk | 18:12 | |
melwitt | I assume we were ... trying to think if this scenario is covered by grenade or how it wouldn't be | 18:13 |
dansmith | well, it should have manifested even in functional tests I would think | 18:13 |
dansmith | we must be in a nested call where we're already in a reader transaction and we call to do a write, right? | 18:14 |
melwitt | yeah, that would be the condition for that error being raised | 18:14 |
dansmith | blanking out the context dumps that so we get a new transaction for the write | 18:14 |
*** esberglu_ is now known as esberglu | 18:16 | |
*** sridharg has quit IRC | 18:16 | |
melwitt | you mean the way that the commenter patched nova/objects/service.py? yeah | 18:18 |
dansmith | yeah they used a new context | 18:18 |
melwitt | yeah | 18:19 |
*** AlexeyAbashkin has joined #openstack-nova | 18:20 | |
*** AlexeyAbashkin has quit IRC | 18:24 | |
*** eharney has joined #openstack-nova | 18:26 | |
*** dave-mccowan has joined #openstack-nova | 18:28 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Revert "Refine waiting for vif plug events during _hard_reboot" https://review.openstack.org/553818 | 18:31 |
*** wolverineav has joined #openstack-nova | 18:32 | |
*** moshele has joined #openstack-nova | 18:32 | |
*** felipemonteiro__ has joined #openstack-nova | 18:32 | |
*** wolverineav has quit IRC | 18:33 | |
*** wolverineav has joined #openstack-nova | 18:33 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Use Queens UCA for nova-multiattach job https://review.openstack.org/554317 | 18:35 |
*** vladikr has quit IRC | 18:36 | |
*** felipemonteiro_ has quit IRC | 18:37 | |
mriedem | dansmith: http://lists.openstack.org/pipermail/openstack-dev/2018-March/128487.html for the setuptools thing you were talking about before | 18:37 |
mriedem | pkg_version | 18:37 |
dansmith | ah cool | 18:37 |
dansmith | migi: ^ | 18:37 |
*** tssurya has quit IRC | 18:38 | |
dansmith | mriedem: did you see my comment on that patch for the obj_relationships thing? | 18:38 |
mriedem | not yet | 18:39 |
*** vladikr has joined #openstack-nova | 18:40 | |
melwitt | dansmith: from what I can tell, we only have unit test coverage for the uuid generation for service where things are mocked (and would anyway be a write to begin with), or calling _from_db_object selectively https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_service.py#L145 | 18:44 |
melwitt | to hit this, we'd have to have a functional test scenario where we read an already existing service record that has no uuid | 18:44 |
melwitt | I can try writing one | 18:45 |
*** yamahata has joined #openstack-nova | 18:46 | |
dansmith | hmm, I thought we did | 18:47 |
dansmith | because that was mriedem | 18:47 |
dansmith | and he looooves functional tests | 18:47 |
dansmith | I would also have expected to hear more noise about this | 18:47 |
dansmith | like, wouldn't the first compute startup after that patch merged cause nova-compute to not update its service record? | 18:48 |
*** openstackgerrit has quit IRC | 18:48 | |
mriedem | it was dtp with me helping, but it's a pretty established pattern, | 18:49 |
mriedem | i.e. he copied it from compute_nodes.uuid, and aggregates.uuid | 18:49 |
dansmith | yeah | 18:49 |
dansmith | was just lookin gup the commit | 18:50 |
* melwitt looks around | 18:50 | |
mriedem | also https://github.com/dpeschman | 18:50 |
mriedem | https://avatars3.githubusercontent.com/u/145098?s=400&v=4 | 18:50 |
mriedem | i would like to know what's going on there | 18:50 |
melwitt | lol | 18:50 |
dansmith | even still, | 18:51 |
dansmith | we're out of the db context at that point, so I'm not sure why this is happening anyway | 18:52 |
dansmith | out of the pick_context_manager I mean | 18:52 |
*** tbachman has joined #openstack-nova | 18:52 | |
*** krtaylor has joined #openstack-nova | 18:59 | |
*** suresh12_ has quit IRC | 18:59 | |
*** suresh12 has joined #openstack-nova | 19:00 | |
*** suresh12 has quit IRC | 19:04 | |
*** jmlowe has quit IRC | 19:05 | |
*** READ10 has quit IRC | 19:05 | |
*** READ10 has joined #openstack-nova | 19:06 | |
*** jmlowe has joined #openstack-nova | 19:07 | |
*** openstackgerrit has joined #openstack-nova | 19:11 | |
openstackgerrit | Merged openstack/nova-specs master: Update placement aggregates spec to clarify generation handling https://review.openstack.org/548237 | 19:11 |
*** suresh12 has joined #openstack-nova | 19:13 | |
*** jpena is now known as jpena|off | 19:15 | |
*** suresh12 has quit IRC | 19:17 | |
*** AlexeyAbashkin has joined #openstack-nova | 19:20 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method https://review.openstack.org/544728 | 19:23 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add aggregates list to Destination object https://review.openstack.org/544729 | 19:23 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add request filter functionality to scheduler https://review.openstack.org/544730 | 19:23 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make get_allocation_candidates() honor aggregate restrictions https://review.openstack.org/547990 | 19:23 |
openstackgerrit | Dan Smith proposed openstack/nova master: [WIP] Add require_tenant_aggregate request filter https://review.openstack.org/545002 | 19:23 |
openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282 | 19:23 |
*** felipemonteiro_ has joined #openstack-nova | 19:24 | |
*** fragatina has quit IRC | 19:24 | |
*** AlexeyAbashkin has quit IRC | 19:24 | |
*** felipemonteiro_ has quit IRC | 19:25 | |
*** felipemonteiro_ has joined #openstack-nova | 19:25 | |
*** jmlowe has quit IRC | 19:26 | |
*** felipemonteiro__ has quit IRC | 19:27 | |
*** jmlowe has joined #openstack-nova | 19:27 | |
*** suresh12 has joined #openstack-nova | 19:29 | |
*** moshele has quit IRC | 19:29 | |
*** suresh12 has quit IRC | 19:30 | |
*** suresh12 has joined #openstack-nova | 19:30 | |
mriedem | debugging weirdness during live migratoin would be a lot easier if we had thread ids in the logs... | 19:35 |
*** liverpooler has quit IRC | 19:36 | |
*** vladikr has quit IRC | 19:36 | |
*** vladikr has joined #openstack-nova | 19:39 | |
mriedem | sean-k-mooney: do you know if unbinding a port makes it automatically go to status=DOWN? | 19:40 |
*** ociuhandu has joined #openstack-nova | 19:41 | |
*** _pewp_ has quit IRC | 19:41 | |
*** tssurya has joined #openstack-nova | 19:42 | |
mriedem | sean-k-mooney: https://review.openstack.org/#/c/434870/11/nova/compute/manager.py@1126 | 19:44 |
mriedem | seeing issues during live migration where we handle the migration completed lifecycle event from the hypervisor and start unbinding the port while we're also going through _post_live_migration | 19:44 |
mriedem | we blow up trying to unplug an unbound VIF | 19:45 |
sean-k-mooney[m] | Im not sure. The status of an unbound port does not really have a meaning as it is effectively just a db entry when unbound. E. G. It dose not exist on the datapath when unbound. | 19:45 |
mriedem | i ask because of https://github.com/openstack/neutron/blob/f2dced777acb7d87dc7048c3efec59dadf37930d/neutron/notifiers/nova.py#L186 | 19:45 |
*** _pewp_ has joined #openstack-nova | 19:45 | |
mriedem | i'm seeing mysterious network-vif-plugged and network-vif-unplugged events around the time of the failure | 19:45 |
mriedem | but they are coming before we unplug vifs on the source host | 19:46 |
mriedem | might be getting network-vif-plugged events on the source host from when they were plugged on the dest host in pre_live_migration, | 19:47 |
mriedem | the events are going to be routed to where the instance currrently lives, which is the source host until we switch | 19:48 |
*** openstackgerrit has quit IRC | 19:48 | |
*** felipemonteiro__ has joined #openstack-nova | 19:48 | |
mriedem | then we live migrate the guest, | 19:48 |
mriedem | get the 'migration completed' lifecycle event from the hypervisor and start unbinding the port, | 19:49 |
dansmith | mriedem: once there is a migration record, they'll be routed to both hosts | 19:49 |
sean-k-mooney[m] | Well in the old workflow we don't unbind the port. We just update the host in the binding details. Yes i think the vif plugged events are from when the destination vif is plugged | 19:49 |
mriedem | ok, i was only checking the source host logs, can check the dest host logs | 19:49 |
*** bswrchrd has joined #openstack-nova | 19:50 | |
mriedem | sean-k-mooney[m]: this is the patch causing problems right now https://review.openstack.org/#/c/434870/ | 19:50 |
mriedem | the thing that tried to do the early switch once were in post-copy | 19:50 |
mriedem | *once we were in | 19:50 |
jroll | jaypipes: dansmith: this is roughly what I recall from PTG about ironic RT stuff, does it make sense to you? https://review.openstack.org/#/c/545479/ | 19:50 |
*** felipemonteiro_ has quit IRC | 19:52 | |
mriedem | dansmith: hmm, not seeing the same event routed to both hosts | 19:53 |
mriedem | but we should have a migration record at this point because i see on the dest host logs that we've plugged the vifs there in pre_live_migration | 19:53 |
dansmith | mriedem: hang on I'm piecing this back together in my head just a sec | 19:55 |
dansmith | jroll: ack, will look ater | 19:55 |
dansmith | mriedem: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4547-L4559 | 19:56 |
jroll | thanks | 19:56 |
*** slagle has quit IRC | 19:56 | |
mriedem | fwiw, i don't see https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4556 in the logs | 19:56 |
dansmith | mriedem: it will require migration_context to be set | 19:56 |
mriedem | yeah that doesn't show up in the n-api logs http://logs.openstack.org/71/551371/6/check/legacy-tempest-dsvm-multinode-live-migration/4d466b2/logs/screen-n-api.txt.gz | 19:56 |
mriedem | do we not lazy-load migration_context? | 19:56 |
dansmith | okay, so maybe we haven't set migration context yet? | 19:56 |
dansmith | mriedem: we query for it specifically in the api side: | 19:56 |
dansmith | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/server_external_events.py#L59-L61 | 19:56 |
mriedem | yup | 19:56 |
dansmith | more likely we haven't .save()d it yet or something | 19:57 |
mriedem | i only see apply_migration_context called in evacuate and resize flows, not live migration, | 19:58 |
mriedem | is it set in the RT? | 19:58 |
dansmith | not that I know of, so maybe we never set it for live migration? | 19:58 |
*** slagle has joined #openstack-nova | 19:58 | |
sean-k-mooney[m] | Right so it kindof makes sense when you activate the binding on the destination it deactivates the binding for the source which i guess results in vif unbound for the old host. How are we getting the vif on the source node that we pass to _nova_to_osvif_vif_unbound we should probably be cache the original vif object in the migration object | 19:59 |
mriedem | right we don't do a move_claim for live migration, | 19:59 |
mriedem | and RT.move_claim is what sets migration_context on the instance | 19:59 |
*** felipemonteiro__ has quit IRC | 19:59 | |
*** germs has joined #openstack-nova | 20:00 | |
mriedem | sean-k-mooney[m]: heh way ahead of you https://review.openstack.org/#/c/515423/10/nova/objects/migrate_data.py@47 :) | 20:00 |
*** felipemonteiro_ has joined #openstack-nova | 20:00 | |
*** felipemonteiro_ has quit IRC | 20:00 | |
*** felipemonteiro_ has joined #openstack-nova | 20:01 | |
sean-k-mooney[m] | :) | 20:01 |
mriedem | artom: fyi re the migration_context thing, since you wrote https://review.openstack.org/#/c/371048/ | 20:02 |
mriedem | but i think ^ was just for evac cases, | 20:02 |
mriedem | where the source is dead so we can't rpc there | 20:02 |
dansmith | blast from ye olde past | 20:02 |
mriedem | reminds me i need to touch up some dings in the woodwork with olde english | 20:03 |
*** germs has quit IRC | 20:04 | |
*** sean-k-mooney2 has joined #openstack-nova | 20:04 | |
*** felipemonteiro__ has joined #openstack-nova | 20:06 | |
*** jackie-truong has joined #openstack-nova | 20:06 | |
*** felipemonteiro_ has quit IRC | 20:09 | |
*** arvindn05 has joined #openstack-nova | 20:11 | |
SamYaple | other than a --hard reboot, is there a way to regenerate the libvirt.xml config? we changed the vnc listening address but we cannot migrate/live-migrate some instances due to pci passthrough of a sata controller | 20:12 |
*** awaugama has quit IRC | 20:13 | |
sean-k-mooney2 | SamYaple: there is no way via nova to regenerate the xml that wont be equivalent to a reboot if you cant live migrate | 20:14 |
sean-k-mooney2 | SamYaple: you could likely just do a domain xml update out of band. | 20:14 |
*** ociuhandu has quit IRC | 20:14 | |
SamYaple | yea unfortunately not, but not for technical reasons | 20:15 |
SamYaple | sean-k-mooney2: awesome. thought not. thanks for confirming | 20:15 |
sean-k-mooney2 | SamYaple: if you update the vnc adress in the nova.conf you will also need to restart nova compute as i dont think that is one of the mutable config options either. | 20:16 |
SamYaple | oh no worries. its all rolled out, just in the "applying" phase | 20:17 |
SamYaple | we already live-mgirated 90% of the instances | 20:17 |
*** sree has joined #openstack-nova | 20:21 | |
*** oomichi has quit IRC | 20:21 | |
*** AlexeyAbashkin has joined #openstack-nova | 20:21 | |
*** Sukhdev_ has joined #openstack-nova | 20:25 | |
*** sree has quit IRC | 20:25 | |
*** AlexeyAbashkin has quit IRC | 20:26 | |
mriedem | sean-k-mooney[m]: yeah so i think changing the port binding changes the status, | 20:26 |
mriedem | this is the source host neutron agent 2 seconds after nova updated the port binding to point at the dest host | 20:27 |
mriedem | http://logs.openstack.org/71/551371/6/check/legacy-tempest-dsvm-multinode-live-migration/4d466b2/logs/screen-q-agt.txt.gz#_Mar_19_14_25_08_374523 | 20:27 |
mriedem | binding host and status changes on the port in there | 20:27 |
*** eharney has quit IRC | 20:28 | |
mriedem | which according to this: https://github.com/openstack/neutron/blob/f2dced777acb7d87dc7048c3efec59dadf37930d/neutron/notifiers/nova.py#L186 will trigger a network-vif-unplugged event | 20:28 |
sean-k-mooney2 | right so looking at that ports binding_details its an ovs vif with ovs_hybrid_plug": true so os-vif will plug the vif on the destination ovs during pre livemigrate and neutron will emit the vif plugged events as it wires them up | 20:29 |
mnaser | does anyone know if there's some sort of variable you can use in docs to output the current version/release | 20:30 |
mnaser | reason - https://docs.openstack.org/nova/queens/cli/nova-manage.html -- this doc and many others have old versions listed | 20:30 |
mnaser | i can update it manually but it'll probably be forgotten again :> | 20:30 |
mriedem | mnaser: https://review.openstack.org/#/c/553903/ | 20:31 |
mnaser | mriedem: oh well there you go, that's a solution | 20:32 |
sean-k-mooney2 | mriedem: ok so the vif plugged event is coming from the pre livemigrate when os-vif adds the port to ovs and the unplug event is coming form activateing the binding on the destenation host. is that the order it happens in | 20:33 |
*** lpetrut has quit IRC | 20:34 | |
*** lpetrut has joined #openstack-nova | 20:34 | |
*** openstackgerrit has joined #openstack-nova | 20:35 | |
openstackgerrit | Merged openstack/nova master: Don't filter out sibling sets with one core https://review.openstack.org/537361 | 20:35 |
mriedem | sean-k-mooney2: yes i think so, | 20:37 |
mriedem | sean-k-mooney2: i'm confused as to why neutron thinks the port is unbound though | 20:37 |
mriedem | this is on both the source and dest neutron agent | 20:37 |
mriedem | http://logs.openstack.org/71/551371/6/check/legacy-tempest-dsvm-multinode-live-migration/4d466b2/logs/screen-q-agt.txt.gz#_Mar_19_14_25_08_821754 | 20:37 |
mriedem | http://logs.openstack.org/71/551371/6/check/legacy-tempest-dsvm-multinode-live-migration/4d466b2/logs/subnode-2/screen-q-agt.txt.gz#_Mar_19_14_25_09_150483 | 20:37 |
sean-k-mooney2 | subnode-2 is the source node correct | 20:38 |
mriedem | yes | 20:38 |
sean-k-mooney2 | this is where the vif is pulled on the destination in prelivemigrate http://logs.openstack.org/71/551371/6/check/legacy-tempest-dsvm-multinode-live-migration/4d466b2/logs/screen-n-cpu.txt.gz#_Mar_19_14_25_03_105245 | 20:38 |
mriedem | this is where the source n-cpu changes the port binding host_id to the dest host http://logs.openstack.org/71/551371/6/check/legacy-tempest-dsvm-multinode-live-migration/4d466b2/logs/subnode-2/screen-n-cpu.txt.gz#_Mar_19_14_25_06_838681 | 20:41 |
mriedem | in q-svc, i see this http://logs.openstack.org/71/551371/6/check/legacy-tempest-dsvm-multinode-live-migration/4d466b2/logs/screen-q-svc.txt.gz#_Mar_19_14_25_06_991106 | 20:41 |
mriedem | Device cd0afcf3-c654-488e-b578-6d3c45f3a5a5 not bound to the agent host ubuntu-xenial-rax-dfw-0003055550 | 20:41 |
mriedem | so i guess that's why the vif type is 'unbound'? | 20:42 |
mriedem | but i'm not sure why that is because we plugged the vif on the dest host before all that | 20:42 |
sean-k-mooney2 | well this is where we plug it on the destination in the q-agt log http://logs.openstack.org/71/551371/6/check/legacy-tempest-dsvm-multinode-live-migration/4d466b2/logs/screen-q-agt.txt.gz#_Mar_19_14_25_03_126952 | 20:48 |
sean-k-mooney2 | and then right after it updates the vif_type to unbound http://logs.openstack.org/71/551371/6/check/legacy-tempest-dsvm-multinode-live-migration/4d466b2/logs/screen-q-agt.txt.gz#_Mar_19_14_25_04_008234 | 20:49 |
openstackgerrit | Merged openstack/nova master: Ensure emulator threads are always calculated https://review.openstack.org/537363 | 20:49 |
openstackgerrit | Merged openstack/nova master: [libvirt] Add _get_numa_memnode() https://review.openstack.org/529906 | 20:50 |
mriedem | sean-k-mooney2: hmm yeah wtf | 20:50 |
sean-k-mooney2 | so its the neutron agent on the destination that is seting the status to down and vif_type to unbound. | 20:50 |
*** gouthamr has quit IRC | 20:51 | |
*** jmlowe has quit IRC | 20:51 | |
sean-k-mooney2 | it kind of makes sense. at this point the vif is still bound on the host and we have not activated the vif on the dest but we just plugged it | 20:51 |
mriedem | this is all before nova hits the lifecycle post-copy event and switches the port binding, so presumably this already all happens today during live migration | 20:51 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Address issues raised in adding member_of to GET /a-c https://review.openstack.org/554357 | 20:51 |
edleafe | efried: ^^ gift-wrapped just for you | 20:51 |
*** fragatina has joined #openstack-nova | 20:52 | |
sean-k-mooney2 | mriedem: do we do the vif plugging in pre_livemigrate today or is it down after we bind the port on the destination after livemigration completes | 20:52 |
mriedem | the dest host plugs the vif in pre_live_migration today | 20:53 |
mriedem | source rpc calls to dest pre_live_migration, and then once that rpc call returns, the source starts live migrating the guest in the hypervisor | 20:53 |
sean-k-mooney2 | ok so ya we should be seeing the same bevavior. | 20:54 |
mriedem | this was part of the thing where the source host needs to wait for the vif plugged event from the dest | 20:54 |
*** tbachman has quit IRC | 20:54 | |
mriedem | before it starts transferring the guest | 20:54 |
mriedem | i haven't coded that part up yet | 20:54 |
mriedem | for all i know, given https://review.openstack.org/#/c/553035/ - can we even reliably wait for vif-plugged on the source if the host binding hasn't changed? | 20:55 |
sean-k-mooney2 | when you call self.network_api.migrate_instance_finish(context, instance,...) does that activate the port binding on the dest | 20:55 |
mriedem | or will opendaylight never send a vif-plugged event in that case? | 20:55 |
mriedem | sean-k-mooney2: it switches the binding host_id yeah, sec | 20:55 |
*** pchavva has quit IRC | 20:55 | |
*** amoralej is now known as amoralej|off | 20:55 | |
mriedem | https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L2577 | 20:56 |
mriedem | in this case, that host variable is the dest host | 20:56 |
mriedem | that's what we see here http://logs.openstack.org/71/551371/6/check/legacy-tempest-dsvm-multinode-live-migration/4d466b2/logs/subnode-2/screen-n-cpu.txt.gz#_Mar_19_14_25_06_838681 | 20:56 |
mriedem | on the source host | 20:56 |
sean-k-mooney2 | mriedem: im not sure if odl will remit the event but i dont think that would be an unreasonable expectation. | 20:57 |
mriedem | sean-k-mooney2: from what i remember of the discussion leading up to https://review.openstack.org/#/c/553035/ with mnaser, ODL will only emit events for host binding changes, not vif unplug/plug | 20:57 |
*** _ix has joined #openstack-nova | 20:58 | |
mriedem | which really kind of kills us as the consumer of this workflow... | 20:58 |
sean-k-mooney2 | yes and when we activate the binding for the dest that should be considered a binding change as we update the host_id in the port bindings_details field on the port | 20:58 |
*** dtruong has joined #openstack-nova | 20:59 | |
mriedem | yeah, but the plan was to not start migrating the guest in the hypervisor until the source got the event that the plug, initiated from the dest, is done | 21:00 |
_ix | Good afternoon, folks. Can anyone explain the pipeline from the fine work that's going into nova and the end repositories at say http://mirror.centos.org/centos/7/cloud/x86_64/openstack-pike/ | 21:00 |
*** burt has quit IRC | 21:01 | |
sean-k-mooney2 | mriedem: right so we might have to start the migrate on a timeout and have the event short circute it, instead of waiting | 21:01 |
mriedem | really kind of need an admin-only field on the port to tell clients, like nova, if we can expect vif plug events or not | 21:01 |
_ix | I'm only seeing latest as 16.0.3 in that repo -- is there a different repository that you mgith be able to recommend for 16.1.0 ? | 21:02 |
mriedem | sean-k-mooney2: we could, but then anyone using ODL is going to wait up to 5 minutes for an event that's not going to come | 21:02 |
mriedem | _ix: you could try asking in #openstack-rpm-packaging | 21:02 |
sean-k-mooney2 | mriedem: ill double check the odl code. ill be back in the office tomorrow so i can try and set up an odl environment | 21:03 |
mriedem | _ix: or maybe #rdo | 21:03 |
mriedem | sean-k-mooney2: ok for now i'm just going to neuter this lifecycle event callback code that calls migrate_instance_finish | 21:03 |
sean-k-mooney2 | well we can still activate the dest binding when we get the hyperviors event that it just paused the source and is about to unpause the dest right | 21:05 |
sean-k-mooney2 | we just dont wait for the vif plugged event | 21:05 |
mriedem | two separate issues, | 21:05 |
mriedem | i'm going to neuter the first right now because it's causing this vif unbound explosion in _post_live_migrate | 21:06 |
*** itlinux has quit IRC | 21:06 | |
mriedem | the latter issue is not something we're hitting right now, but was in the plan as part of this spec | 21:06 |
sean-k-mooney2 | ah ok | 21:06 |
*** mgoddard has quit IRC | 21:14 | |
*** suresh12 has quit IRC | 21:15 | |
openstackgerrit | Merged openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | 21:17 |
_ix | mriedem: Thanks for the tips. | 21:17 |
*** elmaciej has joined #openstack-nova | 21:21 | |
jaypipes | phew, my brain is melting today... | 21:27 |
*** suresh12 has joined #openstack-nova | 21:30 | |
*** felipemonteiro__ has quit IRC | 21:32 | |
*** jmlowe has joined #openstack-nova | 21:34 | |
*** lpetrut has quit IRC | 21:34 | |
*** sar has quit IRC | 21:35 | |
cdent | was it the videophone jaypipes ? | 21:38 |
*** elmaciej has quit IRC | 21:39 | |
*** sean-k-mooney2 has quit IRC | 21:39 | |
edleafe | efried: ugh - I had all those changes, and then rebased poorly after the original member_of patch merged. Fixing... | 21:40 |
efried | edleafe: Here to keep you honest. I feel your rebase pain, homey. | 21:41 |
edleafe | efried: heh, trying to keep things straight while half-paying attention to meetings | 21:42 |
cdent | efried: "I have looked at this." beyond that and the typos, do you think I'm on the right track? | 21:45 |
efried | cdent: The unedited version was something like, "I have looked at this. I have no idea what I'm looking at, so I am abstaining from voting." | 21:46 |
cdent | ah, that makes a bit more sense | 21:46 |
efried | cdent: "...and the time it would take me to figure out what I'm looking at doesn't fit in my current budget (still ploughing through vacation backlog)." | 21:47 |
cdent | if/when you surface from that and you want a tour, let me know | 21:47 |
cdent | thanks for looking in any case, I'm always glad to have your proofing | 21:48 |
*** gyee has quit IRC | 21:50 | |
*** tssurya has quit IRC | 21:51 | |
*** Guest68461 has quit IRC | 21:54 | |
*** jackie-truong has quit IRC | 21:58 | |
*** germs has joined #openstack-nova | 22:00 | |
*** germs has quit IRC | 22:00 | |
*** germs has joined #openstack-nova | 22:00 | |
*** suresh12 has quit IRC | 22:03 | |
*** germs has quit IRC | 22:05 | |
edleafe | efried: let | 22:08 |
edleafe | oops | 22:08 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Address issues raised in adding member_of to GET /a-c https://review.openstack.org/554357 | 22:08 |
edleafe | efried: let's try this again ^^ | 22:08 |
efried | ack | 22:08 |
jaypipes | cdent: nah, trying to fix the nested providers alloc candidates stuff | 22:09 |
*** Darky1UVFKE has joined #openstack-nova | 22:13 | |
*** liusheng has quit IRC | 22:14 | |
*** liusheng has joined #openstack-nova | 22:14 | |
efried | mriedem: 1.19 was for bp placement-aggregate-generation. Gerrit likes to overwrite the topic for all patches when submitting a series, which is how 1.19 ended up in the whiteboard for https://blueprints.launchpad.net/nova/+spec/generation-from-create-provider. FTFY. | 22:19 |
*** mlavalle has quit IRC | 22:20 | |
*** AlexeyAbashkin has joined #openstack-nova | 22:20 | |
efried | mriedem: You can mark https://blueprints.launchpad.net/nova/+spec/placement-aggregate-generation done too. | 22:22 |
mriedem | melwitt: totally random but i was just doing some blame game and came across https://review.openstack.org/#/c/377093/1/nova/rpc.py | 22:22 |
mriedem | melwitt: ever noticed that RequestContext(overwrite) kwarg is not used at all in that patch? | 22:22 |
mriedem | https://review.openstack.org/#/c/377093/1/nova/context.py@72 | 22:23 |
mriedem | efried: done | 22:24 |
efried | thx | 22:24 |
*** AlexeyAbashkin has quit IRC | 22:25 | |
melwitt | mriedem: yeah, it's used in the base context class from oslo https://github.com/openstack/oslo.context/blob/a8d86df/oslo_context/context.py#L225 | 22:25 |
mriedem | oh i see | 22:26 |
melwitt | it's kind of unclear though, being lumped into **kwargs like that | 22:26 |
mriedem | have been trying to figure out wtf periodic tasks are running with a request id which is the same request id i'm tracking for an instance create operation | 22:26 |
*** suresh12 has joined #openstack-nova | 22:27 | |
melwitt | ah, okay. I'd like to know how that happens too | 22:27 |
mriedem | well, when the periodics run, they call get_admin_context | 22:27 |
mriedem | which doesn't overwrite the thread local context | 22:28 |
mriedem | so i guess that's why | 22:28 |
melwitt | oh, so they just get whatever was there. yeah | 22:28 |
mriedem | https://github.com/openstack/nova/blob/master/nova/service.py#L295 | 22:28 |
mriedem | which are run from this thread group timer https://github.com/openstack/nova/blob/master/nova/service.py#L213 | 22:28 |
mriedem | but it makes tracing a failed operation for a specific instance really frustrating if everything else picks up the thread local version | 22:29 |
melwitt | right | 22:29 |
melwitt | I have this old patch up around generating an admin context up front and storing it on the service for use in periodic tasks (to help us get to configuring CellDatabases differently). I'm not sure if it would do any better than the current situation though https://review.openstack.org/#/c/524306 | 22:30 |
melwitt | like, I'm not sure if that initial get_admin_context in the service creation might also get tied to some unhelpful request id | 22:30 |
melwitt | er, service.start() I mean | 22:31 |
*** suresh12 has quit IRC | 22:31 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423 | 22:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 22:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Port binding based on events during live migration https://review.openstack.org/434870 | 22:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170 | 22:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: compute: use port binding extended API during live migration https://review.openstack.org/551371 | 22:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 22:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Turn on new port binding extended live migrate flow https://review.openstack.org/552173 | 22:32 |
mriedem | it shouldn't if we're not calling get_admin_context() every time the periodic runs | 22:33 |
*** rcernin has joined #openstack-nova | 22:34 | |
mriedem | i was going to try setting the request_id='req-%s' % self.service_ref.uuid so we'd have a predictable request id in periodics, but that would just be ignored since get_admin_context uses overwrite=False | 22:34 |
melwitt | okay, then that patch might help then. it got stalled on me not providing a reliable poison fixture for making sure nothing in periodics tries to get_admin_context on their own. and gibi proposed something better around that. but I haven't gotten back to it | 22:35 |
*** elmaciej has joined #openstack-nova | 22:37 | |
*** suresh12 has joined #openstack-nova | 22:39 | |
*** edmondsw has quit IRC | 22:49 | |
dansmith | mriedem: I'm still not sure how that makes sense, | 22:49 |
dansmith | the periodics are their own (eventlet) thread, they should have their own TLS | 22:49 |
*** hongbin has quit IRC | 22:49 | |
dansmith | not sure why they shouldn't be able to set their TLS there | 22:49 |
dansmith | I'd guess that overwrite=False is actually for the case where some user operation needs an admin context to do something specific, and you don't want to leave them with admin when you're done | 22:50 |
dansmith | but periodics probably *should* overwrite TLS in that case | 22:50 |
dansmith | I would also expect TLS to be reset on each spawn/switch of a thread, otherwise you leak things between requests until you set TLS _to_ something | 22:51 |
mriedem | i assume overwrite=False was added to get_admin_context() for a wholly different reason than why we're using get_admin_context() for periodics | 22:51 |
dansmith | that's what I'm saing | 22:52 |
dansmith | for a localized "okay, do this thing for the user as admin" | 22:52 |
dansmith | looking up a fixedip before assignment, etc | 22:52 |
dansmith | that's why I'm saying we probably want overwrite=True for periodics | 22:52 |
dansmith | that kinda goes against the original supposition of melwitt's patch to client RPC though | 22:53 |
melwitt | here's the original bug that led to using overwrite=False https://bugs.launchpad.net/nova/+bug/1627838 | 22:53 |
openstack | Launchpad bug 1627838 in OpenStack Compute (nova) newton "Context is reset to by ClientRouter in nova/rpc.py" [High,Fix committed] - Assigned to Matt Riedemann (mriedem) | 22:53 |
dansmith | hmm, I don't get that | 22:54 |
*** suresh12 has quit IRC | 22:55 | |
*** suresh12 has joined #openstack-nova | 22:56 | |
melwitt | yeah, I guess that was a different thing. normally we run periodics with get_admin_context which does an overwrite=False and when I added a different unrelated periodic I didn't use get_admin_context and inadvertently made it overwrite=True without realizing it | 23:00 |
melwitt | and the reported problem was that the periodic was blowing away other requests auth token in-flight | 23:01 |
*** david-lyle has joined #openstack-nova | 23:01 | |
dansmith | although ClientRouter does schedule periodics quite differently from service it seems | 23:01 |
melwitt | yeah, maybe there is something wrong with how it's running periodics? | 23:03 |
dansmith | well, | 23:03 |
*** hemna_ has quit IRC | 23:03 | |
dansmith | I'm wondering if that's why it needed overwrite=False, but our periodics in compute manager need overwrite=True | 23:03 |
melwitt | oh, I see | 23:04 |
dansmith | melwitt: that thing needed periodics to age out clients but that code seems gone I think, so can we remove thatl ine anyway? | 23:04 |
melwitt | yeah, I was thinking that too | 23:04 |
dansmith | so we're running periodics in our managers via threadgroup timers, which I assume run in threads of their own, but I should check | 23:06 |
dansmith | mriedem: yeah, so that _remove_stale_clients() periodic was removed later, so we should nuke that line | 23:06 |
dansmith | er, melwitt ^ | 23:06 |
melwitt | ack. I can propose that | 23:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove RequestContext.instance_lock_checked https://review.openstack.org/554378 | 23:10 |
mriedem | more cleanup while i was poking around ^ | 23:10 |
dansmith | so I'm not exactly seeing how the way periodics are used in ClientRouter would have worked, other than the single go of them run via __init__ | 23:11 |
dansmith | but my guess is that they're not run the way we run them in our service | 23:11 |
dansmith | melwitt: do we know for sure that those periodics ever ran more than once? | 23:11 |
dansmith | that code wasn't in the tree for very long, so.. | 23:11 |
dansmith | it sure looks like we just fired them off once during __init_ and never ran them again | 23:12 |
*** cdent has quit IRC | 23:12 | |
dansmith | which is also puzzling as to why that would be a problem needing to be reported, but the bug doesn't have much info | 23:13 |
melwitt | dansmith: maybe not. ClientRouter inherits from oslo PeriodicTasks, so it would do this https://github.com/openstack/oslo.service/blob/master/oslo_service/periodic_task.py#L193 | 23:13 |
dansmith | melwitt: well, it does that once when you run it in __init__ | 23:13 |
dansmith | but I don't see where that gets scheduled any other place | 23:14 |
dansmith | service.py will do it, but you're not a service there | 23:14 |
dansmith | and the tests just fire the handler manually | 23:16 |
melwitt | hm | 23:16 |
dansmith | (as expected, we don't really have good coverage over periodics) | 23:16 |
dansmith | https://www.youtube.com/watch?v=C9MG9b4mEU0 | 23:17 |
melwitt | lol, great movie | 23:18 |
dansmith | the only thing I can think of, | 23:18 |
dansmith | would be that we instantiate a new compute rpc class for certain things at runtime, | 23:19 |
dansmith | and maybe we blow away whatever context was current at that time, | 23:19 |
dansmith | because anything running ComputeRPCAPI.__init() will run ClientRouter.__init__(), which will instantiate a context with overwrite=True before your patch | 23:19 |
dansmith | er, rpcapi.ComputeAPI() I mean | 23:20 |
*** AlexeyAbashkin has joined #openstack-nova | 23:20 | |
melwitt | yeah :\ | 23:20 |
dansmith | mriedem: so I think maybe chatting with someone like dhellmann might be a good idea, but I kinda think we should be running with overwrite=True on those periodics' context | 23:22 |
dansmith | I'd also like someone to tell me why TLS isn't cleared when a new thread is spawned and/or would like to confirm that timers as we use them to start periodics in our services are actually running in their own threads and not as if they were signal handlers | 23:24 |
*** AlexeyAbashkin has quit IRC | 23:24 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix message for unexpected external event https://review.openstack.org/554380 | 23:26 |
mriedem | ok, but not tonight | 23:27 |
*** masber has joined #openstack-nova | 23:27 | |
*** yamamoto has joined #openstack-nova | 23:27 | |
*** chyka has quit IRC | 23:30 | |
*** chyka has joined #openstack-nova | 23:31 | |
openstackgerrit | melanie witt proposed openstack/nova master: Remove useless run_periodic_tasks call in ClientRouter https://review.openstack.org/554381 | 23:31 |
melwitt | dansmith, mriedem ^ I left the related overwrite=False regression test case alone thinking we would address that when we get answers about what we should be doing with overwrite + periodics later | 23:34 |
dansmith | melwitt: hmm, I'm not sure what that test is testing TBH | 23:35 |
dansmith | I'm pretty fried and well past punkin time, so I'll look tomorrow with a fresh head | 23:35 |
melwitt | it's testing that we don't overwrite=True during periodics because at the time we thought that was messing up things | 23:35 |
melwitt | okay, cool | 23:36 |
*** suresh12 has quit IRC | 23:39 | |
dansmith | I don't think it does though, because we just start the service and then assert our thing, | 23:42 |
dansmith | I don't think we know that the periodics have run yet | 23:42 |
dansmith | we don't run service.periodic_tasks() until the first time we fire the timer, which is delayed from startup, and that's where context gets generated for the periodic run | 23:43 |
*** claudiub has quit IRC | 23:44 | |
*** Sukhdev_ has quit IRC | 23:44 | |
*** r-daneel has quit IRC | 23:49 | |
*** elmaciej has quit IRC | 23:51 | |
*** mriedem has quit IRC | 23:52 | |
*** elmaciej has joined #openstack-nova | 23:52 | |
*** elmaciej_ has joined #openstack-nova | 23:55 | |
*** elmaciej has quit IRC | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!