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

fix(machine): Check bridges have ip address #833

Merged

Conversation

craciunoiuc
Copy link
Member

@craciunoiuc craciunoiuc commented Sep 20, 2023

Prerequisite checklist

  • Read the contribution guidelines regarding submitting new changes to the project;
  • Tested your changes against relevant architectures and platforms;
  • Ran make fmt on your commit series before opening this PR;
  • Updated relevant documentation.

Description of changes

GitHub-Fixes: #828

Signed-off-by: Cezar Craciunoiu <cezar.craciunoiu@unikraft.io>
@craciunoiuc craciunoiuc added kind/fix This PR fixes an issue or bug area/machine Issue or PR is related to KraftKit's machine instance subsystem labels Sep 20, 2023
@jake-ciolek
Copy link
Contributor

Thanks!

@jake-ciolek jake-ciolek merged commit 5aeaca3 into unikraft:staging Sep 20, 2023
4 checks passed
@craciunoiuc craciunoiuc deleted the craciunoiuc/check-ip-exists branch October 17, 2023 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/machine Issue or PR is related to KraftKit's machine instance subsystem kind/fix This PR fixes an issue or bug
Projects
Status: 🚀 Done
Development

Successfully merging this pull request may close these issues.

kraft run --network bridge panics with a bound check error when no interface present
2 participants