| opendevreview | Douglas Viroel proposed openstack/watcher stable/2024.1: Configure watcher tempest's microversion in devstack https://review.opendev.org/c/openstack/watcher/+/957821 | 01:28 |
|---|---|---|
| opendevreview | Joan Gilabert proposed openstack/watcher master: Enable storage model collector by default https://review.opendev.org/c/openstack/watcher/+/951323 | 05:20 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Handle missing dst_pool parameter in zone_migration https://review.opendev.org/c/openstack/watcher/+/950149 | 05:59 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Add test for zone_migration with instances and volumes https://review.opendev.org/c/openstack/watcher/+/952115 | 06:17 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Fix instance migrations in zone_migration strategy together with volumes https://review.opendev.org/c/openstack/watcher/+/952116 | 06:17 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Support zone migration audit without compute_nodes https://review.opendev.org/c/openstack/watcher/+/950665 | 06:17 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Add test for zone_migration with instances and volumes https://review.opendev.org/c/openstack/watcher/+/952115 | 09:06 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Fix instance migrations in zone_migration strategy together with volumes https://review.opendev.org/c/openstack/watcher/+/952116 | 09:06 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Support zone migration audit without compute_nodes https://review.opendev.org/c/openstack/watcher/+/950665 | 09:06 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Always report standard deviation in workload_stabilization https://review.opendev.org/c/openstack/watcher/+/947267 | 09:12 |
| 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 | 09:57 |
| dviroel | sean-k-mooney: hey, when you have a chance, pls check https://review.opendev.org/c/openstack/watcher/+/956465 - otherwise jobs will continue to fail because they are skipping all tests | 11:13 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Add patch call validation based on allowed_attrs https://review.opendev.org/c/openstack/watcher/+/955999 | 11:16 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Add parameters to force failures in nop action https://review.opendev.org/c/openstack/watcher/+/955813 | 11:16 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Add `status_message` column to Actions, Audits and ActionPlans tables https://review.opendev.org/c/openstack/watcher/+/954745 | 11:16 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Add `status_message` to objects and notifications https://review.opendev.org/c/openstack/watcher/+/956705 | 11:16 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Skip actions automatically based on pre_condition results https://review.opendev.org/c/openstack/watcher/+/954746 | 11:16 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: API changes for skipped actions: patch actions and status_message https://review.opendev.org/c/openstack/watcher/+/955753 | 11:16 |
| sean-k-mooney | dviroel: i have approved them but im not sure why that was required, we shoudl not have merged any changes to tempest that require this yet | 11:23 |
| sean-k-mooney | oh its becaue of https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/956306 | 11:25 |
| sean-k-mooney | ok so we really do need to get the stable jobs in place on master to validate those changes and make sure the senario tests are running | 11:25 |
| sean-k-mooney | the current watcher-tempest-functional-202* jobs are ment to detect and prevent use makign breaign changes like htis but they only run the api tests currently | 11:26 |
| dviroel | sean-k-mooney: yes, we need to discuss about running scenario tests in stable branches too for that repo. | 11:28 |
| dviroel | to avoid that | 11:28 |
| 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 | 11:29 |
| dviroel | sean-k-mooney: openstack/watcher gate only runs api tests... | 11:52 |
| sean-k-mooney | ya that a miscofiugrtion of our ci | 11:53 |
| sean-k-mooney | and not intentional | 11:53 |
| sean-k-mooney | striclty speakign if a job is voting ti ment ot be in both check and gate | 11:53 |
| dviroel | right | 11:54 |
| sean-k-mooney | we need to revisit how the jobs work currently as removign the watcher-tempest-actuator and watcher-tempest-strategies was actully somehtign i had planned to do last cycel | 11:55 |
| sean-k-mooney | we shoudl have one job per tested datasouce in check and gate | 11:55 |
| sean-k-mooney | and for the plugin repo we need at least one job per sable branch for each test | 11:56 |
| opendevreview | Merged openstack/watcher stable/2025.1: Configure watcher tempest's microversion in devstack https://review.opendev.org/c/openstack/watcher/+/956465 | 11:56 |
| opendevreview | Merged openstack/watcher stable/2024.2: Configure watcher tempest's microversion in devstack https://review.opendev.org/c/openstack/watcher/+/957805 | 11:56 |
| opendevreview | Merged openstack/watcher stable/2024.1: Configure watcher tempest's microversion in devstack https://review.opendev.org/c/openstack/watcher/+/957821 | 11:56 |
| sean-k-mooney | we do not need per data souce jobs for each stable branch on the tempest plugin | 11:56 |
| dviroel | agree | 11:57 |
| sean-k-mooney | provded each senairo/api test is actully run at least once | 11:57 |
| dviroel | i added a topic in the ptg etherpad, we can review all | 11:57 |
| sean-k-mooney | +1 | 11:57 |
| sean-k-mooney | for what it worht i think david has partly started on this but not with the explcit objectivs i mentioned above | 11:58 |
| sean-k-mooney | thhe watcher-tempest-functional job shoudl be renamed to watcher-tempest-gnocci | 11:58 |
| sean-k-mooney | we also shoudl proably fix the nameing of the prometheus jobs too but that less imporant | 11:59 |
| sean-k-mooney | we genreally dont use the term integration upstream and prefer the name tempest instead | 12:00 |
| sean-k-mooney | the name is less important hwoever then the fact it runs | 12:00 |
| dviroel | ack, i will check with david if he wants to drive that section too | 12:04 |
| opendevreview | Merged openstack/watcher stable/2025.1: use cinder migrate for swap volume https://review.opendev.org/c/openstack/watcher/+/957770 | 13:39 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Skip actions automatically based on pre_condition results https://review.opendev.org/c/openstack/watcher/+/954746 | 14:37 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: API changes for skipped actions: patch actions and status_message https://review.opendev.org/c/openstack/watcher/+/955753 | 14:37 |
| opendevreview | Merged openstack/watcher stable/2024.2: use cinder migrate for swap volume https://review.opendev.org/c/openstack/watcher/+/957773 | 15:03 |
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Validate endpoint_type option at loading https://review.opendev.org/c/openstack/watcher/+/957749 | 16:45 |
| opendevreview | Merged openstack/watcher stable/2024.1: use cinder migrate for swap volume https://review.opendev.org/c/openstack/watcher/+/957774 | 19:09 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Fix NovaHelper microversion comparison https://review.opendev.org/c/openstack/watcher/+/957977 | 20:26 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Fix NovaHelper microversion comparison https://review.opendev.org/c/openstack/watcher/+/957977 | 20:32 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!