*** skraynev has quit IRC | 00:07 | |
*** Guest3116 has quit IRC | 00:07 | |
*** feleouet has quit IRC | 00:07 | |
*** wendar has quit IRC | 00:07 | |
*** sc68cal has quit IRC | 00:07 | |
*** adam_g has quit IRC | 00:07 | |
*** peopleme1ge has quit IRC | 00:07 | |
*** muhanpong has quit IRC | 00:07 | |
*** Madkiss has quit IRC | 00:07 | |
*** dguitarbite has quit IRC | 00:07 | |
*** shakamunyi has quit IRC | 00:07 | |
*** Madkiss_ has joined #openstack-neutron | 00:07 | |
*** Madkiss_ is now known as Madkiss | 00:07 | |
*** zhiyan_ has joined #openstack-neutron | 00:07 | |
*** wendar_ has joined #openstack-neutron | 00:07 | |
*** francois_eleouet has joined #openstack-neutron | 00:07 | |
*** zacksh has joined #openstack-neutron | 00:07 | |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: merge br-tun into br-int https://review.openstack.org/112488 | 00:07 |
---|---|---|
*** muhanpon1 has joined #openstack-neutron | 00:07 | |
*** nati_ueno has joined #openstack-neutron | 00:07 | |
*** jdag___ has joined #openstack-neutron | 00:07 | |
*** jamespd_ has joined #openstack-neutron | 00:08 | |
*** openstack has joined #openstack-neutron | 00:09 | |
*** zhiyan_ is now known as zhiyan | 00:09 | |
*** jdag___ is now known as jdag__ | 00:10 | |
*** baoli has joined #openstack-neutron | 00:11 | |
*** rook has quit IRC | 00:11 | |
*** swat30 has quit IRC | 00:11 | |
*** jamespd has quit IRC | 00:11 | |
*** mitz_ has quit IRC | 00:11 | |
*** puck has quit IRC | 00:11 | |
*** raorn_ has quit IRC | 00:11 | |
*** swat30_ is now known as swat30 | 00:11 | |
*** padkrish has joined #openstack-neutron | 00:12 | |
*** padkrish has joined #openstack-neutron | 00:12 | |
*** padkrish has quit IRC | 00:15 | |
*** gildub has joined #openstack-neutron | 00:16 | |
*** ivar-lazzaro has quit IRC | 00:19 | |
*** diegows has joined #openstack-neutron | 00:20 | |
*** karimb has quit IRC | 00:21 | |
*** oda-g has joined #openstack-neutron | 00:22 | |
*** diegows has quit IRC | 00:24 | |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Enable local arp responder for local VMs https://review.openstack.org/113160 | 00:24 |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: l2pop: Allow network types overridable https://review.openstack.org/112947 | 00:25 |
*** diegows has joined #openstack-neutron | 00:25 | |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Local arp responder for VLAN https://review.openstack.org/112949 | 00:28 |
*** tomoe_ has joined #openstack-neutron | 00:28 | |
*** ivar-lazzaro has joined #openstack-neutron | 00:30 | |
*** stack__ has joined #openstack-neutron | 00:34 | |
*** rook has joined #openstack-neutron | 00:36 | |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Implement physical_interface_mappings https://review.openstack.org/113166 | 00:37 |
*** padkrish has joined #openstack-neutron | 00:40 | |
*** silverpuff has joined #openstack-neutron | 00:46 | |
*** padkrish has quit IRC | 00:49 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Adds external attachment type validators to ML2 https://review.openstack.org/116187 | 00:52 |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: External Attachment Point Extension https://review.openstack.org/97173 | 00:52 |
*** yongli has joined #openstack-neutron | 00:54 | |
*** padkrish has joined #openstack-neutron | 00:54 | |
*** shakamunyi has joined #openstack-neutron | 00:54 | |
*** yamamoto has quit IRC | 00:55 | |
*** padkrish has quit IRC | 00:56 | |
*** inara has quit IRC | 00:57 | |
*** padkrish has joined #openstack-neutron | 00:57 | |
*** balles has quit IRC | 00:59 | |
*** rook has quit IRC | 00:59 | |
*** shakamunyi has quit IRC | 00:59 | |
*** balles has joined #openstack-neutron | 00:59 | |
*** gildub has quit IRC | 01:01 | |
*** inara has joined #openstack-neutron | 01:02 | |
*** gildub has joined #openstack-neutron | 01:05 | |
*** inara has quit IRC | 01:05 | |
*** sbfox has quit IRC | 01:05 | |
*** prasoon has quit IRC | 01:06 | |
*** inara has joined #openstack-neutron | 01:06 | |
*** ivar-lazzaro has quit IRC | 01:11 | |
*** tsg has joined #openstack-neutron | 01:11 | |
*** stanzgy has joined #openstack-neutron | 01:13 | |
*** ivar-lazzaro has joined #openstack-neutron | 01:14 | |
*** inara has quit IRC | 01:17 | |
*** ivar-lazzaro has quit IRC | 01:17 | |
*** diegows has quit IRC | 01:20 | |
*** inara has joined #openstack-neutron | 01:21 | |
*** dfarrell07 has joined #openstack-neutron | 01:23 | |
*** otherwiseguy has quit IRC | 01:25 | |
*** terryw has joined #openstack-neutron | 01:25 | |
*** ivar-lazzaro has joined #openstack-neutron | 01:25 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: BSN: Bind external ports in ML2 driver https://review.openstack.org/116188 | 01:26 |
*** ivar-lazzaro has quit IRC | 01:30 | |
*** tsg has quit IRC | 01:36 | |
*** dfarrell07 has quit IRC | 01:36 | |
*** popw has joined #openstack-neutron | 01:43 | |
*** dguitarbite has joined #openstack-neutron | 01:44 | |
*** yamamoto has joined #openstack-neutron | 01:45 | |
*** mitarun has joined #openstack-neutron | 01:45 | |
*** mitarun has quit IRC | 01:49 | |
*** SridharRamaswamy has joined #openstack-neutron | 01:49 | |
*** diegows has joined #openstack-neutron | 01:51 | |
nlahouti | Can someone please let me know how to rebase a patch with dependency, where the patch that it depends on is merged? (i.e A -> B and B is merged) | 01:51 |
*** baohua has joined #openstack-neutron | 01:52 | |
*** ivar-lazzaro has joined #openstack-neutron | 01:54 | |
*** shakamunyi has joined #openstack-neutron | 01:55 | |
*** JayJ has joined #openstack-neutron | 01:55 | |
*** rook has joined #openstack-neutron | 01:55 | |
*** mitarun has joined #openstack-neutron | 01:59 | |
StevenK | nlahouti: Okay, so A depends on B, and B is merged, which means B is now in master, so you can rebase A onto an up-to-date master | 01:59 |
*** shakamunyi has quit IRC | 01:59 | |
*** rook has quit IRC | 02:00 | |
*** mitarun has quit IRC | 02:01 | |
*** mitarun has joined #openstack-neutron | 02:02 | |
nlahouti | StevenK: thx. so need to do 1) clone master 2) cherry-pick the patch ... | 02:02 |
*** diegows has quit IRC | 02:03 | |
*** popw has quit IRC | 02:03 | |
StevenK | nlahouti: You should already have master in your local repo -- So, git checkout master and then git pull ; and then you want to git checkout <your A branch> and then run git rebase -i master | 02:03 |
openstackgerrit | Arvind Somya proposed a change to openstack/neutron: ML2 Type Driver refactor part 3 https://review.openstack.org/115151 | 02:03 |
*** prasoon has joined #openstack-neutron | 02:04 | |
nlahouti | StevenK: got it. thx a lot for your reply. And one more thing. | 02:05 |
*** mitarun has quit IRC | 02:06 | |
nlahouti | StevenK: I had case that A-> B and also A-> C and B & C are merged. | 02:06 |
nlahouti | because of second dependency I had to duplicate file in A from C. to have only A-> B dependency. | 02:07 |
*** padkrish has joined #openstack-neutron | 02:08 | |
nlahouti | now when I rebase, in the commit message it shows conflict files. But it doesn't show as part of files getting committed (basically they shouldn't be committed). | 02:08 |
*** yamahata has joined #openstack-neutron | 02:08 | |
*** yamahata_ has quit IRC | 02:09 | |
StevenK | nlahouti: Rebase is a little special | 02:09 |
nlahouti | StevenK: would that cause any issue after git review ? | 02:09 |
*** yamahata_ has joined #openstack-neutron | 02:09 | |
*** prasoon has quit IRC | 02:09 | |
StevenK | nlahouti: Can you pastebin git status ? | 02:09 |
*** terryw has quit IRC | 02:11 | |
*** popw has joined #openstack-neutron | 02:12 | |
nlahouti | git status doesn't show anything now only topic branch and nothing to commit, working directory clean | 02:12 |
nlahouti | I can pastebin the commit message | 02:12 |
*** padkrish has quit IRC | 02:12 | |
StevenK | nlahouti: Huh, what happened after git rebase -i master ? | 02:12 |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Big Switch: Bind IVS ports in ML2 driver https://review.openstack.org/113749 | 02:14 |
nlahouti | StevenK: there was merge errors and I fixed it and added the modified files then git rebase —continue. | 02:15 |
StevenK | nlahouti: Ah, right, excellent. Then git review and you're done | 02:15 |
openstackgerrit | A change was merged to openstack/neutron: Networks are not scheduled to DHCP agents for Cisco N1KV plugin https://review.openstack.org/114091 | 02:17 |
nlahouti | StevenK: Here is the log: http://paste.openstack.org/show/103987/ | 02:18 |
*** otherwiseguy has joined #openstack-neutron | 02:20 | |
nlahouti | StevenK: And here is the steps that I did: http://paste.openstack.org/show/103988/ | 02:25 |
StevenK | nlahouti: Should be fine, but I'd suggest running git show and eyeballing the diff | 02:27 |
*** prasoon has joined #openstack-neutron | 02:32 | |
nlahouti | StevenK: I checked 'git show' and it has all the changes and seems correct. | 02:32 |
nlahouti | StevenK: hopefully 'git review' doesn't give any error. thx a lot for your time. I'll update you on how it goes. | 02:33 |
StevenK | nlahouti: Then you should be good to run git review :-) | 02:33 |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Allow setting a tenant router's external IP https://review.openstack.org/83664 | 02:34 |
*** sungju has quit IRC | 02:36 | |
*** ianw has quit IRC | 02:36 | |
*** ianw has joined #openstack-neutron | 02:36 | |
openstackgerrit | nlahouti proposed a change to openstack/neutron: Cisco DFA ML2 Mechanism Driver - Part 3 https://review.openstack.org/111761 | 02:39 |
*** JayJ has quit IRC | 02:39 | |
*** JayJ has joined #openstack-neutron | 02:40 | |
nlahouti | StevenK: It went through and was okay :) thank a lot again. | 02:40 |
*** zhhuabj has joined #openstack-neutron | 02:41 | |
*** zhhuabj has quit IRC | 02:43 | |
*** zhhuabj has joined #openstack-neutron | 02:44 | |
*** padkrish has joined #openstack-neutron | 02:45 | |
openstackgerrit | Feng Xi Yan proposed a change to openstack/neutron: ML2 Mechanism Driver for vCenter DVS port group creation https://review.openstack.org/111227 | 02:48 |
*** zhhuabj has quit IRC | 02:49 | |
*** zhhuabj has joined #openstack-neutron | 02:49 | |
openstackgerrit | Angus Lees proposed a change to openstack/neutron: Remove chain for correct router during update_routers() https://review.openstack.org/117444 | 02:54 |
openstackgerrit | Angus Lees proposed a change to openstack/neutron: Remove chain for correct router during update_routers() https://review.openstack.org/117444 | 02:56 |
*** shakamunyi has joined #openstack-neutron | 02:56 | |
*** zhhuabj has quit IRC | 02:56 | |
*** rook has joined #openstack-neutron | 02:56 | |
*** tchaypo has joined #openstack-neutron | 02:58 | |
tchaypo | nlahouti: are you familiar with "git reflog"? | 02:59 |
*** shakamunyi has quit IRC | 03:00 | |
*** rook has quit IRC | 03:01 | |
lifeless | tchaypo: thats where you get flogged multiple times right? | 03:01 |
*** zhhuabj has joined #openstack-neutron | 03:01 | |
tchaypo | lifeless: no, that would be bizarre | 03:02 |
tchaypo | oops, I misspelled | 03:02 |
lifeless | re-flogged, right? | 03:03 |
nlahouti | tchaypo: haven't used it. I usually use 'git log' | 03:04 |
tchaypo | If you've done something like a rebase and realised that it didn't give you the result you wanted, `git reflog` can help you step back to just before the rebase | 03:06 |
tchaypo | I used to be scared of rebasing until I got used to using reflog to step back in time. Now I'm less scared about reflog because I know I can just undo it if it goes bad | 03:07 |
StevenK | tchaypo: It's not so much undo as "Pretend it never happened" | 03:09 |
tchaypo | http://stackoverflow.com/questions/134882/undoing-a-git-rebase is probably as good a starting point as any | 03:09 |
nlahouti | tchaypo: thanks for the info. I'll keep it in my mind to use it. I'm really cautious when rebasing and don't want to mess up anything. | 03:10 |
tchaypo | Yeah, that's a good description. If you wanted to be more formal and were used to thinking of the commit history as a graph where each commit is a node, you might say that the reflog is a history of which node "HEAD" points at, which lets you use "git reset --hard" to step directly to any other node in the graph | 03:11 |
tchaypo | But if you start describing it that way you'll probably end up talking about homeomorphic endofunctors, and that's not a road any sane person wants to go down | 03:12 |
StevenK | tchaypo: Commit history *is* a graph :-P | 03:12 |
tchaypo | Sure, but not everyone *thinks* of it that way | 03:12 |
*** baoli has quit IRC | 03:18 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: UTs: Disable auto deletion of ports/subnets/nets https://review.openstack.org/102465 | 03:23 |
*** padkrish has quit IRC | 03:25 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Big Switch: Retry on 503 errors from backend https://review.openstack.org/114379 | 03:26 |
*** tflynn has joined #openstack-neutron | 03:31 | |
*** mitarun has joined #openstack-neutron | 03:33 | |
*** amotoki has joined #openstack-neutron | 03:37 | |
*** LiberTerra has quit IRC | 03:39 | |
*** dfarrell07 has joined #openstack-neutron | 03:40 | |
openstackgerrit | Itsuro Oda proposed a change to openstack/neutron: Add endpoint_type parameter to MetaInterfaceDriver https://review.openstack.org/113411 | 03:41 |
*** networkstatic has quit IRC | 03:41 | |
openstackgerrit | nlahouti proposed a change to openstack/neutron: Cisco DFA ML2 Mechanism Driver - Part 4 https://review.openstack.org/111863 | 03:44 |
*** JayJ has quit IRC | 03:45 | |
*** JayJ has joined #openstack-neutron | 03:45 | |
*** mitarun has quit IRC | 03:49 | |
*** mitarun has joined #openstack-neutron | 03:49 | |
*** tsg has joined #openstack-neutron | 03:50 | |
*** jistr has joined #openstack-neutron | 03:53 | |
*** dfarrell07 has quit IRC | 03:53 | |
*** mitarun has quit IRC | 03:53 | |
*** SridharRamaswamy has quit IRC | 03:54 | |
*** jistr is now known as jistr|doc | 03:55 | |
*** ivar-lazzaro has quit IRC | 03:57 | |
*** shakamunyi has joined #openstack-neutron | 03:57 | |
*** rook has joined #openstack-neutron | 03:57 | |
*** ivar-lazzaro has joined #openstack-neutron | 03:57 | |
*** rook has quit IRC | 04:01 | |
*** shakamunyi has quit IRC | 04:01 | |
*** mitarun has joined #openstack-neutron | 04:07 | |
*** otherwiseguy has quit IRC | 04:07 | |
*** otherwiseguy has joined #openstack-neutron | 04:08 | |
*** fifieldt_ has quit IRC | 04:08 | |
*** saju_m has joined #openstack-neutron | 04:10 | |
*** ijw has joined #openstack-neutron | 04:12 | |
*** JayJ has quit IRC | 04:15 | |
*** ijw has quit IRC | 04:16 | |
*** padkrish has joined #openstack-neutron | 04:22 | |
*** padkrish has quit IRC | 04:23 | |
*** padkrish has joined #openstack-neutron | 04:23 | |
*** dfarrell07 has joined #openstack-neutron | 04:23 | |
*** chandankumar has joined #openstack-neutron | 04:32 | |
*** dfarrell07 has quit IRC | 04:35 | |
*** nati_ueno has quit IRC | 04:36 | |
*** nati_ueno has joined #openstack-neutron | 04:36 | |
*** garyk has quit IRC | 04:40 | |
*** saju_m has quit IRC | 04:43 | |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: Fix a recent ipv6 UT regression https://review.openstack.org/118075 | 04:44 |
*** saju_m has joined #openstack-neutron | 04:47 | |
*** saju_m has quit IRC | 04:47 | |
*** yfried has quit IRC | 04:51 | |
*** _cjones_ has joined #openstack-neutron | 04:52 | |
*** leenheer has joined #openstack-neutron | 04:52 | |
*** chandankumar has quit IRC | 04:53 | |
*** _cjones_ has quit IRC | 04:53 | |
*** simon-AS559 has joined #openstack-neutron | 04:57 | |
*** _cjones_ has joined #openstack-neutron | 04:58 | |
*** rook has joined #openstack-neutron | 04:58 | |
*** shakamunyi has joined #openstack-neutron | 04:58 | |
*** simon-AS5591 has joined #openstack-neutron | 04:59 | |
*** chandankumar has joined #openstack-neutron | 05:01 | |
openstackgerrit | Angus Lees proposed a change to openstack/neutron: Purge use of "PRED and A or B" poor-mans-ternary https://review.openstack.org/110849 | 05:01 |
*** simon-AS559 has quit IRC | 05:01 | |
*** rook has quit IRC | 05:02 | |
*** shakamunyi has quit IRC | 05:02 | |
*** _cjones_ has quit IRC | 05:03 | |
*** _cjones_ has joined #openstack-neutron | 05:03 | |
*** padkrish has quit IRC | 05:04 | |
openstackgerrit | Koteswara Rao Kelam proposed a change to openstack/neutron: shared policy shouldn't have unshared rules https://review.openstack.org/115912 | 05:08 |
*** _cjones_ has quit IRC | 05:08 | |
*** chandankumar has quit IRC | 05:09 | |
openstackgerrit | Angus Lees proposed a change to openstack/neutron: Remove unused arg to config.setup_logging() https://review.openstack.org/111180 | 05:09 |
*** killer_prince is now known as lazy_prince | 05:12 | |
*** kopparam has joined #openstack-neutron | 05:13 | |
*** chandankumar has joined #openstack-neutron | 05:14 | |
*** rotbeard has joined #openstack-neutron | 05:14 | |
*** flaviof_zzz has joined #openstack-neutron | 05:17 | |
*** padkrish has joined #openstack-neutron | 05:21 | |
*** padkrish has quit IRC | 05:24 | |
*** padkrish has joined #openstack-neutron | 05:24 | |
*** padkrish_ has joined #openstack-neutron | 05:27 | |
*** padkrish has quit IRC | 05:27 | |
*** trinaths has joined #openstack-neutron | 05:28 | |
*** JayJ has joined #openstack-neutron | 05:29 | |
*** SridharG has joined #openstack-neutron | 05:32 | |
*** JayJ has quit IRC | 05:33 | |
Guest41143 | Hello, I added a static routes to a router and now, I want to remove it. How is it done? | 05:37 |
Guest41143 | Using 'neutron router-list -c name -c routes' print outs the static routes of a router | 05:38 |
*** LiberTerra has joined #openstack-neutron | 05:39 | |
openstackgerrit | Jamie Lennox proposed a change to openstack/python-neutronclient: Use adapter from keystoneclient https://review.openstack.org/118006 | 05:40 |
*** chandankumar has quit IRC | 05:40 | |
*** simon-AS5591 has quit IRC | 05:41 | |
*** padkrish_ has quit IRC | 05:46 | |
*** padkrish has joined #openstack-neutron | 05:47 | |
*** garyk has joined #openstack-neutron | 05:48 | |
*** jamielennox has joined #openstack-neutron | 05:49 | |
*** sbfox has joined #openstack-neutron | 05:53 | |
*** _cjones_ has joined #openstack-neutron | 05:53 | |
*** nlahouti has quit IRC | 05:55 | |
*** shakamunyi has joined #openstack-neutron | 05:59 | |
*** rook has joined #openstack-neutron | 05:59 | |
*** _cjones_ has quit IRC | 05:59 | |
*** JayJ has joined #openstack-neutron | 06:00 | |
openstackgerrit | Koteswara Rao Kelam proposed a change to openstack/neutron: shared policy shouldn't have unshared rules https://review.openstack.org/115912 | 06:01 |
*** rook has quit IRC | 06:03 | |
*** shakamunyi has quit IRC | 06:03 | |
*** k4n0 has joined #openstack-neutron | 06:04 | |
*** JayJ has quit IRC | 06:04 | |
*** LiberTerra has quit IRC | 06:05 | |
*** sungju has joined #openstack-neutron | 06:05 | |
*** leenheer has quit IRC | 06:05 | |
*** tflynn has quit IRC | 06:10 | |
*** nati_uen_ has joined #openstack-neutron | 06:10 | |
*** nati_ueno has quit IRC | 06:13 | |
*** ijw has joined #openstack-neutron | 06:13 | |
*** ijw has quit IRC | 06:18 | |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy API-2: Classifiers, Actions, and Rules https://review.openstack.org/101816 | 06:18 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy Mapping API-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/100716 | 06:18 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy Plugin-3: Contracts https://review.openstack.org/103798 | 06:18 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy Mapping Plugin-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/101957 | 06:18 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy Plugin-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/96393 | 06:18 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy Plugin-2: Classifiers, Actions, and Rules https://review.openstack.org/103465 | 06:18 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy API-3: Contracts https://review.openstack.org/103486 | 06:18 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy API-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/95900 | 06:18 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/117100 | 06:19 |
*** jamielennox has left #openstack-neutron | 06:20 | |
*** chandankumar has joined #openstack-neutron | 06:20 | |
*** ivar-lazzaro has quit IRC | 06:20 | |
*** ivar-lazzaro has joined #openstack-neutron | 06:20 | |
*** ivar-lazzaro has quit IRC | 06:21 | |
*** yfauser has joined #openstack-neutron | 06:23 | |
*** yfauser has left #openstack-neutron | 06:23 | |
*** yfried has joined #openstack-neutron | 06:26 | |
*** lazy_prince has quit IRC | 06:26 | |
*** hanzhang has joined #openstack-neutron | 06:32 | |
hanzhang | hi all | 06:33 |
*** gildub has quit IRC | 06:33 | |
*** salv-orlando has joined #openstack-neutron | 06:33 | |
*** padkrish has quit IRC | 06:33 | |
*** padkrish has joined #openstack-neutron | 06:34 | |
*** chandankumar has quit IRC | 06:38 | |
*** irenab has joined #openstack-neutron | 06:40 | |
*** RussB has joined #openstack-neutron | 06:40 | |
*** padkrish_ has joined #openstack-neutron | 06:41 | |
*** padkrish has quit IRC | 06:41 | |
*** RussB has left #openstack-neutron | 06:41 | |
*** simon-AS559 has joined #openstack-neutron | 06:41 | |
*** chandankumar has joined #openstack-neutron | 06:43 | |
openstackgerrit | A change was merged to openstack/neutron: Provide a quick way to run flake8 https://review.openstack.org/115327 | 06:46 |
openstackgerrit | yangxurong proposed a change to openstack/neutron: Huawei SDN Mechanism Driver for ML2 Plugin https://review.openstack.org/104103 | 06:49 |
*** devvesa has joined #openstack-neutron | 06:51 | |
*** roeyc has joined #openstack-neutron | 06:51 | |
openstackgerrit | Motohiro Otsuka proposed a change to openstack/neutron: Remove reference to cisco_cfg_agent.ini from setup.cfg again. https://review.openstack.org/118092 | 06:51 |
*** chandankumar has quit IRC | 06:54 | |
*** chandankumar has joined #openstack-neutron | 06:54 | |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: Fix a recent ipv6 UT regression https://review.openstack.org/118075 | 06:58 |
*** pasquier-s has joined #openstack-neutron | 06:59 | |
*** rook has joined #openstack-neutron | 06:59 | |
*** shakamunyi has joined #openstack-neutron | 06:59 | |
openstackgerrit | garyk proposed a change to openstack/neutron: Remove @author from copyright statements https://review.openstack.org/112329 | 07:01 |
*** JayJ has joined #openstack-neutron | 07:01 | |
*** jlibosva has joined #openstack-neutron | 07:01 | |
*** nlahouti has joined #openstack-neutron | 07:01 | |
*** nlahouti has joined #openstack-neutron | 07:02 | |
*** otherwiseguy has quit IRC | 07:02 | |
*** nlahouti has quit IRC | 07:02 | |
*** Longgeek has joined #openstack-neutron | 07:02 | |
*** jschwarz has joined #openstack-neutron | 07:03 | |
*** rook has quit IRC | 07:04 | |
*** kopparam has quit IRC | 07:04 | |
*** shakamunyi has quit IRC | 07:04 | |
*** kopparam has joined #openstack-neutron | 07:04 | |
*** JayJ has quit IRC | 07:05 | |
*** Longgeek has quit IRC | 07:06 | |
*** Longgeek has joined #openstack-neutron | 07:08 | |
*** kopparam has quit IRC | 07:09 | |
*** killer_prince has joined #openstack-neutron | 07:09 | |
*** killer_prince is now known as lazy_prince | 07:09 | |
*** doude has joined #openstack-neutron | 07:09 | |
*** yfauser has joined #openstack-neutron | 07:11 | |
*** padkrish_ has quit IRC | 07:14 | |
*** yfauser has left #openstack-neutron | 07:15 | |
*** gildub has joined #openstack-neutron | 07:20 | |
openstackgerrit | A change was merged to openstack/neutron: Apic drivers enhancements (second approach): L3 refactor https://review.openstack.org/116156 | 07:22 |
*** luqas has joined #openstack-neutron | 07:22 | |
*** luqas has quit IRC | 07:26 | |
*** sungju has quit IRC | 07:27 | |
*** chandankumar has quit IRC | 07:28 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Allow setting a tenant router's external IP https://review.openstack.org/83664 | 07:28 |
*** saurabh_ has quit IRC | 07:31 | |
*** luqas has joined #openstack-neutron | 07:32 | |
*** kopparam has joined #openstack-neutron | 07:33 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: UTs: Disable auto deletion of ports/subnets/nets https://review.openstack.org/102465 | 07:34 |
*** mitarun has quit IRC | 07:37 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Big Switch: Separate L3 functions into L3 service https://review.openstack.org/113653 | 07:37 |
*** mitarun has joined #openstack-neutron | 07:38 | |
kevinbenton | amotoki: ping | 07:39 |
amotoki | kevinbenton: pong | 07:39 |
kevinbenton | amotoki: nevermind. was going to ask how to retrigger the NEC CI but I see in the comment it’s recheck-nec | 07:40 |
kevinbenton | amotoki: had a failure on https://review.openstack.org/#/c/114379/ | 07:40 |
kevinbenton | amotoki: that i doubt was related because the change was all isolated to the Big Switch code | 07:40 |
*** chandankumar has joined #openstack-neutron | 07:41 | |
amotoki | kevinbenton: ah... let me check it is temporary or not. | 07:41 |
*** mitarun has quit IRC | 07:42 | |
*** yamamoto has quit IRC | 07:44 | |
jschwarz | amotoki, morning :) | 07:45 |
amotoki | kevinbenton: two failures occurred due to kombu not found. It seems temporary problem on pypi mirror or our proxy. | 07:45 |
jschwarz | amotoki, could you possibly take a look at https://review.openstack.org/#/c/116832/ ? You +2'd it before and I made some changes to the code :) | 07:47 |
amotoki | jschwarz: sure | 07:47 |
jschwarz | amotoki, thanks :) | 07:47 |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: Make Juno migrations config independent https://review.openstack.org/109768 | 07:51 |
*** Altran has joined #openstack-neutron | 07:51 | |
*** nati_uen_ has quit IRC | 07:55 | |
*** nati_ueno has joined #openstack-neutron | 07:55 | |
*** rm_work|away is now known as rm_work | 07:56 | |
*** _cjones_ has joined #openstack-neutron | 08:00 | |
*** rook has joined #openstack-neutron | 08:00 | |
*** shakamunyi has joined #openstack-neutron | 08:00 | |
*** karimb has joined #openstack-neutron | 08:01 | |
*** JayJ has joined #openstack-neutron | 08:02 | |
*** mitarun has joined #openstack-neutron | 08:03 | |
*** skolekonov has joined #openstack-neutron | 08:03 | |
*** tsg has quit IRC | 08:03 | |
*** _cjones_ has quit IRC | 08:04 | |
*** ygbo has joined #openstack-neutron | 08:04 | |
*** rook has quit IRC | 08:04 | |
*** shakamunyi has quit IRC | 08:05 | |
*** JayJ has quit IRC | 08:06 | |
*** tomoe_ has quit IRC | 08:07 | |
*** tomoe_ has joined #openstack-neutron | 08:08 | |
*** Longgeek has quit IRC | 08:09 | |
*** Longgeek has joined #openstack-neutron | 08:10 | |
kevinbenton | amotoki: thanks | 08:11 |
*** luqas has quit IRC | 08:12 | |
*** yamamoto has joined #openstack-neutron | 08:12 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Adds external attachment type validators to ML2 https://review.openstack.org/116187 | 08:15 |
*** sungju has joined #openstack-neutron | 08:18 | |
*** nati_uen_ has joined #openstack-neutron | 08:18 | |
*** luqas has joined #openstack-neutron | 08:19 | |
*** luqas has quit IRC | 08:22 | |
*** nati_ueno has quit IRC | 08:22 | |
*** matrohon has joined #openstack-neutron | 08:23 | |
*** luqas has joined #openstack-neutron | 08:23 | |
*** padkrish has joined #openstack-neutron | 08:25 | |
*** Dafna has joined #openstack-neutron | 08:28 | |
*** sungju has quit IRC | 08:29 | |
*** padkrish has quit IRC | 08:29 | |
*** kopparam has quit IRC | 08:30 | |
*** kopparam has joined #openstack-neutron | 08:31 | |
*** cnesa9 has joined #openstack-neutron | 08:38 | |
*** amuller has joined #openstack-neutron | 08:38 | |
*** Longgeek has quit IRC | 08:39 | |
*** Longgeek has joined #openstack-neutron | 08:39 | |
*** Longgeek has quit IRC | 08:44 | |
*** lazy_prince is now known as killer_prince | 08:45 | |
*** killer_prince is now known as lazy_prince | 08:45 | |
amuller | amotoki: Good evening | 08:46 |
amuller | amotoki: The keepalive patch: https://review.openstack.org/#/c/68142/ | 08:46 |
amuller | has +2 from Maru | 08:46 |
*** Longgeek has joined #openstack-neutron | 08:46 | |
amuller | amotoki: I thought maybe you'd like to take a look :) | 08:47 |
amotoki | amuller: wait for a while. will visit after finishing the current one. | 08:47 |
*** yamamoto has quit IRC | 08:47 | |
amuller | ok | 08:47 |
*** k4n0 has quit IRC | 08:47 | |
amuller | amotoki: Also I see your DVR patches to Horizon were merged | 08:48 |
amuller | amotoki: I wanted to ask if you're planning on doing something simple for HA? Like admins seeing if a router is HA or not? | 08:48 |
amotoki | amuller: yes. it is in my mind. | 08:48 |
amuller | amotoki: ok cool | 08:49 |
amuller | amotoki: hopefully we'll merge all of the patches very soon | 08:49 |
amotoki | amuller: it is small enough so we can do it after J-3 cut. | 08:49 |
amuller | yep | 08:49 |
amuller | amotoki: Thanks for your work Akihiro | 08:51 |
*** yamamoto has joined #openstack-neutron | 08:52 | |
openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Add functional test for IptablesManager https://review.openstack.org/117852 | 08:53 |
*** salv-orlando has quit IRC | 08:55 | |
*** aix has joined #openstack-neutron | 08:56 | |
openstackgerrit | Sridar Kandaswamy proposed a change to openstack/neutron: Changes to support FWaaS in a DVR based environment https://review.openstack.org/113359 | 08:59 |
*** rook has joined #openstack-neutron | 09:01 | |
*** shakamunyi has joined #openstack-neutron | 09:01 | |
*** rm_work is now known as rm_work|away | 09:01 | |
*** chandankumar has quit IRC | 09:02 | |
*** JayJ has joined #openstack-neutron | 09:02 | |
*** dguerri`afk is now known as dguerri | 09:04 | |
trinaths | Hi all | 09:05 |
*** rook has quit IRC | 09:05 | |
*** shakamunyi has quit IRC | 09:05 | |
*** k4n0 has joined #openstack-neutron | 09:06 | |
*** jp_at_hp has joined #openstack-neutron | 09:07 | |
*** zilberstein has joined #openstack-neutron | 09:07 | |
zilberstein | hello #openstack-neutron | 09:07 |
zilberstein | Is there's a way to assign floating IP to VPS without creating allocation pool for network ? | 09:07 |
*** ivar-lazzaro has joined #openstack-neutron | 09:08 | |
*** safchain has joined #openstack-neutron | 09:08 | |
*** JayJ has quit IRC | 09:08 | |
*** luqas has quit IRC | 09:10 | |
*** nati_uen_ has quit IRC | 09:10 | |
*** nati_ueno has joined #openstack-neutron | 09:11 | |
*** roeyc has quit IRC | 09:11 | |
*** yamamoto has quit IRC | 09:14 | |
*** yamamoto has joined #openstack-neutron | 09:15 | |
*** yamamoto has quit IRC | 09:15 | |
*** ijw has joined #openstack-neutron | 09:15 | |
*** yamamoto has joined #openstack-neutron | 09:16 | |
*** salv-orlando has joined #openstack-neutron | 09:17 | |
*** mitarun has quit IRC | 09:17 | |
*** mitarun has joined #openstack-neutron | 09:18 | |
*** marun has joined #openstack-neutron | 09:18 | |
*** ijw has quit IRC | 09:20 | |
*** nati_uen_ has joined #openstack-neutron | 09:20 | |
*** mitarun has quit IRC | 09:22 | |
zilberstein | what exactly neutron floatingip-create --fixed-ip-address do ? | 09:22 |
openstackgerrit | Sam Betts proposed a change to openstack/neutron: Added TAP_DEVICE_PREFIX info to common/constants https://review.openstack.org/116970 | 09:22 |
openstackgerrit | Trinath Somanchi proposed a change to openstack/neutron: Freescale FWaaS Plugin https://review.openstack.org/109659 | 09:23 |
*** nati_ueno has quit IRC | 09:24 | |
*** lori is now known as lori|away | 09:30 | |
*** cnesa9 has quit IRC | 09:30 | |
*** geekinut1h has quit IRC | 09:30 | |
*** geekinutah has joined #openstack-neutron | 09:31 | |
*** mitarun has joined #openstack-neutron | 09:31 | |
*** mitarun has quit IRC | 09:34 | |
*** xuhanp has joined #openstack-neutron | 09:34 | |
*** mitarun has joined #openstack-neutron | 09:34 | |
*** mitarun has quit IRC | 09:39 | |
*** sambetts has joined #openstack-neutron | 09:42 | |
*** luqas has joined #openstack-neutron | 09:42 | |
*** jistr|doc is now known as jistr | 09:42 | |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: Introduce havana initial state https://review.openstack.org/112732 | 09:43 |
*** dave_tucker_zzz is now known as dave_tucker | 09:51 | |
*** _cjones_ has joined #openstack-neutron | 09:56 | |
amotoki | jschwarz: I just commented to your patch. I am not sure why we need to use defaultdict rather than set(). | 09:58 |
amuller | amotoki: thanks for the approval | 10:00 |
*** xuhanp has quit IRC | 10:00 | |
*** k4n0 has quit IRC | 10:00 | |
*** _cjones_ has quit IRC | 10:01 | |
*** rook has joined #openstack-neutron | 10:01 | |
*** shakamunyi has joined #openstack-neutron | 10:02 | |
*** ZZelle has joined #openstack-neutron | 10:02 | |
*** mitarun has joined #openstack-neutron | 10:04 | |
*** claudiub has joined #openstack-neutron | 10:05 | |
*** amotoki has quit IRC | 10:05 | |
*** yamamoto has quit IRC | 10:06 | |
*** shakamunyi has quit IRC | 10:07 | |
*** sbfox has quit IRC | 10:09 | |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: Remove logic for conditional migrations https://review.openstack.org/117913 | 10:09 |
*** afazekas has joined #openstack-neutron | 10:10 | |
*** kopparam_ has joined #openstack-neutron | 10:11 | |
*** rook has quit IRC | 10:13 | |
*** kopparam has quit IRC | 10:13 | |
openstackgerrit | Trinath Somanchi proposed a change to openstack/neutron: Freescale FWaaS Plugin https://review.openstack.org/109659 | 10:13 |
*** k4n0 has joined #openstack-neutron | 10:13 | |
*** yamamoto has joined #openstack-neutron | 10:14 | |
*** yamamoto has quit IRC | 10:16 | |
*** yamamoto has joined #openstack-neutron | 10:16 | |
*** luqas has quit IRC | 10:17 | |
*** mitarun has quit IRC | 10:17 | |
*** luqas has joined #openstack-neutron | 10:17 | |
*** kopparam_ has quit IRC | 10:17 | |
*** kopparam has joined #openstack-neutron | 10:18 | |
*** tomoe__ has joined #openstack-neutron | 10:20 | |
*** Guest13287 is now known as puck | 10:20 | |
*** tomoe__ has quit IRC | 10:21 | |
*** tomoe_ has quit IRC | 10:23 | |
*** kopparam has quit IRC | 10:23 | |
*** roeyc has joined #openstack-neutron | 10:25 | |
*** jistr has quit IRC | 10:25 | |
*** yamamoto has quit IRC | 10:25 | |
*** mitarun has joined #openstack-neutron | 10:26 | |
*** alexpilotti has joined #openstack-neutron | 10:28 | |
*** yfried_ has joined #openstack-neutron | 10:32 | |
openstackgerrit | Vinod Kumar proposed a change to openstack/neutron: RESTful API to retrieve dvrhostmac for ODL https://review.openstack.org/118126 | 10:32 |
*** luqas has quit IRC | 10:33 | |
*** yfried has quit IRC | 10:34 | |
*** yamahata has quit IRC | 10:41 | |
*** mitarun has quit IRC | 10:44 | |
*** mitarun has joined #openstack-neutron | 10:44 | |
*** Longgeek has quit IRC | 10:46 | |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: NSX: Add a foreign key for gateway devices https://review.openstack.org/93826 | 10:47 |
*** mitarun has quit IRC | 10:48 | |
*** alexpilotti has quit IRC | 10:50 | |
openstackgerrit | Rossella Sblendido proposed a change to openstack/neutron: Remove locking from network and subnet delete op https://review.openstack.org/115360 | 10:51 |
*** _cjones_ has joined #openstack-neutron | 10:55 | |
openstackgerrit | Trinath Somanchi proposed a change to openstack/neutron: Freescale FWaaS Plugin https://review.openstack.org/109659 | 10:55 |
*** tomoe_ has joined #openstack-neutron | 10:55 | |
*** _cjones_ has quit IRC | 11:00 | |
*** rook has joined #openstack-neutron | 11:02 | |
*** shakamunyi has joined #openstack-neutron | 11:03 | |
*** irenab has quit IRC | 11:03 | |
*** Longgeek has joined #openstack-neutron | 11:05 | |
*** JayJ has joined #openstack-neutron | 11:06 | |
*** lori|away is now known as lori | 11:06 | |
*** rook has quit IRC | 11:07 | |
*** yfried_ has quit IRC | 11:07 | |
*** shakamunyi has quit IRC | 11:07 | |
openstackgerrit | Assaf Muller proposed a change to openstack/neutron: Add L3 VRRP HA base classes https://review.openstack.org/64553 | 11:09 |
openstackgerrit | Assaf Muller proposed a change to openstack/neutron: Add a new scheduler for the l3 HA https://review.openstack.org/66347 | 11:09 |
*** JayJ has quit IRC | 11:10 | |
*** luqas has joined #openstack-neutron | 11:12 | |
*** tomoe_ has quit IRC | 11:12 | |
*** kopparam has joined #openstack-neutron | 11:14 | |
*** yamamoto has joined #openstack-neutron | 11:15 | |
*** popw has quit IRC | 11:15 | |
*** yamamoto_ has joined #openstack-neutron | 11:16 | |
*** yamamoto has quit IRC | 11:17 | |
*** prasoon has quit IRC | 11:17 | |
*** ijw has joined #openstack-neutron | 11:17 | |
openstackgerrit | A change was merged to openstack/neutron: ofagent: merge br-tun into br-int https://review.openstack.org/112488 | 11:20 |
openstackgerrit | A change was merged to openstack/neutron: ofagent: Enable local arp responder for local VMs https://review.openstack.org/113160 | 11:20 |
*** yamamoto_ has quit IRC | 11:21 | |
*** balles has quit IRC | 11:21 | |
*** ijw has quit IRC | 11:22 | |
*** sballe has joined #openstack-neutron | 11:22 | |
*** nati_uen_ has quit IRC | 11:23 | |
*** yfried_ has joined #openstack-neutron | 11:24 | |
openstackgerrit | John Schwarz proposed a change to openstack/neutron: Refactor model_query_hooks https://review.openstack.org/117142 | 11:25 |
openstackgerrit | John Schwarz proposed a change to openstack/neutron: Allow filtering routers by network-ids https://review.openstack.org/111580 | 11:25 |
*** chandankumar has joined #openstack-neutron | 11:26 | |
jschwarz | salv-orlando, morning :) Just sent new patch sets to the model_query_hook refactor (https://review.openstack.org/117142) if you get a chance to look at it sometime :) | 11:28 |
*** kopparam has quit IRC | 11:28 | |
*** kopparam has joined #openstack-neutron | 11:29 | |
*** baohua has quit IRC | 11:30 | |
*** rook has joined #openstack-neutron | 11:34 | |
*** baoli has joined #openstack-neutron | 11:35 | |
*** baoli has quit IRC | 11:35 | |
*** baoli has joined #openstack-neutron | 11:36 | |
openstackgerrit | Preeti Mirji proposed a change to openstack/neutron: Adm policy can't use other tenants unshared rules https://review.openstack.org/108952 | 11:37 |
*** yamahata has joined #openstack-neutron | 11:38 | |
*** chandankumar has quit IRC | 11:39 | |
*** Longgeek has quit IRC | 11:39 | |
openstackgerrit | Vinod Kumar proposed a change to openstack/neutron: RESTful API to retrieve dvrhostmac for ODL https://review.openstack.org/118126 | 11:40 |
*** Longgeek has joined #openstack-neutron | 11:40 | |
*** Longgeek has quit IRC | 11:41 | |
*** luqas has quit IRC | 11:41 | |
openstackgerrit | Robert Kukura proposed a change to openstack/neutron: ML2: Fix release of network segments to allocation pools https://review.openstack.org/118140 | 11:41 |
*** Longgeek has joined #openstack-neutron | 11:41 | |
*** nati_ueno has joined #openstack-neutron | 11:42 | |
*** Longgeek_ has joined #openstack-neutron | 11:42 | |
*** Longgeek_ has quit IRC | 11:42 | |
*** Longgeek_ has joined #openstack-neutron | 11:42 | |
*** nati_ueno has quit IRC | 11:43 | |
*** nati_ueno has joined #openstack-neutron | 11:44 | |
rkukura | salv-orlando, nati_ueno: Can you take a quick look at https://review.openstack.org/118140, which fixes a serious issue introduced by a patch that merged yesterday? | 11:45 |
*** alexpilotti has joined #openstack-neutron | 11:45 | |
nati_ueno | rkukura: sure! | 11:46 |
rkukura | nati_ueno: Thanks! | 11:46 |
*** Longgeek has quit IRC | 11:46 | |
*** ZZelle has quit IRC | 11:48 | |
*** jab has joined #openstack-neutron | 11:52 | |
*** jab has quit IRC | 11:52 | |
*** jab has joined #openstack-neutron | 11:52 | |
*** prasoon has joined #openstack-neutron | 11:54 | |
*** rook has quit IRC | 11:55 | |
*** popw has joined #openstack-neutron | 11:58 | |
*** drankis has joined #openstack-neutron | 12:01 | |
*** shakamunyi has joined #openstack-neutron | 12:03 | |
*** yamamoto has joined #openstack-neutron | 12:04 | |
*** gildub has quit IRC | 12:06 | |
*** boris-42_ is now known as boris-42 | 12:06 | |
openstackgerrit | Liping Mao proposed a change to openstack/neutron: Neutron metering do not check overlap ip range https://review.openstack.org/116652 | 12:07 |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: l2pop: Allow network types overridable https://review.openstack.org/112947 | 12:07 |
*** shakamunyi has quit IRC | 12:08 | |
*** markvoelker has joined #openstack-neutron | 12:09 | |
*** nati_ueno has quit IRC | 12:11 | |
*** sballe has quit IRC | 12:11 | |
*** diegows has joined #openstack-neutron | 12:12 | |
*** Longgeek_ has quit IRC | 12:14 | |
*** amotoki has joined #openstack-neutron | 12:17 | |
*** Longgeek has joined #openstack-neutron | 12:17 | |
*** naotokl is now known as zz_naotokl | 12:20 | |
openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: Don't spawn metadata-proxy for non-isolated nets https://review.openstack.org/116832 | 12:22 |
*** kopparam has quit IRC | 12:22 | |
*** kopparam has joined #openstack-neutron | 12:23 | |
openstackgerrit | Assaf Muller proposed a change to openstack/neutron: Add HA support to the l3 agent https://review.openstack.org/70700 | 12:24 |
openstackgerrit | Assaf Muller proposed a change to openstack/neutron: Test HA router failover https://review.openstack.org/117994 | 12:24 |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Enable local arp responder for TYPE_FLAT https://review.openstack.org/114119 | 12:24 |
trinaths | When I check out the latest code, for my unit testing, I get the error "NoSuchOptError: no such option: router_distributed". | 12:25 |
trinaths | I'm any new configuration ? an DVR guys around ? | 12:25 |
*** roeyc has quit IRC | 12:26 | |
jschwarz | trinaths, where do you get that error? | 12:26 |
*** kopparam has quit IRC | 12:27 | |
trinaths | while unit testing. | 12:28 |
trinaths | near create_firewall | 12:28 |
*** luqas has joined #openstack-neutron | 12:29 | |
*** ZZelle has joined #openstack-neutron | 12:29 | |
trinaths | jschwarz: Here is the complete log. http://paste.openstack.org/show/104196/ | 12:29 |
jschwarz | trinaths, beats me :< sorry | 12:30 |
trinaths | jschwarz: in neutron/common/config.py, there is not config option for router_distributed | 12:31 |
trinaths | jschwarz: If we give a place to options "router_distributed". All unit tests pass. | 12:32 |
jschwarz | trinaths, I'm afraid I'm not that much into DVR at all... | 12:33 |
jschwarz | I remember having a similar problem a while back but don't remember what I did to solve it | 12:34 |
trinaths | jschwarz: [Doubt] For any options in neutron.conf, is it mandatory to have cfg.CONF option in /neutron/common/config.py ? | 12:35 |
trinaths | amotoki: ping | 12:35 |
jschwarz | trinaths, I think they can be placed in other files as well, but really not sure | 12:35 |
trinaths | jschwarz: True, but I have found this config option anywhere in neutron code. | 12:36 |
jschwarz | trinaths, I found it in ./neutron/db/l3_dvr_db.py, line 40 | 12:36 |
trinaths | jschwarz: okay. seen it. But since its a global option, why can't it be moved to config.py | 12:38 |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: NSX: Validate gateway device list against DB https://review.openstack.org/118148 | 12:39 |
jschwarz | trinaths, I don't think it's a global option... If DVR is disable, why should there be a router_distirubted configuration? | 12:39 |
trinaths | jschwarz: If, its present in neutron.conf, and not present in config.py. my unit testing doesn't need this DVR config. and its gives me such errors | 12:41 |
*** trinaths has quit IRC | 12:42 | |
*** ZZelle has quit IRC | 12:43 | |
*** prasoon has quit IRC | 12:45 | |
*** prasoon has joined #openstack-neutron | 12:46 | |
*** garyk has quit IRC | 12:47 | |
*** mitarun has joined #openstack-neutron | 12:47 | |
*** irenab has joined #openstack-neutron | 12:50 | |
*** mitarun has quit IRC | 12:51 | |
*** stanzgy has quit IRC | 12:52 | |
Altran | jschwarz: thanks for Your reviews, about https://review.openstack.org/#/c/117219 I think that Your suggestion about logging number of probes cleared is valid and not out of scope of patch - do You think it's enough to log "%d probe(s) cleared" or distinguish 0 and non 0 ? | 12:55 |
*** sballe has joined #openstack-neutron | 12:55 | |
jschwarz | Altran, the more information (especially when it's a %d information), the merrier IMO | 12:55 |
jschwarz | Altran, also your code was nice to review :) | 12:55 |
*** _cjones_ has joined #openstack-neutron | 12:56 | |
Altran | jschwarz: well they were simple fixes after all :) | 12:56 |
jschwarz | XD | 12:56 |
jschwarz | Altran, if you are adding a counter there, then I think a renaming of that function's name is a must | 12:57 |
Altran | jschwarz: is it preferred to use singular in class names ? because its named ClearProbe | 12:59 |
*** lazy_prince is now known as killer_prince | 12:59 | |
*** baoli has quit IRC | 13:00 | |
*** baoli has joined #openstack-neutron | 13:00 | |
*** _cjones_ has quit IRC | 13:00 | |
*** JayJ has joined #openstack-neutron | 13:01 | |
jschwarz | Altran, I'd rename that as well to something like ProbeClearer | 13:01 |
jschwarz | or something more elegant | 13:01 |
jschwarz | ClearProbe also hints that only one probe is cleared... | 13:01 |
Altran | yeah, but there is ListProbe in the same file which has even less sense :D | 13:04 |
*** Longgeek has quit IRC | 13:04 | |
*** shakamunyi has joined #openstack-neutron | 13:04 | |
*** Longgeek has joined #openstack-neutron | 13:04 | |
*** JayJ_ has joined #openstack-neutron | 13:09 | |
*** shakamunyi has quit IRC | 13:09 | |
openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: Fix comments in api.rpc.handlers https://review.openstack.org/118161 | 13:09 |
*** JayJ has quit IRC | 13:10 | |
jschwarz | Altran, I really don't know... it's starting to seem like quite a bit of renaming, so if I were you I'd try to put it in one patch set with a good excuse for the renaming | 13:10 |
jschwarz | Altran, otherwise might be best to have 2 request changes with one of them doing the renaming | 13:11 |
jschwarz | Altran, but I really don't know ;-< | 13:11 |
Altran | jschwarz: I think I'll rename the method but leave class alone - will see what other reviewers will say about it | 13:13 |
jschwarz | Altran, sounds like a plan :) | 13:14 |
*** rotbeard has quit IRC | 13:15 | |
Altran | hmm by the way I just thought I might consider changing my nick on IRC - I've had it registered from non OpenStack related use but it might be not optimal when someone tries to find me on IRC to discuss something | 13:19 |
*** mitarun has joined #openstack-neutron | 13:20 | |
jschwarz | lol yeah, I was quite surprised to find out you were Jacek :P | 13:20 |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: NSX: Add a foreign key for gateway devices https://review.openstack.org/93826 | 13:20 |
jschwarz | You can identify to a different nick than the one you're using | 13:21 |
*** Altran is now known as jswiders | 13:21 | |
*** irenab has quit IRC | 13:21 | |
jschwarz | jswiders++ | 13:23 |
jswiders | :) | 13:23 |
jschwarz | but now js<tab> doesn't always complete to me... :< | 13:23 |
*** JayJ_ has quit IRC | 13:24 | |
jswiders | hehe ^^ | 13:24 |
*** JayJ_ has joined #openstack-neutron | 13:26 | |
*** JayJ_ has quit IRC | 13:28 | |
openstackgerrit | Jacek Swiderski proposed a change to openstack/neutron: Clarify message when no probes are cleared https://review.openstack.org/117219 | 13:32 |
rkukura | salv-orlando, amotoki: Thanks for the quick +2s on https://review.openstack.org/#/c/118140/. Looks like jenkins failed in https://review.openstack.org/#/c/118140/, which I’ve been seeing recently and does not seem possibly related to this patch. Should we go ahead and +A now, or recheck first? | 13:33 |
rkukura | salv-orlando, amotoki: The jenkins failure is tempest.scenario.test_network_advanced_server_ops.TestNetworkAdvancedServerOps.test_server_connectivity_resize in http://logs.openstack.org/40/118140/1/check/check-tempest-dsvm-neutron-full-2/79d3e03/console.html. | 13:33 |
amuller | rkukura: I've seen that fail randomly recently as well | 13:34 |
amuller | unrelated to patch | 13:34 |
amotoki | rkukura: actually both are same. If Jenkins post -1, check job will be run regardless of +A. | 13:34 |
rkukura | amotoki, amuller, salv-orlando: Do we have a bug associated with this random failure mode? | 13:35 |
*** salv-orlando has quit IRC | 13:35 | |
openstackgerrit | Assaf Muller proposed a change to openstack/neutron: Add HA support to the l3 agent https://review.openstack.org/70700 | 13:37 |
openstackgerrit | Assaf Muller proposed a change to openstack/neutron: Test HA router failover https://review.openstack.org/117994 | 13:37 |
amotoki | rkukura: I don't know if we have or not. checking logstash. | 13:38 |
*** silverpuff has quit IRC | 13:39 | |
*** simon-AS559 has quit IRC | 13:39 | |
openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: Fix a bug in Mellanox plugin RPC caused by secgroup RPC refactoring https://review.openstack.org/118165 | 13:40 |
jswiders | rkukura: https://bugs.launchpad.net/grenade/+bug/1349617 seems to be it (its also no1 in elastic recheck) | 13:40 |
*** tomoe_ has joined #openstack-neutron | 13:41 | |
*** seizadi has joined #openstack-neutron | 13:41 | |
rkukura | jswiders: thanks | 13:43 |
jswiders | btw starting this week, weekly meetings are in new time slot ? | 13:44 |
rkukura | jswiders: I thought that started after feature freeze, so next week | 13:45 |
jswiders | rkukura: oh right - thanks :) | 13:46 |
*** Administrator_ has joined #openstack-neutron | 13:46 | |
*** bvandenh has joined #openstack-neutron | 13:47 | |
openstackgerrit | A change was merged to openstack/neutron: Introduces a keepalived manager for HA https://review.openstack.org/68142 | 13:48 |
*** openstack has joined #openstack-neutron | 13:57 | |
*** openstackstatus has joined #openstack-neutron | 13:57 | |
*** ChanServ sets mode: +v openstackstatus | 13:57 | |
openstackgerrit | Jacek Swiderski proposed a change to openstack/neutron: Clarify message when no probes are cleared https://review.openstack.org/117219 | 13:58 |
*** jab has quit IRC | 14:00 | |
*** _cjones_ has quit IRC | 14:00 | |
*** jab has joined #openstack-neutron | 14:02 | |
*** jab has quit IRC | 14:02 | |
*** jab has joined #openstack-neutron | 14:02 | |
*** shakamunyi has joined #openstack-neutron | 14:05 | |
*** lxsli has left #openstack-neutron | 14:06 | |
*** yfauser has joined #openstack-neutron | 14:08 | |
*** yfauser has left #openstack-neutron | 14:08 | |
*** yfried_ has quit IRC | 14:09 | |
*** roeyc has joined #openstack-neutron | 14:09 | |
*** shakamunyi has quit IRC | 14:09 | |
*** jprovazn has joined #openstack-neutron | 14:10 | |
*** sballe has joined #openstack-neutron | 14:12 | |
*** simon-AS559 has joined #openstack-neutron | 14:12 | |
*** balles has joined #openstack-neutron | 14:13 | |
*** Longgeek has quit IRC | 14:15 | |
openstackgerrit | A change was merged to openstack/neutron: Apic drivers enhancements (second approach): Sync https://review.openstack.org/116157 | 14:15 |
openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Add functional test for IptablesManager https://review.openstack.org/117852 | 14:16 |
*** mlavalle has joined #openstack-neutron | 14:18 | |
openstackgerrit | Arvind Somya proposed a change to openstack/neutron: ML2 Type Driver refactor part 3 https://review.openstack.org/115151 | 14:20 |
openstackgerrit | Jacek Swiderski proposed a change to openstack/neutron: Fix condition for IPv6 attributes validation https://review.openstack.org/117799 | 14:20 |
*** Longgeek has joined #openstack-neutron | 14:21 | |
*** BillTheKat has joined #openstack-neutron | 14:23 | |
*** mlavalle has left #openstack-neutron | 14:24 | |
yamamoto | rkukura: ping | 14:26 |
rkukura | yamamoto: hi | 14:26 |
yamamoto | rkukura: do you have any comments on https://review.openstack.org/#/c/112947/ ? | 14:26 |
yamamoto | rkukura: nachi suggested to ask your input | 14:27 |
rkukura | yamamoto: I glanced at it earlier, but will take a closer look ASAP | 14:27 |
yamamoto | rkukura: thank you! | 14:27 |
*** LiberTerra has joined #openstack-neutron | 14:34 | |
*** miguel1 has joined #openstack-neutron | 14:34 | |
rkukura | yamamoto: LGTM! Any specific concerns? | 14:35 |
openstackgerrit | Rossella Sblendido proposed a change to openstack/neutron: Remove locking from network and subnet delete op https://review.openstack.org/115360 | 14:35 |
*** aix has quit IRC | 14:36 | |
yamamoto | rkukura: no specific concerns afaik. nachi suggested to wait for your input because it's ml2 change. | 14:36 |
*** k4n0 has quit IRC | 14:37 | |
rkukura | yamamoto: Gave my +2/+A | 14:38 |
yamamoto | rkukura: thank you! | 14:39 |
*** dgollub has joined #openstack-neutron | 14:39 | |
rkukura | yamamoto: no problem! | 14:39 |
miguel1 | -- | 14:40 |
*** miguel1 has quit IRC | 14:41 | |
*** miguel1 has joined #openstack-neutron | 14:42 | |
jschwarz | jswiders, I added a comment to your ipv6 patch set | 14:42 |
*** miguel1 has quit IRC | 14:43 | |
jschwarz | jswiders, let me know if you don't agree with me :) | 14:43 |
*** miguel1 has joined #openstack-neutron | 14:43 | |
*** tomoe_ has quit IRC | 14:44 | |
*** leenheer has joined #openstack-neutron | 14:46 | |
jswiders | jschwarz: I was about to say why are You so negative with "if you don't agree" from start - but I do check situation with both ra_mode and address_mode - rightfully (or probably not) when I was refactoring my tests to helper function, I've also used it for test_update_subnet_ipv6_attributes from which I originated my tests | 14:47 |
*** yamamoto has quit IRC | 14:47 | |
jschwarz | jswiders, ah yes, you are right :) | 14:49 |
jschwarz | jswiders, one sec | 14:49 |
jschwarz | jswiders, better? :) | 14:49 |
*** miguel1 has quit IRC | 14:49 | |
*** mlavalle_ has joined #openstack-neutron | 14:50 | |
jswiders | jschwarz: yup :) | 14:50 |
*** mlavalle_ has quit IRC | 14:50 | |
jschwarz | jswiders, i'm trying to be positive (sorry if it doesn't work all the time).. if I was negative you'd know ;-) | 14:50 |
*** tomoe_ has joined #openstack-neutron | 14:50 | |
*** dave_tucker is now known as dave_tucker_zzz | 14:51 | |
*** salv-orlando has joined #openstack-neutron | 14:51 | |
jswiders | jschwarz: if I didn't know that You are not him I would ask You if You are one of my friends - he would also apologize when I say he's to negative sometimes | 14:52 |
jschwarz | jswiders, lol... can't help but being nice ;-> | 14:53 |
jswiders | jschwarz: well either way thanks for reviews :) | 14:53 |
jschwarz | jswiders, sure thing... let me know if you have some more down the line | 14:54 |
*** sbfox has joined #openstack-neutron | 14:55 | |
*** _cjones_ has joined #openstack-neutron | 14:56 | |
*** HenryG has joined #openstack-neutron | 14:57 | |
*** BillTheKat has quit IRC | 14:59 | |
*** sbfox has quit IRC | 14:59 | |
*** _cjones_ has quit IRC | 15:01 | |
*** aix has joined #openstack-neutron | 15:01 | |
*** jschwarz has quit IRC | 15:01 | |
*** trinaths has joined #openstack-neutron | 15:02 | |
*** mspreitz has joined #openstack-neutron | 15:03 | |
*** tomoe_ has quit IRC | 15:05 | |
*** shakamunyi has joined #openstack-neutron | 15:06 | |
*** jswiders has quit IRC | 15:07 | |
trinaths | Hi | 15:07 |
trinaths | amotoki: ping, if you are free we can discuss. | 15:07 |
amotoki | trinaths: it's too late to me. now going to bed. | 15:08 |
trinaths | amotoki: okay, np, Good Night | 15:08 |
*** lori is now known as lori|away | 15:10 | |
*** shakamunyi has quit IRC | 15:10 | |
rkukura | amotoki: Jenkins passed this time, so do you want to do the +A on https://review.openstack.org/#/c/118140/? | 15:10 |
openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Implements ProcessMonitor to watch over external processes https://review.openstack.org/112798 | 15:10 |
rkukura | amotoki: Thanks! | 15:11 |
amotoki | rkukura: nice! approved. | 15:11 |
*** sballe has quit IRC | 15:13 | |
*** jogo_ is now known as jogo | 15:14 | |
*** mitarun has quit IRC | 15:14 | |
*** Longgeek_ has joined #openstack-neutron | 15:16 | |
*** markvoel_ has joined #openstack-neutron | 15:19 | |
*** Longgeek has quit IRC | 15:19 | |
*** markvoel_ has quit IRC | 15:19 | |
*** SridharG has left #openstack-neutron | 15:23 | |
*** amotoki has quit IRC | 15:26 | |
*** cnesa9 has joined #openstack-neutron | 15:28 | |
*** mitarun has joined #openstack-neutron | 15:28 | |
*** r-daneel has joined #openstack-neutron | 15:29 | |
*** bvandenh has quit IRC | 15:29 | |
*** r-daneel has quit IRC | 15:29 | |
*** r-daneel has joined #openstack-neutron | 15:29 | |
openstackgerrit | enikanorov proposed a change to openstack/neutron: Fix setup.cfg so setup could succeed https://review.openstack.org/118176 | 15:31 |
*** sgordon_ has joined #openstack-neutron | 15:31 | |
*** a_le has quit IRC | 15:34 | |
*** mitarun has quit IRC | 15:34 | |
*** mitarun has joined #openstack-neutron | 15:34 | |
*** sbfox has joined #openstack-neutron | 15:36 | |
*** pasquier-s has quit IRC | 15:38 | |
*** mitarun has quit IRC | 15:39 | |
*** _cjones_ has joined #openstack-neutron | 15:40 | |
*** seizadi has quit IRC | 15:40 | |
*** a_le has joined #openstack-neutron | 15:40 | |
*** padkrish has joined #openstack-neutron | 15:41 | |
*** padkrish has joined #openstack-neutron | 15:42 | |
*** _cjones_ has quit IRC | 15:43 | |
*** mlavalle_ has joined #openstack-neutron | 15:43 | |
*** _cjones_ has joined #openstack-neutron | 15:43 | |
mlavalle_ | mestery: what time are we meeting today? | 15:43 |
*** ryu25 has joined #openstack-neutron | 15:43 | |
*** mestery_ has joined #openstack-neutron | 15:45 | |
mlavalle_ | meta2-18~ /window -1 | 15:45 |
*** mestery has quit IRC | 15:46 | |
*** yamamoto has joined #openstack-neutron | 15:47 | |
*** _cjones_ has quit IRC | 15:48 | |
*** balles has quit IRC | 15:48 | |
marun | mlavalle_: at late o'clock for all of us on the other side of utc | 15:48 |
marun | mlavalle_: the alternate time doesn't start till next week | 15:48 |
mlavalle_ | marun: yeah, that's really what I wanted to know :-) | 15:49 |
marun | mlavalle_: :) | 15:49 |
*** roeyc has quit IRC | 15:51 | |
*** yamamoto has quit IRC | 15:51 | |
*** mlavalle_ has quit IRC | 15:54 | |
*** skolekonov has quit IRC | 15:54 | |
*** mlavalle_ has joined #openstack-neutron | 15:54 | |
*** thomasem has joined #openstack-neutron | 15:56 | |
*** dbite has quit IRC | 15:57 | |
*** mitarun has joined #openstack-neutron | 15:58 | |
*** devvesa has quit IRC | 16:04 | |
*** shakamunyi has joined #openstack-neutron | 16:06 | |
*** shakamunyi has quit IRC | 16:11 | |
*** otherwiseguy has joined #openstack-neutron | 16:13 | |
*** leenheer has quit IRC | 16:14 | |
*** ZZelle has joined #openstack-neutron | 16:18 | |
openstackgerrit | Jacek Swiderski proposed a change to openstack/neutron: Clarify message when no probes are cleared https://review.openstack.org/117219 | 16:22 |
*** matrohon has quit IRC | 16:22 | |
*** ryu25 has quit IRC | 16:22 | |
*** aix has quit IRC | 16:23 | |
*** simon-AS559 has quit IRC | 16:25 | |
*** baoli has joined #openstack-neutron | 16:25 | |
*** salv-orlando has quit IRC | 16:25 | |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy API-2: Classifiers, Actions, and Rules https://review.openstack.org/101816 | 16:28 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy Mapping API-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/100716 | 16:28 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy Mapping Plugin-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/101957 | 16:28 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy Plugin-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/96393 | 16:28 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy: Resource Mapping Driver (2) https://review.openstack.org/113775 | 16:28 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy Plugin-2: Classifiers, Actions, and Rules https://review.openstack.org/103465 | 16:28 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy API-3: Contracts https://review.openstack.org/103486 | 16:28 |
openstackgerrit | Stephen Wong proposed a change to openstack/neutron: Group Policy API-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/95900 | 16:28 |
*** otherwiseguy has quit IRC | 16:30 | |
*** dfarrell07 has joined #openstack-neutron | 16:30 | |
*** padkrish has quit IRC | 16:31 | |
*** ijw has joined #openstack-neutron | 16:32 | |
openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Add functional test for IptablesManager https://review.openstack.org/117852 | 16:33 |
*** dbite has joined #openstack-neutron | 16:35 | |
*** aix has joined #openstack-neutron | 16:36 | |
*** ijw has quit IRC | 16:37 | |
*** ijw has joined #openstack-neutron | 16:37 | |
*** Longgeek_ has quit IRC | 16:40 | |
*** luqas has quit IRC | 16:40 | |
*** sballe has joined #openstack-neutron | 16:40 | |
*** Longgeek has joined #openstack-neutron | 16:40 | |
*** Longgeek_ has joined #openstack-neutron | 16:41 | |
*** sballe has quit IRC | 16:41 | |
*** afazekas has quit IRC | 16:41 | |
*** _cjones_ has joined #openstack-neutron | 16:42 | |
*** Longgee__ has joined #openstack-neutron | 16:44 | |
*** Longgeek has quit IRC | 16:45 | |
*** yamamoto has joined #openstack-neutron | 16:45 | |
*** Longgeek_ has quit IRC | 16:48 | |
*** simon-AS559 has joined #openstack-neutron | 16:49 | |
openstackgerrit | Trinath Somanchi proposed a change to openstack/neutron: Freescale FWaaS Plugin https://review.openstack.org/109659 | 16:50 |
*** yamamoto has quit IRC | 16:50 | |
*** simon-AS5591 has joined #openstack-neutron | 16:51 | |
*** padkrish has joined #openstack-neutron | 16:51 | |
*** leenheer has joined #openstack-neutron | 16:52 | |
*** simon-AS559 has quit IRC | 16:53 | |
*** tflynn has joined #openstack-neutron | 16:55 | |
*** leenheer has quit IRC | 16:59 | |
*** gizmoguy_ has quit IRC | 16:59 | |
*** ygbo has quit IRC | 16:59 | |
*** rm_work|away is now known as rm_work | 17:01 | |
*** safchain has quit IRC | 17:01 | |
*** diegows has quit IRC | 17:02 | |
*** amaretskiy has quit IRC | 17:05 | |
*** padkrish has quit IRC | 17:06 | |
openstackgerrit | Robert Kukura proposed a change to openstack/neutron: ML2: Driver API changes for hierarchical port binding https://review.openstack.org/115110 | 17:07 |
openstackgerrit | Robert Kukura proposed a change to openstack/neutron: ML2: DB changes for hierarchical port binding https://review.openstack.org/116122 | 17:07 |
openstackgerrit | Robert Kukura proposed a change to openstack/neutron: ML2: Hierarchical port binding https://review.openstack.org/116209 | 17:07 |
*** shakamunyi has joined #openstack-neutron | 17:07 | |
*** rm_work is now known as rm_work|away | 17:10 | |
*** shakamunyi has quit IRC | 17:12 | |
*** amuller has quit IRC | 17:13 | |
*** SumitNaiksatam has quit IRC | 17:14 | |
*** SumitNaiksatam has joined #openstack-neutron | 17:14 | |
*** rm_work|away is now known as rm_work | 17:15 | |
openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Apic drivers enhancements (second approach): Topology https://review.openstack.org/116780 | 17:17 |
*** ijw has quit IRC | 17:17 | |
*** tflynn has quit IRC | 17:17 | |
*** otherwiseguy has joined #openstack-neutron | 17:17 | |
*** ijw has joined #openstack-neutron | 17:18 | |
*** rm_work is now known as rm_work|away | 17:21 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Adds external attachment type validators to ML2 https://review.openstack.org/116187 | 17:21 |
kevinbenton | rkukura: can you take a look at https://review.openstack.org/#/c/116187/ | 17:22 |
kevinbenton | rkukura: it was the port binding of physical ports we talked about before | 17:22 |
kevinbenton | s/was/is/g | 17:22 |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: BSN: Bind external ports in ML2 driver https://review.openstack.org/116188 | 17:24 |
*** yamamoto has joined #openstack-neutron | 17:25 | |
*** gizmoguy has joined #openstack-neutron | 17:28 | |
*** yamamoto has quit IRC | 17:30 | |
*** thomasem has quit IRC | 17:32 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: BSN: Bind external ports in ML2 driver https://review.openstack.org/116188 | 17:32 |
*** padkrish has joined #openstack-neutron | 17:32 | |
*** prasoon has quit IRC | 17:39 | |
*** padkrish has quit IRC | 17:44 | |
*** simon-AS559 has joined #openstack-neutron | 17:44 | |
*** simon-AS5591 has quit IRC | 17:44 | |
openstackgerrit | A change was merged to openstack/neutron: Remove old policies from policy.json https://review.openstack.org/117586 | 17:45 |
*** claudiub has quit IRC | 17:45 | |
*** simon-AS559 has quit IRC | 17:45 | |
*** simon-AS5591 has joined #openstack-neutron | 17:45 | |
*** padkrish has joined #openstack-neutron | 17:46 | |
*** trinaths has quit IRC | 17:51 | |
*** trinaths has joined #openstack-neutron | 17:51 | |
*** thomasem has joined #openstack-neutron | 17:52 | |
*** devvesa has joined #openstack-neutron | 17:52 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Big Switch: Separate L3 functions into L3 service https://review.openstack.org/113653 | 17:53 |
*** thomasem has quit IRC | 17:53 | |
*** gizmoguy has quit IRC | 17:54 | |
*** thomasem has joined #openstack-neutron | 17:54 | |
*** padkrish has quit IRC | 17:56 | |
*** gizmoguy has joined #openstack-neutron | 17:56 | |
*** bjornar_ has joined #openstack-neutron | 17:56 | |
*** devvesa has quit IRC | 17:57 | |
*** rook has joined #openstack-neutron | 17:57 | |
*** JayJ_ has joined #openstack-neutron | 18:02 | |
*** tsg has joined #openstack-neutron | 18:02 | |
*** _cjones_ has quit IRC | 18:03 | |
*** _cjones_ has joined #openstack-neutron | 18:04 | |
*** diegows has joined #openstack-neutron | 18:05 | |
*** karimb has quit IRC | 18:07 | |
*** Zemeio has joined #openstack-neutron | 18:08 | |
*** shakamunyi has joined #openstack-neutron | 18:08 | |
*** _cjones_ has quit IRC | 18:08 | |
*** mestery_ is now known as mestery | 18:10 | |
*** dgollub has quit IRC | 18:12 | |
*** jp_at_hp has quit IRC | 18:12 | |
*** trinaths has quit IRC | 18:12 | |
*** shakamunyi has quit IRC | 18:13 | |
*** Longgee__ has quit IRC | 18:13 | |
*** yamamoto has joined #openstack-neutron | 18:15 | |
*** SumitNaiksatam has quit IRC | 18:16 | |
*** SumitNaiksatam has joined #openstack-neutron | 18:16 | |
*** yfried_ has joined #openstack-neutron | 18:17 | |
*** yamamoto has quit IRC | 18:20 | |
*** rook has quit IRC | 18:20 | |
*** mitarun has quit IRC | 18:30 | |
*** linuxgeek_ has joined #openstack-neutron | 18:30 | |
*** mitarun has joined #openstack-neutron | 18:31 | |
*** JayJ_ has quit IRC | 18:32 | |
rkukura | kevinbenton: I will as soon as I can - probably this evening | 18:33 |
linuxgeek_ | hi, will these two be merged in juno 3? https://review.openstack.org/#/c/111876/ and https://review.openstack.org/#/c/115575/ | 18:33 |
*** _cjones_ has joined #openstack-neutron | 18:35 | |
*** mitarun has quit IRC | 18:36 | |
*** aix has quit IRC | 18:36 | |
openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Apic drivers enhancements (second approach): Topology https://review.openstack.org/116780 | 18:37 |
*** jlibosva has quit IRC | 18:37 | |
*** sbfox has quit IRC | 18:37 | |
*** sbfox has joined #openstack-neutron | 18:39 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Big Switch: Bind IVS ports in ML2 driver https://review.openstack.org/113749 | 18:39 |
*** zhhuabj has quit IRC | 18:40 | |
kevinbenton | rkukura: ok. thanks | 18:40 |
*** JayJ_ has joined #openstack-neutron | 18:44 | |
*** mlavalle_ has quit IRC | 18:46 | |
*** mlavalle_ has joined #openstack-neutron | 18:46 | |
*** SumitNaiksatam has quit IRC | 18:46 | |
*** SumitNaiksatam has joined #openstack-neutron | 18:47 | |
*** mlavalle_ has quit IRC | 18:49 | |
*** mlavalle_ has joined #openstack-neutron | 18:49 | |
*** zhhuabj has joined #openstack-neutron | 18:52 | |
*** gizmoguy has quit IRC | 18:53 | |
kevinbenton | mestery: is there a neutron meeting today, or are we on the new alternating schedule? | 18:55 |
*** diegows has quit IRC | 18:55 | |
*** dgollub has joined #openstack-neutron | 18:56 | |
*** JayJ_ has quit IRC | 18:56 | |
mlavalle_ | kevinbenton: we are meeting at 21:00 utc, as normal | 18:57 |
kevinbenton | mlavalle_: thanks | 18:57 |
*** _cjones_ has quit IRC | 18:58 | |
mlavalle_ | kevinbenton: np :-) | 18:58 |
*** _cjones_ has joined #openstack-neutron | 18:58 | |
*** dfarrell07 has quit IRC | 18:59 | |
*** mlavalle_ has quit IRC | 19:01 | |
*** mlavalle_ has joined #openstack-neutron | 19:01 | |
*** arosen has joined #openstack-neutron | 19:02 | |
*** _cjones_ has quit IRC | 19:03 | |
*** _cjones_ has joined #openstack-neutron | 19:03 | |
*** marun has quit IRC | 19:04 | |
*** mlavalle_ has quit IRC | 19:05 | |
*** mlavalle_ has joined #openstack-neutron | 19:05 | |
*** padkrish has joined #openstack-neutron | 19:06 | |
*** diegows has joined #openstack-neutron | 19:07 | |
*** _cjones_ has quit IRC | 19:08 | |
*** padkrish has quit IRC | 19:11 | |
*** dgollub has quit IRC | 19:11 | |
*** mlavalle_ has quit IRC | 19:14 | |
*** dgollub has joined #openstack-neutron | 19:14 | |
*** mlavalle_ has joined #openstack-neutron | 19:15 | |
*** yamamoto has joined #openstack-neutron | 19:15 | |
ZZelle | SumitNaiksatam, hi | 19:18 |
SumitNaiksatam | ZZelle: hi | 19:18 |
ZZelle | SumitNaiksatam, one question about FWaaS, what is the meaning of the attribute audited in FirewallPolicy object/table? | 19:19 |
*** mlavalle_ has quit IRC | 19:19 | |
*** JayJ_ has joined #openstack-neutron | 19:19 | |
*** mlavalle_ has joined #openstack-neutron | 19:19 | |
SumitNaiksatam | ZZelle: so some context is required to explain that | 19:19 |
*** yamamoto has quit IRC | 19:20 | |
SumitNaiksatam | ZZelle: so usually the IT departments have separate security teams and infra/network admin teams | 19:20 |
SumitNaiksatam | ZZelle: it is expected that the security teams “audit” the firewall policy | 19:20 |
ZZelle | SumitNaiksatam, one of my usual pain | 19:20 |
SumitNaiksatam | ZZelle: :-) | 19:21 |
SumitNaiksatam | ZZelle: the infra team then uses this policy | 19:21 |
SumitNaiksatam | ZZelle: so the security team crafts, say a pci compliant policy | 19:21 |
SumitNaiksatam | ZZelle: it then marks it as audited | 19:22 |
ZZelle | SumitNaiksatam, ok | 19:22 |
ZZelle | SumitNaiksatam, so security team set a policy as audited and becomes unaudited on update but non-security peoples? | 19:22 |
SumitNaiksatam | ZZelle: but it does not care who is using it | 19:22 |
mestery | kevinbenton: There is a neutron meeting today | 19:22 |
ZZelle | s/but/by | 19:22 |
SumitNaiksatam | ZZelle: yeah, so if someone else goes and modifies that policy we un-audit it | 19:22 |
SumitNaiksatam | ZZelle: per the organization workflow, this has to be go back to the security team for auditing | 19:23 |
SumitNaiksatam | ZZelle: the audit used here has nothing to do with audit logs, in case that is the source of confusion | 19:23 |
SumitNaiksatam | ZZelle: this attribute was added per feedback from the operators (who deal with this workflow) and firewall vendors | 19:24 |
*** r-daneel has quit IRC | 19:24 | |
ZZelle | SumitNaiksatam, i better understand because based on neutron code audited is not useful | 19:25 |
SumitNaiksatam | ZZelle: its just a boolean | 19:25 |
SumitNaiksatam | ZZelle: as far as neutron code is concerned | 19:25 |
SumitNaiksatam | ZZelle: with some small amount of knowledge on when its set or unset | 19:26 |
ZZelle | SumitNaiksatam, speaking of audit logs is there any intentions to support LOG action in FWaaS for K or L+ releases? | 19:26 |
SumitNaiksatam | ZZelle: but we dont use it in any other processing | 19:26 |
SumitNaiksatam | ZZelle: there has always been an intention | 19:26 |
*** cnesa9 has quit IRC | 19:26 | |
SumitNaiksatam | ZZelle: but its become very difficult to get any features in | 19:27 |
SumitNaiksatam | ZZelle: in this release we proposed the “service groups” feature (actually it has been proposed for more than two iterations now) | 19:27 |
SumitNaiksatam | ZZelle: but even that will not get in | 19:28 |
SumitNaiksatam | ZZelle: so intention yes, not sure when | 19:28 |
ZZelle | SumitNaiksatam, i would be interested in 2 FWaaS features LOG action and rate limits in FW rules | 19:28 |
SumitNaiksatam | ZZelle: sure, good to know | 19:29 |
SumitNaiksatam | ZZelle: rate limits might be a little difficult to implement with the reference implementation | 19:29 |
ZZelle | waiting for Kilo-specs :) | 19:29 |
SumitNaiksatam | ZZelle: yes, sure please feel free to add the spec | 19:30 |
*** roeyc has joined #openstack-neutron | 19:30 | |
ZZelle | SumitNaiksatam, i didn't look after seeing the discussion about service groups, i thought it was not the right time | 19:30 |
SumitNaiksatam | ZZelle: yeah | 19:31 |
SumitNaiksatam | ZZelle: right now the focus on getting the high priority stuff into J3 | 19:31 |
*** _cjones_ has joined #openstack-neutron | 19:31 | |
*** marun has joined #openstack-neutron | 19:32 | |
*** drjones has joined #openstack-neutron | 19:35 | |
*** _cjones_ has quit IRC | 19:36 | |
*** sgran has quit IRC | 19:39 | |
*** drjones has quit IRC | 19:39 | |
ZZelle | SumitNaiksatam, after looking at update_firewall use, the lockmode in it seems unrequired (https://review.openstack.org/117022) | 19:40 |
*** _cjones_ has joined #openstack-neutron | 19:40 | |
*** devvesa has joined #openstack-neutron | 19:41 | |
*** _cjones_ has quit IRC | 19:45 | |
*** devvesa has quit IRC | 19:46 | |
*** sambetts has quit IRC | 19:49 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Big Switch: Separate L3 functions into L3 service https://review.openstack.org/113653 | 19:50 |
*** dbite has quit IRC | 19:53 | |
*** jprovazn has quit IRC | 19:58 | |
*** mlavalle_ has quit IRC | 20:02 | |
*** mlavalle_ has joined #openstack-neutron | 20:02 | |
*** yamamoto has joined #openstack-neutron | 20:15 | |
*** yamamoto has quit IRC | 20:17 | |
*** yamamoto has joined #openstack-neutron | 20:17 | |
*** sgran has joined #openstack-neutron | 20:21 | |
*** yamamoto has quit IRC | 20:22 | |
*** seizadi has joined #openstack-neutron | 20:24 | |
*** ijw has quit IRC | 20:24 | |
openstackgerrit | A change was merged to openstack/neutron: l2pop: Allow network types overridable https://review.openstack.org/112947 | 20:26 |
*** cnesa9 has joined #openstack-neutron | 20:26 | |
*** terryw has joined #openstack-neutron | 20:29 | |
*** otherwiseguy has quit IRC | 20:30 | |
*** seizadi has quit IRC | 20:31 | |
*** salv-orlando has joined #openstack-neutron | 20:32 | |
*** mlavalle_ has quit IRC | 20:33 | |
*** mlavalle_ has joined #openstack-neutron | 20:33 | |
*** a_le has quit IRC | 20:34 | |
*** mlavalle_ has quit IRC | 20:34 | |
*** mlavalle_ has joined #openstack-neutron | 20:34 | |
*** seizadi has joined #openstack-neutron | 20:36 | |
*** terryw has quit IRC | 20:41 | |
mlavalle_ | yfried_: you around? | 20:43 |
yfried_ | mlavalle_: more or less | 20:43 |
mlavalle_ | yfried_: so, all the changes you are making to tempest.scenario.manager.NetworkScenarioTest are contained here: https://review.openstack.org/#/c/111719/? | 20:45 |
yfried_ | mlavalle_: yeah? | 20:45 |
mlavalle_ | yfried_: I intend to migrate the the load balancer basic scenario test to tempest neutron client. I'll rebase it on top of your patchset | 20:46 |
yfried_ | mlavalle_: I was going to do it tomorrow (today). | 20:47 |
mlavalle_ | yfried_: well, I am working with the lbaas v2 guys. So it will serve me as a warmp up if that's opk with you | 20:48 |
mlavalle_ | yfried_: ok with you^^^ | 20:48 |
*** seizadi has quit IRC | 20:48 | |
mlavalle_ | yfried_: the next step will be to translate that test to the new lbaas | 20:48 |
yfried_ | sure. if you have the time. go ahead | 20:49 |
yfried_ | mlavalle_: just add your name to the pad | 20:50 |
mlavalle_ | yfried_: : this pad https://etherpad.openstack.org/p/tempest-client-scenarios? | 20:51 |
enikanorov__ | yfried_: mlavalle_hii folks, could you help me to push https://review.openstack.org/#/c/116581/ | 20:52 |
enikanorov__ | ? | 20:52 |
enikanorov__ | it's needed to fix some ipv6-related issue in neutron API | 20:53 |
yfried_ | enikanorov__: I'll take a look tomorrow (please ping me in 8 hours) | 20:53 |
enikanorov__ | yfried_: sure | 20:54 |
mlavalle_ | yfried_: is that the etherpad? | 20:54 |
yfried_ | mlavalle_: yes | 20:54 |
mlavalle_ | yfried_: :-) | 20:54 |
mlavalle_ | enikanorov__: yes, of course i'll help | 20:54 |
*** nati_ueno has joined #openstack-neutron | 20:56 | |
*** zhhuabj has quit IRC | 20:57 | |
*** zhhuabj has joined #openstack-neutron | 20:58 | |
*** yamamoto has joined #openstack-neutron | 20:58 | |
*** markmcclain has joined #openstack-neutron | 20:59 | |
*** emagana has joined #openstack-neutron | 20:59 | |
*** doude has quit IRC | 20:59 | |
*** packet has joined #openstack-neutron | 21:00 | |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: NSX: Add a foreign key for gateway devices https://review.openstack.org/93826 | 21:01 |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: NSX: Validate gateway device list against DB https://review.openstack.org/118148 | 21:01 |
*** gizmoguy has joined #openstack-neutron | 21:01 | |
*** stenchlarge has joined #openstack-neutron | 21:01 | |
*** obondarev_ is now known as obondarev | 21:02 | |
*** networkstatic has joined #openstack-neutron | 21:02 | |
*** otherwiseguy has joined #openstack-neutron | 21:04 | |
*** packet has quit IRC | 21:04 | |
*** packet has joined #openstack-neutron | 21:04 | |
*** jlibosva has joined #openstack-neutron | 21:04 | |
*** mspreitz has quit IRC | 21:06 | |
openstackgerrit | Arvind Somya proposed a change to openstack/neutron: ML2 Type Driver refactor part 3 https://review.openstack.org/115151 | 21:06 |
*** moha_hunt has joined #openstack-neutron | 21:07 | |
*** ajo__ has joined #openstack-neutron | 21:09 | |
*** terryw has joined #openstack-neutron | 21:10 | |
*** otherwiseguy has quit IRC | 21:13 | |
openstackgerrit | A change was merged to openstack/neutron: Remove reference to cisco_cfg_agent.ini from setup.cfg again https://review.openstack.org/118092 | 21:14 |
*** pack3t has joined #openstack-neutron | 21:17 | |
*** ivar-lazzaro has quit IRC | 21:18 | |
*** amotoki has joined #openstack-neutron | 21:18 | |
*** packet has quit IRC | 21:18 | |
*** pack3t is now known as packet | 21:18 | |
*** terryw has quit IRC | 21:19 | |
*** otherwiseguy has joined #openstack-neutron | 21:26 | |
*** bjornar_ has quit IRC | 21:27 | |
*** a_le has joined #openstack-neutron | 21:28 | |
*** markmcclain has quit IRC | 21:28 | |
openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Replace range with xrange https://review.openstack.org/118219 | 21:29 |
*** devvesa has joined #openstack-neutron | 21:30 | |
openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Log error for failed update_device|(up/down) https://review.openstack.org/114705 | 21:34 |
*** devvesa has quit IRC | 21:35 | |
rkukura | anyone know how to debug gate-neutron-dsvm-functional failures, as in http://logs.openstack.org/40/118140/1/gate/gate-neutron-dsvm-functional/54fc625/? | 21:37 |
marun | rkukura: was just wondering the same thing | 21:37 |
marun | rkukura: The first step is figuring out why the test output is not being logged | 21:38 |
*** ijw has joined #openstack-neutron | 21:38 | |
marun | salv-orlando: any ideas? http://logs.openstack.org/40/118140/1/gate/gate-neutron-dsvm-functional/54fc625/console.html | 21:38 |
marun | salv-orlando: the test run is failing but there is no output being saved | 21:38 |
*** kakuma has joined #openstack-neutron | 21:38 | |
salv-orlando | marun: looking | 21:39 |
marun | salv-orlando: after failure it looks like it can't find nose or testr results | 21:39 |
salv-orlando | marun: perhas because teste did not even start? checking | 21:40 |
salv-orlando | marun: what about this? http://logs.openstack.org/40/118140/1/gate/gate-neutron-dsvm-functional/54fc625/console.html#_2014-09-01_20_33_33_849 | 21:40 |
marun | salv-orlando: forest for the trees :/ | 21:41 |
rkukura | salv-orlando: That’s the first thing I noticed, but it doesn’t give me any clue what went wrong | 21:42 |
marun | salv-orlando: hmmm, did the changes to support full-ext merge? | 21:42 |
salv-orlando | nope it did not | 21:42 |
marun | very curious | 21:42 |
salv-orlando | what does logstash say? | 21:43 |
marun | looking | 21:43 |
marun | salv-orlando: I look for 'InvocationError' and get nothing for the functional job :/ | 21:44 |
marun | oh, wait | 21:44 |
salv-orlando | it started on friday 29/8 | 21:45 |
salv-orlando | and we had 25 hits - always and only on the functional job | 21:45 |
marun | salv-orlando: *sigh* | 21:47 |
marun | maybe a revert is in order | 21:47 |
kevinbenton | salv-orlando: re: the external router IP patch. based on your feedback I imagine this will not make it into Juno | 21:48 |
marun | salv-orlando: https://github.com/openstack/neutron/commit/66730877ee8c4389550295c77db88029e3b8e605 | 21:48 |
*** otherwiseguy has quit IRC | 21:48 | |
*** otherwiseguy has joined #openstack-neutron | 21:49 | |
kevinbenton | salv-orlando: https://review.openstack.org/#/c/83664/ What do you think about a separate patch to allow reading the external IP so the VPNaaS people can be unblocked? | 21:49 |
*** karimb has joined #openstack-neutron | 21:49 | |
salv-orlando | kevinbenton: I am not putting a block there. I’m just saying that regardless of whether it merges or not - external_gateway_info needs to be reviewed | 21:49 |
salv-orlando | if this is holding something important you should ask for raising priority and get an FFE | 21:49 |
kevinbenton | salv-orlando: only the read access is holding something important | 21:50 |
salv-orlando | kevinbenton: I do not distribute those - that’s mestery’s task. I can only advice you ask for one. | 21:50 |
kevinbenton | salv-orlando: that’s why it might make sense to separate them | 21:50 |
*** mlavalle_ has quit IRC | 21:51 | |
salv-orlando | kevinbenton: ok I do not have the details to judge how important it is. As I said, it’s not a big deal for me if we add yet another sub-attribute there. I just think that external_gateway_info is messy. However, it might be important for you to raise a point about that blueprint re its priority. I can give it another rewier or two before thursday, but another core dev is needed. | 21:51 |
nati_ueno | I thinks this one is important. | 21:52 |
nati_ueno | vpnaas is blocked | 21:53 |
*** mlavalle_ has joined #openstack-neutron | 21:53 | |
nati_ueno | so let me help | 21:53 |
rkukura | marun, salv-orlando: Do we know what’s causing these gate-neutron-dsvm-functional failures, or why there are actual test results logged? | 21:55 |
kevinbenton | nati_ueno: okay. I will address salv-orlando’s latest comments and upload a patch | 21:55 |
*** jlibosva has quit IRC | 21:55 | |
marun | rkukura: the test results are not being logged, so no, we don't know why. | 21:55 |
kevinbenton | nati_ueno: or are you reviewing the current one already? | 21:55 |
marun | rkukura: I'm trying to figure that out now. | 21:56 |
rkukura | I see this has been happening pretty regularly since 8/29: http://logstash.openstack.org/#eyJzZWFyY2giOiJcIkVSUk9SOiBJbnZvY2F0aW9uRXJyb3I6ICcvb3B0L3N0YWNrL25ldy9uZXV0cm9uLy50b3gvZHN2bS1mdW5jdGlvbmFsL2Jpbi9weXRob24gLW0gbmV1dHJvbi5vcGVuc3RhY2suY29tbW9uLmxvY2t1dGlscyBweXRob25cIiIsImZpZWxkcyI6W10sIm9mZnNldCI6MCwidGltZWZyYW1lIjoiNjA0ODAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJ0aW1lIjp7InVzZXJfaW50ZXJ2YWwiOjB9LCJzdGFtcCI6MTQwOTYwODI4NTgyMn0= | 21:56 |
marun | rkukura: it's periodic so a recheck might get your change through while I figure out how to fix or whether to turn off the job | 21:56 |
nati_ueno | kevinbenton: I have reviewed current one, and thinking.. I think what Salvatore's comment makes sense for me | 21:56 |
rkukura | marun: I’ve been fighting this all day on several different patches. | 21:57 |
*** mlavalle_ has quit IRC | 21:58 | |
marun | rkukura: there have been 6 failures today, not exactly a constant | 21:58 |
rkukura | marun: Thanks for investigating | 21:58 |
marun | rkukura: (from the functional job) | 21:58 |
marun | rkukura: I've had more issues with the full job | 21:58 |
marun | rkukura: but making it non-voting is probably the next step. | 21:58 |
marun | rkukura: I didn't realize the test results weren't being logged on failure until now. | 21:58 |
*** baoli has quit IRC | 21:59 | |
salv-orlando | marun: that makes it one of the top 10 gate offenders (I think) | 21:59 |
salv-orlando | marun: one question abotu pre-gate hoot. | 21:59 |
salv-orlando | meh hook | 21:59 |
salv-orlando | marun: does it run devstack-gate as well? | 21:59 |
marun | salv-orlando: it does | 21:59 |
rkukura | marun: When you see in zuul that a job has a random failure such as this, is there a way to interrupt it and restart the check/verify without waiting for the doomed run to complete? | 21:59 |
marun | salv-orlando: and unfortunately we're still waiting on the patches to unstack | 21:59 |
marun | salv-orlando: it's making it through devstack though | 22:00 |
salv-orlando | marun: seen that. | 22:00 |
marun | salv-orlando: given the timing I'm guessing it's the l3 agent test addition that merged on friday | 22:00 |
marun | salv-orlando: I'm thinking making it non-voting (again) is the easiest way forward. | 22:01 |
marun | salv-orlando: It takes so long to get changes pushed that getting logging output might take a while | 22:02 |
*** nati_ueno has quit IRC | 22:03 | |
*** nati_ueno has joined #openstack-neutron | 22:04 | |
salv-orlando | marun: the point is that there is no logging output | 22:05 |
salv-orlando | in my opinion tox is crashing at startup —> Ran 15 tests in 12.477s | 22:05 |
salv-orlando | I think functional tests normally take a lot more that 12 seconds | 22:05 |
marun | salv-orlando: I don't think so | 22:05 |
marun | salv-orlando: there aren't that many | 22:06 |
marun | salv-orlando: I'll check though | 22:06 |
salv-orlando | marun: the other hint is that it run 15 tests. | 22:06 |
marun | salv-orlando: also correct | 22:06 |
salv-orlando | Some test might be calling a sys.exit which quits the test runner | 22:06 |
marun | we're just getting started with functional tests | 22:06 |
marun | I guess we can't rule that out, but without test output it's just guessing | 22:06 |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Ignore unknown l2pop entry removals https://review.openstack.org/114443 | 22:07 |
marun | salv-orlando: I've posted a change to make the job non-voting until we figure it out. | 22:07 |
marun | salv-orlando: I don't see any infra folks to take action thought | 22:08 |
marun | though | 22:08 |
openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Implements the ProcessMonitor in the dhcp_agent and dnsmasq driver https://review.openstack.org/115935 | 22:08 |
*** nati_ueno has quit IRC | 22:08 | |
*** emagana has quit IRC | 22:09 | |
salv-orlando | marun: if you ping them they will respond | 22:10 |
salv-orlando | well most of them are us based | 22:10 |
salv-orlando | and the others are sleeping | 22:11 |
salv-orlando | so maybe not now | 22:11 |
marun | salv-orlando: I've mentioned them in #openstack-infra | 22:11 |
*** ijw has quit IRC | 22:11 | |
marun | also seeing if I can reproduce locally | 22:11 |
marun | it's periodic but maybe with repeated runs I can reproduce | 22:12 |
salv-orlando | marun: I think there is a relationship between this failure and parallel execution | 22:12 |
salv-orlando | it seems the server are starting 4 workers | 22:12 |
salv-orlando | you should to the same | 22:12 |
salv-orlando | and by server I mean the upstream gate | 22:12 |
marun | salv-orlando: :( | 22:13 |
marun | salv-orlando: missing rootwrap filter might be to blame | 22:14 |
openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Implements the ProcessMonitor in the l3_agent https://review.openstack.org/114931 | 22:15 |
marun | false alarm. my config is outdated | 22:15 |
salv-orlando | I think the l3 agent tests are triggering a sys.exit on the test runner. How that happens it is a mistery for me. | 22:17 |
marun | salv-orlando: it doesn't look like passing jobs are saving test results either though :/ | 22:22 |
salv-orlando | marun: it depends on what results you want. The job you’vee defined saves all the logs as you would have in a tempest run | 22:23 |
salv-orlando | while here I think you’re looking for the subunit log | 22:23 |
salv-orlando | which instead is saved by unit test jobs | 22:23 |
salv-orlando | I agree the subunit log should reveal where the test crashes. | 22:23 |
salv-orlando | I’ve been looking at these l3 agent tests - is this “gray box” approach something that has been agreed with you? | 22:24 |
marun | salv-orlando: Hmmm | 22:24 |
marun | salv-orlando: I was one of the approvers | 22:25 |
marun | salv-orlando: I guess i need some educating. | 22:25 |
salv-orlando | I am looking at the l3 agent tests - they do not exercise the running l3 agent, but they call l3 agent internal functions and then verify that the appropriate configuration (namespace, interfaces, etc) has been performed | 22:25 |
salv-orlando | I think it’s fine. | 22:25 |
salv-orlando | I just don’t understand why we’re running a full devstack suite? Maybe because other tests are designe differently? | 22:26 |
marun | salv-orlando: devstack is only needed to install dependencies | 22:26 |
marun | salv-orlando: right now that means a full run | 22:26 |
marun | salv-orlando: the next step is only install neutron and unstack right after - patches are in flight for that | 22:26 |
salv-orlando | marun: got it. | 22:27 |
marun | salv-orlando: and the follow-on will be updating devstack to support installation of dependencies + source tree without starting anything | 22:27 |
marun | salv-orlando: the tests are going to be a bit scattershot at first while we figure out what constitutes good tests of system interaction | 22:28 |
marun | salv-orlando: some will be low-level (i.e. exercising individual functions), some will be high level (exercising whole agents) | 22:28 |
*** moha_hunt has quit IRC | 22:29 | |
salv-orlando | that’s fine for me. I’ve seen that you have correctly added test-results to your publishers. | 22:30 |
marun | salv-orlando: I am clearly ignorant of the possible failure modes, though. | 22:30 |
salv-orlando | so you should have detailed testr loggign | 22:30 |
marun | salv-orlando: testr results don't seem to show up even on success | 22:30 |
marun | salv-orlando: http://logs.openstack.org/65/102465/4/check/check-neutron-dsvm-functional/9b01d7e/console.html | 22:31 |
salv-orlando | marun: they are copied only on failures - see macros.yaml | 22:31 |
marun | salv-orlando: hmmm | 22:31 |
salv-orlando | in config/modules/openstack_project/jenkins_job_builder.config | 22:31 |
marun | salv-orlando: so why aren't they showing up in this case? | 22:31 |
marun | salv-orlando: is there no way to debug sys.exit()-caused failure? | 22:31 |
salv-orlando | well because of this -> http://logs.openstack.org/40/118140/1/gate/gate-neutron-dsvm-functional/54fc625/console.html#_2014-09-01_20_33_51_768 | 22:31 |
marun | salv-orlando: because that prevents output from being saved? | 22:31 |
kevinbenton | mestery: can you take a quick look at https://review.openstack.org/#/c/113653/ again? You were ready to +2 before but were waitng for the Big Switch CI | 22:32 |
salv-orlando | sys.exit are a nightmare | 22:32 |
salv-orlando | we had that problem with ml2 in unit tests for a while | 22:32 |
kevinbenton | raise SystemExit can be caught by the test runner, right? | 22:32 |
salv-orlando | you only saw that the test runner failed but you did not know why and where | 22:32 |
marun | salv-orlando: arg. none of the functional tests thus far have gone anywhere near sys.exit. but the l3 tests are likely exercising something that does | 22:32 |
salv-orlando | marun: probably | 22:33 |
*** mlavalle_ has joined #openstack-neutron | 22:33 | |
salv-orlando | kevinbenton: I’m pretty sure people have tried that already. If that did not work, I trust there might be a reason, even if I do not know which one it might be. I tend to assume people are not dumb ;) | 22:33 |
marun | salv-orlando: we can mock it out | 22:34 |
salv-orlando | you can try that | 22:34 |
marun | salv-orlando: there are tests that do just that. maybe it's time we did one in the base class | 22:34 |
kevinbenton | salv-orlando: i don’t understand. do you mean they tried raising SystemExit? | 22:34 |
salv-orlando | kevinbenton: the problem is that a test can crash the test runner by calling sys.exit | 22:35 |
salv-orlando | the testrunner process will quit | 22:35 |
kevinbenton | several calls to sys.exit() in the linux/daemon.py file http://paste.openstack.org/show/104418/ | 22:35 |
salv-orlando | no tests will be executed naymore | 22:35 |
kevinbenton | salv-orlando: right, i completely understand that. I meant that I think the test runner can catch the systemExits easier | 22:35 |
kevinbenton | salv-orlando: so perhaps we should replace all sys.exits() with raise SystemExit() | 22:36 |
salv-orlando | kevinbenton: yeah we grepped that before, now we’re trying to understand how the functional tests could exercuse that path | 22:36 |
kevinbenton | salv-orlando: ok. i’ll shut up now :-) | 22:36 |
salv-orlando | kevinbenton: maybe yes, it was done for other bits already. Mocking is an option. | 22:36 |
salv-orlando | You can keep talking no worries… I need to go to bed now anyway | 22:37 |
salv-orlando | This time I won’t let you keeping me up until 3 am! | 22:37 |
marun | salv-orlando: I want to go to bed :( | 22:37 |
marun | infra wake up and disable voting! | 22:37 |
marun | (please) | 22:37 |
salv-orlando | right it’s 1.40 your time? | 22:37 |
marun | it's 12:40am my time | 22:37 |
marun | i'm utc+2 | 22:37 |
salv-orlando | I thought you were in israel | 22:37 |
marun | that would be worse :) | 22:37 |
marun | czech | 22:38 |
marun | salv-orlando: so before you go, which is preferable | 22:38 |
marun | mocking sys.exit or changing all sys.exit calls to raising SystemExit? | 22:39 |
salv-orlando | marun: give me 5 minutes to think | 22:39 |
kevinbenton | hmmm. sys.exit() raises systemexit | 22:41 |
kevinbenton | https://docs.python.org/2/library/sys.html#sys.exit | 22:41 |
kevinbenton | second sentence | 22:41 |
kevinbenton | a super-catch for SystemExit around test cases should work | 22:42 |
*** roeyc has quit IRC | 22:42 | |
kevinbenton | not really super-catch | 22:42 |
kevinbenton | just a catch | 22:42 |
marun | kevinbenton: so catching SystemExit is preferable because it covers both | 22:44 |
kevinbenton | my local interpreter corroborates my story | 22:44 |
kevinbenton | marun: yes | 22:44 |
salv-orlando | marun: there a lot more tests catching SystemExit than tests mocking sys.exit (1 only) | 22:44 |
kevinbenton | http://paste.openstack.org/show/104419/ | 22:44 |
salv-orlando | marun unit tests do it this way: with testtools.ExpectedException(SystemExit): | 22:45 |
marun | salv-orlando: though that will cause an assertion failure if it isn't raised | 22:46 |
kevinbenton | doesn’t that error if the exception isn’t thrown? | 22:46 |
salv-orlando | marun: I have not idea what that method does - not sure if that works like assertRaises | 22:46 |
marun | salv-orlando: it does | 22:47 |
marun | I don't think I have a good way of trapping for SystemExit :/ | 22:47 |
marun | it's much easier to mock sys.exit | 22:47 |
marun | ffs | 22:49 |
marun | a lot more code raises than calls sys.exit | 22:49 |
marun | way to make my life difficult :/ | 22:49 |
marun | maybe I need to replace all those calls | 22:50 |
salv-orlando | if you mock it is ok unless.. you’re hiding an error condition where sys.exit should not be called and instead is called | 22:50 |
salv-orlando | but in that case you can assert on the mocks | 22:50 |
salv-orlando | and have logs | 22:50 |
marun | salv-orlando: so add a mock to the base class and rewrite tests that want to test sys.exit to target it? | 22:52 |
marun | salv-orlando: and allow a default expectation of 'no calls' to be overridden? | 22:53 |
*** emagana has joined #openstack-neutron | 22:54 | |
*** changbl has quit IRC | 22:56 | |
salv-orlando | marun: sounds good to me. | 22:58 |
*** emagana has quit IRC | 22:59 | |
marun | salv-orlando: ok, sleep now :) | 22:59 |
marun | salv-orlando: thanks for the help | 22:59 |
*** padkrish has joined #openstack-neutron | 23:02 | |
*** packet has quit IRC | 23:02 | |
*** padkrish has quit IRC | 23:02 | |
*** dfarrell07 has joined #openstack-neutron | 23:02 | |
*** padkrish has joined #openstack-neutron | 23:03 | |
kevinbenton | marun: i found a way that might work | 23:04 |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Prevent SystemExits when running tests https://review.openstack.org/118225 | 23:04 |
kevinbenton | marun: ^^ | 23:04 |
*** rha has quit IRC | 23:04 | |
kevinbenton | marun: try it with tox -v -epy27 neutron.tests.unit.bigswitch.test_servermanager.*.test_no_servers | 23:05 |
marun | kevinbenton: oooh, way better than mocking | 23:06 |
kevinbenton | marun: even includes the traceback so you can see where it raised from | 23:06 |
kevinbenton | marun: or where sys.exit() was called | 23:07 |
marun | we need another core to get this merged asap | 23:07 |
kevinbenton | marun: ok. let me remove the fail | 23:07 |
kevinbenton | marun: any bug to mention? | 23:07 |
marun | kevinbenton: oh, right :/ | 23:07 |
marun | kevinbenton: I should file one | 23:08 |
kevinbenton | marun: ok. ready to re-upload once you have a bug # | 23:08 |
marun | salv-orlando: still around? how to file bug + categorize elasticsearch failure? | 23:08 |
marun | kevinbenton: or do you know? | 23:08 |
marun | HenryG: help? ^ | 23:08 |
kevinbenton | marun: no, i don’t sadly | 23:09 |
kevinbenton | Well you can always file a generic bug about exits being uncaught | 23:10 |
kevinbenton | and we can get the patch merging | 23:10 |
kevinbenton | we don’t need the categorization component yet, right? | 23:10 |
openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Local arp responder for VLAN https://review.openstack.org/112949 | 23:10 |
*** seizadi has joined #openstack-neutron | 23:11 | |
marun | kevinbenton: http://docs.openstack.org/infra/elastic-recheck/readme.html#adding-bug-signatures | 23:11 |
kevinbenton | well and this won’t fix the actual failure either. It will just expose the thing exiting | 23:12 |
*** ZZelle has quit IRC | 23:12 | |
kevinbenton | marun: i assume the underlying functional test exiting is failing for some reason | 23:13 |
*** HenryG has quit IRC | 23:13 | |
kevinbenton | so it might not make sense to link the gate failures to the bug this is fixing | 23:14 |
marun | kevinbenton: well, related-bug at the least | 23:14 |
kevinbenton | marun: yeah | 23:14 |
*** dfarrell07 has quit IRC | 23:15 | |
*** SayajiPatil has quit IRC | 23:16 | |
marun | kevinbenton: almost done | 23:17 |
kevinbenton | marun: ok | 23:17 |
*** devvesa has joined #openstack-neutron | 23:19 | |
marun | kevinbenton: 1364171 | 23:19 |
marun | https://bugs.launchpad.net/neutron/+bug/1364171 | 23:19 |
kevinbenton | marun: ok. so i will just be related-bug: #1364171 | 23:20 |
*** simon-AS5591 has quit IRC | 23:20 | |
marun | kevinbenton: sounds good | 23:20 |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Prevent SystemExits when running tests https://review.openstack.org/118225 | 23:21 |
kevinbenton | marun: done | 23:21 |
openstackgerrit | Karthik Natarajan proposed a change to openstack/neutron: Implementation of Brocade Vyatta vRouter L3 Plugin https://review.openstack.org/102336 | 23:21 |
kevinbenton | marun: i don’t have a unit test though… | 23:22 |
kevinbenton | marun: i could get meta and add a unit test | 23:22 |
marun | kevinbenton: how would you? | 23:22 |
kevinbenton | marun: a test that exits and mark it as an expected failure :-) | 23:22 |
*** tflynn has joined #openstack-neutron | 23:22 | |
marun | raise SystemExit and expect AssertionFailure? | 23:22 |
marun | sure | 23:22 |
kevinbenton | marun: where should i put it? | 23:22 |
marun | neutron/tests/unit/test_system_exit.py | 23:23 |
marun | or maybe test_base.py | 23:23 |
*** devvesa has quit IRC | 23:23 | |
kevinbenton | marun: i’ll do test_base, not worth a whole new file probably | 23:24 |
kevinbenton | marun: well then there will be a new class in the base | 23:24 |
kevinbenton | marun: maybe a different file is better | 23:24 |
kevinbenton | marun: test_tester.py and we can put other stuff in there later? | 23:25 |
kevinbenton | marun: things to test the setup and whatnot | 23:25 |
marun | kevinbenton: test_tester? | 23:25 |
openstackgerrit | A change was merged to openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/114069 | 23:25 |
kevinbenton | marun: thinking of a new name | 23:25 |
kevinbenton | marun: that isn’t specifically tied to this particular test | 23:25 |
*** tsg has quit IRC | 23:26 | |
marun | kevinbenton: we can always change the name later | 23:27 |
*** HenryG has joined #openstack-neutron | 23:28 | |
marun | mestery: around? | 23:29 |
*** SayajiPatil has joined #openstack-neutron | 23:29 | |
marun | mestery: we'll need some help getting this patch merged ^ | 23:29 |
marun | rkukura: ^ | 23:29 |
*** fifieldt has joined #openstack-neutron | 23:32 | |
*** yamamoto has quit IRC | 23:33 | |
*** amotoki has quit IRC | 23:35 | |
kevinbenton | marun: hmm, expectfailure isn’t working correctly here. might just have to go forward | 23:35 |
openstackgerrit | Karthik Natarajan proposed a change to openstack/neutron: Implementation of Brocade Vyatta vRouter L3 Plugin https://review.openstack.org/102336 | 23:36 |
marun | kevinbenton: you should be able to test anyway... | 23:37 |
marun | kevinbenton: or are you already trying to assert that check_for_systemexit causes an assertion failure when passed SystemExit? | 23:37 |
kevinbenton | marun: yeah | 23:38 |
kevinbenton | marun: i think calling self.fail in the handler is causing the test to fail | 23:38 |
*** yamamoto has joined #openstack-neutron | 23:38 | |
kevinbenton | marun: maybe instead I should raise an assertion error instead | 23:38 |
marun | kevinbenton: so maybe raising assertionfailure will be more testable? | 23:38 |
marun | kevinbenton: worth a try | 23:38 |
*** padkrish has quit IRC | 23:41 | |
openstackgerrit | Sayaji Patil proposed a change to openstack/neutron: Implements sync mechanism between Neutron and Nuage VSD https://review.openstack.org/113418 | 23:41 |
kevinbenton | marun: doesn’t seem to work either way. i think by the time it makes it to that handler, it’s already outside of the expectFailure’s catch | 23:41 |
*** padkrish has joined #openstack-neutron | 23:41 | |
marun | the handler? | 23:42 |
kevinbenton | marun: the one from addOnException | 23:42 |
marun | kevinbenton: yeah, there's no way you can test the addOnException addition | 23:42 |
marun | kevinbenton: you should be able to test calling check_for_systemexit directly though | 23:42 |
marun | kevinbenton: better than nothing | 23:42 |
kevinbenton | marun: oh i see | 23:42 |
kevinbenton | marun: i was trying to test sys.exit directly | 23:43 |
lifeless | kevinbenton: what about sys.exit do you want to test? | 23:46 |
*** rdo has quit IRC | 23:46 | |
marun | lifeless: hooray! | 23:46 |
marun | lifeless: https://review.openstack.org/#/c/118225/2/neutron/tests/base.py | 23:46 |
marun | also, https://bugs.launchpad.net/neutron/+bug/1364171 | 23:47 |
*** rdo has joined #openstack-neutron | 23:48 | |
*** emagana has joined #openstack-neutron | 23:48 | |
lifeless | ok so run_user should indeed catch SystemExit today | 23:49 |
marun | lifeless: hmmmm... so our guessing is wrong... | 23:49 |
lifeless | though thats arguably a bug - catching SystemExit is bogus | 23:50 |
marun | lifeless: any other thoughts as to why the bug would exhibit? | 23:50 |
marun | lifeless: There isn't any test output that we can look through on the failing jobs | 23:50 |
marun | lifeless: no output to be found: http://logs.openstack.org/60/115360/4/check/check-neutron-dsvm-functional/366b616/console.html#_2014-09-01_15_51_31_438 | 23:51 |
*** karimb has quit IRC | 23:51 | |
marun | lifeless: it may be that the job simply isn't configured to leave it? | 23:51 |
marun | lifeless: test-output is set but maybe there's something else? | 23:51 |
lifeless | I'm just poking at this quickly | 23:53 |
*** emagana has quit IRC | 23:53 | |
*** ianw has quit IRC | 23:53 | |
lifeless | http://paste.ubuntu.com/8210787/ | 23:53 |
lifeless | python -m testtools.run exit | 23:54 |
lifeless | runs one test | 23:54 |
lifeless | doesn | 23:54 |
lifeless | doesn't abort the runner | 23:54 |
*** ianw has joined #openstack-neutron | 23:54 | |
lifeless | it (correctly) attaches a traceback | 23:54 |
lifeless | testing a handler now | 23:54 |
kevinbenton | marun: so should i drop this patch? | 23:55 |
*** gildub has joined #openstack-neutron | 23:55 | |
marun | kevinbenton: I guess so :/ | 23:55 |
lifeless | hang a second | 23:55 |
lifeless | lets figure out whats happening | 23:55 |
lifeless | rhttp://paste.ubuntu.com/8210826/ | 23:57 |
lifeless | that does what it looks like it should | 23:57 |
kevinbenton | it might be worth having independent just to catch when systemexits are thrown. right now if i run without the handler, a test with sys.exit() just passes | 23:57 |
lifeless | yeah, because testtools is catching it in its guts | 23:58 |
kevinbenton | so would this handler be good anyway? I don’t think we want tests that run sys.exit() to pass | 23:58 |
marun | kevinbenton++ | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!