*** acormier has quit IRC | 00:03 | |
*** tetsuro_ has joined #openstack-nova | 00:04 | |
*** amodi has joined #openstack-nova | 00:11 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in get https://review.openstack.org/531299 | 00:11 |
---|---|---|
*** yangyape_ has quit IRC | 00:33 | |
*** dave-mcc_ has joined #openstack-nova | 00:33 | |
*** armax has joined #openstack-nova | 00:34 | |
*** hiro-kobayashi has joined #openstack-nova | 00:34 | |
*** dave-mccowan has quit IRC | 00:35 | |
*** s1061123_ has joined #openstack-nova | 00:35 | |
*** s1061123 has quit IRC | 00:35 | |
*** hongbin has joined #openstack-nova | 00:38 | |
*** threestrands has joined #openstack-nova | 00:43 | |
*** hiro-kobayashi has quit IRC | 00:44 | |
*** hongbin has quit IRC | 00:47 | |
*** hoangcx has joined #openstack-nova | 00:48 | |
*** edmondsw has joined #openstack-nova | 00:52 | |
*** edmondsw has quit IRC | 00:56 | |
*** zhaochao has joined #openstack-nova | 01:04 | |
*** hshiina has joined #openstack-nova | 01:11 | |
*** phuongnh has joined #openstack-nova | 01:12 | |
*** acormier has joined #openstack-nova | 01:14 | |
*** acormier has joined #openstack-nova | 01:15 | |
openstackgerrit | Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | 01:17 |
*** yassine has joined #openstack-nova | 01:19 | |
*** yassine is now known as Guest53311 | 01:19 | |
*** yangyapeng has joined #openstack-nova | 01:20 | |
*** hongbin has joined #openstack-nova | 01:22 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | 01:24 |
*** acormier has quit IRC | 01:30 | |
*** zhurong has joined #openstack-nova | 01:30 | |
*** acormier has joined #openstack-nova | 01:30 | |
*** claudiub|2 has quit IRC | 01:37 | |
*** namnh has joined #openstack-nova | 01:44 | |
*** moshele has joined #openstack-nova | 01:45 | |
*** takashin has left #openstack-nova | 01:51 | |
*** xinliang has quit IRC | 01:56 | |
*** mtreinish has quit IRC | 01:58 | |
*** mtreinish has joined #openstack-nova | 01:59 | |
*** threestrands has quit IRC | 02:01 | |
*** moshele has quit IRC | 02:05 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 02:05 |
*** itlinux has joined #openstack-nova | 02:06 | |
*** xinliang has joined #openstack-nova | 02:09 | |
*** lei-zh has joined #openstack-nova | 02:11 | |
*** threestrands has joined #openstack-nova | 02:16 | |
*** threestrands has quit IRC | 02:16 | |
*** threestrands has joined #openstack-nova | 02:16 | |
openstackgerrit | Spencer Yu proposed openstack/nova master: project name should be removed from zuul configuration https://review.openstack.org/540695 | 02:20 |
*** Tom-Tom has joined #openstack-nova | 02:21 | |
*** liuzz has joined #openstack-nova | 02:22 | |
*** itlinux has quit IRC | 02:37 | |
*** edmondsw has joined #openstack-nova | 02:40 | |
*** jmlowe has quit IRC | 02:43 | |
*** zhurong has quit IRC | 02:43 | |
*** edmondsw has quit IRC | 02:45 | |
*** jmlowe has joined #openstack-nova | 02:45 | |
*** yamamoto has joined #openstack-nova | 02:47 | |
*** bhujay has joined #openstack-nova | 03:03 | |
*** lyan has joined #openstack-nova | 03:04 | |
*** annp has joined #openstack-nova | 03:05 | |
*** bhujay has quit IRC | 03:18 | |
*** zhenguo has joined #openstack-nova | 03:21 | |
*** acormier has quit IRC | 03:23 | |
*** dave-mcc_ has quit IRC | 03:32 | |
*** sree has joined #openstack-nova | 03:34 | |
*** hshiina is now known as hshiina|afk | 03:35 | |
*** lyan has quit IRC | 03:35 | |
*** hshiina|afk has quit IRC | 03:35 | |
*** gcb has joined #openstack-nova | 03:37 | |
*** amodi has quit IRC | 03:37 | |
*** lyan has joined #openstack-nova | 03:39 | |
*** lyan has quit IRC | 03:47 | |
*** itlinux has joined #openstack-nova | 03:48 | |
*** Tom-Tom has quit IRC | 03:49 | |
*** Tom-Tom has joined #openstack-nova | 03:50 | |
*** gongysh has joined #openstack-nova | 03:54 | |
*** Tom-Tom has quit IRC | 03:55 | |
*** psachin has joined #openstack-nova | 04:06 | |
*** threestrands has quit IRC | 04:07 | |
*** udesale has joined #openstack-nova | 04:08 | |
*** itlinux has quit IRC | 04:12 | |
*** gongysh has quit IRC | 04:13 | |
*** vladikr has joined #openstack-nova | 04:13 | |
*** abhishekk has joined #openstack-nova | 04:13 | |
openstackgerrit | Chiawei Xie proposed openstack/nova stable/ocata: Modify nova-manage db migrations base message https://review.openstack.org/540709 | 04:16 |
*** vladikr has quit IRC | 04:18 | |
*** threestrands has joined #openstack-nova | 04:19 | |
*** threestrands has quit IRC | 04:19 | |
*** threestrands has joined #openstack-nova | 04:19 | |
*** vladikr has joined #openstack-nova | 04:19 | |
*** edmondsw has joined #openstack-nova | 04:28 | |
*** Spazmotic is now known as Spaz-Korea | 04:32 | |
Spaz-Korea | Flight in 5 hours. almost time :) | 04:32 |
Spaz-Korea | See you all in about 2 weeks | 04:32 |
*** edmondsw has quit IRC | 04:33 | |
*** lei-zh has quit IRC | 04:33 | |
*** gongysh has joined #openstack-nova | 04:33 | |
*** bhujay has joined #openstack-nova | 04:36 | |
*** threestrands has quit IRC | 04:36 | |
Spaz-Korea | If you need me for anything drop me a mail at bhkaminski (at) gmail.com | 04:36 |
*** threestrands has joined #openstack-nova | 04:37 | |
*** gongysh has quit IRC | 04:38 | |
*** vladikr has quit IRC | 04:51 | |
*** vladikr has joined #openstack-nova | 04:54 | |
*** threestrands has quit IRC | 05:07 | |
*** claudiub|2 has joined #openstack-nova | 05:20 | |
*** lei-zh has joined #openstack-nova | 05:25 | |
*** ratailor has joined #openstack-nova | 05:29 | |
*** zhenguo has quit IRC | 05:31 | |
*** Tom-Tom has joined #openstack-nova | 05:32 | |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 05:33 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 05:33 |
openstackgerrit | jichenjc proposed openstack/nova master: z/VM driver: add snapshot function https://review.openstack.org/534240 | 05:33 |
*** trinaths has joined #openstack-nova | 05:33 | |
*** zhurong has joined #openstack-nova | 05:38 | |
*** jaosorior has joined #openstack-nova | 05:39 | |
*** hongbin has quit IRC | 05:40 | |
openstackgerrit | jichenjc proposed openstack/nova master: fix race condition of instance host https://review.openstack.org/494458 | 05:41 |
*** jaypipes_ has joined #openstack-nova | 05:42 | |
*** threestrands has joined #openstack-nova | 05:42 | |
*** leakypipes has quit IRC | 05:44 | |
openstackgerrit | jichenjc proposed openstack/nova master: mv generate_glance_url to get_image_endpoint_url https://review.openstack.org/511400 | 05:45 |
*** links has joined #openstack-nova | 05:46 | |
*** lajoskatona has joined #openstack-nova | 05:47 | |
*** threestrands has quit IRC | 05:47 | |
*** threestrands has joined #openstack-nova | 05:47 | |
*** links has quit IRC | 05:49 | |
*** mdnadeem has joined #openstack-nova | 05:50 | |
*** links has joined #openstack-nova | 05:51 | |
*** takashin has joined #openstack-nova | 05:53 | |
*** links has quit IRC | 05:59 | |
*** links has joined #openstack-nova | 06:00 | |
*** jmlowe has quit IRC | 06:04 | |
*** threestrands_ has joined #openstack-nova | 06:04 | |
*** threestrands_ has quit IRC | 06:04 | |
*** threestrands_ has joined #openstack-nova | 06:04 | |
*** jmlowe has joined #openstack-nova | 06:05 | |
*** threestrands has quit IRC | 06:06 | |
*** links has quit IRC | 06:14 | |
*** links has joined #openstack-nova | 06:16 | |
*** edmondsw has joined #openstack-nova | 06:17 | |
*** jmlowe has quit IRC | 06:18 | |
*** jmlowe has joined #openstack-nova | 06:20 | |
*** ameeda has joined #openstack-nova | 06:21 | |
*** edmondsw has quit IRC | 06:21 | |
*** acormier has joined #openstack-nova | 06:25 | |
openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats https://review.openstack.org/526833 | 06:27 |
*** acormier has quit IRC | 06:29 | |
*** tuanla____ has joined #openstack-nova | 06:29 | |
*** sridharg has joined #openstack-nova | 06:30 | |
*** jmlowe has quit IRC | 06:32 | |
*** jmlowe has joined #openstack-nova | 06:33 | |
*** gongysh has joined #openstack-nova | 06:44 | |
*** jmlowe has quit IRC | 06:47 | |
*** moshele has joined #openstack-nova | 06:48 | |
*** jmlowe has joined #openstack-nova | 06:48 | |
*** jaosorior has quit IRC | 06:50 | |
*** pcaruana has joined #openstack-nova | 07:04 | |
*** jmlowe has quit IRC | 07:04 | |
*** josecastroleon has joined #openstack-nova | 07:06 | |
*** jmlowe has joined #openstack-nova | 07:06 | |
*** jaosorior has joined #openstack-nova | 07:12 | |
*** alexchadin has joined #openstack-nova | 07:12 | |
*** kholkina has joined #openstack-nova | 07:16 | |
*** _ioni is now known as ioni | 07:17 | |
*** ioni has left #openstack-nova | 07:18 | |
*** ioni has joined #openstack-nova | 07:18 | |
*** rcernin has quit IRC | 07:18 | |
*** xinliang has quit IRC | 07:18 | |
*** lpetrut has joined #openstack-nova | 07:19 | |
*** maciejjozefczyk has quit IRC | 07:24 | |
*** vladikr has quit IRC | 07:24 | |
*** vladikr has joined #openstack-nova | 07:25 | |
*** vladikr has quit IRC | 07:33 | |
openstackgerrit | jichenjc proposed openstack/nova-specs master: add z/VM Driver in Rocky https://review.openstack.org/536186 | 07:37 |
*** AlexeyAbashkin has joined #openstack-nova | 07:41 | |
*** alexchadin has quit IRC | 07:42 | |
*** alexchadin has joined #openstack-nova | 07:43 | |
*** vladikr has joined #openstack-nova | 07:45 | |
*** AlexeyAbashkin has quit IRC | 07:45 | |
*** AlexeyAbashkin has joined #openstack-nova | 07:50 | |
*** andreas_s has joined #openstack-nova | 07:54 | |
*** vladikr has quit IRC | 07:57 | |
*** ragiman has joined #openstack-nova | 07:58 | |
*** jaosorior has quit IRC | 08:00 | |
*** moshele has quit IRC | 08:00 | |
*** takashin has left #openstack-nova | 08:02 | |
*** moshele has joined #openstack-nova | 08:02 | |
*** edmondsw has joined #openstack-nova | 08:05 | |
*** slaweq has joined #openstack-nova | 08:05 | |
*** maciejjozefczyk has joined #openstack-nova | 08:06 | |
*** alexchadin has quit IRC | 08:08 | |
*** damien_r has joined #openstack-nova | 08:09 | |
*** giblet is now known as gibi | 08:09 | |
*** edmondsw has quit IRC | 08:09 | |
*** andreas_s has quit IRC | 08:10 | |
*** maciejjozefczyk has quit IRC | 08:11 | |
*** vladikr has joined #openstack-nova | 08:12 | |
gibi | edleafe: both zuul's and your environment is missing a bugfix from ovo that my patch is depends on (Depends-On: https://review.openstack.org/540344). If you pull down that bugfix in your env then the test will pass | 08:14 |
gibi | edleafe: this of course means that we can only merge the patch after the ovo fix is merged and the ovo version is bumped in nova | 08:14 |
*** andreas_s has joined #openstack-nova | 08:15 | |
*** hoonetorg has quit IRC | 08:15 | |
*** ralonsoh has joined #openstack-nova | 08:16 | |
*** ssurana has joined #openstack-nova | 08:17 | |
*** ssurana has quit IRC | 08:17 | |
*** naichuans has joined #openstack-nova | 08:18 | |
ameeda | gibi: good morning :) | 08:19 |
ameeda | can you please review this for me ? https://review.openstack.org/#/c/528385/ | 08:19 |
*** masahisa has quit IRC | 08:20 | |
*** masahisa has joined #openstack-nova | 08:22 | |
*** maciejjozefczyk has joined #openstack-nova | 08:24 | |
*** andreas_s has quit IRC | 08:25 | |
*** sahid has joined #openstack-nova | 08:25 | |
*** ccamacho has joined #openstack-nova | 08:27 | |
*** hoonetorg has joined #openstack-nova | 08:28 | |
*** tesseract has joined #openstack-nova | 08:28 | |
*** andreas_s has joined #openstack-nova | 08:30 | |
*** andreas_s has quit IRC | 08:30 | |
*** andreas_s has joined #openstack-nova | 08:30 | |
openstackgerrit | jichenjc proposed openstack/nova master: fix race condition of instance host https://review.openstack.org/494458 | 08:35 |
*** ttsiouts has joined #openstack-nova | 08:36 | |
*** AJaeger has joined #openstack-nova | 08:37 | |
AJaeger | nova cores, could you merge a small docs update, please? https://review.openstack.org/#/c/538163/ I need another +2.. | 08:38 |
*** amoralej|off is now known as amoralej | 08:38 | |
AJaeger | stephenfin, gibi, can you help, please? ^ | 08:39 |
openstackgerrit | jichenjc proposed openstack/nova master: Remove quota reserve/commit/rollback https://review.openstack.org/521470 | 08:41 |
*** amorin has joined #openstack-nova | 08:43 | |
amorin | hello all | 08:43 |
gibi | ameeda: will try to look at it today | 08:43 |
*** Spaz-Korea has quit IRC | 08:43 | |
gibi | AJaeger: looking | 08:43 |
ameeda | gibi: thank you very much, please don't forget | 08:44 |
*** jpena|off is now known as jpena | 08:45 | |
gibi | ameeda: I tend not to forget these. It is more like I run out of time | 08:46 |
ameeda | gibi: Thank you !! | 08:47 |
AJaeger | thanks, gibi ! | 08:49 |
*** AJaeger has left #openstack-nova | 08:50 | |
*** ircuser-1 has joined #openstack-nova | 08:50 | |
amorin | I would like to know if there is any way to create a neutron client from nova, which could use the retries parameter of neutron | 08:52 |
amorin | on nova side: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L137 | 08:52 |
amorin | neutron side: https://github.com/openstack/python-neutronclient/blob/master/neutronclient/v2_0/client.py#L220 | 08:52 |
amorin | seems that nova is not using retries at all | 08:52 |
*** threestrands_ has quit IRC | 08:54 | |
*** tssurya has joined #openstack-nova | 08:56 | |
*** jmlowe has quit IRC | 08:59 | |
*** vladikr has quit IRC | 09:00 | |
*** jmlowe has joined #openstack-nova | 09:01 | |
bauzas | good morning folksq | 09:03 |
gibi | bauzas: good morning | 09:05 |
amorin | morning | 09:05 |
*** moshele has quit IRC | 09:05 | |
*** moshele has joined #openstack-nova | 09:07 | |
openstackgerrit | jichenjc proposed openstack/nova master: deprecate fping_path config option https://review.openstack.org/526602 | 09:17 |
*** sridharg has quit IRC | 09:18 | |
*** derekh has joined #openstack-nova | 09:20 | |
*** sridharg has joined #openstack-nova | 09:21 | |
*** Dinesh_Bhor has joined #openstack-nova | 09:24 | |
*** mgoddard_ has joined #openstack-nova | 09:30 | |
*** tianhui has joined #openstack-nova | 09:30 | |
*** Dinesh_Bhor has quit IRC | 09:32 | |
*** alexchadin has joined #openstack-nova | 09:35 | |
*** alexchadin has quit IRC | 09:39 | |
*** alexchadin has joined #openstack-nova | 09:40 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 09:43 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Trivial fix a missleading comment https://review.openstack.org/540796 | 09:48 |
*** lei-zh has quit IRC | 09:57 | |
openstackgerrit | Merged openstack/nova master: Zuul: Remove project name https://review.openstack.org/540326 | 10:03 |
openstackgerrit | Merged openstack/nova master: Fix SUSE Install Guide: Placement port https://review.openstack.org/538163 | 10:03 |
*** dtantsur|afk is now known as dtantsur | 10:05 | |
*** sree has quit IRC | 10:05 | |
*** sree has joined #openstack-nova | 10:06 | |
gibi | the gate is super fast now :) | 10:08 |
*** annp has quit IRC | 10:09 | |
*** jmlowe has quit IRC | 10:09 | |
*** jmlowe has joined #openstack-nova | 10:10 | |
openstackgerrit | Jianghua Wang proposed openstack/nova master: doc: update the GPU passthrough HPC feature entry https://review.openstack.org/539638 | 10:11 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: Provide support matrix and doc for VGPU https://review.openstack.org/539266 | 10:11 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: Provide support matrix and doc for VGPU https://review.openstack.org/540808 | 10:11 |
*** sree has quit IRC | 10:14 | |
*** phuongnh has quit IRC | 10:20 | |
*** stakeda has quit IRC | 10:26 | |
*** mdnadeem has quit IRC | 10:28 | |
*** mdnadeem has joined #openstack-nova | 10:28 | |
*** Tom-Tom has quit IRC | 10:31 | |
*** Tom-Tom has joined #openstack-nova | 10:32 | |
*** namnh has quit IRC | 10:33 | |
*** Tom-Tom has quit IRC | 10:36 | |
*** jaosorior has joined #openstack-nova | 10:43 | |
*** cdent has joined #openstack-nova | 10:45 | |
ameeda | gibi: replied | 10:50 |
*** lucas-afk is now known as lucasagomes | 10:52 | |
*** tuanla____ has quit IRC | 10:53 | |
*** yangyapeng has quit IRC | 10:56 | |
*** yangyapeng has joined #openstack-nova | 10:57 | |
*** isssp has quit IRC | 10:57 | |
*** ispp has joined #openstack-nova | 10:57 | |
*** Guest53311 has quit IRC | 10:57 | |
*** sambetts|afk is now known as sambetts | 11:00 | |
*** sahid has quit IRC | 11:01 | |
*** jmlowe has quit IRC | 11:02 | |
*** jmlowe has joined #openstack-nova | 11:03 | |
*** gszasz has joined #openstack-nova | 11:06 | |
*** sahid has joined #openstack-nova | 11:07 | |
*** sdague has quit IRC | 11:09 | |
*** abalutoiu_ has joined #openstack-nova | 11:11 | |
gibi | ameeda: OK, I will double check with mriedem about the double detach | 11:11 |
ameeda | gibi: thank you !! | 11:12 |
gibi | ameeda: until then, I will try to make the functional test work without the need for mocking the detach call | 11:12 |
ameeda | gibi: yes please , I try a lot with it with no lucks | 11:13 |
*** abalutoiu__ has quit IRC | 11:15 | |
ameeda | gibi: I have 2 nova reviews needs for 2 +2 here https://review.openstack.org/#/c/539880/ and https://review.openstack.org/#/c/540274/ | 11:16 |
ameeda | gibi: also I need help here https://review.openstack.org/#/c/526900/ | 11:16 |
*** andreas_s has quit IRC | 11:16 | |
ameeda | sorry for inconvenience, thanks for you time !! | 11:16 |
*** andreas_s has joined #openstack-nova | 11:17 | |
*** gongysh has quit IRC | 11:17 | |
*** sahid has quit IRC | 11:19 | |
*** sahid has joined #openstack-nova | 11:19 | |
*** jmlowe has quit IRC | 11:20 | |
*** sahid has quit IRC | 11:20 | |
*** sahid has joined #openstack-nova | 11:20 | |
*** jmlowe has joined #openstack-nova | 11:25 | |
openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003 | 11:25 |
hrw | morning | 11:26 |
*** andreas_s has quit IRC | 11:26 | |
*** udesale has quit IRC | 11:30 | |
*** andreas_s has joined #openstack-nova | 11:31 | |
rabel | hi. can we merge https://review.openstack.org/#/c/535871/ ? | 11:33 |
*** abhishekk has quit IRC | 11:38 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 11:38 |
*** jmlowe has quit IRC | 11:39 | |
*** jmlowe has joined #openstack-nova | 11:39 | |
*** andreas_s has quit IRC | 11:40 | |
*** edmondsw has joined #openstack-nova | 11:41 | |
*** alexchadin has quit IRC | 11:44 | |
*** claudiub has joined #openstack-nova | 11:44 | |
*** alexchadin has joined #openstack-nova | 11:45 | |
*** edmondsw has quit IRC | 11:45 | |
*** andreas_s has joined #openstack-nova | 11:46 | |
*** claudiub|2 has quit IRC | 11:47 | |
*** stvnoyes has quit IRC | 11:49 | |
*** acormier has joined #openstack-nova | 11:54 | |
*** acormier has quit IRC | 11:55 | |
*** sdague has joined #openstack-nova | 11:55 | |
*** acormier has joined #openstack-nova | 11:55 | |
*** andreas_s has quit IRC | 11:55 | |
*** andreas_s has joined #openstack-nova | 11:56 | |
*** stvnoyes has joined #openstack-nova | 11:57 | |
*** gszasz has quit IRC | 11:57 | |
*** moshele has quit IRC | 11:59 | |
*** gszasz has joined #openstack-nova | 12:00 | |
*** masahisa has quit IRC | 12:00 | |
*** ratailor has quit IRC | 12:01 | |
*** pcaruana has quit IRC | 12:02 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add missing encryption_secret_uuid tests https://review.openstack.org/540679 | 12:03 |
lyarwood | ops | 12:03 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add missing encryption_secret_uuid tests https://review.openstack.org/540679 | 12:03 |
lyarwood | there we go | 12:03 |
*** jmlowe has quit IRC | 12:04 | |
*** andreas_s has quit IRC | 12:04 | |
*** andreas_s has joined #openstack-nova | 12:04 | |
*** jmlowe has joined #openstack-nova | 12:05 | |
*** trinaths has quit IRC | 12:07 | |
*** jmlowe has quit IRC | 12:15 | |
*** yassine has joined #openstack-nova | 12:16 | |
*** jaosorior has quit IRC | 12:16 | |
*** gongysh has joined #openstack-nova | 12:17 | |
*** yassine is now known as Guest44860 | 12:17 | |
*** dtantsur is now known as dtantsur|brb | 12:17 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Unmap compute nodes when deleting host mapping https://review.openstack.org/527560 | 12:23 |
*** jpena is now known as jpena|lunch | 12:25 | |
*** jmlowe has joined #openstack-nova | 12:26 | |
ameeda | gibi: replied | 12:26 |
*** yangyapeng has quit IRC | 12:29 | |
*** gcb has quit IRC | 12:30 | |
*** acormier has quit IRC | 12:30 | |
*** yangyapeng has joined #openstack-nova | 12:33 | |
*** yangyapeng has quit IRC | 12:33 | |
openstackgerrit | Lajos Katona proposed openstack/nova master: WIP: ServerMovingTests with custom resources https://review.openstack.org/497399 | 12:34 |
*** sahid has quit IRC | 12:36 | |
*** jmlowe has quit IRC | 12:36 | |
*** sahid has joined #openstack-nova | 12:37 | |
openstackgerrit | Claudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943 | 12:42 |
*** vladikr has joined #openstack-nova | 12:42 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/533733 | 12:44 |
*** amoralej is now known as amoralej|lunch | 12:44 | |
hrw | https://review.openstack.org/#/c/532504/ - someone will give second +2 (and +w) to stable/pike backport for arm64? | 12:45 |
*** tssurya has quit IRC | 12:48 | |
openstackgerrit | Merged openstack/nova stable/ocata: Fix false positive server group functional tests https://review.openstack.org/536995 | 12:48 |
*** tetsuro_ has left #openstack-nova | 12:51 | |
*** psachin has quit IRC | 12:56 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: docs: Fix stable/pike config references https://review.openstack.org/540846 | 12:58 |
*** gongysh has quit IRC | 12:58 | |
*** abalutoiu__ has joined #openstack-nova | 13:00 | |
*** josecastroleon has quit IRC | 13:00 | |
*** josecastroleon1 has joined #openstack-nova | 13:00 | |
*** yamamoto has quit IRC | 13:01 | |
*** gongysh has joined #openstack-nova | 13:02 | |
*** alexchadin has quit IRC | 13:03 | |
*** tssurya has joined #openstack-nova | 13:03 | |
*** abalutoiu_ has quit IRC | 13:03 | |
*** alexchadin has joined #openstack-nova | 13:04 | |
*** jmlowe has joined #openstack-nova | 13:08 | |
gibi | bauzas: hi, I have a quetion in https://review.openstack.org/#/c/539266/9/doc/source/admin/virtual-gpu.rst@133 | 13:10 |
*** edmondsw has joined #openstack-nova | 13:13 | |
*** amoralej|lunch is now known as amoralej | 13:16 | |
bauzas | gibi: replying | 13:18 |
bauzas | gibi: replied ;) | 13:19 |
*** yamamoto has joined #openstack-nova | 13:19 | |
gibi | bauzas: thanks. so what is this '*Error* state' the doc is refer to? | 13:21 |
*** sahid has quit IRC | 13:23 | |
*** sahid has joined #openstack-nova | 13:23 | |
*** gongysh has quit IRC | 13:25 | |
bauzas | gibi: http://paste.openstack.org/show/662486/ | 13:25 |
bauzas | HTH | 13:25 |
*** jpena|lunch is now known as jpena | 13:27 | |
gibi | bauzas: ahh so the suspend instance action will have the error state | 13:28 |
bauzas | exactly | 13:28 |
*** vladikr has quit IRC | 13:28 | |
bauzas | hence not the VM state "ERROR" | 13:29 |
gibi | bauzas: thanks. | 13:29 |
bauzas | (not capitalized") | 13:29 |
gibi | bauzas: I +Wd that patch | 13:30 |
bauzas | cool thanks | 13:30 |
gibi | thank you for creating such a good doc | 13:31 |
*** liverpooler has joined #openstack-nova | 13:31 | |
bauzas | thanks | 13:32 |
*** masahisa has joined #openstack-nova | 13:33 | |
*** yangyapeng has joined #openstack-nova | 13:35 | |
*** liverpooler has quit IRC | 13:36 | |
*** edmondsw has quit IRC | 13:36 | |
*** liverpooler has joined #openstack-nova | 13:36 | |
*** edmondsw has joined #openstack-nova | 13:36 | |
*** udesale has joined #openstack-nova | 13:37 | |
*** jaosorior has joined #openstack-nova | 13:37 | |
*** udesale has quit IRC | 13:37 | |
*** artom has joined #openstack-nova | 13:37 | |
*** udesale has joined #openstack-nova | 13:38 | |
openstackgerrit | sahid proposed openstack/nova master: hardware: only take into account small pages https://review.openstack.org/532168 | 13:39 |
*** udesale has quit IRC | 13:39 | |
*** masahisa has quit IRC | 13:39 | |
*** yangyapeng has quit IRC | 13:39 | |
*** udesale has joined #openstack-nova | 13:41 | |
*** moshele has joined #openstack-nova | 13:41 | |
*** vladikr has joined #openstack-nova | 13:41 | |
*** edmondsw has quit IRC | 13:41 | |
*** edmondsw has joined #openstack-nova | 13:44 | |
*** pchavva has joined #openstack-nova | 13:48 | |
*** edmondsw has quit IRC | 13:50 | |
*** edmondsw has joined #openstack-nova | 13:51 | |
*** edmondsw has quit IRC | 13:55 | |
jaypipes_ | morning supernovas | 13:56 |
edleafe | Scheduler subteam meeting in 3 minutes in #openstack-meeting-alt | 13:57 |
*** lajoskatona has quit IRC | 13:58 | |
*** takashin has joined #openstack-nova | 13:58 | |
*** edmondsw has joined #openstack-nova | 13:59 | |
*** acormier has joined #openstack-nova | 13:59 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Removed unused argument from LibvirtDriver._connect_volume https://review.openstack.org/540859 | 13:59 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Remove unused argument from LibvirtDriver._disconnect_volume https://review.openstack.org/540860 | 13:59 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: libvirt: Collocate encryptor and volume driver calls https://review.openstack.org/540861 | 13:59 |
edleafe | Scheduler subteam meeting running NOW in #openstack-meeting-alt | 14:00 |
*** edmondsw has quit IRC | 14:01 | |
*** edmondsw has joined #openstack-nova | 14:01 | |
*** acormier has quit IRC | 14:04 | |
*** jaypipes_ is now known as jaypipes | 14:06 | |
*** gszasz has quit IRC | 14:06 | |
*** dtantsur|brb is now known as dtantsur | 14:07 | |
*** dave-mccowan has joined #openstack-nova | 14:08 | |
*** acormier has joined #openstack-nova | 14:10 | |
*** sree has joined #openstack-nova | 14:13 | |
*** acormier has quit IRC | 14:14 | |
*** Tom-Tom has joined #openstack-nova | 14:14 | |
bauzas | wow, I feel the breeze of not having internal stuff to do | 14:16 |
*** sree has quit IRC | 14:17 | |
openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003 | 14:18 |
openstackgerrit | Merged openstack/nova stable/pike: Fix the ocata config-reference URLs https://review.openstack.org/515410 | 14:22 |
openstackgerrit | Merged openstack/nova master: doc: update the GPU passthrough HPC feature entry https://review.openstack.org/539638 | 14:22 |
openstackgerrit | Merged openstack/nova master: Provide support matrix and doc for VGPU https://review.openstack.org/539266 | 14:23 |
ameeda | jaypipes_: could you please check this https://review.openstack.org/#/c/526900/ ? | 14:27 |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Remove 2 redundant methods https://review.openstack.org/528325 | 14:27 |
*** david-lyle has quit IRC | 14:30 | |
*** lucasagomes is now known as lucas-hungry | 14:30 | |
*** dklyle has quit IRC | 14:32 | |
*** claudiub|2 has joined #openstack-nova | 14:34 | |
*** abalutoiu_ has joined #openstack-nova | 14:34 | |
gibi | bauzas: If you have some free time, then there is a bugfix that needs some review https://review.openstack.org/#/c/525242/ :) | 14:36 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/540871 | 14:37 |
bauzas | gibi: free time is not exactly what I call as my current workload, but I'll ;) | 14:37 |
*** claudiub has quit IRC | 14:37 | |
*** abalutoiu__ has quit IRC | 14:37 | |
*** hrw has quit IRC | 14:37 | |
gibi | bauzas: Ohh then I was misled by your feeling the breeze comment above :) | 14:38 |
bauzas | gibi: oh, your patch is crucial https://review.openstack.org/#/c/525242/ | 14:39 |
*** dave-mcc_ has joined #openstack-nova | 14:39 | |
*** mriedem has joined #openstack-nova | 14:39 | |
mriedem | lyarwood: i think it would be ok to just leave that as a separate refactor/test coverage change on top | 14:39 |
mriedem | don't really want to hold up https://review.openstack.org/#/c/539739/ since it's breaking cinder 3rd party CI | 14:40 |
*** moshele has quit IRC | 14:40 | |
*** hrw has joined #openstack-nova | 14:40 | |
lyarwood | mriedem: yup np | 14:40 |
*** gongysh has joined #openstack-nova | 14:41 | |
*** dave-mccowan has quit IRC | 14:42 | |
gibi | mriedem: hi, I think we have some indirect disagreement in https://review.openstack.org/#/c/528385 I think the duplicate detach will cause confusions in the logs and in the notification interface | 14:43 |
*** abalutoiu__ has joined #openstack-nova | 14:45 | |
*** takashin has left #openstack-nova | 14:45 | |
efried | cdent: This sentence does not parse for me: "That new structure allowed for the creation a POST to /allocations that allows an atomic action to manage allocations for multiple consumers in one request, supporting race-free migration allocations." | 14:46 |
*** gcb has joined #openstack-nova | 14:46 | |
*** abalutoiu_ has quit IRC | 14:48 | |
*** gongysh has quit IRC | 14:49 | |
*** awaugama has joined #openstack-nova | 14:50 | |
*** bhujay has quit IRC | 14:51 | |
bauzas | gibi: mriedem: I'm a bit concerned by a bugfix having RPC versions bumped but it's understandable given the upcall issue | 14:51 |
bauzas | context being https://review.openstack.org/#/c/525242/14 | 14:51 |
gibi | bauzas: First I tried without the rpc bump but that solution would create yet another cell upcall | 14:51 |
*** esberglu has joined #openstack-nova | 14:52 | |
bauzas | yeah I know | 14:52 |
bauzas | FWIW, I have a long standing blueprint for passing request specs to computes | 14:52 |
bauzas | it's purely procedural tho | 14:52 |
bauzas | I'm fine with approving that change if everything else is ok | 14:53 |
bauzas | just thinking out loud | 14:53 |
*** josecastroleon1 has quit IRC | 14:53 | |
gibi | yes, we need to agree on such a bump together. Especially as dansmith might plan to do a major rpc version bump before the release | 14:54 |
dansmith | yeah, unfortunately, that reqspec cleanup would be really nice to get into place | 14:54 |
* bauzas seriously apologies for not having implemented yet the blueprint | 14:55 | |
*** tssurya has quit IRC | 14:56 | |
mriedem | i got 2 patches in against it for queens! | 14:56 |
gibi | bauzas: if that helps with the current bugfix, I can offer some help with the request spec blueprint in Rocky | 14:56 |
mriedem | with great success | 14:56 |
efried | bauzas: What's a request spec, as distinct from allocation request data? | 14:57 |
efried | That is, what does a request spec has that an allocation request doesn't? | 14:58 |
mriedem | gibi: replied in https://review.openstack.org/#/c/528385/ | 14:58 |
mriedem | efried: all sorts of stuff, | 14:58 |
mriedem | instance groups, scheduler hints, | 14:58 |
mriedem | image id | 14:58 |
mriedem | it's basically the instance | 14:58 |
mriedem | minus host stuff (and BDMs for some reason) | 14:58 |
efried | okay, thanks. | 14:59 |
efried | cause jfyi, we are passing the allocation to the compute (https://review.openstack.org/#/c/511879/) | 15:00 |
bauzas | efried: originally the request spec is what was passed to the scheduler for filtering purposes | 15:00 |
mriedem | efried: in this case we need the scheduling hints | 15:00 |
efried | coo | 15:00 |
bauzas | efried: it became used for more than just the scheduler | 15:00 |
bauzas | because we have the original intent of the user + some quirks | 15:01 |
bauzas | the flaw in that model (and I apologize for not having considered it at the design state) is that we don't know which fields are immutable and which not | 15:01 |
gibi | mriedem: thanks the reply in https://review.openstack.org/#/c/528385/, I'm OK with the extra kwargs | 15:02 |
bauzas | so, when you review a change using that RequestSpec object record, try to consider all fields as potentially mutable | 15:02 |
*** hongbin has joined #openstack-nova | 15:02 | |
gibi | ameeda: I think you have a way forward now in https://review.openstack.org/#/c/528385/ | 15:03 |
*** r-daneel has joined #openstack-nova | 15:03 | |
openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/540879 | 15:04 |
*** acormier has joined #openstack-nova | 15:04 | |
*** mchlumsky has joined #openstack-nova | 15:04 | |
*** josecastroleon has joined #openstack-nova | 15:09 | |
*** tssurya has joined #openstack-nova | 15:11 | |
*** alexchadin has quit IRC | 15:14 | |
lyarwood | mriedem: https://review.openstack.org/#/c/531422/ - happy to +W this now that https://review.openstack.org/#/c/540680/1 is posted? | 15:16 |
*** lucas-hungry is now known as lucasagomes | 15:22 | |
*** tbachman has joined #openstack-nova | 15:24 | |
mriedem | gibi: comments in https://review.openstack.org/#/c/525242/ | 15:25 |
gibi | mriedem: looking | 15:26 |
*** amodi has joined #openstack-nova | 15:27 | |
cdent | efried: sorry, was walking. On the POST, do the nearby links clarify things at all? On the allocations have a resource provider with a generation that's here: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/allocation.py#L210-L220 | 15:28 |
*** lpetrut has quit IRC | 15:28 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Don't use single backticks in man pages https://review.openstack.org/540887 | 15:29 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using openstackdoctheme's extlink extension https://review.openstack.org/540888 | 15:29 |
*** sshwarts has joined #openstack-nova | 15:29 | |
*** armax has quit IRC | 15:32 | |
mriedem | gibi: and the functional test below | 15:33 |
*** armax has joined #openstack-nova | 15:33 | |
mriedem | dansmith: melwitt: easy quick cells v2 layout caveats update for stuff that is now fixed in queens https://review.openstack.org/#/c/540667/ | 15:33 |
gibi | mriedem: thanks | 15:34 |
*** yangyapeng has joined #openstack-nova | 15:34 | |
mriedem | bauzas: this looks nice https://docs.openstack.org/nova/latest/admin/virtual-gpu.html thanks for doing that | 15:34 |
*** READ10 has joined #openstack-nova | 15:37 | |
*** yangyapeng has quit IRC | 15:38 | |
efried | cdent: I knew what you meant; was just commenting that the sentence didn't grammar for me. | 15:40 |
cdent | efried: because of the missing "of" or more than that? | 15:40 |
*** mlavalle has joined #openstack-nova | 15:40 | |
efried | cdent: TBH, I couldn't even interpolate what was missing/wrong. Let me go read it again... | 15:41 |
cdent | i've just put the "of" in | 15:41 |
mriedem | lyarwood: done | 15:41 |
lyarwood | mriedem: thanks | 15:42 |
*** david-lyle has joined #openstack-nova | 15:42 | |
efried | cdent: Oh, yeah, that (or the addition of caffeine since first reading) makes it a lot clearer. | 15:42 |
efried | cdent: And when you get a chance, you said you could point me to where allocations were managing generations. | 15:42 |
mriedem | lyarwood: can you hit the other 2 pike doc backports here? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/pike+topic:bug/1714017 | 15:42 |
lyarwood | mriedem: sure, looking now | 15:43 |
cdent | efried: second part of [t vPW] | 15:43 |
purplerbot | <cdent> efried: sorry, was walking. On the POST, do the nearby links clarify things at all? On the allocations have a resource provider with a generation that's here: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/allocation.py#L210-L220 [2018-02-05 15:28:10.592679] [n vPW] | 15:43 |
*** lbragstad_ is now known as lbragstad | 15:43 | |
*** slaweq has quit IRC | 15:43 | |
*** slaweq has joined #openstack-nova | 15:43 | |
*** slaweq has quit IRC | 15:43 | |
efried | cdent: That's on the return from POST /allocations? | 15:43 |
mriedem | lyarwood: i'm not sure about this https://review.openstack.org/#/c/540846/ | 15:44 |
cdent | that gives the Allocation object and rp (with a genaration), which is then checked at https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L2079-L2085 | 15:44 |
mriedem | it's going to be super annoying if we have to update all doc backports to point at the branch they are on | 15:44 |
mriedem | we don't do that when we branch | 15:44 |
cdent | efried: that's on the _start_ of PUT or POST | 15:44 |
cdent | on the way in, not the way out | 15:44 |
openstackgerrit | Merged openstack/nova master: Fix invalid UUIDs in test_compute.py https://review.openstack.org/531479 | 15:44 |
cdent | efried: so the generation is checked, but the client doesn't provide | 15:44 |
openstackgerrit | Merged openstack/nova stable/pike: Don't update RT in _allocate_network https://review.openstack.org/519942 | 15:44 |
openstackgerrit | Merged openstack/python-novaclient master: Zuul: Remove project name https://review.openstack.org/540304 | 15:44 |
cdent | which seems ... useless? | 15:44 |
efried | cdent: Right. What are we checking the generation *against*? | 15:45 |
efried | cdent: (BTW, in that code block, I still don't see where the generation is being checked, even internally) | 15:45 |
cdent | efried: anything that changes from the start of the PUT to the time when _increment_provider_generation is called | 15:45 |
bauzas | mriedem: thanks for pushing me the idea :-) | 15:46 |
efried | cdent: Okay, well, I should hope that *all* the APIs are doing *that*, to guarantee atomicity of a single call. But yeah, totally not the point I'm actually making in the bug report. | 15:46 |
cdent | efried: yes, your bug report is a different concern | 15:46 |
cdent | and no, we shouldn't be doing that in all single calls | 15:46 |
cdent | it is pointless in allocations | 15:46 |
mriedem | lyarwood: are you planning on backporting this entire series to ocata also https://review.openstack.org/#/c/540861/ ? | 15:47 |
cdent | if the allocation fits, write it | 15:47 |
mriedem | that's a lot of refactor | 15:47 |
cdent | efried: but we don't currently do that | 15:47 |
*** gcb has quit IRC | 15:47 | |
lyarwood | mriedem: still working out if that's even possible | 15:47 |
efried | cdent: So that's the point we need to consider. I'm not sure I agree that we should always write the allocation if it still fits. | 15:47 |
lyarwood | mriedem: yeah, the refactor also fixes a load of connect/disonnect issues | 15:47 |
lyarwood | mriedem: really I should've landed a smaller bugfix ahead of this on master to backport | 15:48 |
mriedem | hmmm, | 15:49 |
mriedem | like what? | 15:49 |
cdent | efried: if you can come up with a cogent explanation of why, that would be great, but to me it violates the spirit of allocations to do anything else and also over constrains the system. | 15:49 |
mriedem | lyarwood: why not just put a change on stable that says if you're trying to swap encrypted volumes, we fail? | 15:49 |
efried | cdent: Okay, example: since we're explicitly *not* handling weighing in placement, supporting generation would be a way the caller (who has to handle weighing) can ensure the weighing assumptions they make in the PUT/POST allocation call actually apply at the time when the allocation is written. | 15:49 |
lyarwood | mriedem: yup that works and avoids this horrid backport, I'll do that now. | 15:50 |
mriedem | lyarwood: since swap volume doesn't work with encrypted volumes until queens (and only if you're not using native luks encryption), it seems ok to just make it outright fail on stable | 15:50 |
cdent | efried: "can actually apply" shouldn't mean generation (when it comes to allocations) it should mean capacity/capability | 15:50 |
cdent | (or if I'm not understanding you, you'll need to be more complete with your example) | 15:51 |
efried | cdent: I agree with that, but "should actually apply" is based on assumptions about the state of providers when the request was calculated. | 15:51 |
*** rodolof has joined #openstack-nova | 15:52 | |
cdent | which is why we 409 if that state has changed, in a way that changes capacity, by checking the capacity at allocation write-time | 15:52 |
efried | cdent: Yeah, capacity conflicts I'm cool with, put that aside. | 15:52 |
edleafe | cdent: I think jaypipes expressed something similar a while ago. If a deployment is trying for spread, and two requests to allocate against the same host come in, the second should fail | 15:52 |
cdent | if we want to rely on generation, then we are vastly decreasing the odds of an allocation being able to land | 15:53 |
edleafe | IMO, that's a huge nova-ism for placement to base its behavior on | 15:53 |
efried | Yes, we may want to consider making the generation handling optional for allocations. But it should at least be possible. | 15:53 |
cdent | efried: I still haven't heard why? | 15:53 |
*** chyka has joined #openstack-nova | 15:54 | |
cdent | was " is trying for spread, and two requests to allocate against the same host come in, the second should fail" a reason why? If so, I don't understand? | 15:54 |
*** chyka has quit IRC | 15:54 | |
*** mdnadeem has quit IRC | 15:54 | |
efried | cdent: Yeah. Scheduler is scheduling three instances. Each of the three threads requests allocation candidates "at the same time". Gets back the same list. Scheduler applies a "least consumed" weigher in each of those three threads, and winds up sorting the candidates in the same order. Scheduler sends all three PUT /allocations/{c} at the same time, with the same allocation request in each. | 15:55 |
edleafe | cdent: because nova thinks the host is relatively empty, so it chooses it. But had the first allocation been known, it would have chosen an emptier host for the second | 15:55 |
efried | cdent: Even assuming there's capacity for all three on the same set of RPs, by the time we've allocated the first one, the "least consumed" condition, upon which the second scheduler thread's PUT request was based, no longer applies. | 15:56 |
*** chyka has joined #openstack-nova | 15:56 | |
efried | cdent: Which may be an acceptable limitation of the scheduler's weigher. But it may not be. | 15:57 |
*** sapcc-bot has quit IRC | 15:57 | |
efried | cdent: Even more so if we have shared RPs involved, and these threads could be hitting from multiple separate compute nodes. | 15:57 |
*** sapcc-bot has joined #openstack-nova | 15:57 | |
edleafe | efried: this all seems like a good argument for placement not being part of nova | 15:57 |
cdent | a) that's not a thing I think placement should concern itself with, b) that's not a concern I think a cloud should concern itself with (in that fashion, it shoul be orchestrated at a different layer) | 15:57 |
*** sree has joined #openstack-nova | 15:57 | |
efried | cdent: That's precisely my point. It should *not* be placement's responsibility. But therefore it *needs* to be the caller's. | 15:58 |
efried | cdent: But with the API as it stands, we provide the caller no mechanism to enable it to execute that responsibility. | 15:58 |
dansmith | mriedem: we kinda disabled the on_shared_storage flag, IIRC.. are we good to remove that in the rpc bump or do we need to keep it so the api can ask us to check it for compat? | 15:58 |
efried | cdent: Generation would be one such mechanism. | 15:58 |
edleafe | efried: so would something like Watcher, which can re-balance things at the cloud level | 15:59 |
cdent | efried: the caller can do what it likes, managing data how it likes, but it shouldn't rely on placement to do it for it, at the level of allocations. It could make requests for allocation candidates how it likes, look at them, analyze then, do whatever. | 15:59 |
mriedem | dansmith: are you asking b/c of gibi's functional test? | 15:59 |
mriedem | or b/c you're bumping major versions? | 15:59 |
efried | edleafe: After the allocation has been made and the instance has been scheduled? Sure. Messy, though. | 15:59 |
edleafe | efried: IOW, detect a crowded host and performt the migration to uncrowd it | 15:59 |
edleafe | efried: it | 15:59 |
edleafe | it's a cloud | 15:59 |
*** chyka has quit IRC | 16:00 | |
*** chyka has joined #openstack-nova | 16:01 | |
efried | edleafe: Right, I get it. I'm saying migration-after-the-fact is a viable solution, but it's more expensive/disruptive to the workload than getting the scheduling "right" in the first place. | 16:01 |
edleafe | efried: it *will* get it right the overwhelming majority of the time | 16:01 |
edleafe | The case we're describing is an edge of an edge case | 16:01 |
efried | edleafe: Actually, I think it'll get it "wrong" (for common definitions of "wrong") quite often. | 16:02 |
*** sree has quit IRC | 16:02 | |
efried | edleafe: Only when lots of concurrent deploys are happening, of course. | 16:02 |
dansmith | mriedem: bumping major versions | 16:02 |
efried | edleafe: Or concurrent migrations (such as an "evacuate host" scenario). | 16:02 |
gibi | mriedem: I'm in the process to updat that patch not to depened on that flag | 16:03 |
cdent | efried: if you're a person deploying 3 instances and the mechanism you want to use to ensure they go different places is "not loaded please", that sounds wrong. Presumably what you really want is anti-affinity and not loaded. | 16:03 |
edleafe | efried: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L411-L422 | 16:03 |
cdent | If you're just going with "not loaded" and you get something weird, isn't that you're own fault? | 16:03 |
cdent | sigh: your | 16:04 |
efried | cdent: I mean, we can document it that way, but it's not very satisfying to the consumer. | 16:04 |
efried | cdent: And I don't agree that you necessarily want anti-affinity. | 16:04 |
edleafe | efried: and also https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L426-L436 | 16:05 |
efried | If host A is 10% full and host B is 90% full, I totally want my instances (each of which consumes 10%) to both go to host A. | 16:05 |
*** pchavva has quit IRC | 16:05 | |
edleafe | There is plenty of race avoidance in the code. | 16:05 |
*** mchlumsky has quit IRC | 16:05 | |
cdent | I dunno, I think your host should just go where it will get what it needs, unless you tell it something special. | 16:05 |
cdent | And providing too many knobs is bad for everyone | 16:06 |
efried | edleafe: Okay, glad those mechanisms are there, but they're somewhat inelegant. (Not that thrashing GET /allocation_candidates is any more elegant - overall I think it's clunkier and less efficient - but as an example, 'twill serve.) | 16:07 |
*** armax has quit IRC | 16:07 | |
mriedem | dansmith: i'd have to take a look at that flag to see what the api does with it | 16:07 |
efried | cdent: Too many knobs, sure, but I thought a "least consumed" weigher was one of the basics. | 16:07 |
dansmith | mriedem: okay I thought you did it which is why I was asking.. I'll add a note and come back to it | 16:07 |
openstackgerrit | Merged openstack/nova stable/pike: doc: Add user index page https://review.openstack.org/540494 | 16:07 |
openstackgerrit | Merged openstack/nova stable/pike: Migrate "launch instance" user guide docs https://review.openstack.org/540495 | 16:08 |
cdent | efried: is it? to me it sounds like a "exposing too much info to the user about this cloud" and "our resource accounting and allocation ratio subscription model is actually a cheat, here we've provided you with this way to get around it rather than being good cloud providers with good ratios" | 16:09 |
mriedem | dansmith: no i don't think so, sounds like a tdurakov thing | 16:09 |
dansmith | mriedem: ack | 16:09 |
* cdent pours one out for tdurakov | 16:09 | |
*** armax has joined #openstack-nova | 16:09 | |
*** mchlumsky has joined #openstack-nova | 16:10 | |
efried | cdent: I don't see what this has to do with allocation ratios. | 16:10 |
cdent | if allocation ratios are set fairly and correctly, then where your instance goes shouldn't matter: it should always have good access to resources whatever host it lands one | 16:11 |
cdent | if hosts are oversiubscribed then of course you're going to try to do tricks to get on an underutilized host | 16:11 |
*** burt has joined #openstack-nova | 16:12 | |
efried | cdent: That makes a lot of assumptions about the possible reasons for a "least consumed" deployment strategy. | 16:12 |
cdent | what other reason can you want other than needing to ensure you "get enough"? | 16:13 |
*** armax has quit IRC | 16:13 | |
efried | cdent, edleafe: Anyway, y'all have convinced me we don't need/want this for Nova. So until/unless we get a request from some other placement consumer (which isn't quite so likely to happen as long as placement is part of nova) I can drop it. | 16:13 |
*** armax has joined #openstack-nova | 16:14 | |
*** kholkina has quit IRC | 16:14 | |
*** armax has left #openstack-nova | 16:14 | |
*** slaweq has joined #openstack-nova | 16:14 | |
* cdent schwews | 16:14 | |
*** armax has joined #openstack-nova | 16:16 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095 | 16:18 |
openstack | bug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer) | 16:18 |
*** sree has joined #openstack-nova | 16:18 | |
gibi | mriedem: ^^ fixed the functional test, now start fixing the bugfix patch | 16:18 |
*** slaweq has quit IRC | 16:18 | |
*** pchavva has joined #openstack-nova | 16:20 | |
*** abalutoiu_ has joined #openstack-nova | 16:23 | |
*** sree has quit IRC | 16:23 | |
*** ragiman has quit IRC | 16:24 | |
mriedem | dansmith: looks like gibi made onSharedStorage optional in evacuate | 16:26 |
*** abalutoiu__ has quit IRC | 16:26 | |
dansmith | mriedem: right, but it was required in earlier api versions | 16:26 |
openstackgerrit | Jay Pipes proposed openstack/nova master: only increment disk address unit for scsi devices https://review.openstack.org/538310 | 16:26 |
mriedem | dansmith: yeah, so it looks like we could kill it from compute... | 16:26 |
dansmith | so I'm wondering if we have to keep the yes/no/unspecified support in the lower layers | 16:26 |
jaypipes | melwitt, mdbooth: really need some assistance from either/both of you on https://review.openstack.org/538310. Please see my desperate cry for help in the second unit test there... | 16:27 |
mriedem | unless there is some scenario where operators actually need to rely on it b/c the auto-check using the driver is wrong, | 16:27 |
mriedem | but that would seem like a bug | 16:27 |
dansmith | mriedem: well, the point being the api used to advertise that it will check your work | 16:27 |
dansmith | so if people used that, and we silently ignore it in compute now... | 16:27 |
mriedem | it used it for some other stuff too, related to the adminPass | 16:28 |
*** udesale has quit IRC | 16:28 | |
*** vladikr has quit IRC | 16:28 | |
mriedem | reading the spec https://specs.openstack.org/openstack/nova-specs/specs/mitaka/implemented/optional-shared-storage-flag-in-evacuate-api.html | 16:29 |
openstackgerrit | Eric Fried proposed openstack/nova master: placement doc: Conflict caveat for DELETE APIs https://review.openstack.org/540919 | 16:30 |
efried | cdent, edleafe: ^ For the remainder of bug 1746373 | 16:30 |
openstack | bug 1746373 in OpenStack Compute (nova) "Placement APIs with missing conflict detection" [Undecided,In progress] https://launchpad.net/bugs/1746373 - Assigned to Eric Fried (efried) | 16:30 |
efried | jaypipes also ^ | 16:30 |
mriedem | dansmith: fine to just leave it in for compat | 16:32 |
*** AlexeyAbashkin has quit IRC | 16:34 | |
*** AlexeyAbashkin has joined #openstack-nova | 16:34 | |
*** yamamoto has quit IRC | 16:36 | |
*** andreas_s has quit IRC | 16:36 | |
*** andreas_s has joined #openstack-nova | 16:37 | |
*** links has quit IRC | 16:37 | |
*** itlinux has joined #openstack-nova | 16:37 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Rename the 'recreate' param to rebuild_instance to 'evacuate' https://review.openstack.org/508190 | 16:39 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Rename the 'recreate' param in rebuild_instance to 'evacuate' https://review.openstack.org/508190 | 16:39 |
*** itlinux has quit IRC | 16:40 | |
*** claudiub has joined #openstack-nova | 16:41 | |
*** vladikr has joined #openstack-nova | 16:42 | |
*** moshele has joined #openstack-nova | 16:42 | |
*** claudiub|2 has quit IRC | 16:45 | |
*** zhaochao has quit IRC | 16:45 | |
*** andreas_s has quit IRC | 16:46 | |
*** moshele has quit IRC | 16:47 | |
hrw | stephenfin: what is docstring? | 16:47 |
openstackgerrit | Merged openstack/python-novaclient master: nova limits ERROR (Exception): Field names must be unique https://review.openstack.org/527922 | 16:47 |
openstackgerrit | Merged openstack/python-novaclient master: Implement hypervisor hostname exact pattern match https://review.openstack.org/520187 | 16:47 |
*** damien_r has quit IRC | 16:48 | |
*** itlinux has joined #openstack-nova | 16:48 | |
*** yamamoto has joined #openstack-nova | 16:49 | |
hrw | ah. """ D: | 16:49 |
*** yamamoto has quit IRC | 16:49 | |
*** yamamoto has joined #openstack-nova | 16:50 | |
*** yamamoto has quit IRC | 16:50 | |
dansmith | mriedem: ack | 16:50 |
*** edmondsw has quit IRC | 16:50 | |
mriedem | artom: https://review.openstack.org/#/c/508190/2/nova/compute/manager.py@2663 | 16:52 |
artom | mriedem, ah, doh | 16:53 |
artom | mriedem, well, my second one still kinda stands :P | 16:53 |
openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003 | 16:53 |
hrw | stephenfin: as you wish ;D | 16:53 |
openstackgerrit | Merged openstack/python-novaclient master: Add CLI to show instance usage audit logs https://review.openstack.org/409413 | 16:53 |
artom | I'll remove the -1 though | 16:54 |
*** gyee has joined #openstack-nova | 16:55 | |
*** edmondsw has joined #openstack-nova | 16:56 | |
stephenfin | hrw: ++ | 16:57 |
hrw | mriedem, jaypipes, sahid: can you take another look at https://review.openstack.org/538003 patch? it is simplest of all versions and comes with extra test | 16:59 |
*** edmondsw has quit IRC | 17:00 | |
*** moshele has joined #openstack-nova | 17:00 | |
stephenfin | bauzas: Gentle reminder for https://review.openstack.org/#/c/530743/ | 17:01 |
mriedem | stephenfin: see the latest comment there, it introduces a bug | 17:03 |
mriedem | so a big -1 from me on the backport | 17:04 |
jaypipes | hrw: I never saw a response from you to sahid's review comments? | 17:04 |
*** moshele has quit IRC | 17:04 | |
*** AlexeyAbashkin has quit IRC | 17:05 | |
hrw | jaypipes: rewrote whole as an answer | 17:05 |
stephenfin | mriedem: Yup, I was focussing on the latter part of the comment though: a partial fix was better than no fix | 17:05 |
jaypipes | hrw: heh, ok :) | 17:05 |
jaypipes | hrw: I don't see a followup from sahid which is why I asked.. | 17:05 |
stephenfin | Probably best to wait for the fixes' fix to be merged though | 17:06 |
hrw | jaypipes: sure | 17:06 |
*** crushil has joined #openstack-nova | 17:06 | |
hrw | jaypipes: when 'a reply' would be just message then I write it. If it is simple fix then I mark Done' | 17:07 |
hrw | jaypipes: if it is complicated then do code mostly | 17:07 |
*** slaweq has joined #openstack-nova | 17:08 | |
hrw | jaypipes: and here comment was more or less 'rewrite the shit' so I did | 17:08 |
*** ccamacho has quit IRC | 17:11 | |
jaypipes | hrw: understood. I'd still like to have sahid's opinion on it, though, since he had the biggest objection to it.. | 17:11 |
hrw | sure | 17:12 |
*** moshele has joined #openstack-nova | 17:12 | |
*** slaweq has quit IRC | 17:14 | |
*** slaweq has joined #openstack-nova | 17:15 | |
*** ccamacho has joined #openstack-nova | 17:15 | |
*** AlexeyAbashkin has joined #openstack-nova | 17:17 | |
*** sahid has quit IRC | 17:20 | |
*** AlexeyAbashkin has quit IRC | 17:21 | |
*** brad[] has quit IRC | 17:23 | |
*** Guest44860 has quit IRC | 17:24 | |
*** moshele has quit IRC | 17:24 | |
*** moshele has joined #openstack-nova | 17:27 | |
*** yangyapeng has joined #openstack-nova | 17:28 | |
hrw | I have an idea for another 'lets add some device' patch just not found yet where to plug it | 17:28 |
*** slaweq has quit IRC | 17:28 | |
hrw | on pcie machines (aarch64, x86/q35) more pcie-root-port controllers can be handy if someone wants to add more disks/networks | 17:29 |
*** yangyapeng has quit IRC | 17:32 | |
*** Tom-Tom has quit IRC | 17:34 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add nova-status check for ironic flavor migration https://review.openstack.org/527541 | 17:34 |
openstackgerrit | Eric Fried proposed openstack/nova master: placement doc: Conflict caveat for DELETE APIs https://review.openstack.org/540919 | 17:34 |
*** moshele has quit IRC | 17:37 | |
hrw | it is like: basic VM instance config gets created with (let's say) 4 pcie root ports (think of them as pcie slot on virtual motherboard). then we plug video card, usb host controller (and usb keyboard to it) so we have just 2 slots left. then we add RNG and memory balloon... hm. have to boot an instance and check | 17:37 |
*** slaweq has joined #openstack-nova | 17:38 | |
*** crushil has quit IRC | 17:40 | |
*** armax has quit IRC | 17:41 | |
*** slaweq has quit IRC | 17:43 | |
openstackgerrit | Merged openstack/nova master: Update cells v2 layout doc caveats for Queens https://review.openstack.org/540667 | 17:44 |
openstackgerrit | Merged openstack/nova stable/ocata: libvirt: Re-initialise volumes, encryptors, and vifs on hard reboot https://review.openstack.org/531422 | 17:44 |
*** sshwarts has quit IRC | 17:44 | |
*** sambetts is now known as sambetts|afk | 17:45 | |
hrw | arm64 guest has 1 pcie slot free when booted with graphical console | 17:49 |
*** yamamoto has joined #openstack-nova | 17:50 | |
*** sridharg has quit IRC | 17:53 | |
*** dtantsur is now known as dtantsur|afk | 17:53 | |
*** josecastroleon has quit IRC | 17:57 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242 | 17:59 |
*** derekh has quit IRC | 18:00 | |
gibi | mriedem: fixed the bugfix patch as well ^^ | 18:00 |
*** david-lyle has quit IRC | 18:01 | |
mriedem | ok | 18:01 |
mriedem | will take a look after lunch | 18:01 |
*** jose-phillips has joined #openstack-nova | 18:02 | |
gibi | mriedem: have nice lunch | 18:02 |
gibi | mriedem: I'm no checking your idea to use wait_for_versioned_notification instead of the for loop and the sleep int he functional test, but I have to leave soon so cannot promise I finish with that today | 18:03 |
mriedem | that's fine, it can be a later cleanup | 18:04 |
gibi | mriedem: OK | 18:05 |
*** yangyapeng has joined #openstack-nova | 18:05 | |
*** yamamoto has quit IRC | 18:06 | |
*** gjayavelu has joined #openstack-nova | 18:06 | |
*** mgoddard_ has quit IRC | 18:07 | |
*** ralonsoh has quit IRC | 18:07 | |
mriedem | cfriesen: i have an idea about the perf hit in pike, | 18:08 |
mriedem | maybe versioned notifications? | 18:09 |
mriedem | cfriesen: are you running with those on? | 18:09 |
mriedem | default is to send both versioned and unversioned notifications, | 18:09 |
mriedem | and the versioned notifications have to sometimes lazy-load fields on the instance for the notification payload | 18:10 |
mriedem | which means more round trips over rpc to the db | 18:10 |
*** yangyapeng has quit IRC | 18:10 | |
jaypipes | hrw: reviewed. | 18:12 |
*** josecastroleon has joined #openstack-nova | 18:12 | |
cfriesen | mriedem: will check | 18:15 |
hrw | jaypipes: reading | 18:15 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095 | 18:18 |
openstack | bug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer) | 18:18 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242 | 18:18 |
cfriesen | mriedem: looks like explicitly unversioned. thanks for the suggestion though. | 18:18 |
gibi | mriedem: managed to remove the sleep from the functional test. | 18:19 |
openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003 | 18:19 |
* gibi goes home | 18:19 | |
*** jpena is now known as jpena|off | 18:19 | |
hrw | jaypipes: thanks | 18:19 |
*** amodi has quit IRC | 18:20 | |
mriedem | cfriesen: damn | 18:21 |
dansmith | mriedem: so, we have some calls like build_and_run_instances() that has basically all optional arguments, even though they're not really optional | 18:27 |
dansmith | mriedem: we're removing one in the middle of those due to deprecations, so I was thinking I'd remove the optional/kwarg-ness from them all, | 18:28 |
dansmith | but that broke a bunch of tests that use the optionalness | 18:28 |
dansmith | I kinda think we might just want to take the hit now and fix that in the process, but do you have a strong opinion either way? | 18:28 |
mriedem | i was going to say it might not be worth the trouble to refactor all of that | 18:29 |
mriedem | but it's not a strong opinion on | 18:29 |
mriedem | *no | 18:29 |
dansmith | okay | 18:29 |
dansmith | well, I guess I'll just be lazy and not do it then | 18:30 |
*** itlinux has quit IRC | 18:31 | |
*** moshele has joined #openstack-nova | 18:32 | |
*** Swami has joined #openstack-nova | 18:32 | |
jaypipes | hrw: +W | 18:35 |
hrw | thx | 18:36 |
*** amodi has joined #openstack-nova | 18:36 | |
*** tssurya has quit IRC | 18:36 | |
openstackgerrit | Yuki Nishiwaki proposed openstack/nova master: Check enable_new_services option in service.py https://review.openstack.org/540962 | 18:36 |
*** moshele has quit IRC | 18:38 | |
*** rodolof has quit IRC | 18:44 | |
openstackgerrit | Chris Dent proposed openstack/nova-specs master: Add generation support in aggregate association https://review.openstack.org/540447 | 18:44 |
*** rodolof has joined #openstack-nova | 18:46 | |
*** mgoddard_ has joined #openstack-nova | 18:49 | |
*** traskat has joined #openstack-nova | 18:50 | |
*** sean-k-mooney has joined #openstack-nova | 18:52 | |
*** david-lyle has joined #openstack-nova | 18:57 | |
mriedem | dansmith: i'm ok with lazy, cleaning up cruft and dropping deprecated stuff is good for me | 18:58 |
mriedem | given we're only a few days out from rc1 | 18:58 |
dansmith | ack, yeah I left it alone | 18:58 |
dansmith | I'm working on all the signature changes for the other stuff in the tests right now | 18:58 |
* jaypipes gets out his sean-k-mooney-dictionary while reading review on https://review.openstack.org/533077 | 18:59 | |
*** ccamacho has quit IRC | 18:59 | |
*** harlowja has joined #openstack-nova | 19:03 | |
hrw | LibvirtConfigGuestPCIeRootPortController is my next class to add. but not today. | 19:06 |
*** itlinux has joined #openstack-nova | 19:07 | |
* hrw off | 19:13 | |
*** amoralej is now known as amoralej|off | 19:14 | |
*** brad[] has joined #openstack-nova | 19:17 | |
openstackgerrit | Merged openstack/nova stable/pike: Import the config drive docs from openstack-manuals https://review.openstack.org/515053 | 19:18 |
ameeda | gibi and mriedem Thank you very much, I will upload new patch soon | 19:18 |
dansmith | mriedem: I already made that cleanup change for a couple of the other calls, which I'm paying for in tests right now, but they're less invasive than build | 19:20 |
ameeda | mriedem: can you please confirm this for me ? https://review.openstack.org/#/c/540274/ its nova-manager doc | 19:21 |
*** slaweq has joined #openstack-nova | 19:23 | |
*** chyka has quit IRC | 19:27 | |
*** edmondsw has joined #openstack-nova | 19:29 | |
sean-k-mooney | dansmith: mriedem so i found a bug at the end of last week that has been in the nova libvirt forever https://bugs.launchpad.net/nova/+bug/1747496 | 19:30 |
openstack | Launchpad bug 1747496 in OpenStack Compute (nova) "MTUs are not set for VIFs if using kernel ovs + hybrid plug = false" [Undecided,New] | 19:30 |
sean-k-mooney | dansmith: mriedem this is not new in queens but my question is the fix is simple. just set the mtu in the libvirt xml or alternitivly use os-vif which is a little more work | 19:31 |
sean-k-mooney | dansmith: mriedem will i trow a path up for rocky or is this something ye want to backport? | 19:31 |
*** slaweq_ has joined #openstack-nova | 19:32 | |
openstackgerrit | Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385 | 19:33 |
sean-k-mooney | using os-vif is not backportable hence why i am leaning towords adding the libvirt element for the mtu. we can always delegate entirly to os-vif in rocky if we like but just wonderin if we want to fix this in queens? | 19:34 |
openstackgerrit | Merged openstack/nova stable/ocata: Set server status to ERROR if rebuild failed https://review.openstack.org/536904 | 19:36 |
*** tesseract has quit IRC | 19:36 | |
ameeda | mriedem, gibi patch uploaded | 19:36 |
*** slaweq_ has quit IRC | 19:36 | |
*** mgoddard_ has quit IRC | 19:36 | |
ameeda | jaypipes, bauzas: could you please review this for me https://review.openstack.org/#/c/526900/ | 19:37 |
*** AlexeyAbashkin has joined #openstack-nova | 19:38 | |
*** tbachman has quit IRC | 19:39 | |
*** lucasagomes is now known as lucas-afk | 19:39 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Update Provider Tree https://review.openstack.org/540111 | 19:39 |
efried | edleafe, cdent, jaypipes: ^ | 19:39 |
*** slaweq has quit IRC | 19:39 | |
*** chyka has joined #openstack-nova | 19:40 | |
*** elod has quit IRC | 19:46 | |
sean-k-mooney | stephenfin: by the way i know you were looking at mtu suff last year so this might be of interest to you. https://bugs.launchpad.net/nova/+bug/1747496 it will cause issue for anyone with jumboframes who want to upgrade to the ovs conntrack driver. | 19:47 |
openstack | Launchpad bug 1747496 in OpenStack Compute (nova) "MTUs are not set for VIFs if using kernel ovs + hybrid plug = false" [Undecided,New] | 19:47 |
*** AlexeyAbashkin has quit IRC | 19:47 | |
mriedem | sean-k-mooney: you can do a 2 part fix where the first uses the xml directly and backport that, and the other fix goes into os-vif and we remove the xml hack | 19:48 |
sean-k-mooney | mriedem: yep i can is that your prefence? also do we want to backport to queens et al now or wait | 19:49 |
sean-k-mooney | mriedem: im just heading home but i can see if i can trow up the xml patch tommorow if i get time | 19:49 |
mriedem | sean-k-mooney: i wouldn't consider it something we have to hold up queens rc1 for | 19:51 |
mriedem | if it's latent | 19:51 |
ameeda | mriedem: when I try to git cherry-pick -x c0aca1d333628f9174e8db53674787e281e7e0d7 I got this error -> fatal: bad object c0aca1d333628f9174e8db53674787e281e7e0d7 | 19:52 |
sean-k-mooney | mriedem: oh i was not suggesting we hold it up just is this something ye want to review for a queens backport e.g. i should fix it right away or can be reviewd for back port later e.g. ill try and fix before ptg | 19:52 |
*** imacdonn has joined #openstack-nova | 19:55 | |
mriedem | ameeda: git fetch https://git.openstack.org/openstack/nova refs/changes/71/540871/1 && git cherry-pick -x FETCH_HEAD | 19:55 |
mriedem | stvnoyes: i think i what might be an issue with boot from volume with multiple instances and the new cinder volume attach flow... | 19:56 |
sean-k-mooney | mriedem: i have 3 patches i need to rebase for rocky later this week so ill add it to the list if it wont effect future backport or ill try to make time to fix it specificlly tomrow if it more urgent. anyway got to run enjoy your evening. | 19:57 |
*** edmondsw_ has joined #openstack-nova | 19:57 | |
mriedem | stvnoyes: we call this for each instance we create in a multi-create request https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1253 | 19:59 |
mriedem | and _validate_bdm will loop over the volumes passed in, and if everything is new enough, create an attachment per volume and store that on the bdm record | 19:59 |
*** edmondsw has quit IRC | 20:00 | |
mriedem | we then make a copy of that list per instance https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1267 | 20:00 |
mriedem | i'm wondering if we're overwriting or losing any of the bdm.attachment_id records when creating multiple instances | 20:00 |
mriedem | maybe it's ok | 20:00 |
mriedem | might be a decent test though, create >1 instances in the same request using the same volume and make sure each instance has unique attachments to that volume | 20:01 |
mriedem | i suppose if you aren't using a multiattach volume, the 2nd instance in the loop would fail to create an attachment to the volume because the volume's attach_status will be 'attaching' | 20:02 |
ameeda | mriedem: now the commit message have those 2 lines | 20:04 |
ameeda | (cherry picked from commit eba33a0e2d8aace6fbc889026fb80dc2c208bc8d) (cherry picked from commit c0aca1d333628f9174e8db53674787e281e7e0d7) | 20:04 |
*** yangyapeng has joined #openstack-nova | 20:04 | |
mriedem | ameeda: those should go on separate lines | 20:05 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: debug nova-api service version check during boot from volume https://review.openstack.org/540992 | 20:05 |
mriedem | but that's ine | 20:05 |
mriedem | fine | 20:05 |
*** itlinux has quit IRC | 20:06 | |
mriedem | dansmith: we haven't been doing proper service version checking in nova-api since pike https://review.openstack.org/#/c/539755/ - should be an easy +W | 20:08 |
ameeda | mriedem: yes they are , so I can do git review now ? | 20:08 |
mriedem | ameeda: yes | 20:08 |
ameeda | mriedem: Thank you !! | 20:08 |
*** yangyapeng has quit IRC | 20:08 | |
openstackgerrit | Ameed Ashour proposed openstack/nova stable/ocata: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/540274 | 20:09 |
ameeda | mriedem: could you please give me +2 here https://review.openstack.org/#/c/528385/ :$ | 20:09 |
*** slaweq has joined #openstack-nova | 20:10 | |
melwitt | jaypipes: replied | 20:10 |
*** acormier has quit IRC | 20:11 | |
*** slaweq_ has joined #openstack-nova | 20:11 | |
*** acormier has joined #openstack-nova | 20:11 | |
mriedem | ameeda: your ocata backport is wrong | 20:12 |
ameeda | I see | 20:14 |
*** slaweq_ has quit IRC | 20:16 | |
*** slaweq has quit IRC | 20:16 | |
*** acormier has quit IRC | 20:16 | |
*** mgoddard_ has joined #openstack-nova | 20:19 | |
*** slaweq has joined #openstack-nova | 20:21 | |
openstackgerrit | Ameed Ashour proposed openstack/nova stable/ocata: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/540274 | 20:23 |
*** itlinux has joined #openstack-nova | 20:23 | |
ameeda | mriedem: I don't know if I did the correct thing, please check it ^ | 20:27 |
mriedem | ameeda: yes that's correct, thanks | 20:28 |
*** dave-mcc_ has quit IRC | 20:29 | |
ameeda | Thank you !! | 20:29 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a nova-caching-scheduler job to the experimental queue https://review.openstack.org/539260 | 20:30 |
ameeda | mriedem: please don't forget to review this https://review.openstack.org/#/c/528385/ | 20:32 |
*** cdent has quit IRC | 20:32 | |
*** mgoddard_ has quit IRC | 20:35 | |
*** READ10 has quit IRC | 20:35 | |
*** acormier has joined #openstack-nova | 20:36 | |
*** acormier has quit IRC | 20:40 | |
*** acormier has joined #openstack-nova | 20:40 | |
*** acormier_ has joined #openstack-nova | 20:43 | |
*** acormier has quit IRC | 20:45 | |
*** acormier_ has quit IRC | 20:45 | |
*** slaweq_ has joined #openstack-nova | 20:48 | |
*** vladikr has quit IRC | 20:51 | |
*** slaweq_ has quit IRC | 20:53 | |
*** yangyapeng has joined #openstack-nova | 20:54 | |
mriedem | efried: i think this might have broken the nova-next job but i'm not sure how https://github.com/openstack-dev/devstack/commit/49144e627120f9ec1cf72dfc84e02b9b81e669b6 | 20:56 |
mriedem | https://bugs.launchpad.net/nova/+bug/1747511 | 20:56 |
openstack | Launchpad bug 1747511 in OpenStack Compute (nova) "legacy-tempest-dsvm-neutron-nova-next-full fails with "Can not find requested image" since 1/31" [Undecided,New] | 20:56 |
*** acormier has joined #openstack-nova | 20:57 | |
mriedem | the only thing unique about the nova-next job at this point is it sets NOVA_USE_SERVICE_TOKEN=True in devstack | 20:57 |
*** yangyapeng has quit IRC | 20:58 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Bump compute RPC API to version 5.0 https://review.openstack.org/541005 | 20:59 |
dansmith | gawd | 20:59 |
mriedem | ufta | 21:00 |
*** vladikr has joined #openstack-nova | 21:02 | |
*** moshele has joined #openstack-nova | 21:02 | |
*** acormier has quit IRC | 21:06 | |
*** acormier has joined #openstack-nova | 21:06 | |
efried | mriedem: Is the bustage consistent? | 21:06 |
*** traskat has quit IRC | 21:07 | |
mriedem | efried: yeah, 100% fail | 21:07 |
*** chyka has quit IRC | 21:07 | |
efried | mriedem: I can propose reversion of the above, then a blank nova patch that Depends-On it, and see if that at least makes it pass. Shall I? | 21:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: debug nova-next image not found failures https://review.openstack.org/541008 | 21:08 |
mriedem | i think we could also just ignore the service_user config in nova as a test | 21:08 |
efried | I have to jet for a doc appt right now, will catch up with you in an hour or so. | 21:08 |
*** moshele has quit IRC | 21:09 | |
*** acormier has quit IRC | 21:11 | |
jaypipes | thx melwitt | 21:12 |
openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Add release note for Aggregate[Core|Ram|Disk]Filter change https://review.openstack.org/541013 | 21:15 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: debug nova-next image not found failures https://review.openstack.org/541008 | 21:16 |
*** acormier has joined #openstack-nova | 21:17 | |
*** acormier_ has joined #openstack-nova | 21:20 | |
*** dave-mccowan has joined #openstack-nova | 21:21 | |
*** acormier has quit IRC | 21:21 | |
*** acormier_ has quit IRC | 21:25 | |
*** rcernin has joined #openstack-nova | 21:26 | |
*** tbachman has joined #openstack-nova | 21:29 | |
openstackgerrit | melanie witt proposed openstack/nova master: Add release note for Aggregate[Core|Ram|Disk]Filter change https://review.openstack.org/541018 | 21:30 |
*** chyka has joined #openstack-nova | 21:37 | |
*** yangyapeng has joined #openstack-nova | 21:44 | |
*** itlinux has quit IRC | 21:45 | |
mriedem | maciejjozefczyk: not sure why you'd +1 this :) https://review.openstack.org/#/c/540992/ | 21:48 |
*** yangyapeng has quit IRC | 21:48 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: debug nova-api service version check during boot from volume https://review.openstack.org/540992 | 21:50 |
*** burt has quit IRC | 21:52 | |
*** moshele has joined #openstack-nova | 21:54 | |
*** itlinux has joined #openstack-nova | 21:54 | |
openstackgerrit | Matthew Edmonds proposed openstack/nova master: Clarify CONF.scheduler.max_attempts https://review.openstack.org/541031 | 21:56 |
edmondsw_ | edleafe noticed this after reading your blog ^ | 21:56 |
*** edmondsw_ is now known as edmondsw | 21:57 | |
edmondsw | mriedem do we have a more general term that encompasses create, resize, and migrate if "build" doesn't do that? | 21:59 |
mriedem | scheduling? | 22:02 |
mriedem | like it said | 22:02 |
edmondsw | we can't say "schedule", because we only talk to the scheduler once. | 22:02 |
melwitt | it's still scheduling though, isn't it? trying alternate hosts *shrug* | 22:03 |
edmondsw | I guess | 22:03 |
edmondsw | at least the "for a chosen host" bit from the old help text is wrong, right? | 22:03 |
edleafe | mriedem: I think the problematic wording is "for a chosen host". But that was wrong before, too. | 22:04 |
edleafe | jinx | 22:04 |
edmondsw | edleafe yeah that is what first caught my eye | 22:04 |
mriedem | yes 'for a chosen host' is wrong | 22:04 |
edmondsw | I can change it back to scheduling and correct that part | 22:04 |
edleafe | "for a given VM build/move operation" | 22:05 |
edmondsw | I like that... | 22:06 |
mriedem | can we somehow make this about ironic and say, 'for a given bm movement' | 22:06 |
*** moshele has quit IRC | 22:06 | |
edleafe | nah, that sounds kinda crappy | 22:06 |
edmondsw | lol | 22:06 |
openstackgerrit | Matthew Edmonds proposed openstack/nova master: Clarify CONF.scheduler.max_attempts https://review.openstack.org/541031 | 22:07 |
efried | mriedem: I have returned. Any update on the glance image snafu? | 22:08 |
mriedem | efried: debug patch is still running | 22:08 |
jroll | edleafe: "for a given VM b/m operation" :P | 22:08 |
*** acormier has joined #openstack-nova | 22:09 | |
mriedem | i can't wait to be in the vm bm room for 2 days in dublin | 22:09 |
mriedem | talking about...keystone | 22:10 |
mriedem | i came for the bm, stayed for the keystone | 22:10 |
*** dave-mccowan has quit IRC | 22:10 | |
openstackgerrit | Matthew Edmonds proposed openstack/nova master: Clarify CONF.scheduler.max_attempts https://review.openstack.org/541031 | 22:10 |
edleafe | I'm SO glad I made it out of third grade | 22:11 |
openstackgerrit | Dan Smith proposed openstack/nova master: Bump compute RPC API to version 5.0 https://review.openstack.org/541005 | 22:11 |
openstackgerrit | Dan Smith proposed openstack/nova master: Compute RPC client bump to 5.0 https://review.openstack.org/541035 | 22:11 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Query all cells for service version in _validate_bdm https://review.openstack.org/541036 | 22:12 |
*** slaweq_ has joined #openstack-nova | 22:12 | |
mriedem | efried: results http://logs.openstack.org/08/541008/2/check/legacy-tempest-dsvm-neutron-nova-next-full/f24c9d2/logs/screen-n-api.txt.gz?level=TRACE | 22:13 |
*** acormier has quit IRC | 22:14 | |
efried | mriedem: Looks to me like neither of your glance.py messages hit. | 22:15 |
efried | which is... weird. | 22:15 |
openstackgerrit | Matthew Edmonds proposed openstack/nova master: Clarify CONF.scheduler.max_attempts https://review.openstack.org/541031 | 22:15 |
*** slaweq_ has quit IRC | 22:16 | |
mriedem | http://logs.openstack.org/08/541008/2/check/legacy-tempest-dsvm-neutron-nova-next-full/f24c9d2/logs/screen-n-api.txt.gz#_Feb_05_21_46_51_455934 | 22:16 |
mriedem | Using glance API servers: [u'http://158.69.92.100/image/v2/'] | 22:16 |
efried | mriedem: http://logs.openstack.org/08/541008/2/check/legacy-tempest-dsvm-neutron-nova-next-full/f24c9d2/logs/screen-g-api.txt.gz#_Feb_05_21_46_51_651885 | 22:17 |
mriedem | efried: maybe the image exists but the context doesn't have auth to GET it? | 22:17 |
efried | mriedem: This is https://bugs.launchpad.net/python-glanceclient/+bug/1707995 | 22:18 |
openstack | Launchpad bug 1707995 in Glance Client "Can't use versioned endpoints" [Undecided,New] | 22:18 |
mriedem | shazbot! | 22:18 |
efried | Though I thought we weren't subject to that with this set of changes. | 22:18 |
mriedem | not sure why the other jobs wouldn't be failing though | 22:18 |
*** vladikr has quit IRC | 22:19 | |
efried | Also checked this through the CI six ways from sunday before we merged it. | 22:19 |
mriedem | devstack doesn't run nova-next | 22:19 |
efried | So some weird combo of environments - or possibly a recent "fix" in ksa? - is exposing this. | 22:19 |
mriedem | i can try to nuke the usage of the service_user in this debug patch and see if things start magically passing | 22:20 |
efried | mriedem: No, but I made DNM nova patches with Depends-On the devstack change, exactly to validate this. | 22:20 |
mriedem | nova-next is also non-voting | 22:20 |
mriedem | so might not have noticed | 22:20 |
efried | oh, so maybe it failed and I neve.. yeah | 22:20 |
jroll | so GET /servers has a "created_at" query parameter to filter by created date... but sounds like that's an "equal to" filter, is there a way to communicate that I want before or after there? | 22:21 |
efried | mriedem: Could also do as I suggested above, put up a reversion of the devstack patch with an empty nova patch that Depends-On it, see if adding back the explicit endpoint also fixes it. | 22:21 |
*** slaweq has quit IRC | 22:22 | |
mriedem | jroll: don't thin kso | 22:23 |
*** slaweq has joined #openstack-nova | 22:23 | |
jroll | mriedem: that is so useful, lol | 22:23 |
*** yangyapeng has joined #openstack-nova | 22:23 | |
mriedem | jroll: i don't even see a created_at filter for listing nodes https://developer.openstack.org/api-ref/baremetal/#id4 | 22:24 |
mriedem | so shut up! | 22:24 |
jroll | heh, I'm not sure which is worse | 22:24 |
*** slaweq has quit IRC | 22:25 | |
efried | created_at=<exact timestamp> ?? | 22:25 |
efried | Wheee | 22:25 |
jroll | :) | 22:25 |
openstackgerrit | Merged openstack/nova master: Doc: Nix os-traits link from POST resource_classes https://review.openstack.org/540559 | 22:25 |
*** slaweq has joined #openstack-nova | 22:25 | |
mriedem | jroll: add a created_before/created_after param | 22:26 |
openstackgerrit | Merged openstack/nova stable/pike: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/540871 | 22:26 |
mriedem | there is changes-since... | 22:26 |
jroll | mriedem: that doesn't help the guy that's trying to use it today :P | 22:26 |
jroll | thanks though | 22:26 |
mriedem | but that's updated_at>=changes-since | 22:26 |
*** rodolof has quit IRC | 22:26 | |
mriedem | let me hop in the old delorean real quick | 22:27 |
jroll | right | 22:27 |
jroll | ok great thanks | 22:27 |
* mriedem grabs some gel for the ride | 22:27 | |
*** pchavva has quit IRC | 22:27 | |
* jroll tells mriedem's boss to give him a gold star | 22:27 | |
mriedem | i have multiple bosses, so good luck | 22:27 |
openstackgerrit | melanie witt proposed openstack/nova master: Make InstanceGroup.get_hosts query all cells https://review.openstack.org/540258 | 22:27 |
*** acormier has joined #openstack-nova | 22:27 | |
*** yangyapeng has quit IRC | 22:28 | |
*** slaweq_ has joined #openstack-nova | 22:28 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: debug nova-api service version check during boot from volume https://review.openstack.org/540992 | 22:30 |
* mriedem goes to wrangle snow | 22:30 | |
*** inara has quit IRC | 22:31 | |
*** vladikr has joined #openstack-nova | 22:32 | |
efried | Anyone proposed (or planning to propose) a Vancouver presentation about what we've done / are doing in placement/scheduler-land? | 22:32 |
*** inara has joined #openstack-nova | 22:33 | |
*** slaweq_ has quit IRC | 22:33 | |
openstackgerrit | Merged openstack/nova master: Query all cells for service version in _validate_bdm https://review.openstack.org/539755 | 22:34 |
openstackgerrit | Merged openstack/nova stable/pike: Import user-data page from openstack-manuals https://review.openstack.org/515054 | 22:34 |
*** slaweq has quit IRC | 22:35 | |
*** jaypipes_ has joined #openstack-nova | 22:35 | |
*** jaypipes has quit IRC | 22:35 | |
efried | jaypipes_: Any plans for a Vancouver presentation about what we've done / are doing in placement/scheduler-land? | 22:36 |
*** jdurgin has joined #openstack-nova | 22:41 | |
*** itlinux has quit IRC | 22:48 | |
edleafe | efried: you should do one. They're probably tired of seeing Jay's ugly mug | 22:58 |
efried | They need a new ugly mug to stare at, is what you're sayin. | 22:59 |
edleafe | zactly | 23:00 |
efried | I haven't done this before; how much flexibility is there to edit the proposal later, add presenters, change the abstract, etc.? | 23:01 |
edmondsw | efried I think you can change presenters later if need be. Not sure about the abstract. They probably don't want that changing too much | 23:05 |
*** edmondsw has quit IRC | 23:07 | |
*** edmondsw has joined #openstack-nova | 23:08 | |
*** edmondsw has quit IRC | 23:12 | |
*** yangyapeng has joined #openstack-nova | 23:13 | |
*** liverpooler has quit IRC | 23:13 | |
*** yamahata has joined #openstack-nova | 23:16 | |
*** yangyapeng has quit IRC | 23:17 | |
*** threestrands has joined #openstack-nova | 23:19 | |
*** threestrands has quit IRC | 23:19 | |
*** threestrands has joined #openstack-nova | 23:19 | |
*** mlavalle has quit IRC | 23:23 | |
*** yamahata has quit IRC | 23:26 | |
*** takashin has joined #openstack-nova | 23:30 | |
efried | edleafe: Can you see this? https://www.openstack.org/summit/vancouver-2018/call-for-presentations/manage/20813/summary | 23:34 |
efried | I have no way of NOT going to the edit page, so I don't know if this is visible to people who aren't logged in as me. | 23:35 |
edleafe | efried: See my comment? | 23:36 |
efried | edleafe: Where? | 23:36 |
edleafe | or do I have to save it first? | 23:36 |
edleafe | yeah, not an etherpad :) | 23:36 |
efried | I'm not even sure how I would view it. | 23:36 |
edleafe | ok, in the abstaract | 23:36 |
edleafe | abstract, even | 23:36 |
efried | oh, jeez, so like anyone can edit it? | 23:37 |
jroll | looks like it | 23:37 |
edleafe | I guess so, if they know the URL | 23:37 |
efried | And now everybody does, since I put it in IRC. | 23:37 |
* jroll enumerates the id | 23:37 | |
*** masahisa has joined #openstack-nova | 23:37 | |
efried | But I imagine your openID is attached to your edits. | 23:38 |
*** slaweq has joined #openstack-nova | 23:38 | |
edleafe | I removed my silly comment | 23:38 |
efried | Okay. But seem reasonable in terms of scope, taggage, etc.? | 23:38 |
jroll | efried: assuming edits are tracked | 23:38 |
efried | ye gods, I hope they are. | 23:38 |
efried | Otherwise anyone could register for an openID and cause havoc | 23:39 |
* jroll wouldn't bet on it | 23:39 | |
edleafe | efried: it's afree-for-all!! | 23:39 |
jroll | fwiw, I'm poking to find out where to report the issue | 23:39 |
* edleafe is about to duck out for the day | 23:40 | |
*** acormier has quit IRC | 23:40 | |
*** yamahata has joined #openstack-nova | 23:42 | |
*** slaweq has quit IRC | 23:42 | |
*** yamahata has quit IRC | 23:43 | |
*** yamahata has joined #openstack-nova | 23:43 | |
openstackgerrit | Merged openstack/nova stable/ocata: Don't wait for vif plug events during _hard_reboot https://review.openstack.org/540680 | 23:45 |
openstackgerrit | Eric Fried proposed openstack/nova master: Test case: new standard resource class unusable https://review.openstack.org/539732 | 23:47 |
openstackgerrit | Eric Fried proposed openstack/nova master: Ensure resource classes correctly https://review.openstack.org/539738 | 23:47 |
openstackgerrit | Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648 | 23:47 |
openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | 23:47 |
openstackgerrit | Eric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 23:47 |
openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | 23:47 |
openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | 23:47 |
openstackgerrit | Eric Fried proposed openstack/nova master: Make generation optional in ProviderTree https://review.openstack.org/539324 | 23:47 |
efried | jaypipes_ edleafe ^ tweaked per y'all's comments. | 23:48 |
efried | Aaaand I'm out. See y'all tomorrow. | 23:49 |
openstackgerrit | Merged openstack/nova stable/ocata: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/540274 | 23:51 |
*** hongbin has quit IRC | 23:53 | |
jaypipes_ | efried: cool, thx | 23:57 |
*** lei-zh has joined #openstack-nova | 23:59 | |
*** vladikr has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!