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]: Using config templates crashes garden dev #5290

Closed
eysi09 opened this issue Oct 23, 2023 · 0 comments · Fixed by #5329
Closed

0.13: [Bug]: Using config templates crashes garden dev #5290

eysi09 opened this issue Oct 23, 2023 · 0 comments · Fixed by #5329

Comments

@eysi09
Copy link
Collaborator

eysi09 commented Oct 23, 2023

Garden Bonsai (0.13) Bug

Current Behavior

I have a project which uses config templates. Every now and then the dev console (garden dev) will crash with the error:

Cannot read properties of undefined (reading 'inputsSchema')

At a glance I think this only happens after the Garden config has been modified (and Garden prints the Config change detected. Project will be reloaded when the next command is run.)

This usually happens when I run commands from within the console or on events triggered by the dashboard if that's open (some of these events trigger internal Garden command where the config resolution fails).

Expected behavior

It doesn't crash.

Reproducible example

I haven't been able to pinpoint exactly when this happens so kind of hoping someone can spot the issue without a solid repro. But it does strike me as some config resolution race condition since it doesn't fail consistently.

Your best bet to repro would be to:

  1. Run garden dev in e.g. this example which uses config templates.
  2. Open the web dashboard on the live page (if only to have it trigger internal Garden commands which may surface this)
  3. Tweak the actual Garden config (e.g. re-naming an action or adding a new one)
  4. Wait for the dashboard to request a new graph via the loadConfig ws message which is called when the config changes, or run some commands from within the console (e.g. validate)
  5. Observe as it crashes. Most likely. Sometimes.

Workaround

No known workaround, it basically makes the dev command un-usable for projects that use config templates.

Your environment

garden version Latest main as of writing.

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

Successfully merging a pull request may close this issue.

1 participant