Skip to content

Conversation

@kezhenxu94
Copy link
Member

No description provided.

@kezhenxu94 kezhenxu94 requested a review from wu-sheng March 14, 2024 03:09
@wu-sheng wu-sheng added this to the 10.0.0 milestone Mar 14, 2024
@wu-sheng wu-sheng added the docker Docker build scripts or images related label Mar 14, 2024
wu-sheng
wu-sheng previously approved these changes Mar 14, 2024
@wu-sheng
Copy link
Member

We could prepare the demo/showcase replacement, but it should be updated when BanyanDB 0.6 is ready.

To tear down the cluster, run the following command:

```shell
docker compose --project-name=skywalking-quickstart down
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One question, could you document the exposed ports for SkyWalking UI and BanyanDB UI?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At the end of the script, the following log will be printed so the users should be very clear which port to use, I can add BanyanDB UI port there if users choose BanyanDB as storage

# ...
SkyWalking is now running. You can send telemetry data at localhost:11800 and access the UI at http://localhost:8080

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good, we should expose BanyanDB UI as well, to query the data from BanyanDB from Browser.

@wu-sheng
Copy link
Member

Is the exposed port visible in the logs too?

@kezhenxu94
Copy link
Member Author

Is the exposed port visible in the logs too?

Added in the website repo apache/skywalking-website@f553257

@wu-sheng wu-sheng merged commit 52d5b25 into apache:master Mar 14, 2024
@kezhenxu94 kezhenxu94 deleted the quickstartdoc branch March 14, 2024 04:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Docker build scripts or images related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants