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

Documentation error on hello-world example #10

Open
pmjordan opened this issue Aug 6, 2021 · 2 comments
Open

Documentation error on hello-world example #10

pmjordan opened this issue Aug 6, 2021 · 2 comments

Comments

@pmjordan
Copy link
Contributor

pmjordan commented Aug 6, 2021

If the hello-world example is deployed before minikube tunnel is running then the turandot service list shows url: http://0.0.0.0:80 not url: http://'<'unknown'>':80 as it did previously.
At least in my environment the output was not updated once minikube tunnel was started. The log messages were:

ERROR [turandot-operator.processor.services] requeuing unfinished work item (36 time) workspace/hello-world: GoError: topology_template.outputs["url"].value: call to tosca.function.get_attribute("hello-world","web","status","loadBalancer","ingress",0,"ip") failed because nested capability "web" attribute "status.loadBalancer.ingress" not found in "hello-world" at internal:tosca.lib.traversal:145:15(58)

INFO [turandot-operator.controller] Event(v1.ObjectReference{Kind:"Service", Namespace:"workspace", Name:"hello-world", UID:"ee674c74-e859-4fb7-996a-720084ad4cca", APIVersion:"turandot.puccini.cloud/v1alpha1", ResourceVersion:"13405", FieldPath:""}): type: 'Warning' reason: 'CloutUpdateError' Service Clout update error: GoError: topology_template.outputs["url"].value: call to tosca.function.get_attribute("hello-world","web","status","loadBalancer","ingress",0,"ip") failed because nested capability "web" attribute "status.loadBalancer.ingress" not found in "hello-world" at internal:tosca.lib.traversal:145:15(58)

From my viewpoint I would be happy to fix this in the documentation by starting the tunnel before the turandot service.

@tliron
Copy link
Owner

tliron commented Aug 6, 2021

Thanks! Yes, this is indeed a change due to the attribute now having constraints (it must be an actual IP address, not an arbitrary string). No worries, I will fix this quickly on my end. Thanks for reporting.

@pmjordan
Copy link
Contributor Author

pmjordan commented Aug 9, 2021

You might also want to start the tunnel in lab/start so that it is ready for lab/test

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

No branches or pull requests

2 participants