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

delete-only-pre-release-versions throws Error: query for oldest version failed. #59

Open
cat-prices opened this issue Dec 17, 2021 · 2 comments

Comments

@cat-prices
Copy link

cat-prices commented Dec 17, 2021

If there are too many steps in a workflow then it throws an error.

Run actions/[email protected]
  with:
    package-name: Package.Name
    min-versions-to-keep: 10
    delete-only-pre-release-versions: true
    num-old-versions-to-delete: 1
    ignore-versions: ^$
    token: ***
(node:1585) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
Error: query for oldest version failed. verify input parameters are correct

I have 50 packages and noticed that the first five or so always seemed to run successfully, even when I changed the order in which packages were deleted. I have one step per package.
As a work around, I split the workflow into ten separate workflows, with five steps in each one. They all run successfully when the number of steps/packages is limited to a few per workflow.

@NamrataJha
Copy link
Contributor

Hey @codan-prices I am unable to repro this on my side. Can you specify what is the rough amount of versions present in the packages?

@cat-prices
Copy link
Author

cat-prices commented Apr 7, 2022

Hey @codan-prices I am unable to repro this on my side. Can you specify what is the rough amount of versions present in the packages?

Sorry, totally missed your question. The number of versions varies for each package but it's a large number. Picked one at random and it has 30 versions per page, and 40 pages. So 1200+ per package. That's after it runs and cleans up prerelease pacakges.

I've just tested this with v3 of the action and am still getting some of the packages return this error

Run actions/delete-package-versions@v3
  with:
    package-name: packagename-sanitised
    min-versions-to-keep: 10
    delete-only-pre-release-versions: true
    num-old-versions-to-delete: 1
    ignore-versions: ^$
    token: ***
Total versions deleted till now: 11
Total versions deleted till now: 86
Total versions deleted till now: 99
Error: query for oldest version failed. verify input parameters are correct

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants