| *** dtantsur_ is now known as dtantsur | 00:08 | |
| *** efried1 is now known as efried | 00:40 | |
| opendevreview | Haidong Pang proposed openstack/nova master: Catch placement 5xx server exception https://review.opendev.org/c/openstack/nova/+/915832 | 08:10 |
|---|---|---|
| dougszu | Any objection to backporting this fix to supported stable branches? https://bugs.launchpad.net/nova/+bug/2015092 | 10:45 |
| dougszu | I can propose the backports but wanted to check first. I /think/ it should be a non-issue to backport. | 10:46 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add remaining missing request body schemas https://review.opendev.org/c/openstack/nova/+/915734 | 10:49 |
| 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 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add 'removed' decorator https://review.opendev.org/c/openstack/nova/+/915736 | 10:49 |
| 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 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add remaining missing query parameter schema https://review.opendev.org/c/openstack/nova/+/915738 | 10:49 |
| 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 |
| opendevreview | Stephen Finucane proposed openstack/nova master: conf: Add '[api] response_validation' option https://review.opendev.org/c/openstack/nova/+/915740 | 10:49 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body validation helper https://review.opendev.org/c/openstack/nova/+/915741 | 10:49 |
| 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 |
| 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 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Fix bugs in schemas https://review.opendev.org/c/openstack/nova/+/915843 | 10:49 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure API schemas are valid https://review.opendev.org/c/openstack/nova/+/915844 | 10:49 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Migrate to JSON Schema Draft 2020-12 https://review.opendev.org/c/openstack/nova/+/915845 | 10:49 |
| sean-k-mooney | dougszu: you can back port that it looks apporprate | 10:53 |
| dougszu | thanks sean-k-mooney | 11:03 |
| 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 |
| opendevreview | sean mooney proposed openstack/nova master: ignore sphinx-lint series in git blame https://review.opendev.org/c/openstack/nova/+/897218 | 11:25 |
| 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:25 |
| 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? | 12:38 |
| bauzas | fwiesel: the etherpad looks a good start for me, no worries | 13:23 |
| * bauzas doing a lot of internal things due to last week | 13:23 | |
| stephenfin | sean-k-mooney: done | 14:25 |
| sean-k-mooney | fwiesel: lazy generation of the metadata can be problemtic just an fyi | 15:35 |
| sean-k-mooney | fwiesel: cloud init will only retry the first request to the metadata endpoint | 15:36 |
| sean-k-mooney | once the first request is sucuscessful it never retries any other request assuming all metadta is aviaable | 15:36 |
| sean-k-mooney | which is why we generate it all and cache it in memcache | 15:37 |
| 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 |
| sean-k-mooney | and require the end user to reboot them or recreate them | 15:38 |
| 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:39 |
| sean-k-mooney | in general im not sure that lazy loading the metadata will actully be a good thing | 15:40 |
| sean-k-mooney | i would also argure that polling the metadata api is not an intended usecase | 15:42 |
| sean-k-mooney | what we could do is provide a way for the client to invladiate the cache | 15:42 |
| sean-k-mooney | i.e. by issuring a Delete to the root for the metadata api | 15:43 |
| sean-k-mooney | and then that way the could know tey are not out os sync | 15:43 |
| opendevreview | Dan Smith proposed openstack/nova master: Add method to determine "ancient" service version https://review.opendev.org/c/openstack/nova/+/915911 | 16:55 |
| opendevreview | Dan Smith proposed openstack/nova master: Add conductor API for service support check https://review.opendev.org/c/openstack/nova/+/915912 | 16:55 |
| opendevreview | Dan Smith proposed openstack/nova master: Make compute auto-disable if too old https://review.opendev.org/c/openstack/nova/+/915913 | 16:55 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a restart in order to apply a patch update | 18:36 | |
| opendevreview | Amit Uniyal proposed openstack/nova master: DNM: test evacuation https://review.opendev.org/c/openstack/nova/+/915923 | 20:09 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!