Thursday, 2020-09-17

*** matteo has quit IRC00:08
*** armax has joined #openvswitch00:27
*** matteo has joined #openvswitch00:34
*** JamesBenson has joined #openvswitch00:57
*** JamesBenson has quit IRC01:02
*** JamesBenson has joined #openvswitch01:29
*** JamesBenson has quit IRC01:34
*** dcbw has quit IRC02:17
*** JamesBenson has joined #openvswitch02:26
*** armax has quit IRC02:38
*** dholler has quit IRC03:01
*** dholler has joined #openvswitch03:14
*** links has joined #openvswitch03:21
*** acidfu has quit IRC04:06
*** acidfu has joined #openvswitch04:11
*** acidfu has quit IRC04:16
*** JamesBenson has quit IRC04:56
*** JamesBenson has joined #openvswitch05:40
*** eelco has joined #openvswitch06:03
*** JamesBenson has quit IRC06:11
*** slaweq_ has joined #openvswitch06:26
*** ralonsoh has joined #openvswitch06:50
*** rcernin has quit IRC06:50
*** jraju__ has joined #openvswitch06:59
*** links has quit IRC07:00
*** jaicaa has quit IRC07:01
*** jaicaa has joined #openvswitch07:04
*** rcernin has joined #openvswitch07:09
*** rcernin has quit IRC07:28
*** darkemon has quit IRC07:33
*** darkemon has joined #openvswitch07:35
*** rcernin has joined #openvswitch08:11
*** rcernin has quit IRC08:17
*** slaweq_ is now known as slaweq08:48
*** rcernin has joined #openvswitch08:50
*** thaller_ has joined #openvswitch08:54
*** thaller has quit IRC08:54
*** rcernin has quit IRC09:03
*** thaller_ has quit IRC09:12
*** thaller has joined #openvswitch09:26
*** thaller has quit IRC09:28
*** slaweq has quit IRC09:38
*** thaller has joined #openvswitch09:38
*** slaweq has joined #openvswitch09:40
*** thaller has quit IRC09:51
*** thaller has joined #openvswitch09:54
*** JamesBenson has joined #openvswitch11:07
*** slaweq has quit IRC11:38
*** slaweq has joined #openvswitch12:12
*** acidfu has joined #openvswitch12:17
*** bostondriver has joined #openvswitch12:40
*** dcbw has joined #openvswitch13:13
kklimondaare various ovn probe timeouts and how to tweak them documented somewhere?13:58
kklimondafor example, afaict changing probe interval between NB and CMS is describe here: https://www.mail-archive.com/ovs-discuss@openvswitch.org/msg07453.html is resulting in error `ovsdb_jsonrpc_server|ERR|ptcp:6641:0.0.0.0: listen failed: Address already in use`14:08
kklimondalater email in this thread mentions to not specify the same connection method on the command line: https://www.mail-archive.com/ovs-discuss@openvswitch.org/msg07476.html14:09
kklimondais command line argument this one: `--remote=ptcp:6641:10.129.10.12` ?14:10
*** armax has joined #openvswitch14:20
*** maciejjozefczyk has joined #openvswitch14:33
*** maciejjozefczyk has quit IRC14:51
*** cpaelzer has quit IRC15:36
kklimondaok, I think I've managed to sort it out15:38
*** acidfu has quit IRC15:51
*** eelco has quit IRC15:54
*** fbl_ has joined #openvswitch16:15
*** fbl has quit IRC16:15
*** fbl_ has quit IRC16:18
*** fbl has joined #openvswitch16:18
*** fbl is now known as Guest2400216:18
*** jraju__ has quit IRC16:28
*** dceara has joined #openvswitch16:50
*** dceara_ has joined #openvswitch16:56
*** dceara has quit IRC16:58
*** dceara_ is now known as dceara16:59
*** dceara has quit IRC17:02
*** dceara has joined #openvswitch17:09
*** acidfu has joined #openvswitch17:11
*** Guest24002 has quit IRC17:16
*** fbl has joined #openvswitch17:16
numansHello17:18
dcearaHi17:18
zhouhanHi17:18
imaximetshi17:18
numanssome one wants to chair the meeting ?17:19
numansI'm not sure if I can be there till the end17:19
zhouhanimaximets used to do it :)17:20
imaximetsok17:20
numansthe meeting name is ovn_community_development_discussion17:20
imaximetsI'll host17:20
numansimaximets, thanks.17:20
zhouhanthanks imaximets17:20
imaximets#startmeeting ovn_community_development_discussion17:20
openstackMeeting started Thu Sep 17 17:20:43 2020 UTC and is due to finish in 60 minutes.  The chair is imaximets. Information about MeetBot at http://wiki.debian.org/MeetBot.17:20
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:20
openstackThe meeting name has been set to 'ovn_community_development_discussion'17:20
imaximetsI do not have much updates.17:21
numansHi17:21
_lore_hi all17:21
imaximetsWho wants to go first?17:21
numansI can go17:21
imaximetsnumans, all yours.17:21
numans#info branch-20.09 is created17:21
numansYesterday I created the branch-20.09.17:21
numansMark is on leave, so I did it.17:22
numansI spent much of the time on the code reviews.17:22
numansReviewed Anton's refactor patches and Han's VxLAN feature patch.17:22
numansIncluded these 2 patch sets before creating the branch.17:22
zhouhannumans: VXLAN patch is not mine. It is from Ihar :)17:23
numanszhouhan, sorry.17:23
numansI got confused :)17:23
zhouhannp17:23
numansIhar has requested for a couple of patches to be included before the hard freeze.17:23
numansI found VxLAN to be fine.17:24
numansApart from that, I also reviewed zhouhan's crash fix patches.17:24
zhouhannumans: I reviewed Ihar's "multiple controllers" patch and posted some commments.17:24
numansAnd I submitted a small bug fix patch.17:24
numanszhouhan, I saw the email. But didn't look into the comments.17:24
numanszhouhan, thanks for it.17:24
zhouhannumans: He requested that to be included in 20.09 as well. How would we handle it?17:24
numansI think its better not to include it.17:25
numansIt seems a bit risky.17:25
zhouhannumans: agree.17:25
numansWith Vxlan, I found the risk to be less and the code seemed fine to me.17:25
numansand I had reviewed earlier patch versions too.17:26
numansOk. that's it from me.17:26
zhouhanthanks numans. May I go next17:26
dcearanumans: re branch-20.09, I'd like to try to have this patch merged there too if possible:17:26
dceara#link http://patchwork.ozlabs.org/project/ovn/patch/20200917125050.19729.41529.stgit@dceara.remote.csb/17:26
numanszhouhan, sure.17:26
dcearazhouhan: sorry, I interrupted, please go ahead17:27
numansdceara, I couldn't look into it. I'll take a look tomorrow.17:27
zhouhanI was mainly working on the bug reported by dceara. Big thanks to dceara for finding a simple way to reproduce it.17:27
numansand if zhouhan can look into that patch, it would be great.17:27
zhouhannumans: I will take a look17:28
numanszhouhan, thanks.17:28
numansthanks to dceara.17:28
zhouhanNow the bug fix is merged. I am going to merge to 20.09 and 20.06, but leave 20.03 as is. Thanks for the reviews from dceara and numans.17:29
dcearazhouhan: np17:30
zhouhanFor Ihar's "multiple controller" patch review, my major concern is when it checks if there is multiple controllers configured, it skips the tunnel and patch port resource cleanup, which I think is a problem.17:30
zhouhanWe will discuss more in ML.17:30
zhouhanThat's it from me.17:30
imaximetsThanks.  Who wants to go next?17:31
imaximetsAnyone?17:32
dcearaI have a quick comment if that's ok.17:33
imaximetsdceara, sure.17:33
dcearaMark started some discussions regarding CI a while ago on the mailing list but there wasn't much conclusion. I was wondering if we should revive that effort and include upstream ovn-k8s/openstack/etc CI to use latest OVN master code. Given the increasing number of regressions we saw recently.17:34
zhouhan+117:35
numansI agree to that.17:35
dcearaIt's probably better to discuss this on the mailing list but I thought it might be good to mention it here first :)17:35
numansas part of this discussion, I think it would be better if we move to github actions instead of travis ci.17:35
numansMay be we can discuss more in ML, but I'd like to know if there any quick comments here.17:36
zhouhannumans: I am not familiar with git actions. What's the motivation?17:36
numansovn-k8s is already using github actions and we can probably use most of the actions to run ovn-k8s tests17:36
numanszhouhan, As an example you can take a look - https://github.com/ovn-org/ovn-kubernetes/blob/master/.github/workflows/test.yml17:37
zhouhannumans: ok, thanks17:37
dcearanumans: that is a good point, I looked a bit at how ovn-k8s build their image for CI testing and I think we can change it quite easily to build latest OVN from source.17:37
numanszhouhan, although we don't use PR, I think once the patch is merged, the tests defined in the actions would be executed.17:37
numansdceara, they already have a docker file to build ovs/ovn from sources.17:37
numanswe can probably use that to build the ovn-k8s image17:38
zhouhannumans: where would the test run?17:38
dcearanumans: hmm, I thought they were only consuming fedora rpms, I'll have a better look :)17:38
numanszhouhan, I think github runs it.17:38
zhouhanOh, I see17:38
numanszhouhan, for public repos, I think it lets you run certain amount of time for free.17:39
numansI don't know the details. but onv-k8s uses it,17:39
zhouhannumans: thanks to Microsoft17:39
numanszhouhan, https://github.com/ovn-org/ovn-kubernetes/pull/1705#partial-pull-merging17:39
numansone such example here17:39
numansI'll try it out on my personal repo and see.17:40
numansand if it is fine, I can propose it to ovn-org/ovn17:40
numansdceara, sorry. go ahead17:40
numansone more link17:41
imaximetsnumans, zhouhan: I think that most of things that could be done in github workflows could also be done in travis.  What's the point of using github workflows?  Does it have some features that travis has not?17:41
numans#link https://github.com/ovn-org/ovn-kubernetes/pull/1705/checks?check_run_id=112186634317:41
dcearanumans: right, but those runs are with ovn-k8s using fedora rpms.17:41
numansimaximets, not sure about the features, but if we can use much of ovn-k8s work flow code, it would be easier.17:42
numansdceara, https://github.com/ovn-org/ovn-kubernetes/blob/master/dist/images/Dockerfile.fedora.dev17:42
dcearanumans: great, thanks!17:43
numansI guess we can discuss that in the ML. I'll just try it out first on my personal ovn repo.17:43
dcearaIn any case, I just wanted to bring this up here, we can probably continue on the mailing list to figure out details like travis vs gh workflows17:44
dcearaThat's it on my side, thanks.17:44
imaximetsOK. Thanks. Any more updates?17:45
imaximetsSo, I think, we could call it a meeting.17:46
imaximetsThanks everyone!17:46
numansThanks everyone!17:47
imaximets#endmeeting17:47
dcearaBye everyone17:47
openstackMeeting ended Thu Sep 17 17:47:10 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:47
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-09-17-17.20.html17:47
zhouhanthanks. bye everyone17:47
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-09-17-17.20.txt17:47
openstackLog:            http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-09-17-17.20.log.html17:47
numansimaximets, thanks again for hosting the meeting.17:47
imaximetsnumans, no problem17:47
imaximetsbye17:48
*** tbachman has joined #openvswitch18:31
*** zhouhan_ has joined #openvswitch18:40
*** donhw has joined #openvswitch18:40
*** zhouhan has quit IRC18:43
*** rebrec has joined #openvswitch18:56
*** ccamel has quit IRC18:57
*** dceara has quit IRC19:01
*** camelCaser has joined #openvswitch19:15
*** ralonsoh has quit IRC19:57
*** rebrec has quit IRC20:06
*** dceara has joined #openvswitch20:33
*** acidfu has quit IRC20:42
*** JamesBenson has quit IRC20:44
*** thaller has quit IRC21:18
*** thaller has joined #openvswitch21:19
*** acidfu has joined #openvswitch22:03
*** slaweq has quit IRC22:07
*** bostondriver has quit IRC22:22
*** dceara has quit IRC22:31
*** acidfu has quit IRC22:36
*** acidfu has joined #openvswitch22:47
*** rcernin has joined #openvswitch23:05

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!