| *** amoralej|off is now known as amoralej | 06:07 | |
| *** auniyal0 is now known as auniyal | 07:45 | |
| hberaud | ttx, elodilles_pto I wonders why this patch isn't already merged https://review.opendev.org/c/openstack/releases/+/883795 .... even rechecks didn't worked | 09:21 |
|---|---|---|
| *** amoralej is now known as amoralej|lunch | 11:26 | |
| frickler | hberaud: need to set ptl-approved manually | 11:35 |
| frickler | affects all patches not actually submitted by ptls. maybe release team members can add themselves as release liasons for the release project? but maybe that check only works if deliverables files are touched | 11:38 |
| *** amoralej|lunch is now known as amoralej | 12:14 | |
| hberaud | AFAIK this kind of check only applies on changes that touches deliverables files, we (the release team) haven't to set the `ptl-approved` flag for our internal changes | 12:35 |
| frickler | hberaud: that may have changed recentish as a result of gerrit updates. I had the same issue myself some weeks ago, c.f. https://review.opendev.org/c/openstack/releases/+/885066 | 13:33 |
| frickler | -. | 13:33 |
| opendevreview | Stephen Finucane proposed openstack/releases master: openstacksdk 1.3.0 https://review.opendev.org/c/openstack/releases/+/886508 | 15:17 |
| ttx | Yeah that's a consequence of making it a commit requirement so that the dashbaord is fixed | 15:26 |
| opendevreview | Merged openstack/releases master: Add a section about the importance of release patch reviews https://review.opendev.org/c/openstack/releases/+/883795 | 15:39 |
| *** amoralej is now known as amoralej|off | 16:47 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!