| *** JamesBenson has quit IRC | 00:08 | |
| *** armax has quit IRC | 00:21 | |
| *** matteo is now known as matteo| | 00:36 | |
| *** JamesBenson has joined #openvswitch | 00:43 | |
| *** JamesBenson has quit IRC | 00:49 | |
| *** tbachman has quit IRC | 00:52 | |
| *** tbachman has joined #openvswitch | 00:53 | |
| *** JamesBenson has joined #openvswitch | 01:21 | |
| *** thaller has quit IRC | 01:24 | |
| *** fkautz has quit IRC | 01:24 | |
| *** fnordahl has quit IRC | 01:24 | |
| *** mnasiadka has quit IRC | 01:24 | |
| *** ghormoon has quit IRC | 01:24 | |
| *** lynxis has quit IRC | 01:24 | |
| *** sri_ has quit IRC | 01:24 | |
| *** flaviof has quit IRC | 01:24 | |
| *** edwarnicke has quit IRC | 01:24 | |
| *** mepholic has quit IRC | 01:24 | |
| *** ghormoon has joined #openvswitch | 01:25 | |
| *** JamesBenson has quit IRC | 01:25 | |
| *** thaller has joined #openvswitch | 01:30 | |
| *** fkautz has joined #openvswitch | 01:30 | |
| *** fnordahl has joined #openvswitch | 01:30 | |
| *** mnasiadka has joined #openvswitch | 01:30 | |
| *** lynxis has joined #openvswitch | 01:30 | |
| *** sri_ has joined #openvswitch | 01:30 | |
| *** flaviof has joined #openvswitch | 01:30 | |
| *** edwarnicke has joined #openvswitch | 01:30 | |
| *** mepholic has joined #openvswitch | 01:30 | |
| *** matteo| has quit IRC | 01:56 | |
| *** matteo has joined #openvswitch | 03:24 | |
| *** spatel has joined #openvswitch | 03:32 | |
| *** dcbw has quit IRC | 04:58 | |
| *** tbachman has quit IRC | 05:22 | |
| *** tbachman has joined #openvswitch | 05:22 | |
| *** acidfu has quit IRC | 05:41 | |
| *** thaller_ has joined #openvswitch | 05:51 | |
| *** acidfu has joined #openvswitch | 05:52 | |
| *** thaller has quit IRC | 05:55 | |
| *** acidfu has quit IRC | 05:56 | |
| *** anilvenkata has joined #openvswitch | 06:02 | |
| *** jaicaa has quit IRC | 06:28 | |
| *** jaicaa has joined #openvswitch | 06:30 | |
| *** spatel has quit IRC | 06:32 | |
| *** apus has quit IRC | 07:08 | |
| *** apus has joined #openvswitch | 07:08 | |
| *** ralonsoh has joined #openvswitch | 07:14 | |
| *** rcernin has quit IRC | 07:14 | |
| *** dholler has joined #openvswitch | 07:20 | |
| *** wondra has quit IRC | 07:24 | |
| *** rcernin has joined #openvswitch | 07:30 | |
| *** fdangelo has joined #openvswitch | 07:31 | |
| *** eelco has joined #openvswitch | 07:33 | |
| *** thaller_ has quit IRC | 07:34 | |
| *** thaller_ has joined #openvswitch | 07:34 | |
| *** slaweq has joined #openvswitch | 07:35 | |
| *** teardown has quit IRC | 07:36 | |
| *** teardown has joined #openvswitch | 07:37 | |
| *** slaweq has quit IRC | 07:38 | |
| *** thaller_ has quit IRC | 07:42 | |
| *** thaller_ has joined #openvswitch | 07:42 | |
| *** slaweq has joined #openvswitch | 07:50 | |
| *** slaweq has quit IRC | 08:01 | |
| *** slaweq has joined #openvswitch | 08:08 | |
| *** mdgray has joined #openvswitch | 08:18 | |
| *** yamamoto has quit IRC | 08:24 | |
| *** yamamoto has joined #openvswitch | 08:25 | |
| *** yamamoto has joined #openvswitch | 08:26 | |
| *** yamamoto has quit IRC | 08:30 | |
| *** yamamoto has joined #openvswitch | 08:31 | |
| *** yamamoto has joined #openvswitch | 08:32 | |
| *** yamamoto has quit IRC | 08:36 | |
| *** rcernin has quit IRC | 08:44 | |
| *** yamamoto has joined #openvswitch | 09:08 | |
| *** elvira has joined #openvswitch | 09:12 | |
| *** yamamoto has quit IRC | 09:17 | |
| *** yamamoto has joined #openvswitch | 09:31 | |
| *** spatel has joined #openvswitch | 09:33 | |
| *** yamamoto has quit IRC | 09:33 | |
| *** spatel has quit IRC | 09:37 | |
| *** yamamoto has joined #openvswitch | 10:21 | |
| *** yamamoto has quit IRC | 10:32 | |
| *** lord4163 has joined #openvswitch | 11:06 | |
| *** yamamoto has joined #openvswitch | 11:45 | |
| *** fbl has joined #openvswitch | 11:54 | |
| *** JamesBenson has joined #openvswitch | 12:01 | |
| *** JamesBenson has quit IRC | 12:07 | |
| *** JamesBenson has joined #openvswitch | 12:07 | |
| *** donhw_ has joined #openvswitch | 13:08 | |
| *** yamamoto has quit IRC | 13:09 | |
| *** spatel has joined #openvswitch | 13:09 | |
| *** donhw has quit IRC | 13:13 | |
| *** spatel has quit IRC | 13:15 | |
| *** ihrachys has quit IRC | 13:21 | |
| *** ihrachys has joined #openvswitch | 13:28 | |
| *** bostondriver has joined #openvswitch | 13:33 | |
| *** spatel has joined #openvswitch | 13:41 | |
| *** dcbw has joined #openvswitch | 14:05 | |
| *** yamamoto has joined #openvswitch | 14:13 | |
| kklimonda_ | I'm trying to change ovn nbdb/sbdb inactivity_probe timeout, and the only way of doing that I could find was creating connection entry in database like: | 14:20 |
|---|---|---|
| kklimonda_ | ```ovn-sbctl -- --id=@conn_uuid create Connection target="ptcp\:6642\:0.0.0.0" inactivity_probe=60000 -- set SB_Global . connections=@conn_uuid``` | 14:20 |
| kklimonda_ | but that makes ovsdb listen on 0.0.0.0, and given how this connection is apparently global between all clusterd ovsdb nodes I'm not even sure what should be put into target to listen only on specific IP address | 14:21 |
| numans | kklimonda_, Yes. That's the only way for the probes from ovsdb-server to the connected clients. | 14:21 |
| numans | kklimonda_, you can put a specific IP. | 14:21 |
| numans | kklimonda_ ovn-sbct set-connection ptcp:6642:IP (for example) | 14:22 |
| numans | or ovn-sbctl set connection . target="ptcp\:6642\:10.0.0.4" for example | 14:22 |
| kklimonda_ | numans: how does that work with clustered ovsdb? I thought that this is shared between all nodes, in which case each one would need a different IP address | 14:23 |
| numans | kklimonda_, ok. you mean raft cluster ? or active/passive ? | 14:23 |
| kklimonda_ | I mean raft cluster | 14:23 |
| *** yamamoto has quit IRC | 14:26 | |
| numans | kklimonda_, this is how openshift sets the probe interval - https://github.com/openshift/cluster-network-operator/blob/master/bindata/network/ovn-kubernetes/ovnkube-master.yaml#L271 | 14:26 |
| numans | let me see what LISTEN_DUAL_STACK can have the value | 14:27 |
| numans | kklimonda_, may be you can just set - target="ptcp:6642" | 14:29 |
| kklimonda_ | numans: thanks, I'll try although it seems setting this outside of k8s (where each pod has one interface) won't really do what I want | 14:42 |
| *** acidfu has joined #openvswitch | 14:48 | |
| *** aserdean has joined #openvswitch | 15:13 | |
| *** armax has joined #openvswitch | 15:34 | |
| *** eelco has quit IRC | 16:03 | |
| *** dcbw has quit IRC | 16:12 | |
| *** yamamoto has joined #openvswitch | 16:24 | |
| *** yamamoto has quit IRC | 16:31 | |
| *** dcbw has joined #openvswitch | 16:40 | |
| *** dceara has joined #openvswitch | 16:40 | |
| *** teardown_ has joined #openvswitch | 17:00 | |
| *** eelco has joined #openvswitch | 17:01 | |
| *** teardown has quit IRC | 17:03 | |
| *** eelco has quit IRC | 17:13 | |
| *** ralonsoh has quit IRC | 17:23 | |
| *** elvira has quit IRC | 17:40 | |
| *** elvira has joined #openvswitch | 17:43 | |
| *** panda has joined #openvswitch | 17:48 | |
| *** panda has quit IRC | 18:11 | |
| *** dyusupov has joined #openvswitch | 18:11 | |
| *** panda has joined #openvswitch | 18:13 | |
| *** zhouhan has joined #openvswitch | 18:15 | |
| mmichelson | It's about that time. I'm going to get the meeting started | 18:15 |
| mmichelson | #startmeeting ovn_community_development_discussion | 18:15 |
| openstack | Meeting started Thu Dec 17 18:15:21 2020 UTC and is due to finish in 60 minutes. The chair is mmichelson. Information about MeetBot at http://wiki.debian.org/MeetBot. | 18:15 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 18:15 |
| openstack | The meeting name has been set to 'ovn_community_development_discussion' | 18:15 |
| mmichelson | Hi everyone | 18:15 |
| *** blp has joined #openvswitch | 18:16 | |
| blp | hi! | 18:16 |
| mmichelson | Tomorrow is the proposed release date for 20.12.0 | 18:16 |
| dceara | Hi | 18:16 |
| mmichelson | So I have sent out the commits just before this meeting started | 18:16 |
| mmichelson | That way we can get it released early-ish (for the western hemisphere) tomorrow | 18:16 |
| mmichelson | In other news, I reworked my unit test patches so that rather than trying to introduce a framework that requires instrumenting source files, it just is a small refactor of some IPAM code and addition of tests using ovstest | 18:17 |
| blp | mmichelson: +1 | 18:17 |
| mmichelson | blp, I'm glad you're here today :) The release of 20.12.0 means we should be in position to merge DDLog to master whenever you think you're ready. | 18:17 |
| blp | mmichelson: That is good news. | 18:18 |
| mmichelson | I understand you're currently working on a new version of your patches | 18:18 |
| blp | I see that Dumitru passed along some more reviews. | 18:18 |
| mmichelson | Would you want to get the latest version posted or would you rather us merge what currently exists? | 18:18 |
| blp | I think that my job is then: | 18:18 |
| blp | - Rebase and deal with whatever changes have happened. | 18:18 |
| blp | - Address Dumitru's feedback. | 18:18 |
| blp | If the changes are relatively small based on those two things, then I'd be inclined to merge it. | 18:19 |
| blp | If they need more work, then I'd be inclined to post a v10. | 18:19 |
| blp | What do you think? | 18:19 |
| blp | And post-merge there are at least two jobs left: | 18:19 |
| blp | - Keep up-to-date with changes. | 18:19 |
| blp | - Run benchmarks and iterate on making it faster and scale better. | 18:20 |
| dceara | blp: The last patch of the series depends on the IDL split changes (which need a rebase). I'm not sure if there are more changes needed in the IDL patches, but with the current version quite a few OVN tests were failing. | 18:20 |
| blp | I think it was Numan who passed along a useful benchmark script. | 18:20 |
| blp | Oh, crap. | 18:20 |
| mmichelson | blp, for the rebase step, I assume you're not going to aim for feature parity, right? | 18:20 |
| blp | mmichelson: I've kept up with feature parity so far. | 18:20 |
| mmichelson | blp, ah, ok. I just know that some of the changes (such as the datapath groups change) could be kind of big. | 18:21 |
| blp | I'll rebase the IDL split changes then. | 18:21 |
| blp | Last time I did that I did not see any test failures, but maybe some were introduced aftewrad. | 18:22 |
| blp | It's hard to keep rebasing a big series against a constantly shifting base. | 18:22 |
| mmichelson | yes it is | 18:22 |
| mmichelson | that's why I was wondering if we should just merge what we have and then work directly in master from this point? | 18:22 |
| mmichelson | That way, devs introducing new features to northd can introduce to C and DDLog simultaneously | 18:23 |
| blp | If the IDL split changes can get into OVS, then we can do that. | 18:23 |
| mmichelson | And we only have to chase that gap that currently exists from between your last posting and now | 18:23 |
| blp | Without the IDL split changes, ovn+ddlog won't compile. | 18:23 |
| mmichelson | got it | 18:23 |
| blp | I agree with the philosophy otherwise though. | 18:23 |
| imaximets | blp, I'm going to review idl-split patch-set tomorrow. | 18:23 |
| blp | imaximets: Thanks! That will enable the rest. | 18:23 |
| blp | imaximets: I'll try to get the rebase of it done today, then. | 18:24 |
| zhouhan | blp: for the post-merge task "keep up-to-date with changes", is it assumed that the developer who submit the change should keep the DDlog part up-to-date? | 18:24 |
| blp | zhouhan: I would hope so, but I am also available to help, either to eduate developers or to help doing the porting. | 18:24 |
| zhouhan | blp: sounds good! | 18:24 |
| imaximets | blp, ok. One quick comment: can we re-name reconnect_try_receive to reconnect_receive_attempt ? Current name sounds like we're requesting it to receive something. | 18:25 |
| blp | imaximets: OK. | 18:25 |
| imaximets | blp, thanks! | 18:25 |
| blp | imaximets: I'll fold that into my rebase then. | 18:27 |
| mmichelson | anyway, I kind of transitioned the conversation to ddlog, but to be clear I also was done talking about the things I had worked on. | 18:27 |
| blp | I don't think I have anything else. | 18:27 |
| blp | 9 | 18:28 |
| blp | oops wrong window | 18:28 |
| imaximets | I have a topic to discuss. | 18:28 |
| imaximets | last time I talked about this patch: https://patchwork.ozlabs.org/project/openvswitch/patch/20201211205447.3874314-1-i.maximets@ovn.org/ for ovsdb-server. | 18:29 |
| *** istokes has joined #openvswitch | 18:29 | |
| blp | imaximets: yes | 18:30 |
| imaximets | basically, it changes the format of file transactions that stored in database file and in raft logs to contain diffs of columns. | 18:30 |
| imaximets | recent tests shows decrease by 90% of memory consumption for northbound database for ovn-k8s tests, 40% for southbound datbase and overall 40% reduction in disk IO. | 18:31 |
| blp | wow! | 18:31 |
| imaximets | blp, zhouhan: it'll be great if you can review it. | 18:31 |
| zhouhan | imaximets: ok! | 18:31 |
| imaximets | main problem, I guess, is the upgrade process, since ovd ovsdb-server will not be able to read new database format. | 18:32 |
| blp | I'll put it on my to-do list. | 18:32 |
| imaximets | that is also a problem if we'll have different versions of ovsdb-server within same raft cluster. | 18:32 |
| imaximets | blp, thanks! | 18:32 |
| blp | Why is that a problem for upgrades? I would expect that to be a problem for downgrades, which are still worth considering but not in the same way. | 18:33 |
| imaximets | It's because if raft leader will send new append request to old follower that will be a problem. | 18:34 |
| blp | Oh, OK, I see, we're changing not just the on-disk format but the format that gets sent around the raft cluster. | 18:34 |
| imaximets | yep | 18:34 |
| imaximets | I also worked on adding memory reports to ovn processes today: https://patchwork.ozlabs.org/project/ovn/patch/20201217181427.639377-1-i.maximets@ovn.org/ | 18:35 |
| imaximets | that's it from my side. | 18:35 |
| blp | Maybe this format parameter should be part of the database definition, so that the cluster has to agree on what format to use from the start. | 18:35 |
| blp | Then, the format would only be upgraded once all the cluster members can handle it. | 18:36 |
| imaximets | blp, problem is that we must be sure that new old server will not be connected to a cluster once format changed. | 18:37 |
| imaximets | blp, and also old ovsdb-server will not understand that something is wrong. | 18:37 |
| panda | can ovsdb raft cluster size change during normal operations ? | 18:38 |
| blp | imaximets: The format upgrade would have to happen after the software upgrade was finished. | 18:38 |
| blp | panda: OVSDB supports adding and removing raft cluster members, yes. | 18:38 |
| *** dholler has quit IRC | 18:39 | |
| blp | panda: And, of course, cluster members can fail and come back at runtime. I don't know whether you consider failures to be normal operations. | 18:39 |
| imaximets | blp, I added a cmdline option to disable new format. After upgrade user will be able to enable it via unixctl or be re-starting without the option. | 18:39 |
| imaximets | *by | 18:39 |
| imaximets | but I'm not sure if that is a best way to handle this. | 18:40 |
| blp | imaximets: That's kind of an admin-intensive way to deal with it. The admin will have to make sure to use the right command-line options during upgrade, and then turn them off afterward. | 18:40 |
| blp | imaximets: It might be harder to automate than telling people to upgrade the database format once they have finished their server upgrade. | 18:41 |
| blp | imaximets: I'm not an admin though. | 18:41 |
| imaximets | blp, I can't think of a correct automatic way that will work in all cases. But I'd like to discuss ideas on a mail-list. | 18:43 |
| blp | imaximets: OK. | 18:43 |
| mmichelson | cool, y'all. Anybody else? | 18:44 |
| dceara | I just wanted to mention that since yesterday OVN CI runs with AddressSanitizer enabled for clang (memleak detection too) on GitHub. Thanks imaximets for the suggestion and review! | 18:45 |
| dceara | That's it on my side, thanks! | 18:45 |
| blp | AddressSanitizer is a big help. Glad to see it in use for CI. | 18:47 |
| blp | Anyone else? | 18:48 |
| mmichelson | I guess that's it | 18:49 |
| *** dyusupov has quit IRC | 18:49 | |
| mmichelson | Thanks everyone! | 18:50 |
| *** dyusupov has joined #openvswitch | 18:50 | |
| mmichelson | #endmeeting | 18:50 |
| blp | Thanks! Bye! | 18:50 |
| openstack | Meeting ended Thu Dec 17 18:50:21 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 18:50 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-12-17-18.15.html | 18:50 |
| dceara | Bye! | 18:50 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-12-17-18.15.txt | 18:50 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-12-17-18.15.log.html | 18:50 |
| * panda grumbles | 18:50 | |
| *** blp has quit IRC | 18:50 | |
| panda | I have to study raft cluster a bit more | 18:50 |
| *** dyusupov has quit IRC | 18:51 | |
| *** dyusupov has joined #openvswitch | 18:52 | |
| *** anilvenkata_ has joined #openvswitch | 18:54 | |
| *** anilvenkata has quit IRC | 18:57 | |
| *** dyusupov has quit IRC | 18:57 | |
| *** teardown has joined #openvswitch | 19:06 | |
| *** teardown_ has quit IRC | 19:09 | |
| *** istokes has left #openvswitch | 19:24 | |
| *** elvira has quit IRC | 19:38 | |
| *** anilvenkata_ has quit IRC | 19:41 | |
| *** zhouhan has quit IRC | 19:43 | |
| *** mdgray has quit IRC | 19:51 | |
| *** dceara has quit IRC | 19:54 | |
| *** apus has quit IRC | 20:00 | |
| *** apus has joined #openvswitch | 20:00 | |
| *** teardown has quit IRC | 20:16 | |
| *** teardown has joined #openvswitch | 20:18 | |
| *** teardown_ has joined #openvswitch | 20:45 | |
| *** teardown has quit IRC | 20:49 | |
| *** slaweq has quit IRC | 20:59 | |
| *** tbachman has quit IRC | 22:01 | |
| *** tbachman has joined #openvswitch | 22:03 | |
| *** yamamoto has joined #openvswitch | 22:10 | |
| *** teardown has joined #openvswitch | 22:14 | |
| *** rcernin has joined #openvswitch | 22:15 | |
| *** teardown_ has quit IRC | 22:17 | |
| *** slaweq has joined #openvswitch | 22:21 | |
| *** slaweq has quit IRC | 22:33 | |
| *** ihrachys has quit IRC | 22:43 | |
| *** bostondriver has quit IRC | 22:46 | |
| *** tbachman has quit IRC | 23:31 | |
| *** tbachman has joined #openvswitch | 23:34 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!