Wednesday, 2020-03-04

*** ileixe has joined #openstack-nova00:06
*** ileixe has quit IRC00:07
*** brinzhang_ has joined #openstack-nova00:09
*** slaweq has joined #openstack-nova00:11
*** vishalmanchanda has joined #openstack-nova00:11
*** brinzhang has quit IRC00:13
*** slaweq has quit IRC00:15
*** tbachman has quit IRC00:20
*** igordc has joined #openstack-nova00:21
*** prometheanfire has quit IRC00:21
*** brinzhang has joined #openstack-nova00:25
*** brinzhang_ has quit IRC00:28
*** macz_ has quit IRC00:33
*** igordc has quit IRC00:40
*** spatel has joined #openstack-nova00:43
*** spatel has quit IRC00:48
*** tbachman has joined #openstack-nova00:54
*** brinzhang_ has joined #openstack-nova00:55
*** brinzhang has quit IRC00:58
*** slaweq has joined #openstack-nova01:11
*** spatel has joined #openstack-nova01:14
*** slaweq has quit IRC01:16
*** zhanglong has joined #openstack-nova01:20
*** igordc has joined #openstack-nova01:20
*** sapd1 has joined #openstack-nova01:21
*** igordc has quit IRC01:33
*** rcernin has quit IRC02:02
*** threestrands has joined #openstack-nova02:06
*** owalsh_ has quit IRC02:09
*** gyee has quit IRC02:09
*** slaweq has joined #openstack-nova02:11
*** slaweq has quit IRC02:15
openstackgerritBrin Zhang proposed openstack/nova master: Add PATCH volume attachments api to os-volume_attachments  https://review.opendev.org/69382802:23
openstackgerritBrin Zhang proposed openstack/nova master: Add functional tests for PATCH volume attachments API  https://review.opendev.org/71096502:23
*** kaisers has quit IRC02:27
*** owalsh has joined #openstack-nova02:29
*** brinzhang has joined #openstack-nova02:31
*** brinzhang_ has quit IRC02:34
*** mkrai has joined #openstack-nova02:49
*** hongbin has joined #openstack-nova02:50
*** rcernin has joined #openstack-nova02:52
*** prometheanfire has joined #openstack-nova03:09
*** slaweq has joined #openstack-nova03:11
*** slaweq has quit IRC03:16
*** links has joined #openstack-nova03:19
*** zhanglong has quit IRC03:24
*** lbragstad has quit IRC03:31
*** psachin has joined #openstack-nova03:33
openstackgerritMerged openstack/nova master: Unplug VIFs as part of cleanup of networks  https://review.opendev.org/66338204:04
*** spatel has quit IRC04:05
*** spatel has joined #openstack-nova04:05
*** slaweq has joined #openstack-nova04:11
*** slaweq has quit IRC04:17
prometheanfirenew oslo.db breaks nova https://review.opendev.org/71103104:34
*** spatel has quit IRC04:39
*** owalsh_ has joined #openstack-nova04:52
*** owalsh has quit IRC04:53
*** owalsh_ has quit IRC04:53
*** hongbin has quit IRC04:54
*** owalsh has joined #openstack-nova04:59
*** artom has quit IRC04:59
*** haleyb is now known as haleyb|away05:05
*** udesale has joined #openstack-nova05:07
*** slaweq has joined #openstack-nova05:11
*** slaweq has quit IRC05:15
*** evrardjp has quit IRC05:35
*** evrardjp has joined #openstack-nova05:35
*** udesale_ has joined #openstack-nova05:38
*** igordc has joined #openstack-nova05:38
*** udesale has quit IRC05:40
*** TxGirlGeek has quit IRC05:44
*** igordc has quit IRC06:11
*** slaweq has joined #openstack-nova06:11
*** slaweq has quit IRC06:16
*** damien_r has joined #openstack-nova06:17
*** yedongcan has joined #openstack-nova06:20
*** damien_r has quit IRC06:21
*** vishalmanchanda has quit IRC06:31
*** dpawlik has joined #openstack-nova06:33
*** threestrands has quit IRC06:34
openstackgerritMichael Bayer proposed openstack/nova master: Remove oslo_db.sqlalchemy.compat reference  https://review.opendev.org/71109306:34
*** udesale has joined #openstack-nova06:34
*** threestrands has joined #openstack-nova06:35
*** threestrands has quit IRC06:36
*** threestrands has joined #openstack-nova06:36
*** udesale_ has quit IRC06:36
*** threestrands has quit IRC06:37
*** threestrands has joined #openstack-nova06:38
*** threestrands has quit IRC06:39
*** udesale_ has joined #openstack-nova06:42
*** udesale has quit IRC06:45
*** udesale has joined #openstack-nova06:46
*** udesale_ has quit IRC06:46
*** vishalmanchanda has joined #openstack-nova06:55
*** damien_r has joined #openstack-nova06:57
*** damien_r has quit IRC06:57
*** damien_r has joined #openstack-nova06:57
*** hamzy has quit IRC07:04
*** hamzy has joined #openstack-nova07:05
*** lpetrut has joined #openstack-nova07:05
*** lpetrut has quit IRC07:06
*** lpetrut has joined #openstack-nova07:06
*** kaisers has joined #openstack-nova07:10
*** slaweq has joined #openstack-nova07:11
*** udesale_ has joined #openstack-nova07:12
*** udesale has quit IRC07:15
*** slaweq has quit IRC07:15
*** happyhemant has joined #openstack-nova07:17
*** damien_r has quit IRC07:25
*** slaweq has joined #openstack-nova07:38
*** slaweq has quit IRC07:43
*** bbowen_ has joined #openstack-nova07:46
*** bbowen has quit IRC07:47
*** bbowen_ has quit IRC07:48
*** bbowen_ has joined #openstack-nova07:48
*** slaweq has joined #openstack-nova07:49
*** kozhukalov has quit IRC07:51
*** tesseract has joined #openstack-nova08:09
*** mkrai has quit IRC08:09
*** hamzy has quit IRC08:12
*** hamzy has joined #openstack-nova08:13
*** kozhukalov has joined #openstack-nova08:13
*** maciejjozefczyk has joined #openstack-nova08:14
*** damien_r has joined #openstack-nova08:14
*** rpittau|afk is now known as rpittau08:16
*** amoralej|off is now known as amoralej08:22
*** mkrai has joined #openstack-nova08:22
*** ralonsoh has joined #openstack-nova08:33
*** brinzhang_ has joined #openstack-nova08:36
*** kozhukalov has quit IRC08:39
*** brinzhang has quit IRC08:39
*** brinzhang has joined #openstack-nova08:40
*** brinzhang_ has quit IRC08:41
*** brinzhang_ has joined #openstack-nova08:42
*** brinzhang has quit IRC08:44
*** brinzhang has joined #openstack-nova08:48
*** brinzhang_ has quit IRC08:51
*** brinzhang_ has joined #openstack-nova08:52
*** brinzhang has quit IRC08:53
*** Liang__ has joined #openstack-nova08:55
*** udesale_ has quit IRC09:08
*** boxiang_ has quit IRC09:17
*** boxiang_ has joined #openstack-nova09:18
*** brinzhang has joined #openstack-nova09:24
*** brinzhang_ has quit IRC09:26
*** boxiang_ has quit IRC09:27
*** tkajinam has quit IRC09:30
*** hamzy has quit IRC09:34
*** hamzy has joined #openstack-nova09:34
*** ociuhandu has joined #openstack-nova09:35
*** martinkennelly has joined #openstack-nova09:37
*** derekh has joined #openstack-nova09:40
openstackgerritJohn Garbutt proposed openstack/nova master: Ensure we pass a target in admin actions  https://review.opendev.org/66309509:43
openstackgerritJohn Garbutt proposed openstack/nova master: Pass the actual target in os-availability-zone policy  https://review.opendev.org/70669109:46
openstackgerritJohn Garbutt proposed openstack/nova master: Add test coverage of existing os-console-auth-tokens policies  https://review.opendev.org/70668709:47
*** hrw has joined #openstack-nova09:47
hrwmorning09:47
openstackgerritJohn Garbutt proposed openstack/nova master: Introduce scope_types in os-agents policy  https://review.opendev.org/70164509:48
hrwjust wanted to inform that we released CirrOS 0.5.0 - https://github.com/cirros-dev/cirros/releases/tag/0.5.009:48
openstackgerritJohn Garbutt proposed openstack/nova master: Add new default roles in os-agents policies  https://review.opendev.org/70164809:48
openstackgerritJohn Garbutt proposed openstack/nova master: Pass the actual target in os-agents policy  https://review.opendev.org/70164909:48
openstackgerritJohn Garbutt proposed openstack/nova master: Add test coverage of existing attach_interfaces policies  https://review.opendev.org/70512609:48
openstackgerritJohn Garbutt proposed openstack/nova master: Remove old policy enforcement in attach_interfaces  https://review.opendev.org/70512709:49
openstackgerritJohn Garbutt proposed openstack/nova master: Introduce scope_types in os-attach-interfaces  https://review.opendev.org/70579909:50
*** brinzhang_ has joined #openstack-nova09:52
*** brinzhang_ has quit IRC09:54
lyarwoodkashyap / stephenfin ; https://review.opendev.org/#/c/702021/ - more trivial clean up if you have time today for a review btw09:54
*** brinzhang_ has joined #openstack-nova09:54
*** xek has joined #openstack-nova09:55
*** brinzhang has quit IRC09:55
openstackgerritJohn Garbutt proposed openstack/nova master: Fix os-os-deferred-delete policy to be admin_or_owner  https://review.opendev.org/70745709:58
openstackgerritJohn Garbutt proposed openstack/nova master: Add test coverage of existing deferred_delete policies  https://review.opendev.org/70745509:58
* kashyap clicks09:59
kashyaplyarwood: Yep, negative-code++09:59
*** ccamacho has quit IRC10:02
*** hamzy has quit IRC10:03
*** ociuhandu has quit IRC10:04
*** hamzy has joined #openstack-nova10:04
*** hamzy has quit IRC10:06
*** hamzy_ has joined #openstack-nova10:06
*** dtantsur|afk is now known as dtantsur10:09
openstackgerritBrin Zhang proposed openstack/nova master: Add PATCH volume attachments api to os-volume_attachments  https://review.opendev.org/69382810:18
openstackgerritBrin Zhang proposed openstack/nova master: Add new policy to PATCH update volume API  https://review.opendev.org/71119410:18
*** ociuhandu has joined #openstack-nova10:19
openstackgerritBrin Zhang proposed openstack/nova master: Add functional tests for PATCH volume attachments API  https://review.opendev.org/71096510:19
*** hamzy has joined #openstack-nova10:20
*** hamzy_ has quit IRC10:21
*** brinzhang_ has quit IRC10:22
openstackgerritLee Yarwood proposed openstack/nova master: api: Introduce microverion 2.83 allowing boot from volume rescue  https://review.opendev.org/70143010:25
openstackgerritLee Yarwood proposed openstack/nova master: compute: Extract _get_bdm_image_metadata into nova.utils  https://review.opendev.org/70521210:25
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Support boot from volume stable device instance rescue  https://review.opendev.org/70143110:25
openstackgerritLee Yarwood proposed openstack/nova master: DNM - Test stable device rescue tests with BFV instances  https://review.opendev.org/71005010:25
*** hamzy has quit IRC10:26
*** hamzy has joined #openstack-nova10:28
*** tosky has joined #openstack-nova10:30
*** hamzy has quit IRC10:33
*** hrw has left #openstack-nova10:36
*** ccamacho has joined #openstack-nova10:58
*** xek_ has joined #openstack-nova11:05
*** mkrai has quit IRC11:07
*** xek has quit IRC11:08
*** rcernin has quit IRC11:08
*** tbachman has quit IRC11:15
*** zhanglong has joined #openstack-nova11:20
*** sapd1 has quit IRC11:21
*** rpittau is now known as rpittau|bbl11:32
*** psachin has quit IRC11:33
*** psachin has joined #openstack-nova11:37
*** zhanglong has quit IRC11:38
*** zhanglong has joined #openstack-nova11:40
*** zhanglong has quit IRC11:44
lyarwoodhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b78/706898/6/check/nova-tox-functional-py36/b781ccf/testr_results.html - this is new, TypeError: 'TestOpenStackClient' object is not subscriptable, has anyone seen this?11:46
*** zhanglong has joined #openstack-nova11:48
*** psachin has quit IRC11:49
*** openstackstatus has joined #openstack-nova11:51
*** ChanServ sets mode: +v openstackstatus11:51
*** mkrai has joined #openstack-nova11:52
*** mvkr has quit IRC11:58
openstackgerritStephen Finucane proposed openstack/nova stable/train: Functional test for UnexpectedDeletingTaskStateError  https://review.opendev.org/71121012:05
openstackgerritLee Yarwood proposed openstack/nova master: functional: Do not provide self.api to _wait_for_server_parameter within test_bug_1831771  https://review.opendev.org/71121112:06
lyarwoodstephenfin: ^ I have no idea how why we are only seeing this failure now, any ideas?12:06
stephenfinNot a clue. Very odd12:07
stephenfinI wonder if that's not being triggered all the time or something12:07
lyarwoodit actually passes locally which is super odd12:12
*** mkrai has quit IRC12:16
*** mvkr has joined #openstack-nova12:19
lyarwoodright, the race isn't actually being triggered all of the time12:19
lyarwoodis there a way to force the functional tests to log in DEBUG (with OS_DEBUG=1) even when they pass?12:21
lyarwoodstephenfin: https://github.com/openstack/nova/blob/5aa8df8f5cd4013b47a381ea929edd7ac425810e/nova/tests/functional/regressions/test_bug_1831771.py#L50-L67 - that isn't guaranteed to race everytime we run the test12:24
*** martinkennelly has quit IRC12:29
*** johnthetubaguy has quit IRC12:29
*** martinkennelly has joined #openstack-nova12:29
*** vishalmanchanda has quit IRC12:31
*** johnthetubaguy has joined #openstack-nova12:32
*** zhanglong has quit IRC12:35
*** mkrai has joined #openstack-nova12:42
*** zhanglong has joined #openstack-nova12:42
*** ygk_12345 has joined #openstack-nova12:45
*** TheJulia has quit IRC12:54
*** TheJulia has joined #openstack-nova12:54
*** _erlon_ has quit IRC12:54
*** vdrok has quit IRC12:54
*** CeeMac has quit IRC12:54
*** _erlon_ has joined #openstack-nova12:55
*** masayukig has quit IRC12:55
*** vdrok has joined #openstack-nova12:55
*** CeeMac has joined #openstack-nova12:56
*** masayukig has joined #openstack-nova12:56
*** nweinber has joined #openstack-nova12:57
*** liuyulong has joined #openstack-nova13:01
stephenfinlyarwood: okay, so dodgy test. I'll look into making it a more deterministic before we backport it13:03
*** udesale has joined #openstack-nova13:03
*** jamesdenton has quit IRC13:07
*** tbachman has joined #openstack-nova13:08
lyarwoodstephenfin: I've got a change locally to do that, I'll push after lunch13:09
*** nicolasbock has joined #openstack-nova13:11
stephenfinack13:15
*** zhanglong has quit IRC13:15
*** nweinber has quit IRC13:16
*** zhanglong has joined #openstack-nova13:17
*** mkrai has quit IRC13:17
*** artom has joined #openstack-nova13:19
*** vdrok has quit IRC13:23
*** amoralej is now known as amoralej|lunch13:24
*** NobodyCam has quit IRC13:24
*** efried1 has joined #openstack-nova13:24
*** vdrok has joined #openstack-nova13:25
*** larainema has joined #openstack-nova13:25
*** efried has quit IRC13:25
*** efried1 is now known as efried13:25
*** NobodyCam has joined #openstack-nova13:25
*** jmlowe has quit IRC13:26
*** ygk_12345 has quit IRC13:27
*** vishalmanchanda has joined #openstack-nova13:36
openstackgerritElod Illes proposed openstack/nova stable/rocky: libvirt: remove conditional on VIR_DOMAIN_EVENT_SUSPENDED_POSTCOPY  https://review.opendev.org/71123113:38
openstackgerritElod Illes proposed openstack/nova stable/rocky: libvirt: check job status for VIR_DOMAIN_EVENT_SUSPENDED_MIGRATED event  https://review.opendev.org/71123313:39
openstackgerritLee Yarwood proposed openstack/nova master: functional: Avoid race and fix use of self.api within test_bug_1831771  https://review.opendev.org/71121113:40
lyarwoodstephenfin: ^ there you go13:40
openstackgerritLucian Petrut proposed openstack/nova master: hyper-v: update support matrix  https://review.opendev.org/71123413:44
*** rpittau|bbl is now known as rpittau13:44
openstackgerritLucian Petrut proposed openstack/nova master: hyper-v: update support matrix  https://review.opendev.org/71123413:45
*** amoralej|lunch is now known as amoralej13:53
*** Liang__ is now known as LiangFang13:56
*** mriedem has joined #openstack-nova13:58
*** mkrai has joined #openstack-nova14:02
*** lbragstad has joined #openstack-nova14:03
*** rishabhhpe has joined #openstack-nova14:05
rishabhhpeHello All, When i am creating instance on a VM which is hosted on nodepool VM .. its not able to ping the DHCP IP and even gateway .. is there any way i can troubleshoot it .. i tried with kernel bridge but it is not working .. is there workarund for this ? as my tempest is failing due to this14:07
openstackgerritBrin Zhang proposed openstack/nova master: Add new policy to PATCH update volume API  https://review.opendev.org/71119414:09
openstackgerritBrin Zhang proposed openstack/nova master: Add functional tests for PATCH volume attachments API  https://review.opendev.org/71096514:10
*** arxcruz is now known as arxcruz|rover14:13
*** sapd1 has joined #openstack-nova14:14
*** psachin has joined #openstack-nova14:16
*** ccamacho has quit IRC14:18
*** nweinber has joined #openstack-nova14:18
*** ccamacho has joined #openstack-nova14:20
*** ccamacho has quit IRC14:20
*** ccamacho has joined #openstack-nova14:23
*** spatel has joined #openstack-nova14:24
gibidansmith: I reviewd the first 3 cyborg integration patch. I still have concerns14:25
dansmithgibi: ack was just catching up14:26
gibiand I have to leave for today14:26
gibiwill be off tomorrow but I will check back on Firday14:27
*** yedongcan has left #openstack-nova14:28
gibiI still left some comments and questions14:28
lyarwoodstephenfin: https://review.opendev.org/#/c/666915/ - did you see my review here btw, I think it's just a docs nit really but I might be missing something.14:28
*** spatel has quit IRC14:29
redrobotHi Nova friends!  I'm still looking for one more review on https://review.opendev.org/#/c/680562/14:36
lyarwoodefried: https://review.opendev.org/#/c/669121/ - blast from the past but this should be good to land now if you have time to review again.14:38
*** ociuhandu has quit IRC14:40
rishabhhpeHello All, can anyone please help me on above problem i raised14:40
lyarwoodredrobot: The people in this channel don't directly control that project, #puppet-openstack is where you will find them. I've +1'd anyway.14:41
redrobotgot it, thanks lyarwood!14:41
*** links has quit IRC14:45
*** dtantsur is now known as dtantsur|brb14:46
*** nicolasbock has quit IRC14:51
*** spatel has joined #openstack-nova14:53
*** mkrai has quit IRC14:54
*** mmethot has quit IRC14:54
*** priteau has joined #openstack-nova14:55
*** david-lyle is now known as dklyle14:57
openstackgerritLee Yarwood proposed openstack/nova master: DNM - Test TEMPEST_EXTEND_ATTACHED_ENCRYPTED_VOLUME  https://review.opendev.org/70759314:58
*** udesale_ has joined #openstack-nova14:59
*** psachin has quit IRC15:00
*** udesale has quit IRC15:02
*** zhanglong has quit IRC15:03
*** nicolasbock has joined #openstack-nova15:04
*** nicolasbock has quit IRC15:10
*** eharney has quit IRC15:10
*** ociuhandu has joined #openstack-nova15:13
*** ociuhandu has quit IRC15:18
*** lpetrut has quit IRC15:18
lyarwoodhttps://review.opendev.org/#/c/711211/ - any non-RH cores able to review this fix, appears we are randomly hitting it in the gate at the moment.15:25
stephenfinlyarwood: yup, just hadn't gotten to it now. Will do so before EOD15:26
stephenfin*to it before now15:27
lyarwoodstephenfin: ack np15:27
openstackgerritStephen Finucane proposed openstack/nova stable/train: Functional test for UnexpectedDeletingTaskStateError  https://review.opendev.org/71121015:27
openstackgerritStephen Finucane proposed openstack/nova stable/train: Unplug VIFs as part of cleanup of networks  https://review.opendev.org/71125115:27
stephenfinmeanwhile, care to review these incoming patches? Have the affected functional and unit tests passing now15:27
stephenfin*those (bad timing)15:27
lyarwoodyup of course15:28
efriedlyarwood: If you want to tweak the commit message, I'll fast approve. If you want to twiddle some of those other things at the same time, ditto (none of them impact logic).15:35
efriedlyarwood: gate fix +A15:37
*** kukacz_ has joined #openstack-nova15:38
lyarwoodefried: thanks, I'll respin the LUKS compat change now.15:38
efriedlyarwood: tbc, the only thing that *needs* to be fixed in the commit message is the name of the renamed helper function.15:40
*** kaisers has quit IRC15:40
*** psachin has joined #openstack-nova15:40
* lyarwood is back with tea15:49
lyarwoodefried: ack I'll just update that quickly then15:49
efriedlyarwood: ight. The other things (renames of test cases) could be done in a fup15:50
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Remove native LUKS compat code  https://review.opendev.org/66912115:50
efried+A15:50
*** sapd1 has quit IRC15:52
*** eharney has joined #openstack-nova15:52
*** psachin has quit IRC15:56
*** ociuhandu has joined #openstack-nova15:56
*** mkrai has joined #openstack-nova15:56
*** spatel has quit IRC15:57
*** dtantsur|brb is now known as dtantsur16:00
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: FUP I41b7c1653c6a887ee4b08e588c5d422409aebfba  https://review.opendev.org/71126216:04
lyarwoodefried: ack thanks again16:04
*** mlavalle has joined #openstack-nova16:04
*** psachin has joined #openstack-nova16:05
kukacz_hello, any idea what might be wrong with instance being unshelved on a host ignoring AggregateInstanceExtraSpecsFilter ? it seems to only happen to instances created before the filtering params were applied on instance types.16:09
*** ociuhandu has quit IRC16:11
*** ociuhandu has joined #openstack-nova16:16
*** amodi has quit IRC16:18
dansmithkukacz_: yeah, that's expected16:19
dansmithkukacz_: when you boot an instance the flavor is crystalized into the instance to insulate it from further changes to the flavor16:19
*** iurygregory has quit IRC16:20
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Parse the 'os' element from domainCapabilities  https://review.opendev.org/67379016:23
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Add methods to detect firmware auto-selection and SB  https://review.opendev.org/68262716:23
kukacz_dansmith: aha. and is there some method how could I push flavor params to existing instances too?16:25
dansmithkukacz_: nope16:25
dansmithkukacz_: you could snapshot and then re-deploy them with the new flavor, but that's about it16:26
kukacz_dansmith: hmm, that's bad. then it seems I don't have a method to safely prevent non-ssd instances being scheduled on ssd node - thinking the classic example used in Nova docs for ages: https://docs.openstack.org/nova/queens/admin/configuration/schedulers.html16:31
dansmithkukacz_: nova is optimized for cattle, not pets16:31
dansmithif you're on something new enough you could potentially use the new isolated aggregates thing for your ssd aggregate16:32
kukacz_dansmith: just for my curiosity - how does the pets/cattle apply in this case?16:34
dansmithkukacz: it's a pet if you can't shoot it in the head and boot another one in its place16:34
dansmithhttps://docs.openstack.org/nova/latest/reference/isolate-aggregates.html16:35
*** waverider has joined #openstack-nova16:37
dansmithalso if you're willing to shelve/unshelve, you could just resize those instances to a flavor that has the new key in it16:37
kukacz_dansmith: unfortunately, this is queens. isolated aggregates seem to be from rocky16:38
dansmithtrain actually16:39
kukacz_dansmith: yes, the resize method is something I was just thinking of. probably a way to go. but it's not my workload. I'll have to ask many customers to follow that rule16:39
dansmithyup16:40
*** gyee has joined #openstack-nova16:40
*** udesale_ has quit IRC16:41
*** ociuhandu has quit IRC16:41
*** tbachman has quit IRC16:48
*** waverider has quit IRC16:49
kukacz_dansmith: one more detail - how is the original set of flavor parameters stored for each instance? in nova.instances table I see instance_type_id pointing at the current flavor. it must be something else, I guess ...16:50
*** maciejjozefczyk_ has joined #openstack-nova16:51
dansmithkukacz_: it's a serialized json blob in instance_extra.flavor16:51
kukacz_dansmith: aha!16:51
*** ociuhandu has joined #openstack-nova16:51
kukacz_dansmith: thank you a lot for all the clarification!16:51
*** maciejjozefczyk has quit IRC16:52
dansmithkukacz_: how come you're willing to shelve something but not resize it? resize should be faster and less disturbing to the workload16:52
*** psachin has quit IRC16:53
kukacz_dansmith: it's not problem of willingness to resize. it's just that our customers use shelving to pause billing on temporarily stopped instances. now we'll have to explain them, that they'll also need to issue resize call to avoid their instance end up running on hosts where we don't want them to be16:54
dansmithoh okay I thought maybe you were shelving proactively16:55
*** maciejjozefczyk_ has quit IRC16:56
openstackgerritLee Yarwood proposed openstack/nova stable/train: libvirt: Ignore DiskNotFound during update_available_resource  https://review.opendev.org/71127616:56
openstackgerritLee Yarwood proposed openstack/nova stable/stein: libvirt: Ignore DiskNotFound during update_available_resource  https://review.opendev.org/71127716:57
openstackgerritLee Yarwood proposed openstack/nova stable/rocky: libvirt: Ignore DiskNotFound during update_available_resource  https://review.opendev.org/71127816:57
openstackgerritLee Yarwood proposed openstack/nova stable/queens: libvirt: Ignore DiskNotFound during update_available_resource  https://review.opendev.org/71127916:58
melwittstephenfin: what made you think https://review.opendev.org/711093 is blocking the gate?16:58
melwittit was just a heads up about an upcoming requirements update and should get normal review16:58
stephenfinmelwitt: We're referencing a module that's been deleted so we're broken with the latest version of oslo.db, no?16:58
stephenfinah, crap. I didn't realize it was pre-emptive16:59
stephenfinI'll pull it out of the queue16:59
melwittyeah but I don't think the change to the latest happened yet, it's proposed. this is a standard thing that happens when requirements team looks to bump things16:59
openstackgerritStephen Finucane proposed openstack/nova master: Remove oslo_db.sqlalchemy.compat reference  https://review.opendev.org/71109316:59
kukacz_dansmith: no, as a service provider I mostly cannot touch customer instances, they manipulate them themselves16:59
stephenfinmelwitt: Gotcha. Yeah, sorry, missed that. Dragged it out ^17:00
*** mkrai has quit IRC17:04
*** damien_r has quit IRC17:11
*** spatel has joined #openstack-nova17:17
*** openstackgerrit has quit IRC17:20
*** igordc has joined #openstack-nova17:33
*** evrardjp has quit IRC17:35
*** evrardjp has joined #openstack-nova17:35
*** openstackgerrit has joined #openstack-nova17:41
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Use domain capabilities to get supported device models  https://review.opendev.org/66691517:41
openstackgerritStephen Finucane proposed openstack/nova master: Add transform_image_metadata request filter  https://review.opendev.org/66577517:41
*** dtantsur is now known as dtantsur|afk17:41
*** rpittau is now known as rpittau|afk17:44
*** amoralej is now known as amoralej|off17:51
*** ociuhandu_ has joined #openstack-nova17:54
*** ociuhandu has quit IRC17:57
*** ociuhandu_ has quit IRC17:58
*** priteau has quit IRC17:59
*** derekh has quit IRC18:01
*** igordc has quit IRC18:04
*** martinkennelly has quit IRC18:04
*** igordc has joined #openstack-nova18:05
*** ozzzo has joined #openstack-nova18:07
*** tbachman has joined #openstack-nova18:10
*** nicolasbock has joined #openstack-nova18:24
*** igordc has quit IRC18:27
*** nicolasbock has quit IRC18:29
*** nicolasbock has joined #openstack-nova18:30
*** igordc has joined #openstack-nova18:33
*** tosky has quit IRC18:44
*** nweinber_ has joined #openstack-nova18:45
*** nweinber has quit IRC18:48
*** tesseract has quit IRC18:55
ozzzocan anyone help me set set pointer_model to None in nova.conf?18:56
ozzzoit appears to be a bug; I opened https://bugs.launchpad.net/nova/+bug/186610618:57
openstackLaunchpad bug 1866106 in OpenStack Compute (nova) "Can't set "pointer_model = None" in nova.conf" [Undecided,New]18:57
ozzzothe help text in nova.conf says to use None or <None> but nova errors on both18:57
ozzzoERROR nova ConfigFileValueError: Value for option pointer_model is not valid: Valid values are [None, ps2mouse, usbtablet], but found 'None'18:57
ozzzoI tried leaving it blank but that fails also18:57
ozzzothe underlying issue is that I need to not load the USB and PS2 drivers to build large-RAM Centos VMs, because Centos chokes while loading drivers on VMs with over 1064304G RAM18:58
*** damien_r has joined #openstack-nova19:02
*** damien_r has quit IRC19:02
*** damien_r has joined #openstack-nova19:03
*** damien_r has quit IRC19:03
*** damien_r has joined #openstack-nova19:03
melwittozzzo: I agree that looks like a bug. the choice should instead be 'none' (I see several other examples, especially in nova/conf/libvirt.py) and handled accordingly in the code. I think the StrOpt won't ever recognize None as a non-string 'None'19:06
ozzzook trying none19:07
melwittozzzo: I mean I think you'll need a patch to do it19:08
ozzzooic ok19:08
melwittit needs to be fixed in order for you to choose none19:08
melwittlemme see if I can propose one real quick for you19:10
ozzzook cool thank you!19:10
melwittthanks for opening the bug and letting us know19:10
ozzzono problem, my users are screaming for giant memory Centos VMs19:11
mriedemthe config is coercing None to 'None'19:11
ozzzoI'm trying to get them to use Ironic instead but they don't want to19:11
mriedemseems like that might be a bug in oslo.config rather than nova19:11
ozzzoshould I ask in their IRC channel?19:13
bnemecThe only way for a config opt to have a None value is for that to be the default and for the opt to be unset.19:13
bnemecSo completely absent from the file, not something like "opt="19:14
mriedembingo https://github.com/openstack/oslo.config/blob/master/oslo_config/types.py#L14219:14
mriedemyeah the default is usbtablet19:14
mriedemso you can't unset it to avoid the default19:14
ozzzoam I on the right track; will setting this to none stop the USB driver from loading?19:14
mriedemso you're looking at what? default being None, but then checking to see if the option is explicitly set to None or default to 'usbtablet' if not set to None explicitly?19:15
ozzzoor is there a better way to not load USB?19:15
mriedembnemec: couldn't oslo.config just fix this line? https://github.com/openstack/oslo.config/blob/master/oslo_config/types.py#L14219:16
mriedemif value is None and None is a valid choice and not the default, then don't coerce to 'None'19:16
bnemecI don't think it would help. I'm pretty sure the argument parser will never return None, it will return a string "None" anyway.19:18
*** rishabhhpe has quit IRC19:18
bnemecI actually think the cast is unnecessary because I'm pretty sure the raw values we get are all strings to begin with.19:18
bnemecI could be wrong though. It's been a while since I've dug into the internals of oslo.config.19:19
*** tbachman has quit IRC19:19
mriedemyeah maybe19:19
*** larainema has quit IRC19:20
mriedemguess you could hack a new choice "unset" or something to mean None19:20
ozzzomy actual goal is to not load the USB and PS2 drivers. Is there a better way?19:22
mriedemno, that's why i'm saying it sounds like you need a new choice on that option19:23
*** tbachman has joined #openstack-nova19:25
mriedemi think that would basically take care of itself here https://github.com/openstack/nova/blob/757fc03b78d542e7262343b65eacea02ce11dd04/nova/virt/libvirt/driver.py#L594919:26
mriedemthat method would return None so a pointer device wouldn't be set in the guest xml https://github.com/openstack/nova/blob/757fc03b78d542e7262343b65eacea02ce11dd04/nova/virt/libvirt/driver.py#L570019:27
*** mgoddard has quit IRC19:27
mriedema choice of 'none' like melwitt suggested would do it19:27
openstackgerritMerged openstack/nova master: Add test coverage of existing os-console-auth-tokens policies  https://review.opendev.org/70668719:28
*** mgoddard has joined #openstack-nova19:28
mordredmriedem: but if you don't pick usb or ps2, how will your keyboard work?19:29
melwittI'm working on a patch now. writing up the commit message19:29
dansmithmordred: ten digit rotary phone dial19:31
ozzzothese giant-memory VMs are running batch jobs; I may be wrong but I think we can live without a keyboard19:31
dansmithmriedem: hack in the characters via their decimal ascii equivalent... duh.19:31
dansmither, mordred ^ :P19:32
mriedemcripes19:32
openstackgerritmelanie witt proposed openstack/nova master: Fix CONF.pointer_model choice None to use 'none' for StrOpt  https://review.opendev.org/71131519:34
openstackgerritGhanshyam Mann proposed openstack/nova master: Introduce scope_types in os-console-auth-tokens  https://review.opendev.org/70668819:38
*** ralonsoh has quit IRC19:38
melwittgmann: hey I was wondering if you saw johnthetubaguy comment on this patch https://review.opendev.org/705127 I couldn't really follow it19:39
gmannmelwitt: yeah, i can move those to deprecated also, i did not do initially as those were ~3 years old and users might have switched to new one.19:41
gmannupdating..19:41
melwittgmann: ok. since I didn't understand it I didn't agree or disagree, but wondered what you thought about it19:42
*** iurygregory has joined #openstack-nova19:46
gmannwhile new policy were introduced, we did not remove those old policy from enforcement from our API so operators might be relying on those old policies default or override.19:48
gmannkeeping them for deprecation phase does not harm much.19:48
*** nweinber__ has joined #openstack-nova19:49
*** nweinber_ has quit IRC19:52
*** damien_r has quit IRC19:53
*** spatel has quit IRC19:58
melwittozzzo, mriedem: I'm weirdly noticing in the code that setting pointer_model='ps2mouse' looks like it would have the same effect (nothing added to libvirt xml) as setting None if setting None were possible20:01
mriedemi was wondering about that too20:02
ozzzointeresting; I'll try it and then look at the XML20:02
mriedemthat pointer_model option was meant to replace use_usb_tablet back in newton but it was never fully removed because the upgrade compat path was really complicated (i tried and abandoned the attempt)20:02
mriedemi wonder if there was a bad refactor at some point that broke the ps2mouse support20:03
melwittI was wondering the same thing20:03
mriedemnote that devstack sets it to ps2mouse explicitly https://github.com/openstack/devstack/blob/93d22d82988ee781feddf84535d65adf44472379/lib/nova_plugins/hypervisor-libvirt#L4420:03
mriedemso you could just check one of the gate job logs for a guest xml to see if the pointer device is not set20:04
melwittoh nice20:04
*** CeeMac has quit IRC20:06
mriedemcode looks basically the same since newton https://github.com/openstack/nova/blob/newton-eol/nova/virt/libvirt/driver.py#L467520:07
melwitthere's a guest xml https://zuul.opendev.org/t/openstack/build/fcc8380b76924d0ca8acf158f7571ad8/log/controller/logs/screen-n-cpu.txt#188820:07
*** CeeMac has joined #openstack-nova20:07
ozzzoI see the USB lines missing from the XML now, but I still see the PS2 mouse and keyboard lines. How can I get rid of those?20:09
mriedemmelwitt: i don't see any input devices in there20:11
mriedemso seems that ps2mouse is doing the deed20:11
melwittyeah I don't either20:11
mriedemozzzo: you set pointer_model=ps2mouse on your computes?20:11
ozzzoonly on the HV20:12
mriedemHV = nova-compute nodes?20:12
ozzzo yes20:12
mriedemrestarted the service and booted a new vm on that node?20:12
ozzzojust the one I'm testing20:12
ozzzono, I stopped and started the VM, but that rewrites the XML20:12
ozzzodo I need to delete it and build a new one?20:12
mriedemyou have to restart the compute service to refresh the config20:12
mriedemthen you could reboot the vm20:13
melwittozzzo: that won't refresh the nova config though20:13
ozzzoI stop/started nova-compute on the compute node20:13
melwittthat reminds me, did SIGHUP ever get fixed?20:13
ozzzothen stop/started the VM20:13
melwittoh ok20:13
mriedempaste your guest xml i guess20:13
mriedemin a paste/gist20:13
melwittwell, I'm wondering if this is what the config help is describing "'Uses default behavior provided by drivers (mouse on PS2 for libvirt x86)"20:14
mriedembut we're running libvirt x86 in the gate20:14
melwitt... true20:14
melwitthm what the heck20:14
mriedemunless y'all moved over to ARM in the last few months20:15
ozzzohttp://paste.openstack.org/show/790313/20:15
*** eharney has quit IRC20:16
mriedemhuh yeah - which version of libvirt?20:16
*** tbachman has quit IRC20:19
melwittlooks like usb tablet is the only input device we're capable of adding https://github.com/openstack/nova/blob/master/nova/virt/libvirt/config.py#L180220:22
melwittI mean, I guess we already knew that but there's where all the configs are20:23
*** tbachman has joined #openstack-nova20:25
melwitthere's danpb stating "We don't specify anything actually. QEMU always gives you a ps2 keyboard + mouse no matter what & AFAIK, there's no way to disable that.  Any other mouse should override the PS2 mouse, provided the guest OS actually initializes the non-ps2 mouse." https://bugzilla.redhat.com/show_bug.cgi?id=953064#c420:25
openstackbugzilla.redhat.com bug 953064 in libvirt "allow disabling ps2 mouse, it overrides tablet on macosx guests" [Medium,Closed: deferred] - Assigned to libvirt-maint20:25
*** spatel has joined #openstack-nova20:26
mriedemok so ozzzo's bug is invalid / won't fix, just need to set pointer_model=ps2mouse20:27
mriedemand remove that None option as a choice since it's invalid - i guess that's the bug20:27
melwittyeah, it's weird. None is supposed to mean "use the libvirt defaults" which it is doing20:28
melwittand setting ps2mouse does the same thing, uses the defaults. almost seems like the choices should be 'usbtablet' and 'default' instead or something20:29
melwittI mean, if None worked, it would be doing what it says, using the defaults20:30
ozzzoso it's not possible to not load the PS2 driver, and that has nothing to do with openstack; it is a qemu limitation?20:30
melwittlooks like it yeah, according to that libvirt bz20:30
*** spatel has quit IRC20:31
*** abaindur has joined #openstack-nova20:32
ozzzook I'll talk to the guys in #centos again I guess; they are telling me to not load the USB and PS2 drivers because it appears that they choke on large RAM20:34
melwittyeah ... we need to chat with a libvirt/qemu person I think too20:35
ozzzothanks for your help and advice!20:35
melwitthttps://libvirt.org/contact.html#irc20:36
melwittthey have a mailing list too20:36
ozzzook I'll try them, ty20:37
*** iurygregory has quit IRC20:43
openstackgerritMerged openstack/nova master: Pass the actual target in os-availability-zone policy  https://review.opendev.org/70669121:00
openstackgerritMerged openstack/nova master: Fix os-os-deferred-delete policy to be admin_or_owner  https://review.opendev.org/70745721:01
openstackgerritMerged openstack/nova master: Add test coverage of existing deferred_delete policies  https://review.opendev.org/70745521:01
melwittozzzo: this might be totally useless but ... I googled around out of curiosity and found this reddit post https://www.reddit.com/r/VFIO/comments/8qo468/stuttering_caused_by_evdev_with_ps2_input_devices/ where they changed their keyboard and mouse bus to 'virtio' and installed the necessary drivers. they're calling this "passing through the mouse and keyboard". and I saw that as of libvirt 1.3.0, 'virtio' is an option for input device21:05
melwitt bus https://libvirt.org/formatdomain.html#elementsInput21:05
*** ociuhandu has joined #openstack-nova21:06
*** tbachman has quit IRC21:06
melwittI would probably try something like that in a test/lab environment to see if it helps (if you don't find any other info when you inquire) by making manual updates to the xml. but I really don't know what I'm talking about so you should probably ignore me. this problem is interesting21:09
*** ociuhandu has quit IRC21:10
openstackgerritGhanshyam Mann proposed openstack/nova master: Add test coverage of existing attach_interfaces policies  https://review.opendev.org/70512621:11
*** kukacz_ has quit IRC21:17
openstackgerritGhanshyam Mann proposed openstack/nova master: Introduce scope_types in os-attach-interfaces  https://review.opendev.org/70579921:18
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in os-atttach-inerfaces policies  https://review.opendev.org/70667221:18
*** tbachman has joined #openstack-nova21:21
*** nweinber__ has quit IRC21:27
*** KeithMnemonic has joined #openstack-nova21:37
*** bbowen_ has quit IRC21:46
*** tbachman has quit IRC21:46
*** bbowen_ has joined #openstack-nova21:46
openstackgerritArtom Lifshitz proposed openstack/nova master: func tests: move _run_periodics() into own mixin  https://review.opendev.org/70554521:50
openstackgerritArtom Lifshitz proposed openstack/nova master: tests: work around malformed serial XML  https://review.opendev.org/70554621:50
openstackgerritArtom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration  https://review.opendev.org/67259521:50
openstackgerritArtom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146  https://review.opendev.org/68740421:50
openstackbug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms)21:50
*** eharney has joined #openstack-nova21:52
openstackgerritGhanshyam Mann proposed openstack/nova master: Introduce scope_types in os-deferred_delete  https://review.opendev.org/70747622:00
*** imacdonn has quit IRC22:04
*** imacdonn has joined #openstack-nova22:04
*** tosky has joined #openstack-nova22:11
*** N3l1x has quit IRC22:21
*** slaweq has quit IRC22:21
*** nicolasbock has quit IRC22:35
*** rcernin has joined #openstack-nova22:39
openstackgerritmelanie witt proposed openstack/nova master: Remove broken CONF.pointer_model choice None  https://review.opendev.org/71131522:43
*** tkajinam has joined #openstack-nova22:59
ozzzomelwitt: that might help, I'll take a look23:06
*** mriedem has left #openstack-nova23:10
*** slaweq has joined #openstack-nova23:11
*** slaweq has quit IRC23:16
*** mlavalle has quit IRC23:16
ozzzoreading further into that thread; they are repeating what was said here: "It doesn't seem possible to disable them QEMU side."23:20
melwittyeah23:20
melwittI found a few other threads in various places too while googling, saying the same23:21
melwittso I got curious how people get around the perf issues that seem to crop up23:25
*** threestrands has joined #openstack-nova23:28
openstackgerritMerged openstack/nova stable/train: libvirt: avoid cpu check at s390x arch  https://review.opendev.org/70152523:33
openstackgerritMerged openstack/nova master: functional: Avoid race and fix use of self.api within test_bug_1831771  https://review.opendev.org/71121123:33
*** tosky has quit IRC23:40
*** brinzhang has joined #openstack-nova23:41
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in os-atttach-inerfaces policies  https://review.opendev.org/70667223:47
brinzhanggmann: https://review.opendev.org/#/c/706672/3/nova/tests/unit/policies/test_attach_interfaces.py@8523:54
openstackgerritEric Xie proposed openstack/nova master: Catch exception when use invalid architecture of image  https://review.opendev.org/71136323:55
*** tbachman has joined #openstack-nova23:55
brinzhanggmann: why the os-volume-attachments policy can unauthorized the self.project_foo_context context? that you can review https://review.opendev.org/#/c/710190/3/nova/tests/unit/policies/test_volumes.py@13123:55
*** dklyle has quit IRC23:56
gmannbrinzhang: just replied on review for instance action policy23:56
gmanni will check volume policy tomorrow.23:57
brinzhanggmann: thanks, I will look23:57
gmannbecause of deprecation rules, we do not see the behavior of new defaults when we stop supporting the old one.23:58
gmannalso we cannot remove the old rules otherwise it break the deployment.23:59
brinzhanggmann: sure23:59

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