*** tongl has quit IRC | 00:27 | |
*** https_GK1wmSU has joined #openstack-neutron-ovn | 02:09 | |
*** https_GK1wmSU has left #openstack-neutron-ovn | 02:13 | |
*** yamamoto has quit IRC | 02:24 | |
*** yamamoto has joined #openstack-neutron-ovn | 02:30 | |
*** yamamoto has quit IRC | 02:35 | |
*** yamamoto has joined #openstack-neutron-ovn | 02:36 | |
*** saphi has joined #openstack-neutron-ovn | 02:39 | |
*** yamamoto has quit IRC | 02:41 | |
*** yamamoto has joined #openstack-neutron-ovn | 02:45 | |
*** rpb has quit IRC | 03:10 | |
*** gtrxcb has joined #openstack-neutron-ovn | 03:21 | |
*** yamamoto has quit IRC | 03:33 | |
*** NetTinkerer has quit IRC | 03:35 | |
*** NetTinkerer has joined #openstack-neutron-ovn | 03:39 | |
*** yamamoto has joined #openstack-neutron-ovn | 03:49 | |
*** yamamoto has quit IRC | 04:05 | |
*** yamamoto has joined #openstack-neutron-ovn | 04:10 | |
*** gongysh has joined #openstack-neutron-ovn | 04:13 | |
*** anilvenkata has joined #openstack-neutron-ovn | 04:45 | |
*** yamamoto has quit IRC | 04:50 | |
*** yamamoto has joined #openstack-neutron-ovn | 05:07 | |
*** janki has joined #openstack-neutron-ovn | 05:41 | |
*** ajo has quit IRC | 06:11 | |
*** mmirecki_netcont has joined #openstack-neutron-ovn | 06:12 | |
*** pcaruana has joined #openstack-neutron-ovn | 06:34 | |
*** zefferno has joined #openstack-neutron-ovn | 06:58 | |
*** janki is now known as janki|lunch | 07:10 | |
*** Guest14 has joined #openstack-neutron-ovn | 07:39 | |
*** gtrxcb has quit IRC | 07:44 | |
*** saphi has quit IRC | 08:17 | |
*** amotoki has joined #openstack-neutron-ovn | 08:18 | |
amotoki | hi, networking-ovn cores, could you review doc-migration patches? | 08:20 |
---|---|---|
amotoki | https://review.openstack.org/#/q/status:open+project:openstack/networking-ovn+branch:master+topic:doc-migration | 08:20 |
amotoki | networking-ovn is one of four neutron stadium projects which have not completed the doc-migration thing. | 08:20 |
*** janki|lunch is now known as janki | 08:25 | |
*** lucas-afk is now known as lucasagomes | 08:32 | |
*** openstackgerrit has quit IRC | 08:33 | |
numans | amotoki, ack. | 09:15 |
numans | i will take a look. | 09:15 |
numans | thanks | 09:15 |
amotoki | numans: thanks | 09:49 |
numans | amotoki, can you please tell me how to test that patch - is there any command i can run | 09:49 |
amotoki | numans: for the first patch "Improve oslo-config-generator setting", you can check expected sample files are generated by running 'tox -e genconfig' | 09:51 |
numans | amotoki, thanks. | 09:51 |
amotoki | numans: in addition, while you may notice it, ./tools/generate_config_file_samples.sh does the same thing. | 09:52 |
numans | amotoki, ok. | 09:53 |
amotoki | numans: I am not sure why networking-ovn does not call ./tools/generate_config_file_samples.sh in tox genconfig target, but I just try to keep the current commands. | 09:53 |
numans | ok | 09:54 |
numans | i will try to see why | 09:54 |
amotoki | i am fine to update the patches. | 09:55 |
*** mmirecki_netcont is now known as mmirecki_lunch | 10:13 | |
*** openstackgerrit has joined #openstack-neutron-ovn | 10:41 | |
openstackgerrit | Daniel Alvarez proposed openstack/networking-ovn master: Fix gate by skipping neutron test https://review.openstack.org/489966 | 10:41 |
dalvarez | numans, lucasagomes anilvenkata ^ | 10:42 |
lucasagomes | dalvarez, thanks! | 10:43 |
dalvarez | openstackgerrit, joined just for me! | 10:43 |
dalvarez | openstackgerrit++ | 10:43 |
Guest14 | numans : https://review.openstack.org/#/c/486098/2/networking_ovn/common/config.py around L115 ^ | 11:11 |
Guest14 | you have a question in there ;D | 11:11 |
*** Guest14 is now known as ajo | 11:11 | |
ajo | yikes | 11:11 |
ajo | I was Guest | 11:11 |
ajo | XD | 11:11 |
*** gongysh has quit IRC | 11:15 | |
*** NetTinkerer has quit IRC | 11:15 | |
ajo | anilvenkata checked your l3ha patch on networking-ovn, makes sense | 11:23 |
ajo | thanks! :D | 11:23 |
anilvenkata | ajo, thanks Miguel, I will work on tests then | 11:23 |
anilvenkata | ajo++ | 11:23 |
numans | amotoki, that's cool :) | 11:35 |
*** mmirecki_lunch is now known as mmirecki_netcont | 11:36 | |
*** lucasagomes is now known as lucas-hungry | 11:37 | |
*** janki has quit IRC | 11:41 | |
amotoki | numans: thanks | 11:45 |
*** mmirecki_netcont has quit IRC | 11:55 | |
*** mmichelson has joined #openstack-neutron-ovn | 12:05 | |
*** mmirecki_netcont has joined #openstack-neutron-ovn | 12:10 | |
*** lucas-hungry is now known as lucasagomes | 12:39 | |
ajo | oh thanks amotoki! | 12:43 |
ajo | (for the doc migration patches) | 12:44 |
amotoki | ajo: np, just waiting for landing and hope networking-ovn will disappear from here soon: https://doughellmann.com/doc-migration/ | 12:45 |
ajo | nice burndown chart | 12:45 |
*** yamamoto has quit IRC | 13:02 | |
dalvarez | lucasagomes, http://logs.openstack.org/66/489966/1/check/gate-networking-ovn-dsvm-functional/4e62537/console.html | 13:06 |
dalvarez | :S | 13:06 |
* lucasagomes checks | 13:07 | |
lucasagomes | oh that's the patch to skip the tests | 13:07 |
lucasagomes | hmmmmmm | 13:07 |
lucasagomes | and if we don't skip, that tests passes ? dalvarez | 13:08 |
* lucasagomes feels like we have no unittest at all heh | 13:08 | |
dalvarez | it does for me likely apparently | 13:08 |
dalvarez | locally* sorry | 13:08 |
dalvarez | seems unrelated | 13:08 |
lucasagomes | yeah | 13:09 |
dalvarez | 2017-08-02 10:55:19.825862 | 2017-08-02 10:55:19.825 | Error: [('system library', 'fopen', 'No such file or directory'), ('BIO routines', 'file_ctrl', 'system lib'), ('SSL routines', 'SSL_CTX_use_PrivateKey_file', 'system lib')] | 13:09 |
dalvarez | weird :S | 13:09 |
dalvarez | lucasagomes, however unit tests pass | 13:10 |
dalvarez | it's functional failing | 13:10 |
*** mmichelson has quit IRC | 13:11 | |
lucasagomes | dalvarez, the other failures seems to be a schema change maybe ? Cause from the two outputs being tests, one include two extra keys: name and severity | 13:13 |
lucasagomes | http://logs.openstack.org/66/489966/1/check/gate-networking-ovn-dsvm-functional/4e62537/console.html#_2017-08-02_10_55_20_173178 | 13:13 |
*** anilvenkata has quit IRC | 13:16 | |
*** yamamoto has joined #openstack-neutron-ovn | 13:18 | |
dalvarez | lucasagomes, looks like | 13:31 |
dalvarez | numans, ^? know about this? | 13:31 |
*** TrailBlazer has joined #openstack-neutron-ovn | 13:42 | |
*** yamamoto has quit IRC | 13:55 | |
*** mlavalle has joined #openstack-neutron-ovn | 13:57 | |
-openstackstatus- NOTICE: We have disable infracloud-vanilla due to the compute host running mirror.regionone.infracloud-vanilla.o.o being offline. Please recheck your failed jobs to schedule them to another cloud. | 13:57 | |
*** mmichelson has joined #openstack-neutron-ovn | 13:58 | |
ajo | dalvarez, I didn't get your metadata agent to load somehow | 14:18 |
dalvarez | ajo, tell me | 14:18 |
ajo | dalvarez https://github.com/mangelajo/vagrants/blob/master/devstack-ovn-multinode/local.computenode.conf | 14:19 |
ajo | what am I missing? | 14:19 |
ajo | oh wait | 14:19 |
ajo | may be it's working this time : ==> compute1: Created symlink from /etc/systemd/system/multi-user.target.wants/devstack@networking-ovn-metadata-agent.service to /etc/systemd/system/devstack@networking-ovn-metadata-agent.service. | 14:19 |
ajo | ==> compute1: 2017-08-02 13:33:14.409 | Starting Nova | 14:19 |
ajo | 1 sec %) | 14:19 |
dalvarez | ajo, :) | 14:20 |
ajo | [vagrant@compute1 ~]$ ps fax | grep metadata | 14:20 |
ajo | 12902 pts/0 S+ 0:00 \_ grep --color=auto metadata | 14:20 |
ajo | nothing there ? | 14:20 |
ajo | * devstack@networking-ovn-metadata-agent.service loaded failed failed Devstack devstack@networking-ovn-metadata-agent.service | 14:21 |
ajo | hhaha | 14:22 |
ajo | . /etc/neutron/neutron.conf does not exist in compute nodes | 14:22 |
ajo | dalvarez what are you using from there? | 14:22 |
dalvarez | ajo, prolly nothing i think we can remove that | 14:23 |
ajo | dalvarez dont' you need the nova metadata endpoint or anything? | 14:23 |
ajo | dalvarez do we need to colocate the neutron-metadata service? | 14:24 |
dalvarez | ajo we can also remove ml2_conf.ini but I need to set [ovn] settings for connection to sb database | 14:24 |
dalvarez | which is currently there | 14:24 |
ajo | ^ | 14:24 |
dalvarez | ajo that's in metadata agent config file | 14:24 |
ajo | ahs | 14:24 |
ajo | aha | 14:24 |
ajo | let me try | 14:24 |
dalvarez | ajo i'll send a patch to devstack to need only metadata config ini by setting the connection strings in ovn section of metadata agent | 14:25 |
ajo | Could not retrieve schema from tcp:127.0.0.1:6642: Connection refused | 14:25 |
dalvarez | ajo yeah that's what i said :P | 14:25 |
ajo | I guess it needs to be pointed to the primary node | 14:25 |
ajo | ack :) | 14:25 |
dalvarez | i need to fix that ajo... will do it shortly | 14:26 |
dalvarez | so that devstack writes the connection settings into metadata agent ini | 14:26 |
otherwiseguy | lucasagomes, are you working on the gate failures with the functional sync tests? | 14:27 |
lucasagomes | otherwiseguy, not really I think dalvarez is on it | 14:28 |
otherwiseguy | (or know if someone is and I missed it?) | 14:28 |
lucasagomes | I just skimmed thru the logs to see if I could find something obvious | 14:28 |
dalvarez | lucasagomes, lucasagomes i just posted the patch to unbreak the unit tests | 14:28 |
dalvarez | otherwiseguy, ^ | 14:28 |
otherwiseguy | ok. just looking at the diff between the dicts it is testing, one has 'name' and 'severity' keys, the other doesn't. | 14:28 |
dalvarez | but didn't look much into the functional tests | 14:28 |
dalvarez | yeah | 14:28 |
otherwiseguy | I have a patch in neutron to unbreak some unit tests as well. | 14:29 |
lucasagomes | otherwiseguy, yeah, it seems to be related to a schema change, right ? | 14:29 |
otherwiseguy | lucasagomes, dalvarez yeah, I assume so. | 14:29 |
otherwiseguy | Haven't looked much other than just writing a script to parse the copy/paste failure output. | 14:30 |
otherwiseguy | just making sure someone was working on it and that I gave what info I had. :p | 14:31 |
otherwiseguy | dalvarez, Here is the quick patch I wrote for neutron: https://review.openstack.org/#/c/489765/ | 14:31 |
otherwiseguy | It's a little hacky, but it was mostly just "unbreak the damn gate" :p | 14:32 |
dalvarez | otherwiseguy, yeah i thought of that too but it's a bit hacky | 14:32 |
dalvarez | otherwiseguy, right i did this instead: https://review.openstack.org/489966 | 14:32 |
dalvarez | i'd like to find out why we have 3 update events but need to finish some stuff by tomorrow and been too busy :( | 14:33 |
otherwiseguy | dalvarez, we set a port state to active it looks like from the output. | 14:33 |
dalvarez | otherwiseguy, and doesn't it happen in neutron? | 14:33 |
ajo | dalvarez about the metadata thing, I guess that if we don't configure the networking-ovn plugin itself, the connection strings are not set on the compute nodes, right? | 14:34 |
otherwiseguy | yamamoto mentioned that it might be happening in _port_provisioned(), but I haven't looked into it. | 14:35 |
dalvarez | ajo exactly that's why i want to ptch devstack to write the connection strings into metadata agent config file too | 14:35 |
dalvarez | otherwiseguy, in _port_provisioned we complete the provisioning block | 14:35 |
dalvarez | otherwiseguy, this causes the plugin to try to bind the port and then port's updated | 14:35 |
dalvarez | and is then set to ACTIVE | 14:36 |
otherwiseguy | dalvarez, it looked like the intent of the patch that addeded the before_after test was really that at least two notifications happened, so it seemed like just doing >= 2 would be fine. might be some more of the test to edit. | 14:36 |
otherwiseguy | but i don't actually know much about the surrounding code, I just wanted to be able to run tests. :p | 14:37 |
dalvarez | otherwiseguy, yeah but also yamamoto's concern seems valid... if we allow more than 2 then we assuming that the first two ones will always be those? | 14:37 |
dalvarez | otherwiseguy, yeah that's why i submitted the patch to skip it :D | 14:37 |
dalvarez | until we have some more time to investigate | 14:37 |
otherwiseguy | Well, if it is happening in _port_provisioned() that sounds like what would always happen last. :D | 14:37 |
dalvarez | i mean... your patch's fine to me too :p | 14:37 |
dalvarez | otherwiseguy, good point | 14:37 |
dalvarez | maybe we can rely on that | 14:38 |
otherwiseguy | I also like deleting unit tests in favor of functional tests as an option too. Because I'm a nut. :P | 14:39 |
otherwiseguy | In any case, it looks like you all are on fixing the gate, so I'll just move back to my "use ovsdbapp" patch. :p | 14:40 |
*** yamamoto has joined #openstack-neutron-ovn | 14:55 | |
dalvarez | otherwiseguy, :p but now functional tests are failing :( | 14:58 |
*** yamamoto has quit IRC | 15:00 | |
*** mmirecki_netcont has quit IRC | 15:03 | |
*** zefferno has quit IRC | 15:41 | |
*** pcaruana has quit IRC | 16:14 | |
*** lucasagomes is now known as lucas-afk | 16:28 | |
openstackgerrit | Merged openstack/ovsdbapp master: Add OVN_Southbound API support https://review.openstack.org/481779 | 18:24 |
*** mmirecki_netcont has joined #openstack-neutron-ovn | 19:33 | |
*** mmirecki_netcont has quit IRC | 20:34 | |
*** mmirecki_netcont has joined #openstack-neutron-ovn | 20:48 | |
openstackgerrit | Terry Wilson proposed openstack/ovsdbapp master: Add RowView versions of db_find and db_list https://review.openstack.org/490199 | 20:48 |
*** yamamoto_ has joined #openstack-neutron-ovn | 21:08 | |
*** yamamoto_ has quit IRC | 21:15 | |
*** yamamoto_ has joined #openstack-neutron-ovn | 21:17 | |
*** mmirecki_netcont has quit IRC | 21:29 | |
*** ajo has quit IRC | 21:49 | |
*** mmichelson has quit IRC | 22:07 | |
*** rpb has joined #openstack-neutron-ovn | 22:16 | |
*** mlavalle has quit IRC | 22:23 | |
*** yamamoto_ has quit IRC | 22:29 | |
*** yamamoto has joined #openstack-neutron-ovn | 22:32 | |
*** yamamoto has quit IRC | 22:36 | |
*** yamamoto has joined #openstack-neutron-ovn | 22:41 | |
*** yamamoto has quit IRC | 22:44 | |
*** yamamoto has joined #openstack-neutron-ovn | 22:56 | |
*** https_GK1wmSU has joined #openstack-neutron-ovn | 22:57 | |
*** https_GK1wmSU has left #openstack-neutron-ovn | 22:59 | |
*** gtrxcb has joined #openstack-neutron-ovn | 23:06 | |
*** yamamoto has quit IRC | 23:11 | |
*** yamamoto has joined #openstack-neutron-ovn | 23:14 | |
*** tongl has joined #openstack-neutron-ovn | 23:26 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!