| openstackgerrit | CAROLINA CABALLERO LOPEZ proposed openstack/kuryr-kubernetes master: Remove redundant parentheses https://review.opendev.org/c/openstack/kuryr-kubernetes/+/789685 | 02:05 |
|---|---|---|
| openstackgerrit | CAROLINA CABALLERO LOPEZ proposed openstack/kuryr-kubernetes master: Fix typo in kuryport.py file https://review.opendev.org/c/openstack/kuryr-kubernetes/+/789694 | 02:21 |
| *** ltomasbo has joined #openstack-kuryr | 06:10 | |
| openstackgerrit | Merged openstack/kuryr-kubernetes master: Remove redundant parentheses https://review.opendev.org/c/openstack/kuryr-kubernetes/+/789685 | 08:41 |
| *** digitalsimboja has joined #openstack-kuryr | 09:21 | |
| openstackgerrit | Xinxin Shen proposed openstack/kuryr-kubernetes master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/kuryr-kubernetes/+/789767 | 09:23 |
| digitalsimboja | Hi all, this patch still fails https://review.opendev.org/c/openstack/kuryr-kubernetes/+/788713 | 09:40 |
| digitalsimboja | Kindly | 09:40 |
| digitalsimboja | assist | 09:40 |
| maysams | hello digitalsimboja, we can take a look at the result of the zuul job to analyze what is the issue | 09:42 |
| maysams | for example, here is the output of one of the jobs https://zuul.opendev.org/t/openstack/build/7c1571994f114753afdf7829ea07fbbd | 09:43 |
| maysams | digitalsimboja: there was a problem locating the go.mod file... it seems that there should be tweak in the paths passed to the "go build" command | 09:44 |
| digitalsimboja | okay | 09:44 |
| maysams | if you ran the command I share a while ago... "docker build ..." you can reproduce the issue on your PC | 09:45 |
| maysams | s/ran/run | 09:45 |
| digitalsimboja | alright | 09:54 |
| maysams | digitalsimboja: let us know how it goes ;) | 10:03 |
| digitalsimboja | sure | 10:03 |
| digitalsimboja | I am getting this right now | 10:06 |
| digitalsimboja | http://paste.openstack.org/show/804971/ | 10:06 |
| maysams | you're missing a dot in the end of the command | 10:07 |
| digitalsimboja | ohh | 10:08 |
| *** digitalsimboja has quit IRC | 10:30 | |
| *** digitalsimboja has joined #openstack-kuryr | 10:34 | |
| dmellado | just read this | 11:33 |
| dmellado | there's also a build script on hack/build-go | 11:33 |
| dmellado | within the cni folder | 11:33 |
| dmellado | let me take a quick look at your patch | 11:33 |
| dmellado | you'll probably have to modify the dockerfile so it runs the go build command from the cni directory | 12:10 |
| *** rh-jelabarre has joined #openstack-kuryr | 12:29 | |
| *** dulek has quit IRC | 13:22 | |
| dmellado | Either that, or move that dockerfile to the cni so it would take it as the go mod root folder | 13:25 |
| dmellado | but I assume that'd need additional changes in the jobs | 13:25 |
| dmellado | maysams: FYI ;) | 13:25 |
| digitalsimboja | @dmellado: You mentioned I move the dockerfile or specify the path to build from the cni directory? Meanwhile I am yet to succeed to start my docker running. Gives me errors | 13:42 |
| digitalsimboja | http://paste.openstack.org/show/804974/ | 13:42 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-tempest-plugin master: Add service hairpin traffic test https://review.opendev.org/c/openstack/kuryr-tempest-plugin/+/788977 | 14:24 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-tempest-plugin master: Add service hairpin traffic test https://review.opendev.org/c/openstack/kuryr-tempest-plugin/+/788977 | 14:27 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Fix NPs for OVN LBs with hairpin traffic https://review.opendev.org/c/openstack/kuryr-kubernetes/+/786035 | 14:36 |
| dmellado | digitalsimboja: is your docker daemon running? | 14:45 |
| dmellado | and, re: the cni and go.mod | 14:45 |
| dmellado | ideally, go mod files would be on the root folder, there's a couple of caveats in not doing so | 14:46 |
| dmellado | if you are not moving the Dockerfile to the cni rootfolder | 14:46 |
| dmellado | then you'd need to modify the working dir to do so ;) | 14:46 |
| dmellado | so i.e. just run the go build command *from* the kuryr_cni dir | 14:46 |
| digitalsimboja | docker daemon is not yet running, trying to get that up | 15:04 |
| openstackgerrit | Michał Dulko proposed openstack/kuryr-kubernetes master: Fix readiness quota check https://review.opendev.org/c/openstack/kuryr-kubernetes/+/789877 | 15:25 |
| *** carcablop19 has joined #openstack-kuryr | 15:35 | |
| *** carcablop19 has quit IRC | 16:30 | |
| *** carcablop19 has joined #openstack-kuryr | 16:31 | |
| *** digitalsimboja has quit IRC | 16:50 | |
| *** ltomasbo has quit IRC | 17:23 | |
| *** carcablop19 has quit IRC | 18:37 | |
| *** CrayZee has joined #openstack-kuryr | 22:00 | |
| *** snapiri has quit IRC | 22:03 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!