| -@gerrit:opendev.org- Zuul merged: [opendev/system-config] Add matrix term accept instruction https://review.opendev.org/c/opendev/system-config/+/805447 | 19:19 | |
| -@gerrit:opendev.org- Tristan Cacqueray https://matrix.to/#/@tristanc_:matrix.org proposed: [opendev/ci-sandbox] Test gerrit events stream for the gerritbot-matrix https://review.opendev.org/c/opendev/ci-sandbox/+/803377 | 19:29 | |
| @tristanc_:matrix.org | Clark: here the identity lookup in action ^, thank you for quickly fixing the bot | 19:29 |
|---|---|---|
| @clarkb:matrix.org | Cool | 19:30 |
| @clarkb:matrix.org | I guess it doesn't lookup on merges (maybe because that info isn't easily accessible depending on whether or not it was a fast forward) | 19:30 |
| @tristanc_:matrix.org | The merge event should have the change author, it's just not displayed at the moment | 19:32 |
| @tristanc_:matrix.org | the format being `event-author event-action: [change.project] change.title` | 19:33 |
| @tristanc_:matrix.org | we could make the event-action of a merge event be `"on behalf of" change.author` when event.author /= change.author | 19:35 |
| @mordred:inaugust.com | tristanC: "on behalf of" is an interesting thing | 20:32 |
| @mordred:inaugust.com | kind of a neat way to express what's going on | 20:33 |
| @tristanc_:matrix.org | it might be too verbose when multiple changes are grouped in a single message | 20:38 |
| @jim:acmegating.com | test message | 20:42 |
| @tristanc_:matrix.org | i mean, when zuul merge 3 changes from 2 authors, should it be `Zuul merged on behalf of @author1, @author2: - change 1\n- change 2\n- change 3` | 20:43 |
| or `- change 1 (by @author1)\n- change 2 (by @author2)` ... | ||
| @jim:acmegating.com | maybe split it into multiple messages if the author changes? | 20:45 |
| @clarkb:matrix.org | Does gerrit batch the merge events? We should always get separate events I think and can report them separately | 20:47 |
| @tristanc_:matrix.org | Clark: the bot wait 5 seconds to try to process multiple events at once | 20:50 |
| @tristanc_:matrix.org | to help mitigate patch bomb spam | 20:51 |
| -@gerrit:opendev.org- Tristan Cacqueray https://matrix.to/#/@tristanc_:matrix.org proposed: | 20:52 | |
| - [opendev/ci-sandbox] Test gerrit events stream for the gerritbot matrix https://review.opendev.org/c/opendev/ci-sandbox/+/803377 | ||
| - [opendev/ci-sandbox] A follow up event test https://review.opendev.org/c/opendev/ci-sandbox/+/805457 | ||
| @tristanc_:matrix.org | Clark: like this ^ | 20:52 |
| @clarkb:matrix.org | I see | 21:04 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Serve matrix well-known files from apache https://review.opendev.org/c/opendev/system-config/+/805458 | 21:29 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Assume gitea reverse proxy https://review.opendev.org/c/opendev/system-config/+/805459 | 21:33 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Remove matrix well-known files from gitea image https://review.opendev.org/c/opendev/system-config/+/805460 | 21:34 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Assume gitea reverse proxy https://review.opendev.org/c/opendev/system-config/+/805459 | 21:38 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Remove matrix well-known files from gitea image https://review.opendev.org/c/opendev/system-config/+/805460 | 21:38 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Move #zuul from OFTC to Matrix https://review.opendev.org/c/opendev/system-config/+/805464 | 21:45 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:47 | |
| - [opendev/system-config] Serve matrix well-known files from apache https://review.opendev.org/c/opendev/system-config/+/805458 | ||
| - [opendev/system-config] Assume gitea reverse proxy https://review.opendev.org/c/opendev/system-config/+/805459 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Remove matrix well-known files from gitea image https://review.opendev.org/c/opendev/system-config/+/805460 | 21:48 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Serve matrix well-known files from apache https://review.opendev.org/c/opendev/system-config/+/805458 | 21:58 | |
| -@gerrit:opendev.org- Tristan Cacqueray https://matrix.to/#/@tristanc_:matrix.org proposed: [opendev/system-config] Update gerritbot-matrix to the latest version for on behalf annotation https://review.opendev.org/c/opendev/system-config/+/805465 | 21:58 | |
| @tristanc_:matrix.org | mordred: so I added "on behalf of" annotation, and ^ pulls in the feature | 21:59 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Assume gitea reverse proxy https://review.opendev.org/c/opendev/system-config/+/805459 | 22:11 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Remove matrix well-known files from gitea image https://review.opendev.org/c/opendev/system-config/+/805460 | 22:12 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] WIP Update Matrix instructions https://review.opendev.org/c/zuul/zuul/+/805467 | 22:32 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-website] Update website for Matrix https://review.opendev.org/c/zuul/zuul-website/+/805469 | 22:39 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [opendev/system-config] Update Gerrit images to most recent releases https://review.opendev.org/c/opendev/system-config/+/805471 | 22:56 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] Serve matrix well-known files from apache https://review.opendev.org/c/opendev/system-config/+/805458 | 22:59 | |
| -@gerrit:opendev.org- Zuul merged: [opendev/system-config] Update gerritbot-matrix to the latest version for on behalf annotation https://review.opendev.org/c/opendev/system-config/+/805465 | 23:12 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!