| 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 | 04:51 |
|---|---|---|
| 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 | 04:58 |
| opendevreview | chandan kumar proposed openstack/watcher-tempest-plugin master: Add scenario test for skip action feature https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/962562 | 05:08 |
| opendevreview | chandan kumar proposed openstack/watcher-tempest-plugin master: Add scenario test for skip action feature https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/962562 | 05:28 |
| 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 | 08:47 |
| 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 | 08:47 |
| 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 | 09:09 |
| opendevreview | sean mooney proposed openstack/watcher master: adopt ruff check https://review.opendev.org/c/openstack/watcher/+/961452 | 10:53 |
| opendevreview | sean mooney proposed openstack/watcher master: Fix E402 violations and enable additional ruff checks https://review.opendev.org/c/openstack/watcher/+/961454 | 10:53 |
| opendevreview | sean mooney proposed openstack/watcher master: Fix ruff check violations https://review.opendev.org/c/openstack/watcher/+/961453 | 10:53 |
| opendevreview | sean mooney proposed openstack/watcher master: Remove dead neutron client code https://review.opendev.org/c/openstack/watcher/+/962990 | 10:53 |
| opendevreview | sean mooney proposed openstack/watcher master: Remove unused glance client integration https://review.opendev.org/c/openstack/watcher/+/963217 | 10:53 |
| opendevreview | sean mooney proposed openstack/watcher master: Remove dead NovaHelper methods https://review.opendev.org/c/openstack/watcher/+/963249 | 10:53 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Add some debug logs to storage model https://review.opendev.org/c/openstack/watcher/+/958781 | 11:31 |
| dviroel | hello sean-k-mooney - ptal when you have some time: https://review.opendev.org/c/openstack/watcher/+/960881 and https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/960381 - both are S changes, not complex things. Tks | 12:08 |
| sean-k-mooney | i saw the email updated for the first ill take a look in a bit. ah i remember talking abotu thte tempest change but ya i ill take a look at that too | 12:11 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: Fix pre-commit hook failures and add pre-commit config https://review.opendev.org/c/openstack/watcher-dashboard/+/963372 | 12:27 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: Adopt Ruff for security; remove Bandit and fix style https://review.opendev.org/c/openstack/watcher-dashboard/+/963378 | 12:57 |
| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Removed anti-pattern of getattr usage https://review.opendev.org/c/openstack/watcher-dashboard/+/963410 | 15:28 |
| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Removed anti-pattern usage of getattr usage https://review.opendev.org/c/openstack/watcher-dashboard/+/963410 | 15:29 |
| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Removed anti-pattern usage of getattr usage https://review.opendev.org/c/openstack/watcher-dashboard/+/963410 | 15:33 |
| chandankumar | sean-k-mooney: dviroel ^^ patch for drop getattr antipattern usage | 15:33 |
| chandankumar | *dropping | 15:33 |
| sean-k-mooney | its a start i guess but you are adding other antipatteres too | 15:35 |
| sean-k-mooney | https://review.opendev.org/c/openstack/watcher-dashboard/+/963410/3/watcher_dashboard/api/watcher.py#46 is not the right way to really set defaults | 15:36 |
| sean-k-mooney | i would prefer to design this properly then to patch over the solution picesmeal | 15:36 |
| chandankumar | sure | 15:37 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: Adopt Ruff for security; remove Bandit and fix style https://review.opendev.org/c/openstack/watcher-dashboard/+/963378 | 15:40 |
| sean-k-mooney | this is my current high level plan https://gist.github.com/SeanMooney/8746be1c29bbb78a63ef88f0104ea54f | 15:44 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Handle optional pool fields in Cinder notification https://review.opendev.org/c/openstack/watcher/+/961667 | 18:15 |
| opendevreview | sean mooney proposed openstack/watcher-specs master: Improve Watcher Dashboard code quality and test infrastructure https://review.opendev.org/c/openstack/watcher-specs/+/963438 | 18:57 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!