wu-sheng commented on PR #83:
URL:
https://github.com/apache/skywalking-showcase/pull/83#issuecomment-1272331054
> > What I am asking is, why don't activate tracing in `frontend("APISIX
with UI (React)")`? I think we should be able to activate tracing to match the
latest graph
> > <img alt="image" width="1101"
src="https://user-images.githubusercontent.com/5441976/194709998-bf540659-ab82-442a-940b-cbd2cd3efdb3.png";>
> > We should visualize the topology and trace as same as this graph shows,
otherwise, this is showing we are missing something.
>
> We can add traces for APISIX in another PR, if you read carefully, the
current graph is
>
>
> If we want to add traces for APISIX, the graph should be something like
APISIX with UI actually means APISIX as UI server container. In the real
world, APISIX doesn't run the UI codes, it just responds the HTML/JS to the
browser(simulator).
When a browser agent traces a HTTP request, it must lands on the APISIX and
forward, unless this graph is not correct.(UI request to `app server` directly.
> If you agree , Let's keep eyes on apisix metrics
My point is whether our topology graph matches the docs, whether trace
APISIX or not, is not the key.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
![https://user-images.githubusercontent.com/5441976/194709998-bf540659-ab82-442a-940b-cbd2cd3efdb3.png"](https://user-images.githubusercontent.com/5441976/194709998-bf540659-ab82-442a-940b-cbd2cd3efdb3.png"){kind=link}