| opendevreview | David proposed openstack/watcher-dashboard master: Add Playwright tests for filter functionality https://review.opendev.org/c/openstack/watcher-dashboard/+/998131 | 07:45 |
|---|---|---|
| opendevreview | David proposed openstack/watcher-dashboard master: Add Playwright tests for filter functionality https://review.opendev.org/c/openstack/watcher-dashboard/+/998131 | 07:59 |
| opendevreview | David proposed openstack/watcher-dashboard master: Add Playwright tests for filter functionality https://review.opendev.org/c/openstack/watcher-dashboard/+/998131 | 08:00 |
| opendevreview | David proposed openstack/watcher-dashboard master: Add Playwright tests for filter functionality https://review.opendev.org/c/openstack/watcher-dashboard/+/998131 | 08:15 |
| *** jgilaber__ is now known as jgilaber | 08:16 | |
| jgilaber | dviroel, are we good to merge https://review.opendev.org/c/openstack/watcher/+/994777 and the follow up? | 10:58 |
| dviroel | jgilaber: yes, i was waiting more ci results | 11:27 |
| opendevreview | Merged openstack/watcher master: Add volume_type, created_at and host fields to volumes in model https://review.opendev.org/c/openstack/watcher/+/994777 | 13:12 |
| opendevreview | Merged openstack/watcher master: Remove Cinder API calls from zone_migration strategy https://review.opendev.org/c/openstack/watcher/+/994778 | 13:26 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Add dataclass wrappers for CinderHelper API objects https://review.opendev.org/c/openstack/watcher/+/994842 | 13:27 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Replace python-cinderclient with openstacksdk https://review.opendev.org/c/openstack/watcher/+/994843 | 13:27 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove python-cinderclient dependency https://review.opendev.org/c/openstack/watcher/+/994844 | 13:27 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Use real objects in volume migration tests https://review.opendev.org/c/openstack/watcher/+/994862 | 13:27 |
| amoralej_ | jgilaber, dviroel thanks for merging those two! | 13:43 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Add dataclass wrappers for CinderHelper API objects https://review.opendev.org/c/openstack/watcher/+/994842 | 13:43 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Replace python-cinderclient with openstacksdk https://review.opendev.org/c/openstack/watcher/+/994843 | 13:43 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove python-cinderclient dependency https://review.opendev.org/c/openstack/watcher/+/994844 | 13:43 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Use real objects in volume migration tests https://review.opendev.org/c/openstack/watcher/+/994862 | 13:43 |
| dviroel | ack, so next are jgilaber patches, which are now rebased | 13:45 |
| dviroel | amoralej_: thanks for proposing and updating! | 13:45 |
| opendevreview | David proposed openstack/watcher-dashboard master: Switch to admin project before Playwright tests https://review.opendev.org/c/openstack/watcher-dashboard/+/997960 | 13:48 |
| dviroel | chandankumar: amoralej_ so this is the translation job that I mentioned: https://review.opendev.org/c/openstack/project-config/+/992648 | 14:48 |
| dviroel | https://review.opendev.org/c/openstack/watcher-dashboard/+/994377 - adds a locale dir to handle translations | 14:48 |
| amoralej_ | i was looking for some information about the topic, but didn't find it yet | 14:50 |
| amoralej_ | ah, here it is https://docs.openstack.org/horizon/latest/contributor/topics/translation.html | 14:51 |
| amoralej_ | i understand that applies also to plugins .. | 14:51 |
| amoralej_ | but yeah, merging that seems a step we need anyway | 14:52 |
| dviroel | i see that in the code some strings are already translatable | 14:53 |
| dviroel | i think that we can local run tools to check the current outputs | 14:53 |
| dviroel | we need to check if there are some other configurations missing too | 14:54 |
| dviroel | like it is mentioned here https://docs.openstack.org/horizon/latest/contributor/topics/translation.html#pseudo-translation-tool | 14:54 |
| amoralej_ | repo is registered in zanata https://translate.openstack.org/iteration/view/watcher-dashboard/master/languages/es?dswid=-7643 although no much translation ...https://translate.openstack.org/iteration/view/watcher-dashboard/master/languages/es?dswid=-7643 | 14:57 |
| amoralej_ | but yeah, good to test that | 14:57 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Replace python-cinderclient with openstacksdk https://review.opendev.org/c/openstack/watcher/+/994843 | 15:05 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove python-cinderclient dependency https://review.opendev.org/c/openstack/watcher/+/994844 | 15:05 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Use real objects in volume migration tests https://review.opendev.org/c/openstack/watcher/+/994862 | 15:05 |
| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Fix babel configs and add translations tox target https://review.opendev.org/c/openstack/watcher-dashboard/+/998324 | 15:14 |
| chandankumar | amoralej_: ^^ will help with the translation | 15:15 |
| chandankumar | will talk in detail during tomorrow irc meeting | 15:15 |
| opendevreview | Merged openstack/watcher-dashboard master: Add directory for locale https://review.opendev.org/c/openstack/watcher-dashboard/+/994377 | 15:45 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!