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
Network Api enhancement and refactoring #154
Conversation
ad72143
to
3c83fda
Compare
/rebase |
Rebase status: success! |
3c83fda
to
5937986
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @alacuku, thanks for this compelling PRs. I suggested some very small changes and additions. I would add a further point: it would be nice to change the version of Liqo API to something less stable than v1, which says that those APIs will not change. Something like a v1alpha1, v1beta1 or similar would be more appropriate. Since you are introducing new APIs in this PR, I was wondering if this can be handled here at least for network configuration.
6f55bf7
to
0d2bfb9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
/rebase |
Refactoring of the network api
Rebase status: success! |
0d2bfb9
to
62dac68
Compare
Description
Adding the
networkconfig.liqonet.liqo.io
CRD to the network api. This CRD is contains all the network parameters to be shared with a possible foreign cluster. It is used to derive thetunnelendpoint.liqonet.liqo.io
CRD which models a VPN tunnel between two clusters.Some refactoring on the network API has been done
This PR is linked to #151