| *** ttsiouts has joined #openstack-containers | 00:08 | |
| *** ttsiouts has quit IRC | 00:26 | |
| *** ttsiouts has joined #openstack-containers | 00:27 | |
| *** ttsiouts has quit IRC | 00:32 | |
| *** fragatina has quit IRC | 01:34 | |
| *** hongbin has joined #openstack-containers | 01:57 | |
| *** hongbin has quit IRC | 01:59 | |
| *** hongbin has joined #openstack-containers | 02:04 | |
| openstackgerrit | Lingxian Kong proposed openstack/magnum master: [WIP] Delete floating ip for load balancer https://review.openstack.org/630820 | 02:07 |
|---|---|---|
| *** sapd1_x has joined #openstack-containers | 02:21 | |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: [k8s_fedora_atomic] Add heat-container-agent to worker node https://review.openstack.org/561858 | 02:26 |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: [k8s_fedora_atomic] Add heat-container-agent to worker node https://review.openstack.org/561858 | 02:32 |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: Remove -U from pip install https://review.openstack.org/528556 | 02:54 |
| openstackgerrit | Merged openstack/python-magnumclient master: Trivial: Update pypi url to new url https://review.openstack.org/563056 | 03:02 |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: [k8s_fedora_atomic] Add heat-container-agent to worker node https://review.openstack.org/561858 | 03:10 |
| *** ykarel|away has joined #openstack-containers | 03:42 | |
| *** udesale has joined #openstack-containers | 04:05 | |
| *** dave-mccowan has quit IRC | 04:11 | |
| *** udesale has quit IRC | 04:15 | |
| *** udesale has joined #openstack-containers | 04:16 | |
| *** hongbin has quit IRC | 04:25 | |
| *** udesale has quit IRC | 04:26 | |
| *** fragatina has joined #openstack-containers | 04:45 | |
| *** fragatina has quit IRC | 04:47 | |
| *** fragatina has joined #openstack-containers | 04:48 | |
| *** fragatina has quit IRC | 04:51 | |
| *** fragatina has joined #openstack-containers | 04:51 | |
| *** fragatina has quit IRC | 04:54 | |
| *** _fragatina_ has joined #openstack-containers | 04:54 | |
| *** _fragatina_ has quit IRC | 04:56 | |
| *** fragatina has joined #openstack-containers | 04:56 | |
| *** fragatina has quit IRC | 04:58 | |
| *** fragatina has joined #openstack-containers | 04:58 | |
| *** fragatina has quit IRC | 05:03 | |
| *** fragatina has joined #openstack-containers | 05:04 | |
| *** fragatina has quit IRC | 05:11 | |
| *** fragatina has joined #openstack-containers | 05:11 | |
| *** fragatina has quit IRC | 05:13 | |
| *** fragatina has joined #openstack-containers | 05:13 | |
| *** fragatina has quit IRC | 05:16 | |
| *** fragatina has joined #openstack-containers | 05:16 | |
| *** fragatina has quit IRC | 05:17 | |
| *** fragatina has joined #openstack-containers | 05:17 | |
| *** fragatina has quit IRC | 05:19 | |
| *** fragatina has joined #openstack-containers | 05:19 | |
| *** fragatina has quit IRC | 05:27 | |
| *** fragatina has joined #openstack-containers | 05:28 | |
| *** fragatina has quit IRC | 05:30 | |
| *** fragatina has joined #openstack-containers | 05:30 | |
| *** fragatina has quit IRC | 05:32 | |
| *** fragatina has joined #openstack-containers | 05:32 | |
| *** fragatina has quit IRC | 05:36 | |
| *** fragatina has joined #openstack-containers | 05:36 | |
| *** ykarel|away is now known as ykarel | 05:38 | |
| *** Bhujay has joined #openstack-containers | 05:43 | |
| *** ykarel has quit IRC | 05:43 | |
| *** Bhujay has quit IRC | 05:44 | |
| *** Bhujay has joined #openstack-containers | 05:45 | |
| *** fragatina has quit IRC | 05:45 | |
| *** Bhujay has quit IRC | 05:46 | |
| *** fragatina has joined #openstack-containers | 05:46 | |
| *** Bhujay has joined #openstack-containers | 05:46 | |
| *** fragatina has quit IRC | 05:51 | |
| *** fragatina has joined #openstack-containers | 05:52 | |
| *** fragatina has quit IRC | 05:59 | |
| *** fragatina has joined #openstack-containers | 05:59 | |
| *** ykarel has joined #openstack-containers | 06:00 | |
| *** fragatina has quit IRC | 06:01 | |
| *** fragatina has joined #openstack-containers | 06:01 | |
| *** udesale has joined #openstack-containers | 06:05 | |
| *** udesale has quit IRC | 06:12 | |
| *** udesale has joined #openstack-containers | 06:13 | |
| *** fragatina has quit IRC | 06:13 | |
| *** fragatina has joined #openstack-containers | 06:14 | |
| *** fragatina has quit IRC | 06:18 | |
| *** fragatina has joined #openstack-containers | 06:18 | |
| *** fragatina has quit IRC | 06:22 | |
| *** fragatina has joined #openstack-containers | 06:23 | |
| *** fragatina has quit IRC | 06:23 | |
| *** fragatina has joined #openstack-containers | 06:24 | |
| *** fragatina has quit IRC | 06:25 | |
| *** fragatina has joined #openstack-containers | 06:25 | |
| *** fragatina has quit IRC | 06:26 | |
| *** fragatina has joined #openstack-containers | 06:26 | |
| *** fragatina has quit IRC | 06:27 | |
| *** fragatina has joined #openstack-containers | 06:27 | |
| *** itlinux_ has joined #openstack-containers | 06:28 | |
| *** fragatina has quit IRC | 06:29 | |
| *** fragatina has joined #openstack-containers | 06:29 | |
| *** itlinux has quit IRC | 06:30 | |
| *** fragatina has quit IRC | 06:31 | |
| *** fragatina has joined #openstack-containers | 06:32 | |
| *** fragatina has quit IRC | 06:32 | |
| *** fragatina has joined #openstack-containers | 06:33 | |
| *** udesale has quit IRC | 06:33 | |
| *** udesale has joined #openstack-containers | 06:33 | |
| *** _fragatina_ has joined #openstack-containers | 06:34 | |
| *** _fragatina_ has quit IRC | 06:37 | |
| *** _fragatina_ has joined #openstack-containers | 06:37 | |
| *** fragatina has quit IRC | 06:38 | |
| *** _fragatina_ has quit IRC | 06:44 | |
| *** fragatina has joined #openstack-containers | 06:44 | |
| *** fragatina has quit IRC | 06:47 | |
| *** fragatina has joined #openstack-containers | 06:48 | |
| *** fragatina has quit IRC | 06:48 | |
| *** itlinux_ has quit IRC | 06:48 | |
| *** fragatina has joined #openstack-containers | 06:48 | |
| *** fragatina has quit IRC | 06:49 | |
| *** fragatina has joined #openstack-containers | 06:50 | |
| *** fragatina has quit IRC | 06:52 | |
| *** fragatina has joined #openstack-containers | 06:53 | |
| *** rcernin has quit IRC | 06:59 | |
| *** Bhujay has quit IRC | 07:01 | |
| openstackgerrit | Merged openstack/magnum-ui master: Update hacking version https://review.openstack.org/628468 | 07:03 |
| *** fragatina has quit IRC | 07:07 | |
| *** fragatina has joined #openstack-containers | 07:08 | |
| *** Bhujay has joined #openstack-containers | 07:10 | |
| *** Bhujay has quit IRC | 07:11 | |
| *** Bhujay has joined #openstack-containers | 07:11 | |
| *** fragatina has quit IRC | 07:18 | |
| *** fragatina has joined #openstack-containers | 07:19 | |
| *** fragatina has quit IRC | 07:25 | |
| *** fragatina has joined #openstack-containers | 07:26 | |
| *** fragatina has quit IRC | 07:31 | |
| *** fragatina has joined #openstack-containers | 07:31 | |
| *** fragatina has quit IRC | 07:44 | |
| *** fragatina has joined #openstack-containers | 07:44 | |
| *** fragatina has quit IRC | 07:49 | |
| *** _fragatina_ has joined #openstack-containers | 07:49 | |
| *** _fragatina_ has quit IRC | 07:59 | |
| *** fragatina has joined #openstack-containers | 08:00 | |
| *** fragatina has quit IRC | 08:13 | |
| *** fragatina has joined #openstack-containers | 08:14 | |
| *** fragatina has quit IRC | 08:22 | |
| *** fragatina has joined #openstack-containers | 08:23 | |
| *** fragatina has quit IRC | 08:26 | |
| *** fragatina has joined #openstack-containers | 08:27 | |
| *** flwang1 has joined #openstack-containers | 08:28 | |
| flwang1 | strigazi: ping | 08:28 |
| flwang1 | strigazi: ping me when you're available | 08:36 |
| *** ricolin has joined #openstack-containers | 08:42 | |
| *** fragatina has quit IRC | 08:45 | |
| *** fragatina has joined #openstack-containers | 08:46 | |
| *** fragatina has quit IRC | 08:46 | |
| *** fragatina has joined #openstack-containers | 08:46 | |
| *** ykarel is now known as ykarel|lunch | 08:55 | |
| *** fragatina has quit IRC | 09:01 | |
| *** fragatina has joined #openstack-containers | 09:01 | |
| *** fragatina has quit IRC | 09:02 | |
| *** fragatina has joined #openstack-containers | 09:03 | |
| *** fragatina has quit IRC | 09:04 | |
| *** fragatina has joined #openstack-containers | 09:05 | |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: [k8s_fedora_atomic] Add heat-container-agent to worker node https://review.openstack.org/561858 | 09:19 |
| strigazi | flwang1: ping | 09:24 |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: [k8s_fedora_atomic] Add heat-container-agent to worker node https://review.openstack.org/561858 | 09:24 |
| flwang1 | strigazi: yes | 09:26 |
| *** pcaruana has joined #openstack-containers | 09:30 | |
| *** ttsiouts has joined #openstack-containers | 09:36 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum master: [k8s] Update cluster health status by native API https://review.openstack.org/572897 | 09:43 |
| *** fragatina has quit IRC | 10:23 | |
| *** fragatina has joined #openstack-containers | 10:24 | |
| *** udesale has quit IRC | 10:25 | |
| *** fragatina has quit IRC | 10:28 | |
| *** fragatina has joined #openstack-containers | 10:29 | |
| *** ricolin has quit IRC | 10:31 | |
| mordred | lxkong: once I have the ci issue fixed on the cloud-provider-openstack patch for annotations and assuming the PR lands - how long does it take for the updated driver to be published to dockerhub? | 10:33 |
| *** salmankhan has joined #openstack-containers | 10:33 | |
| *** udesale has joined #openstack-containers | 10:35 | |
| *** lpetrut has joined #openstack-containers | 10:39 | |
| *** salmankhan has quit IRC | 10:42 | |
| *** salmankhan has joined #openstack-containers | 10:42 | |
| *** brtknr has joined #openstack-containers | 10:42 | |
| *** ykarel|lunch is now known as ykarel | 10:42 | |
| *** salmankhan has quit IRC | 10:46 | |
| *** salmankhan has joined #openstack-containers | 10:53 | |
| *** udesale has quit IRC | 11:01 | |
| lxkong | mordred: hi, my PR https://github.com/kubernetes/cloud-provider-openstack/pull/433 also needs a new release :-) After our PR landed, we could suggest dims and chris hoge to do a new release. | 11:08 |
| mordred | lxkong: cool! | 11:08 |
| lxkong | mordred: btw, my pr will solve the CI failure | 11:08 |
| lxkong | we just need to update the gophercloud revision | 11:08 |
| mordred | oh good. I just did "dep ensure -update github.com/gophercloud/gophercloud" locally and was going to update my PR - should I do that still? | 11:09 |
| lxkong | mordred: you can wait for my PR merged and rebase :-) | 11:10 |
| mordred | awesome. I will do that! :) | 11:10 |
| mordred | it updated more things than I was expecting and I wasn't sure how to decide if those were good changes or not | 11:10 |
| lxkong | mordred: what do you mean by 'updated more things'? Do you mean there are much more changes in gophercloud? | 11:12 |
| openstackgerrit | Merged openstack/magnum master: Support Keystone AuthN and AuthZ for k8s https://review.openstack.org/561783 | 11:12 |
| openstackgerrit | Merged openstack/magnum master: Remove -U from pip install https://review.openstack.org/528556 | 11:12 |
| openstackgerrit | Merged openstack/magnum master: Release note for cluster pre-delete https://review.openstack.org/625138 | 11:12 |
| *** sapd1_x has quit IRC | 11:16 | |
| *** openstackgerrit has quit IRC | 11:22 | |
| *** udesale has joined #openstack-containers | 11:36 | |
| *** ttsiouts has quit IRC | 11:37 | |
| *** ttsiouts has joined #openstack-containers | 11:37 | |
| *** ttsiouts has quit IRC | 11:42 | |
| *** fragatina has quit IRC | 11:54 | |
| *** gyee has joined #openstack-containers | 12:13 | |
| gyee | hi, can someone please help me out with a review? https://review.openstack.org/#/c/628026/ | 12:13 |
| *** ttsiouts has joined #openstack-containers | 12:18 | |
| sayalilunkad | strigazi: Will you have time to take a look at the patch from gyee? | 12:32 |
| strigazi | sayalilunkad: gyee DONE | 12:38 |
| sayalilunkad | strigazi: thanks! | 12:38 |
| gyee | strigazi, thanks | 12:38 |
| strigazi | sayalilunkad: gyee I'll give flwang1 and others change to review until the meeting tonight and then merge. lgtm | 12:39 |
| gyee | great, thanks again | 12:40 |
| sayalilunkad | sounds good | 12:40 |
| *** openstackgerrit has joined #openstack-containers | 12:43 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum master: DNM use python3 for functional tests https://review.openstack.org/629126 | 12:43 |
| *** zul has joined #openstack-containers | 12:44 | |
| *** gyee has quit IRC | 12:44 | |
| *** mkuf_ has joined #openstack-containers | 13:07 | |
| *** mkuf has quit IRC | 13:10 | |
| *** inovas has quit IRC | 13:21 | |
| *** udesale has quit IRC | 13:36 | |
| *** udesale has joined #openstack-containers | 13:36 | |
| *** mkuf_ has quit IRC | 14:39 | |
| *** mkuf_ has joined #openstack-containers | 14:43 | |
| *** Bhujay has quit IRC | 15:02 | |
| *** mkuf_ has quit IRC | 15:04 | |
| *** ykarel has quit IRC | 15:04 | |
| *** mkuf has joined #openstack-containers | 15:05 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum master: Fix py3 compatibility for the ca_handler https://review.openstack.org/630697 | 15:11 |
| *** ttsiouts has quit IRC | 15:19 | |
| *** ttsiouts has joined #openstack-containers | 15:19 | |
| *** ykarel has joined #openstack-containers | 15:23 | |
| *** ttsiouts has quit IRC | 15:24 | |
| *** hongbin has joined #openstack-containers | 15:28 | |
| *** ttsiouts has joined #openstack-containers | 15:28 | |
| *** salmankhan has quit IRC | 15:45 | |
| *** fragatina has joined #openstack-containers | 15:46 | |
| *** salmankhan has joined #openstack-containers | 16:04 | |
| *** munimeha1 has joined #openstack-containers | 16:20 | |
| *** ykarel is now known as ykarel|away | 16:23 | |
| *** fragatina has quit IRC | 16:25 | |
| *** fragatina has joined #openstack-containers | 16:26 | |
| *** ttsiouts has quit IRC | 16:44 | |
| *** ttsiouts has joined #openstack-containers | 16:44 | |
| *** ricolin has joined #openstack-containers | 16:47 | |
| *** fragatina has quit IRC | 16:47 | |
| *** fragatina has joined #openstack-containers | 16:48 | |
| *** fragatina has quit IRC | 16:48 | |
| *** fragatina has joined #openstack-containers | 16:49 | |
| *** ttsiouts has quit IRC | 16:50 | |
| *** salmankhan has quit IRC | 16:50 | |
| *** salmankhan has joined #openstack-containers | 16:50 | |
| *** fragatina has quit IRC | 16:53 | |
| *** fragatina has joined #openstack-containers | 16:53 | |
| *** fragatina has quit IRC | 16:58 | |
| *** fragatina has joined #openstack-containers | 16:59 | |
| *** udesale has quit IRC | 17:04 | |
| *** fragatina has quit IRC | 17:06 | |
| *** ricolin has quit IRC | 17:12 | |
| *** ykarel has joined #openstack-containers | 17:37 | |
| *** ykarel|away has quit IRC | 17:38 | |
| *** ttsiouts has joined #openstack-containers | 17:47 | |
| *** fragatina has joined #openstack-containers | 17:54 | |
| *** pcaruana has quit IRC | 18:01 | |
| *** salmankhan1 has joined #openstack-containers | 18:04 | |
| *** salmankhan has quit IRC | 18:07 | |
| *** salmankhan1 has quit IRC | 18:08 | |
| *** itlinux has joined #openstack-containers | 18:16 | |
| *** ykarel has quit IRC | 18:25 | |
| *** openstacking_123 has joined #openstack-containers | 18:46 | |
| *** inovas has joined #openstack-containers | 19:00 | |
| *** flwang1 has quit IRC | 19:05 | |
| *** lpetrut has quit IRC | 19:38 | |
| *** ttsiouts has quit IRC | 19:45 | |
| *** ttsiouts has joined #openstack-containers | 19:46 | |
| *** ttsiouts has quit IRC | 19:51 | |
| *** itlinux_ has joined #openstack-containers | 20:36 | |
| *** openstacking_123 has quit IRC | 20:37 | |
| *** itlinux has quit IRC | 20:37 | |
| *** itlinux_ has quit IRC | 20:39 | |
| *** colby has joined #openstack-containers | 20:49 | |
| colby | Hey Everyone. We are using magnum on queens and it does not seem to be setting the openstack provider config correctly. We want to use persistent cinder volumes. It appears to only be populating user/pass/authurl but not project or domain. Is magnum supposed to set this up automatically? | 20:50 |
| strigazi | colby: it doesn't need it. it uses the trust-id which contains all the info. I'll get back to you after the meeting. | 20:59 |
| colby | Hmm ok. Im getting "Cloud provider not initialized properly" which is why I wondered if it needed the project/domain | 21:00 |
| strigazi | #startmeeting containers | 21:01 |
| openstack | Meeting started Tue Jan 15 21:01:25 2019 UTC and is due to finish in 60 minutes. The chair is strigazi. Information about MeetBot at http://wiki.debian.org/MeetBot. | 21:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 21:01 |
| *** openstack changes topic to " (Meeting topic: containers)" | 21:01 | |
| openstack | The meeting name has been set to 'containers' | 21:01 |
| strigazi | #topic Roll Call | 21:01 |
| *** openstack changes topic to "Roll Call (Meeting topic: containers)" | 21:01 | |
| strigazi | o/ | 21:01 |
| jakeyip | o/ | 21:02 |
| strigazi | hello jakeyip | 21:03 |
| jakeyip | hi strigazi :) | 21:03 |
| strigazi | #topic Announcements | 21:04 |
| *** openstack changes topic to "Announcements (Meeting topic: containers)" | 21:04 | |
| strigazi | coming releases for rocky and queens: | 21:04 |
| strigazi | queens: https://review.openstack.org/#/c/630959/ | 21:04 |
| strigazi | rocky: https://review.openstack.org/#/c/630960/ | 21:05 |
| strigazi | #topic Stories/Tasks | 21:05 |
| *** openstack changes topic to "Stories/Tasks (Meeting topic: containers)" | 21:05 | |
| strigazi | jakeyip: I think there is only one patch remaining for the tempest plugin | 21:06 |
| jakeyip | the py3 one? | 21:07 |
| strigazi | in a similar context, we need to fix the minor python3 compatibility issues | 21:07 |
| strigazi | jakeyip: yes | 21:07 |
| strigazi | for python 3, this is the CI patch https://review.openstack.org/#/c/629126/ | 21:08 |
| strigazi | and some fixes here: https://review.openstack.org/#/c/618756/ | 21:08 |
| strigazi | I'll anandon my patch: https://review.openstack.org/#/c/630697/ | 21:08 |
| strigazi | jakeyip: if you can take a look to https://review.openstack.org/#/c/618756/ it would be great | 21:09 |
| jakeyip | strigazi: ok. taking a look now. | 21:09 |
| strigazi | Other than that, from my side, I reviewed and merged the keystone auth patch. works great, we just miss the client config | 21:10 |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum master: Fix python3 compatibility https://review.openstack.org/618756 | 21:10 |
| strigazi | I rebased it ^^ | 21:11 |
| eandersson | o/ | 21:11 |
| strigazi | hello eandersson | 21:11 |
| jakeyip | hi eandersson | 21:11 |
| strigazi | we were just talking about the python3 patches. | 21:12 |
| eandersson | Would be nice to get some tests in place that catches these, to make sure we don't have future regressions, but can be done in a follow up patch. | 21:12 |
| strigazi | CI https://review.openstack.org/#/c/629126/ and https://review.openstack.org/#/c/618756/ are the patches. | 21:12 |
| strigazi | eandersson: UTs or functional? | 21:13 |
| eandersson | both ideally, but I think UT is enough here. | 21:13 |
| eandersson | functional should hopefully catch most stuff already | 21:13 |
| strigazi | eandersson: my CI patch makes python3 default | 21:13 |
| strigazi | and I add one more functional for python2 | 21:14 |
| strigazi | for UTs we keep both of course | 21:14 |
| strigazi | eandersson: thoughts? | 21:14 |
| eandersson | sounds good - once we have it working for py3 I might dig into it a bit deeper | 21:14 |
| eandersson | *default | 21:15 |
| jakeyip | strigazi: CI patch looks ok to me. I'm not familiar with zuul (we use jenkins), but I can help test for py3 compatibility in the tests | 21:19 |
| strigazi | jakeyip: ok, the CI patch works, the tests are not passing yet :) | 21:20 |
| eandersson | I think we need this one as well right? https://review.openstack.org/#/c/597994/ | 21:20 |
| strigazi | we may also need the fix from your patch | 21:20 |
| strigazi | correct | 21:20 |
| jakeyip | ok I'll make that change today. sorry have been distracted because our cloud was falling over the last 2 weeks :P | 21:21 |
| strigazi | jakeyip: thanks | 21:22 |
| *** salmankhan has joined #openstack-containers | 21:23 | |
| strigazi | eandersson: jakeyip Do you have anything else to discuss? | 21:27 |
| colin- | o/ | 21:27 |
| colin- | sorry i'm late | 21:27 |
| eandersson | Nothing at this time. I only have one minor commit open. | 21:27 |
| colin- | don't have anything | 21:27 |
| jakeyip | strigazi: nope | 21:27 |
| strigazi | eandersson: which one? | 21:27 |
| eandersson | I'll try to help out with the py3 stuff when I have some time over. | 21:27 |
| eandersson | https://review.openstack.org/#/c/614034/ | 21:28 |
| eandersson | Cleaning up old variables from devstack. | 21:28 |
| eandersson | I also have this commit open for os-loganalyze that will make our devstack logs prettier https://review.openstack.org/#/c/616705/ | 21:28 |
| eandersson | Priorities =] | 21:29 |
| strigazi | :) | 21:29 |
| strigazi | colin-: o/ | 21:30 |
| eandersson | thanks strigazi | 21:30 |
| strigazi | let's end the meeting then, thanks for joining guys, I'll see you next week. (I'll update right now the wiki with the time) | 21:34 |
| eandersson | Thanks! | 21:34 |
| jakeyip | thanks strigazi | 21:34 |
| strigazi | cheers | 21:35 |
| strigazi | #endmeeting | 21:35 |
| colin- | ttyl | 21:35 |
| *** openstack changes topic to "OpenStack Containers Team" | 21:35 | |
| openstack | Meeting ended Tue Jan 15 21:35:07 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:35 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/containers/2019/containers.2019-01-15-21.01.html | 21:35 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/containers/2019/containers.2019-01-15-21.01.txt | 21:35 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/containers/2019/containers.2019-01-15-21.01.log.html | 21:35 |
| *** openstacking_123 has joined #openstack-containers | 21:39 | |
| strigazi | wiki updated: https://wiki.openstack.org/wiki/Meetings/Containers | 21:39 |
| strigazi | eandersson: any clue why 49 lines are excluded? http://logs.openstack.org/56/618756/5/check/openstack-tox-cover/680e067/cover/magnum_common_x509_operations_py.html | 21:41 |
| eandersson | Let me take a quick look | 21:42 |
| eandersson | There are no unit tests for that path strigazi | 21:44 |
| eandersson | or nvm hmm | 21:44 |
| strigazi | eandersson: there is also this error that doesn't make sense to me http://logs.openstack.org/56/618756/5/check/magnum-functional-k8s/8c25391/job-output.txt.gz#_2019-01-15_21_39_42_405704 | 21:49 |
| eandersson | Interesting | 21:51 |
| eandersson | > Wrong type. Expected '<class 'str'>', got '<class 'int'>' | 21:51 |
| strigazi | where do you see that? | 21:52 |
| eandersson | http://logs.openstack.org/56/618756/5/check/magnum-functional-k8s/8c25391/logs/screen-magnum-api.txt.gz#_Jan_15_21_39_40_926258 | 21:53 |
| strigazi | which makes sense. how it is passing on python2? https://git.openstack.org/cgit/openstack/magnum/tree/magnum/api/controllers/v1/cluster.py?id=d9523964488e2325d341782be949bda4973e001a#n113 | 21:56 |
| openstackgerrit | Merged openstack/python-magnumclient master: add python 3.6 unit test job https://review.openstack.org/620496 | 21:56 |
| openstackgerrit | Merged openstack/python-magnumclient master: Use template for lower-constraints https://review.openstack.org/626736 | 21:56 |
| * strigazi left | 21:57 | |
| eandersson | Tested the code path manually and works as intended | 22:06 |
| eandersson | Let me get a patch up | 22:07 |
| openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/magnum master: Use wtypes.text instead of str https://review.openstack.org/631081 | 22:10 |
| *** openstacking_123 has quit IRC | 22:11 | |
| openstackgerrit | Merged openstack/python-magnumclient master: Add Python 3.6 classifier to setup.cfg https://review.openstack.org/621082 | 22:14 |
| openstackgerrit | Merged openstack/magnum master: Removed admin_* from devstack config https://review.openstack.org/614034 | 22:17 |
| eandersson | We might need something like https://github.com/openstack/blazar/blob/stable/rocky/blazar/api/v2/controllers/types.py#L92 | 22:18 |
| jakeyip | hmm, I can no longer replicate the failure condition I needed for the py3 patch https://review.openstack.org/#/c/597994/ . wonder what I'm doing wrong... | 22:31 |
| openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/python-magnumclient master: Use olso_serialization for json https://review.openstack.org/631083 | 22:33 |
| jakeyip | if anyone has logs for py3 failures caused by magnum-tempest-plugin, please pass them to me... | 22:34 |
| *** lpetrut has joined #openstack-containers | 22:34 | |
| *** lpetrut has quit IRC | 22:38 | |
| eandersson | jakeyip, did you look at https://review.openstack.org/#/c/629126/ | 22:40 |
| eandersson | strigazi, my best guess https://review.openstack.org/#/c/631083/ | 22:41 |
| eandersson | but we might have to do what blazar does | 22:43 |
| *** rcernin has joined #openstack-containers | 22:46 | |
| jakeyip | eandersson: thanks, that was helpful! It fails under py35 and not py36. I can fix it up now. | 22:50 |
| *** trident has joined #openstack-containers | 22:52 | |
| openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/magnum master: Use MultiType instead of str https://review.openstack.org/631081 | 23:00 |
| openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/python-magnumclient master: Use olso_serialization for json https://review.openstack.org/631083 | 23:01 |
| *** hongbin has quit IRC | 23:01 | |
| openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/magnum master: [DNM] Testing py36 https://review.openstack.org/631092 | 23:03 |
| openstackgerrit | Jake Yip proposed openstack/magnum-tempest-plugin master: python3 compatibility fix https://review.openstack.org/597994 | 23:08 |
| *** munimeha1 has quit IRC | 23:14 | |
| jakeyip | eandersson, strigazi: py3 patch is up, I've done it another way. | 23:15 |
| eandersson | heh looks very similar to my patch https://review.openstack.org/631083 | 23:17 |
| eandersson | for the client | 23:17 |
| eandersson | hopefully that means I am on the right track | 23:18 |
| jakeyip | that's what openstack docs says https://wiki.openstack.org/wiki/Python3#Serialization:_base64.2C_JSON.2C_etc. | 23:18 |
| openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/magnum master: [DNM] Testing py36 https://review.openstack.org/631092 | 23:19 |
| jakeyip | eandersson: nice, LGTM +1 :) | 23:22 |
| *** salmankhan has quit IRC | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!