Thursday, 2020-04-09

*** ktraynor_ has joined #openvswitch00:03
*** ktraynor has quit IRC00:06
*** dceara has quit IRC00:18
*** yamamoto has joined #openvswitch01:02
*** rebrec79 has quit IRC01:02
*** balkamos has quit IRC02:01
*** balkamos has joined #openvswitch02:01
*** yamamoto has quit IRC02:04
*** dholler has quit IRC02:45
*** dholler has joined #openvswitch02:58
*** yamamoto has joined #openvswitch03:09
*** rebrec79 has joined #openvswitch03:29
*** oanson has quit IRC04:48
*** oanson has joined #openvswitch04:54
*** cpaelzer__ has joined #openvswitch05:02
*** cpaelzer has quit IRC05:02
*** armax has quit IRC05:28
*** mmirecki has joined #openvswitch05:48
*** vkuramshin1 has joined #openvswitch05:50
*** jaicaa has quit IRC05:52
*** jaicaa has joined #openvswitch05:55
*** mmirecki has quit IRC05:55
*** yamamoto has quit IRC05:59
*** mmirecki has joined #openvswitch06:04
*** yogananth has joined #openvswitch06:11
*** mmirecki has quit IRC06:14
*** mmirecki has joined #openvswitch06:14
*** yamamoto has joined #openvswitch06:15
*** mmirecki has quit IRC06:33
*** slaweq_ has quit IRC06:36
*** slaweq_ has joined #openvswitch07:00
*** eelco has joined #openvswitch07:02
*** slaweq_ is now known as slaweq07:03
*** ebail has joined #openvswitch07:04
*** ebail has quit IRC07:05
*** dceara has joined #openvswitch07:09
*** dceara has quit IRC07:28
*** jraju__ has joined #openvswitch07:35
*** ktraynor_ has quit IRC07:58
*** ktraynor has joined #openvswitch07:59
*** mmirecki has joined #openvswitch08:38
*** dceara has joined #openvswitch08:45
*** mmirecki has quit IRC08:45
*** dceara has quit IRC08:49
*** dceara has joined #openvswitch09:11
*** timothy has joined #openvswitch09:26
*** timothy has quit IRC10:46
*** timothy has joined #openvswitch10:49
*** timothy has quit IRC10:56
*** timothy has joined #openvswitch11:04
*** mmirecki has joined #openvswitch11:58
*** mmirecki has quit IRC12:12
*** igordc has quit IRC12:18
*** bostondriver has joined #openvswitch12:55
*** yamamoto has quit IRC12:57
*** yamamoto has joined #openvswitch12:58
*** dcbw has joined #openvswitch13:00
*** acidfoo has quit IRC13:00
*** acidfoo has joined #openvswitch13:01
*** zhouhan_ has quit IRC13:08
*** zhouhan has joined #openvswitch13:08
*** timothy has quit IRC13:17
*** dholler has quit IRC13:18
*** timothy has joined #openvswitch13:20
*** dholler has joined #openvswitch13:30
*** timothy has quit IRC13:51
*** timothy has joined #openvswitch13:53
*** rcernin has quit IRC13:54
*** yamamoto has quit IRC14:00
*** yamamoto has joined #openvswitch14:02
*** yamamoto has quit IRC14:02
*** yamamoto has joined #openvswitch14:02
*** yamamoto has quit IRC14:06
*** yamamoto has joined #openvswitch14:06
*** timothy has quit IRC14:07
*** timothy has joined #openvswitch14:14
*** yamamoto has quit IRC14:32
numansihrachys, one question on your test patch. Earlier how the tests were passing with that buggy macro ?14:49
ihrachysnumans: the macro does nothing, so tests did nothing14:52
ihrachysthe macro never *executed* the buggy function14:53
ihrachysonly defined it14:53
ihrachysI suspect if the regular compare_packets macro would be used after its remove_broadcast peer, it would fail because it would redefine the original function.14:54
ihrachysprobably no tests did that14:54
numansihrachys, ok.14:54
*** cpaelzer__ is now known as cpaelzer14:57
*** slaweq has quit IRC15:06
*** armax has joined #openvswitch15:10
*** yamamoto has joined #openvswitch15:11
*** slaweq has joined #openvswitch15:22
*** yamamoto has quit IRC15:23
*** dholler has quit IRC16:01
*** eelco has quit IRC16:02
*** dholler has joined #openvswitch16:14
*** blp has joined #openvswitch17:00
*** timothy has quit IRC17:04
numansHello17:15
_lore_hi all17:15
numanstime for OVN meeting17:15
blphi17:15
*** jraju__ has quit IRC17:15
*** aginwala has joined #openvswitch17:16
dcearaHi17:16
numans#startmeeting ovn-community-development-discussion17:16
openstackMeeting started Thu Apr  9 17:16:55 2020 UTC and is due to finish in 60 minutes.  The chair is numans. Information about MeetBot at http://wiki.debian.org/MeetBot.17:16
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:16
openstackThe meeting name has been set to 'ovn_community_development_discussion'17:16
numansHello17:17
numansmmichelson is not there today for the meeting17:17
numansWho wants to start ?17:17
numansI can17:18
numansI'm working on handling the runtime data changes in the flow output stage17:18
numansI'm still working on it and hoping to have thsee patches in v3 of the I-P patches next week17:19
blpnumans: Thanks for leading the meeting.17:19
numansI did some code reviews as well17:19
numansblp, welcome17:19
numansThat's it from me.17:19
blpI'm focusing on ddlog for ovn. I've more or less ignored ovs and ovn the last week. Probably annoying some people with that. Sorry.17:20
blpI don't think I have more than that.17:20
_lore_can I go next?17:21
_lore_very quick17:21
numanssure17:21
_lore_this week I resumed my work on iPXE support in OVN adding new feature to dhcp support17:21
_lore_moreover I am woring in ovn-scale-test adding OpenShift Network Policy support17:22
_lore_zhouhan: it is based on my previous PR so when you have some free cycles could you please take a look to it?17:22
_lore_thanks in advance17:22
_lore_that's all from my side17:22
zhouhan_lore_: I feel real sorry that I still didn't review your previous PR. I will do it as soon as I can.17:23
_lore_zhouhan: no worries17:23
_lore_thanks17:23
dcearanumans, may I go next?17:25
numansdceara, sure17:25
dcearaI sent a patch yesterday to fix the missing SB update issue I reported a few weeks ago: https://patchwork.ozlabs.org/patch/1268457/17:26
zhouhanthanks dceara for the great finding!17:26
dcearaIt's due to a bug in the monitor_cond_since implementation. But I'm still a bit uncomfortable with the ovsdb-idl code so looking forward to reviews :)17:26
dcearazhouhan, my pleasure17:27
zhouhandceara: I am reviewing it. Just to confirm, this happens only when client connection lost temporarily, right?17:27
dcearazhouhan, yes, indeed17:28
dcearaalso, I sent a series to backport all recent virtual port related fixes from OVN master/20.03 to OVS 2.12. It would be great if OVS committers could add them to 2.12 as we need them downstream: https://patchwork.ozlabs.org/project/openvswitch/list/?series=16898717:29
dcearathat's it from me, thanks!17:29
zhouhandceara: So, with the fix is it likely to trigger a storm of data downloading if many cilents lost connection and restored during a monitor-condition update.17:30
dcearazhouhan, the chance is higher. But it would happen only if the connection is lost exactly when the monitor-condition is updated.17:31
zhouhandceara: got it17:31
dcearazhouhan, I couldn't think of a better way to deal with it. Especially because the client might reconnect to a different server (if running servers in raft)17:31
zhouhandceara: I will think about it. Maybe it is the best way. Thanks for your fix!17:32
dcearazhouhan, the alternative is to be "reactive" and reconnect when the IDL detects the inconsistency. But that can happen quite late.17:33
dcearazhouhan, thanks!17:33
zhouhanbut reconnect would also trigger the full download, right?17:33
dcearazhouhan, yes, it has to. Otherwise we might end up in a reconnect loop if the last-id txn is still in history.17:34
zhouhandceara: ok, let's discuss in the review17:35
dcearazhouhan, sounds good, thanks17:35
* zhouhan don't have any update this time17:35
imaximetsmay I go next?17:36
numanssure17:36
imaximetsJust an update about the feature I working now.17:36
imaximetsThis is started as a performance test development for SB DB, but now I'm working on "stream-replay" functionality in OVS.17:37
imaximetsThis is a new stream-provider and a bunch of hooks in the main stream.c that allows to record all the connections and data received on them to replay_* files in exact order as they are appearing in the lifetime of OVS-based process and actually replay them making, for example, ovsdb-server to think that it actually receives new connections and data while everything goes from the pre-recorded replay_* files.17:37
imaximetsThis might be used for performance testing if you have recorded replays from your big setup and replaying them without any delays and sleeps between events just to check the raw performance of ovsdb-server.17:37
imaximetsThis might also be used for root cause analysis in case you have recorded bad conditions and replaying them locally, maybe with debugger attached or more logs enabled.17:38
numansimaximets, Sounds interesting.17:38
imaximetsSince implementation is on a "stream" level, this will record all the db connections, openflow and unixctl.  ovsdb-server is a good application to record because it has no other sources of events beside stream.  ovn-controller might be also a good candidate.17:38
imaximetsI have an almost working prototype with one small bug that I'm going to fix soon.17:38
imaximetsthat's it from me.17:38
numansWho wants to go next ?17:39
zhouhanthanks imaximets. sounds interesting. Could you introduce a little about how do you record the events?17:40
imaximetszhouhan, I have a file for each passive and active stream.  File has records.  Each record is a global sequence number, length and the actual received data.17:41
zhouhanimaximets: does it impact performance when recording?17:41
imaximetszhouhan, you could look at the prototype at https://github.com/igsilya/ovs/tree/tmp-stream-replay if you want deep details.  But sure it will impact performance by the fact that each received buffer is written to file.17:42
zhouhanimaximets: got it, thanks!17:43
flaviofmay I drop in next?17:43
imaximetszhouhan, but actual receiving/sending data is not a bottleneck in ovsdb, so I hope that it will not degrade performance too much.17:44
zhouhanimaximets: ok, sounds good!17:44
numansI think we are done with this topic17:47
numansflaviof, I think you can go next.17:47
flaviofty numans17:47
flaviofI don't have anything worth mentioning this week, again. 😜 But would like to chime in on behalf of Maciej on this discussion: https://mail.openvswitch.org/pipermail/ovs-discuss/2020-April/049896.html17:47
flaviofNot sure if any of you have had experience with using selection_method=dp_hash in Openflow, but if you did and have an understanding of how OVN is supposed to interact with that, please respond to that thread.17:47
flaviofThat is all from me. And happy Easter to those of us who celebrate it!17:47
numansflaviof, thanks. I saw the thread. I'll take a look tomorrow17:48
flaviofTY!17:48
numanszhouhan, In case you have some extra cycles, appreciate your comments on the v2 of I-P patch series.17:49
zhouhannumans: oh, you mean the first 4 patches? I will do!17:50
zhouhannumans: sorry for the delay17:50
numanszhouhan, yes. Thanks. No worries17:51
numanszhouhan, you've anything to report ?17:51
zhouhanno updates this time :)17:51
numansOk. Is anyone else who wants to update ?17:52
numansIf not I think we can end the meeting a little early.17:52
blpI don't ahve anything else.17:53
numansOk.17:53
numansI think we can end.17:53
numansThanks everyone17:53
numansBye17:53
dcearaBye17:53
zhouhanthanks, bye17:53
flaviofbye all!17:53
imaximetsbye17:53
numans#endmeeting17:54
openstackMeeting ended Thu Apr  9 17:54:00 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:54
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-04-09-17.16.html17:54
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-04-09-17.16.txt17:54
openstackLog:            http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-04-09-17.16.log.html17:54
*** psahoo has quit IRC18:00
*** aginwala has quit IRC18:06
*** blp has left #openvswitch18:10
*** zhouhan has quit IRC18:20
*** zhouhan has joined #openvswitch18:20
*** markmcclain has quit IRC18:27
*** markmcclain has joined #openvswitch18:29
*** mmirecki has joined #openvswitch18:52
*** mmirecki has quit IRC19:02
*** vkuramshin1 has quit IRC19:19
*** markmcclain has quit IRC19:22
*** gmg has joined #openvswitch19:23
*** markmcclain has joined #openvswitch19:24
*** acidfoo has quit IRC19:34
*** acidfoo has joined #openvswitch19:36
*** fbl has quit IRC19:46
*** gmg has left #openvswitch19:58
*** mmirecki has joined #openvswitch20:13
*** FH_thecat has quit IRC20:14
*** mmirecki has quit IRC20:33
*** markmcclain has quit IRC20:42
*** markmcclain has joined #openvswitch20:42
*** dholler has quit IRC21:44
*** dceara has quit IRC21:52
*** slaweq has quit IRC22:14
*** troulouliou_div2 has joined #openvswitch22:35
*** zhouhan has quit IRC23:30
*** zhouhan has joined #openvswitch23:31

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!