Project

General

Profile

Actions

action #158236

closed

Backlog Limits Checker github workflow fails on pull requests from forks size:S

Added by tinita 7 months ago. Updated 5 months ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Feature requests
Target version:
Start date:
2024-03-28
Due date:
2024-05-03
% Done:

0%

Estimated time:

Description

Observation

https://github.com/openSUSE/backlogger/actions/runs/8468254805/job/23200822772
The workflow is creating a preview of the HTML page in the origin gh-pages branch.
For that, it needs the right permissions. A PR with a branch from origin works, but it fails for forks.

Maybe there are other options to make it work.

Suggestions

  • Research how previews could be done
    • Give permissions to upstream to forks
    • Store the preview elsewhere
    • Lookup other ways to do this in the preview action, or other actions doing this

Files


Related issues 2 (1 open1 closed)

Copied to openQA Project - action #159783: Changes to queries.yaml in backlog status PRs are not reflected in the previewNew2024-03-28

Actions
Copied to openQA Project - action #159888: Open pull requests on Backlog Limits Checker use up too much spaceResolvedlivdywan

Actions
Actions #1

Updated by okurz 7 months ago

  • Target version set to Ready
Actions #2

Updated by okurz 6 months ago

  • Tags set to reactive work
  • Subject changed from Backlog Limits Checker github workflow fails on pull requests on forks to Backlog Limits Checker github workflow fails on pull requests from forks
  • Description updated (diff)
  • Status changed from New to Workable
Actions #3

Updated by tinita 6 months ago

  • Subject changed from Backlog Limits Checker github workflow fails on pull requests from forks to Backlog Limits Checker github workflow fails on pull requests from forks size:S
Actions #4

Updated by ybonatakis 6 months ago

  • Status changed from Workable to In Progress
  • Assignee set to ybonatakis
Actions #5

Updated by ybonatakis 6 months ago

for now i am waiting to get permissions to the https://github.com/openSUSE/backlogger as i cant access the settings

Actions #6

Updated by livdywan 6 months ago

  • Status changed from In Progress to Workable

Please don't leave it in progress unless you're actively working on it

Actions #7

Updated by ybonatakis 6 months ago

  • Status changed from Workable to In Progress

access granted. I can open settings on https://github.com/openSUSE/backlogger now.

Actions #8

Updated by ybonatakis 6 months ago

I added the REDMINE_API_KEY in the forked repo but this didnt work.
I found an open issue which lists some limitation for forked jobs and i commented there, and hope to get some answer/solution https://github.com/rossjrw/pr-preview-action/issues/54

Actions #9

Updated by openqa_review 6 months ago

  • Due date set to 2024-04-24

Setting due date based on mean cycle time of SUSE QE Tools

Actions #10

Updated by ybonatakis 6 months ago

  • Due date deleted (2024-04-24)

From the documentation is clear that the workflow doesnt work from forked repos in the current version. There is a PR[0] to fix the issue in the v2 but it is not ready yet.

[0] https://github.com/rossjrw/pr-preview-action/pull/6

Actions #11

Updated by ybonatakis 6 months ago

  • Status changed from In Progress to Feedback

ybonatakis wrote in #note-8:

I added the REDMINE_API_KEY in the forked repo but this didnt work.
I found an open issue which lists some limitation for forked jobs and i commented there, and hope to get some answer/solution https://github.com/rossjrw/pr-preview-action/issues/54

I got an answer too https://github.com/rossjrw/pr-preview-action/issues/54#issuecomment-2046165272 confirming the state of the project. there is a workaround tho which comes with some security caveats.

Change it to feedback for further discussion

Actions #12

Updated by livdywan 6 months ago

Discussed in the unblock:

Actions #13

Updated by ybonatakis 6 months ago

  • Status changed from Feedback to In Progress
Actions #15

Updated by openqa_review 6 months ago

  • Due date set to 2024-04-25

Setting due date based on mean cycle time of SUSE QE Tools

Actions #16

Updated by ybonatakis 6 months ago

  • Status changed from In Progress to Feedback

waiting for @titina

Actions #17

Updated by ybonatakis 6 months ago

  • Status changed from Feedback to In Progress

reopen due to the comment in the PR[0].

the testing commit is checked out but then Run the action implemented in this repo runs another checkout.

[0] https://github.com/openSUSE/backlogger/pull/38#issuecomment-2064171707

Actions #18

Updated by ybonatakis 6 months ago

  • Status changed from In Progress to Feedback
Actions #19

Updated by ybonatakis 6 months ago · Edited

addressing the comment[0] from @tinita i encountered an unrelated issue with the container. the previews actions looks good tho with the correct commit hash

[0] https://github.com/openSUSE/backlogger/pull/42#discussion_r1575344367

Actions #21

Updated by livdywan 6 months ago

  • Due date changed from 2024-04-25 to 2024-05-03

ybonatakis wrote in #note-20:

i think it works https://github.com/openSUSE/backlogger/actions/runs/8815937064/job/24220533045
but i cant tell for sure how it will affect https://github.com/os-autoinst/qa-tools-backlog-assistant/blob/master/.github/workflows/backlog_checker.yml#L21

Is this the correct link? That's a PR on the upstream repo. We need the preview to work on forks.

Actions #22

Updated by okurz 6 months ago

  • Status changed from Feedback to In Progress
  • Priority changed from Normal to Urgent

To actually address that we missed the planned due date we should bump prio as decided in the daily now.

Actions #23

Updated by livdywan 6 months ago

  • Status changed from In Progress to Workable

Let's discuss this on Monday. Pulling a ticket in progress without confirming someone can work on it is not enough to get it done. If someone did work on, or further discuss this, please do add your comments here of course.

Actions #24

Updated by ybonatakis 5 months ago

  • Status changed from Workable to In Progress
Actions #25

Updated by livdywan 5 months ago

Yannis and I were discussing this this morning (hence in progress). Hopefully we were able to bisect the relevant changes successfully.

I also prepared a PR to verify that this works correctly now https://github.com/openSUSE/backlogger/pull/45

Actions #26

Updated by livdywan 5 months ago · Edited

livdywan wrote in #note-25:

Yannis and I were discussing this this morning (hence in progress). Hopefully we were able to bisect the relevant changes successfully.

I also prepared a PR to verify that this works correctly now https://github.com/openSUSE/backlogger/pull/45

Seems like my changes to query.yaml were ignored. The limits in the generated preview are still the old ones.

Actions #27

Updated by livdywan 5 months ago

Seems like my changes to query.yaml were ignored. The limits in the generated preview are still the old ones.

Reviewing the code we hypothesize that the queries.yaml is being taken from the wrong folder. Since it uses a relative "config" filename.
It seems the fix we came up with doesn't make a difference, though?

Actions #28

Updated by livdywan 5 months ago

As discussed there's 3 more action items to wrap this up:

  • Propose a PR to remove branches: ["**"] from preview.yaml, provided it's not needed
  • Add a brief note to the https://github.com/openSUSE/backlogger README.md, maybe simply under Previews for pull requests, changes to queries.yaml don't currently work in previews.
  • File a follow-up ticket to make previews with changes to queries.yaml work

The implicit AC "make it not fail" is fulfilled.

Actions #29

Updated by livdywan 5 months ago

  • Copied to action #159783: Changes to queries.yaml in backlog status PRs are not reflected in the preview added
Actions #30

Updated by livdywan 5 months ago

  • Propose a PR to remove branches: ["**"] from preview.yaml, provided it's not needed

It seems the setting is needed, at least that's what I conclude from the proof of concept.

https://github.com/openSUSE/backlogger/pull/47

Actions #31

Updated by ybonatakis 5 months ago

i going to close this after https://github.com/openSUSE/backlogger/pull/47
we can tackle the rest in another ticket.
For now, at least, pipeline is green from forked, but gh-pages is not absolute correctly

Actions #32

Updated by ybonatakis 5 months ago

  • Status changed from In Progress to Feedback
Actions #33

Updated by ybonatakis 5 months ago

Side note: this PR seems intends to fix the upstream issue https://github.com/rossjrw/pr-preview-action/pull/6

Actions #34

Updated by ybonatakis 5 months ago

ybonatakis wrote in #note-33:

Side note: this PR seems to intent to fix the upstream issue https://github.com/rossjrw/pr-preview-action/pull/6

Actions #35

Updated by ybonatakis 5 months ago

  • Status changed from Feedback to In Progress
Actions #36

Updated by livdywan 5 months ago

I re-created my branch to update the README.md, and something's wrong with the gh-pages branch.

error: unable to write file pr-preview/pr-47/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-45/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-43/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-41/pr-preview/pr-41/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-41/index.html
fatal: cannot create directory at 'pr-preview/pr-47/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-45/pr-preview/pr-46/pr-preview/pr-46/pr-preview/pr-43/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-41/pr-preview/pr-41/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-42/pr-preview/pr-41/pr-preview': No space left on device

@tinita tried to clean up the gh-pages branch, so right now my PR re-ran successfully. After the second re-run I am once again seeing redundant folders:

I'll see if I can find out if this is a known issue, and come up with a work-around.

Actions #37

Updated by livdywan 5 months ago

  • Copied to action #159888: Open pull requests on Backlog Limits Checker use up too much space added
Actions #38

Updated by livdywan 5 months ago

  • Status changed from In Progress to Resolved
  • Assignee changed from livdywan to ybonatakis
  • Priority changed from Urgent to High

I'm giving back the ticket and resolving it. #159888 deals with the specific issue with redundant files being created. The preview has been improved, and they work in general and I don't believe the disk space issue is specific to forks either.

Actions

Also available in: Atom PDF