Thanks for comments. Yeah, I think since it does not produce extra side effect over the previous behaviour in dead case just something will stop working based on the old assumptions rather than it starts moving incorrect bugs around.
I have prepared a change [1], will give it some test on staging gerrit first and then merge and deploy to production gerrit. [1] https://gerrit.ovirt.org/#/c/97605/ On Wed, Feb 6, 2019 at 11:41 AM Yedidyah Bar David <[email protected]> wrote: > On Wed, Feb 6, 2019 at 12:36 PM Anton Marchukov <[email protected]> > wrote: > > > > Hello All. > > > > I have checked the hooks code. And it indeed just extracts all the links > pointing to bugzilla. This is not correct and fails when somebody just > mentions a bug in commit message. > > > > We are about to fix this and adjust the regexp used to explicitly check > for "Bug-Url" keyword. I think this is the expected behavior for everybody. > > +1 > > > > > But just in case I am sending this pre-announcement about the change. > Let me know if you anticipate any problems. > > I think we'll have enough time to fix such problems. It will only affect > new patches, history would remain as-is. So main risk is if people had > tools/hooks/habits to link to BZs without 'Bug-Url' and expected that to > work, and it will now be ignored. Small risk, imo. > > Thanks! > > > > > Thanks. > > > > On Tue, Feb 5, 2019 at 10:16 AM Nir Soffer <[email protected]> wrote: > >> > >> If a commit message mention another bug, the CI script try to add the > patch > >> to the bug in the commit message, and change the bug to POST. > >> > >> Mentioning another bug in a commit message is good practice, making it > >> easier to follow, and avoiding unclear forms like "bug 1000000" or > "BZ#1000000", > >> or even worse shortened urls like https://goo.gl/bPuFGo. > >> > >> Does it make sense that we cannot link to Red Hat bugzilla like god > >> intended? > >> > >> Here is a proof: > >> https://gerrit.ovirt.org/c/97568/ > >> > >> gerrit-hooks > >> Patch Set 1: > >> > >> Check Bug-Url::10000000::WARN, failed to get bug info (private bug > or bug doesn't exist > >> Check Product::IGNORE, not relevant for branch: master > >> Check TM::IGNORE, not relevant for branch: master > >> Check Backport::IGNORE, not relevant for branch: master > >> Set POST::#10000000::WARN, failed to get bug info (private bug or > bug doesn't exist) > >> Update Tracker::#10000000::WARN, failed to get bug info (private > bug or bug doesn't exist) > >> CI scripts should process urls only inside Bug-Url: tag. > >> > >> Expected behavior: > >> Extract bug urls *only* from Bug-Url: label. > >> > >> The same issue exists with Related-To: label. > >> > >> This is not a new bug. I reported it few years ago but for some reason > the issue > >> was not understood. > >> > >> Nir > >> _______________________________________________ > >> Infra mailing list -- [email protected] > >> To unsubscribe send an email to [email protected] > >> Privacy Statement: https://www.ovirt.org/site/privacy-policy/ > >> oVirt Code of Conduct: > https://www.ovirt.org/community/about/community-guidelines/ > >> List Archives: > https://lists.ovirt.org/archives/list/[email protected]/message/YZKHQTISCF6W3GNXOTWWO3IE4T24SZQQ/ > > > > > > > > -- > > Anton Marchukov > > Team Lead - Release Management - RHV DevOps - Red Hat > > > > _______________________________________________ > > Devel mailing list -- [email protected] > > To unsubscribe send an email to [email protected] > > Privacy Statement: https://www.ovirt.org/site/privacy-policy/ > > oVirt Code of Conduct: > https://www.ovirt.org/community/about/community-guidelines/ > > List Archives: > https://lists.ovirt.org/archives/list/[email protected]/message/MFPBLV2MJEG7QNONFOU3KMV2DAUWP5SM/ > > > > -- > Didi > -- Anton Marchukov Team Lead - Release Management - RHV DevOps - Red Hat
_______________________________________________ Devel mailing list -- [email protected] To unsubscribe send an email to [email protected] Privacy Statement: https://www.ovirt.org/site/privacy-policy/ oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ List Archives: https://lists.ovirt.org/archives/list/[email protected]/message/37VM4RPI2UCSLHVWPFZ32KOGW44KGPWD/
