| opendevreview | chandan kumar proposed openstack/watcher-tempest-plugin master: Added tempest API tests for continous audit https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/956004 | 06:38 |
|---|---|---|
| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Add option to SKIP Actions https://review.opendev.org/c/openstack/watcher-dashboard/+/958209 | 07:03 |
| opendevreview | Joan Gilabert proposed openstack/watcher-tempest-plugin master: Add test for volume migrate with zone migration https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/958644 | 07:15 |
| opendevreview | Joan Gilabert proposed openstack/watcher-tempest-plugin master: Test zone migration volume and compute migrations https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/962702 | 07:15 |
| opendevreview | chandan kumar proposed openstack/watcher-tempest-plugin master: Added tempest API tests for continous audit https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/956004 | 10:28 |
| opendevreview | chandan kumar proposed openstack/watcher-tempest-plugin master: Add Continuous audit dummy strategy scenario test https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/955472 | 12:13 |
| opendevreview | Douglas Viroel proposed openstack/watcher-tempest-plugin master: Add tests for extended compute datamodel https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/956116 | 12:41 |
| opendevreview | Douglas Viroel proposed openstack/watcher-tempest-plugin master: Remove server_state check from data_model test https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/963172 | 12:41 |
| dviroel | chandankumar: sean-k-mooney jgilaber_ : I have small fix for ci here, let mw know if you agree to merge: https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/963172 | 14:39 |
| sean-k-mooney | dviroel: Im ok with making the check less specific, i agree the statue could be cached and not reflective of the actual state so this looks ok to me | 15:16 |
| dviroel | ++ | 16:33 |
| sean-k-mooney | dviroel: jgilaber: would ye mind adding https://review.opendev.org/c/openstack/watcher/+/961450/3 ant the follow up patches to your review queue. i have the neutron client removal patch at the end of that serise and im going to put the glance client patch after that once i write it | 17:36 |
| dviroel | ack, I will review them. | 17:37 |
| sean-k-mooney | dviroel: i filed https://bugs.launchpad.net/watcher/+bug/2126959 by the way for glanceclient removal there are some other dead functions i belive that i may just lump together once that is completed | 18:38 |
| sean-k-mooney | for example im 99% sure delete_instance is never used either | 18:38 |
| sean-k-mooney | that was also used via the old cold migrate flow but once i have these simple cases done ill get my ai agent to do a general dead code review on a few fo the modules to see how much more it can fined | 18:40 |
| sean-k-mooney | we can decied if we want to use bug or something else to track this later | 18:40 |
| dviroel | ack, yes that was my understanding too of the use of glanceclient | 18:41 |
| dviroel | same thing for the delete_instance | 18:41 |
| sean-k-mooney | yep i conisderd foldign that into this or the neturon clinet change but ill leave it in its own commit as the final cleanup related to https://review.opendev.org/c/openstack/watcher/+/555665 | 18:42 |
| dviroel | +1 lgtm | 18:42 |
| opendevreview | sean mooney proposed openstack/watcher master: Remove unused glance client integration https://review.opendev.org/c/openstack/watcher/+/963217 | 19:18 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!