| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-11-26 | |||
| 16:52:47 | bauzas | thanks all | |
| 16:52:50 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-11-26-16.00.log.html | |
| 16:52:50 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-11-26-16.00.txt | |
| 16:52:50 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-11-26-16.00.html | |
| 16:52:50 | opendevmeet | Meeting ended Tue Nov 26 16:52:50 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:52:50 | bauzas | #endmeeting | |
| 16:52:54 | s3rj1k | thanks all | |
| 16:52:56 | tkajinam | thanks ! | |
| 16:53:28 | tkajinam | nothing urgent but I've added a few patches to drop deprecated/unmaintained deps to review priority list just fyi | |
| 16:53:42 | elodilles | thanks o/ | |
| 16:54:03 | gibi | o/ | |
| 17:16:27 | opendevreview | Balazs Gibizer proposed openstack/nova master: Show candidate combinatorial explosion by dev number https://review.opendev.org/c/openstack/nova/+/855885 | |
| 17:26:07 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add unit test coverage of get_machine_ips https://review.opendev.org/c/openstack/nova/+/936287 | |
| 19:01:04 | opendevreview | Douglas Viroel proposed openstack/nova-specs master: Add spec for show scheduler hints in server details https://review.opendev.org/c/openstack/nova-specs/+/936140 | |
| 19:14:37 | opendevreview | Andrei Yachmenev proposed openstack/nova-specs master: Dynamic disk qos updates support https://review.opendev.org/c/openstack/nova-specs/+/936302 | |
| 20:24:37 | opendevreview | Merged openstack/nova master: api: Add response body schemas for remaining server action APIs https://review.opendev.org/c/openstack/nova/+/915743 | |
| #openstack-nova - 2024-11-27 | |||
| 10:14:19 | opendevreview | Nisha Brahmankar proposed openstack/nova master: Retain port binding profile after live migration https://review.opendev.org/c/openstack/nova/+/936338 | |
| 10:26:43 | opendevreview | Nisha Brahmankar proposed openstack/nova master: Retain port binding profile after live migration https://review.opendev.org/c/openstack/nova/+/936338 | |
| 11:21:44 | opendevreview | Ivan Tkachuk proposed openstack/nova master: Reduce calls to qemu-img for disk_info https://review.opendev.org/c/openstack/nova/+/936246 | |
| 12:52:58 | dviroel | hi there nova folks. I would like to request reviews on this nova spec: https://review.opendev.org/c/openstack/nova-specs/+/936140 | |
| 12:53:46 | dviroel | just wondering if I should add it to https://etherpad.opendev.org/p/nova-2025.1-status ? under Blueprint needs approval ? | |
| 13:02:48 | bauzas | dviroel: sure, add it in the etherpad | |
| 13:04:10 | dviroel | ty | |
| 13:28:51 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 13:28:51 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Add contributor docs for response body validation https://review.opendev.org/c/openstack/nova/+/924597 | |
| 13:28:52 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Allow min/max_version arguments to expected_errors https://review.opendev.org/c/openstack/nova/+/936363 | |
| 13:28:52 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove use of microversion constants https://review.opendev.org/c/openstack/nova/+/936362 | |
| 13:28:53 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Adjust validation helpers for a single-method future https://review.opendev.org/c/openstack/nova/+/936365 | |
| 13:28:53 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Allow min/max_version arguments to response https://review.opendev.org/c/openstack/nova/+/936364 | |
| 13:28:54 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add new, simpler api_version decorator https://review.opendev.org/c/openstack/nova/+/936367 | |
| 13:28:54 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Stop using wsgi.Controller.api_version to switch between API versions https://review.opendev.org/c/openstack/nova/+/936366 | |
| 13:28:55 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only run format checks on strings https://review.opendev.org/c/openstack/nova/+/936368 | |
| 13:28:56 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify parameter types https://review.opendev.org/c/openstack/nova/+/936369 | |
| 13:38:56 | s3rj1k | JFYI: I don't see any mentions about adding specs to etherpad in https://docs.opendev.org/opendev/infra-manual/latest/developers.html#working-on-specifications-and-blueprints | |
| 13:41:49 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 13:41:50 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 13:41:50 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 13:41:51 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 13:41:51 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 13:41:52 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 13:41:52 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares https://review.opendev.org/c/openstack/nova/+/860287 | |
| 13:41:53 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 13:41:54 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 13:41:54 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 13:41:56 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 13:41:56 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 13:41:58 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 13:41:58 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 13:42:00 | opendevreview | ribaudr proposed openstack/nova master: Reports instance events to the DB regarding attaching and detaching a share https://review.opendev.org/c/openstack/nova/+/927088 | |
| 13:42:00 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 13:42:02 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 13:42:02 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 13:42:04 | opendevreview | ribaudr proposed openstack/nova master: Refactor test_server_shares: Mock in Base Class and trait verification https://review.opendev.org/c/openstack/nova/+/935861 | |
| 13:42:04 | opendevreview | ribaudr proposed openstack/nova master: Manila shares admin guide documentation https://review.opendev.org/c/openstack/nova/+/871642 | |
| 13:44:14 | sean-k-mooney | s3rj1k: thats because bauzas has an ar to udate the doc :) its not a requirement its just an extra way to highlight things that are avialable to review | |
| 13:45:09 | sean-k-mooney | s3rj1k: ildikov filed a but to crrect the docs gap recently | |
| 13:45:35 | sean-k-mooney | s3rj1k: the etherpad was added 2 releases ago, before that we used review priorities | |
| 13:46:05 | s3rj1k | and there is still an extra step of making stub blueprint | |
| 13:46:24 | sean-k-mooney | that is docuemented | |
| 13:46:31 | sean-k-mooney | all spec need a bluepirnt | |
| 13:46:36 | sean-k-mooney | that in the sepc template too | |
| 13:46:40 | sean-k-mooney | https://bugs.launchpad.net/nova/+bug/2089325 is the bug | |
| 13:47:11 | ildikov | +1, I was just about to throw in the bug report link :) | |
| 13:47:24 | s3rj1k | yea, I mean that there is blueprint for tracking state of spec and there is also etherpad | |
| 13:48:07 | sean-k-mooney | the etherpad is purly to help publisise and priories reviews | |
| 13:48:17 | sean-k-mooney | it is not require at all for our process | |
| 13:48:58 | ildikov | I interpreted the etherpad process as a lightweight project management tool that helps the core team to keep track of things that are ongoing | |
| 13:49:11 | sean-k-mooney | yep | |
| 13:49:54 | sean-k-mooney | somethime things can get "lost" in gerrit or lanchpad | |
| 13:50:02 | s3rj1k | imo, I see this as not so optimal, but ok, if that works | |
| 13:50:36 | sean-k-mooney | its manily working aroudn some limistaion in launchpads and gerrit ui | |
| 13:50:50 | sean-k-mooney | to be clear i almost exclisilvy ignor the etherpad | |
| 13:51:11 | sean-k-mooney | i do look at it form time to time but i look at gerrit much more frequently directly | |
| 13:51:34 | sean-k-mooney | i try to take a look at the ether pad once a week however but dont always fidn time too | |
| 13:52:03 | s3rj1k | could been just automated away based on data from gerrit and launchpad) | |
| 13:52:26 | sean-k-mooney | no the peopel that whated it wanted it to be human qurated | |
| 13:52:37 | sean-k-mooney | *curated | |
| 13:52:58 | sean-k-mooney | if we did an automated sync we woudl be back to the noise problem | |
| 13:55:52 | sean-k-mooney | s3rj1k: i get that process can be furstrating, especially if its not clear why or how it works | |
| 13:56:57 | s3rj1k | it just adds extra layer of bureaucracy and this will impact adoption for new community members | |
| 13:57:15 | sean-k-mooney | s3rj1k: how so. currently its not required at all | |
| 13:58:12 | s3rj1k | I have a gut feeling about this, I guess in time we will see | |
| 13:58:38 | sean-k-mooney | its actully there to allow people to engages async and to ask for reviews | |
| 13:58:45 | sean-k-mooney | so its intended to help continutors | |
| 13:59:06 | sean-k-mooney | i.e. so that they dont have to be on irc to ask for reviews | |
| 13:59:17 | sean-k-mooney | when they are not gettign feedback in gerrit | |
| 13:59:54 | sean-k-mooney | gerrit, irc and the mailing list are the 3 primary engagment points for contibutors with launch pad (bugs) for users | |
| 14:00:47 | s3rj1k | there is still RFE step that is semi-supported | |
| 14:00:50 | sean-k-mooney | s3rj1k: im not discounting your feedback im wonderign howe to improvie it (beyond updating the docs) | |
| 14:01:06 | sean-k-mooney | so nova does not use RFE bugs | |
| 14:01:11 | sean-k-mooney | that a neutron specific thing | |
| 14:01:20 | sean-k-mooney | we have experiemtned with adding it but its not part of our process | |
| 14:01:44 | s3rj1k | yea, but how to propose a draft for a feature to talk about? rfe seems perfect for this | |
| 14:01:46 | sean-k-mooney | most openstack teams dont sue RFE bugs, neutron use it because you cant realy have discussion in bluepirnts due to the ui | |
| 14:02:34 | s3rj1k | it's not like rfe is a call to action for core devs) just a place to put text and talk in comments | |
| 14:02:55 | sean-k-mooney | right but lanuchpad is not where we have those dicussions in general | |
| 14:03:13 | s3rj1k | mailing lists? | |
| 14:03:14 | sean-k-mooney | gerrit, irc or the mailing list is outside fo the ptg | |
| 14:03:41 | sean-k-mooney | so normally unless its a triival feature we would expect to have design discussion in a spec review | |
| 14:04:14 | sean-k-mooney | the actul workflow is ment to be, if you have a feature request bring it up on the mailing list or in the irc meeting | |