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

Move get_machine_name to a more public looking place #7888

Closed
berland opened this issue May 14, 2024 · 0 comments · Fixed by #7965
Closed

Move get_machine_name to a more public looking place #7888

berland opened this issue May 14, 2024 · 0 comments · Fixed by #7965
Assignees
Labels
maintenance Not a bug now but could be one day, repaying technical debt

Comments

@berland
Copy link
Contributor

berland commented May 14, 2024

This function is imported by ert-configurations and should thus not be hidden in the config of the ensemble_evaluator. A more appropriate place is perhaps ert.shared.

def get_machine_name() -> str:
"""Returns a name that can be used to identify this machine in a network
A fully qualified domain name is returned if available. Otherwise returns
the string `localhost`
"""
hostname = socket.gethostname()
try:
# We need the ip-address to perform a reverse lookup to deal with

@berland berland added the maintenance Not a bug now but could be one day, repaying technical debt label May 14, 2024
@berland berland self-assigned this May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Not a bug now but could be one day, repaying technical debt
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant