Hi all, The docs say [1] that for gerrit we can comment 'ci re-merge please'. But for github [2] I don't see anything similar, only 'test|check' and 'build'. Latest merged PR for ovirt-ansible-hosted-engine-setup [3] failed due to an unrelated reason (see thread "OST basic suite fails on 002_bootstrap.add_secondary_storage_domains"), now fixed, and I want CQ to handle [3] again. How? Thanks! [1] https://ovirt-infra-docs.readthedocs.io/en/latest/CI/Using_STDCI_with_Gerrit... [2] https://ovirt-infra-docs.readthedocs.io/en/latest/CI/Using_STDCI_with_GitHub... [3] https://github.com/oVirt/ovirt-ansible-hosted-engine-setup/pull/306 -- Didi
Hi Didi, you need to trigger it from the webhooks. I think this is the one that you need but you need to select the correct change from the list and redeliver it: https://github.com/oVirt/ovirt-ansible-hosted-engine-setup/settings/hooks/43... let me know if you need help, Dafna On Wed, Mar 11, 2020 at 7:15 AM Yedidyah Bar David <didi@redhat.com> wrote:
On Wed, Mar 11, 2020 at 10:54 AM Ehud Yonasi <eyonasi@redhat.com> wrote:
You can trigger the merged job again in jenkins also.
Is this documented? How? Just rebuild: https://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/21146/ ? Also: How to find it? The only reliable way I have right now is from the emails it sends to infra. Can you somehow make the job add a comment in github? Although I admit I now consider moving it to gerrit, as I do not see any advantages to github, and gerrit is definitely more supported.
I'll try and update. Thanks!
-- Didi
On Wed, 11 Mar 2020 at 13:29, Yedidyah Bar David <didi@redhat.com> wrote:
to gerrit, as I do not see any advantages to github, and gerrit is
definitely more supported.
There is no place in GitHub to add such a comment. Once a PR is closed it can no longer be commented upon, unlike patches in Gerrit.
-- Barak Korren RHV DevOps team , RHCE, RHCi Red Hat EMEA redhat.com | TRIED. TESTED. TRUSTED. | redhat.com/trusted
On Wed, Mar 11, 2020 at 1:35 PM Barak Korren <bkorren@redhat.com> wrote:
So how did I comment? See [1]. Was merged Mar 10 evening, I commented in the next morning. [1] https://github.com/oVirt/ovirt-ansible-hosted-engine-setup/pull/306
-- Didi
participants (4)
-
Barak Korren -
Dafna Ron -
Ehud Yonasi -
Yedidyah Bar David