- Notifications
You must be signed in to change notification settings - Fork928
fix: Use explicit resource order when assocating agents#2219
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
d94e224 to1033852CompareThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Tests lgtm
This cleans up agent association code to explicitly map a singleagent to a single resource. This willfix#1884, and unblocka prospect from beginning a POC.
| var findTerraformResources func(mod *tfjson.StateModule) | ||
| findTerraformResources = func(mod *tfjson.StateModule) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
| varfindTerraformResourcesfunc(mod*tfjson.StateModule) | |
| findTerraformResources=func(mod*tfjson.StateModule) { | |
| findTerraformResources:=func(mod*tfjson.StateModule) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Can't do that because it calls itself ;p
This cleans up agent association code to explicitly map a singleagent to a single resource. This willfix#1884, and unblocka prospect from beginning a POC.
This cleans up agent association code to explicitly map a single
agent to a single resource. This willfix#1884, and unblock
a prospect from beginning a POC.