Friday, 2025-03-21

opendevreviewJoan Gilabert proposed openstack/watcher master: Set number of decimal digits in efficacy indicator  https://review.opendev.org/c/openstack/watcher/+/94519910:11
opendevreviewMatthieu Huin proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507913:23
opendevreviewAlfredo Moralejo proposed openstack/watcher master: DNM Test1  https://review.opendev.org/c/openstack/watcher/+/94521314:03
opendevreviewAlfredo Moralejo proposed openstack/watcher master: DNM Test2  https://review.opendev.org/c/openstack/watcher/+/94521414:03
opendevreviewFrancisco Seruca Salgado proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507915:45
opendevreviewFrancisco Seruca Salgado proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507915:48
opendevreviewFrancisco Seruca Salgado proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507915:51
opendevreviewAlfredo Moralejo proposed openstack/watcher-tempest-plugin master: Check expected actions included in plans when executing strategies  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94523316:03
amoralej^ first proposal for check actionplans, i'm keeping it simple and just checking that the expected action types are included in the plan, lemme know if you think that's enough or what else should we check16:09
*** jgilaber__ is now known as jgilaber16:12
jgilaberas a first approach seems ok, should we also check the number of actions?16:13
jgilaberright now for example it can't distinguish between an action plan with 2 or 3 migrations16:13
jgilaberI don't know right now for how many existing tests this could be relevant (if any)16:14
amoralejthat would be a good improvement my doubt is that the same test may create different number of actions in different environments16:15
amoraleji.e. the number of migrations in a consolidation strategies may depend on the number of compute nodes16:16
amoralejso, we'd need to add logic for that16:16
jgilaberthat's true, could be hard in some cases16:16
amoralejbut definetively, it would improve the test coverage16:16
jgilaberI think as it is is already a good improvement16:17
amoralejwe may implement "type": <number> , and have a special value for "any"16:17
amoraleji wanted to have it simple, but It may be worthy to complicate it, feedback is appreciated16:18
amoralejdon't hesitate to comment on the review, thans16:18
amoralejthanks16:18
opendevreviewFrancisco Seruca Salgado proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507916:20
opendevreviewAlfredo Moralejo proposed openstack/watcher-tempest-plugin master: Check expected actions included in plans when executing strategies  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94523316:27
opendevreviewFrancisco Seruca Salgado proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507916:31

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!