Project

General

Profile

Actions

action #165144

closed

OBS SCM/CI Workflow Integration started never creates OBS projects for some pull requests size:S

Added by livdywan about 1 month ago. Updated 5 days ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Regressions/Crashes
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Observation

The *OBS SCM/CI Workflow Integration started * workflow doesn't reliable work for all pull requests. Sometimes it doesn't create a project. Trying to view the results ends up in a redirect to github.com.

https://github.com/os-autoinst/openQA/pull/5845

There is a manual workaround but at this point it looks to be a regular issue so I consider this High priority to fix properly.

Workaround

Delete the project (if it exists) e.g. https://build.opensuse.org/project/show/devel:openQA:GitHub:os-autoinst:openQA:PR-5845 where the number matches the PR number and retrigger the corresponding webhook run.

Acceptance Criteria

  • AC1: OBS checks work reliably for all pull requests

Suggestions


Files

Actions #1

Updated by okurz about 1 month ago

  • Tags changed from alert, infra, reactive work to alert, reactive work
  • Project changed from openQA Infrastructure to openQA Project
  • Category set to Regressions/Crashes
  • Target version set to Ready
Actions #2

Updated by livdywan about 1 month ago

  • Description updated (diff)
Actions #3

Updated by okurz about 1 month ago

  • Subject changed from OBS SCM/CI Workflow Integration started never creates OBS projects for some pull requests to OBS SCM/CI Workflow Integration started never creates OBS projects for some pull requests size:S
  • Description updated (diff)
  • Status changed from New to Workable
Actions #4

Updated by mkittler about 1 month ago

Do we know whether the initial webhook could be delivered correctly? (I guess I'll pay attention to the problem next time I'll encounter it. I don't recall seeing this frequently, though.)

Actions #5

Updated by jbaier_cz about 1 month ago

mkittler wrote in #note-4:

Do we know whether the initial webhook could be delivered correctly? (I guess I'll pay attention to the problem next time I'll encounter it. I don't recall seeing this frequently, though.)

In case of failed webhooks, maybe https://docs.github.com/en/webhooks/using-webhooks/automatically-redelivering-failed-deliveries-for-a-repository-webhook is something we can try.

Actions #6

Updated by tinita 29 days ago

I found this issue: https://github.com/openSUSE/open-build-service/issues/12661
One difference is that in our webhook logs I usually see a 504 http status, not a 400.

Actions #7

Updated by tinita 29 days ago

  • Status changed from Workable to In Progress
  • Assignee set to tinita
Actions #8

Updated by tinita 29 days ago

  • Description updated (diff)
Actions #9

Updated by tinita 29 days ago · Edited

  • Status changed from In Progress to Feedback

ci: Add helper for reporting OBS url:

https://github.com/os-autoinst/openQA/pull/5872
https://github.com/os-autoinst/os-autoinst/pull/2537

edit: closed, will create new ones

Actions #10

Updated by livdywan 28 days ago

We talked about it in the unblock. We're not expecting the OBS check fixed upstream considering the abstractions and the metadata not being available directly.

In case of failed webhooks, maybe https://docs.github.com/en/webhooks/using-webhooks/automatically-redelivering-failed-deliveries-for-a-repository-webhook is something we can try.

I will give the redelivery boilerplate a go. Maybe we can delete the project via API call and redeliver.

Actions #11

Updated by livdywan 28 days ago

I will give the redelivery boilerplate a go. Maybe we can delete the project via API call and redeliver.

Turns out someone did make this into a re-usable action afterall, so os-autoinst/openQA#5877 actually uses gateixeira/retrigger-webhook-action as opposed to implementing it from scratch. This action also supports multiple hooks without hard-coding a specific one.

Actions #12

Updated by tinita 27 days ago

Actions #13

Updated by tinita 26 days ago

https://github.com/os-autoinst/openQA/pull/5879 merged but:
https://github.com/os-autoinst/openQA/pull/5888 Run OBS helper for pull_request_target (also merged now)

Actions #15

Updated by tinita 20 days ago

  • Status changed from Feedback to Blocked

No further comments on the automatic redeliver approach.
Blocking on https://github.com/openSUSE/open-build-service/issues/12661

Actions #16

Updated by livdywan 14 days ago

Actions #17

Updated by livdywan 7 days ago

I asked on help-obs on Slack if someone could take another look at the ticket. If nothing else to confirm if a fix is plausible or we should pursue work-arounds like redelivery or other options.

Actions #19

Updated by livdywan 5 days ago

  • Status changed from Blocked to Resolved

tinita wrote in #note-18:

That helped.
https://github.com/openSUSE/open-build-service/pull/16834

Merged 🤓

Actions

Also available in: Atom PDF