Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-01-07
16:33:57 Sundar dansmith: Thanks, I see it now. Happy New Year, and hope you had a good break.
16:49:25 stephenfin guess there's a first time for everything
16:49:57 stephenfin efried: Meant to get back to you earlier. Was still stuck on the same stuff from before Christmas but I think that's done now. Should get to it tomorrow
16:50:05 stephenfin ditto for huaqiang ^
16:53:14 openstackgerrit Merged openstack/nova master: FUP to Iff8194c868580facb1cc81b5567d66d4093c5274 https://review.opendev.org/700202
17:01:41 efried thanks stephenfin
17:27:54 gmann mriedem: gibi : replied in review. this will fix the policy warning things, i missed to notify here early. - https://review.opendev.org/#/c/700797/
17:58:11 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix the suppress of policy deprecation warnings https://review.opendev.org/700797
18:22:32 dansmith efried: isn't this the thing that doesn't work? https://review.opendev.org/#/c/699554/2/nova/scheduler/request_filter.py@231
18:52:24 efried dansmith: it'll need https://review.opendev.org/#/c/699050/ which is still in progress
18:52:42 dansmith yeah, I thought so
18:53:15 efried ...but which I should get back to
18:53:42 efried oh, right, cause I really wanted your look at that dansmith, since basically whatever you tell me OVO-wise is what I'll go do.
18:54:11 dansmith okay
18:54:39 efried I can respin per mriedem's comments, but probably save a step to find out now how much your opinion doesn't overlap with that.
19:03:30 dansmith efried: didn't review enough to +2, but enough to add weight to the -1
19:03:47 efried thanks
19:15:23 openstackgerrit Lee Yarwood proposed openstack/nova master: virt: Provide block_device_info during rescue https://review.opendev.org/700811
19:15:23 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Add support for stable device rescue https://review.opendev.org/700812
19:15:24 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP compute: Report COMPUTE_BFV_RESCUE and check during rescue https://review.opendev.org/701429
19:15:24 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP api: Introduce microverion 2.82 allowing boot from volume rescue https://review.opendev.org/701430
19:15:25 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP libvirt: Support boot from volume instance rescue https://review.opendev.org/701431
19:22:29 efried dangit
19:22:52 efried dansmith: I think I'm going to need to write the rules for nullable on a post-it and stick it to my montior
19:23:16 efried Every time you explain it, I think I understand it. But then if more than a day or two passes til I have to think about it again, I forget.
19:23:29 dansmith nullable means exactly what it means for sql... "can be set to none".. that is ALL it means
19:23:31 efried Should request_level_params be nullable=True like requested_resources?
19:23:44 dansmith can/should it be set to none?
19:24:16 efried IMO no, IMO it should always be a RequestLevelParams even if that guy is "empty" (whatever that means for it). Which is why I had aggressively set the default.
19:24:31 dansmith aggressively setting the default is good
19:24:41 efried ...but not via default=
19:24:42 dansmith if it shouldn't be set to None, then it should not be nullable
19:25:20 dansmith default= doesn't actually default it, as you know,
19:25:48 dansmith but I just don't like the using of default= for this because it's a whole object and we're depending on deepcopy to do the right thing for it, and also it's probably more expensive to deepcopy those things than to create them fresh
19:33:51 efried dansmith: under what circumstances does obj_load_attr get invoked? Trying to ferret out how to do the defaulting there...
19:34:08 dansmith efried: it gets called if you reference a field that is not set
19:34:21 dansmith nothing to do with nullability
19:34:31 efried okay, neat, that makes what's already in there make more sense to me.
19:34:44 efried I was like, wait, won't these overwrite an existing field with a blank one??
19:39:15 efried It would be neat if there was a way to enforce that this field *only* be initialized that way. Because then I can control how the sub-fields are initialized. But there doesn't seem to be a precedent for e.g. making a field "private" and using a @prop reader for its public incarnation.
19:41:28 efried guess I'll do it via comment
19:45:01 dansmith that's not really what ovo is trying to do, it's trying to make sure you're structured
19:45:24 dansmith but yes, I know this kind of thing seems to mystify people a lot
20:10:06 efried It's one of those things that's very powerful, and with great power comes great responsibility, which is fine for someone who's "mature" enough to handle the responsibility, but tough for fly-by-nighters like me.
20:10:41 efried As long as savvy reviewers are around to hand-hold, we should be able to avert the apocalypse.
20:17:33 openstackgerrit Eric Fried proposed openstack/nova master: Use Placement 1.35 (root_required) https://review.opendev.org/699050
20:17:42 efried dansmith: done ^
20:17:42 efried Thanks as always for the guidance.
20:20:58 efried ...and remembered to post my comments just now. Weird where the vacation rust shows up.
21:12:34 openstackgerrit Merged openstack/nova master: Deprecate [glance]api_servers https://review.opendev.org/692227
22:50:31 openstackgerrit Eric Fried proposed openstack/nova master: Use Placement 1.35 (root_required) https://review.opendev.org/699050
23:50:09 openstackgerrit melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478
23:50:09 openstack bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929
#openstack-nova - 2020-01-08
01:30:08 openstackgerrit Merged openstack/nova master: nova-net: Remove firewall support (pt. 1) https://review.opendev.org/696514
01:55:21 openstackgerrit Merged openstack/nova master: nova-net: Remove firewall support (pt. 2) https://review.opendev.org/700510
03:59:59 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341
03:59:59 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029
04:00:00 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522
04:00:00 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460
05:59:56 openstackgerrit Merged openstack/nova master: Fix the suppress of policy deprecation warnings https://review.opendev.org/700797
06:27:04 openstackgerrit Merged openstack/nova master: Remove 'nova-xvpvncproxy' https://review.opendev.org/687909
09:02:20 gibi gmann: ack, thanks for the headsup abandoned my policy warning patch
09:08:16 gibi stephenfin: hi! could you check back to https://review.opendev.org/#/c/701364 it is green now
10:03:22 openstackgerrit Merged openstack/nova master: libvirt: avoid cpu check at s390x arch https://review.opendev.org/696228
10:06:03 openstackgerrit jichenjc proposed openstack/nova stable/train: libvirt: avoid cpu check at s390x arch https://review.opendev.org/701525
10:06:14 openstackgerrit jichenjc proposed openstack/nova stable/stein: libvirt: avoid cpu check at s390x arch https://review.opendev.org/701526
12:15:37 stephenfin gibi: Sorry, missed the ping. Will hit now
12:22:02 stephenfin gibi: done
12:34:03 stephenfin efried: question on https://review.opendev.org/#/c/686804/12/specs/ussuri/approved/add-emulated-virtual-tpm.rst@365
12:34:22 stephenfin wdym we can't do an API check? How's this different from the check sean-k-mooney added for the NUMA stuff?
12:35:01 stephenfin basically if the old and new images have different vTPM metadata, fail?
12:48:42 sean-k-mooney stephenfin: i was wondering that too
12:49:32 sean-k-mooney i had acess to the old image which is embded in the instance, the new image and the flavor
12:50:08 sean-k-mooney so that should be enough to do a check in the rebuild case
13:01:14 openstackgerrit Merged openstack/nova master: Fix excessive runtime of test test_migrate_within_cell https://review.opendev.org/701364
13:09:44 gibi stephenfin: thanks
13:57:23 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove firewall support (pt. 3) https://review.opendev.org/700511
13:57:24 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Copy shared utils from nova-net module https://review.opendev.org/696515
13:57:24 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove final references to nova-network https://review.opendev.org/696516
13:57:25 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'MetadataManager' https://review.opendev.org/696748
13:57:25 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Add TODOs to remove security group-related objects https://review.opendev.org/696517
13:57:26 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove dependency on nova-net from fake cache https://review.opendev.org/696749
13:57:26 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Kill it https://review.opendev.org/696518
13:57:27 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove unused exceptions https://review.opendev.org/697149
13:57:27 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove unnecessary 'neutronv2' prefixes https://review.opendev.org/696776
13:57:28 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Rename 'nova.network.neutronv2' -> 'nova.network' https://review.opendev.org/696745
13:57:28 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Make the security group API a module https://review.opendev.org/696746
13:57:29 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove remaining nova-network quotas https://review.opendev.org/697152
13:57:29 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'FloatingIP' object https://review.opendev.org/697153
13:57:30 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'Network' object https://review.opendev.org/697154
13:57:30 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'SecurityGroup' object https://review.opendev.org/697155
13:57:31 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove unused nova-network objects https://review.opendev.org/697156
14:32:54 shilpasd mriedem: thanks for review
14:44:29 stephenfin gibi: when you've a chance, could you hit https://review.opendev.org/#/c/700511/ again? Just addressed two of efried's nits
14:47:12 gibi stephenfin: looking...
14:48:42 gibi stephenfin: done
14:49:04 gibi stephenfin: I will try to look into the next set of patches in the series during today or tomorrow
14:57:46 bauzas gibi: can I ask you a question ?
14:58:01 bauzas gibi: I'm looking at your comments on https://review.opendev.org/#/c/670112/11/nova/tests/functional/test_nova_manage.py
14:58:27 bauzas gibi: I honestly prefer to leave castascall rather than having more than one method call per contextmanager

Earlier   Later