| *** ysandeep|out is now known as ysandeep | 06:05 | |
| *** amoralej|off is now known as amoralej | 07:56 | |
| *** ysandeep is now known as ysandeep|lunch | 08:30 | |
| *** jpena|off is now known as jpena | 08:37 | |
| *** ysandeep|lunch is now known as ysandeep | 09:17 | |
| *** ministry is now known as __ministry | 09:48 | |
| *** sboyron_ is now known as sboyron | 10:44 | |
| *** rlandy|out is now known as rlandy|ruck | 11:01 | |
| *** bhagyashris_ is now known as bhagyashris|ruck | 11:05 | |
| *** dviroel|afk is now known as dviroel | 11:20 | |
| *** sshnaidm|afk is now known as sshnaidm | 11:35 | |
| *** jcapitao is now known as jcapitao_lunch | 11:56 | |
| *** jcapitao_lunch is now known as jcapitao | 12:40 | |
| *** amoralej is now known as amoralej|lunch | 13:23 | |
| *** dasm|off is now known as dasm | 13:34 | |
| *** ysandeep is now known as ysandeep|brb | 13:35 | |
| opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Drop gerritbot from #osism channel https://review.opendev.org/c/openstack/project-config/+/826671 | 13:41 |
|---|---|---|
| *** ysandeep|brb is now known as ysandeep | 14:03 | |
| *** pmatulis2 is now known as pmatulis | 14:17 | |
| pmatulis | `is:mergeable` is now illegal in gerrit searches - what is the new sexy? | 14:18 |
| frickler | pmatulis: iiuc gerrit no longer checks this at all, you would need to rely on zuul jobs to detect merge conflicts | 14:19 |
| *** amoralej|lunch is now known as amoralej | 14:23 | |
| pmatulis | that's unfortunate. i wonder why we can't search on this since gerrit clearly labels PRs with merge conflicts or WIP | 14:24 |
| pmatulis | fine: https://www.gerritcodereview.com/3.4.html#ismergeable-predicate-is-disabled-per-default | 14:29 |
| frickler | infra-root: ^^ I wasn't aware that this is still a selectable option. since we don't seem to have any load issues, maybe we want to re-enable this for our setup? | 14:53 |
| fungi | frickler: yes, it was something we discussed briefly but should probably have another conversation about. if gerrit upstream has plans to rip it out in a future release now that it's no longer enabled by default, we may just want to follow their lead now rather than later | 14:56 |
| *** Guest292 is now known as diablo_rojo_phone | 15:01 | |
| frickler | fungi: the gerrit doc doesn't mention a deprecation even for 3.5.0 afaict. I'll add it to our agenda to discuss | 15:09 |
| fungi | thanks, i agree it seems like maybe they had some complaints and switched to a more conservative default, but that we could potentially turn it back on if our users found it genuinely useful | 15:11 |
| frickler | for me it is one of the indicators I use in deciding whether a patch has been abandoned by the author and can thus also be abandoned in gerrit | 15:17 |
| frickler | and it seems there is some interest by others, too | 15:18 |
| *** dviroel is now known as dviroel|lunch | 16:13 | |
| clarkb | frickler: fungi: the main downside for us is during reindexing I think | 16:16 |
| clarkb | that information has to be calculated when reindexing | 16:17 |
| opendevreview | Merged openstack/project-config master: Drop gerritbot from #osism channel https://review.opendev.org/c/openstack/project-config/+/826671 | 16:30 |
| *** jpena is now known as jpena|off | 16:43 | |
| *** ysandeep is now known as ysandeep|out | 17:21 | |
| *** dviroel|lunch is now known as dviroel | 17:26 | |
| *** amoralej is now known as amoralej|off | 17:30 | |
| *** sshnaidm is now known as sshnaidm|afk | 18:13 | |
| *** dviroel is now known as dviroel|out | 20:20 | |
| opendevreview | M Camp proposed openstack/project-config master: Update StarlingX docs promote job for R6 release https://review.opendev.org/c/openstack/project-config/+/826775 | 21:33 |
| opendevreview | Merged openstack/project-config master: Update StarlingX docs promote job for R6 release https://review.opendev.org/c/openstack/project-config/+/826775 | 22:03 |
| *** dasm is now known as dasm|off | 22:58 | |
| *** rlandy|ruck is now known as rlandy|ruck|bbl | 23:26 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!