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

get connect bridge name from docker network name #1

Merged
merged 2 commits into from
Aug 31, 2020
Merged

Conversation

karimra
Copy link
Member

@karimra karimra commented Aug 31, 2020

This fixes the bridge name that corresponds to the docker network used for mgmt interface.
Go docker api uses bridge name br-$id, where $id is the 12 first characters from the docker network ID

if the docker network already exist, the modified function will get the network ID to build the bridge name.

@henderiw
Copy link
Contributor

approved

@henderiw henderiw merged commit 7efc786 into master Aug 31, 2020
@henderiw henderiw deleted the bridge-name branch August 31, 2020 05:17
hellt pushed a commit that referenced this pull request Oct 21, 2021
hellt pushed a commit that referenced this pull request Apr 8, 2022
hellt added a commit that referenced this pull request Nov 2, 2022
use mgmt server readiness file instead of commit #1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants