Mismatch of tenantId between demo channel and arc view

[hsudbrock]

Currently, chatting with the demo agents using the arc view leads to errors, because the demo channel uses tenantId "acme", while the default system context of the installed arc view uses tenantId "de" - and, hence, the channel cannot be resolved by the lmos-runtime:

[hsudbrock]

I wonder how to fix this.

One option is to adapt the channel configuration from the lmos-demo to the default tenantId from the arc view. That would be a simple change, but sounds somehow wrong (next time someone changes the default the default tenantId from the arc view, the demo would be broken again).

The "nicer" solution, IMHO, would be to configure the default tenantId used by the arc view via configuration in the helm chart used to install the arc view. From what I see that is currently not possible. I was looking where this could be changed, but failed finding the place where the arc view helm chart is even built and published (the lmos-demo still uses a helm chart from the "old" lmos-ai group in github).

@patwlan Do you have a hint/ pointer for me where I could adapt the arc view helm chart (for making the default system context configurable)?

[patwlan]

The arc view does not usually have a default tenant id. we should look into updating the arc view we use for the demo.

[hsudbrock]

I was wondering where we actually publish an image and a helm chart for the arc view; I could not find anything in https://github.com/eclipse-lmos/arc-view - do I miss something, or is this simply not there yet?

[kaikreuzer]

@hsudbrock See eclipse-lmos/arc-view#16. We should actually add this issue to the arc 0.1.0 release project...

[kaikreuzer]

The "nicer" solution, IMHO, would be to configure the default tenantId used by the arc view via configuration in the helm chart used to install the arc view

This would have also been my wish here. The demo channels are on purpose configured as tenant "acme" and the "de" stuff that is coded in arc-view is still a left-over from the original DT code.

[hsudbrock]

OK, so let's change the arc-demo first to provide a helm chart (where we can, in particular, configure the tenantId somehow), and then get back to address this issue here afterwards.