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

Vxlan operator nodes map #687

Merged
merged 1 commit into from Jun 18, 2021
Merged

Conversation

alacuku
Copy link
Member

@alacuku alacuku commented Jun 17, 2021

Description

The vxlan operator populates a map for each node of the k8s cluster saving its IP address. The map is consumed by the symmetric-routing operator introduced in #686.

How Has This Been Tested?

Unit tests have been modified to take in account the modified code.

@adamjensenbot
Copy link
Collaborator

Hi @alacuku. Thanks for your PR!

I am @adamjensenbot.
You can interact with me issuing a slash command in the first line of a comment.
Currently, I understand the following commands:

  • /rebase: Rebase this PR onto the master branch
  • /merge: Merge this PR into the master branch
  • /ok-to-test Enable testing for PRs with base on forks

Make sure this PR appears in the liqo changelog, adding one of the following labels:

  • kind/breaking: 💥 Breaking Change
  • kind/feature: 🚀 New Feature
  • kind/bug: 🐛 Bug Fix
  • kind/cleanup: 🧹 Code Refactoring
  • kind/docs: 📝 Documentation

@alacuku
Copy link
Member Author

alacuku commented Jun 17, 2021

/rebase

@alacuku
Copy link
Member Author

alacuku commented Jun 18, 2021

/rebase

@alacuku
Copy link
Member Author

alacuku commented Jun 18, 2021

/merge

@adamjensenbot adamjensenbot added the merge-requested Request bot merging (automatically managed) label Jun 18, 2021
@adamjensenbot adamjensenbot merged commit 165892a into master Jun 18, 2021
@adamjensenbot adamjensenbot deleted the kcl/vxlan-operator-nodesMap branch June 18, 2021 18:15
@adamjensenbot adamjensenbot removed the merge-requested Request bot merging (automatically managed) label Jun 18, 2021
@alacuku alacuku mentioned this pull request Jun 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants