[flagd-ui] Spans with error

[jack5341]

[flagd-ui] Spans with error

Changes

Switch flagd-ui navbar links from plain anchors to LiveView navigation (.link navigate) to avoid full page reloads and websocket teardown when switching between Basic and Advanced.

Fix LiveSocket URL construction under the /feature base path so the websocket connects to /feature/live (was /featurelive), preventing spurious disconnects.

Merge Requirements

For new features contributions, please make sure you have completed the following
essential items:

• CHANGELOG.md updated to document new feature additions
• Appropriate documentation updates in the docs
• Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: julianocosta89 / name: Juliano Costa (34f66ab)

[julianocosta89]

@jack5341 thx for that!

I still see an error when accessing the flagd-ui service though:

This error happens every time I open /feature.

[jack5341]

Did you guys faced something like that before in this repository?

Not only the CSS but also the JS files are unreachable, so I can’t make any function calls. Even when I switch to another branch, I still encounter the same problem. Yesterday, everything was working fine, this issue started today.

Additionaly I use Orbstack for docker.

My console

GET http://localhost:32935/feature/assets/css/app-6f5d86242cf5220b8531adc7351da8bc.css?vsn=d net::ERR_ABORTED 404 (Not Found)Understand this error
(index):10  

GET http://localhost:32935/feature/assets/js/app-fb088dfe3c12b4ebb739348d1a2a3a57.js?vsn=d net::ERR_ABORTED 404 (Not Found)

[julianocosta89]

Did you guys faced something like that before in this repository?

Not really, and today I was able to run your branch fine.
I had to access the /feature to get the traces with error, and everything worked fine

[jack5341]

Did you guys faced something like that before in this repository?

Not really, and today I was able to run your branch fine. I had to access the /feature to get the traces with error, and everything worked fine

you have any idea what can cause this problem?

[jack5341]

To help other users see that issue, make sure the project is always initiated with make start command. I am on my way again.

[jack5341]

Hey @julianocosta89, I’ve fixed it. The issue was caused by the protocol switch during WebSocket connections. HTTP 101 responses were being treated as errors, which is actually expected behavior for WebSockets. The simple and reliable fix was to add a filter to the otel-collector processor to set http.status_code 101 entries to UNSET.

[jack5341]

update

[julianocosta89]

hey @jack5341 now I'm not sure if we should fix this on the demo or if we should open an issue on the instrumentation repo.
If 101 shouldn't be an error, the instrumentation is wrong and it should be fixed.

WDYT?!

[jack5341]

I’d suggest fixing this in the instrumentation repo as well as here, since we don’t know when it will be released and when it might affect this repository.

[julianocosta89]

In that case we should do the following:

• Open an issue in the Elixir instrumentation repo
• Add a comment with the link to the issue in the Collector rule here
• Limit the scope of the rule to only Flagd-UI spans (at the moment the rule is configured to update ALL spans with status code 101)

[jack5341]

In that case we should do the following:

• Open an issue in the Elixir instrumentation repo
• Add a comment with the link to the issue in the Collector rule here
• Limit the scope of the rule to only Flagd-UI spans (at the moment the rule is configured to update ALL spans with status code 101)

1- Is this repository the correct one for Elixir?
2- Do you mean I should add a comment to the changes here and include a link to the issue created in the Elixir repository?
3- Why not keep it global? I mean, 101 shouldn’t really be treated as an error, am I wrong?

[julianocosta89]

1- Is this repository the correct one for Elixir?

yes

2- Do you mean I should add a comment to the changes here and include a link to the issue created in the Elixir repository?

Yes, same as we have done here: https://github.com/open-telemetry/opentelemetry-demo/blob/main/src/otel-collector/otelcol-config.yml#L150

3- Why not keep it global? I mean, 101 shouldn’t really be treated as an error, am I wrong?

Good question. I wonder if other instrumentations set the span status to error.
If so then this is a specification problem.

I can take a further look next week on that.