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

Add sys.hostname configuration for supervisor #7741

Open
jsirex opened this issue Jun 11, 2020 · 5 comments
Open

Add sys.hostname configuration for supervisor #7741

jsirex opened this issue Jun 11, 2020 · 5 comments
Labels
Team:Habitat All issues tied to Chef habitat team Type: Bug Issues that describe broken functionality

Comments

@jsirex
Copy link
Contributor

jsirex commented Jun 11, 2020

Like for sys.ip it will be nice to have possibility to advertise servers' hostname.

Usually I have "internal name" for server, like internal-server-01.local.domain.
However, I can assign additional DNS(-es) to it, like public.name.official.domain.

I want to use binding and service discovery. But if I use TLS/SSL encryption I must use "correct" name. But bind.backend.first.sys.hostname will resolve to the local domain name, so I will end up with "common name mismatch".

Current workaround is to hardcode connection names.

@christophermaier
Copy link
Contributor

The hostname would end up being something like internal-server-01 or public, given your examples above.

From your description, it sounds like having access to a particular fully-qualified domain name might work better for you (e.g., adding an option like hab sup run --fqdn=public.name.official.domain)... is that accurate?

@jsirex
Copy link
Contributor Author

jsirex commented Jul 7, 2020

Yep. fqdn options is good here.

I'm also thinking about an general approach to catch all future use-cases if any..
For example, introduce general options for the sys data.
Run with: hab sup run --system-info fqdn=f.q.d.n --system-info foo-bar=baz --system-info tag=az1
Use with template:

tag = {{bind.backend.sys.tag }}
conn = {{bind.backend.sys.fqdn}}
text = {{bind.backend.sys.foo-bar}}

@christophermaier
Copy link
Contributor

We can likely add fqdn without much issue.

More general metadata would be interesting, and there is some prior art in the event stream feature, but it might take a bit of thinking to sort out the best way forward. Would you mind creating a separate issue for just that?

Thanks! 🙇

@christophermaier christophermaier added Type: Bug Issues that describe broken functionality and removed C-bug labels Jul 24, 2020
@christophermaier christophermaier removed their assignment Jan 11, 2021
@rahulgoel1 rahulgoel1 added the Team:Habitat All issues tied to Chef habitat team label Jul 30, 2021
@stale
Copy link

stale bot commented Jul 31, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We value your input and contribution. Please leave a comment if this issue still affects you.

1 similar comment
@stale
Copy link

stale bot commented Aug 12, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We value your input and contribution. Please leave a comment if this issue still affects you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Habitat All issues tied to Chef habitat team Type: Bug Issues that describe broken functionality
Projects
None yet
Development

No branches or pull requests

3 participants