Hi I got -1 on gerrit-hooks please check why. https://gerrit.ovirt.org/#/c/79829/ https://gerrit.ovirt.org/#/c/79828/ Thank you, -- SHIRLY RADCO BI SOFTWARE ENGINEER Red Hat Israel <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted>
It seems it tells you that right in the gerrit comment: - Check Bug-Url::WARN, no bug url/s found (-1) - Check Backport::WARN, The patch wasn't backported to all the relevant stable branches. (-1) On 26 July 2017 at 13:46, Shirly Radco <sradco@redhat.com> wrote:
Hi
I got -1 on gerrit-hooks please check why.
https://gerrit.ovirt.org/#/c/79829/ https://gerrit.ovirt.org/#/c/79828/
Thank you,
--
SHIRLY RADCO
BI SOFTWARE ENGINEER
Red Hat Israel <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted>
_______________________________________________ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
-- Barak Korren RHV DevOps team , RHCE, RHCi Red Hat EMEA redhat.com | TRIED. TESTED. TRUSTED. | redhat.com/trusted
Check Backport::WARN, The patch wasn't backported to all the relevant stable branches. (-1) this means the patch isn't merged yet on master branch. The hooks tries to verify you merge patches in the right order: master -> stable branch. On Wed, Jul 26, 2017 at 1:46 PM, Shirly Radco <sradco@redhat.com> wrote:
Hi
I got -1 on gerrit-hooks please check why.
https://gerrit.ovirt.org/#/c/79829/ https://gerrit.ovirt.org/#/c/79828/
Thank you,
--
SHIRLY RADCO
BI SOFTWARE ENGINEER
Red Hat Israel <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted>
_______________________________________________ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
-- Eyal edri ASSOCIATE MANAGER RHV DevOps EMEA VIRTUALIZATION R&D Red Hat EMEA <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> phone: +972-9-7692018 irc: eedri (on #tlv #rhev-dev #rhev-integ)
On Wed, Jul 26, 2017 at 1:51 PM, Eyal Edri <eedri@redhat.com> wrote:
Check Backport::WARN, The patch wasn't backported to all the relevant stable branches. (-1) this means the patch isn't merged yet on master branch.
The hooks tries to verify you merge patches in the right order: master -> stable branch.
Since this is a patch which only relevant to the stable branch, this is false positive, but we can't really differ today between patches like that to others. This is why any project owner has permissions to remove any voting from patches on his project if he things its correct. So in this case, feel free to remove the voting.
On Wed, Jul 26, 2017 at 1:46 PM, Shirly Radco <sradco@redhat.com> wrote:
Hi
I got -1 on gerrit-hooks please check why.
https://gerrit.ovirt.org/#/c/79829/ https://gerrit.ovirt.org/#/c/79828/
Thank you,
--
SHIRLY RADCO
BI SOFTWARE ENGINEER
Red Hat Israel <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted>
_______________________________________________ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
--
Eyal edri
ASSOCIATE MANAGER
RHV DevOps
EMEA VIRTUALIZATION R&D
Red Hat EMEA <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> phone: +972-9-7692018 <+972%209-769-2018> irc: eedri (on #tlv #rhev-dev #rhev-integ)
-- Eyal edri ASSOCIATE MANAGER RHV DevOps EMEA VIRTUALIZATION R&D Red Hat EMEA <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> phone: +972-9-7692018 irc: eedri (on #tlv #rhev-dev #rhev-integ)
First it also failed on missing bug URL Suggestion, can't a certain commit message differ between bugs that needs to be back ported and those who aren't? On Wed, Jul 26, 2017 at 1:52 PM, Eyal Edri <eedri@redhat.com> wrote:
On Wed, Jul 26, 2017 at 1:51 PM, Eyal Edri <eedri@redhat.com> wrote:
Check Backport::WARN, The patch wasn't backported to all the relevant stable branches. (-1) this means the patch isn't merged yet on master branch.
The hooks tries to verify you merge patches in the right order: master -> stable branch.
Since this is a patch which only relevant to the stable branch, this is false positive, but we can't really differ today between patches like that to others.
This is why any project owner has permissions to remove any voting from patches on his project if he things its correct. So in this case, feel free to remove the voting.
On Wed, Jul 26, 2017 at 1:46 PM, Shirly Radco <sradco@redhat.com> wrote:
Hi
I got -1 on gerrit-hooks please check why.
https://gerrit.ovirt.org/#/c/79829/ https://gerrit.ovirt.org/#/c/79828/
Thank you,
--
SHIRLY RADCO
BI SOFTWARE ENGINEER
Red Hat Israel <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted>
_______________________________________________ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
--
Eyal edri
ASSOCIATE MANAGER
RHV DevOps
EMEA VIRTUALIZATION R&D
Red Hat EMEA <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> phone: +972-9-7692018 <+972%209-769-2018> irc: eedri (on #tlv #rhev-dev #rhev-integ)
--
Eyal edri
ASSOCIATE MANAGER
RHV DevOps
EMEA VIRTUALIZATION R&D
Red Hat EMEA <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> phone: +972-9-7692018 <+972%209-769-2018> irc: eedri (on #tlv #rhev-dev #rhev-integ)
_______________________________________________ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
participants (4)
-
Barak Korren -
Eyal Edri -
Gil Shinar -
Shirly Radco