-
Notifications
You must be signed in to change notification settings - Fork 33
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
[BUG] Deadloop for duplicated configuration #478
Comments
I think there's two ways to solve this issue, one of which is really easy to just put in place (which is why I marked this as First off, I think it would be a good idea to add an additional manifest validation function that ensures that configuration names don't conflict from within a single manifest. Including this in the validation code makes sure that an application with duplicated config names would be denied at Second, I think it would be good to enhance the These could be done at the same time, or in one PR, and happy to advise for how to get this done! Help appreciated on knocking out the first bit |
…loud#478) Signed-off-by: Florian Fürstenberg <[email protected]>
…asmCloud#478) Signed-off-by: Florian Fürstenberg <[email protected]>
…loud#478) Signed-off-by: Florian Fürstenberg <[email protected]>
…loud#478) Signed-off-by: Florian Fürstenberg <[email protected]>
…asmCloud#478) Signed-off-by: Florian Fürstenberg <[email protected]>
…link config names (wasmCloud#478) Signed-off-by: Florian Fürstenberg <[email protected]>
Signed-off-by: Florian Fürstenberg <[email protected]>
…478) Signed-off-by: Florian Fürstenberg <[email protected]>
…link config names (#478) Signed-off-by: Florian Fürstenberg <[email protected]>
Affected project(s)
Describe the bug
(I'm not sure if it's the problem with wadm or wasmcloud host but) if there are two configurations sharing the same name for the same link, there will be a dead loop.
Steps to reproduce
Define an application where a link has a configuration with the same name
Expected behavior
I would expect to see some error / warning, instead of logging insanely
putting configuration
etc.Environment
Screenshots / Logs / Additional context
The text was updated successfully, but these errors were encountered: