You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
defget_service_info(self, service, port):
""" Returns the host and the port for one of the services. Parameters ---------- service: str Name of the docker compose service port: int The internal port to get the host for Returns ------- (str, str): The hostname for the service, The port for the service """port_cmd=self.docker_compose_command() + ["port", service, str(port)]
output=subprocess.check_output(port_cmd, cwd=self.filepath).decode("utf-8")
result=str(output).rstrip().split(":")
iflen(result) !=2ornotall(result):
raiseNoSuchPortExposed(f"port {port} is not exposed for service {service}")
returnresult[0], result[1]
# old approachhost=compose.get_service_host("hub", 4444)
port=compose.get_service_port("hub", 4444)
# new approachhost, port=compose.get_service_info("hub", 4444)
Activity
tillahoffmann commentedon Aug 28, 2022
Sounds good. Fancy sending a PR?
csikb commentedon Aug 28, 2022
Sure, I alredy did the coding part, let me look trough it if everything is OK with it.
tillahoffmann commentedon Aug 28, 2022
Should just require removing the underscore in
compose.py, right?csikb commentedon Aug 28, 2022
And adding a test case
csikb commentedon Aug 28, 2022
#240
totallyzen commentedon Mar 19, 2024
this is now obsolete! see the new docker compose v2 implementation!