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

Default port 10000 overlaps with juju-backup-all #133

Closed
przemeklal opened this issue Dec 11, 2023 · 3 comments · Fixed by #153
Closed

Default port 10000 overlaps with juju-backup-all #133

przemeklal opened this issue Dec 11, 2023 · 3 comments · Fixed by #153
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@przemeklal
Copy link
Member

By default hardware-observer uses port 10000 which overlaps with another Solutions Engineering project https://code.launchpad.net/juju-backup-all. In our deployments, juju-backup-all usually runs on physical machines so almost every time we run into this conflict.

I'm aware of juju config hardware-observer exporter-port.

Can the defaults be updated so there's no overlap between these two projects?

@przemeklal przemeklal added the enhancement New feature or request label Dec 11, 2023
@przemeklal przemeklal changed the title Port 1000 overlaps with juju-backup-all Default port 10000 overlaps with juju-backup-all Dec 11, 2023
@Pjack Pjack added this to the 23.10.3 milestone Dec 26, 2023
@dashmage dashmage self-assigned this Jan 24, 2024
@dashmage
Copy link
Contributor

This seems simple enough to fix. Is there any preference or can I just pick a random default port which isn't 10000 (say 9000)?

@Pjack
Copy link

Pjack commented Jan 24, 2024

Maybe we can try to allocate a port here.
https://github.com/prometheus/prometheus/wiki/Default-port-allocations

@dashmage
Copy link
Contributor

Maybe we can try to allocate a port here. https://github.com/prometheus/prometheus/wiki/Default-port-allocations

10200 looks good, what do you say? Doesn't seem to be taken up by any other default port allocations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants