Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-27
21:03:18 cfriesen sean-k-mooney[m]: you're correct that it's no worse than it is now
21:03:18 sean-k-mooney[m] cfriesen: yes and today that can happen too. the guest sees 1 numa node but on the host ist ram can be allocate form multiple because it floats
21:03:28 jaypipes My brain is, frankly, pretty fried.
21:03:32 efried I've thought through all the things you can't express with it, and it boils down to just one thing: you can't express "separate these request groups".
21:03:41 jaypipes I really need to eat and refresh my brain juices.
21:03:54 efried ...but adding that feature would be relatively straightforward with the addition of some new syntax.
21:04:16 efried Yeah, for my part I really ought to go review some more specs that aren't already approved.
21:06:19 sean-k-mooney[m] cfriesen: :) so i was suggesting was if you did not spcify a numa topology then we leave it up to the virt-driver but it then has to reflect the topolgy to the guest. it would be no wose then today but actully fix the 4k pages issue and technically i dont think we are breaking any guartees as i dont hink we define that if numa_nodes is not set its 1 numa node
21:07:41 sean-k-mooney[m] cfriesen: anyway that an issue for another day
21:11:22 openstackgerrit Hongbin Lu proposed openstack/nova-specs master: Choose default network on ambiguity https://review.openstack.org/520247
21:11:50 openstackgerrit Ed Leafe proposed openstack/nova-specs master: Add Generation to Consumers https://review.openstack.org/556971
21:12:09 edleafe cdent: efried: jaypipes: ^^ updated with your suggestions
21:14:38 efried edleafe: Check yer whitespace
21:15:46 edleafe efried: ugh, do I have to? <whine>
21:15:55 efried edleafe: But hold, there's another typo
21:16:02 efried edleafe: And another thing that needs to be said.
21:16:54 edleafe well, I wasn't gonna push a rev for whitespace until everyone had a crack at it
21:20:23 efried edleafe: Never mind that last thing. I was gonna ask if the generation in PUT /allocations/{c} was going to quit being ignored. But that's a RP generation, and not really relevant here.
21:21:04 efried edleafe: Consider me cracked.
21:22:24 sean-k-mooney[m] you know that feeling when you find the souce of a bug you hit and you wish you had not...
21:23:04 sean-k-mooney[m] here we convert form the disk bus the use asked for to a prefix https://github.com/openstack/nova/blob/ef0ce4d692d28a7f5a0079e24acdbfe7d2767e8b/nova/virt/libvirt/blockinfo.py#L124-L143
21:23:15 cdent cracked
21:23:23 sean-k-mooney[m] here we convert form the prefix to the disk bus https://github.com/openstack/nova/blob/ef0ce4d692d28a7f5a0079e24acdbfe7d2767e8b/nova/virt/libvirt/blockinfo.py#L297-L304
21:23:54 sean-k-mooney[m] we map sata scis and usb to sd then hard code sd = scsi
21:24:40 edleafe efried: I always have
21:24:40 sean-k-mooney[m] that mean on kvm you can request sata or usb disk which is why i had to go to IDE to fix my centos image
21:25:10 sean-k-mooney[m] *cant request
21:26:30 openstackgerrit Hongbin Lu proposed openstack/nova-specs master: Choose default network on ambiguity https://review.openstack.org/520247
21:27:20 sean-k-mooney[m] setting the disk_bus for kvm host has apreantely been broken for 5 years since https://github.com/openstack/nova/commit/7be531fe9462f2b07d4a1abf6687f649d1dfbb89 was merged ...
21:28:25 openstackgerrit Ed Leafe proposed openstack/nova-specs master: Add Generation to Consumers https://review.openstack.org/556971
21:29:18 openstackgerrit Merged openstack/nova-specs master: Amend the migration paging spec for uuid in server migrations response https://review.openstack.org/532904
21:31:01 edleafe cdent: efried: jaypipes: ^^ fixed those ghastly errors
21:31:18 efried edleafe: +1
21:35:29 mriedem jaypipes: i'll come back to the mirror aggregates in placement spec, but not today, burned out
21:37:54 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857
21:39:02 cdent thanks for putting some eyes on the error code spec jaypipes
21:41:34 openstackgerrit Sylvain Bauza proposed openstack/nova-specs master: Proposes Multiple GPU types https://review.openstack.org/557065
21:41:54 bauzas jaypipes: mriedem: just created the spec we discussed about multiple types
21:42:10 bauzas it's a very simple spec, but needs agreement
21:43:06 openstackgerrit Merged openstack/nova-specs master: Address review comments from afdc828db3c9d0205b6ded268db24f5cdf857fa6 https://review.openstack.org/554251
21:46:31 mriedem bauzas: i don't remember talking about that at all
21:46:33 melwitt bauzas: on https://blueprints.launchpad.net/nova/+spec/vgpu-rocky it looks like you can just use that bp to link with your spec since it was meant to cover the multiple vgpu types part anyway. the spec is just needed to facilitate discussion on the details, right?
21:46:40 mriedem i might have been in ireland when it happened, but didn't talk about it
21:47:50 melwitt yeah, I don't remember particulars about the vgpu discussion other than, there's more work to do this cycle and we're agreed to do it and review it
21:48:16 bauzas mriedem: no worries, I pinged you a while ago just about whether I should use another BP for it, and you told me to ping melwitt
21:48:41 bauzas melwitt: yup, zactly
21:48:46 bauzas melwitt: I linked that BP to the spec
21:48:58 melwitt okay, cool. thanks
21:49:12 bauzas so, I just want to make clear that the spec only covers the problem in it
21:49:20 bauzas which requires a consensus, hence a spec
21:49:37 bauzas other feature patches won't need it
21:49:38 melwitt right, just have to agree how to implement the multiple types
21:49:41 melwitt yeah
21:50:12 bauzas I'd love jianghuaw_ to voice on that spec too
21:54:03 bauzas anyway, calling it a day \o
22:11:14 jaypipes mriedem_afk: you and me both, man :)
22:21:13 openstackgerrit Ed Leafe proposed openstack/nova-specs master: Add Generation to Consumers https://review.openstack.org/556971
22:21:29 edleafe ^^ damn rST double colons!
22:37:42 sean-k-mooney[m] bauzas: can you review the bug fix i have up for the libvirt mtu here https://review.openstack.org/#/c/553072/ when you have time. i think this is also a good backport candiate.
22:38:04 sean-k-mooney[m] i also found another lovely bug today https://bugs.launchpad.net/nova/+bug/1759420
22:38:05 openstack Launchpad bug 1759420 in OpenStack Compute (nova) "nova does not correctly support HW_DISK_BUS=sata or usb for kvm/qemu" [Undecided,New]
22:40:40 sean-k-mooney[m] because of ^ i had to for a centos vm that was given to me to run with disks attached to ide because it was freaking out with scsi or virtio disks. the virtio_blk and virtio_scsi drivers were causeing the kernel to lock up
22:47:35 openstackgerrit Eric Fried proposed openstack/nova master: Remove usage of [placement]os_region_name https://review.openstack.org/557086
22:47:38 efried edleafe: ^
22:56:12 openstackgerrit Eric Fried proposed openstack/nova master: Slugification utilities for placement names https://review.openstack.org/556628
22:58:14 openstackgerrit Michael Still proposed openstack/nova master: Move configurable mkfs to privsep. https://review.openstack.org/551921
22:58:14 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi xenstore_read's to privsep. https://review.openstack.org/552241
22:58:15 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242
22:58:15 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi partition copies to privsep. https://review.openstack.org/553605
22:58:16 openstackgerrit Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437
22:58:16 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
22:58:17 openstackgerrit Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439
22:59:39 mikal efried: I've replied to your comments on the first patch, but the only changes were in comments. I'm not sure that was worth a -1.
23:01:03 sean-k-mooney[m] mikal: no more rootwap will be quite nice to see :)
23:01:20 sean-k-mooney[m] mikal: also i like you blueprint url
23:01:46 mikal sean-k-mooney[m]: ta, I do try. Other blueprints you might enjoy include execs-ive-had-a-few...
23:06:44 mlavalle jaypipes: could we structure this RP tree https://review.openstack.org/#/c/502306/22/specs/rocky/approved/bandwidth-resource-provider.rst@432 so an allocation can be satisfied by ANY of the networks in the tree?
23:11:18 jaypipes mlavalle: not sure I'm following your question...
23:12:37 mlavalle jaypipes: I'll pose my question in the spec be cause I have to run in 15 minutes
23:12:44 mlavalle thanks for replying
#openstack-nova - 2018-03-28
00:29:27 openstackgerrit Merged openstack/nova-specs master: Add serial numbers for local disks https://review.openstack.org/556565
00:55:01 openstackgerrit Michael Still proposed openstack/nova master: Remove duplicative implementation of temporary directories. https://review.openstack.org/554791
00:55:01 openstackgerrit Michael Still proposed openstack/nova master: Use a pythonic delete. https://review.openstack.org/554792
00:55:02 openstackgerrit Michael Still proposed openstack/nova master: Use a pythonic delete, with a retry. https://review.openstack.org/554793
01:14:33 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Allow abort live migrations in queued status https://review.openstack.org/536722
01:23:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (2/3) https://review.openstack.org/550648
01:24:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (3/3) https://review.openstack.org/550659
01:25:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
01:58:51 Spaz-Home Morning novafriends
02:21:38 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925
02:36:50 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Extract generate_hostid method into utils.py https://review.openstack.org/555282
02:42:40 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
02:45:13 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
03:10:14 openstackgerrit Ghanshyam Mann proposed openstack/nova-specs master: Spec for Granular API policy https://review.openstack.org/547850
03:18:39 openstackgerrit Nguyen Hai proposed openstack/nova-specs master: Enhance nova-specs webpage and clean up repo https://review.openstack.org/551802
03:20:27 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
03:33:43 openstackgerrit Jianle He proposed openstack/nova-specs master: Back up volume-backed-instance https://review.openstack.org/530214
04:02:29 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova-specs master: Fix typo in GET os-instance-actions url. https://review.openstack.org/557143
04:09:30 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: [WIP] Add host/hostId to instance action events API https://review.openstack.org/557145
04:51:10 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Fix comments in get_all_with_shared() https://review.openstack.org/533195

Earlier   Later