| opendevreview | Joan Gilabert proposed openstack/watcher master: Check result of retype action based on type and status https://review.opendev.org/c/openstack/watcher/+/951513 | 07:57 |
|---|---|---|
| opendevreview | chandan kumar proposed openstack/watcher-tempest-plugin master: Check for single instance migration to backup node https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/958174 | 09:10 |
| dviroel | sean-k-mooney1: hi - wrt these 2 patches from Alfredo: https://review.opendev.org/q/topic:%22blueprint-add-skip-actions%22+project:+openstack/watcher (in watcher) | 12:03 |
| dviroel | I can propose another patch to fix the issues from both | 12:03 |
| dviroel | and we can proceed with Alfredo's patches | 12:04 |
| opendevreview | Merged openstack/watcher master: Validate endpoint_type option at loading https://review.opendev.org/c/openstack/watcher/+/957749 | 12:06 |
| sean-k-mooney1 | dviroel: ack i was going to try and sync with you on that today | 12:08 |
| sean-k-mooney1 | that works for me | 12:08 |
| dviroel | ok, so I will start working on a patch | 12:09 |
| sean-k-mooney1 | so to finish that feature we will have the exisitng 2 patches and the followup your goign to create. the only other feature that will be impacted by thrudays freeze is your https://review.opendev.org/c/openstack/watcher/+/955827 patch | 12:10 |
| chandankumar | I will test https://review.opendev.org/c/openstack/watcher-dashboard/+/958209 patch and update it . | 12:14 |
| opendevreview | Merged openstack/watcher-tempest-plugin master: Check for single instance migration to backup node https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/958174 | 12:27 |
| sean-k-mooney1 | dviroel: has that been updated to relfect the change in scope we dicussed last week? | 12:39 |
| dviroel | sean-k-mooney1: yes! I also updated the spec to reflect the same | 12:39 |
| sean-k-mooney1 | oh i should fix my nic | 12:54 |
| *** sean-k-mooney1 is now known as sean-k-mooney | 12:54 | |
| sean-k-mooney | i have been redoing some of my home networking. i have more planned but ya my client would have discconected while i had everythign unplugged... | 12:55 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Fix doc section to enable cinder notifications https://review.opendev.org/c/openstack/watcher/+/957668 | 12:57 |
| jgilaber | sean-k-mooney, thanks for the comment on ^, that clarified the problem for me. I opened a docs bug https://bugs.launchpad.net/watcher/+bug/2121384 and linked the patch there | 12:59 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Fix doc section to enable cinder notifications https://review.opendev.org/c/openstack/watcher/+/957668 | 12:59 |
| jgilaber | IIUC we would need more changes in this topic to fix the current watcher defaults for the exchange and topics but at least with up to date docs it should be usable | 13:00 |
| sean-k-mooney | we shoudl change the default value for the decsion engine form watcher.notificoansn or whatever to openstack.notifications | 13:01 |
| sean-k-mooney | which will have it listen to the shared/common notificaiton topic | 13:01 |
| sean-k-mooney | then it shoudl "just work" | 13:01 |
| sean-k-mooney | for reciving notificaion form any service that is configured to send them | 13:02 |
| jgilaber | ok to to in the same patch as the docs change? | 13:02 |
| sean-k-mooney | am that up to you. it can be or we can split it | 13:02 |
| sean-k-mooney | smaller patchs are trivial to review so either works for me | 13:03 |
| jgilaber | ack, will do add it then, thanks! | 13:03 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Fix doc section to enable cinder notifications https://review.opendev.org/c/openstack/watcher/+/957668 | 14:15 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Enable storage model collector by default https://review.opendev.org/c/openstack/watcher/+/951323 | 14:19 |
| opendevreview | Joan Gilabert proposed openstack/watcher-tempest-plugin master: Add test for volume retype with zone migration https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/954625 | 14:19 |
| *** melwitt_ is now known as jgwentworth | 16:43 | |
| *** jgwentworth is now known as melwitt | 16:44 | |
| opendevreview | Merged openstack/watcher master: Fix NovaHelper microversion comparison https://review.opendev.org/c/openstack/watcher/+/957977 | 19:18 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Follow up changes for skip-action blueprint https://review.opendev.org/c/openstack/watcher/+/958469 | 19:38 |
| *** haleyb is now known as haleyb|out | 20:58 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!