| opendevreview | Takashi Kajinami proposed openstack/watcher master: Extend default query_timeout https://review.opendev.org/c/openstack/watcher/+/958504 | 00:47 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Rename confusing query timeout options https://review.opendev.org/c/openstack/watcher/+/958507 | 01:14 |
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Rename confusing query timeout options https://review.opendev.org/c/openstack/watcher/+/958507 | 01:44 |
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Rename confusing query timeout options https://review.opendev.org/c/openstack/watcher/+/958507 | 02:03 |
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Rename confusing query timeout options https://review.opendev.org/c/openstack/watcher/+/958507 | 04:52 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Enable storage model collector by default https://review.opendev.org/c/openstack/watcher/+/951323 | 07:38 |
| dviroel | hi sean-k-mooney, i proposed a follow up for skip actions here: https://review.opendev.org/c/openstack/watcher/+/958469 | 11:52 |
| dviroel | we may want to proceed with the proposal order: | 11:53 |
| dviroel | 1. skip action blueprint from alfredo | 11:53 |
| dviroel | 2. takashi's 958348: Fix missing watcher_workflow_engines.taskflow section | https://review.opendev.org/c/openstack/watcher/+/958348 | 11:53 |
| dviroel | 3. follow up change for skip actions? since it will conflict with takashi's fix | 11:53 |
| dviroel | so I can rebase the follow up | 11:54 |
| sean-k-mooney | that sounds reasonable to me | 11:55 |
| sean-k-mooney | so lets do that. | 11:56 |
| sean-k-mooney | ill review the folloup if im happy with that ill add my +2 to all the relevent patchs and then ill let you +2w them in the correct order | 11:56 |
| dviroel | ack | 11:58 |
| sean-k-mooney | ok im +2 on all 4 patches. the followup adressed my main points so those are good to go | 12:04 |
| sean-k-mooney | after those 4 have merged i guess your extra info patch will be next then we can move on to jgilaber's bug fixes | 12:04 |
| dviroel | +1 | 12:06 |
| sean-k-mooney | im going to try an rebase my make monasca optional patch today as well but im inclined to merge that after your patch is in and treate is as a bug fix | 12:08 |
| sean-k-mooney | but we can discuss that in the irc meeting if its not merged by FF | 12:08 |
| opendevreview | Merged openstack/watcher master: Skip actions automatically based on pre_condition results https://review.opendev.org/c/openstack/watcher/+/954746 | 12:33 |
| opendevreview | Merged openstack/watcher master: API changes for skipped actions: patch actions and status_message https://review.opendev.org/c/openstack/watcher/+/955753 | 12:54 |
| opendevreview | Merged openstack/watcher master: Fix missing watcher_workflow_engines.taskflow section https://review.opendev.org/c/openstack/watcher/+/958348 | 13:16 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Follow up changes for skip-action blueprint https://review.opendev.org/c/openstack/watcher/+/958469 | 13:28 |
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Rename confusing query timeout options https://review.opendev.org/c/openstack/watcher/+/958507 | 14:07 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Extend compute model attributes https://review.opendev.org/c/openstack/watcher/+/955827 | 14:39 |
| opendevreview | Merged openstack/watcher master: Follow up changes for skip-action blueprint https://review.opendev.org/c/openstack/watcher/+/958469 | 15:32 |
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Rename confusing query timeout options https://review.opendev.org/c/openstack/watcher/+/958507 | 16:00 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Set max value for nova_client.api_version configuration https://review.opendev.org/c/openstack/watcher/+/958092 | 19:06 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!