| opendevreview | melanie witt proposed openstack/cyborg master: Add API sample structural validation tests https://review.opendev.org/c/openstack/cyborg/+/994830 | 05:11 |
|---|---|---|
| opendevreview | chandan kumar proposed openstack/cyborg master: pci-sim: Add CentOS Stream 10 support https://review.opendev.org/c/openstack/cyborg/+/995935 | 05:21 |
| opendevreview | melanie witt proposed openstack/cyborg master: Add functional test job to Zuul CI https://review.opendev.org/c/openstack/cyborg/+/996389 | 05:29 |
| opendevreview | melanie witt proposed openstack/cyborg master: Add contributor docs for API sample validation tests https://review.opendev.org/c/openstack/cyborg/+/996390 | 05:34 |
| opendevreview | melanie witt proposed openstack/cyborg master: Fix api-ref inaccuracies and add missing program endpoint https://review.opendev.org/c/openstack/cyborg/+/995533 | 05:39 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Use cyborg-tempest-py3 for master job https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | 07:34 |
| chandankumar | jgilaber: Hi, https://review.opendev.org/c/openstack/project-config/+/996426 will enable stable branch gerrit review notification on cyborg channel | 08:17 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Use cyborg-tempest-py3 for master job https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | 08:53 |
| jgilaber | thanks chandankumar! I wanted to look for that | 08:54 |
| opendevreview | chandan kumar proposed openstack/cyborg master: pci-sim: Add CentOS Stream 10 support https://review.opendev.org/c/openstack/cyborg/+/995935 | 12:00 |
| sean-k-mooney | oh i was not aware we hadd a brnach filter applied | 12:38 |
| *** chandank` is now known as chandankumar | 13:14 | |
| opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove [glance] debug https://review.opendev.org/c/openstack/cyborg/+/995707 | 15:24 |
| opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove unused image utilities https://review.opendev.org/c/openstack/cyborg/+/995534 | 15:24 |
| melwitt | anyone else care to take a gander at a local hacking check for logging calls? https://review.opendev.org/c/openstack/cyborg/+/995561 sean-k-mooney has reviewed it so far (thanks) | 16:01 |
| sean-k-mooney | by the way i am undescided if i have broken teim-reivew on monday | 16:15 |
| opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove unused image utilities https://review.opendev.org/c/openstack/cyborg/+/995534 | 16:15 |
| sean-k-mooney | its now actully enfocing the conditions for reporting issues | 16:15 |
| sean-k-mooney | but it also now has many report with no issues | 16:15 |
| sean-k-mooney | so... ya i need to look into it more. | 16:16 |
| melwitt | I noticed it has quieted down a lot but the one inline comment I saw from it today was at 0.8 confidence so it seems to be working correctly? but it is a lot quieter | 16:26 |
| sean-k-mooney | ya so i had been tweakign the threshold but they were not being enfoced deteminsiticly | 17:06 |
| sean-k-mooney | so maybe they are too high | 17:06 |
| sean-k-mooney | i might just allow it to report anythingin the html report so i know that that is working and i can compare or drop tthe html thresholds | 17:07 |
| sean-k-mooney | i think i mentioned before that the min value was incorrectly set to 0.6 so now it using the full range form 0.0-1.0 | 17:08 |
| sean-k-mooney | https://minio-api.teim.app/zuul-logs/f86/main/f86334d7e10540e99d401ca0c8fc3be1/code-review/review-report.html | 17:08 |
| sean-k-mooney | has a html only suggestion at 0.8 | 17:08 |
| sean-k-mooney | the reshsodl are now deterministic https://github.com/SeanMooney/openstack-ai-style-guide/blob/master/tools/normalize_review_report.py#L32 | 17:11 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Use cyborg-tempest-py3 for master job https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | 17:26 |
| opendevreview | Merged openstack/cyborg master: Remove [glance] debug https://review.opendev.org/c/openstack/cyborg/+/995707 | 17:27 |
| melwitt | deterministic is good | 18:02 |
| opendevreview | Merged openstack/cyborg master: Add C300 hacking check for set literal in logging calls https://review.opendev.org/c/openstack/cyborg/+/995561 | 18:27 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!