Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-24
20:28:00 dansmith the combo
20:28:41 mriedem i should have sniffed this out sooner on the backports given i knew about the earlier work from HPE to try and solve the case insensitivity issues in the api
20:29:02 mriedem mdbooth was also working on something related to this recently
20:29:36 mriedem http://lists.openstack.org/pipermail/openstack-dev/2018-August/thread.html#133202
20:29:51 melwitt yeah, that was the metadata keys case insensitive thing
20:29:59 dansmith mriedem:
20:30:44 mriedem beware then https://review.openstack.org/#/c/504885/
20:30:48 openstackgerrit Dan Smith proposed openstack/nova master: Revert "Make host_aggregate_map dictionary case-insensitive" https://review.openstack.org/604898
20:30:56 melwitt it was confusing thinking about it, but the proposal was to make the key column case sensitive by changing the collation, but that would break anyone who was relying on the case insensitive behavior. and then I got fatigued thinking about it
20:31:30 dansmith yep,
20:31:31 melwitt so I was trying to determine whether there would be any similar thing here
20:31:38 dansmith the difference here is that a hostname with a different name is not legit
20:31:42 dansmith in an aggregate mapping
20:31:56 dansmith since metadata is free-form, it is legit (although silly) to use two keys that differ only by case
20:32:09 dansmith er, "hostname with a different case"
20:32:32 melwitt ok, I see
20:33:06 dansmith mriedem: I can stack the check on top of this if you'll write the tests when I run out of time
20:35:04 dansmith mriedem: we could also just use the name from the mapping instead of the one they asked for,
20:35:18 dansmith which means they won't get a NotFound, and we'll add the mapping they intended
20:35:27 dansmith but they may fail similarly later if they keep using the wrong name
20:35:34 dansmith so I tend to err on the side of fail fast,
20:35:41 dansmith especially since PG will already cause us to behave that way
20:35:44 dansmith thoughts?
20:35:51 mriedem agree on the api being explicitly
20:35:53 mriedem *explicit
20:38:15 melwitt yeah it sounds like fail fast would be better
20:40:56 openstackgerrit Jack Ding proposed openstack/nova master: Correct instance port binding for rebuilds/reboots https://review.openstack.org/603844
20:47:08 jaypipes yeah, I definitely won't be reviewing 504885. Tired of approving these patches and then needing them to be reverted.
20:47:25 jaypipes I'll just stick to reviewing anything that isn't in the API layer I guess.
20:47:41 jaypipes since our API layer is so friggin eggshells.
20:49:26 openstackgerrit Dan Smith proposed openstack/nova master: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/604906
20:49:36 dansmith mriedem: all yours if that fails other tests
20:50:15 openstackgerrit Dan Smith proposed openstack/nova master: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/604906
20:55:00 mriedem ack
21:05:42 melwitt reminder to everyone: forum topic submission deadline is Wed Sep 26
21:06:05 melwitt https://etherpad.openstack.org/p/nova-forum-stein
21:15:37 openstackgerrit Dan Smith proposed openstack/nova master: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/604906
21:15:49 mriedem i don't know if we're going to do a placement extraction session at the forum
21:16:19 mriedem kind of a big thing *not* to have a session about it for at least a status check and fyi
21:16:24 mriedem for people that weren't at the ptg
21:18:19 melwitt that's fair enough
21:18:37 melwitt a status check would be good, at the least
21:20:43 mriedem so i guess i'm submitting that one
21:21:47 melwitt if you could, that would be greeeeaaat
21:22:36 melwitt honestly, it's kinda hard to be thinking about what we need to talk about at the forum when we literally were just at the ptg
21:23:03 melwitt other than picking out, which of these topics should be a repeat at summit time
21:23:13 melwitt or a generic "nova ops/users feedback session"
21:31:38 mriedem https://www.openstack.org/summit/berlin-2018/vote-for-speakers#/22780
21:33:57 melwitt thanks
21:44:36 openstackgerrit Merged openstack/osc-placement stable/queens: import zuul job settings from project-config https://review.openstack.org/601397
#openstack-nova - 2018-09-25
00:11:56 openstackgerrit Merged openstack/nova stable/queens: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/603358
00:12:02 openstackgerrit Merged openstack/nova stable/rocky: libvirt: Use os.stat and os.path.getsize for RAW disk inspection https://review.openstack.org/603488
00:29:19 openstackgerrit Jonte Watford proposed openstack/nova master: Modified version of 0027-Numa-object-string-representations.patch with some updates from the current numa files for nova: numa.py instance_numa_topology.py https://review.openstack.org/600269
01:14:59 brinzhang dansmith: hello, I have a question with this patch(https://review.openstack.org/#/c/604687) ^^
01:16:44 brinzhang dansmith: Is this using NEW_COLUMN_NAME(volume_type) to represent name or uuid in the block_device_mapping table?
01:51:37 openstackgerrit Merged openstack/python-novaclient master: docs: Add redirects https://review.openstack.org/604796
02:31:51 openstackgerrit Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252
02:41:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345
03:20:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629
04:04:50 openstackgerrit Brin Zhang proposed openstack/nova master: Add volume_type field to BlockDeviceMapping object https://review.openstack.org/604687
06:00:17 openstackgerrit Merged openstack/osc-placement stable/rocky: import zuul job settings from project-config https://review.openstack.org/601402
06:24:19 openstackgerrit Merged openstack/python-novaclient master: Add support changes-before for microversion 2.66 https://review.openstack.org/603549
06:39:43 pvc_ anyone guys?
06:39:44 pvc_ ResourceProviderTraitRetrievalFailed: Failed to get traits for resource provider with UUID 206fd270-1c55-4601-947c-a2584ed9be68
06:45:10 openstackgerrit Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252
07:01:24 openstackgerrit Merged openstack/nova master: Wait for network-vif-plugged on resize revert https://review.openstack.org/595069
07:01:57 openstackgerrit Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252
07:12:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix wrong bold decoration https://review.openstack.org/604986
07:50:10 openstackgerrit Przemyslaw Lal proposed openstack/os-vif master: add nested DPDK VIF class for kuryr-kubernetes https://review.openstack.org/604992
07:51:31 bauzas good morning Nova
08:05:46 openstackgerrit huanhongda proposed openstack/nova master: Allow ability for non admin users to use status=DELETED filter on server list https://review.openstack.org/604995
08:16:24 openstackgerrit Naichuan Sun proposed openstack/nova master: os-xenapi(n-rp): add traits for vgpu n-rp https://review.openstack.org/604269
08:18:30 openstackgerrit Naichuan Sun proposed openstack/nova master: os-xenapi(n-rp): add traits for vgpu n-rp https://review.openstack.org/604269
08:43:39 openstackgerrit Merged openstack/os-vif stable/rocky: import zuul job settings from project-config https://review.openstack.org/601404
08:55:37 openstackgerrit Merged openstack/nova stable/queens: Fix service list for disabled compute using MC driver https://review.openstack.org/592311
08:56:45 openstackgerrit Tao Li proposed openstack/nova master: Don't persist retry information to database https://review.openstack.org/605011
08:57:12 openstackgerrit Jason SUN proposed openstack/nova master: doc:update virtual gpu doc https://review.openstack.org/605012
09:01:42 openstackgerrit Jason SUN proposed openstack/nova master: doc:update virtual gpu doc https://review.openstack.org/605012
09:16:54 openstackgerrit Balazs Gibizer proposed openstack/nova master: Consumer gen support for delete instance allocations https://review.openstack.org/591597
09:17:53 gibi jaypipes, efried: ^^ I had to respin it as I failed to create a py3 compatible test
09:18:35 bauzas gibi: I'm working on a respin, but you're next in my todolist
09:19:08 gibi bauzas: cool, thanks
09:34:24 openstackgerrit Vladyslav Drok proposed openstack/nova master: Clean up instance on target node if evacuate fails https://review.openstack.org/375623
09:35:19 openstackgerrit Merged openstack/nova stable/queens: Filter deleted computes from get_all_by_uuids() https://review.openstack.org/604448
09:38:20 openstackgerrit Brin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance https://review.openstack.org/579360
09:51:05 openstackgerrit Matthew Booth proposed openstack/nova master: Raise error on timeout in wait_for_versioned_notifications https://review.openstack.org/604859
10:10:25 openstackgerrit Merged openstack/nova master: Validate transport_url in nova-manage cell_v2 commands https://review.openstack.org/569858
10:17:37 openstackgerrit Brin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance https://review.openstack.org/579360
10:31:03 openstackgerrit Matthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate https://review.openstack.org/578846
10:31:03 openstack bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)
10:31:03 openstackgerrit Matthew Booth proposed openstack/nova master: Add regression test for bug 1550919 https://review.openstack.org/591733
10:52:26 stephenfin moshele: It seems I can use the mstflint package to configure MLX cards and the mlx4_en driver can be configured as-is. Is there any reason to download and run the mlnxofedinstall tool described here https://community.mellanox.com/docs/DOC-2365
10:52:50 stephenfin Just making sure I'm not shooting myself in the foot, heh
10:53:15 lyarwood sigh
10:53:17 lyarwood ops
10:53:29 lyarwood wrong channel, ignore ;)
10:59:49 jaypipes gibi: ack
11:00:55 jaypipes gibi: the remainder of that series seems to need a rebase.. are you working on that?
11:01:18 gibi jaypipes: sure, I'm doing that right now. Also I have to fix general comments from efried there as well
11:01:33 gibi jaypipes: I will ping you when the next patch is up

Earlier   Later