Folks,
As discussed at this week's VPP Community Meeting, I have completed
development of gerrit change auto-abandonment and incorporated the
reporting of auto-abandoned gerrit changes in the daily
vpp-gerrit-report [0] generation scripts.
The gerrit instance is being stressed with the large number of open
changes and leaving hundreds of open gerrit changes reduces the
signal-to-noise ration for reviewers. Abandoned changes are not deleted
from gerrit, so they can always be restored. If you want them to remain
visible, then restore the change and mark them either as
work-in-progress (publicly visible) or private (visible to reviewers +
author). This will remove them from regular CI processing that gerrit
does and help improve gerrit performance.
Alternatively, gerrit changes that are to be merged can simply be
rebased (which they should be prior to merging to ensure testing with
the latest codebase prior to merge).
I have also written a shellscript to search the repo for gerrit changes
that were auto-abandoned by the daily scripts to make it easy to find
and restore auto-abandoned changes. Here's how to use it:
1. git clone https://github.com/vpp-dev/vpp-gerrit-report && cd
vpp-gerrit-report
2. ./scripts/find_abandoned_changes.sh <email>
Here is a sample output of the shellscript:
/scratch/dwallacelf/github.com/vpp-dev/vpp-gerrit-report$
./scripts/find_abandoned_changes.sh dwallac...@gmail.com
**Dave Wallace** <dwallac...@gmail.com>:
Found in
https://github.com/vpp-dev/vpp-gerrit-report/blob/6d844808a029747406e2e2ca39b3c6d62e56e09a/VPP-GERRIT-REPORT.rst
| `33727 <https:////gerrit.fd.io/r/c/vpp/+/33727>`_ [A 0]: tests:
relicense
| `33707 <https:////gerrit.fd.io/r/c/vpp/+/33707>`_ [A 0]: papi:
relicense
Found in
https://github.com/vpp-dev/vpp-gerrit-report/blob/33cdcd6ef678873c9b5feacb81296ac6243dc352/VPP-GERRIT-REPORT.rst
| `37088 <https:////gerrit.fd.io/r/c/vpp/+/37088>`_ [A 0]: misc:
patch to test CI infra changes
Found in
https://github.com/vpp-dev/vpp-gerrit-report/blob/4f1f0aeed38480d39c188a2dc0d338de356a9818/VPP-GERRIT-REPORT.rst
| `37088 <https:////gerrit.fd.io/r/c/vpp/+/37088>`_ [A 0]: misc:
patch to test CI infra changes
Found in
https://github.com/vpp-dev/vpp-gerrit-report/blob/88e5d41324dec032db40eba4639573f2f6d37cca/VPP-GERRIT-REPORT.rst
| `37088 <https:////gerrit.fd.io/r/c/vpp/+/37088>`_ [A 0]: misc:
patch to test CI infra changes
Thanks,
-daw-
[0] https://github.com/vpp-dev/vpp-gerrit-report
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#22235): https://lists.fd.io/g/vpp-dev/message/22235
Mute This Topic: https://lists.fd.io/mt/95224721/21656
Group Owner: vpp-dev+ow...@lists.fd.io
Unsubscribe: https://lists.fd.io/g/vpp-dev/leave/1480452/21656/631435203/xyzzy
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
