Monday, 2019-03-04

*** janki has joined #openstack-kuryr03:52
*** snapiri has joined #openstack-kuryr05:19
*** ccamposr has joined #openstack-kuryr07:00
*** gcheresh_ has joined #openstack-kuryr07:27
*** maysams has joined #openstack-kuryr07:49
*** snapiri has quit IRC07:52
*** pcaruana has joined #openstack-kuryr08:25
*** janki has quit IRC09:00
*** janki has joined #openstack-kuryr09:00
*** celebdor has joined #openstack-kuryr09:15
*** jistr is now known as jistr|sick09:42
*** gkadam has joined #openstack-kuryr09:50
*** maysams has quit IRC09:59
*** aperevalov has joined #openstack-kuryr10:28
*** aperevalov has quit IRC10:29
*** maysams has joined #openstack-kuryr10:40
*** janki has quit IRC12:16
*** janki has joined #openstack-kuryr12:16
*** danil has joined #openstack-kuryr12:33
dulekltomasbo: We were discussing SR-IOV stuff with danil.12:44
dulekmaysams: ^12:45
dulekI'll just paste our talk…12:45
dulek<danil> as you remember, in last patches for sriov functionality neutron ports remains in down state. I have investigated this issue and I want to fix it.12:45
dulek<dulek> Sure, what's the question?12:45
dulek<danil> in case of nova instances , exactly nova runs through the instances, looks for pci devices that are attached for instances and then updates details for these ports neutron db12:45
dulek<danil> but nova doesn't handle PODs12:45
dulek<danil> so I think I should do next things:12:45
dulek<danil> kuryr-k8s contoller should watch for PODs state and make a update in neutron db for sriov-ports if POD is Running12:45
dulek<danil> also it will necessary do duplicate some details from nova config for kuryr-k8s controller too (I mean pci_whitelist etc.)12:45
dulek<dulek> How do you want Kuryr to update Neutron DB?12:45
dulek<dulek> It may not even have any access to MySQL IP.12:45
dulek<dulek> That would be a hard no from us, I think…12:45
dulekdanil: So you'd like to have Neutron DB credentials in kuryr.conf and access it from Kuryr?12:46
dulekdanil: Is Nova really doing that?12:46
dulekdanil: I thought it was using RabbitMQ.12:46
dulekdanil: I really don't want to depend from Nova's DB design.12:46
danilyes, RabbitMQ exactly. Not with Neutron DB credentials12:46
dulekTo be honest I'd rather not depend from RabbitMQ RPC API either.12:47
danilhm, is there any way to update neutron ports ? We need to update binding:profile12:47
dulekdanil: If Neutron REST API doesn't allow it, then probably no…12:49
dulekdanil: I'm looking up code in Nova that does it. Are you able to point me to it?12:49
danilin /usr/lib/python2.7/site-packages/nova/network/neutronv2/api.py12:51
danildef _update_port_binding_for_instance12:52
dulekdanil: Hey, this is using REST API!12:53
dulekdanil: In such case feel free to call Neutron however you like. :D12:53
ltomasbodulek, danil: is this sriov for baremetal case, right? what about the nested one?12:53
ltomasbodanil, dulek: should it work in a completely different way, like attaching a new interface to the VM and then fully allocate it to a given container?12:54
danilI'm just about baremetal case12:54
ltomasbodanil, dulek: perhaps in that way nova already takes care of the port status...12:54
danilhaven't tested  nested one12:54
dulekltomasbo: Yeah, it should probably work that way, but I don't think we ever figured that out.12:55
danilthanks  a lot. I'll prepare a desision and push it for review12:56
dulekdanil: Okay, so just for you to understand why I had those doubts - DB schema or even RabbitMQ's RCP aren't stable, public API's.12:57
dulekdanil: REST API's on the other hand are considered solid, stable and any change in their behavior is considered a bug.12:57
dulekdanil: That's why I'm concerned about using anything else than REST API to12:58
dulekcontact any OpenStack service.12:58
dulekdanil: Because we want to support older versions of OpenStack with a newer Kuryr - e.g. there's nothing that prevents using Stein Kuryr with Queens cloud.12:59
dulekAnd using some internal API would make it much harder to keep such compatibility.12:59
danildulek: sure, I see. Thank sfor explanation13:00
danildulek: I agree13:00
dulek:)13:00
*** jistr|sick is now known as jistr|sick|mtg13:01
ltomasbodulek, danil: plus, if you run kuryr-kubernetes containerized on top of a nested env, you will not have access to the rabbitmq queues, right?13:08
dulekltomasbo: Well, you could if they're exposed, depends on configuration.13:09
dulekltomasbo: But on a well-designed cloud those would not be accessible to VM's.13:10
ltomasbodulek, you could, but it is not what you are going to get most of the time I suppose13:10
dulekltomasbo: Sure thing, consider a public cloud, for example. I'm 100% sure nobody is exposing RPC to VM's. ;)13:11
*** rh-jelabarre has joined #openstack-kuryr13:17
openstackgerritMaysa de Macedo Souza proposed openstack/kuryr-kubernetes master: Fix LBaaS sg rules update on deployment scale  https://review.openstack.org/63718613:19
dulekltomasbo: Had you succeeded with deploying Kuryr with networking-ovn?13:22
ltomasbodulek, yes!13:45
ltomasbodulek, do you want the local.conf?13:46
dulekltomasbo: I already have it and I'm trying to install it.13:46
dulekltomasbo: At the moment installations fail waiting for the K8s API LB.13:46
dulekltomasbo: But I've noticed some weird errors in o-cw, related to Octavia templates formatting.13:47
dulekltomasbo: So maybe it was that and I just need to update Octavia.13:47
dulekltomasbo: So until I hit it again, no questions. ;)13:47
ltomasbodulek, ok!13:50
dulekNeutron did not start'13:53
dulekOh come on!13:53
*** aperevalov has joined #openstack-kuryr13:54
*** aperevalov has quit IRC13:56
*** janki has quit IRC14:19
*** janki has joined #openstack-kuryr14:24
*** pcaruana has quit IRC14:31
maysams#openstack-meeting-414:39
openstackgerritMerged openstack/kuryr-kubernetes master: Add option to tag Neutron resources created by us  https://review.openstack.org/63658614:42
*** jistr|sick|mtg is now known as jistr|sick15:02
danildulek, ltomasbo: how do you think, what is the best way to pass some host information (pci adress and pci_vendor for used virtual function) to master?15:09
danilwe can use k8s_client to annotate k8s master15:09
danilI'd prefet to use pod annotations to pass this information. Is is possible?15:10
dulekdanil: You want to pass them from kuryr-daemon? Yeah, annotations sound like an option.15:13
danildulek, it doesn't matter. I just want to use a simplest way. I can collect information on minion's side and I just want to pass it to k8s master15:15
danilinformation related to used VFs15:15
dulekdanil: I think minion only has access to K8s API, so annotations or CRD's.15:16
*** danil_ has joined #openstack-kuryr15:21
danil_dulek, I left my laptop. Could you please repeat your answer?15:22
danil_What is the best way do you think15:22
*** pcaruana has joined #openstack-kuryr15:25
*** danil_ has quit IRC15:27
dulekdanil: Annotations or CRD, whatever works better for you.15:35
dulekltomasbo: Okay, I'm hitting this again: TemplateSyntaxError: expected token 'end of statement block', got '.'15:40
dulekltomasbo: Should I use some older version of Octavia or what?15:40
ltomasbodulek, that is wehn deploying?15:41
ltomasbodulek, I've never seen that one15:41
dulekltomasbo: No, it's in o-cw.15:41
ltomasboumm, ovn backend + ovn-octavia driver?15:41
dulekltomasbo: maysams was pointing me to a specific patchset from https://review.openstack.org/#/c/608862/. Maybe I should just use the newest one?15:42
*** janki has quit IRC15:42
ltomasbodulek, I installed this morning using a specific one15:42
dulekltomasbo: And master Octavia?15:42
maysamsdulek, maybe it's good to share your local.conf15:42
ltomasbobut of course with a few other components that were older...15:42
ltomasboso, probably better to use the latest15:43
ltomasbofor everything15:43
ltomasboor I can point you to the octavia commit I'm using (if that helps)15:43
dulekltomasbo: And why are you using that patchset instead of newest one?15:44
dulekltomasbo, maysams: http://pastebin.test.redhat.com/73075715:45
ltomasbodulek, because I know that one is working... after that some were broken15:46
ltomasboand I haven't tried the latest one15:46
ltomasboI know 16 and 30-something was working again15:46
dulekltomasbo: Can you point to the Octavia version you use?15:50
ltomasbodulek, sure, latest commit I have is: commit 816abab6a470b33a9441f7df8ec135b3e383ecbc15:50
ltomasbocommit 816abab6a470b33a9441f7df8ec135b3e383ecbc15:50
ltomasbofrom Wed Dec 1215:51
ltomasboMerge "Add certificate configs to config reference"15:51
dulekltomasbo: Oh man, I'm sure this may help. :D15:51
ltomasbofingers crossed!15:51
dulekltomasbo: Although it shouldn't matter really, because amphorae is used for k8s API LB.15:52
dulekBut still, it must be something.15:52
openstackgerritMaysa de Macedo Souza proposed openstack/kuryr-kubernetes master: Fix LBaaS sg rules update on deployment scale  https://review.openstack.org/63718615:57
*** gcheresh_ has quit IRC16:00
ltomasbodulek, I didn't mean that was the commit to blame... just that it is working if you go all the way to that commit16:07
ltomasbodulek, anything from that one and master could be the one to blame16:07
dulekltomasbo: Oh yes, yes.16:08
dulekltomasbo: I was just surprised you're using a December's commit.16:08
ltomasboyep! I'm just updating kuryr on that server...16:08
*** maysams has quit IRC16:30
dulekltomasbo: Progress!16:45
dulekltomasbo: Nope, I've said that too soon. :(16:45
ltomasbo:)16:47
dulekThat was outdated jinja overwritten by yum.17:22
dulekOkay, we have at least one bug with OVN…18:07
dulekltomasbo: Oh my, but services are working flawlessly.18:13
*** ccamposr has quit IRC18:21
*** ccamposr has joined #openstack-kuryr18:21
*** ccamposr has quit IRC18:22
*** mrostecki has quit IRC18:54
*** pcaruana has quit IRC18:56
*** openstackgerrit has quit IRC19:23

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