Skip to content

Conversation

@sebyx07
Copy link
Contributor

@sebyx07 sebyx07 commented Apr 4, 2025

Changes

  • allow --hostname=my-server to be passed

Issues

#5550

@sebyx07 sebyx07 force-pushed the add-hostname branch 2 times, most recently from 316d3b3 to f066607 Compare April 4, 2025 17:38
@sebyx07 sebyx07 changed the title Add hostname Add --hostname=myhost # to docker Apr 4, 2025
@sebyx07 sebyx07 changed the title Add --hostname=myhost # to docker Improve --hostname=myhost to docker custom image Apr 4, 2025
@sebyx07
Copy link
Contributor Author

sebyx07 commented Apr 4, 2025

this helps me with sidekiq(to give a better name to the workers) and sentry. and I think will help others too

@andrasbacsai
Copy link
Member

Thank you for the PR! I refactored a bit and now it works as it should.

@andrasbacsai andrasbacsai merged commit d6d76ad into coollabsio:next Apr 23, 2025
1 check passed
@jonathan-reisdorf
Copy link
Contributor

jonathan-reisdorf commented Apr 30, 2025

Thank you! Would there also be an option to do something like this:

--hostname=$(hostname)

for a multi-server setup? Currently, the hostname in the container is then a literal $(hostname). I can circumvent the issue though to a certain extent by mounting /etc/hostname:/etc/hostname:ro, which will not fully work as iirc it also needs to be passed to the kernel? but I can then inside the container at least access the hostname with cat /etc/hostname, export it to an environment var etc., so all good if it doesn't work :) Thanks!

@sebyx07 sebyx07 deleted the add-hostname branch May 16, 2025 15:05
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants