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

Add CONFIG_IP_SET for CNI #972

Merged
merged 1 commit into from
Jan 8, 2017
Merged

Add CONFIG_IP_SET for CNI #972

merged 1 commit into from
Jan 8, 2017

Conversation

r2d4
Copy link
Contributor

@r2d4 r2d4 commented Jan 6, 2017

Calico and other CNI plugins need the ipset kernel syscall enabled with CONFIG_IP_SET

From some discussions on slack, it looks like calico needs ipset enabled on the VM

2017-01-06T23:29:12.412819915Z 2017-01-06 23:29:12,409 [ERROR][15636/7] calico.felix.ipsets 467: Failed to check if ipset exists
2017-01-06T23:29:12.412823678Z Traceback (most recent call last):
2017-01-06T23:29:12.412826318Z   File "site-packages/calico/felix/ipsets.py", line 461, in exists
2017-01-06T23:29:12.412839121Z   File "site-packages/calico/felix/futils.py", line 355, in check_call
2017-01-06T23:29:12.412841739Z FailedSystemCall: Failed system call (retcode : 1, args : ('ipset', 'list', 'felix-all-ipam-pools-tmp'))
2017-01-06T23:29:12.412885664Z   stdout  : 
2017-01-06T23:29:12.412889933Z   stderr  : ipset v6.29: Kernel error received: Invalid argument
2017-01-06T23:29:12.412892454Z 
2017-01-06T23:29:12.412894776Z   input  : None

I've uploaded a new ISO at https://storage.googleapis.com/minikube/iso/minikube-v1.0.2.iso

Calico and other CNI plugins need the ipset kernel syscall enabled
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jan 6, 2017
@r2d4
Copy link
Contributor Author

r2d4 commented Jan 6, 2017

Should fix #973

@codecov-io
Copy link

codecov-io commented Jan 7, 2017

Current coverage is 34.81% (diff: 100%)

Merging #972 into master will not change coverage

@@             master       #972   diff @@
==========================================
  Files            43         43          
  Lines          1985       1985          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits            691        691          
  Misses         1158       1158          
  Partials        136        136          

Powered by Codecov. Last update 1bfe476...32aa787

@r2d4 r2d4 merged commit c4df022 into kubernetes:master Jan 8, 2017
@r2d4 r2d4 deleted the ip_set branch January 8, 2017 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants