Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-02
20:27:27 mriedem a couple of issues with this in nova scheduler right now: 1. we don't have the requested networks in the RequestSpec 2. we don't yet have the ability to put a RequestGroup in the spec that is not filtering on some kind of resource (e.g. filtering on just traits or aggregates), but the good news is i think both of those are in progress
20:28:30 mriedem 1: https://review.opendev.org/#/c/570201/
20:29:45 mriedem gibi_cape: don't you have a patch for supporting an unnumbered request group that doesn't require resources?
20:30:44 mriedem https://review.opendev.org/#/c/647396
20:33:59 bauzas gibi_cape: I'm curious when looking at https://review.opendev.org/#/c/652608/ is that you don't need to pass allocations to finish_resize
20:34:10 bauzas or is it just an implementation concern ?
20:34:16 jroll ah sorry, you're right about the provider id, not aggregate id
20:38:50 gibi_cape mriedem: that patch https://review.opendev.org/#/c/647396 makes sure that the unnumbered group is created based on the flavor _early_ in the RequestSpec so traits or aggregates request can be added to the unnumbered group directly and don't have to be added to the flavor extra spec
20:39:51 gibi_cape bauzas: I think I need to pass the RequestSpect to finish_resize
20:39:58 gibi_cape bauzas: so the spec might miss that
20:40:15 gibi_cape bauzas: here, the PoC code https://review.opendev.org/#/c/655721/4/nova/compute/rpcapi.py
20:40:28 bauzas oh wait
20:40:49 bauzas you were talking of the conductor's finish_resize() method
20:40:51 bauzas gibi_cape: ^
20:40:54 bauzas not the compute one
20:41:09 bauzas of course, then yes you need the ReqSpec
20:41:25 bauzas for once the allocation is made, you also need to pass it to the compute service I guess
20:41:40 bauzas and then you hit https://review.opendev.org/#/c/589085/1/nova/compute/manager.py
20:42:51 gibi_cape bauzas: I don't have to pass the allocation to the virt driver as the virt driver does not do anything about the bandwidth resource
20:43:03 mriedem gibi_cape: ah ok
20:44:07 gibi_cape mriedem: supporting request group without resources is only discussed on the ML so far
20:44:51 cdent gibi_cape, mriedem: the emerging consensus on that is also "yeah, we'll probably need to do that"
20:50:24 mriedem ok so until then i can continue to hitch a ride with the flavor extra specs in the request spec yeehaw
20:54:44 gibi_cape mriedem: sure you can :)
20:57:25 bauzas gibi_cape: oh ok gtk
20:57:34 bauzas and yeah that makes sense, my bad
20:57:46 bauzas you care about neutron, not the virt driver
20:59:06 gibi_cape bauzas: yes
21:20:11 openstackgerrit melanie witt proposed openstack/nova master: Set [quota]count_usage_from_placement = True in nova-next https://review.opendev.org/653146
21:20:11 openstackgerrit melanie witt proposed openstack/nova master: Count instances from mappings and cores/ram from placement https://review.opendev.org/638073
21:20:12 openstackgerrit melanie witt proposed openstack/nova master: Use instance mappings to count server group members https://review.opendev.org/638324
21:20:12 openstackgerrit melanie witt proposed openstack/nova master: Add documentation for counting quota usage from placement https://review.opendev.org/653845
21:24:45 openstackgerrit melanie witt proposed openstack/nova master: Add documentation for counting quota usage from placement https://review.opendev.org/653845
21:26:15 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Fix a description for config_drive parameter https://review.opendev.org/653683
21:55:39 openstackgerrit Merged openstack/nova master: Query `in_tree` to placement https://review.opendev.org/649535
21:55:47 openstackgerrit Merged openstack/nova master: Remove 'instance_update_at_top', 'instance_destroy_at_top' https://review.opendev.org/651301
22:11:31 stephenfin jaypipes: Wanna look at this again https://review.opendev.org/#/c/629589/ and close out SR-IOV live migration? IIRC, the only change was a comment but there might be more to say I didn't fast approve it
22:13:06 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Hey let's support routed networks y'all! https://review.opendev.org/656885
22:14:07 mriedem jroll: ^
22:14:35 mriedem the scheduler piece needs to be sorted out since the request filters are processed before we create the request group stuff (where the requested aggregates are going to go)
22:28:38 melwitt is anyone else getting this error running unit tests locally? oslo_config.cfg.NoSuchOptError: no such option transport_url in group [DEFAULT]
22:29:05 openstackgerrit melanie witt proposed openstack/nova master: Fix SynchronousThreadPoolExecutorFixture mock of Future https://review.opendev.org/650171
22:29:05 openstackgerrit melanie witt proposed openstack/nova master: Revert "Fix target_cell usage for scatter_gather_cells" https://review.opendev.org/653894
22:29:06 openstackgerrit melanie witt proposed openstack/nova master: Use futurist.ThreadPoolExecutor in scatter_gather_cells https://review.opendev.org/650172
22:35:58 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Hey let's support routed networks y'all! https://review.opendev.org/656885
22:36:03 mriedem melwitt: nope
22:36:05 mriedem stale pycs?
22:38:23 melwitt mriedem: that was it. thanks for the reminder. I haven't had a stale pyc problem in forever
22:39:15 mriedem tox used to auto remove those
22:39:18 mriedem but it was removed
22:39:21 mriedem not tox, our tox.ini
22:40:15 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Hey let's support routed networks y'all! https://review.opendev.org/656885
22:40:57 melwitt yeah, I remember now that you mention it. probably because I switch to stable branches and run tests is how they get generated at all
22:41:25 melwitt because I thought along with that something was done to tell python not to generate the pycs
22:42:24 melwitt (for newer branches)
22:51:47 stephenfin melwitt: I backported that patch to the stable branches. I guess some of them weren't merged
22:53:13 melwitt stephenfin: maybe I'm remembering wrong but I thought in the past there was something about how the value is not always respected? in earlier versions of something
22:53:43 stephenfin melwitt: Earlier versions of tox, yeah. We hardcode the minimum tox version now though so you should have that
22:53:53 melwitt ah ok
22:54:51 igordc hi everyone, any discussion to be had about https://blueprints.launchpad.net/nova/+spec/sriov-bond? can't find anything in ptg etherpad
22:55:25 stephenfin melwitt: Yeah, it goes back to stable/pike. Weird https://review.opendev.org/#/q/I779a17afade78997ab084909a9e6a46b0f91f055
22:55:44 melwitt stephenfin: ok, I dunno what I did to make pycs happen
22:55:52 melwitt thanks for checking
22:56:00 stephenfin Note that we kept the 'rm -rf' on stable just in case
22:56:02 stephenfin np
23:00:37 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove conductor_api and _last_host_check from manager.py https://review.opendev.org/651059
23:01:04 openstackgerrit Tushar Patil proposed openstack/nova-specs master: Support filtering of allocation_candidates by forbidden aggregates https://review.opendev.org/609960
23:24:43 larsks It's -f for foreground.
23:25:00 larsks (I'm running a containerized znc, just checked)
23:25:08 larsks Ugh, whoops.
23:36:54 openstackgerrit Dan Smith proposed openstack/nova master: Enable image type query support in nova-next https://review.opendev.org/656903
23:41:29 openstackgerrit Merged openstack/os-vif master: Remove unused vif_plug_ovs.i18n module https://review.opendev.org/656815
23:43:12 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reset the stored logs at each notification test steps https://review.opendev.org/656844
23:44:38 mriedem for whenever efried is back, i struck the "fixing osc's live migration command" item from the saturday morning agenda since it's already been sorted out in the forum session and recap was posted to the ML
#openstack-nova - 2019-05-03
00:28:32 openstackgerrit melanie witt proposed openstack/nova master: Add 'path' query parameter to console access url https://review.opendev.org/649372
00:35:37 openstackgerrit Merged openstack/nova master: Require at least oslo.versionedobjects>=1.35.0 https://review.opendev.org/656633
01:48:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reset the stored logs at each notification test steps https://review.opendev.org/656844
06:35:44 openstackgerrit Ghanshyam Mann proposed openstack/nova-specs master: Spec for API policy updates https://review.opendev.org/547850
06:36:00 gmann melwitt: johnthetubaguy ^^ updated spec for policy updates.
11:38:25 jroll wow, efried summarizes with the quickness
13:53:19 efried jroll: ..while fresh in my head
13:56:12 jroll :)
13:57:20 gibi_cape efried: the summaries are really good. Thanks!
13:57:39 efried thanks gibi_cape
14:16:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716
14:22:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716
15:37:22 johnthetubaguy gmann: cool, thanks, will take a peak
15:37:38 johnthetubaguy jaypipes: I think this is what I had in my head: https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/compute/manager.py#L1838
15:38:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716
15:38:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656
15:46:03 mriedem johnthetubaguy: this is where the global lock is during the resource claim https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/compute/manager.py#L2196
15:53:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova-status upgrade check for minimum required cinder API version https://review.opendev.org/649759
15:53:21 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop pre-cinder 3.44 version compatibility https://review.opendev.org/621061
16:05:04 mriedem mdbooth: are you cool with dansmith's image disk_format capability patch now? https://review.opendev.org/#/c/653454/
16:09:35 mdbooth mriedem: Looking, btw
16:27:55 imacdonn mdbooth: still around ?
16:28:11 dansmith mriedem: totally fine with waiting to approve that stack until I write the docs, btw
16:28:42 mdbooth imacdonn: I am. In #nova
16:29:31 imacdonn mdbooth: #nova ?
16:29:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove unused param from _fill_provider_mapping https://review.opendev.org/655107

Earlier   Later