| *** fzdarsky__ has joined #openstack-neutron-ovn | 01:49 | |
| *** fzdarsky_ has quit IRC | 01:53 | |
| *** yamamoto_ has joined #openstack-neutron-ovn | 02:09 | |
| *** rpb has quit IRC | 03:34 | |
| *** anilvenkata has joined #openstack-neutron-ovn | 04:05 | |
| *** yamamot__ has joined #openstack-neutron-ovn | 04:20 | |
| *** yamamoto_ has quit IRC | 04:21 | |
| *** kevinbenton has quit IRC | 04:21 | |
| *** kevinbenton has joined #openstack-neutron-ovn | 04:23 | |
| *** janki has joined #openstack-neutron-ovn | 04:55 | |
| *** yamamot__ has quit IRC | 05:07 | |
| *** yamamoto_ has joined #openstack-neutron-ovn | 05:07 | |
| *** pcaruana has joined #openstack-neutron-ovn | 05:15 | |
| *** anilvenkata has quit IRC | 05:39 | |
| *** zefferno has joined #openstack-neutron-ovn | 06:10 | |
| *** ajo has quit IRC | 06:34 | |
| openstackgerrit | xurong00037997 proposed openstack/networking-ovn master: DHCP options for subnet synchronize each time when run ovn db sync or restart neutron-server, it is because there is no classless_static_route for metadata. https://review.openstack.org/501621 | 08:13 |
|---|---|---|
| *** lucas-afk is now known as lucasagomes | 08:31 | |
| lucasagomes | numans, apparently the patch didn't work | 09:09 |
| lucasagomes | numans, I think we will need to change tempest to accomodate that missing ping from ovn | 09:09 |
| numans | lucasagomes, :(. i am looking into the tempest changes. | 09:11 |
| lucasagomes | cool, we can add a hacky patch just to see if that's the cause of the job being failing | 09:11 |
| lucasagomes | like just fake the result of that method to always succeed | 09:11 |
| lucasagomes | if it works we can then upload a proper patch | 09:11 |
| openstackgerrit | xurong00037997 proposed openstack/networking-ovn master: DHCP options for subnet synchronize each time https://review.openstack.org/501621 | 09:13 |
| *** yamamoto_ has quit IRC | 09:28 | |
| *** ajo has joined #openstack-neutron-ovn | 09:45 | |
| *** yamamoto has joined #openstack-neutron-ovn | 09:46 | |
| *** anilvenkata has joined #openstack-neutron-ovn | 09:49 | |
| *** yamamoto has quit IRC | 10:14 | |
| openstackgerrit | Merged openstack/networking-ovn master: Set requested-chassis with binding host_id. https://review.openstack.org/499820 | 10:16 |
| *** yamamoto has joined #openstack-neutron-ovn | 10:19 | |
| *** livelace has joined #openstack-neutron-ovn | 10:23 | |
| numans | lucasagomes, fingeres crossed - https://review.openstack.org/#/c/500815/ https://review.openstack.org/#/c/500609/ | 10:50 |
| * lucasagomes looks | 10:50 | |
| lucasagomes | numans, good stuff on forking tempest for the test (-: | 10:51 |
| numans | lucasagomes, luckily ooo-extras provided the option :) | 10:51 |
| lucasagomes | yeah | 10:51 |
| numans | lucasagomes, here is the tempest clone - https://github.com/numansiddique/tempest/commit/4af28ef8f171d7a5abf6b120f6a32323b2fda395 | 10:52 |
| numans | lucasagomes, i have by passed the ping validation and put some prints. hope it passes now :) | 10:53 |
| lucasagomes | cool :D | 10:55 |
| *** ajo has quit IRC | 10:57 | |
| numans | lucasagomes, http://logs.openstack.org/20/499820/2/check/gate-tempest-dsvm-networking-ovn-ovs-release/35a19b4/logs/tempest_conf.txt.gz | 10:59 |
| numans | lucasagomes, if you grep for 'run_validation' it is set to False | 10:59 |
| numans | i wonder if we should set it to true | 10:59 |
| numans | any thoughts ? | 10:59 |
| lucasagomes | numans, I would have to dig in the tempest code to see exactly what that config does to be honest | 10:59 |
| lucasagomes | by it's name it sounds like yeah we want to run the validations | 11:00 |
| lucasagomes | but, config names in openstack can be misleading sometimes heh | 11:00 |
| * lucasagomes looks | 11:00 | |
| lucasagomes | numans, it has always been false ? | 11:01 |
| lucasagomes | cause the patch increasing the ping count uses validations, no ? | 11:01 |
| numans | lucasagomes, yes. | 11:02 |
| numans | lucasagomes, in the case of tripleo yes, its set to true | 11:02 |
| numans | lucasagomes, let me check | 11:02 |
| lucasagomes | ok | 11:02 |
| numans | lucasagomes, ignore my comment, | 11:03 |
| numans | lucasagomes, i think its ok :) | 11:03 |
| numans | got confused with the names. still confused | 11:04 |
| lucasagomes | hah ok we will find out in a few anyway | 11:04 |
| lucasagomes | I hear you, sometimes even the help string does not help much | 11:04 |
| numans | lucasagomes, http://logs.openstack.org/20/499820/2/check/gate-tempest-dsvm-networking-ovn-ovs-release/35a19b4/logs/tempest.txt.gz#_2017-09-07_03_58_28_071 | 11:06 |
| numans | lucasagomes, if you see this log in networking-ovn job, we see the same ERROR but still the tests pass | 11:06 |
| lucasagomes | otherwiseguy, around ? Am I doing something wrong here or the ovsdb locks does not work ? http://paste.openstack.org/show/620610/ | 11:07 |
| lucasagomes | numans, ^ if you have a min | 11:07 |
| lucasagomes | numans, looking | 11:07 |
| numans | lucasagomes, i think its a bit early for otherwiseguy :) | 11:07 |
| lucasagomes | numans, hmmmm strange yeah... It's hard cause I don't see any other explicit errors on that oooq run | 11:08 |
| lucasagomes | so I thought the ping could have something to deal with it | 11:08 |
| lucasagomes | but now I don't know, if this other tests is passing even when pinging is failing | 11:08 |
| numans | lucasagomes, i would suggest you enable the debug logs in ovsdb-server and monitor the log, you would clearly see a json rpc message to acquire the lock | 11:08 |
| lucasagomes | numans, lemme do it | 11:09 |
| numans | lucasagomes, you could use ovs-appclt -t <CONTROL_SOCKET_PATH> vlog/set dbg | 11:09 |
| numans | i suppose | 11:09 |
| numans | your test app exits after the print ? | 11:09 |
| lucasagomes | yeah | 11:10 |
| numans | lucasagomes, may be you can put a while loop or sleep of 5 seconds and then see the status | 11:10 |
| lucasagomes | I'm just checking whether it got the lock or not | 11:10 |
| lucasagomes | hmm good point | 11:10 |
| lucasagomes | lemme see | 11:10 |
| numans | lucasagomes, it would take some time for the lock reply | 11:10 |
| lucasagomes | numans, oh right | 11:10 |
| lucasagomes | yeah that could be it then | 11:11 |
| numans | :) | 11:11 |
| lucasagomes | thanks numans ! | 11:11 |
| numans | welcome | 11:11 |
| lucasagomes | numans, yup does work :D | 11:13 |
| numans | lucasagomes, cool | 11:13 |
| lucasagomes | numans, hah I will dig into it but, something is odd about this locks: http://paste.openstack.org/show/620614/ | 11:19 |
| lucasagomes | basically both idls got the lock with the same name | 11:19 |
| lucasagomes | there's a race there maybe ?! | 11:19 |
| numans | lucasagomes, i think the problem is both of them have the same IDL connection :) | 11:19 |
| numans | lucasagomes, its as per design :) | 11:20 |
| numans | lucasagomes, let me point you to the code | 11:20 |
| lucasagomes | oh | 11:20 |
| * lucasagomes checks the get_connection() | 11:20 | |
| lucasagomes | cause I'm trying to figure out how I'm going to use the locks for the maintenance thread | 11:20 |
| numans | lucasagomes, https://github.com/openstack/networking-ovn/blob/master/networking_ovn/ovsdb/impl_idl_ovn.py#L116 | 11:21 |
| lucasagomes | oh gotcha | 11:21 |
| numans | lucasagomes, https://github.com/openstack/networking-ovn/blob/master/networking_ovn/ovsdb/impl_idl_ovn.py#L51 | 11:21 |
| numans | :) | 11:21 |
| lucasagomes | oh right it's a class attribute | 11:22 |
| numans | lucasagomes, you have to either fork or run two test apps | 11:22 |
| *** amuller has joined #openstack-neutron-ovn | 11:22 | |
| * lucasagomes thinks how he will use these locks now | 11:22 | |
| *** lucasagomes is now known as lucas-hungry | 11:32 | |
| *** lucas-hungry is now known as lucasagomes | 12:10 | |
| *** ajo has joined #openstack-neutron-ovn | 12:26 | |
| *** jamesdenton has quit IRC | 12:27 | |
| *** jamesdenton has joined #openstack-neutron-ovn | 12:33 | |
| *** mmichelson has joined #openstack-neutron-ovn | 12:42 | |
| *** mmichelson has quit IRC | 12:43 | |
| *** mmichelson has joined #openstack-neutron-ovn | 12:46 | |
| lucasagomes | numans, I said I didn't need that patch to acquire multiple locks with different names in the same idl but, I'm hitting it now with my code heh I probably will work around it another way | 12:59 |
| lucasagomes | but yeah, now ovsdb_monitor and the maintenance thread is trying to get the lock | 12:59 |
| lucasagomes | and end up overwritting one another | 12:59 |
| lucasagomes | *sigh* | 12:59 |
| numans | lucasagomes, so do you need that feature ? | 13:00 |
| numans | lucasagomes, or you have a different way to address ? | 13:00 |
| lucasagomes | numans, well it would come handy with my current design | 13:00 |
| lucasagomes | dunno if I actually need it | 13:00 |
| lucasagomes | I will see | 13:00 |
| numans | ok | 13:00 |
| lucasagomes | I was reusing the same idl I had on the mech driver | 13:00 |
| lucasagomes | I could probably just instantiate a new one, but, let's see... | 13:01 |
| * lucasagomes wants to avoid create moar in-memory replicas | 13:01 | |
| russellb | Hi everyone ... this has come up a few times before, but we're long overdue for retiring this IRC channel and just using #openstack-neutron ... so let's do that starting now. | 13:07 |
| amuller | +1 | 13:07 |
| lucasagomes | +1 | 13:08 |
| russellb | hm ... I don't have ops in here anymore | 13:08 |
| russellb | will look into how to get ops to update topic ... | 13:08 |
| *** ChanServ sets mode: +o russellb | 13:09 | |
| *** russellb changes topic to "THIS CHANNEL IS DEPRECATED. Please use #openstack-neutron for all OpenStack+OVN discussion." | 13:09 | |
| russellb | https://review.openstack.org/501722 | 13:12 |
| *** russellb sets mode: +m | 13:14 | |
| *** nmagnezi has joined #openstack-neutron-ovn | 13:15 | |
| *** otherwiseguy has left #openstack-neutron-ovn | 13:27 | |
| *** amuller has left #openstack-neutron-ovn | 13:28 | |
| *** otherwiseguy has joined #openstack-neutron-ovn | 13:28 | |
| *** jamesdenton has left #openstack-neutron-ovn | 13:44 | |
| *** thegreenhundred has joined #openstack-neutron-ovn | 13:50 | |
| *** janki has quit IRC | 14:36 | |
| *** anilvenkata is now known as anilvenkata|AFK | 14:45 | |
| *** otherwiseguy has left #openstack-neutron-ovn | 15:04 | |
| *** irenab has quit IRC | 15:12 | |
| *** irenab has joined #openstack-neutron-ovn | 15:12 | |
| *** mmichelson has left #openstack-neutron-ovn | 15:44 | |
| *** anilvenkata|AFK has quit IRC | 16:13 | |
| *** zefferno has quit IRC | 16:34 | |
| *** lucasagomes has left #openstack-neutron-ovn | 16:39 | |
| *** ajo has quit IRC | 16:55 | |
| *** tongl has joined #openstack-neutron-ovn | 17:26 | |
| *** livelace has quit IRC | 17:38 | |
| *** livelace has joined #openstack-neutron-ovn | 17:50 | |
| *** livelace has quit IRC | 17:56 | |
| *** livelace has joined #openstack-neutron-ovn | 18:13 | |
| *** amuller has joined #openstack-neutron-ovn | 18:43 | |
| *** amuller has quit IRC | 19:14 | |
| *** amuller has joined #openstack-neutron-ovn | 19:17 | |
| *** amuller has quit IRC | 19:28 | |
| *** pcaruana has quit IRC | 19:31 | |
| *** dmellado has quit IRC | 19:45 | |
| *** ajo has joined #openstack-neutron-ovn | 20:22 | |
| *** ajo has quit IRC | 21:36 | |
| *** amuller has joined #openstack-neutron-ovn | 21:58 | |
| *** thegreenhundred has quit IRC | 22:11 | |
| *** amuller has left #openstack-neutron-ovn | 22:35 | |
| *** tongl has quit IRC | 23:25 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!