| opendevreview | Joan Gilabert proposed openstack/watcher master: Prepare to use openstacksdk instead of novaclient https://review.opendev.org/c/openstack/watcher/+/974710 | 10:57 |
|---|---|---|
| opendevreview | Joan Gilabert proposed openstack/watcher master: Complete migration from novaclient to openstacksdk https://review.opendev.org/c/openstack/watcher/+/974924 | 10:57 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove usage of novaclient from Watcher https://review.opendev.org/c/openstack/watcher/+/974925 | 10:57 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove nova_helper retries for openstacksdk params https://review.opendev.org/c/openstack/watcher/+/975498 | 10:57 |
| opendevreview | Merged openstack/watcher stable/2025.2: Revert "Update migration notification" https://review.opendev.org/c/openstack/watcher/+/974896 | 13:25 |
| opendevreview | Merged openstack/watcher master: Add wrapper classes for novaclient objects https://review.opendev.org/c/openstack/watcher/+/972912 | 14:18 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: [DNM] Debug transaction errors in threading mode with parallel engine https://review.opendev.org/c/openstack/watcher/+/975522 | 15:01 |
| sean-k-mooney | so open question how many people actully read the code review report form my ai code review job? i upgraded the intelagnce of the model over the weekend and am planning to imporve the quality fo the reivew more over the coming weeks | 18:51 |
| sean-k-mooney | i aks because it just found a bug in https://review.opendev.org/c/openstack/watcher/+/974925 in a file that was not modifed by the patch https://minio-api.teim.app/zuul-logs/d3b/main/d3beb0b6303a424f84ca5d13181c4cbc/code-review/review-report.html | 18:53 |
| sean-k-mooney | zuul drop the comment when it did the inline comments | 18:54 |
| sean-k-mooney | but its present in the comment file https://minio-api.teim.app/zuul-logs/d3b/main/d3beb0b6303a424f84ca5d13181c4cbc/code-review/zuul-comments.json so if that was a file that was modifeid it woudl have been reported | 18:54 |
| dviroel | sean-k-mooney: I do read the comments that are placed in gerrit only, when the change is in a ready state, but not at every patch set | 21:02 |
| dviroel | i don't usually check the job results, but indeed, zuul is dropping important comments there | 21:03 |
| sean-k-mooney | im probaly going to make the job fail if there is a critial issue found | 21:06 |
| sean-k-mooney | as i said i enabled a much smarter model at the weekend and im planning to spend a day or two workign on it in the next few weeks | 21:06 |
| sean-k-mooney | to see if i can impvoe the quality | 21:06 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!