| opendevreview | David proposed openstack/watcher-dashboard master: Add HTTPS certificate error handling for Playwright tests https://review.opendev.org/c/openstack/watcher-dashboard/+/996069 | 09:04 |
|---|---|---|
| opendevreview | David proposed openstack/watcher-dashboard master: Use ID-based selector for login button https://review.opendev.org/c/openstack/watcher-dashboard/+/996215 | 09:06 |
| opendevreview | David proposed openstack/watcher-dashboard master: Add Playwright tests to validate goals and strategies https://review.opendev.org/c/openstack/watcher-dashboard/+/996067 | 09:06 |
| opendevreview | David proposed openstack/watcher-dashboard master: Use ID-based selector for login button https://review.opendev.org/c/openstack/watcher-dashboard/+/996215 | 09:09 |
| jgilaber | Hi all! IRC meeting starting in ~1 hour add your topics to the agenda https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L31 | 12:01 |
| opendevreview | Merged openstack/watcher-dashboard master: Add HTTPS certificate error handling for Playwright tests https://review.opendev.org/c/openstack/watcher-dashboard/+/996069 | 12:08 |
| opendevreview | Merged openstack/watcher-dashboard master: Use ID-based selector for login button https://review.opendev.org/c/openstack/watcher-dashboard/+/996215 | 12:08 |
| jgilaber | #startmeeting watcher | 13:00 |
| opendevmeet | Meeting started Thu Jul 16 13:00:31 2026 UTC and is due to finish in 60 minutes. The chair is jgilaber. 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 |
| jgilaber | Hi! Who is around today? | 13:00 |
| morenod | o/ | 13:00 |
| jgilaber | courtesy ping: dviroel amoralej sean-k-mooney chandankumar rlandy | 13:01 |
| amoralej | o/ | 13:01 |
| dviroel | o/ | 13:01 |
| rlandy | o/ | 13:01 |
| jgilaber | let wait another minute and we'll get started | 13:01 |
| jgilaber | you can add topics to the agenda in the meantime https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L31 | 13:02 |
| jgilaber | let's start with the first topic | 13:03 |
| jgilaber | #topic Priority Label is on | 13:03 |
| chandankumar | o/ | 13:03 |
| jgilaber | #link https://review.opendev.org/c/openstack/project-config/+/995886 | 13:03 |
| jgilaber | I guess dviroel added this? | 13:03 |
| dviroel | oh yeah | 13:04 |
| jgilaber | the patch merged and the label is avilable now | 13:04 |
| jgilaber | I don't think we've used it yet? | 13:04 |
| dviroel | yes, we can already set a priority to changes in watcher repois | 13:04 |
| dviroel | but yeah, I didn't use it yet | 13:04 |
| dviroel | so for higher priority changes we can start using it | 13:04 |
| dviroel | which I don't think we have one atm | 13:05 |
| dviroel | but yeah, just to mention that | 13:05 |
| amoralej | how can we set it? | 13:05 |
| dviroel | since we decided this a few meetings ago | 13:06 |
| sean-k-mooney | o/ | 13:06 |
| sean-k-mooney | amoralej: you shoudl just be able to set it when you click reply | 13:06 |
| dviroel | amoralej: you should see below your vote options | 13:06 |
| sean-k-mooney | in gerrit the same as when you leave +/-1 | 13:06 |
| amoralej | I see it now, thanks! | 13:06 |
| sean-k-mooney | it will break your musel memory however | 13:06 |
| sean-k-mooney | so be prepared to think your +w'ing something or +2ing ad set the priorty instead | 13:07 |
| sean-k-mooney | definetly not speakign for experince in nova... | 13:07 |
| jgilaber | thanks dviroel, any other comment before moving on? | 13:09 |
| dviroel | no, just to mention that is available now | 13:09 |
| dviroel | thanks | 13:09 |
| jgilaber | ack, next topic | 13:09 |
| jgilaber | #topic Watcher releases for stable branches done | 13:10 |
| jgilaber | #link https://review.opendev.org/c/openstack/releases/+/995942 | 13:10 |
| jgilaber | we merged the patch to publish a milestone 2 release of the stable branches | 13:10 |
| jgilaber | for all of them | 13:10 |
| jgilaber | there is a bit more detail on the patch, just wanted to highlight it here | 13:10 |
| dviroel | nice jgilaber++ | 13:10 |
| dviroel | ack, the security patch got in all, which is great | 13:11 |
| jgilaber | so I believe we can continue unless there are questions? | 13:11 |
| dviroel | s/got/got merged | 13:11 |
| dviroel | ++ | 13:11 |
| jgilaber | ok, moving to reviews | 13:12 |
| jgilaber | #topic Reviews | 13:12 |
| amoralej | thanks jgilaber | 13:12 |
| jgilaber | #link https://etherpad.opendev.org/p/watcher-2026.2-status | 13:12 |
| jgilaber | this is the status etherpad where we have a good list of patches up for review | 13:12 |
| jgilaber | does anyone want to higlight anything? | 13:13 |
| dviroel | i would call out the initial part of audit pipeline feature | 13:13 |
| jgilaber | #link https://etherpad.opendev.org/p/watcher-2026.2-status#L40 | 13:13 |
| dviroel | which is the default_parameters for audit-template | 13:13 |
| dviroel | #link (core) 993453: Add default_parameters support to Audit Templates | https://review.opendev.org/c/openstack/watcher/+/993453 | 13:14 |
| dviroel | #link (client) 993661: Add default_parameters support to Audit Template | https://review.opendev.org/c/openstack/python-watcherclient/+/993661 | 13:14 |
| dviroel | #link (tempest) 993672: Add API tests for audit template default_parameters | https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/993672 | 13:14 |
| dviroel | these 3 | 13:14 |
| amoralej | I just came back from pto but i will review it | 13:14 |
| dviroel | they can be merged earlier, before all audit-pipeline feature proposal | 13:14 |
| jgilaber | I saw those and have them in my review list as well | 13:15 |
| jgilaber | but I might get to them next week | 13:15 |
| dviroel | the rest of the patche can wait a bit, until we get all ready | 13:15 |
| dviroel | they are partially validated, and may required changes based on the core functionality work | 13:15 |
| dviroel | but if you have cycles, are good for a initial review too | 13:15 |
| dviroel | but lets focus first in the 3 mentioned above | 13:16 |
| dviroel | thanks amoralej jgilaber | 13:16 |
| jgilaber | sounds good dviroel, any questions on this patches? | 13:16 |
| jgilaber | ok, moving to morenod's dashboard patches | 13:17 |
| jgilaber | we have three, but the first two are already merged | 13:17 |
| jgilaber | #link https://review.opendev.org/c/openstack/watcher-dashboard/+/996067 | 13:17 |
| jgilaber | that is the remaining one | 13:17 |
| dviroel | i am reviewing it now yeah | 13:18 |
| jgilaber | do you want to highlight anything morenod? | 13:18 |
| morenod | thanks dviroel | 13:18 |
| morenod | just what we talked last week | 13:18 |
| morenod | I did updated the https one to always check https by default on zuul jobs | 13:18 |
| morenod | the var will be available for us to disable it on our local labs, but jobs will verify https | 13:19 |
| dviroel | ++ | 13:19 |
| jgilaber | +1, that seems like a good default, thanks for working on this | 13:19 |
| jgilaber | that was the last point in the agenda, do we have any last minute topic to discuss? | 13:20 |
| chandankumar | I have two watcher-dashboard reviews that needs attension | 13:21 |
| jgilaber | sure, go ahead chandankumar | 13:21 |
| chandankumar | 976594: Add Playwright integration test for skip action workflow | https://review.opendev.org/c/openstack/watcher-dashboard/+/976594 - skip action workflow test | 13:21 |
| chandankumar | And another one is 996062: Address review comments on Playwright E2E framework | https://review.opendev.org/c/openstack/watcher-dashboard/+/996062 - it is a follow up | 13:22 |
| chandankumar | from the playwright poc review feedback | 13:22 |
| dviroel | adding them to the end of the review etherpad | 13:24 |
| dviroel | not sure if were there already | 13:24 |
| jgilaber | was going to suggest that as well thanks dviroel | 13:24 |
| dviroel | together with morenod one | 13:24 |
| chandankumar | that's it from my side! thank you! | 13:24 |
| jgilaber | ok, looks like we've reached the end of the agenda, since dviroel++ volunteered to chair next week | 13:26 |
| jgilaber | any last minute comment before closing? | 13:26 |
| sean-k-mooney | sorry a littel distacted with a downstream conversation in parralel | 13:27 |
| dviroel | jgilaber++ thanks for chairing the last 2 meetings | 13:27 |
| sean-k-mooney | im reading back periodicly | 13:27 |
| morenod | thanks jgilaber++ | 13:27 |
| jgilaber | no problem sean-k-mooney do you want to raise anything before we close? | 13:27 |
| jgilaber | we've had a short agenda today | 13:28 |
| jgilaber | closing then, if something comes up we can always chat in the channel | 13:29 |
| jgilaber | thanks all for attending! | 13:29 |
| jgilaber | #endmeeting | 13:29 |
| opendevmeet | Meeting ended Thu Jul 16 13:29:19 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:29 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-07-16-13.00.html | 13:29 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-07-16-13.00.txt | 13:29 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-07-16-13.00.log.html | 13:29 |
| amoralej | Thanks jgilaber for chairing! | 13:31 |
| sean-k-mooney | amoralej: os while you were away i was reflecting on the funtional testing | 13:31 |
| amoralej | and? :) | 13:31 |
| sean-k-mooney | i see your point | 13:31 |
| amoralej | about any specific part? | 13:31 |
| sean-k-mooney | in nova we normally want to model no trivial nova toplogy and we want to do that programticly for nova | 13:32 |
| sean-k-mooney | but for the fixture that emulate cidner and neutron | 13:32 |
| sean-k-mooney | we startd form a baseline of static dicts in memory with some helper fucniton and classes aorund that | 13:32 |
| sean-k-mooney | amoralej: so in watcher its constnet to treat nova the way nova treats neutoron | 13:32 |
| sean-k-mooney | its a external service that we can have a reletively static toplogy for | 13:33 |
| sean-k-mooney | and as such codifying it move via data then impertivly is ok | 13:33 |
| amoralej | so, you mean about expressing topologies as dicts, right? | 13:35 |
| dviroel | right, but for watcher, the nova/cinder content is the non trivial topology that we need to handle | 13:37 |
| dviroel | i think that for more complex scenarios, the dicts are hard to read/maintain | 13:37 |
| amoralej | so, what'd you propose? using some kind of classes instead? | 13:40 |
| dviroel | yes, something that would generate a topology. Otherwise, how would we test in larger topologies? I don't think that upload a large json/xml would be a good option. | 13:47 |
| dviroel | i'm thinking on scaling tests in the future | 13:47 |
| dviroel | i also how we could easily modify existing test topologies | 13:48 |
| dviroel | if we want to add sche_hints info, i would need to update all existing topologies around, like we do for the xml (or try to do at least) | 13:48 |
| dviroel | we could still support simple dicts as input I think, but would be good to have a way to generate larger ones, that would make our life easier to maintain them | 13:50 |
| amoralej | So "something that would generate a topology", what do you mean? that's beyond the fact of using simple freeform dicts or Server, Instance, Volume, etc... objects | 13:57 |
| amoralej | right? | 13:57 |
| amoralej | you mean expressing the topology description instead of each element? | 13:58 |
| dviroel | yes, i could think on using objects only, that would allow us to build topologies using python loops for example | 14:06 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!