| *** mhen_ is now known as mhen | 02:15 | |
| opendevreview | Biser Milanov proposed openstack/tempest master: test_instances_with_cinder_volumes: Make sure attachments in guest are correct https://review.opendev.org/c/openstack/tempest/+/939633 | 08:59 |
|---|---|---|
| opendevreview | Joel Capitao proposed openstack/devstack master: lib/glance: Migrate Glance to WSGI module path https://review.opendev.org/c/openstack/devstack/+/932702 | 09:25 |
| opendevreview | Joel Capitao proposed openstack/devstack master: Support CentOS Stream 10 https://review.opendev.org/c/openstack/devstack/+/937251 | 09:25 |
| opendevreview | Joel Capitao proposed openstack/devstack master: Support CentOS Stream 10 https://review.opendev.org/c/openstack/devstack/+/937251 | 10:50 |
| opendevreview | Biser Milanov proposed openstack/tempest master: test_instances_with_cinder_volumes: Make sure attachments in guest are correct https://review.opendev.org/c/openstack/tempest/+/939633 | 11:11 |
| chandankumar | Hello gmann! I wanted to take about a new project: prometheus devstack plugin. | 13:42 |
| chandankumar | Since #openstack-qa office hour will be very late for me. I will drop the message here. | 13:42 |
| chandankumar | Recently I have started working on watcher project. In watcher, we are about to add prometheus as a datasource https://review.opendev.org/c/openstack/watcher/+/934423. Currently we were installing prometheus from sg-core devstack plugin repo. The same plugin is used in ceilometer repo. | 13:43 |
| chandankumar | Watcher project also needs to install node exporter and pushgateway. We proposed patch to sg-core repo https://github.com/openstack-k8s-operators/sg-core/pull/14 to add pushgateway support (needed for watcher-tempest-plugin to push custom metrics). But sg-core repo is meant for holding sg-core related stuff. | 13:43 |
| chandankumar | Since multiple projects are now using prometheus. So they suggested me to create prometheus devstack plugin and keep all prometheus projects installation and configuration there. | 13:43 |
| chandankumar | Before creating the repo, I wanted to check with #openstack-qa team. Who can take care of managing the prometheus devstack plugin repo? It would be #openstack-qa or telemetry team. | 13:43 |
| chandankumar | I will keep an eye on the reply here. Thank you! | 13:44 |
| chandankumar | gmann: I have added it under openstack-qa office hour agenda. rlandy or malinga will be there to discuss the same. thank you! | 14:02 |
| rlandy | thanks chandankumar | 14:05 |
| tosky | I can't answer as I'm not a core, but there are examples of devstack plugins whose management is shared: iirc devstack-plugin-ceph has both people from some of the affectes projects (manila, cinder for sure) AND also QA people | 14:07 |
| tosky | same for devstack-plugin-nfs, which has both cinder-core and devstack-core in its core reviewer group | 14:08 |
| tosky | so I suspect the answer will be along the line of "both telemetry and QA team" | 14:08 |
| rlandy | thanks for the input tosky | 14:18 |
| chandankumar | thank you tosky for the suggestion! | 14:35 |
| chandankumar | prometheus devstack plugin will be used by both telemetry and watcher project. | 14:37 |
| chandankumar | We will remove existing prometheus bits from sg_core repo. | 14:37 |
| sean-k-mooney | the orgial proposal in the watcher ptg was to add the prometious supprot to core devstack | 14:39 |
| sean-k-mooney | but if the prefernce is a plugin like the ceph one | 14:39 |
| sean-k-mooney | then that works equally well | 14:40 |
| frickler | with the declining number of devstack reviewers, having a plugin that can mostly be maintained by interested stakeholders sounds like a better solution to me, too | 14:45 |
| sean-k-mooney | if nothing else it allwos a wider set of core reviewers | 14:57 |
| sean-k-mooney | we can add the devstack, telemetry and watcher core groups to teh plugin core group | 14:58 |
| sean-k-mooney | without needign to do that on devstack itself | 14:58 |
| sean-k-mooney | chandankumar: are you goign to propose a patch to the project-config repo to create it? | 14:59 |
| chandankumar | sean-k-mooney: yes, I will do that tomorrow! | 15:00 |
| sean-k-mooney | from the governace repo poitn of view i think puting it under QA makes the most sence https://github.com/openstack/governance/blob/master/reference/projects.yaml#L2072 | 15:01 |
| sean-k-mooney | but its a devstack plugin so we are not really going to do release of it so that more so it can be in the openstack namespace then anything else | 15:01 |
| sean-k-mooney | the nfs one is there basically for that reason https://github.com/openstack/governance/blob/master/reference/projects.yaml#L2153 | 15:02 |
| sean-k-mooney | once the proejct-config change is proposed you shoudl propsoe a patch to the governace repo to add it too | 15:02 |
| chandankumar | sean-k-mooney: sure, thank you for the links! | 15:11 |
| *** gthiemon1e is now known as gthiemonge | 15:58 | |
| opendevreview | Balazs Gibizer proposed openstack/devstack master: Follow up for creating parent dir for config files https://review.opendev.org/c/openstack/devstack/+/939825 | 16:05 |
| opendevreview | Joel Capitao proposed openstack/devstack master: Support CentOS Stream 10 https://review.opendev.org/c/openstack/devstack/+/937251 | 16:51 |
| opendevreview | Ashish Gupta proposed openstack/tempest master: Add host filtering for test_aggregate_add_existent_host and test_aggregate_remove_host_as_user tests https://review.opendev.org/c/openstack/tempest/+/939766 | 17:04 |
| rlandy | sean-k-mooney: are we settled here with the solution "we can add the devstack, telemetry and watcher core groups to the plugin core group"? | 17:40 |
| rlandy | I'll still attend the meeting but the discussion/decision looks to be taken already? | 17:40 |
| sean-k-mooney | i think thats a resonable starting poitn. so how i woudl do this is add 1 or 2 people that will be maintianing the plugin directly to a new devstack-promethus-core group adn add the other 3 core groups as fallback | 17:42 |
| sean-k-mooney | there is no direct expectation for the core grops of the ohter project ot review the new plugin but they would have the opertunity too | 17:43 |
| rlandy | ok it's still on the meeting agenda - if no objections - you can go forward - thanks | 17:43 |
| sean-k-mooney | over time we can either add the active popel direcly and dropt the included groups or leave it | 17:43 |
| gmann | #startmeeting qa | 18:00 |
| opendevmeet | Meeting started Wed Jan 22 18:00:55 2025 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot. | 18:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 18:00 |
| opendevmeet | The meeting name has been set to 'qa' | 18:00 |
| gmann | #link https://wiki.openstack.org/wiki/Meetings/QATeamMeeting#Agenda_for_next_Office_hours | 18:01 |
| gmann | today agenda ^^ | 18:01 |
| gmann | who all here today | 18:01 |
| rlandy | o/ | 18:01 |
| gmann | rlandy: hi | 18:02 |
| gmann | let's go through the regular items first and then we can discuss about devstack new plugin | 18:03 |
| mtembo | Hello | 18:03 |
| gmann | mtembo: hi | 18:03 |
| gmann | #topic Announcement and Action Item (Optional) | 18:03 |
| gmann | nothing for this weel | 18:03 |
| gmann | week | 18:03 |
| gmann | #topic Priority Items progress | 18:03 |
| gmann | #link https://etherpad.opendev.org/p/qa-epoxy-priority | 18:04 |
| gmann | there is no significant progress on items so we can check it next time | 18:05 |
| gmann | #topic OpenStack Events Updates and Planning (Optional) | 18:05 |
| gmann | no event planned in near future but we do have next cycle election coming | 18:06 |
| gmann | #link https://governance.openstack.org/election/ | 18:06 |
| gmann | if there is no candidate for qa PTL role, we either go for DPL model or I will see if I can help as PTL role in next cycle | 18:07 |
| gmann | #topic Gate Status / Fix Checks | 18:07 |
| gmann | #link https://review.opendev.org/q/label:Review-Priority%253D%252B2+status:open+(project:openstack/tempest+OR+project:openstack/patrole+OR+project:openstack/devstack+OR+project:openstack/grenade) | 18:07 |
| gmann | nothing is tagged as urgent fix but afaik I did not see any blocker here | 18:07 |
| gmann | #link Changes with Review-Priority == +1 | 18:08 |
| gmann | #topic Changes with Review-Priority == +1 | 18:08 |
| gmann | #link https://review.opendev.org/q/label:Review-Priority%253D%252B1+status:open+(project:openstack/tempest+OR+project:openstack/patrole+OR+project:openstack/devstack+OR+project:openstack/grenade) | 18:08 |
| gmann | there is one review tagged as priority, I will check today | 18:08 |
| gmann | #topic Periodic jobs Status Checks | 18:08 |
| gmann | Periodic stable full: | 18:09 |
| gmann | #link https://zuul.openstack.org/builds?pipeline=periodic-stable&job_name=tempest-full-2023-2&job_name=tempest-full-2024-1&job_name=tempest-full-2024-2 | 18:09 |
| gmann | all green here | 18:09 |
| gmann | Periodic stable slow: | 18:09 |
| gmann | #link https://zuul.openstack.org/builds?job_name=tempest-slow-2024-2&job_name=tempest-slow-2024-1&job_name=tempest-slow-2023-2 | 18:09 |
| gmann | Periodic extra tests: | 18:09 |
| gmann | #link https://zuul.openstack.org/builds?job_name=tempest-full-2024-2-extra-tests&job_name=tempest-full-2024-1-extra-tests&job_name=tempest-full-2023-2-extra-tests | 18:09 |
| gmann | Periodic master: | 18:09 |
| gmann | #link https://zuul.openstack.org/builds?project=openstack%2Ftempest&project=openstack%2Fdevstack&pipeline=periodic | 18:10 |
| gmann | it seems tempest-full-test-account-no-admin-py3 job failing consistently | 18:11 |
| gmann | Details: {'type': 'PolicyNotAuthorized', 'message': 'rule:create_network is disallowed by policy', 'detail': ''} | 18:11 |
| gmann | some neutron rbac policy changed which we might need to check here | 18:12 |
| gmann | I will try if I can find some time to debug it this week | 18:12 |
| gmann | Distros check: | 18:12 |
| gmann | Centos 9: https://zuul.openstack.org/builds?job_name=tempest-full-centos-9-stream&job_name=devstack-platform-centos-9-stream&skip=0 | 18:12 |
| gmann | Debian: https://zuul.openstack.org/builds?job_name=devstack-platform-debian-bookworm&skip=0 | 18:12 |
| gmann | Rocky: https://zuul.openstack.org/builds?job_name=devstack-platform-rocky-blue-onyx | 18:12 |
| gmann | Jammy: https://zuul.opendev.org/t/openstack/builds?job_name=devstack-platform-ubuntu-jammy-ovn-source&job_name=devstack-platform-ubuntu-jammy-ovs&job_name=devstack-platform-ubuntu-jammy&skip=0 | 18:12 |
| gmann | Noble: https://zuul.opendev.org/t/openstack/builds?job_name=devstack-platform-ubuntu-noble&job_name=devstack-platform-ubuntu-noble-ovn-source&job_name=devstack-platform-ubuntu-noble-ovs&skip=0 | 18:12 |
| gmann | c9s job which is nv is failing some time which I can say not stable | 18:13 |
| gmann | other job is ok, they are failing sometime but not consistently | 18:14 |
| gmann | also, I will noble job from this list as we are now testing every job on Noble | 18:14 |
| gmann | #topic Open Discussion | 18:15 |
| gmann | rlandy/chandankumar: Ownership of prometheus devstack plugin (repo creation review will be sent after that). Context | 18:15 |
| gmann | rlandy: please go ahead | 18:15 |
| rlandy | thanks gmann | 18:15 |
| rlandy | I think we pretty much have a suggestion | 18:15 |
| rlandy | per sean-k-mooney and chandankumar's discussion | 18:16 |
| rlandy | ""we can add the devstack, telemetry and watcher core groups to the plugin core group" | 18:16 |
| rlandy | if there are no objections, we can go forward | 18:16 |
| rlandy | gmann: ^^ you thoughts? | 18:16 |
| gmann | I read the log | 18:16 |
| rlandy | for recording ... | 18:17 |
| rlandy | prometheus devstack plugin will be used by both telemetry and watcher projec | 18:17 |
| rlandy | so ownership spreads teams | 18:17 |
| gmann | first of all we have to understand what exactly meaning of adding a new project under QA. | 18:17 |
| gmann | Every project (I will say subproject) under QA are maintained by their own core member team. Like tempest, devstack, grenade all have their own core group even some/many of them overlap with same people but idea if to maintained them by their team | 18:18 |
| gmann | similar way if we add any devstack plugin, we need to have a separate plugin team to maintain it | 18:18 |
| gmann | seeing it is related to devstack setup it is all ok to add it under QA project governance but ownership and maintaining responsibility needs to be taken care by this plugin team | 18:19 |
| rlandy | (adding a bit more context): https://meetings.opendev.org/irclogs/%23openstack-qa/%23openstack-qa.2025-01-22.log.html ... I think the general understanding is that some combination of the two teams will look after it | 18:19 |
| rlandy | but the general structure will be under QA | 18:20 |
| gmann | for example, if plugin is active now and later we do not have any maintainer to maintain it then we might need to retire it as current QA team cannot maintaint it | 18:20 |
| rlandy | understood | 18:20 |
| gmann | This is same for all other devstack plugin | 18:20 |
| gmann | rlandy: yeah, we create the core group from both project or any other project also | 18:21 |
| rlandy | +1 | 18:21 |
| gmann | but my suggestion is to add a explicit list instead of adding watcher core or telemetry core group itself | 18:21 |
| gmann | and this plugin list can grow if someone other than these two project show interest | 18:22 |
| rlandy | I am not sure how the telemetry group is organized but we will pass on your recommendation | 18:22 |
| rlandy | we are working closely with them now | 18:22 |
| gmann | rlandy: yeah, maybe we can start with some of you there and then you can check if other member would like to join | 18:23 |
| gmann | perfect than | 18:23 |
| rlandy | ok - thank you gmann | 18:23 |
| gmann | as next step, please start the repo creation and propose the governance change to add this under QA with separate group of maintainer | 18:23 |
| gmann | please let me know if any help you need in that | 18:24 |
| gmann | also, frickler is not here at this time but he can also give input on this | 18:24 |
| rlandy | will do - this will start tomorrow | 18:24 |
| gmann | I just ready his feedback in log so we all good here | 18:25 |
| gmann | rlandy: perfect | 18:25 |
| gmann | anything else | 18:25 |
| rlandy | not from our side | 18:26 |
| gmann | thanks rlandy and chandankumar | 18:26 |
| gmann | I do not see any other topic in agenda, but I will keep it open for couple of min if anything else | 18:27 |
| gmann | let's close if nothing else, thanks everyone for joining | 18:29 |
| gmann | #endmeeting | 18:29 |
| opendevmeet | Meeting ended Wed Jan 22 18:29:24 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 18:29 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/qa/2025/qa.2025-01-22-18.00.html | 18:29 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/qa/2025/qa.2025-01-22-18.00.txt | 18:29 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/qa/2025/qa.2025-01-22-18.00.log.html | 18:29 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: Remove standalone nfs-ganesha https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/939907 | 19:52 |
| sp-bmilanov | sean-k-mooney: re: https://review.opendev.org/c/openstack/tempest/+/939633 when you say that LUN devices do not have a `/dev/block/<DEV>/serial`, do you mean that the file is empty or that is missing entirely? If it is missing, the way the check is done now will waste ~2minutes of time before it reverts to the previous behaviour (relying on the Nova/libvirt mapping) | 19:55 |
| sean-k-mooney | i dont know. so qemu does not allow you to pass a serial, im not sure if the device file will exisit with a seiral coming form the storage backend or if the file will be entirly missing | 20:01 |
| sean-k-mooney | sp-bmilanov: if you have a devstack env with lvm cinder you could modify this test https://github.com/openstack/tempest/blob/bd02c7355cb6bf327d109ef01f79c13cd2b263c6/tempest/api/compute/volumes/test_attach_volume.py#L469 to check | 20:02 |
| sp-bmilanov | ah, nice, will check it out later | 20:09 |
| sean-k-mooney | i suspect it might be there btu it will have a serial form the storage backend | 20:12 |
| sean-k-mooney | the specific code you are changing is not impacted by htis | 20:12 |
| opendevreview | Rajat Dhasmana proposed openstack/devstack stable/2024.2: Update glance image size limit https://review.opendev.org/c/openstack/devstack/+/939917 | 21:11 |
| whoami-rajat | gmann, kopecmartin hey, thanks for your reviews on the devstack change, here is the backport for 2024.2 https://review.opendev.org/c/openstack/devstack/+/939917 | 21:11 |
| gmann | whoami-rajat: ack, will check after gate result | 21:18 |
| whoami-rajat | thanks! | 21:20 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a restart to put some database compaction config changes into effect, and will return within a few minutes | 22:54 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!