-
Notifications
You must be signed in to change notification settings - Fork 66
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
[#487][#488] Fix cloud2edge post-install hook, fix tour instructions. #489
Conversation
Kafka usage is now the default. Signed-off-by: Carsten Lohmann <[email protected]>
Signed-off-by: Carsten Lohmann <[email protected]>
@@ -63,27 +66,27 @@ The demo device's digital twin supports a temperature property which will be set | |||
by means of the following command: | |||
|
|||
{% clipboard %} | |||
curl -i -u [email protected]:demo-secret -H 'application/json' --data-binary '{ | |||
curl -i -k -u [email protected]:demo-secret -H 'application/json' --data-binary '{ |
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.
FYI: I've added the -k
(i.e. --insecure
) option here for now. Better would be to fetch the used CA cert beforehand (via kubectl get secret
) and use that here instead. But it seems the CA cert is currently only set in a K8s secret if Kafka messaging is being used. That means a generic solution requires changes also to the Hono chart.
0fd241a
to
b0702a2
Compare
Using new REGISTRY_BASE_URL and HTTP_ADAPTER_BASE_URL environment variables that will contain either the secure or insecure endpoint base URL depending on the chart configuration. Signed-off-by: Carsten Lohmann <[email protected]>
Signed-off-by: Carsten Lohmann <[email protected]>
Signed-off-by: Carsten Lohmann <[email protected]>
b0702a2
to
3cbabd0
Compare
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.
LGTM
Includes these fixes:
Using new REGISTRY_BASE_URL and HTTP_ADAPTER_BASE_URL environment variables that will contain either the secure or insecure endpoint base URL depending on the chart configuration.
Kafka usage is now the default.