Monday, 2019-09-30

*** victor286 has joined #openstack-nova00:31
*** ricolin has joined #openstack-nova00:32
*** victor287 has joined #openstack-nova00:33
*** victor286 has quit IRC00:33
*** victor287 has quit IRC00:33
*** victor286 has joined #openstack-nova00:33
*** victor287 has joined #openstack-nova00:37
*** victor286 has quit IRC00:40
*** victor287 is now known as victor28600:40
*** Liang__ has joined #openstack-nova01:08
*** Liang__ is now known as LiangFang01:17
*** brinzhang has quit IRC01:29
*** brinzhang has joined #openstack-nova01:29
*** victor287 has joined #openstack-nova01:35
*** brinzhang_ has joined #openstack-nova01:38
*** victor286 has quit IRC01:39
*** victor287 is now known as victor28601:39
*** brinzhang has quit IRC01:41
*** BjoernT has joined #openstack-nova01:50
*** BjoernT has quit IRC01:59
*** BjoernT has joined #openstack-nova02:05
*** tkajinam_ has joined #openstack-nova02:06
*** BjoernT has quit IRC02:08
*** tkajinam has quit IRC02:09
*** brinzhang has joined #openstack-nova02:10
*** brinzhang_ has quit IRC02:14
*** brinzhang_ has joined #openstack-nova02:17
*** brinzhang has quit IRC02:20
*** gbarros has joined #openstack-nova02:21
*** victor287 has joined #openstack-nova02:27
*** victor286 has quit IRC02:29
*** victor287 is now known as victor28602:29
*** brinzhang has joined #openstack-nova02:43
*** gbarros has quit IRC02:43
*** brinzhang_ has quit IRC02:46
*** markvoelker has joined #openstack-nova02:46
*** davee_ has joined #openstack-nova03:03
*** davee__ has quit IRC03:04
*** brinzhang_ has joined #openstack-nova03:13
openstackgerritRui Zang proposed openstack/nova master: make virtual pmem feature compatible with python3  https://review.opendev.org/68559303:16
*** brinzhang has quit IRC03:17
*** BjoernT has joined #openstack-nova03:19
*** brinzhang has joined #openstack-nova03:20
*** BjoernT_ has joined #openstack-nova03:23
*** brinzhang_ has quit IRC03:23
*** BjoernT has quit IRC03:24
*** mkrai has joined #openstack-nova03:41
*** psachin has joined #openstack-nova03:55
*** udesale has joined #openstack-nova04:04
*** toabctl has joined #openstack-nova04:07
*** brinzhang_ has joined #openstack-nova04:12
*** brinzhang has quit IRC04:15
*** BjoernT_ has quit IRC04:18
*** BjoernT has joined #openstack-nova04:21
*** ociuhandu has joined #openstack-nova04:30
*** ociuhandu has quit IRC04:35
*** brinzhang has joined #openstack-nova04:57
*** Luzi has joined #openstack-nova05:01
*** brinzhang_ has quit IRC05:01
*** ircuser-1 has joined #openstack-nova05:01
*** brinzhang_ has joined #openstack-nova05:02
*** markvoelker has quit IRC05:02
*** jaosorior has joined #openstack-nova05:03
*** brinzhang has quit IRC05:03
*** BjoernT has quit IRC05:08
*** BjoernT has joined #openstack-nova05:11
*** BjoernT has quit IRC05:23
*** brinzhang has joined #openstack-nova05:23
*** pcaruana has joined #openstack-nova05:24
*** ratailor has joined #openstack-nova05:25
*** brinzhang_ has quit IRC05:27
*** dpawlik has joined #openstack-nova05:28
*** pcaruana has quit IRC05:30
*** janki has joined #openstack-nova05:35
*** brinzhang_ has joined #openstack-nova05:38
*** brinzhang has quit IRC05:41
openstackgerritTakashi NATSUME proposed openstack/nova master: Replace 'fake' with a real project ID  https://review.opendev.org/68564505:43
*** markvoelker has joined #openstack-nova05:58
openstackgerritRui Zang proposed openstack/nova master: make virtual pmem feature compatible with python3  https://review.opendev.org/68559305:58
*** dpawlik has quit IRC06:01
*** markvoelker has quit IRC06:02
*** slaweq has joined #openstack-nova06:23
*** dpawlik has joined #openstack-nova06:29
*** avolkov has joined #openstack-nova06:37
*** luksky has joined #openstack-nova06:38
*** pcaruana has joined #openstack-nova06:41
*** damien_r has joined #openstack-nova06:56
*** damien_r has quit IRC06:56
*** damien_r has joined #openstack-nova06:57
*** maciejjozefczyk has joined #openstack-nova07:12
*** rcernin has quit IRC07:14
*** tesseract has joined #openstack-nova07:17
*** jchhatbar has joined #openstack-nova07:23
openstackgerritLee Yarwood proposed openstack/nova master: DNM libvirt: extract _start_heterogeneous_computes  https://review.opendev.org/68262007:23
openstackgerritLee Yarwood proposed openstack/nova master: Add regression test for bug #1843639  https://review.opendev.org/68262107:23
openstackbug 1843639 in OpenStack Compute (nova) "libvirt: post_live_migration failures to disconnect volumes result in the rollback of live migrations" [Medium,In progress] https://launchpad.net/bugs/1843639 - Assigned to Lee Yarwood (lyarwood)07:23
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Ignore volume exceptions during post_live_migration  https://review.opendev.org/68262207:23
openstackgerritLee Yarwood proposed openstack/nova master: compute: Remove stale BDMs on reserve_block_device_name failure  https://review.opendev.org/68259407:25
*** arxcruz|zzz is now known as arxcruz|ruck07:25
*** janki has quit IRC07:25
openstackgerritwangfaxin proposed openstack/nova master: fixd notes: neuton is replaced with neutron in line 756.  https://review.opendev.org/68550907:29
*** tesseract has quit IRC07:43
*** tesseract has joined #openstack-nova07:43
*** LiangFang has quit IRC07:43
*** dpawlik has quit IRC07:45
*** rpittau|afk is now known as rpittau07:52
*** markvoelker has joined #openstack-nova08:00
*** ralonsoh has joined #openstack-nova08:00
*** tkajinam_ has quit IRC08:00
*** ttsiouts has joined #openstack-nova08:02
*** dpawlik has joined #openstack-nova08:04
*** markvoelker has quit IRC08:04
*** takashin has left #openstack-nova08:05
*** ttsiouts has quit IRC08:16
*** ttsiouts has joined #openstack-nova08:20
*** xek has joined #openstack-nova08:23
*** dtantsur|afk is now known as dtantsur08:24
*** ttsiouts has quit IRC08:31
*** derekh has joined #openstack-nova08:38
*** dpawlik has quit IRC08:39
*** cdent has joined #openstack-nova08:56
*** cdent has left #openstack-nova08:57
*** tssurya has joined #openstack-nova08:57
*** dpawlik has joined #openstack-nova08:59
openstackgerritSeyeong Kim proposed openstack/nova master: block_device: Optionally recreate attachments when refreshing connection_info  https://review.opendev.org/57900409:00
*** dpawlik has quit IRC09:04
*** ttsiouts has joined #openstack-nova09:08
openstackgerritBrin Zhang proposed openstack/nova-specs master: Fix invalid link index  https://review.opendev.org/68566409:12
*** dpawlik has joined #openstack-nova09:13
*** ricolin has quit IRC09:13
*** jangutter has joined #openstack-nova09:13
*** smcginnis has quit IRC09:14
bauzasgibi: <3 you09:15
bauzasgibi: found a shit number of helpers09:15
bauzasin the integrated helpers for my functional tests09:15
bauzaswill make them easier to read09:15
gibibauzas: I'm glad that those can help you09:15
*** ttsiouts has quit IRC09:17
bauzasgibi: the only issue is that I discovered those helpers after some time09:20
* bauzas needs more to look at the existing modules09:20
gibiyeah it is hard how to make such (and other) helper codes discoverable09:21
bauzasunless by reviewing others's patches :)09:21
gibibut nobody can review every incoming patch in nova to know what happens09:22
*** zbr is now known as zbr|ruck09:24
bauzas...also if they have a lot of other things to do internally :)09:27
aspiersIs it just me or is https://docs.openstack.org/glance/latest/admin/useful-image-properties.html wrong regarding the default of hw_scsi_model being virtio-blk?09:29
*** dpawlik has quit IRC09:29
aspiersAFAICS, if hw_scsi_model is not specified, there will be no SCSI controller09:31
aspiersbut also, "virtio-blk" is not a valid value in nova.objects.fields.SCSIModel09:32
*** sapd1_x has joined #openstack-nova09:33
aspiersand in fact that doc is missing all the other valid values09:36
*** dpawlik has joined #openstack-nova09:48
*** ociuhandu has joined #openstack-nova09:52
*** ttsiouts has joined #openstack-nova09:55
*** mkrai has quit IRC09:57
*** victor286 has quit IRC10:11
aspiersah, I think I figured it out10:16
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Use the `preferred` memory allocation mode for NUMA  https://review.opendev.org/68437510:16
aspiersthe default when hw_scsi_model is not specified is no SCSI controller, and something like <target dev='vda' bus='virtio'/>10:16
aspierswhich is colloquially referred to as virtio-blk, but it's just 'virtio' in the code10:17
*** sean-k-mooney has quit IRC10:18
*** sean-k-mooney has joined #openstack-nova10:22
sean-k-mooneyaspiers: yes the defualt for libvirt/kvm is virtio or virtio-blk or virtio block10:23
*** mkrai has joined #openstack-nova10:23
aspierssean-k-mooney: thanks for confirmation. just submitted https://review.opendev.org/#/c/685674/10:23
sean-k-mooneyif you want to use virto-scsi you need to set the disk bus to scsi and then set the scsi model to virtio-scsi10:24
sean-k-mooneythe default storage contole depend on the hypervisor and virt type10:25
sean-k-mooneyit wont alwyas be virtio if you dont set hw_disk_bus10:25
*** avolkov has quit IRC10:27
sean-k-mooneyaspiers: this is the funciton in the libvirt driver that chooes the defult disk bus https://github.com/openstack/nova/blob/master/nova/virt/libvirt/blockinfo.py#L221-L29610:28
sean-k-mooneyi should also point out that there is this horrible code that tries to incorrectly determin the optimal default disk and cd buss form the image os10:31
sean-k-mooneyhttps://github.com/openstack/nova/blob/master/nova/virt/osinfo.py#L114-L14710:31
sean-k-mooneybut its actully broken by desgin in that it expect the OS version to be concatonated into the os_distro field instead of actully reading the os_version field so it does not actully work correctly10:33
openstackgerritDongcan Ye proposed openstack/nova master: libvirt: Check the guest support UEFI  https://review.opendev.org/68567810:39
*** ociuhandu has quit IRC10:40
*** ociuhandu has joined #openstack-nova10:53
*** ricolin has joined #openstack-nova10:54
*** ttsiouts has quit IRC10:56
*** jchhatbar has quit IRC11:00
*** henriqueof has joined #openstack-nova11:01
*** markvoelker has joined #openstack-nova11:01
*** markvoelker has quit IRC11:06
*** udesale has quit IRC11:13
*** luksky has quit IRC11:14
*** rcernin has joined #openstack-nova11:28
sean-k-mooneylyarwood: care to hit some easy sable patches in os-vif11:32
sean-k-mooneyhttps://review.opendev.org/#/c/683489/ https://review.opendev.org/#/c/683488/11:33
sean-k-mooneywell not backports release stuff. just the git review and constraits file updates for train11:33
*** ociuhandu has quit IRC11:35
*** ociuhandu has joined #openstack-nova11:36
*** mkrai has quit IRC11:38
*** xek has quit IRC11:39
*** xek_ has joined #openstack-nova11:39
*** mkrai has joined #openstack-nova11:45
lyarwoodsean-k-mooney: ack looking11:48
*** awalende has joined #openstack-nova11:50
lyarwoodsean-k-mooney: done11:52
*** ttsiouts has joined #openstack-nova11:52
*** ttsiouts has quit IRC12:06
*** ttsiouts has joined #openstack-nova12:06
*** Kevin_Zheng has quit IRC12:10
*** luksky has joined #openstack-nova12:13
openstackgerritSeyeong Kim proposed openstack/nova master: block_device: Optionally recreate attachments when refreshing connection_info  https://review.opendev.org/57900412:15
*** rcernin has quit IRC12:50
*** sapd1_x has quit IRC12:58
*** dtantsur is now known as dtantsur|afk12:58
*** gbarros has joined #openstack-nova12:59
*** nweinber_ has joined #openstack-nova12:59
*** dklyle has quit IRC13:00
*** markvoelker has joined #openstack-nova13:02
*** maciejjozefczyk_ has joined #openstack-nova13:03
openstackgerritMerged openstack/os-vif master: Update the constraints url  https://review.opendev.org/68387313:04
*** maciejjozefczyk has quit IRC13:05
*** ttsiouts has quit IRC13:06
*** markvoelker has quit IRC13:06
*** ratailor has quit IRC13:10
openstackgerritMerged openstack/os-vif master: Update master for stable/train  https://review.opendev.org/68349013:12
*** mkrai has quit IRC13:12
*** BjoernT has joined #openstack-nova13:14
*** dave-mccowan has joined #openstack-nova13:17
*** ttsiouts has joined #openstack-nova13:20
*** mriedem has joined #openstack-nova13:25
bauzasmriedem: morningh13:27
bauzasmriedem: reviewing a shit number of times https://review.opendev.org/#/c/678100/13:27
bauzasIIUC, we leak the source allocation by https://review.opendev.org/#/c/678100/1/nova/tests/functional/wsgi/test_services.py13:29
*** BjoernT has quit IRC13:33
*** maciejjozefczyk_ has quit IRC13:35
openstackgerritMerged openstack/os-vif stable/train: Update .gitreview for stable/train  https://review.opendev.org/68348813:35
*** maciejjozefczyk has joined #openstack-nova13:35
*** dpawlik has quit IRC13:36
*** Luzi has quit IRC13:37
*** BjoernT_ has joined #openstack-nova13:43
mriedembauzas: i'm not following the question13:43
bauzassorry then13:43
*** dklyle has joined #openstack-nova13:43
mriedemfor evacuate we leak the source allocation when deleting the source compute service13:44
mriedemb/c we fail to delete the resource provider b/c it has allocatoins against it from the evacuate13:44
bauzastrying to exactly understand how to get an orphan allocation when resizeing13:44
bauzasas I want to write some functest13:44
mriedemfor resize, you'd migrate the server so it gets to VERIFY_RESIZE, then delete the source compute service13:44
bauzasfor the moment, I did : create the instance, resize, delete the service13:44
mriedemyes13:44
mriedemthe source resource provider will have allocations held by the migration record13:44
bauzasbut when confirming the resize, I get an exception13:45
*** awalende has quit IRC13:45
mriedemthat's separate from the leaked allocation issue13:45
*** awalende has joined #openstack-nova13:45
bauzashum, so, once I delete the service, I'll just delete the instance?13:46
mriedemfor one thing, if you restart the source compute service it will create a new compute node with a new uuid and a new resource provider, so the migration record won't be able to find allocations on that one13:46
mriedemi guess i'd have to know what the error is when you go to confirm the resize13:46
bauzassecf13:46
bauzasmriedem: http://paste.openstack.org/show/780354/13:47
*** dklyle has quit IRC13:48
bauzaswhen confirming the resize, I get an exception saying that it doesn't get the source compute UUID13:48
*** david-lyle has joined #openstack-nova13:48
*** munimeha1 has joined #openstack-nova13:49
mriedemdoes the test stop/start a new compute service after deleting the original source compute service?13:49
mriedemstarting a new compute service is going to create a new compute node record13:49
mriedemif you didn't fully stop the service it's going to have residual cached information in the ResourceTracker.compute_nodes dict13:49
*** awalende has quit IRC13:50
mriedemwhich is likely a bug in your test, though someone could totally forget to stop the compute service before deleting it, we have warnings in the API about that13:50
openstackgerritMerged openstack/os-vif stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train  https://review.opendev.org/68348913:51
bauzasmriedem: okay, I can restart the service before confirming, then13:51
bauzasthanks13:51
mriedemwell, to do things the proper way, you should be stopping the source compute service, then deleting it, and then starting a new one13:52
mriedemhttps://docs.openstack.org/api-ref/compute/?expanded=delete-compute-service-detail#delete-compute-service13:52
*** gbarros has quit IRC13:53
mriedemso what is happening i think is that because you're confirming with a stale ResourceTracker.compute_nodes entry, it's trying to update a record that was deleted when you deleted the compute service13:53
mriedemw/o stopping it13:53
*** jangutter has quit IRC13:53
*** jamesdenton has quit IRC13:54
mriedemone could argue that drop_move_claim should handle that since we don't prevent you from deleting the compute service in the api w/o stopping it first13:54
mriedembut it's a different bug13:54
bauzasokay, I'll quickly write it13:55
*** gbarros has joined #openstack-nova13:56
mriedemb/c of this https://opendev.org/openstack/nova/src/branch/master/nova/compute/resource_tracker.py#L714 is why we don't create a new compute node record when you start the service after deleting it13:56
mriedemand we're using the old now-deleted version13:56
mriedemhttps://review.opendev.org/#/c/684849/ is a similar issue13:57
mriedemi've found a few problems in the RT code lately b/c of that caching13:57
*** TxGirlGeek has joined #openstack-nova13:57
mriedemhttps://review.opendev.org/#/q/I9fa1d509a3de405d6246fb8670612c65c10cc93b13:57
*** ociuhandu has quit IRC13:58
mriedemfeel free to hit the queens backport for ^ before you get a bz downstream for it anyway13:58
*** redrobot has joined #openstack-nova13:59
mriedemneed another stable core to approve this backport for train rc2 https://review.opendev.org/#/c/685387/14:02
bauzasmriedem: can do14:02
*** TxGirlGeek has quit IRC14:05
*** jamesdenton has joined #openstack-nova14:06
mriedembauzas: btw i might have misled you with a comment on your placement audit patch about recreating an evacuate scenario with leaked allocations, if i said there is a migration-based allocatoin on the source provider during evacuate, that was wrong14:06
mriedemwe only do migration-based allocation swap-a-roo for cold and live migrate14:07
mriedem*and resize14:07
bauzasmriedem: hmmm14:07
mriedemthere is an allocation on the source provider, it's just not held by a migration record, it's held by the instance consumer14:07
*** jangutter has joined #openstack-nova14:08
bauzasmriedem: that's what I found14:08
bauzasmriedem: I already have a test for evacuate which works14:08
*** smcginni1 has joined #openstack-nova14:08
bauzasmriedem: https://review.opendev.org/#/c/670112/5/nova/tests/functional/test_nova_manage.py@1470 (but I need to rewrite this test to use the existing helpers thanks to gibi :) )14:08
*** adriant has quit IRC14:09
*** smcginni1 is now known as smcginnis_14:11
*** pcaruana has quit IRC14:18
*** ttsiouts has quit IRC14:20
*** ttsiouts has joined #openstack-nova14:20
*** ttsiouts has quit IRC14:25
*** ociuhandu has joined #openstack-nova14:33
*** BjoernT_ is now known as BjoernT14:40
*** smcginnis_ is now known as smcginnis14:46
*** slaweq has quit IRC14:48
*** slaweq has joined #openstack-nova14:50
*** TxGirlGeek has joined #openstack-nova14:54
*** awalende has joined #openstack-nova14:56
*** maciejjozefczyk has quit IRC14:57
*** awalende has quit IRC15:01
*** markvoelker has joined #openstack-nova15:02
*** mlavalle has joined #openstack-nova15:04
*** nweinber_ has quit IRC15:04
*** nweinber has joined #openstack-nova15:05
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Use nova-net explicitly in '_IntegratedTestBase' subclasses  https://review.opendev.org/68432615:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Use nova-net explicitly in 'ApiSampleTestBase' subclasses  https://review.opendev.org/68432815:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Use nova-net explicitly in 'ServersSampleBase' subclasses  https://review.opendev.org/68433015:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_availability_zone' functional tests  https://review.opendev.org/68433215:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ip_pools' functional tests  https://review.opendev.org/68433315:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_quota_sets' functional tests  https://review.opendev.org/68433415:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_server_tags' functional tests  https://review.opendev.org/68433515:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_servers' functional tests  https://review.opendev.org/68433615:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hosts' functional tests  https://review.opendev.org/68433715:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_networks_associate' functional tests  https://review.opendev.org/68433815:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_rescue' functional tests  https://review.opendev.org/68433915:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hypervisors' functional tests  https://review.opendev.org/68434015:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_attach_interfaces' functional tests  https://review.opendev.org/68434115:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_simple_tenant_usage' functional tests  https://review.opendev.org/68434215:14
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Neutron fixture cleanup  https://review.opendev.org/68434315:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ips' functional tests  https://review.opendev.org/68434415:14
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests  https://review.opendev.org/68434515:14
*** Jeffrey4l has quit IRC15:19
*** Jeffrey4l has joined #openstack-nova15:20
*** mkrai has joined #openstack-nova15:24
openstackgerritAdam Spiers proposed openstack/nova master: Also enable iommu for virtio controllers in libvirt  https://review.opendev.org/68482515:25
openstackgerritAdam Spiers proposed openstack/nova master: Also enable iommu for virtio controllers in libvirt  https://review.opendev.org/68482515:25
*** luksky has quit IRC15:26
openstackgerritMerged openstack/nova stable/train: Stop filtering out 'accepted' for in-progress migrations  https://review.opendev.org/68538715:29
*** gyee has joined #openstack-nova15:30
openstackgerritEric Fried proposed openstack/nova master: make virtual pmem feature compatible with python3  https://review.opendev.org/68559315:34
*** jdillaman has joined #openstack-nova15:35
dansmithefried: looks like that should be an rc2 thing eh?15:36
*** jmlowe has quit IRC15:38
*** damien_r has quit IRC15:39
dansmithreally, the guest config module should handle that15:39
efrieddansmith: Could make an argument for RC2, yeah.15:41
efriedsince basically py3 is busted otherwise.15:42
dansmithit's a regression and a broken feature we added in train, so it seems clear to me15:42
efrieddansmith: Since Alex was +2 and I just changed the test, would you feel good about fast approving it?15:42
efried"the guest config module should handle" like by knowing which fields are supposed to be int and casting them before stringifying them?15:43
dansmithI'm pulling it down to mess with it15:43
dansmithefried: uhyeah.. right now it would stringify an object in that field if you gave it to it15:43
dansmithwhich is dumb15:43
dansmithobviously not for this fix though15:44
efriedyeah, seems like probably that would be a pervasive problem throughout15:44
efriedthough I haven't looked15:44
dansmithit is, from a quick look15:44
efriedIt took me a hot minute to figure out why we weren't getting by on the fact that the ovo fields are IntegerField()15:45
efriedbut it's because we're doing the conversion after that15:45
efrieds/conversion/division/15:45
dansmithyeah, this is way after any object stuff15:45
*** pcaruana has joined #openstack-nova15:46
*** rpittau is now known as rpittau|afk15:48
*** tssurya has quit IRC15:49
openstackgerritDan Smith proposed openstack/nova stable/train: make virtual pmem feature compatible with python3  https://review.opendev.org/68574215:50
*** tesseract has quit IRC15:55
dansmithI guess if we15:58
dansmithare going to merge things based purely on manual testing by the submitter, we should ask for them to do it in py2 and py315:58
dansmith(or not do that thing)15:58
*** xek_ is now known as xek15:59
efriedWell, it wasn't manual, it was 3pCI, but I never thought to check that it was running py3.16:00
efriedAre we planning to continue supporting py2 in ussuri?16:00
efriedthe argument could be made that testing (3pCI et al) should *only* be py316:01
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Use nova-net explicitly in functional tests  https://review.opendev.org/68432616:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_availability_zone' functional tests  https://review.opendev.org/68433216:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ip_pools' functional tests  https://review.opendev.org/68433316:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_quota_sets' functional tests  https://review.opendev.org/68433416:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_server_tags' functional tests  https://review.opendev.org/68433516:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_servers' functional tests  https://review.opendev.org/68433616:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hosts' functional tests  https://review.opendev.org/68433716:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_networks_associate' functional tests  https://review.opendev.org/68433816:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_rescue' functional tests  https://review.opendev.org/68433916:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hypervisors' functional tests  https://review.opendev.org/68434016:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_attach_interfaces' functional tests  https://review.opendev.org/68434116:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_simple_tenant_usage' functional tests  https://review.opendev.org/68434216:03
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Neutron fixture cleanup  https://review.opendev.org/68434316:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ips' functional tests  https://review.opendev.org/68434416:03
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests  https://review.opendev.org/68434516:03
bauzasmriedem: you know what ? can't confirm a resize after stopping/deleting/restarting the original compute16:03
mriedemthat's not surprising16:04
bauzasmriedem: after this, I'm stopping to do this, I'll just mock out the allocation delete16:04
bauzasso it'll just leave the orphaned migration allocation16:04
mriedembauzas: i'm assuming the failure is the same as before? or something new because on restart of the source compute service, we've created a new compute node record with a new uuid and a new resource provider and we can't cleanup the migration-based allocations b/c we're looking for the wrong provider?16:05
bauzasmriedem: a different exception16:05
*** itlinux has joined #openstack-nova16:06
bauzasmriedem: a conflict because we want to recreate a new RP with the same name16:06
mriedemdo you have a paste?16:06
bauzassure, sec16:06
mriedemnvm that's a known bug16:06
mriedemhttps://bugs.launchpad.net/nova/+bug/181783316:07
openstackLaunchpad bug 1817833 in OpenStack Compute (nova) "Check compute_id existence when nova-compute reports info to placement" [Medium,In progress] - Assigned to Matt Riedemann (mriedem)16:07
mriedemcalled out in https://review.opendev.org/#/c/678100/16:07
*** maciejjozefczyk has joined #openstack-nova16:07
bauzasmriedem: http://paste.openstack.org/show/780370/16:08
bauzasmriedem: anyway, like I said, I'll just mock out the allocation delete16:08
bauzasso once we fix the bug, we won't need to modify the functional test16:08
bauzas(or I'll just recreate the migration allocation)16:08
bauzasin order to make sure the nova-manage audit command sees it and then deletes it if asked16:09
mriedemi'd make sure to have a note explaining it's due to bug 181783316:10
openstackbug 1817833 in OpenStack Compute (nova) "Check compute_id existence when nova-compute reports info to placement" [Medium,In progress] https://launchpad.net/bugs/1817833 - Assigned to Matt Riedemann (mriedem)16:10
mriedemif you're mocking things out in a functional test16:10
bauzasmriedem: I'll write something and rushing to upload it, so you can provide comments16:11
mriedemyou're assuming i'm going to review that today...16:12
mriedembut sure :)16:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Use nova-net explicitly in functional tests  https://review.opendev.org/68432616:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_availability_zone' functional tests  https://review.opendev.org/68433216:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ip_pools' functional tests  https://review.opendev.org/68433316:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_quota_sets' functional tests  https://review.opendev.org/68433416:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_server_tags' functional tests  https://review.opendev.org/68433516:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_servers' functional tests  https://review.opendev.org/68433616:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hosts' functional tests  https://review.opendev.org/68433716:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hypervisors' functional tests  https://review.opendev.org/68434016:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_attach_interfaces' functional tests  https://review.opendev.org/68434116:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_simple_tenant_usage' functional tests  https://review.opendev.org/68434216:12
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Neutron fixture cleanup  https://review.opendev.org/68434316:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ips' functional tests  https://review.opendev.org/68434416:12
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests  https://review.opendev.org/68434516:12
*** ociuhandu has quit IRC16:13
bauzasmriedem: nah, it's 6:13pm, my wife is looking at me angrily16:13
bauzasmriedem: so I'll do this tomorrow morning16:13
mriedemi understand16:13
bauzaswell, my wife understands more my upstream work than other folks :)16:14
*** Sundar has joined #openstack-nova16:15
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests  https://review.opendev.org/68434516:19
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Migrate old style volume attachments on nova-compute startup  https://review.opendev.org/54913016:21
openstackgerritMatt Riedemann proposed openstack/nova master: Extract some helper functions from DriverVolumeBlockDevice  https://review.opendev.org/68575216:21
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ips' functional tests  https://review.opendev.org/68434416:24
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests  https://review.opendev.org/68434516:24
openstackgerritMerged openstack/nova-specs master: Fix invalid link index  https://review.opendev.org/68566416:26
*** yaawang has quit IRC16:28
*** yaawang has joined #openstack-nova16:30
*** cgoncalves has quit IRC16:42
*** cgoncalves has joined #openstack-nova16:42
*** mkrai has quit IRC16:43
*** cgoncalves has quit IRC16:43
*** cgoncalves has joined #openstack-nova16:45
openstackgerritAdam Spiers proposed openstack/nova master: Also enable iommu for virtio controllers in libvirt  https://review.opendev.org/68482516:45
*** jaosorior has quit IRC16:46
openstackgerritAdam Spiers proposed openstack/nova stable/train: Also enable iommu for virtio controllers in libvirt  https://review.opendev.org/68575616:47
*** zbr|ruck is now known as zbr16:52
*** gbarros has quit IRC16:53
*** gbarros has joined #openstack-nova16:55
*** slaweq has quit IRC16:59
*** cgoncalves has quit IRC16:59
*** gbarros has quit IRC17:00
*** derekh has quit IRC17:00
*** cgoncalves has joined #openstack-nova17:00
*** cfriesen has joined #openstack-nova17:01
*** slaweq has joined #openstack-nova17:01
*** gbarros has joined #openstack-nova17:03
*** jmlowe has joined #openstack-nova17:03
efriedaspiers: RC2? ^17:05
efriednot clear from the bug report what actually breaks17:06
*** igordc has joined #openstack-nova17:08
*** owalsh has quit IRC17:09
*** owalsh has joined #openstack-nova17:10
sean-k-mooneyefried: im guessing it breaks sev guest when you set the disk bus to scsi and select virt-scsi as the model17:11
*** gbarros has quit IRC17:13
efriedsean-k-mooney: is that something that happens frequently?17:14
sean-k-mooneyyes17:14
*** gbarros has joined #openstack-nova17:14
efriedand by "breaks" -- the guest won't boot? or won't be SEV'd? or...?17:14
sean-k-mooneyhw_disk_mode=scsi and hw_scsi_model=virtio-scsi is the recommend mode for ceph17:14
sean-k-mooneyim not sure what aspiers is chaning is enabling the iommu integration so i would guest it unencyrped but i dont think tha tis allowd for sev guest so it might just not boot17:15
aspiersefried: I think it probably causes a kernel panic17:17
sean-k-mooneyin the guest right17:17
aspiersfrom the spec: "The iommu attribute is on for all virtio devices. Despite the name, this does not require the guest or host to have an IOMMU device, but merely enables the virtio flag which indicates that virtualized DMA should be used. This ties into the SEV code to handle memory encryption/decryption, and prevents IO buffers being shared between host and guest."17:17
*** markvoelker has quit IRC17:17
aspiersI've definitely seen kernel panics from incorrectly configured guests, I can't remember if I explicitly tested virtio-scsi without iommu17:17
aspiersbut it's required for sure17:17
efriedk, would be good to understand all of that to know whether this should be considered critical for RC2. It sounds like it probably is. <== dansmith mriedem17:17
sean-k-mooneywell you have to opt into enabling the scsi disk bus17:18
efriedaspiers: imo [unencrypted when encrypted was expected] would be worse than [kernel panic] (assuming the latter is only affecting the guest, not the whole host)17:18
*** BjoernT has quit IRC17:18
sean-k-mooneysince that is not the default17:18
aspierssean-k-mooney: not if config drives or cdroms are used17:18
sean-k-mooneybut it would be very common to do so if you are using ceph17:18
efriedbut either way it's a bug in a new feature and therefore RC potential IIUC17:19
sean-k-mooneyaspiers: config drive should now defualt to sata17:19
aspiersefried: [unencrypted when encrypted was expected] will not happen17:19
sean-k-mooneyand it used to default to ide17:19
efriedwell that's good anyway :)17:19
*** ociuhandu has joined #openstack-nova17:19
aspiersefried: the <launchSecurity> element will be there regardless of any iommu stuff17:19
*** markvoelker has joined #openstack-nova17:20
dansmithefried: in a sec17:22
dansmithmriedem: do you know if another db_sync --all-cells patch was floated somewhere? The one I was thinking of never merged and was abandoned silently in july17:23
dansmithmriedem: https://review.opendev.org/#/c/519275/17:23
dansmithnot sure why, but that probably took it off anyone's radar17:23
dansmithor hmm, maybe we made it hit all cells by default... gosh, all this fell out of my head17:24
dansmithah, no --local-cell is just skipping cell0, so it still doesn't fan like I thought17:25
*** maciejjozefczyk has quit IRC17:26
*** ociuhandu has quit IRC17:27
dansmithefried: probably depends on what the fix looks like and what impacts it might have, and what the impact of not having it is (like you say expecting safe, but not safe)17:29
sean-k-mooneythats the main part of the fix https://review.opendev.org/#/c/684825/4/nova/virt/libvirt/designer.py17:30
*** maciejjozefczyk has joined #openstack-nova17:32
sean-k-mooneyaspiers: by the way the only think that stikes me about the fix is that we are testing with a fake xml we cannot generate17:33
mriedemdansmith: i don't remember another, though i thought i had reported a bug at some point about making db sync support all cells but i might just be thinking of the archive command17:33
sean-k-mooneye.g. its not posibel to have two scsi contolers with a different model based on how nova generates the xml17:34
dansmithmriedem: yeah, so code in tree does not fan out (except to cell0) and that other patch got abandoned in July for some reason17:34
sean-k-mooneythat said it is vaild in libvirt to do that17:34
mriedemi also remember bringing it up at some summit, i.e. should the nova-manage commands hit all cells? and it was a low priority response - that might have been sydney...17:34
mriedemdansmith: my guess is they abandoned it b/c it sat since july with no replies17:34
dansmithcould be17:34
mriedemlincanwei is still around though, he's the watcher ptl17:34
dansmithoh, nm, this was abandoned in Jan, last patch was in july 2018, I see17:39
dansmithhmm, I thought there was a more recent attempt at this t hen17:39
mriedemi see a duplicate of the archive all-cells patch17:40
mriedemhttps://review.opendev.org/#/c/599050/ ?17:40
mriedemoh no that's different17:41
dansmithno17:41
dansmithyeah17:41
dansmithanyway, not a big deal17:41
gmannmriedem: dansmith : seems like sec groups are added for down cell response for detail GET API case only (it is not included in Show API  case)17:41
mriedemdansmith: heh https://review.opendev.org/#/c/420973/17:42
mriedemit was me!17:42
gmannand that is because sec grps are added explicitly for detail case-  https://github.com/openstack/nova/blob/961c2945491ebcea3cf1cb175a06d057155aa5a5/nova/api/openstack/compute/views/servers.py#L41017:42
dansmithmriedem: that's not the one I was thinking of, but..funny17:42
gmannis it missed in 2.69 microversion ?17:42
*** gbarros has quit IRC17:43
gmannwith nova-net it was not added as there were no sec grp in DB nut with stephenfin patch to run sample tests with neutron fail - https://review.opendev.org/#/c/684335/5/doc/api_samples/servers/v2.69/servers-details-resp.json@1217:44
mriedemit was probably an oversight17:45
mriedemi'm also not sure how much we need to care if the cell that the instance is in is down17:45
gmannand same for host_status ?17:45
mriedemif the cell that the instance is in is down, the host_status likely doesn't matter17:46
mriedemyou can't do anything with that instance17:46
mriedemexcept maybe delete it17:46
*** awalende has joined #openstack-nova17:46
mriedemthe whole point with 2.69 is return a minimal set of stuff based on what's in the API DB for the instance17:47
mriedemnot to return everything we possibly can17:47
mriedeme.g. we could also proxy to cinder to get attached volume info but we're not going to do that (like the neutron api proxy call to get security groups)17:47
mriedemgmann: for host_status this method won't work really https://github.com/openstack/nova/blob/961c2945491ebcea3cf1cb175a06d057155aa5a5/nova/compute/api.py#L488117:48
gmannyeah, it would not hurt to return. only difference will be GET and GET details response. Not sure is it worth to fix though.17:48
mriedeminstance.host wouldn't be set so at best we'd be returning NONE which is potentially not accurate - UNKNOWN would be more appropriate17:49
mriedemif you're using 2.69+ and getting a repsonse from a down cell, there are already going to be a lot of differences17:49
mriedemfor which you need to account client-side17:49
mriedemhttps://docs.openstack.org/api-guide/compute/down_cells.html gives the details on the fields that are returned17:50
*** gbarros has joined #openstack-nova17:50
*** imacdonn has joined #openstack-nova17:50
*** awalende has quit IRC17:50
sean-k-mooneyaspiers: commented on https://review.opendev.org/#/c/684825/4 over all this address the reported bug but it misses another edgecase where qemu virtio channeles are not handeled17:52
gmanni see. then let's include in sample files as it is returned.17:53
mriedemgmann: sorry, what is returned?17:54
mriedemhost_status nor security_groups are returned for GET /servers/{server_id} when the cell is down17:55
mriedemshow and detail are different in the down cell case b/c in the detail case we won't even get to https://github.com/openstack/nova/blob/961c2945491ebcea3cf1cb175a06d057155aa5a5/nova/api/openstack/compute/views/servers.py#L146 because we will have already filtered out the instances from the down cells17:55
gmannmriedem: yeah but 'security_groups' are returned in GET /servers/details case17:55
mriedemnot if the instance is in a down cell17:56
gmannmriedem: due to this - https://github.com/openstack/nova/blob/961c2945491ebcea3cf1cb175a06d057155aa5a5/nova/api/openstack/compute/views/servers.py#L41017:56
*** psachin has quit IRC17:56
mriedemgmann: you will not get there with an instance from a down cell17:56
mriedemthe multi-cell instance lister code will filter out those results17:56
gmannbecause for detail case it is added after show method return17:56
mriedem...17:57
mriedemagain,17:57
mriedemGET /servers/detail will filter out instances from down cells17:57
mriedemso we will not get to https://github.com/openstack/nova/blob/961c2945491ebcea3cf1cb175a06d057155aa5a5/nova/api/openstack/compute/views/servers.py#L410 for instances from down cells17:57
mriedemso i don't see the problem17:57
mriedemoh i think i see17:58
mriedemhttps://github.com/openstack/nova/blob/961c2945491ebcea3cf1cb175a06d057155aa5a5/nova/api/openstack/compute/views/servers.py#L39617:58
mriedemself._list_view(self.show17:58
mriedembut still,17:58
mriedeminstances from a down cell won't get that far17:58
mriedemgmann: are you just basing this on code inspection?17:58
gmannmriedem: this is from tests which I am not sure does actual down cell handling - https://review.opendev.org/#/c/684335/5/doc/api_samples/servers/v2.69/servers-details-resp.json17:59
*** itlinux has quit IRC17:59
gmannso instance from down-cell will be there in GET detail case or not17:59
mriedemwill not18:00
*** gbarros has quit IRC18:00
gmann*so not sure if  instance18:00
gmannok18:00
mriedemcompute API get_all will filter those out18:00
*** jmlowe has quit IRC18:00
mriedemyou would need to follow this thread https://github.com/openstack/nova/blob/961c2945491ebcea3cf1cb175a06d057155aa5a5/nova/compute/api.py#L276018:00
*** gbarros has joined #openstack-nova18:02
gmannbut cell-down-support will be true for > 2.69 - https://github.com/openstack/nova/blob/961c2945491ebcea3cf1cb175a06d057155aa5a5/nova/api/openstack/compute/servers.py#L32918:02
gmannthen it will include the instances form down cell also18:03
mriedemno18:03
mriedemthat's not what that means18:03
mriedemyou need to look at the InstanceLister code18:03
aspierssean-k-mooney: I've replied and emailed the SEV gurus to ask them18:03
*** gbarros has quit IRC18:03
aspierssean-k-mooney: I would prefer to deal with that in a follow-up though18:03
aspiersespecially considering we don't actually know for sure yet it's an issue18:04
sean-k-mooneyyep we could18:04
sean-k-mooneythe code as written should be correct and adress the current bug18:04
mriedemgmann: in here https://github.com/openstack/nova/blob/961c2945491ebcea3cf1cb175a06d057155aa5a5/nova/compute/multi_cell_list.py#L41418:05
sean-k-mooneybut i think we should file a bug for the qemu-channel too18:05
sean-k-mooneyaspiers: if ye file a bug for it im +1 i just dont want to loose track of it18:05
sean-k-mooneyaspiers: we can always close it as invalid if its not required18:05
*** gbarros has joined #openstack-nova18:06
sean-k-mooneyaspiers: did ye try to boot a vm with a spice console and sev?18:06
aspierssean-k-mooney: I'll wait until the gurus respond, they're normally very fast and that can potentially save me the effort of filing18:06
aspierssean-k-mooney: yes I booted with spice fine18:06
aspiersor at least I'm pretty sure18:06
aspiersI tried a lot of different combos, can't remember all the details18:06
aspiersdefinitely saw one minor virtio device without iommu18:07
aspiersand it didn't crash18:07
aspiersin fact I think I checked about that with our guru and he said don't worry about it... think it was mouse or USB input of some kind18:07
aspiersheading off now o/18:09
sean-k-mooneyaspiers: its used for spices usb fowradign support that allows usb devices to be hotpluged at runtime18:09
sean-k-mooneywithout requireing libvirt to modify the xml18:09
sean-k-mooneyi belive it also allows spcie to do remote hotpug18:09
sean-k-mooneye.g. a local usb to a remove vm like RDP does with drives and printers18:10
gmannmriedem: thanks. that param in multi cell control only exception or skip to prepare the minimum response in APi view.18:15
gmannwe need to mock the same in sample tests also.18:18
*** gmann is now known as gmann_afk18:19
mriedemgmann_afk: ok. there are samples for 2.69 which use a down cell fixture so maybe something is just missing from that, idk.18:21
*** ricolin has quit IRC18:27
*** elod has quit IRC18:28
*** jmlowe has joined #openstack-nova18:41
*** ociuhandu has joined #openstack-nova18:45
*** Sundar has quit IRC18:53
*** maciejjozefczyk has quit IRC18:54
*** elod has joined #openstack-nova18:55
*** BjoernT has joined #openstack-nova18:56
mriedemdansmith: maybe we should finally fix this :) https://bugs.launchpad.net/nova/+bug/178128618:56
openstackLaunchpad bug 1781286 in OpenStack Compute (nova) "CantStartEngineError in cell conductor during reschedule - get_host_availability_zone up-call" [Medium,Triaged]18:56
mriedemi think it's just a matter of shoving the AZ into the Selection object that we pass around in conductor18:57
mriedemtricky thing is functional testing since we don't have a great way to poison conductor on a reschedule to not hit the API DB18:58
dansmithack18:58
dansmithmriedem: in your reply you said cern is running without a superconductor.. you just mean their cell conductors have api creds right?18:59
mriedemyeah18:59
mriedembut,18:59
mriedemi don't think they make a distinction between a top and cell level conductor18:59
dansmithbecause unless I'm missing something, you can't not have superconductors, as you need something listening on the conductor topic that isn't going to be confused for cell traffic18:59
dansmithI guess you could have your apis set to penalize one cell's conductors to do the super work, if that made sense in your topology (i.e. one big cell and a bunch of smaller ones or something)19:00
mriedemhmm, i just assumed they were doing whatever devstack does for singleconductor19:00
mriedemwhich still has cell0 and cell119:00
mriedemright, that's what i'm assuming cern does19:00
dansmithwell, if you do, then cell1 ends up doing all the work for cell2 for example19:00
mriedemor just a handful of big conductor19:00
mriedemwe could ask them but no one in channel19:00
*** jmlowe has quit IRC19:00
dansmithfor a deployment tool, they'd have to have some way of letting you pick which cell should be burdened with the extra work19:01
dansmithwhich seems less than ideal to me19:01
mriedemas for the console proxy auth stuff,19:03
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/rocky/implemented/convert-consoles-to-objects.html19:03
mriedem"instead we will resolve the cell database issue by running console proxies per cell instead of global to a deployment, such that the cell database is local to the console proxy"19:03
mriedemi couldn't find that in our docs19:03
mriedemwhich seems...bad19:03
mriedemlike it should be in https://docs.openstack.org/nova/latest/user/cellsv2-layout.html19:03
mriedemmelwitt: ^?19:03
mriedemdid the console proxy per cell thing get into the docs?19:03
dansmithyeah, I couldn't either, which is why I punted to her19:04
*** ociuhandu has quit IRC19:04
mriedemnothing here either https://docs.openstack.org/nova/latest/admin/remote-console-access.html19:04
mriedemmgoddard: when you say kolla-ansible is adding support for cells v2, you mean multiple cells specifically right? because cells v2 (cell0, cell1, mappings, etc) has been required since ocata19:11
dansmithmriedem: I think he said specifically adding a cell to an existing deployment19:12
mriedemah yeah at the end, "I'm trying to design it such that it's possible to deploy a new cell in an existing environment"19:13
dansmithaye19:13
mriedemhere is a docs bug for the console proxy per cell thing anyway https://bugs.launchpad.net/nova/+bug/184604519:18
openstackLaunchpad bug 1846045 in OpenStack Compute (nova) "Docs don't mention running console proxies per cell" [Undecided,New]19:18
*** jmlowe has joined #openstack-nova19:20
*** ccamacho has joined #openstack-nova19:22
*** luksky has joined #openstack-nova19:23
openstackgerritEric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated  https://review.opendev.org/68518119:38
*** nweinber has quit IRC20:01
*** markvoelker has quit IRC20:04
openstackgerritEric Fried proposed openstack/nova master: DNM: Alt: Extract some helper functions from DriverVolumeBlockDevice  https://review.opendev.org/68578220:14
*** markguz_ has joined #openstack-nova20:40
*** ralonsoh has quit IRC20:49
*** ociuhandu has joined #openstack-nova20:49
*** pcaruana has quit IRC20:50
*** markvoelker has joined #openstack-nova20:52
*** irclogbot_3 has quit IRC20:55
*** irclogbot_0 has joined #openstack-nova20:56
*** ccamacho has quit IRC20:57
*** irclogbot_0 has quit IRC21:01
*** irclogbot_1 has joined #openstack-nova21:02
openstackgerritMerged openstack/nova master: make virtual pmem feature compatible with python3  https://review.opendev.org/68559321:03
efrieddansmith: ^21:04
*** irclogbot_1 has quit IRC21:07
*** irclogbot_3 has joined #openstack-nova21:08
*** ociuhandu has quit IRC21:12
*** irclogbot_3 has quit IRC21:13
*** irclogbot_3 has joined #openstack-nova21:14
*** xek has quit IRC21:17
*** markguz_ has quit IRC21:18
*** irclogbot_3 has quit IRC21:19
*** gbarros has quit IRC21:19
*** irclogbot_0 has joined #openstack-nova21:20
*** irclogbot_0 has quit IRC21:25
*** irclogbot_3 has joined #openstack-nova21:26
*** irclogbot_3 has quit IRC21:33
*** BjoernT has quit IRC21:34
*** irclogbot_0 has joined #openstack-nova21:36
*** markguz_ has joined #openstack-nova21:39
markguz_Hi, Anyone know of way to group pci passthrough cards together based on a label or tagging?21:39
markguz_for example. I have a compute node with 4 Identical Fiber Channel cards. 2 are connected to SAN 1 and 2 are connected to SAN 2.21:40
markguz_i wanted to be able to group the cards based on what they were connected to, but so far I have not found a way of doing that21:40
markguz_am i out of luck?21:40
*** irclogbot_0 has quit IRC21:41
*** irclogbot_0 has joined #openstack-nova21:42
*** TxGirlGeek has quit IRC21:43
*** TxGirlGeek has joined #openstack-nova21:46
*** irclogbot_0 has quit IRC21:47
*** irclogbot_2 has joined #openstack-nova21:48
efriedmarkguz_: I defer to sean-k-mooney for all things pci alias, but yeah, I don't think you can e.g. have each of two aliases grouping together the cards on the same SAN.21:52
*** irclogbot_2 has quit IRC21:53
*** irclogbot_1 has joined #openstack-nova21:54
markguz_efried: Yeah I thought maybe I could make an alias for a pci slot, but alias doesn't support it21:55
efriednope, only whitelist has wildcarding I think21:55
efriedonce we have placement tracking of devices, your SAN is a trait...21:56
markguz_efried: and I suppose there's anyway to reference the whitelist directly in a flavor?21:56
efriedno, the alias is the level of abstraction available to flavors.21:57
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add Selection.availability_zone field  https://review.opendev.org/68580721:59
*** irclogbot_1 has quit IRC21:59
markguz_efried: bummer.  thanks though.21:59
*** irclogbot_0 has joined #openstack-nova22:00
*** irclogbot_0 has quit IRC22:05
*** dave-mccowan has quit IRC22:05
*** irclogbot_0 has joined #openstack-nova22:06
openstackgerritEric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated  https://review.opendev.org/68518122:06
*** luksky has quit IRC22:13
*** adriant has joined #openstack-nova22:14
*** slaweq has quit IRC22:14
*** irclogbot_0 has quit IRC22:15
*** irclogbot_2 has joined #openstack-nova22:16
*** gbarros has joined #openstack-nova22:22
*** rcernin has joined #openstack-nova22:24
*** mlavalle has quit IRC22:25
*** munimeha1 has quit IRC22:28
*** TxGirlGeek has quit IRC22:31
efriedmriedem: we used to have something in nova-specs that enforced conformance to the template, no??22:39
efriedah, it's hidden in pep822:41
*** TxGirlGeek has joined #openstack-nova22:42
*** mriedem has quit IRC22:44
*** gbarros has quit IRC22:45
*** mriedem has joined #openstack-nova22:45
*** TxGirlGeek has quit IRC22:55
*** markguz_ has quit IRC22:58
openstackgerritEric Fried proposed openstack/nova-specs master: Add 'Core Liaison' section to spec template  https://review.opendev.org/68585722:59
*** gbarros has joined #openstack-nova22:59
*** tkajinam has joined #openstack-nova23:02
*** mriedem has quit IRC23:08
*** sapd1_x has joined #openstack-nova23:12
*** sapd1_x has quit IRC23:22

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