bithavoc opened a new pull request #9555:
URL: https://github.com/apache/pulsar/pull/9555
Fixes #4044, completes #290
### Motivation
When you google "docker compose pulsar", the first results points to
`docker-compose/standalone-dashboard/docker-compose.yml` which works for the
most part if you use CLI commands such as `pulsar-admin`, but the dashboard
never shows any tenant, topics, etc. This has been reported in issues such as
#4044 and officially given a recommendation in #290, however,
`docker-compose/standalone-dashboard/docker-compose.yml` is still outdated.
### Modifications
- Modified `docker-compose/standalone-dashboard/docker-compose.yml` to
include `--advertised-address` with the name of the network host inside the
docker-compose network that the dashboard can actually use.
### Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
This change added tests and can be verified as follows:
- without applying this patch, run `docker-compose up`, even after 2
minutes the dashboard still doesn't show tenants or topics
- apply the patch and then run `docker-compose up`, refresh the dashboard
UI after 20 seconds and the tenants and topics show up.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]