Skip to content

feat: use unix domain socket to interact with runner#4

Open
kyujin-cho wants to merge 3 commits intomasterfrom
feature/unix-domain-socket
Open

feat: use unix domain socket to interact with runner#4
kyujin-cho wants to merge 3 commits intomasterfrom
feature/unix-domain-socket

Conversation

@kyujin-cho
Copy link
Copy Markdown
Member

@kyujin-cho kyujin-cho commented Oct 28, 2022

Updating libbaihook to this version will break compatibility with Backend.AI krunner prior to 22.09.5.

@kyujin-cho kyujin-cho self-assigned this Oct 28, 2022
@kyujin-cho kyujin-cho requested a review from achimnol October 28, 2022 07:12
@kyujin-cho kyujin-cho changed the title feat: use unix domain socket to interact with runenr feat: use unix domain socket to interact with runner Oct 28, 2022
Copy link
Copy Markdown
Member

@achimnol achimnol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For compatibility, let's first check the existence/file-type of the input_sock and fallback to the prior implementation if it does not exist.

@kyujin-cho kyujin-cho requested a review from achimnol November 2, 2022 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants