18:03:14 #startmeeting networking_policy 18:03:15 Meeting started Thu Oct 15 18:03:14 2015 UTC and is due to finish in 60 minutes. The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot. 18:03:16 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 18:03:19 The meeting name has been set to 'networking_policy' 18:03:57 #info agenda https://wiki.openstack.org/wiki/Meetings/GroupBasedPolicy#Oct_15th_2015 18:04:19 lets start with the packaging 18:04:26 #topic Packaging 18:04:47 there was an issue with CLI/client compatibility with RHEL OSP 6 18:05:17 SumitNaiksatam: Is this the same issue as with the Fedora juno package? 18:05:27 our stable/juno does not work with the the distro since the distro uses a different version of the libs 18:05:33 that we dependent on 18:05:37 rkukura: yes, same issue 18:06:27 here is the fix for it: #link https://review.openstack.org/#/c/235037 18:06:52 SumitNaiksatam: Your patch looks reasonable, but I haven’t had a chance to test it 18:06:52 we try to get the client to work in both cases by conditionally importing 18:06:58 rkukura: okay np 18:07:14 so jishnu tested this on RHEL OSP 6 installation 18:07:39 and the compatibility with stable/juno is tested by this patch: 18:07:57 https://review.openstack.org/#/c/235045/ 18:08:29 * tested that the compatibility is stable/juno is preserved 18:09:11 if the approach looks fine, we will need to get it merged at the earliest 18:09:43 rkukura: over to you now for the packaging, any updates that you want to share? 18:11:13 Nothing new - my next step on this is to test that the f22 (juno) and f23 (kilo) packages install successfully on fedora installations that have never had other openstack pacakges installed on them. 18:11:28 I should be able to do this today or tomorrow. 18:11:56 rkukura: okay 18:12:09 I’ll probably want to update the f22 python-gbpclient package to include release of Sumits fix 18:12:24 Then ask the Red Hat folks to pull them into RDO 18:12:46 rkukura: at the appropriate please let me know when to release a new version of the juno client 18:13:02 this will of course have to be after the above fix is merged in 18:13:24 Then I’ll do some RDO testing and update the wiki instructions, and let the Red Hat folks pull the f23 RDO pacakge into delorean 18:13:40 rkukura: okay 18:13:53 SumitNaiksatam: right - as soon as it merges, I guess, unless we have other CLI patches in the pipeline 18:14:08 okay so talking about CLI patches... 18:14:25 rkukura: if you are done with that update, I can switch to the CLI topic 18:14:27 ? 18:14:41 that all from me 18:14:45 rkukura: thanks! 18:14:56 #topic CLI changes to improve usability 18:15:04 there is this other patch i have posted: 18:15:19 #link https://review.openstack.org/234156 18:15:32 it addresses a bunch of related things 18:16:00 so if we are releasing a new version of the client, i would like to get the above in that version as well 18:16:26 SumitNaiksatam: makes sense to me 18:16:35 rkukura: okay 18:17:02 so i have requested folks in the team to test this out 18:17:06 and also requesting you here 18:17:47 rkukura: so lets coordinate on reviewing, validating and merging of these two patches 18:18:00 rkukura: say by end of tomorrow 18:18:19 rkukura: and then you can take it from there for generating the packages? 18:19:05 Sounds good 18:19:21 rkukura: okay 18:19:37 #topic Stackforge namespace retirement - Oct 17th 18:19:45 #link https://wiki.openstack.org/wiki/Stackforge_Namespace_Retirement 18:20:03 so i updated the above wiki when the notification was sent out 18:20:21 the move itself will be automated 18:20:37 at our end we need to modify all places where we need make this namespace change 18:21:59 i just created a google doc here: #link https://docs.google.com/document/d/1vwycTgXFhwiGek0SFN9IAAypZuTaoUzVyeAVoOdUzUk/edit 18:22:46 lets collect input on what places we need to update to accomodate this change 18:23:19 the repo location changes, so our intgration jobs which clone the repos get affected 18:23:24 and we will need to update those 18:23:46 please add any other such items that you can think of 18:24:26 SumitNaiksatam: will the ex-stackforge projects get any special big tent tag? 18:24:45 it will also be helpful if the cores can watch their emails over the weekend to quickly mege any patches that we will need to post adapt to this migration 18:24:58 ivar-lazzaro: not that i am aware of 18:25:10 SumitNaiksatam: If this changes the git repo URLs, I’ll need to update the RPM package specs. 18:25:23 ivar-lazzaro: at least not as a part of the migration 18:25:29 rkukura: right 18:25:57 So it probably makes sense to do those updates in the f22 and f23 packages before they get cloned to RDO. 18:26:11 any place where we are using http://*stackforge*/ will have to change 18:26:35 rkukura: yeah that makes sense 18:27:00 So probably no need to update the python-gbpclient package prior to that, right? 18:27:22 rkukura: okay 18:27:34 rkukura: so you can just locally test if my patch helps 18:27:48 I plan to do that, SumitNaiksatam 18:27:49 rkukura: then lest merge the CLI/client changes 18:28:00 rkukura: then lets wait for the namespace migration to happen 18:28:14 rkukura: and any associated action that we need to take to support that 18:28:38 rkukura: and once this is settled (lets say by monday), we can proceed with the package generation? 18:28:51 SumitNaiksatam: +1 18:28:58 rkukura: okay 18:29:26 anything else we need to discuss on this topic 18:29:51 requesting everybody in the team to stay on watch for the next few days during this migration 18:30:04 SumitNaiksatam: Your google doc does seem to allow write access 18:30:13 SumitNaiksatam: will do 18:30:23 s/does/does not/ 18:30:24 rkukura: yeah its open to all, thanks! 18:30:29 rkukura: oh 18:30:59 rkukura: sorry, fixed 18:31:10 #topic Node Composition Plugin/Driver/Plumber enhancements 18:31:42 i think its time to move forward with merging the following two specs: 18:31:59 #link https://review.openstack.org/#/c/228693 (Traffic stitching & Proxy Group spec) 18:32:20 and #link https://review.openstack.org/#/c/203226/ (New Terminology and Implementation) 18:32:37 ivar-lazzaro: it seems like they are up-to-date 18:32:49 yes 18:32:57 so IIRC 18:33:10 mageshgv: i know you were tracking these more carefully 18:33:17 the only blueprint we are missing at this point is the one defining HA PTs 18:33:18 mageshgv: can we move forward? 18:33:32 ivar-lazzaro: i was just going to ask about that ;-) 18:33:37 was the next thing 18:33:55 SumitNaiksatam: I'll have a new blueprint on that 18:33:58 but anyway, i think the above two can be merged since we also have the implementation 18:34:02 ivar-lazzaro: okay 18:34:04 SumitNaiksatam: yes , the current specs are fine 18:34:05 SumitNaiksatam: doesn't really belong to any of the existing ones 18:34:13 ivar-lazzaro: okay 18:34:17 mageshgv: ok good 18:34:43 anyone else have any issues with the state of the above two specs (or the design proposed therein)? 18:34:59 not sure if igordcard_ is here today 18:36:09 okay probably not 18:36:41 i had some minor grammar/typo related comments in the second spec but it does not have to wait for that 18:37:13 implementation patches: #link https://review.openstack.org/#/q/status:open+project:stackforge/group-based-policy+branch:master+topic:bp/node-centric-chain-plugin,n,z 18:37:35 ivar-lazzaro: yesterday we reviewed and tried to merge a couple of the patches but there seemed to be merge conflicts 18:37:41 can you take a look? 18:38:06 SumitNaiksatam: I think one failed because it finished the gate job before its dependency 18:38:19 #link https://review.openstack.org/#/c/229614/ 18:38:39 sorry wrong topic link earlier: #link https://review.openstack.org/#/q/status:open+project:stackforge/group-based-policy+branch:master+topic:bp/sg-managers-in-rmd,n,z 18:39:03 not sure what happened to #link https://review.openstack.org/#/c/227562/ 18:39:11 ivar-lazzaro: right, but this one did not merge: #link https://review.openstack.org/#/c/227562/ 18:39:25 SumitNaiksatam: FYI I just checked, and it does not seem the fedora spec files contain the git URLs, just the launchpad and tarball URLs, which I don’t believe are changing. 18:39:52 rkukura: oh nice, yes those will not change 18:39:54 SumitNaiksatam: ok I'll solve the conflict 18:39:57 rkukura: definitely not the launchpad 18:40:24 ivar-lazzaro: i ran into wierd merge conflicts in the last week or so 18:40:52 ivar-lazzaro: i tried a few things but did not help, so i just locally rebased and posted another patchset 18:41:13 unfortunately the merge log is not visible 18:41:22 so difficult to tell what went wrong 18:42:01 we will have to ask that tiny little birdie there ;-) 18:42:35 I think it 18:42:39 is fixed now 18:42:47 ivar-lazzaro: nice 18:43:24 ivar-lazzaro: oh so there were merge conflicts? 18:44:22 yes, just a small thing in the apic mapping 18:44:34 ivar-lazzaro: ah okay 18:44:41 but not on the first patch of the series 18:44:53 which makes me wonder why that one failed to merge 18:46:06 ivar-lazzaro: i thought you just rebased the first patch in the series which was failing to merge 18:46:32 I always rebase the whole thing, that's my workflow 18:46:54 ivar-lazzaro: yeah, and the merge conflict seems to be in the first patch which failed to merge 18:47:13 ivar-lazzaro: patchset 21 and 22 has a diff 18:47:46 SumitNaiksatam: that was done autocatically 18:47:50 automatically* 18:47:53 that aside, i think the three patches apart from #link https://review.openstack.org/229673 are good 18:47:55 I haven't manually solved that 18:48:02 ivar-lazzaro: okay 18:48:26 we can review merge https://review.openstack.org/229673 once the spec is posted 18:48:37 ivar-lazzaro: thanks for the update 18:48:46 mageshgv: ivar-lazzaro: anything else to discuss on this today? 18:49:13 SumitNaiksatam: nothing from my side 18:50:09 mageshgv: okay 18:50:29 #topic Bugs 18:51:01 so the high priority bug was this one #link https://review.openstack.org/232274 18:51:28 rkukura: thanks for working on this 18:51:42 hi, sorry, I couldn't get to the meeting in time today 18:51:49 rkukura: so now i think all rally tests are 100% except one 18:51:52 igordcard_: no worries 18:52:01 igordcard_: thanks for stopping by anyway 18:52:05 igordcard: hi 18:52:19 rkukura: is the one that is failing the one that ajay needed to tweak? 18:52:27 SumitNaiksatam: Right, that test needs to be fixed to not use the default IP pool for multiple L3Ps for the same tenant. 18:52:36 rkukura: sweet! 18:53:14 i dont think this patch needs to wait for ajay to fix that 18:53:35 agreed 18:53:45 ivar-lazzaro: mageshgv: if there are no objections, lets try to get #link https://review.openstack.org/232274 in asap 18:53:53 SumitNaiksatam: ok 18:54:01 SumitNaiksatamL okay 18:54:07 ivar-lazzaro: i believe you have a follow up patch as well? 18:54:12 I've pushed the apic side of the fix, was someone reproducing it on the apic driver? 18:54:47 ivar-lazzaro: not that i am aware of, perhaps you need to coordinate with jishnu 18:54:54 ok 18:55:20 ivar-lazzaro: the other option is to merge and make this available in the local packages 18:55:30 ivar-lazzaro: and let them test it from that 18:55:42 ivar-lazzaro: assuming you feel confident of the fix ;-) 18:55:53 SumitNaiksatam: I don't :) 18:55:58 ivar-lazzaro: lol 18:56:28 any other critical bugs to discuss today? 18:56:51 #topic Open Discusion 18:57:16 No meeting next week since people will be preparing to travel to Tokyo 18:57:31 and the week after we will be at the summit in Tokyo, so we will meet in person 18:57:54 we can decide if we want to take a week’s break after the summit 18:58:15 but at least for now, no IRC meeting in the next couple of weeks 18:58:22 anything else we missed today? 18:58:29 SumitNaiksatam: Are we scheduling any meetings at the summit? 18:58:48 rkukura: we definitely can in an informal way 18:59:02 rkukura: we dont have formal session rooms though 18:59:16 That was my understanding 18:59:21 rkukura: right 18:59:43 alright thanks everyone for joining 18:59:46 and see you in Tokyo 18:59:49 bye! 18:59:51 cya 18:59:57 bye 19:00:02 #endmeeting