Thursday, 2022-05-26

*** rlandy|bbl is now known as rlandy01:23
*** ysandeep|out is now known as ysandeep01:27
*** rlandy is now known as rlandy|out01:31
*** swalladge is now known as Guest28401:32
*** ysandeep is now known as ysandeep|afk02:50
*** diablo_rojo_phone is now known as Guest36005:08
*** mnasiadka_ is now known as mnasiadka05:08
*** tkajinam_ is now known as tkajinam05:11
*** bshephar is now known as bshephar|afk07:11
*** ykarel_ is now known as ykarel07:33
*** ysandeep|rover is now known as ysandeep|rover|lunch07:35
sshnaidmwho can add channel operators?07:43
*** ysandeep|rover|lunch is now known as ysandeep|rover09:23
*** rlandy|out is now known as rlandy10:27
*** carloss_ is now known as carloss10:31
*** dviroel_ is now known as dviroel11:17
dpawlikclarkb: Hey, about the performance.json file and where the results should be stored in same index or external. We were talking about 2 solutions: create a tool that is taking data from logstash index with performance.json content, then make many fields and push it to the separate index. Another idea was to push into the same index and the grafana11:28
dpawlikwill get took the values from opensearch (but it can make high utilization on Opensearch side). I was thinking about third solution: another tool that is periodically quering Opensearch for getting new performance.json informations and stored it somewhere else11:28
fungisshnaidm: propose an addition in https://opendev.org/openstack/project-config/src/branch/master/accessbot/channels.yaml with a channel-specific "ops" list of nicks11:30
fungireference documentation is at https://docs.opendev.org/opendev/system-config/latest/irc.html#accessbot11:32
dpawlikclarkb: alternatively, I can try to add a parameter to the logsender to send it to another index, but if it will overload the tool, I will revert the change/disable function11:36
*** ysandeep|rover is now known as ysandeep|rover|break11:44
opendevreviewdaniel.pawlik proposed openstack/ci-log-processing master: DNM Include performance fields in same index  https://review.opendev.org/c/openstack/ci-log-processing/+/84046611:59
*** ysandeep|rover|break is now known as ysandeep|rover12:36
*** mnaser_ is now known as mnaser12:41
opendevreviewdaniel.pawlik proposed openstack/ci-log-processing master: Parse performance.json file and send to separate index  https://review.opendev.org/c/openstack/ci-log-processing/+/84046613:00
opendevreviewdaniel.pawlik proposed openstack/ci-log-processing master: Ignore directories that does not exists or don't have access  https://review.opendev.org/c/openstack/ci-log-processing/+/84297713:03
*** bhagyashris is now known as bhagyashris|ruck14:22
*** ysandeep|rover is now known as ysandeep|dinner14:27
*** ysandeep|dinner is now known as ysandeep14:50
*** Guest360 is now known as diablo_rojo_phone15:00
clarkbdpawlik: the problem with storing it somewhere else is you need a somewhere else and elasticsearch/opensearch is already a json document store eg it is purpose built for this use case15:19
clarkbwhether or not you use the same tool to push into opensearch is a different question but I think opensearch is really well suited to this particular problem15:20
dpawlikclarkb: ok, proposed a change https://review.opendev.org/c/openstack/ci-log-processing/+/84046615:21
dpawlikneed to write some functional tests for that, but it might be difficult...15:21
dpawlikbut I will figure out something15:22
opendevreviewMerged openstack/project-config master: elastic-recheck: allow releasers to merge/delete  https://review.opendev.org/c/openstack/project-config/+/84045515:33
opendevreviewShnaidman Sagi (Sergey) proposed openstack/project-config master: Add ops to openstack-ansible-sig channel  https://review.opendev.org/c/openstack/project-config/+/84349215:33
*** gce108_ is now known as gce10815:36
fungijm1[m]: mnaser: your +1 on ^ is requested15:42
* mnaser signs in15:42
mnaserdone15:43
*** gce108 is now known as gce108_15:49
fungidpawlik: frenzyfriday|ruck: who should be in the elastic-recheck-release group on gerrit (to tag new releases, merge to/from the rdo branch, eventually delete the rdo branch, et cetera)?15:52
*** frenzyfriday|ruck is now known as frenzy_friday15:54
frenzy_fridayfungi, from ci team it would be me and dasm (dsmigiel@redhat.com)15:55
fungifrenzy_friday: awesome, i'll go ahead and add you. just be aware that gerrit will no longer prevent you from pushing merge commits or (signed) tags to that repository, so be extra careful you don't accidentally do that when you don't mean to15:57
frenzy_fridayfungi, ack15:58
frenzy_fridaythanks15:58
clarkbsimilar with deleting branches15:58
fungii think the acl will still prevent accidental branch deletion on push15:58
*** gce108_ is now known as gce10815:58
clarkbif/when you do that it is often a good idea to tag the branch first, make note of the sha HEAD, then delete the branch15:59
fungibut you'll be able to delete branches with the api or webui, so still yes be careful15:59
clarkbfungi: yup but you added perms for them to delete branches whic they didn't have before15:59
clarkbvia push you need force push perms to delete branches which aren't present15:59
fungiright15:59
fungifrenzy_friday: just double-checking, you're anbanerj right?16:00
fungii've added the two of you to elastic-recheck-release now, and removed ancient entries for mriedem and mtreinish as they're no longer actively involved in that project anyway16:02
frenzy_fridayfungi, yep16:03
*** ysandeep is now known as ysandeep|out16:06
*** gce108 is now known as gce108__16:09
opendevreviewDr. Jens Harbott proposed openstack/project-config master: Add a repository for the Large Scale SIG  https://review.opendev.org/c/openstack/project-config/+/84353417:29
opendevreviewDr. Jens Harbott proposed openstack/project-config master: Add a repository for the Large Scale SIG  https://review.opendev.org/c/openstack/project-config/+/84353417:31
*** dviroel is now known as dviroel|out21:00
opendevreviewDr. Jens Harbott proposed openstack/project-config master: Fix propose-updates job for requirements (3rd attempt)  https://review.opendev.org/c/openstack/project-config/+/84355021:06
*** rlandy is now known as rlandy|biab21:23
opendevreviewMerged openstack/project-config master: Fix propose-updates job for requirements (3rd attempt)  https://review.opendev.org/c/openstack/project-config/+/84355021:48

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