| *** goldyfruit has quit IRC | 00:03 | |
| *** goldyfruit has joined #openstack-containers | 00:07 | |
| *** rcernin has quit IRC | 00:17 | |
| *** rcernin has joined #openstack-containers | 00:18 | |
| *** rcernin has quit IRC | 00:49 | |
| *** rcernin has joined #openstack-containers | 00:50 | |
| *** hongbin has joined #openstack-containers | 01:10 | |
| *** ricolin has joined #openstack-containers | 01:13 | |
| *** udesale has quit IRC | 02:15 | |
| *** goldyfruit has quit IRC | 02:56 | |
| *** pcaruana has joined #openstack-containers | 04:22 | |
| *** ykarel|afk has joined #openstack-containers | 04:31 | |
| *** ivve has quit IRC | 04:32 | |
| *** hongbin has quit IRC | 04:49 | |
| *** janki has joined #openstack-containers | 04:51 | |
| *** ykarel|afk is now known as ykarel | 05:10 | |
| *** ykarel_ has joined #openstack-containers | 05:13 | |
| *** ykarel has quit IRC | 05:15 | |
| *** ykarel__ has joined #openstack-containers | 05:16 | |
| *** ykarel_ has quit IRC | 05:19 | |
| *** jchhatbar has joined #openstack-containers | 05:27 | |
| *** janki has quit IRC | 05:30 | |
| *** xiangdong-huang has quit IRC | 05:33 | |
| *** ivve has joined #openstack-containers | 05:38 | |
| *** lpetrut has joined #openstack-containers | 06:01 | |
| *** ykarel__ is now known as ykarel|afk | 06:14 | |
| *** yolanda has quit IRC | 06:38 | |
| *** ramishra has joined #openstack-containers | 06:48 | |
| *** ykarel|afk is now known as ykarel | 06:55 | |
| *** jhesketh has quit IRC | 07:11 | |
| *** ricolin_ has joined #openstack-containers | 07:12 | |
| *** ricolin has quit IRC | 07:15 | |
| *** jhesketh has joined #openstack-containers | 07:29 | |
| *** logan- has quit IRC | 07:33 | |
| *** logan- has joined #openstack-containers | 07:36 | |
| *** ykarel is now known as ykarel|lunch | 08:00 | |
| *** jaewook_oh has joined #openstack-containers | 08:05 | |
| *** nfakhir_ is now known as nfakhir | 08:26 | |
| *** nfakhir is now known as nfakhir_ | 08:28 | |
| *** mgoddard has joined #openstack-containers | 08:41 | |
| *** sapd1_x has joined #openstack-containers | 08:53 | |
| *** jaewook_oh has quit IRC | 08:57 | |
| *** sapd1_x has quit IRC | 09:10 | |
| *** jchhatba_ has joined #openstack-containers | 09:18 | |
| *** jchhatbar has quit IRC | 09:21 | |
| *** ykarel|lunch is now known as ykarel | 09:33 | |
| *** sapd1_x has joined #openstack-containers | 09:54 | |
| *** ricolin_ has quit IRC | 10:02 | |
| *** ramishra has quit IRC | 10:11 | |
| *** nfakhir_ has quit IRC | 10:19 | |
| *** ramishra has joined #openstack-containers | 10:25 | |
| *** dave-mccowan has joined #openstack-containers | 11:14 | |
| *** ttsiouts has joined #openstack-containers | 11:33 | |
| *** ttsiouts has quit IRC | 11:38 | |
| *** mkuf_ has joined #openstack-containers | 11:56 | |
| *** mkuf has quit IRC | 12:00 | |
| *** mkuf_ has quit IRC | 12:21 | |
| *** mkuf_ has joined #openstack-containers | 12:24 | |
| *** henriqueof has quit IRC | 12:56 | |
| *** dave-mccowan has quit IRC | 13:00 | |
| *** henriqueof has joined #openstack-containers | 13:04 | |
| *** zul has joined #openstack-containers | 13:08 | |
| *** jchhatba_ has quit IRC | 13:08 | |
| *** jchhatba_ has joined #openstack-containers | 13:08 | |
| *** goldyfruit has joined #openstack-containers | 13:11 | |
| *** goldyfruit_ has joined #openstack-containers | 13:25 | |
| *** goldyfruit has quit IRC | 13:27 | |
| *** goldyfruit__ has joined #openstack-containers | 13:35 | |
| *** goldyfruit_ has quit IRC | 13:37 | |
| *** henriqueof has quit IRC | 13:49 | |
| *** jchhatba_ has quit IRC | 13:55 | |
| *** itlinux has quit IRC | 14:19 | |
| *** lpetrut has quit IRC | 14:30 | |
| *** itlinux has joined #openstack-containers | 14:32 | |
| *** ykarel has quit IRC | 14:35 | |
| *** dasp has quit IRC | 14:36 | |
| *** ykarel has joined #openstack-containers | 14:53 | |
| *** lpetrut has joined #openstack-containers | 14:59 | |
| *** udesale has joined #openstack-containers | 15:00 | |
| *** ivve has quit IRC | 15:03 | |
| *** nfakhir has joined #openstack-containers | 15:08 | |
| *** ykarel is now known as ykarel|away | 15:13 | |
| *** nfakhir is now known as nfakhir_ | 15:26 | |
| *** itlinux has quit IRC | 15:35 | |
| *** ykarel|away has quit IRC | 15:38 | |
| *** lpetrut has quit IRC | 15:38 | |
| *** spiette_ has joined #openstack-containers | 15:41 | |
| *** spiette has quit IRC | 15:41 | |
| *** dioguerra has quit IRC | 15:51 | |
| *** ykarel|away has joined #openstack-containers | 16:09 | |
| *** ricolin_ has joined #openstack-containers | 16:10 | |
| *** ivve has joined #openstack-containers | 16:22 | |
| *** goldyfruit__ has quit IRC | 16:22 | |
| *** ramishra has quit IRC | 16:33 | |
| *** itlinux has joined #openstack-containers | 16:35 | |
| *** itlinux has quit IRC | 16:43 | |
| *** itlinux has joined #openstack-containers | 16:46 | |
| *** itlinux has quit IRC | 16:58 | |
| *** ricolin_ has quit IRC | 17:00 | |
| *** itlinux has joined #openstack-containers | 17:01 | |
| *** cbrumm__ has quit IRC | 17:02 | |
| *** sapd1_x has quit IRC | 17:05 | |
| *** goldyfruit has joined #openstack-containers | 17:14 | |
| *** irclogbot_1 has quit IRC | 17:17 | |
| *** irclogbot_0 has joined #openstack-containers | 17:18 | |
| *** ykarel|away has quit IRC | 17:34 | |
| *** ykarel|away has joined #openstack-containers | 17:35 | |
| *** dtruong has quit IRC | 17:43 | |
| *** dtruong has joined #openstack-containers | 17:44 | |
| *** ttsiouts has joined #openstack-containers | 17:49 | |
| *** ttsiouts has quit IRC | 17:59 | |
| *** ykarel|away has quit IRC | 18:01 | |
| *** itlinux has quit IRC | 18:02 | |
| *** hongbin has joined #openstack-containers | 18:03 | |
| *** itlinux has joined #openstack-containers | 18:17 | |
| *** lpetrut has joined #openstack-containers | 18:47 | |
| *** lpetrut has quit IRC | 18:51 | |
| *** itlinux has quit IRC | 19:09 | |
| *** itlinux has joined #openstack-containers | 19:13 | |
| openstackgerrit | Bharat Kunwar proposed openstack/python-magnumclient master: Conditional hidden arg for backward compatibility https://review.opendev.org/659298 | 19:17 |
|---|---|---|
| *** pcaruana has quit IRC | 19:30 | |
| *** goldyfruit has quit IRC | 19:31 | |
| *** goldyfruit has joined #openstack-containers | 19:34 | |
| *** dasp has joined #openstack-containers | 20:20 | |
| *** dasp has quit IRC | 20:38 | |
| *** dasp has joined #openstack-containers | 20:38 | |
| jakeyip | flwang: is there going to be a meeting today/ | 21:02 |
| flwang | jakeyip: yes | 21:04 |
| flwang | Spyros said he will be online but seems no | 21:04 |
| flwang | jakeyip: we can have a casual chat if you don't mind | 21:04 |
| jakeyip | sure | 21:05 |
| flwang | do you want me start first? or you have some topics you want to discuss first? | 21:05 |
| jakeyip | sure I can start | 21:06 |
| flwang | cool, i'm listening | 21:06 |
| jakeyip | https://review.opendev.org/659298 - thanks Bharat Kunwar for working on this. This is good for now; I am wondering if we need microversion to solve this properly. | 21:06 |
| flwang | jakeyip: we do need a release | 21:07 |
| flwang | jakeyip: can you fix the pep8 issue and cover job (if you can)? | 21:08 |
| flwang | then i can approve and release it | 21:08 |
| jakeyip | I think he's still actively working on it | 21:08 |
| flwang | jakeyip: ok, no problem | 21:08 |
| *** dave-mccowan has joined #openstack-containers | 21:08 | |
| flwang | brtknr: are you there? | 21:08 |
| jakeyip | hm. want to continue while we wait? | 21:11 |
| jakeyip | https://review.opendev.org/#/c/657915/ - this I think is the last patch that's preventing our cluster from coming up after a reboot of master | 21:13 |
| flwang | jakeyip: sure | 21:14 |
| jakeyip | normally we can carry patches in-house; we don't need to bug upstream. this needs help from upstream because containers are on dockerhub. | 21:14 |
| jakeyip | thanks flwang :) | 21:14 |
| jrosser | i have a patch that i'd like reviews/opinions on - my rocky deployment was broken by this https://review.opendev.org/#/c/660316/ | 21:14 |
| flwang | i just added spyros for reviewing that one | 21:15 |
| flwang | jrosser: let me see | 21:15 |
| flwang | jrosser: i cannot see any comment why this one needs to be reverted | 21:16 |
| flwang | jrosser: would you mind adding comments to explain more? | 21:16 |
| jrosser | could you help me understand what is unclear? | 21:17 |
| flwang | jrosser: it could help the reviewers to understand why it's needed | 21:17 |
| flwang | jrosser: ah, sorry, you changed the commit message | 21:17 |
| jrosser | right - the commit message is quite extensive - if the proxy config needed for the magnum api service is different from the end user VM then everything is broken | 21:17 |
| flwang | jrosser: i see | 21:17 |
| flwang | jrosser: cool, i will take a look | 21:18 |
| jakeyip | jrosser: I'll take a look too. which is the file that allows a user config to set conductor's proxy? | 21:19 |
| jrosser | it depends on the deployment, in openstack-ansible it would be environment config in a systemd unit for example | 21:20 |
| *** itlinux has quit IRC | 21:20 | |
| jrosser | but imho this is the busness of deployment tooling, not the service itself | 21:20 |
| flwang | jrosser: did you change any code when doing revert ? or just simple revert it by the button on gerrit dashboard? | 21:21 |
| jrosser | i think it needed a very minor adjustment by hand, one moment | 21:21 |
| jrosser | the only thing which was not totally clean was the imports at the top of this file https://review.opendev.org/#/c/660316/1/magnum/drivers/heat/template_def.py | 21:23 |
| *** dave-mccowan has quit IRC | 21:23 | |
| *** itlinux has joined #openstack-containers | 21:24 | |
| flwang | jrosser: ok | 21:25 |
| jrosser | it's late here i have to head off - thankyou for your time | 21:25 |
| flwang | for general cases, it doesn't make sense if we just revert the commit for a stable branch, we need to make sure if we also have the same issue in master branch | 21:26 |
| jakeyip | it seems like there needs to be a clear definition between proxy config for clients and magnum components | 21:26 |
| jrosser | so that is a revert for master | 21:27 |
| jrosser | i was a bit surprised a change like that got backported so far | 21:27 |
| jakeyip | there might be cases where proxy is neded for VMs but not magnum services, vice-versae | 21:27 |
| jrosser | yes this is exactly my situation | 21:27 |
| jrosser | being clear about internal / external networks is very important in my environment | 21:28 |
| jakeyip | zigo mentioned that his VMs has internet connectivity and not his services, so it is not a rare situation | 21:29 |
| flwang | jrosser: sorry, i misunderstood, because i was thinking it's for stable/rocky | 21:29 |
| jrosser | flwang: no i revert initially to master and will do the work on the same for the stable branches if there is a agreement on a solution | 21:29 |
| jakeyip | jrosser: is there any issues with you setting proxy on your magnum services currently? | 21:29 |
| jrosser | no, becasue it can be done trivially in the systemd unit that runs the service | 21:30 |
| jrosser | and it needs a different setting to user VM | 21:30 |
| flwang | jrosser: i appreciate for the work, but I will be very careful, because we have customers using proxy in their env and i may need time to make sure there is no regression issue | 21:30 |
| jrosser | flwang: understood - but again i think really that the proxy environment of the services is something that should be handled by deployment tooling | 21:31 |
| jrosser | there are no other openstack services that i know of which have any specific handling of proxies | 21:31 |
| *** udesale has quit IRC | 21:31 | |
| flwang | jrosser: i agree, but we may need a solution to fix both issues | 21:33 |
| jrosser | flwang: to be clear i do have proxies in my environment, and i need magnum to work with proxy A for the service and B for the VM | 21:33 |
| flwang | i mean maybe not a simply revert, does that make sense? | 21:33 |
| jrosser | reverting this patch allows that to work | 21:33 |
| flwang | jrosser: i understand your goal to make your setting works | 21:33 |
| flwang | but what i mean is, we may need a "new" patch(patchset) to fix the original issue and yours | 21:34 |
| jakeyip | I feel we need to be less aggressive with backports | 21:34 |
| jrosser | ok, no problem, like i say i feel the original issue could have been solved without a patch to magnum | 21:35 |
| flwang | jrosser: could be, we may need your help to do some tests later | 21:36 |
| flwang | jrosser: may i know your timezone and company? | 21:37 |
| jrosser | flwang: UK TZ and i work at the BBC | 21:37 |
| flwang | jrosser: fantastic | 21:38 |
| flwang | good to know BBC is using Magnum | 21:38 |
| jrosser | most of my contribution is to #openstack-ansible where i've added tons of proxy and isolated control-plane stuff | 21:38 |
| flwang | oh, is BBC using Magnum or K8s in house? | 21:39 |
| jrosser | magnum as a route to small trials with k8s, but very early days so far | 21:39 |
| flwang | jrosser: i see. then please feel free to ask questions here | 21:40 |
| jrosser | sure, thanks again :) now really i must go sleep! | 21:41 |
| flwang | CERN, NEctar, Blizzard, Catalyst Cloud, we're all using magnum on prod | 21:41 |
| flwang | jrosser: cool, have a good night | 21:41 |
| flwang | jakeyip: let's continue? | 21:41 |
| jakeyip | good night jrosser | 21:41 |
| jakeyip | ok flwang | 21:42 |
| jakeyip | flwang: I only have one more patch https://review.opendev.org/#/c/655875/ this has tripped more than one user up | 21:43 |
| jakeyip | I don't think 0.2.0 works very well and we should do away with it by default | 21:43 |
| flwang | jakeyip: https://review.opendev.org/#/c/655875/3 is easy one | 21:44 |
| flwang | I can +2 it now | 21:44 |
| flwang | but for previous stable version, i'd suggest define the version in template | 21:45 |
| jakeyip | thanks. I do not think we should backport this | 21:45 |
| flwang | jakeyip: i will be back in 10 mins, stand up | 21:45 |
| jakeyip | ok | 21:45 |
| jakeyip | oh by the way let me know if you want release notes for this change | 21:51 |
| flwang | jakeyip: it would be nice if we can have a release note for this version change | 21:52 |
| jakeyip | ok | 21:56 |
| jakeyip | last thing I need your help with - I can't get devstack working so couldn't test your change. If you have a working local.conf please let me know. | 21:58 |
| *** itlinux has quit IRC | 22:02 | |
| *** rcernin has quit IRC | 22:06 | |
| *** dave-mccowan has joined #openstack-containers | 22:12 | |
| flwang | jakeyip: ah, sorry, i forgot that one, wait a sec | 22:18 |
| flwang | jakeyip: http://paste.openstack.org/show/752196/ | 22:19 |
| jakeyip | thanks! | 22:21 |
| *** absubram has joined #openstack-containers | 22:21 | |
| flwang | jakeyip: sorry for the late | 22:28 |
| flwang | jakeyip: i'm working on the rolling upgrade as spyros is away, my work is on https://review.opendev.org/660851 | 22:30 |
| flwang | it's basically working now | 22:30 |
| *** dave-mccowan has quit IRC | 22:31 | |
| *** goldyfruit has quit IRC | 22:49 | |
| *** goldyfruit has joined #openstack-containers | 23:03 | |
| *** dave-mccowan has joined #openstack-containers | 23:14 | |
| *** rcernin has joined #openstack-containers | 23:21 | |
| *** absubram has quit IRC | 23:22 | |
| *** goldyfruit has quit IRC | 23:28 | |
| *** goldyfruit has joined #openstack-containers | 23:32 | |
| *** dave-mccowan has quit IRC | 23:33 | |
| *** dave-mccowan has joined #openstack-containers | 23:46 | |
| *** dave-mccowan has quit IRC | 23:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!