*** rlandy|bbl is now known as rlandy | 01:23 | |
*** ysandeep|out is now known as ysandeep | 01:27 | |
*** rlandy is now known as rlandy|out | 01:31 | |
*** swalladge is now known as Guest284 | 01:32 | |
*** ysandeep is now known as ysandeep|afk | 02:50 | |
*** diablo_rojo_phone is now known as Guest360 | 05:08 | |
*** mnasiadka_ is now known as mnasiadka | 05:08 | |
*** tkajinam_ is now known as tkajinam | 05:11 | |
*** bshephar is now known as bshephar|afk | 07:11 | |
*** ykarel_ is now known as ykarel | 07:33 | |
*** ysandeep|rover is now known as ysandeep|rover|lunch | 07:35 | |
sshnaidm | who can add channel operators? | 07:43 |
---|---|---|
*** ysandeep|rover|lunch is now known as ysandeep|rover | 09:23 | |
*** rlandy|out is now known as rlandy | 10:27 | |
*** carloss_ is now known as carloss | 10:31 | |
*** dviroel_ is now known as dviroel | 11:17 | |
dpawlik | clarkb: 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 grafana | 11:28 |
dpawlik | will 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 else | 11:28 |
fungi | sshnaidm: propose an addition in https://opendev.org/openstack/project-config/src/branch/master/accessbot/channels.yaml with a channel-specific "ops" list of nicks | 11:30 |
fungi | reference documentation is at https://docs.opendev.org/opendev/system-config/latest/irc.html#accessbot | 11:32 |
dpawlik | clarkb: 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 function | 11:36 |
*** ysandeep|rover is now known as ysandeep|rover|break | 11:44 | |
opendevreview | daniel.pawlik proposed openstack/ci-log-processing master: DNM Include performance fields in same index https://review.opendev.org/c/openstack/ci-log-processing/+/840466 | 11:59 |
*** ysandeep|rover|break is now known as ysandeep|rover | 12:36 | |
*** mnaser_ is now known as mnaser | 12:41 | |
opendevreview | daniel.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/+/840466 | 13:00 |
opendevreview | daniel.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/+/842977 | 13:03 |
*** bhagyashris is now known as bhagyashris|ruck | 14:22 | |
*** ysandeep|rover is now known as ysandeep|dinner | 14:27 | |
*** ysandeep|dinner is now known as ysandeep | 14:50 | |
*** Guest360 is now known as diablo_rojo_phone | 15:00 | |
clarkb | dpawlik: 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 case | 15:19 |
clarkb | whether 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 problem | 15:20 |
dpawlik | clarkb: ok, proposed a change https://review.opendev.org/c/openstack/ci-log-processing/+/840466 | 15:21 |
dpawlik | need to write some functional tests for that, but it might be difficult... | 15:21 |
dpawlik | but I will figure out something | 15:22 |
opendevreview | Merged openstack/project-config master: elastic-recheck: allow releasers to merge/delete https://review.opendev.org/c/openstack/project-config/+/840455 | 15:33 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/project-config master: Add ops to openstack-ansible-sig channel https://review.opendev.org/c/openstack/project-config/+/843492 | 15:33 |
*** gce108_ is now known as gce108 | 15:36 | |
fungi | jm1[m]: mnaser: your +1 on ^ is requested | 15:42 |
* mnaser signs in | 15:42 | |
mnaser | done | 15:43 |
*** gce108 is now known as gce108_ | 15:49 | |
fungi | dpawlik: 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_friday | 15:54 | |
frenzy_friday | fungi, from ci team it would be me and dasm (dsmigiel@redhat.com) | 15:55 |
fungi | frenzy_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 to | 15:57 |
frenzy_friday | fungi, ack | 15:58 |
frenzy_friday | thanks | 15:58 |
clarkb | similar with deleting branches | 15:58 |
fungi | i think the acl will still prevent accidental branch deletion on push | 15:58 |
*** gce108_ is now known as gce108 | 15:58 | |
clarkb | if/when you do that it is often a good idea to tag the branch first, make note of the sha HEAD, then delete the branch | 15:59 |
fungi | but you'll be able to delete branches with the api or webui, so still yes be careful | 15:59 |
clarkb | fungi: yup but you added perms for them to delete branches whic they didn't have before | 15:59 |
clarkb | via push you need force push perms to delete branches which aren't present | 15:59 |
fungi | right | 15:59 |
fungi | frenzy_friday: just double-checking, you're anbanerj right? | 16:00 |
fungi | i'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 anyway | 16:02 |
frenzy_friday | fungi, yep | 16:03 |
*** ysandeep is now known as ysandeep|out | 16:06 | |
*** gce108 is now known as gce108__ | 16:09 | |
opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Add a repository for the Large Scale SIG https://review.opendev.org/c/openstack/project-config/+/843534 | 17:29 |
opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Add a repository for the Large Scale SIG https://review.opendev.org/c/openstack/project-config/+/843534 | 17:31 |
*** dviroel is now known as dviroel|out | 21:00 | |
opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Fix propose-updates job for requirements (3rd attempt) https://review.opendev.org/c/openstack/project-config/+/843550 | 21:06 |
*** rlandy is now known as rlandy|biab | 21:23 | |
opendevreview | Merged openstack/project-config master: Fix propose-updates job for requirements (3rd attempt) https://review.opendev.org/c/openstack/project-config/+/843550 | 21:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!