| dviroel | sean-k-mooney: i think that is helping yes, even if some comments are not so useful. Indeed I didn't stop to think about min and max number of stages. But I would expect a min of 2. A single stage pipeline would behave like an regular audit (but with the addition of a metric cache only - which can also be added to all strategies in the end)\ | 10:57 |
|---|---|---|
| dviroel | tl;dr; is not distracting, and some comments are really helping, so we can keep it I think | 10:58 |
| sean-k-mooney | dviroel: ok i wont do much to it for now although i may work on a hacking.rst for the specs repo. if that file exist in the repo my ci will read it for repo sepcific instruction to follow | 11:16 |
| dviroel | ++ | 11:16 |
| sean-k-mooney | for code it actully pretty easy to tune but for somethign architecual like a spec where each can be very differnet in scope its a little harder to reason about. | 11:17 |
| sean-k-mooney | i started reviewing your last night but didnt push my comments because i realised i needed to do a deeper read over the propsoal | 11:17 |
| sean-k-mooney | im not entirly sure about the direction of the db change on if we should add a mapping table ro nulable colum on the actionplan | 11:18 |
| dviroel | sean-k-mooney: ack, we can discuss more tomorrow in the irc meeting too, or defer to next Thursday too if you folks need some more time to read it. | 11:39 |
| sean-k-mooney | it more that i have started to read it a few time but have not sat down and gong top to bottom in one pass in a while so i keep loosing context | 11:40 |
| opendevreview | Takashi Kajinami proposed openstack/grian-ui master: Declare Python 3.13 support https://review.opendev.org/c/openstack/grian-ui/+/974088 | 12:22 |
| opendevreview | Merged openstack/watcher-tempest-plugin master: Add stable/2026.1 job https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/981220 | 12:56 |
| opendevreview | sean mooney proposed openstack/grian-ui master: Add pypi-check tox env and fix README rst syntax https://review.opendev.org/c/openstack/grian-ui/+/983723 | 13:34 |
| opendevreview | Merged openstack/watcher-tempest-plugin master: Add project badge https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/973408 | 13:37 |
| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Add Playwright-based E2E testing framework https://review.opendev.org/c/openstack/watcher-dashboard/+/970353 | 14:55 |
| opendevreview | sean mooney proposed openstack/grian-ui master: Add pypi-check tox env and fix README rst syntax https://review.opendev.org/c/openstack/grian-ui/+/983723 | 16:28 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!