Monday, 2019-11-18

*** Liang__ has joined #openstack-nova00:04
*** Liang__ has quit IRC00:23
*** ivve has quit IRC00:33
*** brinzhang has joined #openstack-nova01:08
*** ociuhandu has joined #openstack-nova01:31
*** chenhaw has joined #openstack-nova01:35
*** ociuhandu has quit IRC01:41
*** macz has joined #openstack-nova02:05
*** alex_xu has joined #openstack-nova02:11
*** zhanglong has joined #openstack-nova02:24
*** ociuhandu has joined #openstack-nova02:26
*** ociuhandu has quit IRC02:30
*** brinzhang_ has joined #openstack-nova02:47
*** brinzhang has quit IRC02:50
*** macz has quit IRC02:51
*** ociuhandu has joined #openstack-nova03:00
*** ociuhandu has quit IRC03:07
*** artom has quit IRC03:33
*** artom has joined #openstack-nova03:34
*** mkrai has joined #openstack-nova03:34
*** mkrai_ has joined #openstack-nova03:40
*** mkrai has quit IRC03:41
*** udesale has joined #openstack-nova04:02
*** davee__ has joined #openstack-nova04:03
*** davee_ has quit IRC04:04
*** bhagyashris has joined #openstack-nova04:20
*** ociuhandu has joined #openstack-nova04:27
*** ociuhandu has quit IRC04:29
*** ociuhandu has joined #openstack-nova04:30
*** ociuhandu has quit IRC04:36
*** bhagyashris has quit IRC04:45
*** abhishekk has joined #openstack-nova04:51
*** ociuhandu has joined #openstack-nova04:53
*** ociuhandu has quit IRC04:58
*** ociuhandu has joined #openstack-nova04:58
*** mkrai_ has quit IRC04:59
*** bhagyashris has joined #openstack-nova05:03
*** ociuhandu has quit IRC05:03
*** mkrai has joined #openstack-nova05:11
*** mkrai has quit IRC05:14
*** mkrai has joined #openstack-nova05:14
*** mkrai_ has joined #openstack-nova05:22
*** ratailor has joined #openstack-nova05:23
*** links has joined #openstack-nova05:24
*** mkrai has quit IRC05:24
*** bhagyashris has quit IRC05:27
*** ileixe has quit IRC05:29
*** ociuhandu has joined #openstack-nova05:30
*** bhagyashris has joined #openstack-nova05:31
*** ileixe has joined #openstack-nova05:32
*** ociuhandu has quit IRC05:35
*** ircuser-1 has quit IRC05:35
*** cp has joined #openstack-nova05:36
*** slaweq has joined #openstack-nova05:48
*** huaqiang has quit IRC05:49
*** brinzhang has joined #openstack-nova05:54
*** slaweq has quit IRC05:57
*** brinzhang_ has quit IRC05:58
*** abaindur has joined #openstack-nova05:58
*** abaindur has quit IRC06:05
*** Luzi has joined #openstack-nova06:11
*** awalende has joined #openstack-nova06:16
*** awalende has quit IRC06:20
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.opendev.org/69471706:21
*** bhagyashris has quit IRC06:45
*** ileixe has quit IRC06:51
*** ileixe has joined #openstack-nova06:52
*** bhagyashris has joined #openstack-nova06:52
*** abaindur has joined #openstack-nova07:01
*** ociuhandu has joined #openstack-nova07:02
*** rcernin has quit IRC07:04
*** ociuhandu has quit IRC07:06
*** abaindur has quit IRC07:07
*** rpittau|afk is now known as rpittau07:28
*** dpawlik has joined #openstack-nova07:34
*** maciejjozefczyk has joined #openstack-nova07:36
*** slaweq has joined #openstack-nova07:38
*** damien_r has joined #openstack-nova07:57
*** igordc has joined #openstack-nova07:58
*** tkajinam has quit IRC08:02
*** igordc has quit IRC08:02
*** mkrai_ has quit IRC08:06
*** luksky has joined #openstack-nova08:08
*** tesseract has joined #openstack-nova08:15
*** ociuhandu has joined #openstack-nova08:22
*** ociuhandu has quit IRC08:29
bauzasgood morning Nova08:36
*** ratailor_ has joined #openstack-nova08:40
*** mkrai has joined #openstack-nova08:42
*** ratailor has quit IRC08:42
*** ivve has joined #openstack-nova08:46
gibibauzas: good morning08:47
bauzas:)08:47
bauzasta, you too08:47
*** jaosorior has joined #openstack-nova08:48
*** ralonsoh has joined #openstack-nova08:49
*** mkrai has quit IRC08:50
*** mkrai has joined #openstack-nova08:51
*** xek has joined #openstack-nova08:59
*** ociuhandu has joined #openstack-nova09:04
*** martinkennelly has joined #openstack-nova09:05
*** nanzha has joined #openstack-nova09:10
*** awalende has joined #openstack-nova09:16
*** awalende has quit IRC09:18
*** awalende has joined #openstack-nova09:26
brinzhanggood morning :)09:29
*** awalende has quit IRC09:30
*** awalende has joined #openstack-nova09:36
*** awalende has quit IRC09:37
*** ileixe has quit IRC09:40
*** ratailor__ has joined #openstack-nova09:41
*** ileixe has joined #openstack-nova09:41
*** mkrai has quit IRC09:42
* kashyap goes to update the libvirt distro support matrix ... and then version bump and clean-up ...09:42
*** ratailor_ has quit IRC09:43
bauzasbrinzhang: heh, good morning !09:49
*** brinzhang_ has joined #openstack-nova09:52
*** mkrai has joined #openstack-nova09:54
*** brinzhang has quit IRC09:55
brinzhang_bauzas: gibi: I was updated the flavor explosion spec after PTG, and base gibi's comments updated again, please review, Thanks :P09:56
*** mkrai has quit IRC09:56
*** ociuhandu has quit IRC09:57
*** mkrai has joined #openstack-nova09:57
brinzhang_bauzas: gibi: spec https://review.opendev.org/#/c/663563/09:57
bauzascool, I'll try09:57
*** ociuhandu has joined #openstack-nova09:58
*** ociuhandu has quit IRC10:03
*** kaisers has quit IRC10:04
*** lennyb has quit IRC10:06
*** zhanglong has quit IRC10:06
*** mkrai has quit IRC10:07
*** bhagyashris has quit IRC10:10
*** ileixe has quit IRC10:10
*** pcaruana has joined #openstack-nova10:10
*** nanzha has quit IRC10:11
*** ileixe has joined #openstack-nova10:11
*** nanzha has joined #openstack-nova10:12
*** owalsh has quit IRC10:12
*** abhishekk has quit IRC10:12
*** bhagyashris has joined #openstack-nova10:12
*** brinzhang_ has quit IRC10:13
gibiack10:13
kashyapaspiers: If you're about, want to update the SLES versions here? -- https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix#Distro_minimum_versions10:16
*** kaisers has joined #openstack-nova10:20
*** owalsh has joined #openstack-nova10:22
openstackgerritpangliye proposed openstack/nova master: Fix misspell word  https://review.opendev.org/69474110:29
openstackgerritStephen Finucane proposed openstack/nova master: Stop testing Python 2  https://review.opendev.org/68795410:32
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Make functional job inherit from openstack parents  https://review.opendev.org/68842510:32
kashyapAnyone familiar with Debian here: why was "sid" (the 'unstable' version) chcked here? -- https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix10:33
kashyapzigo: ^10:34
bauzaskashyap: ask zigo10:34
bauzasheh10:34
kashyap:-)10:36
kashyap"sid" is the unstable edition; normally I'd check the "stable" version to be there10:36
kashyapDoes Infra gate uses Debian "sid"?  Maybe it's for #openstack-infra10:36
zigokashyap: This looks wrong in many ways, for example, the qemu version ...10:37
kashyapzigo: Yeah!  I'm right now undoing the mess, whoever did that10:37
*** CeeMac has joined #openstack-nova10:37
kashyapzigo: So, to be clear, these are the versions from 'stable':10:37
kashyap- libvirt: 5.0.0-410:37
zigokashyap: In buster, we have qemu 1:3.1+dfsg-8+deb10u310:38
zigo(ie: 3.1)10:38
kashyap- QEMU: 3.110:38
kashyapzigo: Indeed10:38
zigolibvirt 5.0.010:38
kashyapzigo: BTW, the version you see there for QEMU was checked on 2018-APR-05 -- so at _that_ time Buster indeed had 2.11 QEMU10:39
zigokashyap: I believe it's just outdated ...10:39
kashyapSo that's correct based on what was there at that time10:39
zigoRight.10:39
kashyapYes, it is outdated.  Updating these MediaWiki pages are a *pain* in the neck10:39
zigoYeah, the syntax is ... hum... suboptimal, to use nice words! :)10:40
kashyapzigo: Refresh the page, and check if the Buster ("stable") looks correct now10:41
zigokashyap: Well, it looks ok to me, though instead of Buster ("sid")  you may want to write Bullseye ("sid") and write what's there.10:42
kashyapzigo: Right, let me do that10:43
kashyapI'll update the Buster ("sid") --> Bullseye ("sid")10:44
kashyapzigo: Am I correct in saying that what's at top-left (the 4.1.1) indicates the Bullseye ("sid") version? -- https://tracker.debian.org/pkg/qemu10:45
kashyapAlso what does the "1:" mean?10:45
zigokashyap: That's an "EPOCH".... basically, this is for when we do a mistake with the upstream version, and need to "upgrade" to a lower version.10:46
zigoSo, that's distro specific ...10:46
kashyapAh, right.  The same concept exists in Fedora.10:46
zigo1:2.3.4 is higher that 2.3.510:46
kashyapMuch frowned upon, though. :-)10:46
kashyapRight.10:47
zigoWe try to avoid it as much as possible, because that's very confusing when writting dependency, and a mistake may happen very fast...10:47
kashyapYeah, it's an emergency hammer10:47
zigoFor example, if you want version >= 2.3.5 and forget the 1:, then you get 1:2.2.4 satisfying your dependency and you're screwed.10:47
zigoDoes anyone know if Nova fixed the issue of volume live migration in Rocky, and if yes, in what point release?10:48
zigoI've tried upgrading os-brick to 2.5.8 in a compute, it didn't help...10:49
zigoI'll try nova and cinder next...10:49
kashyapzigo: Can you refresh the page to now check if Debian stuff is all kosher?10:50
zigokashyap: LGTM ! :)10:50
kashyapstephenfin: Any ideas to transfer this to a better place than MediaWiki? -- can we do proper tables in rST?10:51
kashyaphttps://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix10:51
kashyapThe MediaWiki syntax sucks10:51
kashyapWe _can_ do tables in rST; I've done it myself.  Perhaps will consider moving this page in-tree10:51
*** rcernin has joined #openstack-nova10:52
*** chenhaw has quit IRC10:59
*** ociuhandu has joined #openstack-nova11:02
openstackgerritMerged openstack/nova master: Convert legacy nova-live-migration and nova-multinode-grenade to py3  https://review.opendev.org/69237411:04
*** mkrai has joined #openstack-nova11:05
*** luksky has quit IRC11:06
zigoOk, I found it, the issue is in Nova, and fixed by the point release in Rocky.11:12
*** awalende has joined #openstack-nova11:14
*** awalende has quit IRC11:14
*** udesale has quit IRC11:17
*** awalende has joined #openstack-nova11:20
*** awalende has quit IRC11:21
*** luksky has joined #openstack-nova11:22
*** brinzhang has joined #openstack-nova11:23
stephenfinkashyap: Yeah, that probably makes sense in doc/source/reference11:37
stephenfinand you should be able to do it in tables, yup11:37
kashyapstephenfin: Thanks!  I'll get to it.11:40
*** mkrai has quit IRC11:44
*** brinzhang has quit IRC11:47
zigoDoes anyone know which commit has the cinder volume live migration fix in Rocky?11:49
*** mkrai has joined #openstack-nova11:49
lyarwoodzigo: That's pretty generic, do you have a launchpad bug refrence I could check?11:50
zigolyarwood: Basically, in 18.1.0, live migrating a cinder volume never finshes, with 18.2.3 that's fixed...11:51
zigoI don't know about any bug number though, I just tested and saw it was fixed.11:51
zigoI'm trying to lookup commits now ...11:51
lyarwoodzigo: ah retyping a volume?11:52
zigolyarwood: Just live-migrating a volume:11:53
zigoopenstack volume migrate --host <volume-hostname> <volume-id>11:53
lyarwoodzigo: I landed https://review.opendev.org/#/q/I1bdf3431bda2da98380e0dcaa9f952e6768ca3af to fix that11:53
zigoIn 18.1.0, the new volume is re-attached ...11:53
*** mkrai has quit IRC11:54
zigo... but the old one never deleted (and both new and old are still in the bad migration status)11:54
*** mkrai_ has joined #openstack-nova11:54
lyarwoodack yeah that came in with 18.2.111:54
zigolyarwood: Thanks, this looks like it.11:55
zigolyarwood: FYI, I need to have the exact patch, so I get this fast in Buster (upgrading to 18.2.3 has a lot of chances to be unfortunately rejected by the Debian stable release team...)11:55
lyarwoodhttps://review.opendev.org/#/c/657577/ is the change11:56
zigoYeah, got it, thanks a lot.11:56
*** bhagyashris has quit IRC11:56
*** bhagyashris has joined #openstack-nova11:57
*** dpawlik has quit IRC11:58
zigolyarwood: I need that one too, right?12:00
zigohttps://review.opendev.org/#/c/639331/12:00
zigoAh no, that's for tempest ...12:00
*** dpawlik has joined #openstack-nova12:01
*** brinzhang has joined #openstack-nova12:03
*** zhanglong has joined #openstack-nova12:04
lyarwoodzigo: yeah you don't technically need that, it will however ensure things are tested correctly if Debian stable has CI for OpenStack12:04
zigolyarwood: Yeah, I may also patch the tempest package... :)12:05
*** ociuhandu has quit IRC12:13
*** Liang__ has joined #openstack-nova12:15
*** brinzhang has quit IRC12:19
*** mkrai_ has quit IRC12:19
*** brinzhang has joined #openstack-nova12:20
*** bhagyashris has quit IRC12:21
openstackgerritPavel Gluschak (scsnow) proposed openstack/nova master: Don't delete compute node, when deleting service other that nova-compute  https://review.opendev.org/69475612:22
openstackgerritPavel Gluschak (scsnow) proposed openstack/nova master: Don't delete compute node, when deleting service other than nova-compute  https://review.opendev.org/69475612:27
openstackgerritPavel Gluschak (scsnow) proposed openstack/nova master: Don't delete compute node, when deleting service other than nova-compute  https://review.opendev.org/69475612:28
*** rcernin has quit IRC12:31
*** damien_r has quit IRC12:41
*** lpetrut has joined #openstack-nova12:42
*** ociuhandu has joined #openstack-nova12:43
efriedo/ nova12:43
efriedI'm going to be out today.12:43
efriedEnjoy.12:43
*** efried is now known as efried_pto12:43
*** luksky has quit IRC12:47
*** ratailor__ has quit IRC12:51
gibiefried_pto: enjoy your day too13:00
*** zhanglong has quit IRC13:02
*** ociuhandu has quit IRC13:04
*** damien_r has joined #openstack-nova13:04
*** zhanglong has joined #openstack-nova13:07
*** lennyb has joined #openstack-nova13:08
*** zhanglong has quit IRC13:13
*** sapd1 has quit IRC13:16
*** dtantsur is now known as dtantsur|bbl13:21
*** nanzha has quit IRC13:24
*** nanzha has joined #openstack-nova13:25
*** zhanglong has joined #openstack-nova13:29
*** nweinber has joined #openstack-nova13:37
*** luksky has joined #openstack-nova13:39
*** KeithMnemonic has joined #openstack-nova13:40
*** mriedem has joined #openstack-nova13:52
*** yan0s has joined #openstack-nova13:52
*** awalende has joined #openstack-nova13:55
*** zhanglong has quit IRC13:55
*** zhanglong has joined #openstack-nova13:57
*** slaweq has quit IRC14:02
*** slaweq has joined #openstack-nova14:04
*** lennyb has quit IRC14:14
mriedemi wonder, maybe we should drop using the neutron-grenade-multinode job which is 2 node grenade + smoke tests. we already have nova-grenade-multinode which is just live migration tests in both a block storage + lvm and shared storage + ceph setup. if we changed nova-grenade-multinode to run smoke tests + live migration for the block storage + lvm case, before changing the env to ceph, then we'd cover what the neutron-grenade-m14:23
mriedemnode job was doing14:23
mriedemoh nova-grenade-multinode also runs a cold migrate test since train14:24
*** awalende has quit IRC14:26
*** jbernard has quit IRC14:27
sean-k-mooneymriedem: that sounds resonable. have you looked at https://review.opendev.org/#/c/548936/ it woudl be good to eventually get that merged to have a native grenade job14:27
mriedemnope i haven't. it's huge and it scares me.14:27
*** davee__ has quit IRC14:28
*** beekneemech is now known as bnemec14:29
*** jbernard has joined #openstack-nova14:29
sean-k-mooneyit is, it does appear to work at least. that said the last ci run was july so it might not anymore14:29
*** brinzhang has quit IRC14:33
*** zhanglong has quit IRC14:34
mriedemgrenade-core includes devstack-core and the devstack / qa team have done a lot of the initial zuulv3-ification of stuff, starting with devstack and tempest, so i'll defer to them for reviewing that massive zuulv3 chnage14:34
mriedem*change14:34
openstackgerritMatt Riedemann proposed openstack/nova master: FUP to Ie1a0cbd82a617dbcc15729647218ac3e9cd0e5a9  https://review.opendev.org/69478714:38
*** jbernard_ has joined #openstack-nova14:47
openstackgerritMatt Riedemann proposed openstack/nova master: Drop neutron-grenade-multinode job  https://review.opendev.org/69478914:50
*** francoisp has joined #openstack-nova14:57
artomIf you do a Follow Up Patch to a Follow Up Patch, is it a Follow Up Patch Addendum?14:57
*** eharney has joined #openstack-nova14:57
*** Luzi has quit IRC14:59
*** tesseract has quit IRC15:01
*** tesseract has joined #openstack-nova15:01
*** dtantsur|bbl is now known as dtantsur15:01
mriedemso making nova-grenade-multinode run smoke tests adds about 14-15 minutes to the job unless we prune down the smoke tests that it runs which we might want to consider doing - to just run compute API and scenario smoke tests, though then we lose some of the coverage that neutron-grenade-multinode gave us,15:07
mriedemthough a nova change likely isn't going to break an identity/image/object/network/volume API change that doesn't use nova at all15:07
*** dpawlik has quit IRC15:14
mriedemi would need some help on the regexing though..15:19
* bauzas is having laptop issues today, sorry folks (upgrading)15:19
*** TxGirlGeek has joined #openstack-nova15:19
mriedemartom: you like a good regex test right?15:20
mriedemsay i have a test string of: tempest.api.compute.flavors.test_flavors.FlavorsV2TestJSON.test_get_flavor[id-1f12046b-753d-40d2-abb6-d8eb8b30cb2f,smoke]15:20
mriedemi want to match tempest.api.compute.* and tempest.scenario.* but only for those strings containing 'smoke'15:20
mriedemi think (tempest.api.compute.*|tempest.scenario.*)([\bsmoke\b]) might work?15:21
artommriedem, I think aspiers was the regex master, but I'll try15:21
artommriedem, that should work15:22
*** udesale has joined #openstack-nova15:22
artomI'd have done it as 'tempest\.(api\.compute|scenario)\..*smoke'15:23
mriedemyeah that seems to work as long as smoke comes last15:25
mriedemwell, i guess there is no $ on the regex so i guess it's ok15:26
artomSo 'tempest\.(api\.compute|scenario)\..*smoke.*' then :)15:26
mriedemtempest.(api.compute|scenario)\..*smoke.*15:26
*** ociuhandu has joined #openstack-nova15:26
mriedemyeah15:26
*** ociuhandu has quit IRC15:28
*** links has quit IRC15:36
sean-k-mooneybauzas: could you take a look at https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1763766 ideally im aiming to get this merged before m115:36
sean-k-mooneystephenfin: similar for https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/vm-scoped-sriov-numa-affinity if i can get that merged before m1 it would make peopel happy so if you could review it would be nice before i start asking others15:37
*** Liang__ has quit IRC15:48
*** cdent has joined #openstack-nova15:55
*** cdent has left #openstack-nova15:56
*** dklyle has quit IRC15:57
*** macz has joined #openstack-nova15:58
*** dklyle has joined #openstack-nova15:58
bauzassean-k-mooney: ack, will look15:59
*** luksky has quit IRC15:59
*** hamzy has quit IRC16:01
*** gyee has joined #openstack-nova16:02
*** mlavalle has joined #openstack-nova16:04
*** udesale has quit IRC16:06
*** jmlowe has joined #openstack-nova16:07
openstackgerritMark Goddard proposed openstack/nova master: [ironic] Fix node rebalance race issues  https://review.opendev.org/69480216:10
mriedemmgoddard: hi https://review.opendev.org/#/c/684849/16:14
kashyapCan someone update the openSUSE stuff here? -- https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix16:14
kashyap(I've updated for Debian, Ubuntu, RHEL and Fedora.)16:14
*** ociuhandu has joined #openstack-nova16:14
kashyapWho's the contact point for Oracle Linux again?16:14
mgoddardmriedem: thanks, I'll check it out16:15
artomkashyap, probably Satan16:18
* mriedem finds the code of conduct link16:19
artomDammit ><16:19
kashyapartom: Yeah, c'mon16:19
*** ociuhandu has quit IRC16:19
* artom stops, apologies16:19
mgoddardmriedem: it could be because I'm using Rocky, but I don't see any errors in RT following rebalance16:22
mriedemon either node?16:24
mriedems/node/compute service host/16:24
mgoddardyeah16:25
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests  https://review.opendev.org/68434516:27
openstackgerritStephen Finucane proposed openstack/nova master: Remove 'os-security-group-default-rules' REST API  https://review.opendev.org/68680716:27
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove unused '*_default_rules' security group DB APIs  https://review.opendev.org/68680816:27
openstackgerritStephen Finucane proposed openstack/nova master: Remove (most) '/os-networks' REST APIs  https://review.opendev.org/68680916:27
openstackgerritStephen Finucane proposed openstack/nova master: Remove '/os-tenant-networks' REST API  https://review.opendev.org/68681016:27
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove 'USE_NEUTRON' from functional tests  https://review.opendev.org/68681116:27
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove 'networks' quota  https://review.opendev.org/68681216:27
openstackgerritStephen Finucane proposed openstack/nova master: Remove nova-manage network, floating commands  https://review.opendev.org/68681316:27
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove associate, disassociate network APIs  https://review.opendev.org/68681416:27
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-dhcpbridge' binary  https://review.opendev.org/68681516:27
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-network' binary  https://review.opendev.org/68681616:27
openstackgerritStephen Finucane proposed openstack/nova master: docs: Blast most references to nova-network  https://review.opendev.org/68681716:27
openstackgerritStephen Finucane proposed openstack/nova master: WIP  https://review.opendev.org/68681816:27
mriedemmgoddard: your commit message says you have sql errors in the logs, likely due to duplicate entry errors?16:28
mgoddardmriedem: I never really got to the bottom of that one. http://paste.openstack.org/show/786272/16:29
mriedemdo you have this patch? https://review.opendev.org/#/c/676514/16:29
mriedemwhat version of rocky do you have?16:29
mgoddardyes I have that patch16:29
mgoddardI have 18.2.0, plus a few backports (including that one)16:30
*** ociuhandu has joined #openstack-nova16:30
mgoddardmriedem: I have a few things to look at here. I could try my reproducer with your patch, and try your func test with my patch16:33
mriedemyeah it would be good to know if they aren't duplicate bugs but something else16:35
openstackgerritMatt Riedemann proposed openstack/nova master: Handle NoSuchOptError in nova-manage gracefully  https://review.opendev.org/69480616:40
*** mdbooth has joined #openstack-nova16:48
*** hamzy has joined #openstack-nova16:51
mgoddardmriedem: my patch passes the func test, with two modifications16:53
mgoddard1. I didn't hit "Skipping removal of allocations for deleted instances"16:53
mgoddard2. Part of my change prevents the orphan deletion happening in that case, so I rolled that part back for testing16:54
mgoddardhttps://review.opendev.org/#/c/694802/1/nova/db/sqlalchemy/api.py16:54
*** damien_r has quit IRC16:54
mgoddard^ that prevents host2 from deleting a compute node after host1 has taken ownership of it16:54
mgoddardwhich should prevent us from hitting this issue16:55
mgoddardthe rest of the change is to help clean up if we have hit it previously16:55
*** ralonsoh has quit IRC16:58
mriedemmgoddard: ok i guess it's up to king solomon then to figure out which patch to go with16:59
mriedemor just -1 my change if you want16:59
mriedemeither way i think the functional test needs to stay b/c unit tests aren't really sufficient for this kind of multi-service and db interaction17:00
openstackgerritMatt Riedemann proposed openstack/nova master: Handle NoSuchOptError in nova-manage gracefully  https://review.opendev.org/69480617:01
*** ralonsoh has joined #openstack-nova17:01
*** yan0s has quit IRC17:02
*** luksky has joined #openstack-nova17:02
openstackgerritMatt Riedemann proposed openstack/nova master: Print help if nova-manage subcommand is not specified  https://review.opendev.org/69480617:02
*** jmlowe has quit IRC17:04
*** ociuhandu has quit IRC17:17
*** rpittau is now known as rpittau|afk17:18
*** nanzha has quit IRC17:20
*** hamzy has quit IRC17:21
*** hamzy has joined #openstack-nova17:22
*** nweinber has quit IRC17:26
*** ociuhandu has joined #openstack-nova17:28
*** ircuser-1 has joined #openstack-nova17:40
*** ociuhandu has quit IRC17:40
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: Use admin neutron client to see if instance has qos ports  https://review.opendev.org/69466817:41
artomsean-k-mooney, left you things on https://review.opendev.org/#/c/687957/517:44
sean-k-mooneyjust a couple :)17:50
sean-k-mooneyill adress them tomorrow. over all are you ok with the patch17:51
artomYeah, once we've taken image difference out of the picture, I don't see issues with not running the NUMATopologyFilter on rebuilds17:56
artom(Famous last words)17:56
artomMight be good to get jparker to look at it17:57
artomHe came up with interesting test cases for NUMA LM, he might find a way to break it :)17:57
artomNot necessarily the code itself, just the idea of it17:57
sean-k-mooneywell the idea is if we assert no numa toplogy constraits change in the api we dont need to use the filter to assert that again since rebuilds cant change host17:58
sean-k-mooneyso the existing claim should be valid17:58
artomYep :) I'm just paranoid17:59
sean-k-mooneyyou could break the rebuild in other ways17:59
artomRight, that's out of scope17:59
sean-k-mooneye.g. adding a trait that is not present on the host17:59
openstackgerritKashyap Chamarthy proposed openstack/nova master: [RFC] Pick NEXT_MIN libvirt/QEMU versions for "V" release  https://review.opendev.org/69482117:59
sean-k-mooneyya im not trying to adress that although i hope we already do17:59
artom... do we check placement for rebuild?18:00
sean-k-mooneyi hope so but im not sure. its on my todo list to look at in general18:00
artomYour ocean-spanning todo list :D18:01
sean-k-mooneyyes well when i fix things i keep findind other thngs that need to be fixed18:01
*** tbachman has joined #openstack-nova18:02
artomhttps://xkcd.com/1739/18:02
sean-k-mooneylike i found out that apparently we allow you to attach port that violate numa toplogy and you will only find out if you rebuild or otherwise have to go back to the schduler18:02
artomOh are those your comments on the second patch in that stack?18:02
sean-k-mooneythey are on a different patch one sec18:02
*** dtantsur is now known as dtantsur|afk18:03
sean-k-mooneyartom: https://review.opendev.org/#/c/674072/7/nova/tests/unit/virt/test_hardware.py@424018:03
sean-k-mooneyno wrong one18:04
sean-k-mooneyalso found that issue...18:04
sean-k-mooneyhttps://review.opendev.org/#/c/689861/3/nova/tests/functional/libvirt/test_numa_servers.py@110618:05
sean-k-mooneyi need to make that less wordy but ya that existing test was wrong18:06
artomOr make it into its own patch? I haven't dug in yet, is what related to NUMA rebuilds?18:10
artom(That last bit)18:11
sean-k-mooneyfixing the attachment api to reject interace if it would volate the numa aware vstichs feature is a spereate thing18:12
sean-k-mooneyat present it does not check and then a rebuild or resize to the same host woudl fail18:12
sean-k-mooneyi need to file a seperate bug and address it seperately18:13
artomsean-k-mooney, I mean just putting the comments in a separate patch :)18:17
artomAnyways, lemme review that properly18:17
sean-k-mooneyi needed to chagne the test so i added the comment to explain why18:18
artomAck18:18
artomAs I said, I hadn't dug in yet18:18
sean-k-mooneyi proably should change teh NOTE to a FIXME18:18
openstackgerritArchit Modi proposed openstack/nova stable/pike: doc: fix and clarify --block-device usage in user docs  https://review.opendev.org/69445018:21
*** mvkr has quit IRC18:32
*** tbachman has quit IRC18:36
*** igordc has joined #openstack-nova18:46
*** martinkennelly has quit IRC18:48
mriedemthis is a good pike-only fix for something that was regressed with a backport https://review.opendev.org/#/q/topic:bug/1852207+(status:open+OR+status:merged)18:50
*** gouthamr_ is now known as gouthamr18:51
*** tesseract has quit IRC18:52
*** tbachman has joined #openstack-nova18:52
*** ralonsoh has quit IRC18:56
artomsean-k-mooney, grah, forgot the backporting thing - might have left comments that make no sense in that context19:01
openstackgerritMatt Riedemann proposed openstack/nova master: FUP to Ie1a0cbd82a617dbcc15729647218ac3e9cd0e5a9  https://review.opendev.org/69478719:24
openstackgerritMatt Riedemann proposed openstack/nova master: Drop neutron-grenade-multinode job  https://review.opendev.org/69478919:24
*** dpawlik has joined #openstack-nova19:26
*** jdillaman has joined #openstack-nova19:30
*** abaindur has joined #openstack-nova19:34
*** abaindur has quit IRC19:35
*** abaindur has joined #openstack-nova19:36
*** rafaelweingartne has joined #openstack-nova19:43
rafaelweingartneHey guys, we are found on Nova that might be a bug19:44
rafaelweingartnewhen a user is updating something in the VM (e.g. its name), this information is not updated in the KVM XML that is used to describe the VM in the compute host19:44
rafaelweingartneis that expected? Or, is it a bug?19:44
dansmithrafaelweingartne: generally only something like a volume attach will change the libvirt xml at runtime19:47
rafaelweingartnehmm19:47
rafaelweingartneso, that is by design?19:47
dansmithrafaelweingartne: if we store the display name in the xml (do we?) we wouldn't update that until a migration or hard reboot type event19:47
rafaelweingartneyes, we do store some data there19:47
rafaelweingartnenot just the display name, but also project for instance19:47
rafaelweingartneand this is troublesome for Ceilometer, as it was trusting that data19:48
dansmithI see, in metadata. No, we don't change that on the fly19:48
rafaelweingartnehmm19:48
rafaelweingartneshould we fix it in Ceilometer or in Nova?19:48
dansmithnot even sure the compute gets notified when that changes.. I suspect not19:48
rafaelweingartneI opened a bug report: https://bugs.launchpad.net/ceilometer/+bug/185304819:48
openstackLaunchpad bug 1853048 in OpenStack Compute (nova) "Nova not updating VM's XML in KVM" [Undecided,New]19:48
dansmithrafaelweingartne: ceilometer scraping anything out of the xml underneath nova would be at its own peril for sure19:49
rafaelweingartnehmm19:50
*** abaindur has quit IRC19:50
dansmithyeah, confirmed, the compute node won't even be notified that such things have changed via the api,19:50
rafaelweingartneSo, this is by design in Nova? If someone were to fix it, you guys would prefer to fix on Ceilometer side, right?19:51
*** abaindur has joined #openstack-nova19:51
dansmithit would just re-read that info from the db the next time it needs to generate the XML, which like I say would be some significant livecycle event19:51
dansmithrafaelweingartne: yes, by design19:51
dansmithrafaelweingartne: if you mean ceilometer update the xml itself, then no, but if you mean change ceilometer to get that info from somewhere else, then yes19:51
rafaelweingartneand, what about changing Nova to update the data when the user is updating something (e.g. display name of a VM)?19:52
dansmithrafaelweingartne: I would be -2 on that without some really really strong use-case, and ceilometer scraping that info and expecting it to be reliable is not good enough19:54
rafaelweingartneok19:54
rafaelweingartnegot it19:54
*** rafaelweingartne has quit IRC19:58
*** eharney has quit IRC20:02
*** rafaelweingartne has joined #openstack-nova20:03
*** jbernard has quit IRC20:14
*** jmlowe has joined #openstack-nova20:16
*** jbernard has joined #openstack-nova20:17
*** dpawlik has quit IRC20:22
*** rafaelweingartne has quit IRC20:24
*** dpawlik has joined #openstack-nova20:27
*** dpawlik has quit IRC20:41
*** dpawlik has joined #openstack-nova20:44
*** mgariepy has quit IRC20:57
openstackgerritMerged openstack/nova master: Remove service_uuids_online_data_migration  https://review.opendev.org/69342121:01
openstackgerritMerged openstack/nova master: functional: Change order of two classes  https://review.opendev.org/68917821:01
openstackgerritMerged openstack/nova master: functional: Rework '_delete_server'  https://review.opendev.org/68917921:02
*** ozzzo has joined #openstack-nova21:08
ozzzo I created an aggregate with property "allflavors='0'"21:08
ozzzohttps://paste.fedoraproject.org/paste/X8lzKsU474FD1Ffj2x29uQ21:08
ozzzoThen I created VMs and specified "allflavors='1'"21:08
ozzzoopenstack server create --flavor s1.1cx4g --image NOT-QSC-CentOS7.6-19P1-v4 --network it-network --property allflavors='1' --max 20 alberttest21:08
ozzzobut some of the VMs I created went to hv005 which should be restricted by "allflavors"21:09
ozzzohttps://paste.fedoraproject.org/paste/AhxIfQnQjeWPgMw33xmbKQ21:09
ozzzowhat am I doing wrong?21:09
ozzzoI tried asking in #openstack but nobody is answering questions there21:09
*** eharney has joined #openstack-nova21:12
*** awalende has joined #openstack-nova21:16
*** awalende has quit IRC21:21
*** awalende has joined #openstack-nova21:26
*** dpawlik has quit IRC21:29
*** awalende has quit IRC21:31
*** awalende has joined #openstack-nova21:36
openstackgerritMerged openstack/nova master: Stop testing Python 2  https://review.opendev.org/68795421:38
*** ociuhandu has joined #openstack-nova21:38
*** awalende has quit IRC21:40
*** abaindur has quit IRC21:42
*** jbernard has quit IRC21:42
*** dpawlik has joined #openstack-nova21:43
*** ociuhandu has quit IRC21:43
*** abaindur has joined #openstack-nova21:43
*** jbernard has joined #openstack-nova21:45
*** awalende has joined #openstack-nova21:46
*** jbernard_ has quit IRC21:46
*** abaindur has quit IRC21:50
mriedemthat's an exclusion problem with the aggregates filters21:51
mriedemand --property is server metadata, not part of the flavor extra spec21:52
mriedemsee https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#aggregateinstanceextraspecsfilter21:52
*** awalende has quit IRC21:56
*** awalende has joined #openstack-nova21:57
ozzzomriedem: does that mean that we can't specify extra-specs on the command line? Does it have to be in a flavor?21:59
mriedemcorrect22:00
mriedemadmins control extra specs in the flavors, not users22:00
mriedemby default policy rules anyway22:00
ozzzoso we can only delegate to users by giving them a menu of flavors?22:00
ozzzoor by setting specs in image?22:01
mriedemcorrect22:01
ozzzook that is very helpful, ty!22:01
mriedemfiltering by image properties is a different filter22:01
mriedemhttps://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#aggregateimagepropertiesisolation22:01
ozzzook got it22:02
mriedemyou could of course write your own filter that processes server metadata linked to a host aggregate but that could be semi expensive since the RequestSpec object passed to the filter interface does not have the server metadata in it (you'd have to get that from the db per instance in the request)22:03
sean-k-mooneyozzzo: you may be interested in a spec that is currently being proposed in this area22:03
sean-k-mooneyhttps://review.opendev.org/#/c/663563/22:04
sean-k-mooneythis is still changing alot and is not that popular but if you have usecase that would requrie something other then what flaovrs or images provide today22:05
ozzzothat does look interesting, ty!22:05
sean-k-mooneyit would be good to read that and provide feedback22:05
sean-k-mooneychanging how this works is problematic as alot of the code and external tooling assumes flavor and image are the way resouce requests and qualitive aspects of an instance are defiened22:06
*** awalende has quit IRC22:07
*** awalende has joined #openstack-nova22:07
sean-k-mooneythat does not mean we cant discuss changing it its just hard not to break things in this if we do22:07
*** dpawlik has quit IRC22:08
*** awalende_ has joined #openstack-nova22:08
*** awalende has quit IRC22:12
*** abaindur has joined #openstack-nova22:20
*** abaindur has quit IRC22:24
*** pcaruana has quit IRC22:26
*** kaisers1 has joined #openstack-nova22:28
*** kaisers has quit IRC22:29
*** mriedem has quit IRC22:31
*** maciejjozefczyk has quit IRC22:39
*** maciejjozefczyk has joined #openstack-nova22:39
*** mvkr has joined #openstack-nova22:40
*** abaindur has joined #openstack-nova22:44
*** abaindur has joined #openstack-nova22:45
*** awalende_ has quit IRC22:46
*** rcernin has joined #openstack-nova22:46
*** slaweq has quit IRC22:50
*** luksky has quit IRC23:02
*** tkajinam has joined #openstack-nova23:09
*** aloga_ has quit IRC23:10
*** slaweq has joined #openstack-nova23:11
*** slaweq has quit IRC23:17
*** jaosorior has quit IRC23:21
*** mlavalle has quit IRC23:24
*** ivve has quit IRC23:26
*** Liang__ has joined #openstack-nova23:30
*** ociuhandu has joined #openstack-nova23:31
*** ociuhandu has quit IRC23:36
openstackgerritMerged openstack/nova master: Move rng device checks to the appropriate method  https://review.opendev.org/69321023:39
*** aloga has joined #openstack-nova23:40
*** zhanglong has joined #openstack-nova23:47

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