Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Saved state not found #1124

Closed
2 of 5 tasks
rcomer opened this issue Dec 20, 2023 · 6 comments
Closed
2 of 5 tasks

Saved state not found #1124

rcomer opened this issue Dec 20, 2023 · 6 comments
Labels
bug Something isn't working

Comments

@rcomer
Copy link

rcomer commented Dec 20, 2023

Description:
Workflow ran out of operations on the first run but still started from the beginning on the second.

Action version:
9

Platform:

  • Ubuntu
  • macOS
  • Windows

Runner type:

  • Hosted
  • Self-hosted

Repro steps:
Here is our workflow: Label inactive PRs.

This has run twice since updating to v9. On the first run, the output indicated state: persisting info about 203 issue(s). On the next run, The saved state was not found, the process starts from the first issue.

We have operations-per-run set low in order to work slowly through a large backlog of issues, but didn't want steps like removing the label after update or closing the issue to wait until we'd worked through the whole backlog. So when we updated to v9, we also added a second workflow using the action with days-before-stale=-1 and a much higher operations-per-run (see my PR here matplotlib/matplotlib#27523). The two workflows run on alternate days. Perhaps using the second workflow is interfering with the state somehow? Though the state was also not found by the second workflow when it ran in between.

Expected behavior:
Original workflow finds state and works from where it got to last time.

Actual behavior:
Original workflow does not find state, starts from the beginning.

@rcomer rcomer added bug Something isn't working needs triage labels Dec 20, 2023
@marko-zivic-93
Copy link
Contributor

Hello @rcomer
Thank you for creating this issue. We will investigate it and come back to you as soon as we have some feedback.

@rcomer
Copy link
Author

rcomer commented Jan 3, 2024

I tried to reproduce the problem in a dummy repo and couldn't. Both workflows successfully interact with the cached state. Any suggestions for troubleshooting why Matplotlib's workflows are not finding the state?

For Matplotlib's use-case we would want the second workflow to ignore the cached state that the first one saved. Is that something that would be easy to implement? Or is there a better way to make sure the already-labelled issues get closed or their labels removed in a timely manner?

Thanks!

@rcomer
Copy link
Author

rcomer commented Jan 4, 2024

Actually the second workflow did find the cache when it ran two days ago (and then deleted it because it worked through all the remaining issues). A new cache was then saved by the first workflow one day ago, and is still there, but was not found by the most recent run of the second workflow.

So I fear that whatever is going wrong here is not reproducible ☹️

@anomiex
Copy link

anomiex commented Feb 22, 2024

Possibly this is the same as #1136?

@rcomer
Copy link
Author

rcomer commented Feb 22, 2024

We do indeed have many caches!

@rcomer
Copy link
Author

rcomer commented Feb 22, 2024

I'm going to close this as a duplicate of #1136 which has debugging details. I will open a new feature request for our issue about using more than one workflow.

Edit: feature request now at #1137

@rcomer rcomer closed this as not planned Won't fix, can't repro, duplicate, stale Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants