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

0.13: [Bug]: stack graph resolution performance degradation #4763

Closed
hnicke opened this issue Jul 6, 2023 · 4 comments · Fixed by #4823
Closed

0.13: [Bug]: stack graph resolution performance degradation #4763

hnicke opened this issue Jul 6, 2023 · 4 comments · Fixed by #4823
Assignees
Labels
0.13 bug community performance regression A return to a previous and less advanced or worse state

Comments

@hnicke
Copy link
Contributor

hnicke commented Jul 6, 2023

Bug

Current Behavior

I'm about to upgrade our stack to bonsai.
We are seeing a drastic performance degradation in the module / action resolution step.

On v12, garden validate takes ~6 seconds.
On v13, garden validate takes ~51 seconds.

At first I was suspecting that the module-to-action conversion takes so much longer, but according to the logs this conversion only takes ~3 seconds.
However, it takes about ~44 seconds to perform the module scanning.
Please do note that the number of Found xx files in the logs are the same in v12 and v13.

PS: After further debugging I found out that the exclude action / module configuration is extremely slow in v0.13.
After migrating to .gardenignore files the performance issue is gone.

Expected behavior

I'd expect garden validate to take <10 seconds.

Workaround

Use .gardenignore files instead of exclude in action / module config.

Additional context

FYI, most of our modules reside in remote sources.

Your environment

  • OS: arch linux
  • How I'm running Kubernetes: GKE

garden version
0.12.61
0.13.6

@edvald
Copy link
Collaborator

edvald commented Jul 6, 2023

You're definitely running into some sort of a performance bug here. Typically it's quite the opposite when moving over. It might be best if we troubleshoot this over in our discord where you could perhaps send detailed logs with timestamps. Would you mind posting there? Someone on our team (even myself) will then help out. Whatever we find we'll then post back here for posterity.

@worldofgeese worldofgeese added bug performance regression A return to a previous and less advanced or worse state 0.13 community labels Jul 6, 2023
@worldofgeese worldofgeese moved this to Candidate in Core Weekly Jul 6, 2023
@hnicke
Copy link
Contributor Author

hnicke commented Jul 6, 2023

Great, thank you! discord thread: https://discord.com/channels/817392104711651328/1126473787693604884

@hnicke
Copy link
Contributor Author

hnicke commented Jul 10, 2023

After further debugging I found out that the exclude action / module configuration is extremely slow in v0.13.
After migrating to .gardenignore files the performance issue is gone.

@vvagaytsev
Copy link
Collaborator

@hnicke the fix has been released in 0.13.9, please try it out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.13 bug community performance regression A return to a previous and less advanced or worse state
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

5 participants