Earlier  
Posted Nick Remark
#openstack-nova - 2019-09-27
21:34:18 mriedem dude, just submit it as a cycle goal
21:34:22 mriedem to de-paste openstack
21:34:25 efried ooo
21:34:28 mriedem people will just sign up to do it
21:35:30 lbragstad lol
21:36:38 efried see ^
21:36:48 efried lbragstad is just itching to rip paste out of nova
21:36:57 mriedem no that's just a rash
21:37:00 mriedem he's rashy
21:37:04 mriedem diaper wise
21:37:17 efried razor burn
21:37:26 mriedem "rashy lance" we used to call him at ibm
21:37:49 mriedem with all of that helpfulness, i'm signing off, o/
21:37:53 efried o/
21:43:13 openstackgerrit Eric Fried proposed openstack/nova-specs master: Re-proposed Nova Cyborg interaction specification. https://review.opendev.org/682637
22:08:44 openstackgerrit Merged openstack/nova-specs master: Re-proposed Nova Cyborg interaction specification. https://review.opendev.org/682637
23:00:49 openstackgerrit Merged openstack/nova stable/rocky: lxc: make use of filter python3 compatible https://review.opendev.org/676498
#openstack-nova - 2019-09-28
00:00:27 openstackgerrit wangfaxin proposed openstack/nova master: fixd notes: neuton is replayced with neutron in line 756. https://review.opendev.org/685509
02:02:22 lbragstad efried regarding the out-of-tree extension stuff... we broke that for keystone
02:03:30 lbragstad we formally pulled relevant middleware into the project and then added a few config options enable it
02:04:09 lbragstad otherwise - we punted on allowing people to put their own middleware in paste, kmalloc did all of that conversion for us
02:04:20 lbragstad TMMV
02:04:33 lbragstad s/TMMV/YMMV/
07:24:03 openstackgerrit Merged openstack/nova master: Stop filtering out 'accepted' for in-progress migrations https://review.opendev.org/684409
09:15:30 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/train: Imported Translations from Zanata https://review.opendev.org/685534
#openstack-nova - 2019-09-29
01:54:19 openstackgerrit hulina proposed openstack/nova master: Fix spelling error https://review.opendev.org/685540
09:00:29 openstackgerrit Luyao Zhong proposed openstack/nova master: make virtual pmem feature compatible with python3 https://review.opendev.org/685593
09:04:59 openstackgerrit Luyao Zhong proposed openstack/nova master: make virtual pmem feature compatible with python3 https://review.opendev.org/685593
23:35:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: Update contributor guide for Ussuri https://review.opendev.org/685630
#openstack-nova - 2019-09-30
03:16:21 openstackgerrit Rui Zang proposed openstack/nova master: make virtual pmem feature compatible with python3 https://review.opendev.org/685593
05:43:29 openstackgerrit Takashi NATSUME proposed openstack/nova master: Replace 'fake' with a real project ID https://review.opendev.org/685645
05:58:54 openstackgerrit Rui Zang proposed openstack/nova master: make virtual pmem feature compatible with python3 https://review.opendev.org/685593
07:23:51 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM libvirt: extract _start_heterogeneous_computes https://review.opendev.org/682620
07:23:55 openstackgerrit Lee Yarwood proposed openstack/nova master: Add regression test for bug #1843639 https://review.opendev.org/682621
07:23:55 openstack bug 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:59 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Ignore volume exceptions during post_live_migration https://review.opendev.org/682622
07:25:16 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Remove stale BDMs on reserve_block_device_name failure https://review.opendev.org/682594
07:29:37 openstackgerrit wangfaxin proposed openstack/nova master: fixd notes: neuton is replaced with neutron in line 756. https://review.opendev.org/685509
09:00:03 openstackgerrit Seyeong Kim proposed openstack/nova master: block_device: Optionally recreate attachments when refreshing connection_info https://review.opendev.org/579004
09:12:12 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Fix invalid link index https://review.opendev.org/685664
09:15:17 bauzas gibi: <3 you
09:15:24 bauzas gibi: found a shit number of helpers
09:15:35 bauzas in the integrated helpers for my functional tests
09:15:39 bauzas will make them easier to read
09:15:50 gibi bauzas: I'm glad that those can help you
09:20:33 bauzas gibi: the only issue is that I discovered those helpers after some time
09:21:07 gibi yeah it is hard how to make such (and other) helper codes discoverable
09:21:35 bauzas unless by reviewing others's patches :)
09:22:14 gibi but nobody can review every incoming patch in nova to know what happens
09:27:28 bauzas ...also if they have a lot of other things to do internally :)
09:29:54 aspiers Is 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:31:36 aspiers AFAICS, if hw_scsi_model is not specified, there will be no SCSI controller
09:32:10 aspiers but also, "virtio-blk" is not a valid value in nova.objects.fields.SCSIModel
09:36:44 aspiers and in fact that doc is missing all the other valid values
10:16:03 aspiers ah, I think I figured it out
10:16:29 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Use the `preferred` memory allocation mode for NUMA https://review.opendev.org/684375
10:16:36 aspiers the default when hw_scsi_model is not specified is no SCSI controller, and something like <target dev='vda' bus='virtio'/>
10:17:03 aspiers which is colloquially referred to as virtio-blk, but it's just 'virtio' in the code
10:23:23 sean-k-mooney aspiers: yes the defualt for libvirt/kvm is virtio or virtio-blk or virtio block
10:23:58 aspiers sean-k-mooney: thanks for confirmation. just submitted https://review.opendev.org/#/c/685674/
10:24:06 sean-k-mooney if you want to use virto-scsi you need to set the disk bus to scsi and then set the scsi model to virtio-scsi
10:25:16 sean-k-mooney the default storage contole depend on the hypervisor and virt type
10:25:32 sean-k-mooney it wont alwyas be virtio if you dont set hw_disk_bus
10:28:37 sean-k-mooney aspiers: 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-L296
10:31:52 sean-k-mooney i 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 os
10:31:54 sean-k-mooney https://github.com/openstack/nova/blob/master/nova/virt/osinfo.py#L114-L147
10:33:16 sean-k-mooney but 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 correctly
10:39:24 openstackgerrit Dongcan Ye proposed openstack/nova master: libvirt: Check the guest support UEFI https://review.opendev.org/685678
11:32:53 sean-k-mooney lyarwood: care to hit some easy sable patches in os-vif
11:33:05 sean-k-mooney https://review.opendev.org/#/c/683489/ https://review.opendev.org/#/c/683488/
11:33:37 sean-k-mooney well not backports release stuff. just the git review and constraits file updates for train
11:48:41 lyarwood sean-k-mooney: ack looking
11:52:09 lyarwood sean-k-mooney: done
12:15:07 openstackgerrit Seyeong Kim proposed openstack/nova master: block_device: Optionally recreate attachments when refreshing connection_info https://review.opendev.org/579004
13:04:34 openstackgerrit Merged openstack/os-vif master: Update the constraints url https://review.opendev.org/683873
13:12:28 openstackgerrit Merged openstack/os-vif master: Update master for stable/train https://review.opendev.org/683490
13:27:12 bauzas mriedem: morningh
13:27:20 bauzas mriedem: reviewing a shit number of times https://review.opendev.org/#/c/678100/
13:29:10 bauzas IIUC, we leak the source allocation by https://review.opendev.org/#/c/678100/1/nova/tests/functional/wsgi/test_services.py
13:35:38 openstackgerrit Merged openstack/os-vif stable/train: Update .gitreview for stable/train https://review.opendev.org/683488
13:43:38 mriedem bauzas: i'm not following the question
13:43:50 bauzas sorry then
13:44:02 mriedem for evacuate we leak the source allocation when deleting the source compute service
13:44:13 mriedem b/c we fail to delete the resource provider b/c it has allocatoins against it from the evacuate
13:44:14 bauzas trying to exactly understand how to get an orphan allocation when resizeing
13:44:25 bauzas as I want to write some functest
13:44:33 mriedem for resize, you'd migrate the server so it gets to VERIFY_RESIZE, then delete the source compute service
13:44:43 bauzas for the moment, I did : create the instance, resize, delete the service
13:44:46 mriedem yes
13:44:56 mriedem the source resource provider will have allocations held by the migration record
13:45:04 bauzas but when confirming the resize, I get an exception
13:45:22 mriedem that's separate from the leaked allocation issue
13:46:03 bauzas hum, so, once I delete the service, I'll just delete the instance?
13:46:04 mriedem for 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 one
13:46:20 mriedem i guess i'd have to know what the error is when you go to confirm the resize
13:46:35 bauzas secf
13:47:36 bauzas mriedem: http://paste.openstack.org/show/780354/
13:48:22 bauzas when confirming the resize, I get an exception saying that it doesn't get the source compute UUID
13:49:20 mriedem does the test stop/start a new compute service after deleting the original source compute service?
13:49:30 mriedem starting a new compute service is going to create a new compute node record

Earlier   Later