| 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 | 05:35 |
|---|---|---|
| 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 | 05:45 |
| 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 | 06:07 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Cache node utilization to fix vm_workload_consolidation at scale https://review.opendev.org/c/openstack/watcher/+/992676 | 06:19 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Fix flaky test_do_while_futures_modify_timeout unit test https://review.opendev.org/c/openstack/watcher/+/998423 | 09:19 |
| amoralej_ | dviroel ^ I hope that help with recent failures as in https://zuul.opendev.org/t/openstack/build/6380cd88987e429193749a9cb51d2cbf | 09:27 |
| dviroel | thanks amoralej_ i will check, i was about to open a LP to track that later... | 11:24 |
| dviroel | I think i was the one that proposed that part of the code | 11:24 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Add dataclass wrappers for CinderHelper API objects https://review.opendev.org/c/openstack/watcher/+/994842 | 11:33 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Replace python-cinderclient with openstacksdk https://review.opendev.org/c/openstack/watcher/+/994843 | 11:33 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove python-cinderclient dependency https://review.opendev.org/c/openstack/watcher/+/994844 | 11:33 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Use real objects in volume migration tests https://review.opendev.org/c/openstack/watcher/+/994862 | 11:33 |
| dviroel | hi all o/, the watcher meeting will start in 3 minutes | 12:57 |
| dviroel | #startmeeting watcher | 13:00 |
| opendevmeet | Meeting started Thu Jul 23 13:00:17 2026 UTC and is due to finish in 60 minutes. The chair is dviroel. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
| opendevmeet | The meeting name has been set to 'watcher' | 13:00 |
| dviroel | hi all, who is around? | 13:00 |
| jgilaber | o/ | 13:00 |
| rlandy | o/ | 13:01 |
| amoralej_ | o/ | 13:01 |
| morenod | o/ | 13:01 |
| dviroel | courtesy ping: sean-k-mooney chandank` | 13:02 |
| dviroel | this ^ should be chandankumar current's nick i guess :) | 13:02 |
| dviroel | ok, lets starts | 13:02 |
| dviroel | today's meeting agenda | 13:02 |
| dviroel | #link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L31 (Meeting agenda) | 13:03 |
| dviroel | ok, first topic | 13:03 |
| dviroel | #topic Announcements | 13:03 |
| dviroel | Release Schedule: Feature Freeze Aug 24-28 (5 weeks away) | 13:04 |
| dviroel | #link https://releases.openstack.org/hibiscus/schedule.html | 13:04 |
| dviroel | just a reminder that next milestone is 5 weeks aways | 13:04 |
| dviroel | s/aways/away | 13:04 |
| dviroel | and that's feature freeze | 13:04 |
| dviroel | which means that by the end of the week | 13:04 |
| dviroel | we should get proposed features merged | 13:05 |
| dviroel | so, if you have a feature proposal, it is better to start pushing patches soon | 13:05 |
| dviroel | so reviewers can have some time to review them | 13:05 |
| dviroel | and we don't need to rush in the last week :) | 13:06 |
| dviroel | alright | 13:06 |
| dviroel | nest in the announcements is from jgilaber | 13:06 |
| jgilaber | yes, related, I'll be on PTO and away from my laptop most of August | 13:07 |
| dviroel | right | 13:07 |
| jgilaber | so feel free to ping for anything that could use my reviews | 13:07 |
| jgilaber | I'll try to prioritize reviews before leaving | 13:07 |
| dviroel | ack thanks jgilaber, and we should prioritize your patches that are open for review too | 13:07 |
| dviroel | so yo have time to update them | 13:07 |
| jgilaber | yes, I'll talk briefly about that later | 13:08 |
| dviroel | thanks for sharing | 13:08 |
| amoralej_ | I will also prioritize reviews in existing patches and updating my inflight ones | 13:08 |
| dviroel | nice, thanks | 13:08 |
| dviroel | next topic | 13:08 |
| dviroel | #topic watcher-dashboard - current translation status | 13:09 |
| dviroel | morenod raised that topic yesterday | 13:09 |
| dviroel | and we were checking current status of translation in watcher-dashboard | 13:10 |
| dviroel | chandank` also did some reasearch in that topic | 13:10 |
| dviroel | and added some information in the meeting etherpad | 13:10 |
| dviroel | as far as i can see, there is some configurations set | 13:11 |
| dviroel | and also strings marked for translation, in watcher-dashboard | 13:11 |
| dviroel | what we don't see is the .pot files in watcher-dashboard repo | 13:11 |
| dviroel | which are generate by the translation server | 13:12 |
| dviroel | one example of zanata translations patch, proposed in manila-ui | 13:13 |
| dviroel | #link https://review.opendev.org/c/openstack/manila-ui/+/987789 | 13:13 |
| jgilaber | is https://review.opendev.org/c/openstack/project-config/+/992648 a blocker for this work? | 13:14 |
| chandank` | I am back | 13:14 |
| jgilaber | I see there are some comments there questioning the effort | 13:14 |
| chandank` | https://review.opendev.org/c/openstack/watcher-dashboard/+/998324 generates the pot file | 13:14 |
| chandank` | some seetings were missing from babel.cfg file | 13:14 |
| chandank` | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_651/openstack/651a921bc59b43eb939457845a5f03ac/tox/translations/index.html | 13:15 |
| dviroel | thanks chandank` - i will review that | 13:16 |
| chandank` | currently if we check the tarball https://tarballs.opendev.org/openstack/translation-source/watcher-dashboard | 13:17 |
| chandank` | here we can find translation files for release notes | 13:17 |
| amoralej_ | translation of tables names, column names, etc... are tried by default by django ? | 13:17 |
| amoralej_ | buttons, etc... | 13:17 |
| morenod | column names are not translated on other pages, like nova, cinder or neutron | 13:18 |
| morenod | only menu buttons | 13:18 |
| chandank` | so there are two things: One coming from the api and other coming from templates where tables, button are rendered | 13:18 |
| chandank` | in api, if https://github.com/openstack/watcher-dashboard/blob/master/watcher_dashboard/api/watcher.py#L155 anything ending with _( | 13:19 |
| chandank` | _("Unable to retrieve audit") will be extract by django babel config | 13:19 |
| chandank` | if it is coming from template https://github.com/openstack/watcher-dashboard/blob/db9d191c332c1beb4f2bcf3448a929801f06492e/watcher_dashboard/templates/infra_optim/logs/index.html#L3 | 13:19 |
| amoralej_ | also column names needs explicit usage of _ https://github.com/openstack/watcher-dashboard/blob/master/watcher_dashboard/content/actions/tables.py#L99 | 13:19 |
| amoralej_ | etc ... | 13:19 |
| chandank` | {% trans 'Logs' %} | 13:19 |
| chandank` | It gets extracted by horizon custom django module | 13:21 |
| chandank` | amoralej_: yes | 13:21 |
| dviroel | everything that we want to translate needs explicit usage | 13:21 |
| dviroel | so we choose what don't make sense to translate | 13:22 |
| amoralej_ | ok | 13:22 |
| amoralej_ | let's see how far we can go but it may be a goal for next release ... | 13:22 |
| chandank` | If we check the pot file https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_651/openstack/651a921bc59b43eb939457845a5f03ac/tox/translations/django.pot it will tell from which file and with line number text is extracted | 13:23 |
| dviroel | ok, this is something | 13:25 |
| opendevreview | Merged openstack/watcher master: Fix flaky test_do_while_futures_modify_timeout unit test https://review.opendev.org/c/openstack/watcher/+/998423 | 13:25 |
| amoralej_ | yep, that's a good progress | 13:26 |
| dviroel | chandank`: were you able to translate to another language and validate? | 13:26 |
| chandank` | dviroel: yes, I can translate in my native language like Hindi and rebuild the dashboard and see | 13:28 |
| chandank` | In India, user prefer english | 13:28 |
| dviroel | ack | 13:28 |
| dviroel | ok, so I will review your patch and try also in my end | 13:29 |
| dviroel | and we can follow up next week | 13:29 |
| chandank` | sure! | 13:29 |
| dviroel | thank for persuing this chandank` | 13:29 |
| dviroel | thanks morenod for raising that gap | 13:29 |
| dviroel | anyone wants to raise anything else wrt this topic? | 13:30 |
| dviroel | otherwise we can move forward | 13:30 |
| dviroel | #topic openstacksdk status | 13:30 |
| dviroel | jgilaber: do you want to highlight the progress of your changes? | 13:31 |
| jgilaber | I wanted to give a quick summary of the situation | 13:31 |
| dviroel | ++ | 13:31 |
| jgilaber | I have rebased the cinder helper series on amoralej_ patches removing cinder calls from zone migration | 13:31 |
| jgilaber | thanks amoralej_ and dviroel ++ for the reviews there | 13:31 |
| jgilaber | so I think this is making good progress | 13:31 |
| jgilaber | in parallel I've been working locally on an ironic emulator for functional tests | 13:32 |
| amoralej_ | i started reviewing the list | 13:32 |
| jgilaber | based on the existing work pushed by amoralej_ | 13:32 |
| jgilaber | given the PTO situation and the functional testing work being in progress I was thinking on leaving the ironic helper for next cycle | 13:32 |
| jgilaber | I can repropose the spec with that + adding watcher support to the openstacksdk | 13:33 |
| jgilaber | any thoughts on that? | 13:33 |
| dviroel | sure, you can do that after your pto then | 13:34 |
| dviroel | if you want, you can also propose the ironic ones as WIP | 13:34 |
| amoralej_ | so, iiuc the only strategy that uses that is saving_energy ? | 13:35 |
| jgilaber | yes, I'll try to push them as WIP next week | 13:35 |
| jgilaber | amoralej_, yes, I think that is correct | 13:35 |
| jgilaber | I'm using that strategy in the functional tests | 13:35 |
| amoralej_ | ack | 13:35 |
| jgilaber | and a final note on the sdk work | 13:36 |
| jgilaber | #link https://review.opendev.org/c/openstack/watcher/+/975498 | 13:36 |
| jgilaber | is ready for review too | 13:37 |
| jgilaber | it's a follow-up for the nova helper work | 13:37 |
| dviroel | ack, i will back to review this one too | 13:37 |
| jgilaber | that is all from me, thanks! | 13:38 |
| dviroel | alright, thanks jgilaber! | 13:38 |
| dviroel | #topic Reviews | 13:38 |
| dviroel | #link https://etherpad.opendev.org/p/watcher-2026.2-status | 13:38 |
| dviroel | so, since we expect more patches being proposed | 13:39 |
| dviroel | it is a good idea to maintain them listed in the status etherpad | 13:39 |
| dviroel | I have been proposing patches for audit-pipeline, but they do not yet implement the entire functionality, so not yet calling out for reviews | 13:40 |
| dviroel | only audit-template default parameters were ready for review | 13:40 |
| amoralej_ | i was about to ask about that | 13:40 |
| dviroel | thanks jgilaber and amoralej_ for reviews, I still need to push an update | 13:40 |
| amoralej_ | in your series of patches for pipeline, adding the rest api wouldn't be the last one? | 13:41 |
| chandank` | I also need feedback on playwright integration test patches, listed them under Improvements Dashboard | 13:41 |
| dviroel | there will be 2 api chnages | 13:41 |
| dviroel | 1.7 for default_parameters | 13:41 |
| dviroel | 1.8 for audit pipeline | 13:41 |
| dviroel | the 1.8 can then be moved up yes | 13:42 |
| dviroel | once i push the core audit-pipeline feature | 13:42 |
| amoralej_ | exactly | 13:43 |
| dviroel | it happens that is easier to test the funcionality with an api | 13:43 |
| amoralej_ | i undertand we need to 1. create objects 2. add core implementation in decision-engine 3. add the api to CRU | 13:43 |
| amoralej_ | ^ high level | 13:43 |
| dviroel | yes | 13:44 |
| amoralej_ | ack | 13:44 |
| dviroel | just set that review to WIP | 13:45 |
| dviroel | thanks amoralej_ | 13:45 |
| amoralej_ | yeah, sure, no problem, i just wanted to make sure | 13:45 |
| dviroel | chandank`: ack, yeah, you requested some reviews in the dashboard improvements | 13:45 |
| dviroel | but I didn't reviewed them yet | 13:45 |
| chandank` | No problem, take your time. thank you! | 13:46 |
| dviroel | thanks for updating the status etherpad too | 13:47 |
| dviroel | anyone want to highlight any other change? | 13:47 |
| dviroel | if your patches are not in the status etherpad, please add them | 13:47 |
| dviroel | if they are merged, you can also strikethrough | 13:48 |
| dviroel | #topic Bugs | 13:48 |
| dviroel | thanks for adding bugs to the list | 13:49 |
| * dviroel i guess my agent is not looking at watcher-dashboad project in lp | 13:49 | |
| dviroel | first one in the list | 13:49 |
| dviroel | #link https://bugs.launchpad.net/watcher-dashboard/+bug/2161426 | 13:50 |
| morenod | yep | 13:50 |
| dviroel | Filters on Audits section do not works | 13:50 |
| morenod | chandank` is already working on it | 13:50 |
| morenod | i have been working on testing the filters on all the watcher pages, and only the filter for audit is not working | 13:50 |
| dviroel | intereting | 13:51 |
| dviroel | i never used that filter option | 13:51 |
| dviroel | only to filter actionplan based on audits | 13:51 |
| chandank` | https://ibb.co/35xZDbGN - here is the audit template page | 13:51 |
| chandank` | there is only one filter Audit template, if we see the coulumn, there is no audit template | 13:52 |
| chandank` | It is a valid bug. | 13:52 |
| morenod | so there are two things here. first is fixing the current one, as there is no way to select an audit template | 13:52 |
| amoralej_ | audit templates are not stored for the audit object | 13:52 |
| dviroel | right, so that was my question | 13:53 |
| morenod | but the other thing, is to add more filters, that make this feature usable | 13:53 |
| chandank` | Regarding filter option, Do we want to filter Audit tupe, state, goal, strategy ? | 13:53 |
| amoralej_ | templates should not be available as filtering field i think | 13:53 |
| dviroel | if this filter exits in current api | 13:53 |
| chandank` | i have not checked the api, I was relying on cli putput | 13:53 |
| morenod | we can only use the same filter than API? we cannot filter on the table? | 13:54 |
| amoralej_ | nop https://docs.openstack.org/api-ref/resource-optimization/#list-audit | 13:54 |
| morenod | maybe it could be useful to filter by state, or audit type... I mean, any of the columns could be useful | 13:55 |
| dviroel | so about audit-template, we don't support, and I don't think we will support | 13:55 |
| amoralej_ | yeah, i don't know if the dashboard can apply filters to the table without additional api calls | 13:55 |
| amoralej_ | maybe yes | 13:55 |
| amoralej_ | i mean, for status and type | 13:56 |
| dviroel | ok, so for the bug created, the issue is that Audit Template is not a valid filter for audit, and needs to be removed.. | 13:57 |
| chandank` | dviroel: yes correct | 13:57 |
| amoralej_ | yes | 13:57 |
| morenod | we could track as new feature. just fix the bug changing the filter from audit template to any other available parameter, and develop filters in a more deeper way as a new feature | 13:57 |
| chandank` | I have not started looking into implementation | 13:57 |
| dviroel | could be a new LP to track as RFE | 13:58 |
| chandank` | but looking into filtering on table based on state and type | 13:58 |
| dviroel | and fix the current invalid filter in this LP | 13:58 |
| chandank` | *look | 13:58 |
| dviroel | morenod: so based on your validation, can you create a new LP to track this improvement? | 13:59 |
| morenod | sure | 13:59 |
| dviroel | proposing improving the filtering | 13:59 |
| dviroel | thanks | 13:59 |
| dviroel | chandank`: ok to set this to Low? | 14:00 |
| amoralej_ | https://github.com/openstack/watcher-dashboard/blob/master/watcher_dashboard/content/audits/tables.py#L45 | 14:00 |
| chandank` | yes | 14:00 |
| amoralej_ | i suspect ^ that's where the issue is | 14:00 |
| chandank` | yes | 14:01 |
| dviroel | oh, we are out of time | 14:01 |
| chandank` | fix would look like this https://github.com/openstack/watcher-dashboard/blob/db9d191c332c1beb4f2bcf3448a929801f06492e/watcher_dashboard/content/audit_templates/tables.py#L36 | 14:01 |
| amoralej_ | Low to Medium seems apropiate to me | 14:01 |
| chandank` | we can add filter name | 14:01 |
| chandank` | and that will work | 14:01 |
| amoralej_ | yep, looks like so | 14:01 |
| dviroel | ok, setting to Low for now.. | 14:02 |
| dviroel | that is for bugs | 14:02 |
| dviroel | thanks morenod and chandank` | 14:02 |
| dviroel | #topic Volunteers to chair next week | 14:02 |
| dviroel | someone wants to chair? | 14:02 |
| dviroel | otherwise i can chair again | 14:02 |
| jgilaber | I can do it dviroel | 14:02 |
| dviroel | thanks jgilaber | 14:02 |
| dviroel | let's wrap up for today | 14:03 |
| dviroel | thank you all for participating | 14:03 |
| dviroel | #endmeeting | 14:03 |
| opendevmeet | Meeting ended Thu Jul 23 14:03:16 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:03 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-07-23-13.00.html | 14:03 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-07-23-13.00.txt | 14:03 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-07-23-13.00.log.html | 14:03 |
| morenod | thanks dviroel++ for chairing | 14:03 |
| amoralej_ | thanks dviroel++ | 14:03 |
| opendevreview | Merged openstack/watcher master: Cache node utilization to fix vm_workload_consolidation at scale https://review.opendev.org/c/openstack/watcher/+/992676 | 21:45 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!