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

[Bug] skywalking-showcase docker deployment has some incorrect parts #12077

Closed
2 of 3 tasks
yswdqz opened this issue Apr 8, 2024 · 7 comments
Closed
2 of 3 tasks

[Bug] skywalking-showcase docker deployment has some incorrect parts #12077

yswdqz opened this issue Apr 8, 2024 · 7 comments
Assignees
Labels
bug Something isn't working and you are sure it's a bug! showcase

Comments

@yswdqz
Copy link
Member

yswdqz commented Apr 8, 2024

Search before asking

  • I had searched in the issues and found no similar issues.

Apache SkyWalking Component

Showcase (apache/skywalking-showcase)

What happened

  1. rocketmq-feature seems incorrect:
image I believe that a feature should utilize a common OAP(such as single-node) instead of spinning up a new one. Additionally, there are issues with the configuration of this OAP.
  1. whether we need to expose this port?
image
  1. oap dependency
image Some feature depend on oap. However, in some cases, we only need these services and do not require the built-in OAP.

and also, some config files use oap as host:
image
so we can't easily edit Makefile to change oap address.
image

Are you willing to submit a pull request to fix on your own?

  • Yes I am willing to submit a pull request on my own!

Code of Conduct

@yswdqz yswdqz added bug Something isn't working and you are sure it's a bug! showcase labels Apr 8, 2024
@wu-sheng
Copy link
Member

wu-sheng commented Apr 8, 2024

@kezhenxu94 Could you guide @yswdqz about this? We may need to polish showcase.

@kezhenxu94
Copy link
Member

  1. rocketmq-feature seems incorrect:

image I believe that a feature should utilize a common OAP(such as single-node) instead of spinning up a new one. Additionally, there are issues with the configuration of this OAP.

Nice catch! I think we can fix this by simply removing the OAP section in this docker compose file.

  1. whether we need to expose this port?
image

This is not mandatory as we have a loadgen to trigger traffic, however, if users want to manually trigger traffic, they can open this port in their browser.

  1. oap dependency

image Some feature depend on oap. However, in some cases, we only need these services and do not require the built-in OAP.

Can you elaborate? OAP is required to be up and running for those components that need to talk to OAP, in your screenshot, the vm exporter needs to send data to OAP so it needs to wait for OAP's readiness.

and also, some config files use oap as host: image so we can't easily edit Makefile to change oap address. image

Good catch, this can be replace with the env var BACKEND_SERVICE.

Will you file a pull request @yswdqz ?

@yswdqz
Copy link
Member Author

yswdqz commented Apr 8, 2024

Yes, but I might not have much time recently. I'll submit the PR over the weekend.

@yswdqz
Copy link
Member Author

yswdqz commented Apr 8, 2024

Can you elaborate? OAP is required to be up and running for those components that need to talk to OAP, in your
screenshot, the vm exporter needs to send data to OAP so it needs to wait for OAP's readiness

Maybe we just need VM-monitor feature, and I just want run with "make deploy.docker FEATURE_FLAGS=vm"(not FEATURE_FLAGS=vm,single-node)

@kezhenxu94
Copy link
Member

Can you elaborate? OAP is required to be up and running for those components that need to talk to OAP, in your
screenshot, the vm exporter needs to send data to OAP so it needs to wait for OAP's readiness

Maybe we just need VM-monitor feature, and I just want run with "make deploy.docker FEATURE_FLAGS=vm"(not FEATURE_FLAGS=vm,single-node)

This doesn't make sense as this is a "skywalking" showcase, we don't bother to make it work without skywalking

@yswdqz
Copy link
Member Author

yswdqz commented Apr 9, 2024

This doesn't make sense as this is a "skywalking" showcase, we don't bother to make it work without skywalking

Sorry, my expression was not very clear.
My meaning is, some people (including myself) may want to import data into an external OAP, rather than using a single-node.

@wu-sheng
Copy link
Member

wu-sheng commented Apr 9, 2024

This doesn't make sense as this is a "skywalking" showcase, we don't bother to make it work without skywalking

Sorry, my expression was not very clear.

My meaning is, some people (including myself) may want to import data into an external OAP, rather than using a single-node.

That is not showcase. We don't need to consider that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working and you are sure it's a bug! showcase
Projects
None yet
Development

No branches or pull requests

3 participants