Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-26
19:46:52 mriedem ovo actually has a ComparableVersionedObject mixin
19:47:01 mriedem we only use it in one place in nova's objects, the Selection object
19:47:41 efried but you only want to use the UUID to compare the lists?
19:54:04 mriedem i think i might have it
19:55:16 mriedem yup got it
19:55:41 mriedem ffs, you know how sometimes you come across a programming problem which seems like something you dealt with on a daily basis when you were a junior in college?
19:56:00 mriedem and now, getting paid high professional tech person salary, you feel dumb as a brick
19:56:38 efried college is way harder than real life
19:57:41 efried imacdonn: Okay, I'm there. The bug is the .update, which is patently wrong.
19:57:51 efried imacdonn: I convinced myself by solving it a different way: http://paste.openstack.org/show/730962/
19:58:00 efried which isn't better than what you've done, just different.
19:59:04 imacdonn efried: I tried to do it your way, but I got a complaint about tuples ... oh, you changed it to a list
19:59:12 efried yeah
19:59:19 efried no reason for it to be a tuple afaics
19:59:41 efried anyway, I'm approving your patch. Good catch, good fix.
19:59:56 imacdonn cool, thanks!
20:51:28 openstackgerrit Merged openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
20:51:37 openstackgerrit Merged openstack/nova master: cells: Be explicit in docs about service restarts https://review.openstack.org/603588
20:51:47 openstackgerrit Merged openstack/nova master: api-ref: add 'migrations' param to GET /os-migrations https://review.openstack.org/605232
20:51:55 openstackgerrit Merged openstack/nova master: Ignore VirtDriverNotReady in _sync_power_states periodic task https://review.openstack.org/604376
20:58:48 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Ignore VirtDriverNotReady in _sync_power_states periodic task https://review.openstack.org/605533
21:27:35 imacdonn mriedem: something's troubling me about this online migrations stuff
21:27:40 imacdonn https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L374-L386
21:28:26 imacdonn if there are more than max-count non-migratable rows, wouldn't the method return 50 found and 0 done... even though there could be some that are migratable ?
21:29:16 imacdonn is the migration method supposed to be smart enough to attempt the migratable ones first ?
21:44:10 mriedem i'm not sure what a non-migrateable record is
21:44:29 mriedem dansmith would be the best person to ask about that comment but he's out the rest of this week
21:45:42 imacdonn my interpretation was that there are some migrations that cannot always be applied to all rows .... tho it's not clear what the implications would be
22:05:34 openstackgerrit Chris Friesen proposed openstack/nova-specs master: Add support for emulated virtual TPM https://review.openstack.org/571111
22:08:34 openstackgerrit Chris Friesen proposed openstack/nova-specs master: Add support for emulated virtual TPM https://review.openstack.org/571111
23:08:07 mriedem holy god 609 things in the check queue
23:23:48 openstackgerrit melanie witt proposed openstack/nova master: Revert "Deprecate the nova-consoleauth service" https://review.openstack.org/605250
23:23:48 openstackgerrit melanie witt proposed openstack/nova master: Un-deprecate the nova-consoleauth service https://review.openstack.org/605255
23:25:32 openstackgerrit melanie witt proposed openstack/nova master: Revert "Deprecate the nova-consoleauth service" https://review.openstack.org/605250
23:25:33 openstackgerrit melanie witt proposed openstack/nova master: Un-deprecate the nova-consoleauth service https://review.openstack.org/605255
23:29:50 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Cross-cell resize https://review.openstack.org/603930
#openstack-nova - 2018-09-27
01:25:46 alex_xu gmann: sorry, I didn't back at that time yesterday, I have late meeting at yesterday lunchtime
01:27:04 gmann alex_xu: no prob.
01:28:12 gmann cfriesen: did not get completely about query on microversion need. are you saying to return 400 if image properties and flavor extra-spec if in rebuild/resize/create request ?
01:52:40 openstackgerrit zhaodan7597 proposed openstack/nova master: Unable to delete volume when a vmware instance bfv is failed. https://review.openstack.org/571112
02:03:29 cfriesen gmann: we're planning on adding extra validation for flavor extra-specs and image properties for operations that could change either or both of them. if the combination of flavour extraspecs and image properties doesn't make sense, we will return an error (400 presumably) back to the user.
02:04:19 cfriesen gmann: currently this request would be accepted but would fail later on down on the compute node, but that's an RPC cast and so the user wouldn't get an error message.
02:08:07 gmann cfriesen: ok, what all APIs? i am wondering if that is taken care by additionalProperties or not.
03:07:34 openstackgerrit Brin Zhang proposed openstack/nova master: Add support volume_type in compute api https://review.openstack.org/605573
03:16:42 openstackgerrit Brin Zhang proposed openstack/nova master: Add support volume_type in compute api https://review.openstack.org/605573
03:20:39 openstackgerrit zhaodan7597 proposed openstack/nova master: Unable to delete volume when a vmware instance bfv is failed. https://review.openstack.org/571112
03:21:50 openstackgerrit zhaodan7597 proposed openstack/nova master: Unable to delete volume when a vmware instance bfv is failed. https://review.openstack.org/571112
03:26:08 brinzhang Kevin_zheng: Take a look at this patch https://review.openstack.org/#/c/605573/2/nova/compute/api.py
03:27:40 brinzhang Kevin_zheng: VOLUME_TYPE_MIN_COMPUTE_VERSION = 52 this variable is not need, instead of it with CINDER_V3_VOLUME_TYPE_MIN_COMPUTE_VERSION = 35
03:28:31 brinzhang to check the volume type is supported in cinder min version.
03:29:33 Kevin_Zheng you should re arrange your patchsets, it now seems very hard to follow
03:33:52 Kevin_Zheng replied in your new patch
03:40:20 brinzhang Yeah, updating
04:11:32 openstackgerrit Merged openstack/nova master: Revert "Make host_aggregate_map dictionary case-insensitive" https://review.openstack.org/604898
04:12:07 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
04:12:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106
04:13:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110
05:42:50 openstackgerrit fupingxie proposed openstack/nova master: Don't recreate inst_base on source when using rbd backend in resize https://review.openstack.org/605590
06:05:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
06:05:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
06:13:18 openstackgerrit Tao Li proposed openstack/nova master: Rollback instance vm_state to original where instance claims failed https://review.openstack.org/592252
06:35:07 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: placement: Always reset conf.CONF when starting the wsgi app https://review.openstack.org/604694
06:45:31 openstackgerrit Bryan Song proposed openstack/nova master: Creation image for volume-backend instance should use volume size in image property 'min_disk' https://review.openstack.org/605596
07:03:35 openstackgerrit Tao Li proposed openstack/nova master: Don't persist retry information into database https://review.openstack.org/605011
07:20:30 openstackgerrit Merged openstack/nova master: consumer gen: move_allocations https://review.openstack.org/591810
07:24:48 openstackgerrit huanhongda proposed openstack/nova master: Allow to attach/detach port when vm_state is soft-delete https://review.openstack.org/605602
08:00:02 openstackgerrit huanhongda proposed openstack/nova master: Allow to attach/detach port when vm_state is soft-delete https://review.openstack.org/605602
08:21:05 kashyap gibi: Morning, want to put this through: https://review.openstack.org/#/c/605060/
08:21:12 kashyap gibi: We got confirmation from all the relevant distros
08:21:23 gibi kashyap: good morning. looking
08:21:33 kashyap Thank you!
08:25:51 gibi kashyap: there are couple of FIXMEs in https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix regarding minimum libvirt and qemu versions
08:26:18 gibi kashyap: I guess if we get the relevant infos from the distros then we can fill those out now
08:26:48 kashyap gibi: Yep, I just sent the two reminders to Iain from Oracle and Colleen from SUSE to fill the FIXME there
08:27:08 kashyap gibi: I added the FIXMEs there :-)
08:27:18 gibi kashyap: cool :)
08:27:40 gibi kashyap: +2
08:28:01 kashyap Sweet, thank you!
08:28:53 gibi kashyap: thank you for picking this work up
08:29:39 kashyap gibi: No worries; I did that last cycle, and once or twice before too. Thought I'd "remove the bandage quickly" this time too :-)
08:41:25 stephenfin kashyap: I assume you're going to follow that up with a patch to bump the current minimums?
08:41:39 kashyap stephenfin: Yeah, indeed.
08:41:56 kashyap stephenfin: Want to ACK the above, already got one from gibi
08:42:22 stephenfin I can. Just reading through the notes on the Wiki first
08:42:25 kashyap Sure
08:42:59 kashyap stephenfin: In short: Oracle Linux already has the relevant versions we bumped to and SLES will have it (Colleen confirmed on the review).
08:43:06 kashyap And the rest of the distributions already have it.
08:43:26 kashyap That's the thread on the list: lists.openstack.org/pipermail/openstack-operators/2018-September/015929.html
08:43:57 kashyap Clicakable: http://lists.openstack.org/pipermail/openstack-operators/2018-September/015929.html
08:53:38 openstackgerrit Radoslav Gerganov proposed openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116
09:17:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628
09:34:16 openstackgerrit Balazs Gibizer proposed openstack/nova master: Follow up for Ib6f95c22ffd3ea235b60db4da32094d49c2efa2a https://review.openstack.org/604743
10:00:36 openstackgerrit Chen proposed openstack/nova master: remove commented-out code https://review.openstack.org/605635
10:18:03 openstackgerrit Brin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance https://review.openstack.org/579360
10:35:30 openstackgerrit Matthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate https://review.openstack.org/578846
10:35:34 openstackgerrit tianhui proposed openstack/nova master: Update doc: launch-instance-from-volume https://review.openstack.org/605640
11:15:48 openstackgerrit Merged openstack/nova master: Pick next minimum libvirt / QEMU versions for "T" release https://review.openstack.org/605060
11:27:17 openstackgerrit Balazs Gibizer proposed openstack/nova master: Followup for Iba230201803ef3d33bccaaf83eb10453eea43f20 https://review.openstack.org/605653
11:57:11 openstackgerrit Balazs Gibizer proposed openstack/nova master: Follow up for Iba230201803ef3d33bccaaf83eb10453eea43f20 https://review.openstack.org/605653
11:57:12 openstackgerrit Balazs Gibizer proposed openstack/nova master: Follow up for Ie991d4b53e9bb5e7ec26da99219178ab7695abf6 https://review.openstack.org/605658

Earlier   Later