Using ipset for security group（iptables too long from N*N -> N?）

https://review.openstack.org/#/c/100761/

Change-Id:I8b9a849c6a4612f6e043c70ca1269cdd6cdc0afb

Owner

shihanzhang

Projectopenstack/neutron-specsBranchmasterTopic

bp/add-ipset-to-security

UploadedJun 18, 2014 11:52 AMUpdatedAug 21, 2014 1:54 PMStatusMerged

Commit Message

Permalink

Using ipset for security group

Change-Id: I8b9a849c6a4612f6e043c70ca1269cdd6cdc0afbImplements blueprint add-ipset-to-security

Reviewer Code-ReviewVerifiedWorkflow

shihanzhang

    

Jay Pipes

    

Ihar Hrachyshka

    

Carl Baldwin

    

Liping Mao

    

Miguel Angel Ajo

    

QianLin

    

keshava

    

Isaku Yamahata

    

Gary Duan

    

Bertrand Lallau

    

YAMAMOTO Takashi

    

Jenkins

   

Sudhakar Babu Gariganti

 +1  

Kyle Mestery

   

Salvatore Orlando

 +1  

Sumit Naiksatam

  

Tomoe Sugihara

 +1  

Jenkins check (2 rechecks)Jul 21, 2014 11:02 AMgate-neutron-specs-python27SUCCESS in 2m 04sgate-neutron-specs-docsSUCCESS in 1m 44sIncluded in Dependencies  Reference Version:BasePatch Set 1Patch Set 2Patch Set 3Patch Set 4Patch Set 5Patch Set 6Patch Set 7Patch Set 8Patch Set 9Patch Set 10Patch Set 11Patch Set 12Patch Set 13Patch Set 14Patch Set 15Patch Set 16Patch Set 17Patch Set 18Patch Set 19Patch Set 20Patch Set 21Patch Set 22

Patch Set 1 5093c825aac7d319d39ab596858e8e5c17891ae3 (gitweb) Patch Set 2 4888024616eedddd08829a00fc68c60088cc3ebe (gitweb) Patch Set 3 7b9f48bab0c8cae2545c66be53418bc44fc925c2 (gitweb) Patch Set 4 1b28b3928d411822b5424c629899d9ffa537c3dd (gitweb) Patch Set 5 5a15c10ff257a7b473d6b5cf424c62574b527396 (gitweb) Patch Set 6 960a8ca80bb4f51cab047e5f2e264ebd2858f093 (gitweb) Patch Set 7 4532c87fb6963131fafa1cccd2fc994103545605 (gitweb) Patch Set 8 3c9e869dd69c1ba00c5f32207ab6e991f302fe4b (gitweb) Patch Set 9 69f30929c648c5cf4e55a30ca774a806f1879cfa (gitweb) Patch Set 10 7c3de8bffa47c3b152fee0e2499b427535e07e7b (gitweb) Patch Set 11 d700ecf9f5554d772631c2c8036d1c2b7ee3dfb5 (gitweb) Patch Set 12 d3f24e3ebb3fdabaf7fe3e809337227bd81fd6b5 (gitweb) Patch Set 13 5bfe163305b8558b33c99a1ad7b89396d94b88eb (gitweb) Patch Set 14 478459bdf17cb6f85cf6e4ceab13ab9cc16e229e (gitweb) Patch Set 15 bc0d7f43f7ee25da1fc0133792bf64c56f8a89e4 (gitweb) Patch Set 16 8a0aae7b47e141eb7da625fb7ca06c906ce3afeb (gitweb) Patch Set 17 7f41b943ec6c719b4686f9baca0b08bceea23020 (gitweb) Patch Set 18 0281b2f89d5b9f881ec7d9bcd2110da67ebee2d6 (gitweb) Patch Set 19 bdae4320a59d33c958e94ced7dadc563151ea1b7 (gitweb) Patch Set 20 f64c01aaa27797138929fd13eb2943df4aab08b5 (gitweb) Patch Set 21 6d2e91c1b4c4bd6dd6a337c2d13f0b5b617fdf4e (gitweb) Patch Set 22 2bc957d4c41fc48115c6513cd10eb88b40ea32a1 (gitweb)

Author

shihanzhang

<[e-mail]>Jun 18, 2014 11:46 AM

Committer

shihanzhang

<[e-mail]>Jul 14, 2014 7:26 PM

Parent(s)72e27e38dfce050a2d83096d0d397a3dfe6ef85fMerge "a10-lbaas-driver"Download

checkoutpullcherry-pickpatch

Anonymous HTTP

git fetch https://review.openstack.org/openstack/neutron-specs refs/changes/61/100761/22 && git checkout FETCH_HEAD

  File PathCommentsSizeDiff Commit Message  Side-by-SideUnified Aspecs/juno/add-ipset-to-security.rst2 comments176 linesSide-by-SideUnified  +176, -0All Side-by-SideAll Unified

Comments

Expand RecentExpand AllCollapse All

 shihanzhangUploaded patch set 1.Jun 18, 2014

 shihanzhangUploaded patch set 2.Jun 18, 2014

 shihanzhangUploaded patch set 3.Jun 18, 2014

 YAMAMOTO TakashiPatch Set 3: Code-Review-1 (4 comments)Jun 18, 2014

 shihanzhangPatch Set 3: (4 comments)Jun 18, 2014

 shihanzhangUploaded patch set 4.Jun 18, 2014

 Tomoe SugiharaPatch Set 4: Code-Review-1 (1 comment)Jun 18, 2014

 shihanzhangPatch Set 4: (1 comment)Jun 18, 2014

 shihanzhangUploaded patch set 5.Jun 18, 2014

 shihanzhangUploaded patch set 6.Jun 18, 2014

 shihanzhangUploaded patch set 7.Jun 18, 2014

 Salvatore OrlandoPatch Set 7: Code-Review-1 (6 comments) Please provide more details.Jun 18, 2014

 shihanzhangPatch Set 7: (6 comments)Jun 18, 2014

 shihanzhangUploaded patch set 8.Jun 18, 2014

 YAMAMOTO TakashiPatch Set 8: Code-Review-1 (4 comments)Jun 18, 2014

 shihanzhangPatch Set 8: (4 comments)Jun 19, 2014

 shihanzhangUploaded patch set 9.Jun 19, 2014

 keshavaPatch Set 9: (1 comment)Jun 19, 2014

 YAMAMOTO TakashiPatch Set 9: Code-Review+1Jun 19, 2014

 Sudhakar Babu GarigantiPatch Set 9: Code-Review-1 (2 comments) Like the idea. Thanks for …Jun 20, 2014

 shihanzhangPatch Set 9: (2 comments)Jun 21, 2014

 shihanzhangUploaded patch set 10.Jun 23, 2014

 shihanzhangUploaded patch set 11.Jun 23, 2014

 Sudhakar Babu GarigantiPatch Set 11: (1 comment)Jun 23, 2014

 shihanzhangPatch Set 11: (1 comment)Jun 23, 2014

 Miguel Angel AjoPatch Set 9: (2 comments)Jun 23, 2014

 shihanzhangPatch Set 9: (3 comments)Jun 23, 2014

 shihanzhangUploaded patch set 12.Jun 23, 2014

 shihanzhangUploaded patch set 13.Jun 23, 2014

 shihanzhangUploaded patch set 14.Jun 23, 2014

 QianLinPatch Set 14: Code-Review+1 LGTM.Jun 27, 2014

 Miguel Angel AjoPatch Set 14: (4 comments)Jul 1, 2014

 Miguel Angel AjoPatch Set 14: Code-Review-1 -1 just to get attention into the comments, I …Jul 1, 2014

 shihanzhangPatch Set 14: (3 comments)Jul 2, 2014

 shihanzhangUploaded patch set 15.Jul 2, 2014

 YAMAMOTO TakashiPatch Set 15: Code-Review-1 (3 comments)Jul 2, 2014

 Liping MaoPatch Set 15: Code-Review-1 (4 comments)Jul 2, 2014

 shihanzhangPatch Set 15: (7 comments)Jul 2, 2014

 Miguel Angel AjoPatch Set 15: Code-Review-1 (1 comment)Jul 2, 2014

 shihanzhangPatch Set 15: (1 comment)Jul 2, 2014

 shihanzhangUploaded patch set 16.Jul 3, 2014

 shihanzhangUploaded patch set 17.Jul 3, 2014

 Miguel Angel AjoPatch Set 17: (7 comments)Jul 4, 2014

 Salvatore OrlandoPatch Set 17: Code-Review-1 (4 comments) I still do not understand if …Jul 4, 2014

 shihanzhangPatch Set 17: (10 comments)Jul 4, 2014

 shihanzhangUploaded patch set 18.Jul 4, 2014

 Miguel Angel AjoPatch Set 17: (1 comment)Jul 4, 2014

 Miguel Angel AjoPatch Set 18: (2 comments)Jul 4, 2014

 shihanzhangPatch Set 18: (2 comments)Jul 5, 2014

 shihanzhangUploaded patch set 19.Jul 7, 2014

 Miguel Angel AjoPatch Set 19: Code-Review+1Jul 8, 2014

 Carl BaldwinPatch Set 19: Code-Review-1 (2 comments) I think I might know what …Jul 11, 2014

 shihanzhangPatch Set 19: (2 comments)Jul 11, 2014

 shihanzhangUploaded patch set 20.Jul 11, 2014

 shihanzhangUploaded patch set 21.Jul 11, 2014

 Salvatore OrlandoPatch Set 21: Code-Review-1 I am a chicken and a wuss, and I'm giving -1 …Jul 11, 2014

 shihanzhangPatch Set 21: hi Salvatore Orlando, thanks for your review. I agree with …Jul 11, 2014

 Ihar HrachyshkaPatch Set 21: @Salvatoro, I respectfully disagree. If the author will …Jul 11, 2014

 Liping MaoPatch Set 21: Code-Review+1 I think this is OK, if user can choose if or …Jul 11, 2014

 Miguel Angel AjoPatch Set 21: @Salvatore, I agree with them, making the feature …Jul 14, 2014

 Miguel Angel AjoPatch Set 21: I'm thinking about the plugin implementation, with …Jul 14, 2014

 Salvatore OrlandoPatch Set 21: (1 comment) I just want to understand how that's made …Jul 14, 2014

 shihanzhangPatch Set 21: (1 comment)Jul 14, 2014

 shihanzhangUploaded patch set 22.Jul 14, 2014

 Kyle MesteryPatch Set 22: Code-Review-2 Thank you for your spec submission. …Jul 14, 2014

 Salvatore OrlandoPatch Set 22: Code-Review+1 I don't want to open the "season of core …Jul 15, 2014

 Tomoe SugiharaPatch Set 22: Code-Review+1Jul 15, 2014

 shihanzhangPatch Set 22: thanks all, my spec has commit for a long time and the …Jul 15, 2014

 Sudhakar Babu GarigantiPatch Set 22: Code-Review+1 I agree with Salvatore's comments. This …Jul 17, 2014

 Kyle MesteryPatch Set 22: Code-Review+2 We'll see if this lands, though we're packed …Jul 21, 2014

 Sumit NaiksatamPatch Set 22: Code-Review+2 Workflow+1 Although not explicitly stated in …Jul 21, 2014

 JenkinsChange has been successfully merged into the git repository.Jul 21, 2014

 Jay PipesPatch Set 22: (1 comment)Aug 20, 2014

 Miguel Angel Ajo Aug 21, 2014

Patch Set 22:

(1 comment)

Add Comment

Press '?' to view keyboard shortcuts | Powered by Gerrit Code Review (2.8.4-15-g6dc8444) | Report Bug