-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Dependabot enforcing engine-strict but with an old npm on Action Runners in last few days #10982
Comments
Looking at our logs Was fine on
Failing on
|
I searched for the error and saw 52 events in the last hour. So it seems fairly common I think it might be related to this change in Core #10944 |
if that ticket is it, looks like it's behind a feature flag. as a customer, can we opt out of that flag in the meantime? or is there a corepack version file we need to have in our repo to avoid this? |
Hi @broksonic21, Could I get the CC: @abdulapopoola , @carlincherry |
Hi @broksonic21 and @carlincherry, The change has been deployed. If the CC: @abdulapopoola |
.npmrc:
in package-lock:
|
This made it way worse unfortunately @kbukum1 One repo is getting lock file v1 locks now, with update PRs for packages that are already up to date other repos are just plain failing with no noticeable log on why, just say unknown error from dependabot. I’m happy to hop on a repro call tomorrow if it helps, or even later this evening. |
Hi @broksonic21 , I am unable to view the log because of permission. Will it be possible if you can share logs with me? URL: /~https://github.com/broksonic21/dependabot-engine/network/updates/920865298 |
Example pr after this change: /~https://github.com/broksonic21/repro-increase-dependabot-grouped/pull/5/files note: seeing this in every repo, work and personal, that’s on lock file 3. |
Hi @broksonic21 , I checked the last error and I saw the following error. Will it be possible to use rerun the process through Insight -> Dependency Graph -> Dependabot -> by clicking the button "Recent update jobs" for [package.json]?
|
Hi @broksonic21, It looks like the problem has been resolved. Please let me know if you encounter any further issues. If everything is working as expected, we’ll proceed to close this issue. |
I see the problem here. Checking this one. |
@kbukum1 I fixed that issue, but it is still broken. see broksonic21/dependabot-engine#17 it is reverting to lock file 1… and repos that doesn’t work on are just failing I can meet at 5:30 your time to show you if that works. |
Hi @broksonic21, The feature has been disabled, so it should now work as how it was working before. We are currently investigating the issue on our end to identify a solution. This behavior occurs when running npm |
@kbukum1 Thanks - I can confirm that fixed our work repos and my test repos - both the ones creating wrong PRs/reverting to lockfile 1, and the ones that wouldn’t run at all (likely because they used a feature that whatever npm/lockfile 1 version was running didn’t support) Thanks for diligence and quick follow up! I’ll leave my test repos up and happy to test again if you have a potential fix. |
Thank @broksonic21 for reporting that quickly and providing information. |
The change is completed and enabled for your test repositories.
Is it possible if you can check everything is ok? If everything is ok we plan to rollout the feature. |
@kbukum1 that looks right on my both my test repositories. Let me know after you rollout and we can confirm the work ones too. Else @carlincherry has my contact info if you want to reach out and test their first |
Thank you for quick check. I am going to rollout soon. I will let you know when the rollout is done. |
The rollout is done. Please let me know if you see any problem in your repositories. If there is no problem found we can close this issue. |
@kbukum1 work repositories are all working like I'd expect. I think you are good here, I'll let you know if anything changes |
Thanks for the support. |
For me it's saying it's using npm 9.6.5 (actual) and 10.8.2 at the same time 🤣
Context
|
Is there an existing issue for this?
Package ecosystem
npm
Package manager version
9
Language version
22
Manifest location and content before the Dependabot update
package.json:
with .npmrc:
see
/~https://github.com/broksonic21/dependabot-engine
dependabot.yml content
/~https://github.com/broksonic21/dependabot-engine/blob/main/.github/dependabot.yml
Updated dependency
No response
What you expected to see, versus what you actually saw
In the last day or so, this started happening from dependabot and fails to run at all - silently, unless you look at the dependabot UI. Note: this only happens with dependabot on github action runners - i wasn't able to repro this if I turn off github action runners.
See: /~https://github.com/broksonic21/dependabot-engine/network/updates/920658676
Note: the error message says NPM 10.8.2.
But the logs say you are running npm 8.19.4
We required npm 9 and above -> and all of a sudden all our dependabot runs fail to run.
Can this get fixed up (both the messaging, and the fact that you are running on an old version)
Logs show:
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
/~https://github.com/broksonic21/dependabot-engine/network/updates/920658676
Smallest manifest that reproduces the issue
No response
The text was updated successfully, but these errors were encountered: