| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-04-11 | |||
| 14:41:21 | bauzas | we're running another topic for 20 mins | |
| 15:04:17 | bauzas | stephenfin: joining us ? | |
| #openstack-nova - 2024-04-12 | |||
| 08:14:30 | opendevreview | Kien Nguyen proposed openstack/python-novaclient master: (WIP) feat: support list and get server's metadata. https://review.opendev.org/c/openstack/python-novaclient/+/915546 | |
| 10:42:10 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/nova/+/915562 | |
| 10:47:10 | ratailor_ | stephenfin, could you please +w on https://review.opendev.org/c/openstack/python-openstackclient/+/904569 I think we are out of freeze now ? | |
| 10:49:36 | opendevreview | Stephen Finucane proposed openstack/placement master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/placement/+/915568 | |
| 11:55:45 | opendevreview | sean mooney proposed openstack/nova master: fix sphinx-lint errors in docs and add ci https://review.opendev.org/c/openstack/nova/+/897089 | |
| 11:55:46 | opendevreview | sean mooney proposed openstack/nova master: ignore sphinx-lint series in git blame https://review.opendev.org/c/openstack/nova/+/897218 | |
| 11:56:43 | sean-k-mooney | stephenfin: melwitt ^ ye were previously +2w on those but they failed to merge so i just rebased them. can you take a look at them again when you have time | |
| 11:57:03 | sean-k-mooney | that the ci automation for sphinx lint which was ment to be merged last cycle | |
| 13:02:37 | bauzas | sean-k-mooney: wanting to join us ? | |
| 13:19:49 | opendevreview | Stephen Finucane proposed openstack/nova-specs master: Add openapi spec https://review.opendev.org/c/openstack/nova-specs/+/909448 | |
| 14:19:22 | mikal | Thanks for the chat guys. I'm off to bed. | |
| 14:20:26 | bauzas | ++ | |
| 14:22:56 | bauzas | jayaanand_: we will discuss your topic in the next 30 mins (either now or after berendts), are you able to join us ? | |
| 14:23:13 | bauzas | for the moment, we are on a break until 2:30pmUTC | |
| 14:23:47 | bauzas | oh nevermind, I see you in meetpad :) | |
| 14:23:57 | bauzas | so, we'll discuss your topic after christian | |
| 14:24:02 | bauzas | probably around 3pm | |
| 14:24:18 | jayaanand_ | i am already in | |
| 14:24:20 | jayaanand_ | thank you! | |
| 14:24:39 | bauzas | jayaanand_: I don't see msaravan in the chan | |
| 14:24:58 | bauzas | but he/she's on the courtesy ping list | |
| 14:28:20 | jayaanand_ | i will ping him | |
| 14:57:59 | SaravananManickam[m] | I joined | |
| 14:58:14 | SaravananManickam[m] | Thank you bauzas jayaanand_ | |
| 16:05:59 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add missing Controller inheritance https://review.opendev.org/c/openstack/nova/+/914083 | |
| 16:06:00 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove FlavorManageController https://review.opendev.org/c/openstack/nova/+/914084 | |
| 16:06:00 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Keep track of action controllers https://review.opendev.org/c/openstack/nova/+/915732 | |
| 16:06:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add request body schemas for SG APIs https://review.opendev.org/c/openstack/nova/+/915733 | |
| 16:06:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add remaining missing request body schemas https://review.opendev.org/c/openstack/nova/+/915734 | |
| 16:06:02 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a request body schema https://review.opendev.org/c/openstack/nova/+/915735 | |
| 16:06:02 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add 'removed' decorator https://review.opendev.org/c/openstack/nova/+/915736 | |
| 16:06:03 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Don't do version check if nothing required https://review.opendev.org/c/openstack/nova/+/915737 | |
| 16:06:03 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add remaining missing query parameter schema https://review.opendev.org/c/openstack/nova/+/915738 | |
| 16:06:05 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a request query schema https://review.opendev.org/c/openstack/nova/+/915739 | |
| 16:06:05 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Add '[api] response_validation' option https://review.opendev.org/c/openstack/nova/+/915740 | |
| 16:06:07 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body validation helper https://review.opendev.org/c/openstack/nova/+/915741 | |
| 16:06:07 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server action APIs https://review.opendev.org/c/openstack/nova/+/915742 | |
| 16:06:09 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for remaining server action APIs https://review.opendev.org/c/openstack/nova/+/915743 | |
| #openstack-nova - 2024-04-14 | |||
| 21:40:35 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add remaining missing request body schemas https://review.opendev.org/c/openstack/nova/+/915734 | |
| 21:40:35 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a request body schema https://review.opendev.org/c/openstack/nova/+/915735 | |
| 21:40:36 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add 'removed' decorator https://review.opendev.org/c/openstack/nova/+/915736 | |
| 21:40:36 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Don't do version check if nothing required https://review.opendev.org/c/openstack/nova/+/915737 | |
| 21:40:37 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add remaining missing query parameter schema https://review.opendev.org/c/openstack/nova/+/915738 | |
| 21:40:37 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a request query schema https://review.opendev.org/c/openstack/nova/+/915739 | |
| 21:40:38 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Add '[api] response_validation' option https://review.opendev.org/c/openstack/nova/+/915740 | |
| 21:40:39 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body validation helper https://review.opendev.org/c/openstack/nova/+/915741 | |
| 21:40:39 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server action APIs https://review.opendev.org/c/openstack/nova/+/915742 | |
| 21:40:41 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for remaining server action APIs https://review.opendev.org/c/openstack/nova/+/915743 | |
| #openstack-nova - 2024-04-15 | |||
| 08:10:35 | opendevreview | Haidong Pang proposed openstack/nova master: Catch placement 5xx server exception https://review.opendev.org/c/openstack/nova/+/915832 | |
| 10:45:19 | dougszu | Any objection to backporting this fix to supported stable branches? https://bugs.launchpad.net/nova/+bug/2015092 | |
| 10:46:02 | dougszu | I can propose the backports but wanted to check first. I /think/ it should be a non-issue to backport. | |
| 10:49:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add remaining missing request body schemas https://review.opendev.org/c/openstack/nova/+/915734 | |
| 10:49:28 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a request body schema https://review.opendev.org/c/openstack/nova/+/915735 | |
| 10:49:29 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add 'removed' decorator https://review.opendev.org/c/openstack/nova/+/915736 | |
| 10:49:29 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Don't do version check if nothing required https://review.opendev.org/c/openstack/nova/+/915737 | |
| 10:49:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add remaining missing query parameter schema https://review.opendev.org/c/openstack/nova/+/915738 | |
| 10:49:30 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a request query schema https://review.opendev.org/c/openstack/nova/+/915739 | |
| 10:49:31 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Add '[api] response_validation' option https://review.opendev.org/c/openstack/nova/+/915740 | |
| 10:49:31 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body validation helper https://review.opendev.org/c/openstack/nova/+/915741 | |
| 10:49:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server action APIs https://review.opendev.org/c/openstack/nova/+/915742 | |
| 10:49:33 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for remaining server action APIs https://review.opendev.org/c/openstack/nova/+/915743 | |
| 10:49:33 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Fix bugs in schemas https://review.opendev.org/c/openstack/nova/+/915843 | |
| 10:49:35 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure API schemas are valid https://review.opendev.org/c/openstack/nova/+/915844 | |
| 10:49:35 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Migrate to JSON Schema Draft 2020-12 https://review.opendev.org/c/openstack/nova/+/915845 | |
| 10:53:11 | sean-k-mooney | dougszu: you can back port that it looks apporprate | |
| 11:03:58 | dougszu | thanks sean-k-mooney | |
| 11:25:39 | opendevreview | sean mooney proposed openstack/nova master: fix sphinx-lint errors in docs and add ci https://review.opendev.org/c/openstack/nova/+/897089 | |
| 11:25:40 | opendevreview | sean mooney proposed openstack/nova master: ignore sphinx-lint series in git blame https://review.opendev.org/c/openstack/nova/+/897218 | |
| 12:25:51 | sean-k-mooney | melwitt: stephenfin can ye reappove the sphinx-lint seriese, stephen i adressed your nits in the latest verion too but let me know if i missed anything | |
| 12:38:23 | fwiesel | bauzas: I've started with an etherpad for a Blueprint for the lazy metadata loading (https://etherpad.opendev.org/p/nova-lazy-metadata-loading) Should put that url in the blueprint registration? Or does it come later? | |
| 13:23:14 | bauzas | fwiesel: the etherpad looks a good start for me, no worries | |
| 14:25:05 | stephenfin | sean-k-mooney: done | |
| 15:35:50 | sean-k-mooney | fwiesel: lazy generation of the metadata can be problemtic just an fyi | |
| 15:36:12 | sean-k-mooney | fwiesel: cloud init will only retry the first request to the metadata endpoint | |
| 15:36:46 | sean-k-mooney | once the first request is sucuscessful it never retries any other request assuming all metadta is aviaable | |
| 15:37:10 | sean-k-mooney | which is why we generate it all and cache it in memcache | |
| 15:38:14 | sean-k-mooney | so if we allow lazy loading we would have to enforce some kind of stict timing aroudn how long that can take or it will cause vms to fail to get ssh keys ectra | |
| 15:38:25 | sean-k-mooney | and require the end user to reboot them or recreate them | |
| 15:39:13 | sean-k-mooney | fwiesel: the othger thing to kep in mind is i belivew we use the network info cache i dont think we query neturon for any of the metadata | |
| 15:40:35 | sean-k-mooney | in general im not sure that lazy loading the metadata will actully be a good thing | |
| 15:42:38 | sean-k-mooney | i would also argure that polling the metadata api is not an intended usecase | |
| 15:42:59 | sean-k-mooney | what we could do is provide a way for the client to invladiate the cache | |
| 15:43:15 | sean-k-mooney | i.e. by issuring a Delete to the root for the metadata api | |
| 15:43:24 | sean-k-mooney | and then that way the could know tey are not out os sync | |
| 16:55:03 | opendevreview | Dan Smith proposed openstack/nova master: Add method to determine "ancient" service version https://review.opendev.org/c/openstack/nova/+/915911 | |
| 16:55:04 | opendevreview | Dan Smith proposed openstack/nova master: Add conductor API for service support check https://review.opendev.org/c/openstack/nova/+/915912 | |
| 16:55:04 | opendevreview | Dan Smith proposed openstack/nova master: Make compute auto-disable if too old https://review.opendev.org/c/openstack/nova/+/915913 | |
| 20:09:44 | opendevreview | Amit Uniyal proposed openstack/nova master: DNM: test evacuation https://review.opendev.org/c/openstack/nova/+/915923 | |
| #openstack-nova - 2024-04-16 | |||
| 05:49:08 | opendevreview | Amit Uniyal proposed openstack/nova master: DNM: test evacuation https://review.opendev.org/c/openstack/nova/+/915923 | |
| 09:23:28 | opendevreview | Kien Nguyen proposed openstack/python-novaclient master: support get and list server's metadata https://review.opendev.org/c/openstack/python-novaclient/+/915977 | |
| 11:27:30 | opendevreview | Amit Uniyal proposed openstack/nova master: DNM: test evacuation https://review.opendev.org/c/openstack/nova/+/915923 | |
| 11:33:09 | sean-k-mooney | auniyal: ^ fyi when you create DNM patches like that its better to disable the jobs that you dont need to save ci resouces | |
| 13:50:28 | dansmith | gibi: I cc'd you but just in case it's under a mountain of other notification emails, I'm looking for pre-approval on this method of determining the trailing threshold: https://review.opendev.org/c/openstack/nova/+/915911/1 | |
| 13:50:33 | dansmith | from our version support discussion | |
| 13:51:00 | dansmith | just the algorithm for picking the version, and then I'll flesh out everything else | |
| 14:01:57 | auniyal | sean-k-mooney ack, but I am not sure how to disable jobs, | |
| 14:02:05 | auniyal | by doing -1 to workflow ? | |
| 14:03:00 | auniyal | its a depends-on patch, so I do not really need to run any job in nova-ci for this | |