Monday, 2018-02-05

*** acormier has quit IRC00:03
*** tetsuro_ has joined #openstack-nova00:04
*** amodi has joined #openstack-nova00:11
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in get  https://review.openstack.org/53129900:11
*** yangyape_ has quit IRC00:33
*** dave-mcc_ has joined #openstack-nova00:33
*** armax has joined #openstack-nova00:34
*** hiro-kobayashi has joined #openstack-nova00:34
*** dave-mccowan has quit IRC00:35
*** s1061123_ has joined #openstack-nova00:35
*** s1061123 has quit IRC00:35
*** hongbin has joined #openstack-nova00:38
*** threestrands has joined #openstack-nova00:43
*** hiro-kobayashi has quit IRC00:44
*** hongbin has quit IRC00:47
*** hoangcx has joined #openstack-nova00:48
*** edmondsw has joined #openstack-nova00:52
*** edmondsw has quit IRC00:56
*** zhaochao has joined #openstack-nova01:04
*** hshiina has joined #openstack-nova01:11
*** phuongnh has joined #openstack-nova01:12
*** acormier has joined #openstack-nova01:14
*** acormier has joined #openstack-nova01:15
openstackgerritHironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue  https://review.openstack.org/41648701:17
*** yassine has joined #openstack-nova01:19
*** yassine is now known as Guest5331101:19
*** yangyapeng has joined #openstack-nova01:20
*** hongbin has joined #openstack-nova01:22
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform aggregate.update_metadata notification  https://review.openstack.org/46062501:24
*** acormier has quit IRC01:30
*** zhurong has joined #openstack-nova01:30
*** acormier has joined #openstack-nova01:30
*** claudiub|2 has quit IRC01:37
*** namnh has joined #openstack-nova01:44
*** moshele has joined #openstack-nova01:45
*** takashin has left #openstack-nova01:51
*** xinliang has quit IRC01:56
*** mtreinish has quit IRC01:58
*** mtreinish has joined #openstack-nova01:59
*** threestrands has quit IRC02:01
*** moshele has quit IRC02:05
openstackgerritZhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware  https://review.openstack.org/50748602:05
*** itlinux has joined #openstack-nova02:06
*** xinliang has joined #openstack-nova02:09
*** lei-zh has joined #openstack-nova02:11
*** threestrands has joined #openstack-nova02:16
*** threestrands has quit IRC02:16
*** threestrands has joined #openstack-nova02:16
openstackgerritSpencer Yu proposed openstack/nova master: project name should be removed from zuul configuration  https://review.openstack.org/54069502:20
*** Tom-Tom has joined #openstack-nova02:21
*** liuzz has joined #openstack-nova02:22
*** itlinux has quit IRC02:37
*** edmondsw has joined #openstack-nova02:40
*** jmlowe has quit IRC02:43
*** zhurong has quit IRC02:43
*** edmondsw has quit IRC02:45
*** jmlowe has joined #openstack-nova02:45
*** yamamoto has joined #openstack-nova02:47
*** bhujay has joined #openstack-nova03:03
*** lyan has joined #openstack-nova03:04
*** annp has joined #openstack-nova03:05
*** bhujay has quit IRC03:18
*** zhenguo has joined #openstack-nova03:21
*** acormier has quit IRC03:23
*** dave-mcc_ has quit IRC03:32
*** sree has joined #openstack-nova03:34
*** hshiina is now known as hshiina|afk03:35
*** lyan has quit IRC03:35
*** hshiina|afk has quit IRC03:35
*** gcb has joined #openstack-nova03:37
*** amodi has quit IRC03:37
*** lyan has joined #openstack-nova03:39
*** lyan has quit IRC03:47
*** itlinux has joined #openstack-nova03:48
*** Tom-Tom has quit IRC03:49
*** Tom-Tom has joined #openstack-nova03:50
*** gongysh has joined #openstack-nova03:54
*** Tom-Tom has quit IRC03:55
*** psachin has joined #openstack-nova04:06
*** threestrands has quit IRC04:07
*** udesale has joined #openstack-nova04:08
*** itlinux has quit IRC04:12
*** gongysh has quit IRC04:13
*** vladikr has joined #openstack-nova04:13
*** abhishekk has joined #openstack-nova04:13
openstackgerritChiawei Xie proposed openstack/nova stable/ocata: Modify nova-manage db migrations base message  https://review.openstack.org/54070904:16
*** vladikr has quit IRC04:18
*** threestrands has joined #openstack-nova04:19
*** threestrands has quit IRC04:19
*** threestrands has joined #openstack-nova04:19
*** vladikr has joined #openstack-nova04:19
*** edmondsw has joined #openstack-nova04:28
*** Spazmotic is now known as Spaz-Korea04:32
Spaz-KoreaFlight in 5 hours. almost time :)04:32
Spaz-KoreaSee you all in about 2 weeks04:32
*** edmondsw has quit IRC04:33
*** lei-zh has quit IRC04:33
*** gongysh has joined #openstack-nova04:33
*** bhujay has joined #openstack-nova04:36
*** threestrands has quit IRC04:36
Spaz-KoreaIf you need me for anything drop me a mail at bhkaminski (at) gmail.com04:36
*** threestrands has joined #openstack-nova04:37
*** gongysh has quit IRC04:38
*** vladikr has quit IRC04:51
*** vladikr has joined #openstack-nova04:54
*** threestrands has quit IRC05:07
*** claudiub|2 has joined #openstack-nova05:20
*** lei-zh has joined #openstack-nova05:25
*** ratailor has joined #openstack-nova05:29
*** zhenguo has quit IRC05:31
*** Tom-Tom has joined #openstack-nova05:32
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver  https://review.openstack.org/52338705:33
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765805:33
openstackgerritjichenjc proposed openstack/nova master: z/VM driver: add snapshot function  https://review.openstack.org/53424005:33
*** trinaths has joined #openstack-nova05:33
*** zhurong has joined #openstack-nova05:38
*** jaosorior has joined #openstack-nova05:39
*** hongbin has quit IRC05:40
openstackgerritjichenjc proposed openstack/nova master: fix race condition of instance host  https://review.openstack.org/49445805:41
*** jaypipes_ has joined #openstack-nova05:42
*** threestrands has joined #openstack-nova05:42
*** leakypipes has quit IRC05:44
openstackgerritjichenjc proposed openstack/nova master: mv generate_glance_url to get_image_endpoint_url  https://review.openstack.org/51140005:45
*** links has joined #openstack-nova05:46
*** lajoskatona has joined #openstack-nova05:47
*** threestrands has quit IRC05:47
*** threestrands has joined #openstack-nova05:47
*** links has quit IRC05:49
*** mdnadeem has joined #openstack-nova05:50
*** links has joined #openstack-nova05:51
*** takashin has joined #openstack-nova05:53
*** links has quit IRC05:59
*** links has joined #openstack-nova06:00
*** jmlowe has quit IRC06:04
*** threestrands_ has joined #openstack-nova06:04
*** threestrands_ has quit IRC06:04
*** threestrands_ has joined #openstack-nova06:04
*** jmlowe has joined #openstack-nova06:05
*** threestrands has quit IRC06:06
*** links has quit IRC06:14
*** links has joined #openstack-nova06:16
*** edmondsw has joined #openstack-nova06:17
*** jmlowe has quit IRC06:18
*** jmlowe has joined #openstack-nova06:20
*** ameeda has joined #openstack-nova06:21
*** edmondsw has quit IRC06:21
*** acormier has joined #openstack-nova06:25
openstackgerritChen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats  https://review.openstack.org/52683306:27
*** acormier has quit IRC06:29
*** tuanla____ has joined #openstack-nova06:29
*** sridharg has joined #openstack-nova06:30
*** jmlowe has quit IRC06:32
*** jmlowe has joined #openstack-nova06:33
*** gongysh has joined #openstack-nova06:44
*** jmlowe has quit IRC06:47
*** moshele has joined #openstack-nova06:48
*** jmlowe has joined #openstack-nova06:48
*** jaosorior has quit IRC06:50
*** pcaruana has joined #openstack-nova07:04
*** jmlowe has quit IRC07:04
*** josecastroleon has joined #openstack-nova07:06
*** jmlowe has joined #openstack-nova07:06
*** jaosorior has joined #openstack-nova07:12
*** alexchadin has joined #openstack-nova07:12
*** kholkina has joined #openstack-nova07:16
*** _ioni is now known as ioni07:17
*** ioni has left #openstack-nova07:18
*** ioni has joined #openstack-nova07:18
*** rcernin has quit IRC07:18
*** xinliang has quit IRC07:18
*** lpetrut has joined #openstack-nova07:19
*** maciejjozefczyk has quit IRC07:24
*** vladikr has quit IRC07:24
*** vladikr has joined #openstack-nova07:25
*** vladikr has quit IRC07:33
openstackgerritjichenjc proposed openstack/nova-specs master: add z/VM Driver in Rocky  https://review.openstack.org/53618607:37
*** AlexeyAbashkin has joined #openstack-nova07:41
*** alexchadin has quit IRC07:42
*** alexchadin has joined #openstack-nova07:43
*** vladikr has joined #openstack-nova07:45
*** AlexeyAbashkin has quit IRC07:45
*** AlexeyAbashkin has joined #openstack-nova07:50
*** andreas_s has joined #openstack-nova07:54
*** vladikr has quit IRC07:57
*** ragiman has joined #openstack-nova07:58
*** jaosorior has quit IRC08:00
*** moshele has quit IRC08:00
*** takashin has left #openstack-nova08:02
*** moshele has joined #openstack-nova08:02
*** edmondsw has joined #openstack-nova08:05
*** slaweq has joined #openstack-nova08:05
*** maciejjozefczyk has joined #openstack-nova08:06
*** alexchadin has quit IRC08:08
*** damien_r has joined #openstack-nova08:09
*** giblet is now known as gibi08:09
*** edmondsw has quit IRC08:09
*** andreas_s has quit IRC08:10
*** maciejjozefczyk has quit IRC08:11
*** vladikr has joined #openstack-nova08:12
gibiedleafe: 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 pass08:14
gibiedleafe: this of course means that we can only merge the patch after the ovo fix is merged and the ovo version is bumped in nova08:14
*** andreas_s has joined #openstack-nova08:15
*** hoonetorg has quit IRC08:15
*** ralonsoh has joined #openstack-nova08:16
*** ssurana has joined #openstack-nova08:17
*** ssurana has quit IRC08:17
*** naichuans has joined #openstack-nova08:18
ameedagibi: good morning :)08:19
ameedacan you please review this for me ? https://review.openstack.org/#/c/528385/08:19
*** masahisa has quit IRC08:20
*** masahisa has joined #openstack-nova08:22
*** maciejjozefczyk has joined #openstack-nova08:24
*** andreas_s has quit IRC08:25
*** sahid has joined #openstack-nova08:25
*** ccamacho has joined #openstack-nova08:27
*** hoonetorg has joined #openstack-nova08:28
*** tesseract has joined #openstack-nova08:28
*** andreas_s has joined #openstack-nova08:30
*** andreas_s has quit IRC08:30
*** andreas_s has joined #openstack-nova08:30
openstackgerritjichenjc proposed openstack/nova master: fix race condition of instance host  https://review.openstack.org/49445808:35
*** ttsiouts has joined #openstack-nova08:36
*** AJaeger has joined #openstack-nova08:37
AJaegernova 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 amoralej08:38
AJaegerstephenfin, gibi, can you help, please? ^08:39
openstackgerritjichenjc proposed openstack/nova master: Remove quota reserve/commit/rollback  https://review.openstack.org/52147008:41
*** amorin has joined #openstack-nova08:43
amorinhello all08:43
gibiameeda: will try to look at it today08:43
*** Spaz-Korea has quit IRC08:43
gibiAJaeger: looking08:43
ameedagibi: thank you very much, please don't forget08:44
*** jpena|off is now known as jpena08:45
gibiameeda: I tend not to forget these. It is more like I run out of time08:46
ameedagibi: Thank you !!08:47
AJaegerthanks, gibi !08:49
*** AJaeger has left #openstack-nova08:50
*** ircuser-1 has joined #openstack-nova08:50
amorinI would like to know if there is any way to create a neutron client from nova, which could use the retries parameter of neutron08:52
amorinon nova side: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L13708:52
amorinneutron side: https://github.com/openstack/python-neutronclient/blob/master/neutronclient/v2_0/client.py#L22008:52
amorinseems that nova is not using retries at all08:52
*** threestrands_ has quit IRC08:54
*** tssurya has joined #openstack-nova08:56
*** jmlowe has quit IRC08:59
*** vladikr has quit IRC09:00
*** jmlowe has joined #openstack-nova09:01
bauzasgood morning folksq09:03
gibibauzas: good morning09:05
amorinmorning09:05
*** moshele has quit IRC09:05
*** moshele has joined #openstack-nova09:07
openstackgerritjichenjc proposed openstack/nova master: deprecate fping_path config option  https://review.openstack.org/52660209:17
*** sridharg has quit IRC09:18
*** derekh has joined #openstack-nova09:20
*** sridharg has joined #openstack-nova09:21
*** Dinesh_Bhor has joined #openstack-nova09:24
*** mgoddard_ has joined #openstack-nova09:30
*** tianhui has joined #openstack-nova09:30
*** Dinesh_Bhor has quit IRC09:32
*** alexchadin has joined #openstack-nova09:35
*** alexchadin has quit IRC09:39
*** alexchadin has joined #openstack-nova09:40
openstackgerritZhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware  https://review.openstack.org/50748609:43
openstackgerritZhenyu Zheng proposed openstack/nova master: Trivial fix a missleading comment  https://review.openstack.org/54079609:48
*** lei-zh has quit IRC09:57
openstackgerritMerged openstack/nova master: Zuul: Remove project name  https://review.openstack.org/54032610:03
openstackgerritMerged openstack/nova master: Fix SUSE Install Guide: Placement port  https://review.openstack.org/53816310:03
*** dtantsur|afk is now known as dtantsur10:05
*** sree has quit IRC10:05
*** sree has joined #openstack-nova10:06
gibithe gate is super fast now :)10:08
*** annp has quit IRC10:09
*** jmlowe has quit IRC10:09
*** jmlowe has joined #openstack-nova10:10
openstackgerritJianghua Wang proposed openstack/nova master: doc: update the GPU passthrough HPC feature entry  https://review.openstack.org/53963810:11
openstackgerritJianghua Wang proposed openstack/nova master: Provide support matrix and doc for VGPU  https://review.openstack.org/53926610:11
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: Provide support matrix and doc for VGPU  https://review.openstack.org/54080810:11
*** sree has quit IRC10:14
*** phuongnh has quit IRC10:20
*** stakeda has quit IRC10:26
*** mdnadeem has quit IRC10:28
*** mdnadeem has joined #openstack-nova10:28
*** Tom-Tom has quit IRC10:31
*** Tom-Tom has joined #openstack-nova10:32
*** namnh has quit IRC10:33
*** Tom-Tom has quit IRC10:36
*** jaosorior has joined #openstack-nova10:43
*** cdent has joined #openstack-nova10:45
ameedagibi: replied10:50
*** lucas-afk is now known as lucasagomes10:52
*** tuanla____ has quit IRC10:53
*** yangyapeng has quit IRC10:56
*** yangyapeng has joined #openstack-nova10:57
*** isssp has quit IRC10:57
*** ispp has joined #openstack-nova10:57
*** Guest53311 has quit IRC10:57
*** sambetts|afk is now known as sambetts11:00
*** sahid has quit IRC11:01
*** jmlowe has quit IRC11:02
*** jmlowe has joined #openstack-nova11:03
*** gszasz has joined #openstack-nova11:06
*** sahid has joined #openstack-nova11:07
*** sdague has quit IRC11:09
*** abalutoiu_ has joined #openstack-nova11:11
gibiameeda: OK, I will double check with mriedem about the double detach11:11
ameedagibi: thank you !!11:12
gibiameeda: until then, I will try to make the functional test work without the need for mocking the detach call11:12
ameedagibi: yes please , I try a lot with it with no lucks11:13
*** abalutoiu__ has quit IRC11:15
ameedagibi: 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
ameedagibi: also I need help here  https://review.openstack.org/#/c/526900/11:16
*** andreas_s has quit IRC11:16
ameedasorry for inconvenience, thanks for you time !!11:16
*** andreas_s has joined #openstack-nova11:17
*** gongysh has quit IRC11:17
*** sahid has quit IRC11:19
*** sahid has joined #openstack-nova11:19
*** jmlowe has quit IRC11:20
*** sahid has quit IRC11:20
*** sahid has joined #openstack-nova11:20
*** jmlowe has joined #openstack-nova11:25
openstackgerritMarcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console  https://review.openstack.org/53800311:25
hrwmorning11:26
*** andreas_s has quit IRC11:26
*** udesale has quit IRC11:30
*** andreas_s has joined #openstack-nova11:31
rabelhi. can we merge https://review.openstack.org/#/c/535871/ ?11:33
*** abhishekk has quit IRC11:38
openstackgerritZhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware  https://review.openstack.org/50748611:38
*** jmlowe has quit IRC11:39
*** jmlowe has joined #openstack-nova11:39
*** andreas_s has quit IRC11:40
*** edmondsw has joined #openstack-nova11:41
*** alexchadin has quit IRC11:44
*** claudiub has joined #openstack-nova11:44
*** alexchadin has joined #openstack-nova11:45
*** edmondsw has quit IRC11:45
*** andreas_s has joined #openstack-nova11:46
*** claudiub|2 has quit IRC11:47
*** stvnoyes has quit IRC11:49
*** acormier has joined #openstack-nova11:54
*** acormier has quit IRC11:55
*** sdague has joined #openstack-nova11:55
*** acormier has joined #openstack-nova11:55
*** andreas_s has quit IRC11:55
*** andreas_s has joined #openstack-nova11:56
*** stvnoyes has joined #openstack-nova11:57
*** gszasz has quit IRC11:57
*** moshele has quit IRC11:59
*** gszasz has joined #openstack-nova12:00
*** masahisa has quit IRC12:00
*** ratailor has quit IRC12:01
*** pcaruana has quit IRC12:02
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Add missing encryption_secret_uuid tests  https://review.openstack.org/54067912:03
lyarwoodops12:03
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Add missing encryption_secret_uuid tests  https://review.openstack.org/54067912:03
lyarwoodthere we go12:03
*** jmlowe has quit IRC12:04
*** andreas_s has quit IRC12:04
*** andreas_s has joined #openstack-nova12:04
*** jmlowe has joined #openstack-nova12:05
*** trinaths has quit IRC12:07
*** jmlowe has quit IRC12:15
*** yassine has joined #openstack-nova12:16
*** jaosorior has quit IRC12:16
*** gongysh has joined #openstack-nova12:17
*** yassine is now known as Guest4486012:17
*** dtantsur is now known as dtantsur|brb12:17
openstackgerritLee Yarwood proposed openstack/nova stable/pike: Unmap compute nodes when deleting host mapping  https://review.openstack.org/52756012:23
*** jpena is now known as jpena|lunch12:25
*** jmlowe has joined #openstack-nova12:26
ameedagibi: replied12:26
*** yangyapeng has quit IRC12:29
*** gcb has quit IRC12:30
*** acormier has quit IRC12:30
*** yangyapeng has joined #openstack-nova12:33
*** yangyapeng has quit IRC12:33
openstackgerritLajos Katona proposed openstack/nova master: WIP: ServerMovingTests with custom resources  https://review.openstack.org/49739912:34
*** sahid has quit IRC12:36
*** jmlowe has quit IRC12:36
*** sahid has joined #openstack-nova12:37
openstackgerritClaudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM  https://review.openstack.org/47894312:42
*** vladikr has joined #openstack-nova12:42
openstackgerritLee Yarwood proposed openstack/nova stable/pike: Do not attempt volume swap when guest is stopped/suspended  https://review.openstack.org/53373312:44
*** amoralej is now known as amoralej|lunch12:44
hrwhttps://review.openstack.org/#/c/532504/ - someone will give second +2 (and +w) to stable/pike backport for arm64?12:45
*** tssurya has quit IRC12:48
openstackgerritMerged openstack/nova stable/ocata: Fix false positive server group functional tests  https://review.openstack.org/53699512:48
*** tetsuro_ has left #openstack-nova12:51
*** psachin has quit IRC12:56
openstackgerritLee Yarwood proposed openstack/nova stable/pike: docs: Fix stable/pike config references  https://review.openstack.org/54084612:58
*** gongysh has quit IRC12:58
*** abalutoiu__ has joined #openstack-nova13:00
*** josecastroleon has quit IRC13:00
*** josecastroleon1 has joined #openstack-nova13:00
*** yamamoto has quit IRC13:01
*** gongysh has joined #openstack-nova13:02
*** alexchadin has quit IRC13:03
*** tssurya has joined #openstack-nova13:03
*** abalutoiu_ has quit IRC13:03
*** alexchadin has joined #openstack-nova13:04
*** jmlowe has joined #openstack-nova13:08
gibibauzas: hi, I have a quetion in https://review.openstack.org/#/c/539266/9/doc/source/admin/virtual-gpu.rst@13313:10
*** edmondsw has joined #openstack-nova13:13
*** amoralej|lunch is now known as amoralej13:16
bauzasgibi: replying13:18
bauzasgibi: replied ;)13:19
*** yamamoto has joined #openstack-nova13:19
gibibauzas: thanks. so what is this '*Error* state' the doc is refer to?13:21
*** sahid has quit IRC13:23
*** sahid has joined #openstack-nova13:23
*** gongysh has quit IRC13:25
bauzasgibi: http://paste.openstack.org/show/662486/13:25
bauzasHTH13:25
*** jpena|lunch is now known as jpena13:27
gibibauzas: ahh so the suspend instance action will have the error state13:28
bauzasexactly13:28
*** vladikr has quit IRC13:28
bauzashence not the VM state "ERROR"13:29
gibibauzas: thanks.13:29
bauzas(not capitalized")13:29
gibibauzas: I +Wd that patch13:30
bauzascool thanks13:30
gibithank you for creating such a good doc13:31
*** liverpooler has joined #openstack-nova13:31
bauzasthanks13:32
*** masahisa has joined #openstack-nova13:33
*** yangyapeng has joined #openstack-nova13:35
*** liverpooler has quit IRC13:36
*** edmondsw has quit IRC13:36
*** liverpooler has joined #openstack-nova13:36
*** edmondsw has joined #openstack-nova13:36
*** udesale has joined #openstack-nova13:37
*** jaosorior has joined #openstack-nova13:37
*** udesale has quit IRC13:37
*** artom has joined #openstack-nova13:37
*** udesale has joined #openstack-nova13:38
openstackgerritsahid proposed openstack/nova master: hardware: only take into account small pages  https://review.openstack.org/53216813:39
*** udesale has quit IRC13:39
*** masahisa has quit IRC13:39
*** yangyapeng has quit IRC13:39
*** udesale has joined #openstack-nova13:41
*** moshele has joined #openstack-nova13:41
*** vladikr has joined #openstack-nova13:41
*** edmondsw has quit IRC13:41
*** edmondsw has joined #openstack-nova13:44
*** pchavva has joined #openstack-nova13:48
*** edmondsw has quit IRC13:50
*** edmondsw has joined #openstack-nova13:51
*** edmondsw has quit IRC13:55
jaypipes_morning supernovas13:56
edleafeScheduler subteam meeting in 3 minutes in #openstack-meeting-alt13:57
*** lajoskatona has quit IRC13:58
*** takashin has joined #openstack-nova13:58
*** edmondsw has joined #openstack-nova13:59
*** acormier has joined #openstack-nova13:59
openstackgerritLee Yarwood proposed openstack/nova stable/pike: Removed unused argument from LibvirtDriver._connect_volume  https://review.openstack.org/54085913:59
openstackgerritLee Yarwood proposed openstack/nova stable/pike: Remove unused argument from LibvirtDriver._disconnect_volume  https://review.openstack.org/54086013:59
openstackgerritLee Yarwood proposed openstack/nova stable/pike: libvirt: Collocate encryptor and volume driver calls  https://review.openstack.org/54086113:59
edleafeScheduler subteam meeting running NOW in #openstack-meeting-alt14:00
*** edmondsw has quit IRC14:01
*** edmondsw has joined #openstack-nova14:01
*** acormier has quit IRC14:04
*** jaypipes_ is now known as jaypipes14:06
*** gszasz has quit IRC14:06
*** dtantsur|brb is now known as dtantsur14:07
*** dave-mccowan has joined #openstack-nova14:08
*** acormier has joined #openstack-nova14:10
*** sree has joined #openstack-nova14:13
*** acormier has quit IRC14:14
*** Tom-Tom has joined #openstack-nova14:14
bauzaswow, I feel the breeze of not having internal stuff to do14:16
*** sree has quit IRC14:17
openstackgerritMarcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console  https://review.openstack.org/53800314:18
openstackgerritMerged openstack/nova stable/pike: Fix the ocata config-reference URLs  https://review.openstack.org/51541014:22
openstackgerritMerged openstack/nova master: doc: update the GPU passthrough HPC feature entry  https://review.openstack.org/53963814:22
openstackgerritMerged openstack/nova master: Provide support matrix and doc for VGPU  https://review.openstack.org/53926614:23
ameedajaypipes_: could you please check this https://review.openstack.org/#/c/526900/ ?14:27
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: Remove 2 redundant methods  https://review.openstack.org/52832514:27
*** david-lyle has quit IRC14:30
*** lucasagomes is now known as lucas-hungry14:30
*** dklyle has quit IRC14:32
*** claudiub|2 has joined #openstack-nova14:34
*** abalutoiu_ has joined #openstack-nova14:34
gibibauzas: If you have some free time, then there is a bugfix that needs some review  https://review.openstack.org/#/c/525242/ :)14:36
openstackgerritLee Yarwood proposed openstack/nova stable/pike: add "--until-complete" option for nova-manage db archive_deleted_rows.  https://review.openstack.org/54087114:37
bauzasgibi: free time is not exactly what I call as my current workload, but I'll ;)14:37
*** claudiub has quit IRC14:37
*** abalutoiu__ has quit IRC14:37
*** hrw has quit IRC14:37
gibibauzas: Ohh then I was misled by your feeling the breeze comment above :)14:38
bauzasgibi: oh, your patch is crucial https://review.openstack.org/#/c/525242/14:39
*** dave-mcc_ has joined #openstack-nova14:39
*** mriedem has joined #openstack-nova14:39
mriedemlyarwood: i think it would be ok to just leave that as a separate refactor/test coverage change on top14:39
mriedemdon't really want to hold up https://review.openstack.org/#/c/539739/ since it's breaking cinder 3rd party CI14:40
*** moshele has quit IRC14:40
*** hrw has joined #openstack-nova14:40
lyarwoodmriedem: yup np14:40
*** gongysh has joined #openstack-nova14:41
*** dave-mccowan has quit IRC14:42
gibimriedem: 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 interface14:43
*** abalutoiu__ has joined #openstack-nova14:45
*** takashin has left #openstack-nova14:45
efriedcdent: 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-nova14:46
*** abalutoiu_ has quit IRC14:48
*** gongysh has quit IRC14:49
*** awaugama has joined #openstack-nova14:50
*** bhujay has quit IRC14:51
bauzasgibi: mriedem: I'm a bit concerned by a bugfix having RPC versions bumped but it's understandable given the upcall issue14:51
bauzascontext being https://review.openstack.org/#/c/525242/1414:51
gibibauzas: First I tried without the rpc bump but that solution would create yet another cell upcall14:51
*** esberglu has joined #openstack-nova14:52
bauzasyeah I know14:52
bauzasFWIW, I have a long standing blueprint for passing request specs to computes14:52
bauzasit's purely procedural tho14:52
bauzasI'm fine with approving that change if everything else is ok14:53
bauzasjust thinking out loud14:53
*** josecastroleon1 has quit IRC14:53
gibiyes, we need to agree on such a bump together. Especially as dansmith might plan to do a major rpc version bump before the release14:54
dansmithyeah, unfortunately, that reqspec cleanup would be really nice to get into place14:54
* bauzas seriously apologies for not having implemented yet the blueprint14:55
*** tssurya has quit IRC14:56
mriedemi got 2 patches in against it for queens!14:56
gibibauzas: if that helps with the current bugfix, I can offer some help with the request spec blueprint in Rocky14:56
mriedemwith great success14:56
efriedbauzas: What's a request spec, as distinct from allocation request data?14:57
efriedThat is, what does a request spec has that an allocation request doesn't?14:58
mriedemgibi: replied in https://review.openstack.org/#/c/528385/14:58
mriedemefried: all sorts of stuff,14:58
mriedeminstance groups, scheduler hints,14:58
mriedemimage id14:58
mriedemit's basically the instance14:58
mriedemminus host stuff (and BDMs for some reason)14:58
efriedokay, thanks.14:59
efriedcause jfyi, we are passing the allocation to the compute (https://review.openstack.org/#/c/511879/)15:00
bauzasefried: originally the request spec is what was passed to the scheduler for filtering purposes15:00
mriedemefried: in this case we need the scheduling hints15:00
efriedcoo15:00
bauzasefried: it became used for more than just the scheduler15:00
bauzasbecause we have the original intent of the user + some quirks15:01
bauzasthe 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 not15:01
gibimriedem: thanks the reply in https://review.openstack.org/#/c/528385/, I'm OK with the extra kwargs15:02
bauzasso, when you review a change using that RequestSpec object record, try to consider all fields as potentially mutable15:02
*** hongbin has joined #openstack-nova15:02
gibiameeda: I think you have a way forward now in https://review.openstack.org/#/c/528385/15:03
*** r-daneel has joined #openstack-nova15:03
openstackgerritPeter Hamilton proposed openstack/nova-specs master: Add support for certificate validation  https://review.openstack.org/54087915:04
*** acormier has joined #openstack-nova15:04
*** mchlumsky has joined #openstack-nova15:04
*** josecastroleon has joined #openstack-nova15:09
*** tssurya has joined #openstack-nova15:11
*** alexchadin has quit IRC15:14
lyarwoodmriedem: 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 lucasagomes15:22
*** tbachman has joined #openstack-nova15:24
mriedemgibi: comments in https://review.openstack.org/#/c/525242/15:25
gibimriedem: looking15:26
*** amodi has joined #openstack-nova15:27
cdentefried: 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-L22015:28
*** lpetrut has quit IRC15:28
openstackgerritStephen Finucane proposed openstack/nova master: doc: Don't use single backticks in man pages  https://review.openstack.org/54088715:29
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using openstackdoctheme's extlink extension  https://review.openstack.org/54088815:29
*** sshwarts has joined #openstack-nova15:29
*** armax has quit IRC15:32
mriedemgibi: and the functional test below15:33
*** armax has joined #openstack-nova15:33
mriedemdansmith: melwitt: easy quick cells v2 layout caveats update for stuff that is now fixed in queens https://review.openstack.org/#/c/540667/15:33
gibimriedem: thanks15:34
*** yangyapeng has joined #openstack-nova15:34
mriedembauzas: this looks nice https://docs.openstack.org/nova/latest/admin/virtual-gpu.html thanks for doing that15:34
*** READ10 has joined #openstack-nova15:37
*** yangyapeng has quit IRC15:38
efriedcdent: I knew what you meant; was just commenting that the sentence didn't grammar for me.15:40
cdentefried: because of the missing "of" or more than that?15:40
*** mlavalle has joined #openstack-nova15:40
efriedcdent: TBH, I couldn't even interpolate what was missing/wrong.  Let me go read it again...15:41
cdenti've just put the "of" in15:41
mriedemlyarwood: done15:41
lyarwoodmriedem: thanks15:42
*** david-lyle has joined #openstack-nova15:42
efriedcdent: Oh, yeah, that (or the addition of caffeine since first reading) makes it a lot clearer.15:42
efriedcdent: And when you get a chance, you said you could point me to where allocations were managing generations.15:42
mriedemlyarwood: 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/171401715:42
lyarwoodmriedem: sure, looking now15:43
cdentefried: 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 lbragstad15:43
*** slaweq has quit IRC15:43
*** slaweq has joined #openstack-nova15:43
*** slaweq has quit IRC15:43
efriedcdent: That's on the return from POST /allocations?15:43
mriedemlyarwood: i'm not sure about this https://review.openstack.org/#/c/540846/15:44
cdentthat 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-L208515:44
mriedemit's going to be super annoying if we have to update all doc backports to point at the branch they are on15:44
mriedemwe don't do that when we branch15:44
cdentefried: that's on the _start_ of PUT or POST15:44
cdenton the way in, not the way out15:44
openstackgerritMerged openstack/nova master: Fix invalid UUIDs in test_compute.py  https://review.openstack.org/53147915:44
cdentefried: so the generation is checked, but the client doesn't provide15:44
openstackgerritMerged openstack/nova stable/pike: Don't update RT in _allocate_network  https://review.openstack.org/51994215:44
openstackgerritMerged openstack/python-novaclient master: Zuul: Remove project name  https://review.openstack.org/54030415:44
cdentwhich seems ... useless?15:44
efriedcdent: Right.  What are we checking the generation *against*?15:45
efriedcdent: (BTW, in that code block, I still don't see where the generation is being checked, even internally)15:45
cdentefried: anything that changes from the start of the PUT to the time when _increment_provider_generation is called15:45
bauzasmriedem: thanks for pushing me the idea :-)15:46
efriedcdent: 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
cdentefried: yes, your bug report is a different concern15:46
cdentand no, we shouldn't be doing that in all single calls15:46
cdentit is pointless in allocations15:46
mriedemlyarwood: are you planning on backporting this entire series to ocata also https://review.openstack.org/#/c/540861/ ?15:47
cdentif the allocation fits, write it15:47
mriedemthat's a lot of refactor15:47
cdentefried: but we don't currently do that15:47
*** gcb has quit IRC15:47
lyarwoodmriedem: still working out if that's even possible15:47
efriedcdent: 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
lyarwoodmriedem: yeah, the refactor also fixes a load of connect/disonnect issues15:47
lyarwoodmriedem: really I should've landed a smaller bugfix ahead of this on master to backport15:48
mriedemhmmm,15:49
mriedemlike what?15:49
cdentefried: 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
mriedemlyarwood: why not just put a change on stable that says if you're trying to swap encrypted volumes, we fail?15:49
efriedcdent: 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
lyarwoodmriedem: yup that works and avoids this horrid backport, I'll do that now.15:50
mriedemlyarwood: 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 stable15:50
cdentefried: "can actually apply" shouldn't mean generation (when it comes to allocations) it should mean capacity/capability15:50
cdent(or if I'm not understanding you, you'll need to be more complete with your example)15:51
efriedcdent: 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-nova15:52
cdentwhich is why we 409 if that state has changed, in a way that changes capacity, by checking the capacity at allocation write-time15:52
efriedcdent: Yeah, capacity conflicts I'm cool with, put that aside.15:52
edleafecdent: 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 fail15:52
cdentif we want to rely on generation, then we are vastly decreasing the odds of an allocation being able to land15:53
edleafeIMO, that's a huge nova-ism for placement to base its behavior on15:53
efriedYes, we may want to consider making the generation handling optional for allocations.  But it should at least be possible.15:53
cdentefried: I still haven't heard why?15:53
*** chyka has joined #openstack-nova15:54
cdentwas " 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 IRC15:54
*** mdnadeem has quit IRC15:54
efriedcdent: 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
edleafecdent: 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 second15:55
efriedcdent: 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-nova15:56
efriedcdent: Which may be an acceptable limitation of the scheduler's weigher.  But it may not be.15:57
*** sapcc-bot has quit IRC15:57
efriedcdent: 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-nova15:57
edleafeefried: this all seems like a good argument for placement not being part of nova15:57
cdenta) 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-nova15:57
efriedcdent: That's precisely my point.  It should *not* be placement's responsibility.  But therefore it *needs* to be the caller's.15:58
efriedcdent: But with the API as it stands, we provide the caller no mechanism to enable it to execute that responsibility.15:58
dansmithmriedem: 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
efriedcdent: Generation would be one such mechanism.15:58
edleafeefried: so would something like Watcher, which can re-balance things at the cloud level15:59
cdentefried: 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
mriedemdansmith: are you asking b/c of gibi's functional test?15:59
mriedemor b/c you're bumping major versions?15:59
efriededleafe: After the allocation has been made and the instance has been scheduled?  Sure.  Messy, though.15:59
edleafeefried: IOW, detect a crowded host and performt the migration to uncrowd it15:59
edleafeefried: it15:59
edleafeit's a cloud15:59
*** chyka has quit IRC16:00
*** chyka has joined #openstack-nova16:01
efriededleafe: 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
edleafeefried: it *will* get it right the overwhelming majority of the time16:01
edleafeThe case we're describing is an edge of an edge case16:01
efriededleafe: Actually, I think it'll get it "wrong" (for common definitions of "wrong") quite often.16:02
*** sree has quit IRC16:02
efriededleafe: Only when lots of concurrent deploys are happening, of course.16:02
dansmithmriedem: bumping major versions16:02
efriededleafe: Or concurrent migrations (such as an "evacuate host" scenario).16:02
gibimriedem: I'm in the process to updat that patch not to depened on that flag16:03
cdentefried: 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
edleafeefried: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L411-L42216:03
cdentIf you're just going with "not loaded" and you get something weird, isn't that you're own fault?16:03
cdentsigh: your16:04
efriedcdent: I mean, we can document it that way, but it's not very satisfying to the consumer.16:04
efriedcdent: And I don't agree that you necessarily want anti-affinity.16:04
edleafeefried: and also https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L426-L43616:05
efriedIf 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 IRC16:05
edleafeThere is plenty of race avoidance in the code.16:05
*** mchlumsky has quit IRC16:05
cdentI dunno, I think your host should just go where it will get what it needs, unless you tell it something special.16:05
cdentAnd providing too many knobs is bad for everyone16:06
efriededleafe: 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 IRC16:07
mriedemdansmith: i'd have to take a look at that flag to see what the api does with it16:07
efriedcdent: Too many knobs, sure, but I thought a "least consumed" weigher was one of the basics.16:07
dansmithmriedem: okay I thought you did it which is why I was asking.. I'll add a note and come back to it16:07
openstackgerritMerged openstack/nova stable/pike: doc: Add user index page  https://review.openstack.org/54049416:07
openstackgerritMerged openstack/nova stable/pike: Migrate "launch instance" user guide docs  https://review.openstack.org/54049516:08
cdentefried: 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
mriedemdansmith: no i don't think so, sounds like a tdurakov thing16:09
dansmithmriedem: ack16:09
* cdent pours one out for tdurakov16:09
*** armax has joined #openstack-nova16:09
*** mchlumsky has joined #openstack-nova16:10
efriedcdent: I don't see what this has to do with allocation ratios.16:10
cdentif 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 one16:11
cdentif hosts are oversiubscribed then of course you're going to try to do tricks to get on an underutilized host16:11
*** burt has joined #openstack-nova16:12
efriedcdent: That makes a lot of assumptions about the possible reasons for a "least consumed" deployment strategy.16:12
cdentwhat other reason can you want other than needing to ensure you "get enough"?16:13
*** armax has quit IRC16:13
efriedcdent, 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-nova16:14
*** kholkina has quit IRC16:14
*** armax has left #openstack-nova16:14
*** slaweq has joined #openstack-nova16:14
* cdent schwews16:14
*** armax has joined #openstack-nova16:16
openstackgerritBalazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407  https://review.openstack.org/52609516:18
openstackbug 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-nova16:18
gibimriedem: ^^ fixed the functional test, now start fixing the bugfix patch16:18
*** slaweq has quit IRC16:18
*** pchavva has joined #openstack-nova16:20
*** abalutoiu_ has joined #openstack-nova16:23
*** sree has quit IRC16:23
*** ragiman has quit IRC16:24
mriedemdansmith: looks like gibi made onSharedStorage optional in evacuate16:26
*** abalutoiu__ has quit IRC16:26
dansmithmriedem: right, but it was required in earlier api versions16:26
openstackgerritJay Pipes proposed openstack/nova master: only increment disk address unit for scsi devices  https://review.openstack.org/53831016:26
mriedemdansmith: yeah, so it looks like we could kill it from compute...16:26
dansmithso I'm wondering if we have to keep the yes/no/unspecified support in the lower layers16:26
jaypipesmelwitt, 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
mriedemunless there is some scenario where operators actually need to rely on it b/c the auto-check using the driver is wrong,16:27
mriedembut that would seem like a bug16:27
dansmithmriedem: well, the point being the api used to advertise that it will check your work16:27
dansmithso if people used that, and we silently ignore it in compute now...16:27
mriedemit used it for some other stuff too, related to the adminPass16:28
*** udesale has quit IRC16:28
*** vladikr has quit IRC16:28
mriedemreading the spec https://specs.openstack.org/openstack/nova-specs/specs/mitaka/implemented/optional-shared-storage-flag-in-evacuate-api.html16:29
openstackgerritEric Fried proposed openstack/nova master: placement doc: Conflict caveat for DELETE APIs  https://review.openstack.org/54091916:30
efriedcdent, edleafe: ^ For the remainder of bug 174637316:30
openstackbug 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
efriedjaypipes also ^16:30
mriedemdansmith: fine to just leave it in for compat16:32
*** AlexeyAbashkin has quit IRC16:34
*** AlexeyAbashkin has joined #openstack-nova16:34
*** yamamoto has quit IRC16:36
*** andreas_s has quit IRC16:36
*** andreas_s has joined #openstack-nova16:37
*** links has quit IRC16:37
*** itlinux has joined #openstack-nova16:37
openstackgerritMatt Riedemann proposed openstack/nova master: Rename the 'recreate' param to rebuild_instance to 'evacuate'  https://review.openstack.org/50819016:39
openstackgerritMatt Riedemann proposed openstack/nova master: Rename the 'recreate' param in rebuild_instance to 'evacuate'  https://review.openstack.org/50819016:39
*** itlinux has quit IRC16:40
*** claudiub has joined #openstack-nova16:41
*** vladikr has joined #openstack-nova16:42
*** moshele has joined #openstack-nova16:42
*** claudiub|2 has quit IRC16:45
*** zhaochao has quit IRC16:45
*** andreas_s has quit IRC16:46
*** moshele has quit IRC16:47
hrwstephenfin: what is docstring?16:47
openstackgerritMerged openstack/python-novaclient master: nova limits ERROR (Exception): Field names must be unique  https://review.openstack.org/52792216:47
openstackgerritMerged openstack/python-novaclient master: Implement hypervisor hostname exact pattern match  https://review.openstack.org/52018716:47
*** damien_r has quit IRC16:48
*** itlinux has joined #openstack-nova16:48
*** yamamoto has joined #openstack-nova16:49
hrwah. """ D:16:49
*** yamamoto has quit IRC16:49
*** yamamoto has joined #openstack-nova16:50
*** yamamoto has quit IRC16:50
dansmithmriedem: ack16:50
*** edmondsw has quit IRC16:50
mriedemartom: https://review.openstack.org/#/c/508190/2/nova/compute/manager.py@266316:52
artommriedem, ah, doh16:53
artommriedem, well, my second one still kinda stands :P16:53
openstackgerritMarcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console  https://review.openstack.org/53800316:53
hrwstephenfin: as you wish ;D16:53
openstackgerritMerged openstack/python-novaclient master: Add CLI to show instance usage audit logs  https://review.openstack.org/40941316:53
artomI'll remove the -1 though16:54
*** gyee has joined #openstack-nova16:55
*** edmondsw has joined #openstack-nova16:56
stephenfinhrw: ++16:57
hrwmriedem, jaypipes, sahid: can you take another look at https://review.openstack.org/538003 patch? it is simplest of all versions and comes with extra test16:59
*** edmondsw has quit IRC17:00
*** moshele has joined #openstack-nova17:00
stephenfinbauzas: Gentle reminder for https://review.openstack.org/#/c/530743/17:01
mriedemstephenfin: see the latest comment there, it introduces a bug17:03
mriedemso a big -1 from me on the backport17:04
jaypipeshrw: I never saw a response from you to sahid's review comments?17:04
*** moshele has quit IRC17:04
*** AlexeyAbashkin has quit IRC17:05
hrwjaypipes: rewrote whole as an answer17:05
stephenfinmriedem: Yup, I was focussing on the latter part of the comment though: a partial fix was better than no fix17:05
jaypipeshrw: heh, ok :)17:05
jaypipeshrw: I don't see a followup from sahid which is why I asked..17:05
stephenfinProbably best to wait for the fixes' fix to be merged though17:06
hrwjaypipes: sure17:06
*** crushil has joined #openstack-nova17:06
hrwjaypipes: when 'a reply' would be just message then I write it. If it is simple fix then I mark Done'17:07
hrwjaypipes: if it is complicated then do code mostly17:07
*** slaweq has joined #openstack-nova17:08
hrwjaypipes: and here comment was more or less 'rewrite the shit' so I did17:08
*** ccamacho has quit IRC17:11
jaypipeshrw: understood. I'd still like to have sahid's opinion on it, though, since he had the biggest objection to it..17:11
hrwsure17:12
*** moshele has joined #openstack-nova17:12
*** slaweq has quit IRC17:14
*** slaweq has joined #openstack-nova17:15
*** ccamacho has joined #openstack-nova17:15
*** AlexeyAbashkin has joined #openstack-nova17:17
*** sahid has quit IRC17:20
*** AlexeyAbashkin has quit IRC17:21
*** brad[] has quit IRC17:23
*** Guest44860 has quit IRC17:24
*** moshele has quit IRC17:24
*** moshele has joined #openstack-nova17:27
*** yangyapeng has joined #openstack-nova17:28
hrwI have an idea for another 'lets add some device' patch just not found yet where to plug it17:28
*** slaweq has quit IRC17:28
hrwon pcie machines (aarch64, x86/q35) more pcie-root-port controllers can be handy if someone wants to add more disks/networks17:29
*** yangyapeng has quit IRC17:32
*** Tom-Tom has quit IRC17:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add nova-status check for ironic flavor migration  https://review.openstack.org/52754117:34
openstackgerritEric Fried proposed openstack/nova master: placement doc: Conflict caveat for DELETE APIs  https://review.openstack.org/54091917:34
*** moshele has quit IRC17:37
hrwit 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 check17:37
*** slaweq has joined #openstack-nova17:38
*** crushil has quit IRC17:40
*** armax has quit IRC17:41
*** slaweq has quit IRC17:43
openstackgerritMerged openstack/nova master: Update cells v2 layout doc caveats for Queens  https://review.openstack.org/54066717:44
openstackgerritMerged openstack/nova stable/ocata: libvirt: Re-initialise volumes, encryptors, and vifs on hard reboot  https://review.openstack.org/53142217:44
*** sshwarts has quit IRC17:44
*** sambetts is now known as sambetts|afk17:45
hrwarm64 guest has 1 pcie slot free when booted with graphical console17:49
*** yamamoto has joined #openstack-nova17:50
*** sridharg has quit IRC17:53
*** dtantsur is now known as dtantsur|afk17:53
*** josecastroleon has quit IRC17:57
openstackgerritBalazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild  https://review.openstack.org/52524217:59
*** derekh has quit IRC18:00
gibimriedem: fixed the bugfix patch as well ^^18:00
*** david-lyle has quit IRC18:01
mriedemok18:01
mriedemwill take a look after lunch18:01
*** jose-phillips has joined #openstack-nova18:02
gibimriedem: have nice lunch18:02
gibimriedem: 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 today18:03
mriedemthat's fine, it can be a later cleanup18:04
gibimriedem: OK18:05
*** yangyapeng has joined #openstack-nova18:05
*** yamamoto has quit IRC18:06
*** gjayavelu has joined #openstack-nova18:06
*** mgoddard_ has quit IRC18:07
*** ralonsoh has quit IRC18:07
mriedemcfriesen: i have an idea about the perf hit in pike,18:08
mriedemmaybe versioned notifications?18:09
mriedemcfriesen: are you running with those on?18:09
mriedemdefault is to send both versioned and unversioned notifications,18:09
mriedemand the versioned notifications have to sometimes lazy-load fields on the instance for the notification payload18:10
mriedemwhich means more round trips over rpc to the db18:10
*** yangyapeng has quit IRC18:10
jaypipeshrw: reviewed.18:12
*** josecastroleon has joined #openstack-nova18:12
cfriesenmriedem: will check18:15
hrwjaypipes: reading18:15
openstackgerritBalazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407  https://review.openstack.org/52609518:18
openstackbug 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
openstackgerritBalazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild  https://review.openstack.org/52524218:18
cfriesenmriedem: looks like explicitly unversioned.  thanks for the suggestion though.18:18
gibimriedem: managed to remove the sleep from the functional test.18:19
openstackgerritMarcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console  https://review.openstack.org/53800318:19
* gibi goes home18:19
*** jpena is now known as jpena|off18:19
hrwjaypipes: thanks18:19
*** amodi has quit IRC18:20
mriedemcfriesen: damn18:21
dansmithmriedem: so, we have some calls like build_and_run_instances() that has basically all optional arguments, even though they're not really optional18:27
dansmithmriedem: 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
dansmithbut that broke a bunch of tests that use the optionalness18:28
dansmithI 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
mriedemi was going to say it might not be worth the trouble to refactor all of that18:29
mriedembut it's not a strong opinion on18:29
mriedem*no18:29
dansmithokay18:29
dansmithwell, I guess I'll just be lazy and not do it then18:30
*** itlinux has quit IRC18:31
*** moshele has joined #openstack-nova18:32
*** Swami has joined #openstack-nova18:32
jaypipeshrw: +W18:35
hrwthx18:36
*** amodi has joined #openstack-nova18:36
*** tssurya has quit IRC18:36
openstackgerritYuki Nishiwaki proposed openstack/nova master: Check enable_new_services option in service.py  https://review.openstack.org/54096218:36
*** moshele has quit IRC18:38
*** rodolof has quit IRC18:44
openstackgerritChris Dent proposed openstack/nova-specs master: Add generation support in aggregate association  https://review.openstack.org/54044718:44
*** rodolof has joined #openstack-nova18:46
*** mgoddard_ has joined #openstack-nova18:49
*** traskat has joined #openstack-nova18:50
*** sean-k-mooney has joined #openstack-nova18:52
*** david-lyle has joined #openstack-nova18:57
mriedemdansmith: i'm ok with lazy, cleaning up cruft and dropping deprecated stuff is good for me18:58
mriedemgiven we're only a few days out from rc118:58
dansmithack, yeah I left it alone18:58
dansmithI'm working on all the signature changes for the other stuff in the tests right now18:58
* jaypipes gets out his sean-k-mooney-dictionary while reading review on https://review.openstack.org/53307718:59
*** ccamacho has quit IRC18:59
*** harlowja has joined #openstack-nova19:03
hrwLibvirtConfigGuestPCIeRootPortController is my next class to add. but not today.19:06
*** itlinux has joined #openstack-nova19:07
* hrw off19:13
*** amoralej is now known as amoralej|off19:14
*** brad[] has joined #openstack-nova19:17
openstackgerritMerged openstack/nova stable/pike: Import the config drive docs from openstack-manuals  https://review.openstack.org/51505319:18
ameedagibi and mriedem Thank you very much, I will upload new patch soon19:18
dansmithmriedem: 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 build19:20
ameedamriedem: can you please confirm this for me ? https://review.openstack.org/#/c/540274/ its nova-manager doc19:21
*** slaweq has joined #openstack-nova19:23
*** chyka has quit IRC19:27
*** edmondsw has joined #openstack-nova19:29
sean-k-mooneydansmith: 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/174749619:30
openstackLaunchpad 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-mooneydansmith: 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 work19:31
sean-k-mooneydansmith: mriedem will i trow a path up for rocky or is this something ye want to backport?19:31
*** slaweq_ has joined #openstack-nova19:32
openstackgerritAmeed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails  https://review.openstack.org/52838519:33
sean-k-mooneyusing 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
openstackgerritMerged openstack/nova stable/ocata: Set server status to ERROR if rebuild failed  https://review.openstack.org/53690419:36
*** tesseract has quit IRC19:36
ameedamriedem, gibi patch uploaded19:36
*** slaweq_ has quit IRC19:36
*** mgoddard_ has quit IRC19:36
ameedajaypipes, bauzas: could you please review this for me https://review.openstack.org/#/c/526900/19:37
*** AlexeyAbashkin has joined #openstack-nova19:38
*** tbachman has quit IRC19:39
*** lucasagomes is now known as lucas-afk19:39
openstackgerritEric Fried proposed openstack/nova-specs master: Update Provider Tree  https://review.openstack.org/54011119:39
efriededleafe, cdent, jaypipes: ^19:39
*** slaweq has quit IRC19:39
*** chyka has joined #openstack-nova19:40
*** elod has quit IRC19:46
sean-k-mooneystephenfin: 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
openstackLaunchpad 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 IRC19:47
mriedemsean-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 hack19:48
sean-k-mooneymriedem: yep i can is that your prefence? also do we want to backport to queens et al now or wait19:49
sean-k-mooneymriedem: im just heading home but i can see if i can trow up the xml patch tommorow if i get time19:49
mriedemsean-k-mooney: i wouldn't consider it something we have to hold up queens rc1 for19:51
mriedemif it's latent19:51
ameedamriedem: when I try to  git cherry-pick -x c0aca1d333628f9174e8db53674787e281e7e0d7 I got this error -> fatal: bad object c0aca1d333628f9174e8db53674787e281e7e0d719:52
sean-k-mooneymriedem: 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 ptg19:52
*** imacdonn has joined #openstack-nova19:55
mriedemameeda: git fetch https://git.openstack.org/openstack/nova refs/changes/71/540871/1 && git cherry-pick -x FETCH_HEAD19:55
mriedemstvnoyes: 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-mooneymriedem: 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-nova19:57
mriedemstvnoyes: we call this for each instance we create in a multi-create request https://github.com/openstack/nova/blob/master/nova/compute/api.py#L125319:59
mriedemand _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 record19:59
*** edmondsw has quit IRC20:00
mriedemwe then make a copy of that list per instance https://github.com/openstack/nova/blob/master/nova/compute/api.py#L126720:00
mriedemi'm wondering if we're overwriting or losing any of the bdm.attachment_id records when creating multiple instances20:00
mriedemmaybe it's ok20:00
mriedemmight 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 volume20:01
mriedemi 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
ameedamriedem: now the commit message have those 2 lines20:04
ameeda(cherry picked from commit eba33a0e2d8aace6fbc889026fb80dc2c208bc8d) (cherry picked from commit c0aca1d333628f9174e8db53674787e281e7e0d7)20:04
*** yangyapeng has joined #openstack-nova20:04
mriedemameeda: those should go on separate lines20:05
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: debug nova-api service version check during boot from volume  https://review.openstack.org/54099220:05
mriedembut that's ine20:05
mriedemfine20:05
*** itlinux has quit IRC20:06
mriedemdansmith: we haven't been doing proper service version checking in nova-api since pike https://review.openstack.org/#/c/539755/ - should be an easy +W20:08
ameedamriedem: yes they are , so I can do git review now ?20:08
mriedemameeda: yes20:08
ameedamriedem: Thank you !!20:08
*** yangyapeng has quit IRC20:08
openstackgerritAmeed Ashour proposed openstack/nova stable/ocata: add "--until-complete" option for nova-manage db archive_deleted_rows.  https://review.openstack.org/54027420:09
ameedamriedem: could you please give me +2 here https://review.openstack.org/#/c/528385/ :$20:09
*** slaweq has joined #openstack-nova20:10
melwittjaypipes: replied20:10
*** acormier has quit IRC20:11
*** slaweq_ has joined #openstack-nova20:11
*** acormier has joined #openstack-nova20:11
mriedemameeda: your ocata backport is wrong20:12
ameedaI see20:14
*** slaweq_ has quit IRC20:16
*** slaweq has quit IRC20:16
*** acormier has quit IRC20:16
*** mgoddard_ has joined #openstack-nova20:19
*** slaweq has joined #openstack-nova20:21
openstackgerritAmeed Ashour proposed openstack/nova stable/ocata: add "--until-complete" option for nova-manage db archive_deleted_rows.  https://review.openstack.org/54027420:23
*** itlinux has joined #openstack-nova20:23
ameedamriedem: I don't know if I did the correct thing, please check it ^20:27
mriedemameeda: yes that's correct, thanks20:28
*** dave-mcc_ has quit IRC20:29
ameedaThank you !!20:29
openstackgerritMatt Riedemann proposed openstack/nova master: Add a nova-caching-scheduler job to the experimental queue  https://review.openstack.org/53926020:30
ameedamriedem: please don't forget to review this https://review.openstack.org/#/c/528385/20:32
*** cdent has quit IRC20:32
*** mgoddard_ has quit IRC20:35
*** READ10 has quit IRC20:35
*** acormier has joined #openstack-nova20:36
*** acormier has quit IRC20:40
*** acormier has joined #openstack-nova20:40
*** acormier_ has joined #openstack-nova20:43
*** acormier has quit IRC20:45
*** acormier_ has quit IRC20:45
*** slaweq_ has joined #openstack-nova20:48
*** vladikr has quit IRC20:51
*** slaweq_ has quit IRC20:53
*** yangyapeng has joined #openstack-nova20:54
mriedemefried: i think this might have broken the nova-next job but i'm not sure how https://github.com/openstack-dev/devstack/commit/49144e627120f9ec1cf72dfc84e02b9b81e669b620:56
mriedemhttps://bugs.launchpad.net/nova/+bug/174751120:56
openstackLaunchpad 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-nova20:57
mriedemthe only thing unique about the nova-next job at this point is it sets NOVA_USE_SERVICE_TOKEN=True in devstack20:57
*** yangyapeng has quit IRC20:58
openstackgerritDan Smith proposed openstack/nova master: Bump compute RPC API to version 5.0  https://review.openstack.org/54100520:59
dansmithgawd20:59
mriedemufta21:00
*** vladikr has joined #openstack-nova21:02
*** moshele has joined #openstack-nova21:02
*** acormier has quit IRC21:06
*** acormier has joined #openstack-nova21:06
efriedmriedem: Is the bustage consistent?21:06
*** traskat has quit IRC21:07
mriedemefried: yeah, 100% fail21:07
*** chyka has quit IRC21:07
efriedmriedem: 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
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: debug nova-next image not found failures  https://review.openstack.org/54100821:08
mriedemi think we could also just ignore the service_user config in nova as a test21:08
efriedI have to jet for a doc appt right now, will catch up with you in an hour or so.21:08
*** moshele has quit IRC21:09
*** acormier has quit IRC21:11
jaypipesthx melwitt21:12
openstackgerritmelanie witt proposed openstack/nova stable/ocata: Add release note for Aggregate[Core|Ram|Disk]Filter change  https://review.openstack.org/54101321:15
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: debug nova-next image not found failures  https://review.openstack.org/54100821:16
*** acormier has joined #openstack-nova21:17
*** acormier_ has joined #openstack-nova21:20
*** dave-mccowan has joined #openstack-nova21:21
*** acormier has quit IRC21:21
*** acormier_ has quit IRC21:25
*** rcernin has joined #openstack-nova21:26
*** tbachman has joined #openstack-nova21:29
openstackgerritmelanie witt proposed openstack/nova master: Add release note for Aggregate[Core|Ram|Disk]Filter change  https://review.openstack.org/54101821:30
*** chyka has joined #openstack-nova21:37
*** yangyapeng has joined #openstack-nova21:44
*** itlinux has quit IRC21:45
mriedemmaciejjozefczyk: not sure why you'd +1 this :) https://review.openstack.org/#/c/540992/21:48
*** yangyapeng has quit IRC21:48
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: debug nova-api service version check during boot from volume  https://review.openstack.org/54099221:50
*** burt has quit IRC21:52
*** moshele has joined #openstack-nova21:54
*** itlinux has joined #openstack-nova21:54
openstackgerritMatthew Edmonds proposed openstack/nova master: Clarify CONF.scheduler.max_attempts  https://review.openstack.org/54103121:56
edmondsw_edleafe noticed this after reading your blog ^21:56
*** edmondsw_ is now known as edmondsw21:57
edmondswmriedem do we have a more general term that encompasses create, resize, and migrate if "build" doesn't do that?21:59
mriedemscheduling?22:02
mriedemlike it said22:02
edmondswwe can't say "schedule", because we only talk to the scheduler once.22:02
melwittit's still scheduling though, isn't it? trying alternate hosts *shrug*22:03
edmondswI guess22:03
edmondswat least the "for a chosen host" bit from the old help text is wrong, right?22:03
edleafemriedem: I think the problematic wording is "for a chosen host". But that was wrong before, too.22:04
edleafejinx22:04
edmondswedleafe yeah that is what first caught my eye22:04
mriedemyes 'for a chosen host' is wrong22:04
edmondswI can change it back to scheduling and correct that part22:04
edleafe"for a given VM build/move operation"22:05
edmondswI like that...22:06
mriedemcan we somehow make this about ironic and say, 'for a given bm movement'22:06
*** moshele has quit IRC22:06
edleafenah, that sounds kinda crappy22:06
edmondswlol22:06
openstackgerritMatthew Edmonds proposed openstack/nova master: Clarify CONF.scheduler.max_attempts  https://review.openstack.org/54103122:07
efriedmriedem: I have returned.  Any update on the glance image snafu?22:08
mriedemefried: debug patch is still running22:08
jrolledleafe: "for a given VM b/m operation" :P22:08
*** acormier has joined #openstack-nova22:09
mriedemi can't wait to be in the vm bm room for 2 days in dublin22:09
mriedemtalking about...keystone22:10
mriedemi came for the bm, stayed for the keystone22:10
*** dave-mccowan has quit IRC22:10
openstackgerritMatthew Edmonds proposed openstack/nova master: Clarify CONF.scheduler.max_attempts  https://review.openstack.org/54103122:10
edleafeI'm SO glad I made it out of third grade22:11
openstackgerritDan Smith proposed openstack/nova master: Bump compute RPC API to version 5.0  https://review.openstack.org/54100522:11
openstackgerritDan Smith proposed openstack/nova master: Compute RPC client bump to 5.0  https://review.openstack.org/54103522:11
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Query all cells for service version in _validate_bdm  https://review.openstack.org/54103622:12
*** slaweq_ has joined #openstack-nova22:12
mriedemefried: results http://logs.openstack.org/08/541008/2/check/legacy-tempest-dsvm-neutron-nova-next-full/f24c9d2/logs/screen-n-api.txt.gz?level=TRACE22:13
*** acormier has quit IRC22:14
efriedmriedem: Looks to me like neither of your glance.py messages hit.22:15
efriedwhich is... weird.22:15
openstackgerritMatthew Edmonds proposed openstack/nova master: Clarify CONF.scheduler.max_attempts  https://review.openstack.org/54103122:15
*** slaweq_ has quit IRC22:16
mriedemhttp://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_45593422:16
mriedemUsing glance API servers: [u'http://158.69.92.100/image/v2/']22:16
efriedmriedem: 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_65188522:17
mriedemefried: maybe the image exists but the context doesn't have auth to GET it?22:17
efriedmriedem: This is https://bugs.launchpad.net/python-glanceclient/+bug/170799522:18
openstackLaunchpad bug 1707995 in Glance Client "Can't use versioned endpoints" [Undecided,New]22:18
mriedemshazbot!22:18
efriedThough I thought we weren't subject to that with this set of changes.22:18
mriedemnot sure why the other jobs wouldn't be failing though22:18
*** vladikr has quit IRC22:19
efriedAlso checked this through the CI six ways from sunday before we merged it.22:19
mriedemdevstack doesn't run nova-next22:19
efriedSo some weird combo of environments - or possibly a recent "fix" in ksa? - is exposing this.22:19
mriedemi can try to nuke the usage of the service_user in this debug patch and see if things start magically passing22:20
efriedmriedem: No, but I made DNM nova patches with Depends-On the devstack change, exactly to validate this.22:20
mriedemnova-next is also non-voting22:20
mriedemso might not have noticed22:20
efriedoh, so maybe it failed and I neve.. yeah22:20
jrollso 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
efriedmriedem: 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 IRC22:22
mriedemjroll: don't thin kso22:23
*** slaweq has joined #openstack-nova22:23
jrollmriedem: that is so useful, lol22:23
*** yangyapeng has joined #openstack-nova22:23
mriedemjroll: i don't even see a created_at filter for listing nodes https://developer.openstack.org/api-ref/baremetal/#id422:24
mriedemso shut up!22:24
jrollheh, I'm not sure which is worse22:24
*** slaweq has quit IRC22:25
efriedcreated_at=<exact timestamp> ??22:25
efriedWheee22:25
jroll:)22:25
openstackgerritMerged openstack/nova master: Doc: Nix os-traits link from POST resource_classes  https://review.openstack.org/54055922:25
*** slaweq has joined #openstack-nova22:25
mriedemjroll: add a created_before/created_after param22:26
openstackgerritMerged openstack/nova stable/pike: add "--until-complete" option for nova-manage db archive_deleted_rows.  https://review.openstack.org/54087122:26
mriedemthere is changes-since...22:26
jrollmriedem: that doesn't help the guy that's trying to use it today :P22:26
jrollthanks though22:26
mriedembut that's updated_at>=changes-since22:26
*** rodolof has quit IRC22:26
mriedemlet me hop in the old delorean real quick22:27
jrollright22:27
jrollok great thanks22:27
* mriedem grabs some gel for the ride22:27
*** pchavva has quit IRC22:27
* jroll tells mriedem's boss to give him a gold star22:27
mriedemi have multiple bosses, so good luck22:27
openstackgerritmelanie witt proposed openstack/nova master: Make InstanceGroup.get_hosts query all cells  https://review.openstack.org/54025822:27
*** acormier has joined #openstack-nova22:27
*** yangyapeng has quit IRC22:28
*** slaweq_ has joined #openstack-nova22:28
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: debug nova-api service version check during boot from volume  https://review.openstack.org/54099222:30
* mriedem goes to wrangle snow22:30
*** inara has quit IRC22:31
*** vladikr has joined #openstack-nova22:32
efriedAnyone 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-nova22:33
*** slaweq_ has quit IRC22:33
openstackgerritMerged openstack/nova master: Query all cells for service version in _validate_bdm  https://review.openstack.org/53975522:34
openstackgerritMerged openstack/nova stable/pike: Import user-data page from openstack-manuals  https://review.openstack.org/51505422:34
*** slaweq has quit IRC22:35
*** jaypipes_ has joined #openstack-nova22:35
*** jaypipes has quit IRC22:35
efriedjaypipes_: Any plans for a Vancouver presentation about what we've done / are doing in placement/scheduler-land?22:36
*** jdurgin has joined #openstack-nova22:41
*** itlinux has quit IRC22:48
edleafeefried: you should do one. They're probably tired of seeing Jay's ugly mug22:58
efriedThey need a new ugly mug to stare at, is what you're sayin.22:59
edleafezactly23:00
efriedI haven't done this before; how much flexibility is there to edit the proposal later, add presenters, change the abstract, etc.?23:01
edmondswefried I think you can change presenters later if need be. Not sure about the abstract. They probably don't want that changing too much23:05
*** edmondsw has quit IRC23:07
*** edmondsw has joined #openstack-nova23:08
*** edmondsw has quit IRC23:12
*** yangyapeng has joined #openstack-nova23:13
*** liverpooler has quit IRC23:13
*** yamahata has joined #openstack-nova23:16
*** yangyapeng has quit IRC23:17
*** threestrands has joined #openstack-nova23:19
*** threestrands has quit IRC23:19
*** threestrands has joined #openstack-nova23:19
*** mlavalle has quit IRC23:23
*** yamahata has quit IRC23:26
*** takashin has joined #openstack-nova23:30
efriededleafe: Can you see this?  https://www.openstack.org/summit/vancouver-2018/call-for-presentations/manage/20813/summary23:34
efriedI 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
edleafeefried: See my comment?23:36
efriededleafe: Where?23:36
edleafeor do I have to save it first?23:36
edleafeyeah, not an etherpad :)23:36
efriedI'm not even sure how I would view it.23:36
edleafeok, in the abstaract23:36
edleafeabstract, even23:36
efriedoh, jeez, so like anyone can edit it?23:37
jrolllooks like it23:37
edleafeI guess so, if they know the URL23:37
efriedAnd now everybody does, since I put it in IRC.23:37
* jroll enumerates the id23:37
*** masahisa has joined #openstack-nova23:37
efriedBut I imagine your openID is attached to your edits.23:38
*** slaweq has joined #openstack-nova23:38
edleafeI removed my silly comment23:38
efriedOkay.  But seem reasonable in terms of scope, taggage, etc.?23:38
jrollefried: assuming edits are tracked23:38
efriedye gods, I hope they are.23:38
efriedOtherwise anyone could register for an openID and cause havoc23:39
* jroll wouldn't bet on it23:39
edleafeefried: it's afree-for-all!!23:39
jrollfwiw, I'm poking to find out where to report the issue23:39
* edleafe is about to duck out for the day23:40
*** acormier has quit IRC23:40
*** yamahata has joined #openstack-nova23:42
*** slaweq has quit IRC23:42
*** yamahata has quit IRC23:43
*** yamahata has joined #openstack-nova23:43
openstackgerritMerged openstack/nova stable/ocata: Don't wait for vif plug events during _hard_reboot  https://review.openstack.org/54068023:45
openstackgerritEric Fried proposed openstack/nova master: Test case: new standard resource class unusable  https://review.openstack.org/53973223:47
openstackgerritEric Fried proposed openstack/nova master: Ensure resource classes correctly  https://review.openstack.org/53973823:47
openstackgerritEric Fried proposed openstack/nova master: New-style _set_inventory_for_provider  https://review.openstack.org/53764823:47
openstackgerritEric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree  https://review.openstack.org/53382123:47
openstackgerritEric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker  https://review.openstack.org/52024623:47
openstackgerritEric Fried proposed openstack/nova master: Fix nits in update_provider_tree series  https://review.openstack.org/53126023:47
openstackgerritEric Fried proposed openstack/nova master: Move refresh time from report client to prov tree  https://review.openstack.org/53551723:47
openstackgerritEric Fried proposed openstack/nova master: Make generation optional in ProviderTree  https://review.openstack.org/53932423:47
efriedjaypipes_ edleafe ^ tweaked per y'all's comments.23:48
efriedAaaand I'm out.  See y'all tomorrow.23:49
openstackgerritMerged openstack/nova stable/ocata: add "--until-complete" option for nova-manage db archive_deleted_rows.  https://review.openstack.org/54027423:51
*** hongbin has quit IRC23:53
jaypipes_efried: cool, thx23:57
*** lei-zh has joined #openstack-nova23:59
*** vladikr has quit IRC23:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!