| opendevreview | Ghanshyam proposed openstack/tempest master: Add new tempest job enable the rbac scope checks and new defaults https://review.opendev.org/c/openstack/tempest/+/614484 | 05:23 |
|---|---|---|
| *** jpena|off is now known as jpena | 08:39 | |
| opendevreview | Merged openstack/devstack master: [Doc] Fix tox command option to run smoke tests https://review.opendev.org/c/openstack/devstack/+/860915 | 09:26 |
| *** jpena is now known as jpena|off | 09:50 | |
| frickler | kopecmartin: how about we move to a single-core-reviewer rule formally, at least for non-functional changes like the stable job cleanups? they have been pending long enough now IMO | 09:53 |
| kopecmartin | frickler: sure, i usually wait and if there isn't any objection to my +2 I'll merge it | 09:56 |
| kopecmartin | i'm ok merging those 2 job cleaning patches | 09:57 |
| *** jpena|off is now known as jpena | 10:01 | |
| frickler | kopecmartin: IMO if the patch is already multiple weeks old, one can assume there has been enough time to state any objection, I don't see why reviewers would wait for that until after a +2 vote has been made | 10:05 |
| kopecmartin | that's true, only one situation comes to my mind when a reviewer forgot about the review, it dropped down in their review list and a new vote basically reminds them | 10:13 |
| frickler | kopecmartin: that might happen, but maybe then we can say "bad luck". also a revert is still possible then | 10:31 |
| opendevreview | Merged openstack/devstack stable/stein: Cleanup failing platform job https://review.opendev.org/c/openstack/devstack/+/860798 | 11:12 |
| opendevreview | Merged openstack/devstack stable/wallaby: Drop platform jobs for bionic and centos-8-stream https://review.opendev.org/c/openstack/devstack/+/842968 | 11:44 |
| opendevreview | Martin Kopec proposed openstack/devstack master: Extend single-core-review for non-functional changes https://review.opendev.org/c/openstack/devstack/+/863646 | 13:34 |
| kopecmartin | frickler: gmann ^ | 13:34 |
| frickler | +2 | 13:40 |
| opendevreview | Rodolfo Alonso proposed openstack/devstack master: Add new service "file_tracker" https://review.opendev.org/c/openstack/devstack/+/863324 | 14:09 |
| dansmith | gmann: frickler: can one of you +W? https://review.opendev.org/c/openstack/devstack/+/860976 | 15:27 |
| opendevreview | Merged openstack/tempest master: Clarify the enforce_scope config option help message https://review.opendev.org/c/openstack/tempest/+/861497 | 16:48 |
| opendevreview | Merged openstack/tempest master: [Doc] Fix python version in unit tests document https://review.opendev.org/c/openstack/tempest/+/861034 | 16:48 |
| *** jpena is now known as jpena|off | 17:33 | |
| opendevreview | Merged openstack/devstack master: Extend single-core-review for non-functional changes https://review.opendev.org/c/openstack/devstack/+/863646 | 18:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!