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

[RA2 Ch4]: Removal of VMware NCP #1379

Merged
merged 2 commits into from Apr 6, 2020

Conversation

CsatariGergely
Copy link
Collaborator

Fixes #1378

Fixes anuket-project#1378

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
@CsatariGergely CsatariGergely added this to In Progress in old-RA2 via automation Mar 26, 2020
Copy link
Collaborator

@tomkivlin tomkivlin left a comment

Choose a reason for hiding this comment

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

I don't think the whole merge needs reversing. I am ok with NCP being listed, but we do need to highlight that it is not open source, perhaps a reference to the CNTT principles in the comparison table?

@ramkri123
Copy link
Collaborator

Agree with Tom's observation.

There is no mandate for the CNI plugin to be open source as long as the plugin adheres to the CNI specification -- https://github.com/containernetworking/cni. Other examples of non-open-source plugins are ACI, Nuage etc. -- they are described in https://kubernetes.io/docs/concepts/cluster-administration/addons/.

@tomkivlin
Copy link
Collaborator

Agree with Tom's observation.

There is no mandate for the CNI plugin to be open source as long as the plugin adheres to the CNI specification -- https://github.com/containernetworking/cni. Other examples of non-open-source plugins are ACI, Nuage etc. -- they are described in https://kubernetes.io/docs/concepts/cluster-administration/addons/.

I think the point is that there is a CNTT principle that we only used open source technologies. However, it is a principle, so I think worth adding but highlighting.

@ramkri123
Copy link
Collaborator

Hi Tom,

Would the following changes help clarify.

  1. In the NCP text
    Note that NCP is not a open source CNI plugin, but adheres to the CNI specification -- https://github.com/containernetworking/cni. Other examples of non-open-source CNI plugins are ACI, Nuage etc. -- they are described in https://kubernetes.io/docs/concepts/cluster-administration/addons/.

  2. In the NCP column in the table
    Not open source

Thanks,
Ramki

@CsatariGergely
Copy link
Collaborator Author

I don't think the whole merge needs reversing. I am ok with NCP being listed, but we do need to highlight that it is not open source, perhaps a reference to the CNTT principles in the comparison table?

  1. [RA2 Ch4] updates to include NSX Container plugin(NCP) #1370 was merged before reaching a consensus.
  2. I think having a principle to use only open source technologies would mean that we use only open source technologies.
  3. If I understand correctly this CNI assumes that the Kubernetes cluster runs on top of VM-s in a VMware vCloud environment.
  4. I'm still not convinced that this is a metaplugin

@CsatariGergely CsatariGergely added the question Further information is requested label Apr 2, 2020
@tomkivlin
Copy link
Collaborator

Meeting 2nd April:

  • Tom agreed with points 1/2 above
  • Principle compliance should be made, unless there isn't an option that complies with all principles
  • Do we need a discussion about "open source" - what it means - is an open source software owned by one company with contributions only from that company any better than proprietary? Yes - but not in this PR :) @tomkivlin to raise an issue.

@CsatariGergely CsatariGergely removed the question Further information is requested label Apr 3, 2020
Copy link
Collaborator

@xmulligan xmulligan left a comment

Choose a reason for hiding this comment

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

another is one word, otherwise lgtm

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
@CsatariGergely
Copy link
Collaborator Author

another is one word, otherwise lgtm

Done

@rabi-abdel rabi-abdel merged commit fbe5271 into anuket-project:master Apr 6, 2020
old-RA2 automation moved this from In Progress to Done Apr 6, 2020
@rabi-abdel rabi-abdel added the Archive Archive Item label May 15, 2020
@project-bot project-bot bot moved this from Done to Archive in old-RA2 May 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Archive Archive Item
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[RA2 Ch4]: Remove not open source CNI plugin
5 participants