| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-07-20 | |||
| 16:18:37 | bauzas | August 5th | |
| 16:18:41 | bauzas | that | |
| 16:18:42 | sean-k-mooney | ok cool | |
| 16:18:43 | gibi | first Thursday every month | |
| 16:18:48 | bauzas | first Thursday every month | |
| 16:18:49 | sean-k-mooney | ah ok | |
| 16:18:52 | bauzas | grraaah | |
| 16:18:54 | gibi | :D | |
| 16:18:56 | gibi | jinx | |
| 16:19:01 | bauzas | burned | |
| 16:19:42 | gibi | OK, if no other topic then lets close this and go back to work a bit | |
| 16:20:12 | gibi | thanks for joining | |
| 16:20:14 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2021/nova.2021-07-20-16.00.log.html | |
| 16:20:14 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2021/nova.2021-07-20-16.00.txt | |
| 16:20:14 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2021/nova.2021-07-20-16.00.html | |
| 16:20:14 | opendevmeet | Meeting ended Tue Jul 20 16:20:14 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:20:14 | gibi | #endmeeting | |
| 16:21:40 | bauzas | gibi: thanks | |
| 16:22:59 | elodilles | thanks o/ | |
| 16:45:32 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP neutron: Allow to spawn VMs with port without IP address https://review.opendev.org/c/openstack/nova/+/669411 | |
| 16:52:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | |
| 16:52:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | |
| 16:55:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 16:55:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | |
| 16:56:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 17:52:12 | stephenfin | gibi: I've the bulk of the precursor patches for the minimum BW series done. Main comments are that I think we should avoid using nova.utils where possible and that the functional tests and fixtures (including existing tests and fixtures) should go in their own file. All IMO, of course | |
| 17:52:23 | stephenfin | Will tackle the rest tomorrow | |
| 18:15:19 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Validate AZ values https://review.opendev.org/c/openstack/nova/+/801523 | |
| 18:22:53 | opendevreview | Stephen Finucane proposed openstack/nova master: neutron: Rework how we check for extensions https://review.opendev.org/c/openstack/nova/+/705792 | |
| 18:22:54 | opendevreview | Stephen Finucane proposed openstack/nova master: neutron: Remove references to out-of-tree 'qos-queue' extension https://review.opendev.org/c/openstack/nova/+/754448 | |
| 18:25:03 | stephenfin | sean-k-mooney: can you re-review https://review.opendev.org/c/openstack/nova/+/782410/1 and unblock the rest of the series? | |
| 19:12:02 | gibi | stephenfin: thanks, I will check the comments tomorrow | |
| 20:15:15 | sean-k-mooney | stephenfin: yes | |
| 20:16:17 | sean-k-mooney | you have adressed my nits in https://review.opendev.org/c/openstack/nova/+/784066/2 right | |
| 20:20:04 | sean-k-mooney | stephenfin: so ya the iisue i had are adressed in the followup so im ok with moving forward with the current patch and adressing those issue in the later patch | |
| 23:57:39 | opendevreview | Akihiro Motoki proposed openstack/nova master: DNM: Test tempest-full/slow jobs https://review.opendev.org/c/openstack/nova/+/801550 | |
| #openstack-nova - 2021-07-21 | |||
| 08:30:01 | bauzas | stephenfin: around ? I just discovered a new argument for a Opt definition that allows dynamic grouping https://docs.openstack.org/oslo.config/latest/reference/defining.html#dynamic-groups | |
| 08:30:11 | bauzas | but I lack of understand how this works | |
| 08:31:39 | bauzas | context is, we already dynamically register vgpu opts at startup but I want to add a deprecated group for mdevs | |
| 08:34:18 | bauzas | ah nevermind, saw your internal email about you being sick, hope you recover soon | |
| 09:03:52 | sean-k-mooney | bauzas: i think that just allows you to specify a static config option that holds the names of the dynmic groups | |
| 09:07:03 | sean-k-mooney | bauzas: https://opendev.org/openstack/oslo.config/src/branch/master/oslo_config/cfg.py#L1470 https://opendev.org/openstack/oslo.config/src/branch/master/oslo_config/_list_opts.py#L68 | |
| 09:14:16 | opendevreview | Sylvain Bauza proposed openstack/nova master: Rename vgpu options to mdev https://review.opendev.org/c/openstack/nova/+/801607 | |
| 09:15:46 | bauzas | sean-k-mooney: nevermind, I just found a way to have a deprecated group alias easily ^ | |
| 09:17:29 | opendevreview | Sylvain Bauza proposed openstack/nova master: Rename vgpu options to mdev https://review.opendev.org/c/openstack/nova/+/801607 | |
| 09:19:20 | sean-k-mooney | just using deprecated_name='enabled_vgpu_types', ya | |
| 09:19:30 | sean-k-mooney | at lest for the static part | |
| 09:19:43 | sean-k-mooney | ah | |
| 09:19:48 | sean-k-mooney | you are also adding deprecated_group='vgpu_%s' % mdev_type) | |
| 09:19:56 | sean-k-mooney | in the dynmaic option too | |
| 10:07:42 | gibi | could somebody (other than bauzas) hit https://review.opendev.org/c/openstack/placement/+/796595 to unblock the placement gate? | |
| 10:17:18 | lyarwood | gibi: done | |
| 10:17:25 | gibi | lyarwood: thanks! | |
| 10:17:39 | lyarwood | seems like an odd thing to test | |
| 10:17:53 | gibi | it is an intentional cannary | |
| 10:17:59 | lyarwood | kk | |
| 10:18:18 | gibi | there is no reason to have rc names in os-rc-lib that are not usable via the placement API | |
| 10:24:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump oslo.db to 10.0.0 https://review.opendev.org/c/openstack/nova/+/801610 | |
| 10:31:40 | sean-k-mooney | gibi: did the weekly job catch that | |
| 10:32:09 | sean-k-mooney | yep https://zuul.openstack.org/build/0e135bb912b240c8bc2aa96049727a1a | |
| 10:33:55 | gibi | sean-k-mooney: yes | |
| 10:34:05 | gibi | sean-k-mooney: but the patch was up before it caught :) | |
| 10:38:40 | sean-k-mooney | ya the periodic are just incase we dont have activ patches to placement in flight | |
| 10:39:08 | sean-k-mooney | but im gald that we have at leat one red result on it since it definetly show it will catch this now | |
| 10:40:05 | gibi | yepp, it is useful to have | |
| 11:52:56 | opendevreview | Merged openstack/placement master: Bump os-resource-classes requirements https://review.opendev.org/c/openstack/placement/+/796595 | |
| 14:56:21 | yonglihe | gibi，thanks your time reviewing the smartnic patches set, thats lots of work. | |
| 15:06:49 | gibi | yonglihe: hi! I know I should get back to the those as you responded | |
| 21:07:22 | NobodyCam | Good Afternoon Nova folks, crazy question is it safe to delete a stale record out of nova_api.build_requests, or is there a better safer way to clean records up in build_requests? | |
| 23:18:50 | melwitt | zzzeek: random question but I'm trying to maintain a single db transaction across several different method calls and I'm seeing my context.session drop/disappear when I call a method from a different module than I'm currently in. is that expected? | |
| 23:19:48 | melwitt | that is, inside the method in the different module, my context.session is gone even though context is passed in | |
| 23:55:46 | melwitt | zzzeek: nevermind, ignore me. I made a mistake in the thing I was testing /o\ | |
| #openstack-nova - 2021-07-22 | |||
| 01:34:31 | opendevreview | melanie witt proposed openstack/placement master: Add consumer_types migration, database and object changes https://review.opendev.org/c/openstack/placement/+/669170 | |
| 01:34:38 | opendevreview | melanie witt proposed openstack/placement master: Microversion 1.37: API support for consumer types https://review.opendev.org/c/openstack/placement/+/679441 | |
| 01:34:42 | opendevreview | melanie witt proposed openstack/placement master: Switch ConsumerType to use an AttributeCache https://review.opendev.org/c/openstack/placement/+/679486 | |
| 01:40:41 | melwitt | gibi: finally I have an update for consumer types https://review.opendev.org/c/openstack/placement/+/679441 sorry for the long delay | |
| 07:15:34 | opendevreview | Merged openstack/nova-specs master: Move placement specs to placement repo https://review.opendev.org/c/openstack/nova-specs/+/800775 | |
| 07:20:21 | gibi | melwitt: OK, I've added it to my queue | |
| 07:21:59 | gibi | melwitt: I have the RP re-parenting patch ready too https://review.opendev.org/c/openstack/placement/+/784020 | |
| 07:22:05 | gibi | if you have time | |
| 08:10:57 | opendevreview | Lee Yarwood proposed openstack/nova master: libvirt: Handle silent failures to extend volume within os-brick https://review.opendev.org/c/openstack/nova/+/801714 | |
| 09:12:57 | stephenfin | lyarwood: Could you review a few outstanding docs changes for me today? https://review.opendev.org/c/openstack/nova/+/782410/1 | |
| 09:13:12 | lyarwood | stephenfin: sure thing | |
| 09:13:29 | stephenfin | Thanks. I addressed your comments in that one specifically too, gibi ^ | |
| 09:13:48 | gibi | stephenfin: ack | |
| 09:19:38 | stephenfin | bauzas: In case you didn't see it, I addressed your issues on https://review.opendev.org/c/openstack/nova/+/798145 with a follow-up https://review.opendev.org/c/openstack/nova/+/801523 | |
| 09:19:50 | bauzas | stephenfin: ok, will look | |
| 09:19:54 | bauzas | and thanks | |
| 09:25:40 | opendevreview | Sylvain Bauza proposed openstack/nova master: DNM (yet) : Expose the mdev class https://review.opendev.org/c/openstack/nova/+/801743 | |
| 09:26:04 | lyarwood | https://github.com/openstack/nova/blob/3545356ae3a719442833cb8c3c911408d4bd3c15/nova/conductor/manager.py#L166-L184 - This is likely a stupid idea but why aren't we logging anything here as breadcrumbs for RPC timeouts etc? | |
| 09:26:26 | lyarwood | Have an example downstream of a bdm.save() timing out on the compute and I've got no way of seeing what n-cond was doing AFAICT | |
| 09:28:17 | lyarwood | I guess any breadcrumbs need to land in the actual object methods being exercised | |
| 09:29:34 | sean-k-mooney | ya i dont know really | |
| 09:29:45 | sean-k-mooney | i guess one question would be where | |
| 09:30:03 | sean-k-mooney | as in in what log would they end up | |
| 09:30:16 | sean-k-mooney | the api/conductor wont know where it actuly got too | |
| 09:30:59 | lyarwood | it should end up in n-cond right? | |
| 09:31:47 | lyarwood | almost need a TRACE log level below DEBUG for stuff like this | |
| 09:34:37 | gibi | stephenfin: what should I do to enable ':oslo.config:option:' directive in the api-guide doc build? I've added 'oslo_config.sphinxext' extension to the config but it fails with https://paste.opendev.org/show/807644/ | |
| 09:40:43 | lyarwood | nvm another n-cond log is full of `Too many heartbeats missed` errors from oslo.msg | |