| *** redrobot2 is now known as redrobot | 05:58 | |
| *** gibi is now known as gibi_pto_back_thu | 06:11 | |
| frickler | kopecmartin: I would like to abandon old devstack reviews in order to make the backlog manageable again. with a patch like https://paste.opendev.org/show/810357/ it would be possible to "borrow" the script neutron is using regularly. we can also discuss at the next meeting https://paste.opendev.org/show/810357/ | 08:13 |
|---|---|---|
| frickler | ianw: yoctozepto: gmann: clarkb: other devstack reviewers: ^^ interested in your opinion, too | 08:14 |
| kopecmartin | frickler: i think it's a good idea , i'd propose to do this in tempest as well .. i've been thinking about this for some time, I wanted to write to the ML asking owners to either update their patch or abandon it .. however, script approach is much faster | 09:34 |
| kopecmartin | i would start with a dry run and check how many patches will be affected by 6 month setting .. if too many , we can make it even higher for start | 09:35 |
| opendevreview | Lee Yarwood proposed openstack/tempest master: zuul: Add a CentOS 8 stream job to integrated-gate-compute https://review.opendev.org/c/openstack/tempest/+/797614 | 09:35 |
| frickler | kopecmartin: I did a dry-run for devstack and it found 200 patches | 09:36 |
| frickler | kopecmartin: I'd also assume 90% of the patch owners are no longer active or have forgotten about those | 09:37 |
| kopecmartin | hmm, that's around 80% of the currently open patches | 09:39 |
| kopecmartin | i see the point though, 6 months is a very long time and this step would remind the owners they have patches open | 09:41 |
| kopecmartin | there is no benefit of having the old reviews open, we can't go over them ourselves and verify if they are still valid, this way we have a chance the reviews get updated | 09:44 |
| kopecmartin | i can't estimate how fast and many patches will be restored and updated , but maybe we should keep an eye even more on the open patches | 09:45 |
| kopecmartin | after the automatic abandoning | 09:46 |
| kopecmartin | it wouldn't send a good signal if we abandon a patch twice due to a delay on our side | 09:46 |
| frickler | kopecmartin: maybe we should go for 12m instead of 6, that would leave a bit more room, at least initially | 09:51 |
| kopecmartin | that sounds good, let's wait on couple of more opinions and then we can execute | 09:55 |
| ianw | frickler: to be completely honest open changes (and I have more than a few) don't bother me that much; it just kind of indicates to me that the problem they are solving are not that interesting right now -- but one day, for whatever reason, it may be | 10:02 |
| ianw | but, i will certainly not object if others feel that abandoning old things is what they want either | 10:03 |
| kopecmartin | maybe those not important patches atm could be marked as WIP? it's something I'm trying to do myself lately because i have also a few open patches (lots of them DNM ones where i test or experiment with something) | 10:08 |
| *** jpena|off is now known as jpena | 10:09 | |
| opendevreview | Lukas Piwowarski proposed openstack/patrole master: Add cleanup for test_show_auto_allocated_topology test https://review.opendev.org/c/openstack/patrole/+/788316 | 10:18 |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: DNM: Test Bullseye with more swap https://review.opendev.org/c/openstack/devstack/+/815958 | 10:34 |
| frickler | yoctozepto: ^^ you might be interested in that, came after looking at why centos jobs don't seem to be affected and also recent discussion in #-nova | 10:34 |
| *** dviroel|out is now known as dviroel|rover | 11:21 | |
| opendevreview | Merged openstack/devstack master: Stop creating a keystone admin site https://review.opendev.org/c/openstack/devstack/+/735472 | 12:30 |
| opendevreview | Merged openstack/devstack master: Make creation of keystone admin endpoint optional https://review.opendev.org/c/openstack/devstack/+/777339 | 12:30 |
| yoctozepto | frickler: ack, thanks; I found it earlier and rechecked | 14:25 |
| yoctozepto | if swap works reliably, we can do it | 14:26 |
| yoctozepto | kopecmartin: wip is wip, let's not mix the two ideas | 14:28 |
| yoctozepto | frickler: I'm all in about the abandoning | 14:29 |
| yoctozepto | 12m sounds less harsh though | 14:29 |
| yoctozepto | and for sure those 12m old with these properties are unlikely to get merged ever | 14:29 |
| gmann | frickler: kopecmartin ianw yoctozepto I am ok with the cleanup of old reviews. What i used to do in Tempest is 1. add comment of saying that if this is still needed please update otherwise next week we will abandon it 2. next week or so I abandon those with saying feel free to restore if this is still needed | 14:59 |
| frickler | gmann: do you have scripts for that? I only found the ones neutron uses | 15:06 |
| gmann | frickler: ah not script, I do it like manually. with last tab page in gerrit | 15:08 |
| gmann | +1 on neutron script | 15:08 |
| frickler | gmann: yeah, doing it manually for 200+ patches isn't something I would consider | 15:11 |
| yoctozepto | frickler: well, it's kinda relaxing to just go and drop :-) | 15:12 |
| yoctozepto | but yeah, still better to script it away | 15:12 |
| gmann | yeah, i do like one set of it in month or biweekly etc | 15:14 |
| clarkb | frickler: no objection from me on abandoning old patches. I've probably got a few in there that were DNM to test something or other that I should've cleaned up when done | 15:21 |
| *** jpena is now known as jpena|off | 17:48 | |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Add barbican to our devstack job https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/779043 | 17:51 |
| opendevreview | Ghanshyam proposed openstack/devstack stable/train: WIP: Pin Tempest on stable/train https://review.opendev.org/c/openstack/devstack/+/816549 | 19:28 |
| opendevreview | Ghanshyam proposed openstack/devstack master: Clarify error message for ERROR_ON_CLONE=True https://review.opendev.org/c/openstack/devstack/+/814233 | 19:34 |
| *** dviroel|rover is now known as dviroel|rover|afk | 19:45 | |
| opendevreview | Ghanshyam proposed openstack/tempest master: Test all supported python version for stable branch https://review.opendev.org/c/openstack/tempest/+/816553 | 20:05 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Test all supported python version for stable branch https://review.opendev.org/c/openstack/tempest/+/816553 | 20:09 |
| opendevreview | Ghanshyam proposed openstack/openstack-health master: Test python3.9 as voting job https://review.opendev.org/c/openstack/openstack-health/+/816558 | 20:19 |
| opendevreview | Ghanshyam proposed openstack/bashate master: Test python3.9 as voting job https://review.opendev.org/c/openstack/bashate/+/816559 | 20:19 |
| opendevreview | Ghanshyam proposed openstack/coverage2sql master: Test python3.9 as voting job https://review.opendev.org/c/openstack/coverage2sql/+/816560 | 20:19 |
| opendevreview | Ghanshyam proposed openstack/os-performance-tools master: Test python3.9 as voting job https://review.opendev.org/c/openstack/os-performance-tools/+/816561 | 20:30 |
| opendevreview | Ghanshyam proposed openstack/os-testr master: Test python3.9 as voting job https://review.opendev.org/c/openstack/os-testr/+/816562 | 20:30 |
| opendevreview | Ghanshyam proposed openstack/devstack-tools master: Test python3.9 as voting job https://review.opendev.org/c/openstack/devstack-tools/+/816563 | 20:30 |
| opendevreview | Ghanshyam proposed openstack/stackviz master: Test python3.9 as voting job https://review.opendev.org/c/openstack/stackviz/+/816564 | 20:34 |
| opendevreview | Ghanshyam proposed openstack/tempest-stress master: Test python3.9 as voting job https://review.opendev.org/c/openstack/tempest-stress/+/816565 | 20:34 |
| opendevreview | Ghanshyam proposed openstack/openstack-health master: Test python3.9 as voting job https://review.opendev.org/c/openstack/openstack-health/+/816558 | 20:36 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Remove broken tempest-full-py3-opensuse15 job https://review.opendev.org/c/openstack/tempest/+/816569 | 20:50 |
| opendevreview | Ghanshyam proposed openstack/bashate master: Test python3.9 as voting job https://review.opendev.org/c/openstack/bashate/+/816559 | 21:07 |
| opendevreview | Ghanshyam proposed openstack/coverage2sql master: Test python3.9 as voting job https://review.opendev.org/c/openstack/coverage2sql/+/816560 | 21:08 |
| opendevreview | Ghanshyam proposed openstack/os-performance-tools master: Test python3.9 as voting job https://review.opendev.org/c/openstack/os-performance-tools/+/816561 | 21:09 |
| opendevreview | Ghanshyam proposed openstack/devstack-tools master: Test python3.9 as voting job https://review.opendev.org/c/openstack/devstack-tools/+/816563 | 21:09 |
| opendevreview | Ghanshyam proposed openstack/stackviz master: Test python3.9 as voting job https://review.opendev.org/c/openstack/stackviz/+/816564 | 21:10 |
| *** dviroel|rover|afk is now known as dviroel|rover | 21:29 | |
| *** dviroel|rover is now known as dviroel|rover|out | 22:18 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!