| *** henriqueof1 has joined #openstack-containers | 00:45 | |
| *** henriqueof has quit IRC | 00:45 | |
| *** dave-mccowan has quit IRC | 01:31 | |
| *** hongbin has joined #openstack-containers | 02:32 | |
| *** sapd1 has joined #openstack-containers | 03:00 | |
| *** ramishra has joined #openstack-containers | 03:16 | |
| *** hongbin has quit IRC | 03:34 | |
| *** ykarel|away has joined #openstack-containers | 04:06 | |
| *** ykarel|away has quit IRC | 04:43 | |
| *** pcaruana has joined #openstack-containers | 05:03 | |
| *** sapd1_x has joined #openstack-containers | 05:05 | |
| *** udesale has joined #openstack-containers | 05:08 | |
| *** ykarel|away has joined #openstack-containers | 05:13 | |
| *** ykarel|away is now known as ykarel | 06:03 | |
| brtknr | flwang: u there? | 06:17 |
|---|---|---|
| *** sapd1_x has quit IRC | 06:20 | |
| *** henriqueof has joined #openstack-containers | 06:27 | |
| *** henriqueof1 has quit IRC | 06:28 | |
| *** sapd1_x has joined #openstack-containers | 06:46 | |
| brtknr | flwang: i have added comment to bfv patch | 06:49 |
| brtknr | ps12 doesnt work for me either | 06:54 |
| *** lpetrut has joined #openstack-containers | 06:54 | |
| brtknr | none of the cases, no boot_volume_size specified, boot_volume_size=0 and boot_volume_size=20 | 06:54 |
| *** trident has quit IRC | 06:55 | |
| *** sapd1_x has quit IRC | 06:59 | |
| *** udesale has quit IRC | 07:04 | |
| *** trident has joined #openstack-containers | 07:07 | |
| *** Florian has joined #openstack-containers | 07:10 | |
| *** henriqueof has quit IRC | 07:12 | |
| *** henriqueof has joined #openstack-containers | 07:12 | |
| *** ttsiouts has joined #openstack-containers | 07:26 | |
| *** ramishra has quit IRC | 07:26 | |
| *** ramishra has joined #openstack-containers | 07:28 | |
| openstackgerrit | Andreas Jaeger proposed openstack/magnum master: Remove unneeded Zuul branch matcher https://review.opendev.org/681976 | 07:29 |
| *** ttsiouts has quit IRC | 07:36 | |
| *** ttsiouts has joined #openstack-containers | 07:37 | |
| *** ttsiouts has quit IRC | 07:41 | |
| *** ykarel is now known as ykarel|lunch | 07:56 | |
| *** ttsiouts has joined #openstack-containers | 07:59 | |
| *** ivve has joined #openstack-containers | 08:00 | |
| *** AJaeger has joined #openstack-containers | 08:26 | |
| AJaeger | Hi magnum team, the container-build job is failing whenever it runs, see https://review.opendev.org/681976 or https://review.opendev.org/#/c/621734/ - could you investigate what's broken, please? | 08:27 |
| *** ttsiouts has quit IRC | 09:02 | |
| *** ttsiouts has joined #openstack-containers | 09:03 | |
| *** ttsiouts has quit IRC | 09:07 | |
| *** ykarel|lunch is now known as ykarel | 09:08 | |
| strigazi | AJaeger: I left a comment, I found the issue | 09:08 |
| strigazi | brtknr: flwang here? | 09:09 |
| brtknr | strigazi: yeah, argh, fedora:rawhide is constantly changing | 09:09 |
| brtknr | there was no /usr/bin/python there before | 09:10 |
| strigazi | adding na if, should be enough | 09:11 |
| strigazi | is flwang here? I want to sort out the BFV nightmare where I spent a lot of time to do something write | 09:12 |
| strigazi | is flwang here? I want to sort out the BFV nightmare where I spent a lot of time to do something right | 09:12 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Remove unneeded Zuul branch matcher https://review.opendev.org/681976 | 09:13 |
| brtknr | strigazi: ive make it conditional | 09:13 |
| brtknr | strigazi: no he's not here... | 09:14 |
| brtknr | but is your the patch you proposed applied on top of master or PS12? | 09:14 |
| *** ttsiouts has joined #openstack-containers | 09:14 | |
| strigazi | on master | 09:14 |
| brtknr | strigazi: this one right? http://paste.openstack.org/show/775607/ | 09:15 |
| *** udesale has joined #openstack-containers | 09:15 | |
| strigazi | the current ps12 or any PS of that change is crazy, I couldn't spent time to adapt that one. | 09:15 |
| strigazi | yes, I can do it from scratch, it will be faster | 09:15 |
| AJaeger | thanks, strigazi | 09:15 |
| AJaeger | thanks, brtknr - want to update commit message as well to mention the change? | 09:16 |
| brtknr | AJaeger: done | 09:17 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Remove unneeded Zuul branch matcher https://review.opendev.org/681976 | 09:17 |
| *** Florian has quit IRC | 09:19 | |
| *** FlorianFa has joined #openstack-containers | 09:19 | |
| AJaeger | thanks | 09:23 |
| brtknr | strigazi: i dont have strong opinions on bfv, but if it can be done with fewer lines of code, I'd like to see it | 09:26 |
| brtknr | strigazi: it would be nice to get it sorted though, been a lingering issue | 09:28 |
| *** ivve has quit IRC | 09:32 | |
| brtknr | strigazi: even with this change https://review.opendev.org/681922, some pods in the kube-system namespace remain stuck in pending... do we need openstack-controller-manager to untaint nodes? | 09:51 |
| brtknr | strigazi: makes the cluster unusable when cluster_user_trust=False | 09:52 |
| *** udesale has quit IRC | 09:56 | |
| *** udesale has joined #openstack-containers | 09:57 | |
| *** ttsiouts has quit IRC | 09:57 | |
| *** ttsiouts has joined #openstack-containers | 09:57 | |
| brtknr | strigazi: when i remove the taint manually, it works | 10:00 |
| *** ttsiouts has quit IRC | 10:02 | |
| *** ivve has joined #openstack-containers | 10:05 | |
| openstackgerrit | Andreas Jaeger proposed openstack/magnum master: Remove unneeded Zuul branch matcher https://review.opendev.org/681976 | 10:10 |
| *** pcaruana has quit IRC | 10:11 | |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Fix k8s deployment when cluster_user_trust=False https://review.opendev.org/681922 | 11:07 |
| AJaeger | brtknr, strigazi , https://review.opendev.org/681976 now passes, the container-build job is fixed | 11:08 |
| brtknr | strigazi: its fixed finally! | 11:08 |
| brtknr | AJaeger: cool | 11:09 |
| AJaeger | thanks for your help! | 11:09 |
| brtknr | AJaeger: np | 11:10 |
| brtknr | strigazi: mgoddard: andrein: https://review.opendev.org/681922 fix for cluster_user_trust=False | 11:10 |
| brtknr | also goldyfruit ^ | 11:11 |
| mgoddard | nice one brtknr | 11:12 |
| *** lpetrut has quit IRC | 11:20 | |
| *** sapd1_x has joined #openstack-containers | 11:22 | |
| *** pcaruana has joined #openstack-containers | 11:24 | |
| *** udesale has quit IRC | 11:40 | |
| *** udesale has joined #openstack-containers | 11:41 | |
| *** lpetrut has joined #openstack-containers | 11:42 | |
| andrein | brtknr: nice! | 11:45 |
| *** lpetrut has quit IRC | 11:59 | |
| *** lpetrut has joined #openstack-containers | 11:59 | |
| goldyfruit | brtknr, yw | 12:07 |
| *** goldyfruit has quit IRC | 12:12 | |
| *** ttsiouts has joined #openstack-containers | 12:19 | |
| *** sapd1_x has quit IRC | 12:31 | |
| brtknr | strigazi: any thoughts on whether we ought to override CLOUD_PROVIDER_ENABLED to false when cluster_user_trust is false? | 12:43 |
| *** dave-mccowan has joined #openstack-containers | 12:44 | |
| strigazi | Didn't we have this? | 12:45 |
| *** AJaeger has left #openstack-containers | 12:45 | |
| brtknr | strigazi: not that i can see | 12:46 |
| *** lpetrut has quit IRC | 12:48 | |
| *** henriqueof1 has joined #openstack-containers | 12:49 | |
| *** henriqueof has quit IRC | 12:50 | |
| brtknr | strigazi: my question is... whether to fail or override? | 12:50 |
| strigazi | We could do that, not sure if it is a big improvement. It is not bad to have it | 12:50 |
| brtknr | at the moment, it is not clear why the cluster fails when cluster_user_trust is false | 12:51 |
| *** lpetrut has joined #openstack-containers | 12:53 | |
| brtknr | strigazi: but I dont have strong views on this, it was more because mgoddard was asking | 12:53 |
| brtknr | I would like to remove repetitions of `if [ -n "$TRUST_ID" ] && [ "$(echo "${CLOUD_PROVIDER_ENABLED}" | tr '[:upper:]' '[:lower:]')" = "true" ]; then` | 12:55 |
| mgoddard | if the combination of cluster_user_trust=False and CLOUD_PROVIDER_ENABLED=True is always going to fail, it would be nice to catch that case early and explain to the user what they're doing wrong | 12:55 |
| mgoddard | 1 because it's hard to debug why it went wrong and how to fix | 12:55 |
| mgoddard | 2 because if we just skip enabling the cloud provider when cluster_user_trust=False, then we have not done what the user asked | 12:56 |
| mgoddard | if I've misunderstood just say and I'll skulk off back to #openstack-kolla | 12:57 |
| *** dave-mccowan has quit IRC | 12:58 | |
| brtknr | given that we dont test for cluster_user_trust=False scenarios on devstack since its true by default, it does make sense to provide a helpful pointer. | 12:59 |
| brtknr | equally, if cluster_user_trust=false, this would lead to 100% failure rate | 13:05 |
| brtknr | if we were to fail conditional on cluster_user_trust=false and cloud_provider_enabled=true | 13:06 |
| *** goldyfruit has joined #openstack-containers | 13:28 | |
| *** ttsiouts has quit IRC | 13:31 | |
| *** ttsiouts has joined #openstack-containers | 13:32 | |
| *** ttsiouts_ has joined #openstack-containers | 13:34 | |
| *** ttsiouts has quit IRC | 13:35 | |
| openstackgerrit | Merged openstack/magnum master: Remove unneeded Zuul branch matcher https://review.opendev.org/681976 | 13:36 |
| *** ykarel is now known as ykarel|afk | 14:01 | |
| *** ykarel|afk has quit IRC | 14:10 | |
| *** goldyfruit has quit IRC | 14:14 | |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Fix k8s deployment when cluster_user_trust=False https://review.opendev.org/681922 | 14:32 |
| brtknr | mgoddard: ^ | 14:32 |
| *** goldyfruit has joined #openstack-containers | 14:34 | |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Fix k8s deployment when cluster_user_trust=False https://review.opendev.org/681922 | 14:35 |
| *** goldyfruit_ has joined #openstack-containers | 14:42 | |
| *** rcernin has quit IRC | 14:42 | |
| *** lpetrut has quit IRC | 14:43 | |
| *** goldyfruit has quit IRC | 14:44 | |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Fix k8s deployment when cluster_user_trust=False https://review.opendev.org/681922 | 14:56 |
| *** ykarel|afk has joined #openstack-containers | 15:16 | |
| *** ivve has quit IRC | 15:29 | |
| *** udesale has quit IRC | 15:30 | |
| *** udesale has joined #openstack-containers | 15:31 | |
| *** ttsiouts_ has quit IRC | 15:37 | |
| *** ttsiouts has joined #openstack-containers | 15:38 | |
| *** ykarel|afk is now known as ykarel|away | 15:39 | |
| *** ttsiouts has quit IRC | 15:42 | |
| *** ramishra has quit IRC | 15:51 | |
| *** udesale has quit IRC | 17:36 | |
| *** goldyfruit_ has quit IRC | 17:46 | |
| *** goldyfruit_ has joined #openstack-containers | 17:49 | |
| *** goldyfruit___ has joined #openstack-containers | 18:59 | |
| *** goldyfruit_ has quit IRC | 19:02 | |
| *** lpetrut has joined #openstack-containers | 19:14 | |
| *** lpetrut has quit IRC | 19:15 | |
| *** lpetrut has joined #openstack-containers | 19:16 | |
| *** lpetrut has quit IRC | 19:58 | |
| *** ykarel|away has quit IRC | 20:18 | |
| *** mgoerens has quit IRC | 20:41 | |
| *** openstackgerrit has quit IRC | 20:51 | |
| *** henriqueof1 has quit IRC | 20:54 | |
| *** pcaruana has quit IRC | 21:03 | |
| *** KeithMnemonic has quit IRC | 21:19 | |
| *** goldyfruit___ has quit IRC | 22:49 | |
| *** rcernin has joined #openstack-containers | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!