Thursday, 2018-06-28

*** markguz has quit IRC00:03
*** itlinux has joined #openstack-containers00:23
*** Nel1x has joined #openstack-containers00:24
*** spotz has quit IRC01:04
*** yolanda_ has joined #openstack-containers01:05
*** georgem1 has joined #openstack-containers01:07
*** yolanda has quit IRC01:08
*** itlinux has quit IRC01:08
*** yolanda has joined #openstack-containers01:09
*** yolanda_ has quit IRC01:11
*** itlinux has joined #openstack-containers01:12
*** yolanda_ has joined #openstack-containers01:13
*** yolanda has quit IRC01:16
*** yolanda__ has joined #openstack-containers01:19
*** spotz has joined #openstack-containers01:20
*** yolanda_ has quit IRC01:22
*** mdnadeem has joined #openstack-containers01:23
*** georgem1 has quit IRC01:39
*** hongbin has joined #openstack-containers02:06
*** AlexeyAbashkin has joined #openstack-containers02:10
*** ricolin has joined #openstack-containers02:17
*** Nel1x has quit IRC02:21
*** AlexeyAbashkin has quit IRC02:39
*** lpetrut has joined #openstack-containers03:49
*** udesale has joined #openstack-containers03:53
openstackgerritFeilong Wang proposed openstack/magnum master: Make etcd loadbalancer optional  https://review.openstack.org/57454004:00
*** hongbin has quit IRC04:27
*** nguyenhai has left #openstack-containers04:31
*** lpetrut has quit IRC04:32
*** AlexeyAbashkin has joined #openstack-containers04:32
*** ramishra has joined #openstack-containers04:34
*** ykarel has joined #openstack-containers05:00
*** janki has joined #openstack-containers05:00
*** AlexeyAbashkin has quit IRC05:25
*** openstackgerrit has quit IRC05:34
*** ozerovandrei has quit IRC05:34
*** iranzo has joined #openstack-containers05:44
*** openstackstatus has quit IRC05:51
*** openstackstatus has joined #openstack-containers05:51
*** ChanServ sets mode: +v openstackstatus05:51
*** yolanda__ is now known as yolanda05:59
*** flwang1 has quit IRC06:06
*** ramishra has quit IRC06:10
*** ramishra has joined #openstack-containers06:12
*** mjura has joined #openstack-containers06:16
*** lpetrut has joined #openstack-containers06:28
*** AlexeyAbashkin has joined #openstack-containers06:29
*** openstackgerrit has joined #openstack-containers06:30
openstackgerritMerged openstack/magnum stable/queens: Allow multimaster lb with no floating ip option  https://review.openstack.org/57823606:30
*** ramishra has quit IRC06:35
*** ramishra has joined #openstack-containers06:37
*** AlexeyAbashkin has quit IRC06:41
openstackgerritMerged openstack/magnum master: Rename scripts  https://review.openstack.org/56245406:45
*** ramishra has quit IRC06:55
*** ramishra has joined #openstack-containers07:00
*** serlex has joined #openstack-containers07:01
*** peereb has joined #openstack-containers07:09
*** pcaruana has joined #openstack-containers07:10
*** mago_ has joined #openstack-containers07:22
*** ramishra has quit IRC07:23
*** ramishra has joined #openstack-containers07:24
*** ramishra has quit IRC07:29
*** ramishra has joined #openstack-containers07:30
*** yolanda_ has joined #openstack-containers07:35
*** yolanda is now known as Guest6344807:37
*** yolanda_ is now known as yolanda07:37
*** mgoddard has joined #openstack-containers07:37
*** Guest63448 has quit IRC07:38
*** ramishra has quit IRC07:39
*** yasemin has quit IRC07:44
openstackgerritMerged openstack/magnum master: Add option to specify Cgroup driver for Kubelet  https://review.openstack.org/57158307:49
*** ispp has joined #openstack-containers07:54
strigaziflwang: are you here?07:55
*** ykarel is now known as ykarel|lunch08:03
*** threestrands has joined #openstack-containers08:06
*** threestrands has quit IRC08:06
*** threestrands has joined #openstack-containers08:06
*** threestrands has quit IRC08:13
openstackgerritBharat Kunwar proposed openstack/magnum stable/queens: Add option to specify Cgroup driver for Kubelet  https://review.openstack.org/57872508:13
*** mgoddard has quit IRC08:20
*** ramishra has joined #openstack-containers08:25
openstackgerritBharat Kunwar proposed openstack/magnum master: Support disabling floating IPs in swarm mode  https://review.openstack.org/57120008:30
openstackgerritBharat Kunwar proposed openstack/magnum master: Support disabling floating IPs in swarm mode  https://review.openstack.org/57120008:32
openstackgerritBharat Kunwar proposed openstack/magnum master: Support disabling floating IPs in swarm mode  https://review.openstack.org/57120008:33
*** flwang1 has joined #openstack-containers08:36
openstackgerritBharat Kunwar proposed openstack/magnum stable/queens: Add option to specify Cgroup driver for Kubelet  https://review.openstack.org/57872508:40
openstackgerritBharat Kunwar proposed openstack/magnum stable/queens: Add option to specify Cgroup driver for Kubelet  https://review.openstack.org/57872508:43
*** armaan has joined #openstack-containers09:17
*** salmankhan has joined #openstack-containers09:17
*** ykarel|lunch is now known as ykarel09:23
openstackgerritDavid Rabel proposed openstack/magnum master: Docs: Replace non-existing command  https://review.openstack.org/57874109:28
*** pcichy has joined #openstack-containers09:30
*** pcichy has quit IRC09:35
*** pcichy has joined #openstack-containers09:35
*** rcernin has quit IRC09:56
*** armaan has quit IRC10:02
*** armaan has joined #openstack-containers10:02
*** armaan has quit IRC10:03
*** armaan has joined #openstack-containers10:04
*** ispp has quit IRC10:25
*** armaan has quit IRC10:26
*** rabel has joined #openstack-containers10:26
*** armaan has joined #openstack-containers10:27
rabelhi. i get a "Unknown attributes for argument cluster: master_flavor_id, docker_volume_size, flavor_id (HTTP 400)" error from magnum (ocata) when openstack coe cluster create with the newest openstackclient and magnumclient from pypi. all three parameters are already specified in the cluster template. what am i doing wrong?10:29
rabeladditionally specifying them in the openstack coe cluster create does not help either10:30
strigazirabel: you use a newer version of the client than the server10:31
rabelstrigazi: is that a problem? i thought that is why there are api versions and microversions10:31
*** armaan has quit IRC10:31
*** udesale has quit IRC10:32
strigazirabel: i think so, you can try with the ocata client to veridy10:33
strigazirabel: i think so, you can try with the ocata client to verify10:33
openstackgerritDavid Rabel proposed openstack/magnum master: Remove wrong blank line in doc/source/user/index.rst  https://review.openstack.org/57875810:43
*** peereb has quit IRC10:43
*** ramishra has quit IRC10:44
rabelstrigazi: seems that ocata version of magnumclient is noch integrated in openstackclient. so i work directly with magnumclient now. and the error message has changed, which is good :)10:44
*** mdnadeem has quit IRC10:45
*** salmankhan has quit IRC10:45
*** ispp has joined #openstack-containers10:49
*** zul has joined #openstack-containers11:39
*** pcichy has quit IRC11:43
ftersinstrigazi: hi. are you around?11:43
strigaziftersin: yes11:45
ftersinstrigazi: could you please explain the usage of kube:kube in https://github.com/openstack/magnum/commit/faa9e90402bcf78acdd166198fff9612fa8be81c#diff-bcbe4fe100320ff5c4fd8d35ef9b068eR1311:46
ftersinthere is no such user in that container, so it does not work for me11:46
ftersindo i miss something?11:47
*** zul has quit IRC11:49
*** serlex has quit IRC11:49
strigaziftersin: let me check, it seems that you are right. I was able to sign certs with kubernetes, give me a sec11:49
*** ramishra has joined #openstack-containers11:50
strigaziftersin: the deployment is failing or just this line doesn't work?11:51
*** ramishra has quit IRC11:51
*** ramishra has joined #openstack-containers11:51
ftersinstrigazi: my version quite differs from the upstream and it fails there. but i think upstream just doesn't work in this line11:56
*** pcaruana has quit IRC11:56
ftersinit writes: [INFO] {"deploy_stdout": "", "deploy_stderr": "chown: invalid user: 'kube:kube'\n", "deploy_status_code": 0}11:56
ftersinstrigazi: http://paste.openstack.org/show/724517/11:57
openstackgerritMerged openstack/magnum stable/queens: Add option to specify Cgroup driver for Kubelet  https://review.openstack.org/57872512:05
*** sheel has joined #openstack-containers12:06
strigaziftersin: you are right this shouldn't work. I wonder why it works in my env. I'll check later. Just delete these two lines chmod and chown to unblock yourself. I'll check my env12:13
*** Nel1x has joined #openstack-containers12:14
ftersinstrigazi: ok. should i file a bug for that?12:15
*** markguz has joined #openstack-containers12:17
*** markguz has quit IRC12:22
*** zul has joined #openstack-containers12:37
strigaziftersin yes please, a story in storyboard.openstack.org12:40
*** pcaruana has joined #openstack-containers12:44
*** Nel1x has quit IRC12:48
ftersinstrigazi: thank you. the bug: https://storyboard.openstack.org/#!/story/200274812:51
strigaziftersin: thank you12:51
*** rcernin has joined #openstack-containers12:53
*** serlex has joined #openstack-containers12:56
*** chhagarw has joined #openstack-containers13:04
*** chhavi__ has joined #openstack-containers13:04
*** rcernin has quit IRC13:08
*** georgem1 has joined #openstack-containers13:09
*** markguz has joined #openstack-containers13:12
*** chhagarw has quit IRC13:18
*** udesale has joined #openstack-containers13:20
*** markguz has quit IRC13:24
*** hongbin has joined #openstack-containers13:50
*** markguz has joined #openstack-containers13:52
*** ykarel is now known as ykarel|afk13:59
*** lpetrut has quit IRC14:03
*** ykarel|afk has quit IRC14:04
*** mjura has quit IRC14:12
*** ispp has quit IRC14:14
rabelis magnumclient integration into the openstackclient only in master branch?14:23
strigazirabel: in pike http://git.openstack.org/cgit/openstack/python-magnumclient/tree/magnumclient/osc/v1?h=stable/pike14:24
*** itlinux has quit IRC14:26
rabelstrigazi: thanks. so with magnumclient and openstackclient from pike something like openstack cluster list should work, right?14:28
rabelah, sorry. my fault. yes it works :O)14:30
*** lpetrut has joined #openstack-containers14:38
*** markguz has quit IRC14:40
*** markguz has joined #openstack-containers14:41
*** markguz has quit IRC14:45
*** markguz has joined #openstack-containers14:46
*** imdigitaljim has quit IRC14:52
*** mago_ has quit IRC14:58
rabelstrigazi: back to what we were talking about a few hours ago: so the problem seems to be that openstackclient passes parameters to the api that i do not specifiy, with value null. https://git.openstack.org/cgit/openstack/python-magnumclient/tree/magnumclient/osc/v1/clusters.py#n12315:02
rabelcould we fix that by only adding only the parameters to args that are not null?15:02
strigazirabel: I think this was fixed15:03
strigazirabel: no, it isn't in osc15:04
strigazirabel: for labels it is https://github.com/openstack/python-magnumclient/blob/master/magnumclient/osc/v1/clusters.py#L13615:04
rabelstrigazi: yes, if we could do that for the other three parameters, too. it would be perfekt15:05
rabelstrigazi: i can upload a change later, if you want me to15:06
strigazirabel: go for it15:06
rabel:)15:07
*** itlinux has joined #openstack-containers15:11
*** yamamoto has quit IRC15:12
*** yamamoto has joined #openstack-containers15:12
openstackgerritJim Bach proposed openstack/magnum master: Masters are now given the role label for master  https://review.openstack.org/57662315:15
openstackgerritDavid Rabel proposed openstack/python-magnumclient master: osc: Don't pass parameters with null value  https://review.openstack.org/57884315:18
*** imdigitaljim has joined #openstack-containers15:18
*** digitaljim has joined #openstack-containers15:18
*** digitaljim has left #openstack-containers15:19
*** imdigitaljim has quit IRC15:19
*** imdigitaljim has joined #openstack-containers15:19
*** digitaljim has joined #openstack-containers15:19
*** ftersin has quit IRC15:19
*** digitaljim has quit IRC15:19
imdigitaljimstrigazi: you here still?15:20
strigaziimdigitaljim: yeap15:20
imdigitaljimthanks for checking with flannel i was curious with the behavior15:21
imdigitaljimlike I was saying a bit ago, i understand what you meant with the previous discussion. i think until we restore kubelet ill have to use that conditional15:21
imdigitaljimim gonna work on converging all cni deployments here shortly to make it easier going forward15:22
imdigitaljimso the only variation is the cni15:22
strigaziimdigitaljim I was thinking the following15:22
strigaziimdigitaljim: we could run configure minion everywhere to have the kubelet everywhere15:23
strigaziimdigitaljim: then we wouldn't have this issue15:23
strigaziimdigitaljim: it is not a small a change it would clean things up15:24
strigaziimdigitaljim: it is not a small a change, but it would clean things up15:24
imdigitaljimhmmm yeah that could be a start until we do some refactor for naming/etc15:24
imdigitaljimbut yeah that could be the quick fix15:24
strigaziimdigitaljim: Then I want to move flannel to the hosted deployment that uses the kubernetes api for storage15:25
imdigitaljimflannel still works with etcd right?15:26
imdigitaljimshould we make it optional?15:26
imdigitaljimbut that would be a neat feature option15:26
strigaziimdigitaljim: yes, it still works with etcd15:27
strigaziimdigitaljim: I think we kube-api only is better15:27
imdigitaljimfair enough, i think that would be a great option to allow :)15:28
imdigitaljimmaybe we can make it available for all and not CNI dependent15:28
imdigitaljimcalico can do the same15:28
strigazicalico docs are strange for this15:29
imdigitaljimhttps://docs.projectcalico.org/v3.0/getting-started/kubernetes/installation/hosted/kubernetes-datastore/15:29
strigaziin canal they say go for kube-api15:29
imdigitaljimi think we should consider writing it in a generic way as much as possible15:29
strigaziimdigitaljim: IMO, every CNI should have its own deployment script. Isn't that generic enough?15:34
*** lpetrut has quit IRC15:36
imdigitaljimthat sounds great for configuring the .yaml deployment but major control plane changes (etcd/kubelet/proxy) etc should all be left intact15:36
*** lpetrut has joined #openstack-containers15:37
imdigitaljimetcd vs kubeapi as well15:37
strigaziimdigitaljim: oh yes, of course15:37
strigaziimdigitaljim: but etcd kube-api is left in the deployment method of the cni15:37
imdigitaljimthats the one id say can be still chosen separate of the CNI15:38
imdigitaljimbut for now id say using it with just flannel would be a great start15:38
imdigitaljimas in baking it into flannel now sounds good and we can revisit it later making it more generic15:39
*** lxkong has quit IRC15:39
*** ispp has joined #openstack-containers15:40
imdigitaljimits sounds like a good path forward :]15:40
*** ykarel has joined #openstack-containers15:45
*** iranzo has quit IRC15:46
*** sheel has quit IRC15:55
*** udesale has quit IRC15:56
*** udesale has joined #openstack-containers15:56
*** lpetrut has quit IRC16:05
*** ispp has quit IRC16:20
*** janki has quit IRC16:25
*** raopajay has quit IRC16:26
*** udesale has quit IRC16:31
*** ramishra has quit IRC16:34
*** jmlowe has quit IRC16:41
openstackgerritDavid Rabel proposed openstack/python-magnumclient master: osc: Don't pass parameters with null value  https://review.openstack.org/57884316:53
*** jmlowe has joined #openstack-containers16:56
strigaziimdigitaljim: flwang are we going to have a meeting?16:58
flwang1strigazi: i'm around16:59
*** ramishra has joined #openstack-containers16:59
strigaziimdigitaljim: ping16:59
imdigitaljimyeah im here16:59
strigazi#startmeeting containers17:00
openstackMeeting started Thu Jun 28 17:00:44 2018 UTC and is due to finish in 60 minutes.  The chair is strigazi. Information about MeetBot at http://wiki.debian.org/MeetBot.17:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:00
*** openstack changes topic to " (Meeting topic: containers)"17:00
openstackThe meeting name has been set to 'containers'17:00
strigazi#topic Roll Call17:00
*** openstack changes topic to "Roll Call (Meeting topic: containers)"17:00
strigazio/17:01
imdigitaljimo/17:01
flwang1o/17:01
colin-hello17:01
jslatero/17:01
colin-    \o17:01
strigazi#topic Review Action Items17:02
*** openstack changes topic to "Review Action Items (Meeting topic: containers)"17:02
strigazistrigazi to summarize the meeting in a ML mail DONE17:02
strigazi#link http://lists.openstack.org/pipermail/openstack-dev/2018-June/131769.html17:04
strigaziI had two more, to create stories but I wanted to confirm before creating noise in the already heavy storyboard17:05
strigaziWe can discuss them in:17:05
strigazi#topic Blueprints/Bugs/Ideas17:05
*** openstack changes topic to "Blueprints/Bugs/Ideas (Meeting topic: containers)"17:05
strigaziThe first one is pretty clear and it is about moving kubelet/proxy on all nodes.17:06
flwang1s/moving/having ?17:06
strigaziWe had a quick chat with imdigitaljim. I propose to run the minion configuration on all nodes17:06
strigaziyes s/moving/having/17:06
imdigitaljimim gonna post a refactor draft here soon on the k8s bootstrapping files, please discuss changes on it. =) if we like it ill continue forward and validate/enhance it.17:07
imdigitaljimquick fix minion configuration sounds good17:07
imdigitaljimlongterm needs to be cleaner17:07
strigaziwhat does quickfix mean?17:08
strigaziWhich changes?17:08
flwang1i would like to postpone the changing to Stein17:08
flwang1not in Rocky17:08
imdigitaljimthats fine, its just for visibility, doesnt have to be accepted17:09
strigaziflwang1: So only small patches?17:09
flwang1it's milestone 3 now17:09
flwang1i'm not sure if we have enough time to test every corner17:09
strigaziok17:10
imdigitaljimadding the minion configuration to master should work though, lets try it out17:11
flwang1i just wanna to see a stable/rocky17:12
strigaziThe current situation that we configure kubelet in the master is a bit comlplicated. If the change is not big we can go for it17:12
flwang1besides, k8s 1.11 just released, we may need to bump the k8s version in rocky17:13
strigaziflwang1: this is irrelevant17:13
flwang1strigazi: if there is a patch in next 2 weeks, i'm happy to review17:13
strigazichanging the kube verision is easy and we can't make decisions based on that17:14
flwang1yep, 1.11 is not releated, just thinking a loud to avoid i forgot17:14
strigaziwe did that in the past, config should be generic enough17:14
strigaziok, let's see a patch and we decide in the coming weeks.17:15
flwang1Jul 23 - Jul 27R-5Rocky-3 milestone17:15
strigazitwo weeks17:15
flwang1we have about 4 weeks for R-317:15
flwang1or for the worst case, if we can have a patch in good shape by the end of july, i'm happy17:16
strigaziThe next change would be upgrading the heat templates to pike17:16
strigazihow we should make this decision?17:17
strigaziflwang1: imdigitaljim what heat version are you running?17:17
flwang1strigazi: we will be on queens shortly17:18
strigazishould we fork to _v2 drivers for this change?17:18
flwang1strigazi: that would be better to physically isolate the changes17:19
flwang1so yes17:19
imdigitaljimwe're still using the juno ones shipped with magnum17:19
imdigitaljima v2 would be good for stability/going forward as well17:19
*** lpetrut has joined #openstack-containers17:20
strigaziI'm in favor of v2 but keep the scripts common, thoughts?17:20
colin-yeah that sounds resaonable17:20
flwang1scripts is fine17:21
strigaziok then17:21
flwang1and if we do that, i would suggest us resondiering the folder structure17:21
strigaziflwang1: what do you mean?17:21
strigazinot k8s_fedora_atomic_v2?17:22
flwang1i'm happy with k8s_fedora_atomic_v217:23
strigaziok, one moment17:23
cbrummwould moving to a self hosted control plane be possible in a v2 driver?17:23
imdigitaljim^17:23
flwang1i'm talking about magnum/drivers/common/templates/kubernetes/fragments17:24
flwang1i'm a bit weird for me though i haven't got a better idea yet17:24
flwang1s/i'm/it's17:24
imdigitaljimif you're not familiar with self-hosted17:24
imdigitaljimhttps://coreos.com/blog/self-hosted-kubernetes.html17:24
imdigitaljimheres a decent doc to explain17:24
imdigitaljimkubernetes is moving towards self-hosted model17:24
imdigitaljimbut in other words your control plane containers run in k8s instead of as systemd17:25
strigazi#link https://storyboard.openstack.org/#!/story/200275017:25
imdigitaljimjust the kubelet would be systemd17:25
*** ricolin has quit IRC17:25
strigaziwe could do that, but from experience the benefit is not great17:25
flwang1imdigitaljim: we did discussed that way17:26
flwang1and we even want to drop master nodes from user's tenant, no mater it's in vm or container17:26
*** ramishra has quit IRC17:26
colin-for my part i've enjoyed being able to interact with all components of the control plane via the API, much in the same patern the project encourages elsewhere for self-managing resources17:27
flwang1imdigitaljim: and i agree with strigazi, we need to understand the pros and cons17:27
strigazicolin-: I agree on it, it is much easier17:27
strigazibut17:27
strigazistreamlining the process we deliver kubelet is more important and if we do it for kubelet we can do it for all components17:28
imdigitaljimcontrol plane updates in place, improved HA, better introspection on the cluster,17:28
colin-ok17:28
imdigitaljimcontrol plane access through k8s API17:28
cbrummI'm sure a doc could be put together to help weigh the idea17:29
*** kbyrne has quit IRC17:29
imdigitaljim#link https://coreos.com/blog/self-hosted-kubernetes.html17:29
strigaziimdigitaljim: colin- cbrumm we had this actually before.17:29
*** kbyrne has joined #openstack-containers17:29
strigaziThe reason we shifted from was that the way to deliver the same software kubernetes whose being fragmented.17:30
strigaziThe reason we shifted from was that the way to deliver the same software kubernetes was fragmented.17:30
strigaziI'm not against it in principle, but at the moment it would complicate things17:31
cbrummok, I just figured that if a v2 was being considered it might be a good time to bring the control plane in line with kubernetes current recommendations. Use kubeadm etc17:31
cbrummbut it's not small consideration for sure17:32
flwang1those kinds of things really need specs17:32
strigaziself-hosted kubernetes would be certainly done with kubeamd.17:32
imdigitaljimi think it could cut our bootstrapping code into about 1/4 of what it is17:32
flwang1we can't make decision here17:32
*** ykarel has quit IRC17:32
cbrummflwang1 agreed17:32
imdigitaljim^17:32
strigaziso, we need a detailed proposal and target it to S17:34
cbrummthat sounds good17:34
cbrummI can't commit that we'll get to that immediately, but if we can't then it's only our fault if it slips17:35
strigazi#link https://storyboard.openstack.org/#!/story/200275117:35
cbrummthanks strigazi17:36
openstackgerritJim Bach proposed openstack/magnum master: Kubernetes Bootrapping IDEAS (not intended as PR)  https://review.openstack.org/57894517:36
strigazinext, I have one thing we need for Rocky and discovered this week17:38
strigaziWhile doing some benchmarks with spark and kubernetes. we fould out that the way magnum is interacting with heat, actually hurts heat a lot.17:39
imdigitaljimoh?17:39
imdigitaljimwhat does it do?17:39
strigaziIn magnum, we don't have locking between the conductor17:39
strigaziand when the conductor(s) try to sync the status with heat they do a stack get.17:40
strigaziWhen a stack is CREATE|UDPATE_IN_PROGRESS we only need the status17:40
strigazinot the outputs of the stack or any metadata.17:41
cbrummYeah that sounds like overkill17:41
strigaziFor large stacks, evaluating the stack outputs  is expensive17:41
colin-have seen similar patterns in other projects17:41
colin-is there a better query?17:42
strigaziand when many processes hit the heat api and queue requestsm rabbit gets overloaded and eventually heat goes down17:42
strigaziThe better query is heat list and filter of one id17:42
imdigitaljimthat sounds like a great patch to get done17:42
flwang1strigazi: can we use stack list instead of get?17:42
flwang1and then matching the id in magnum17:42
strigazifilters={"id": some_stack_id}17:42
colin-nice17:43
flwang1strigazi: can the fileters accept a list of id?17:43
strigazia list takes a second, a get takes a minute for a cluster with 200 nodes17:43
strigaziyeap tested it17:43
flwang1but anyway, we should be able to do that17:43
flwang1strigazi: cool, have a patch already?17:43
flwang1i'm keen to review it17:43
strigaziI'll push a patch I have it almost ready17:44
imdigitaljimanother issue with heat/magnum cleanup. if the cloud-controller-manager allocates something like a loadbalancer and the cluster is deleted.. the LB is dangling and not cleaned up because heat nor magnum know about it as it stands17:44
strigaziThere is a patch in review17:44
strigaziCan you dig it? I can send it you too17:44
strigaziCan you dig it? I can send it to you too17:44
flwang1strigazi: the heat issue? sure17:45
strigaziflwang1: the lbaas clean up17:45
strigazifor the heat issue I'll push it17:45
flwang1in our testing, we can see the lb is deleted successfully17:45
imdigitaljimis this allocated through cloud-controller-manager?17:46
strigazifor lbass (created by the cloud provider)17:46
flwang1without cloud-controller-manager17:46
imdigitaljimusing kubernetes17:46
strigazithere is a patch17:46
imdigitaljimyeah the normal LB's allocated by heat are fine17:46
*** jmlowe has quit IRC17:46
strigazigive me a sec17:46
flwang1we haven't tried that way17:46
flwang1i'm happy to test and fix it17:46
imdigitaljimin other words, kubernetes users need to use load balancers17:46
imdigitaljimkubernetes talks to neutron to get a LB17:46
flwang1imdigitaljim: i see17:46
flwang1it's for service LB not masters LB17:47
strigazi#link https://review.openstack.org/#/c/497144/17:47
imdigitaljimneutron/octavia17:47
imdigitaljimcorrect17:47
imdigitaljimim not sure what all we can/should do to rectify it, we might want to setup a BP for that17:47
flwang1strigazi: thanks, will dig it17:47
strigaziit's the linked one, have a look17:48
imdigitaljimoh yes17:48
imdigitaljimstrigazi: ill review this https://review.openstack.org/#/c/497144/3/magnum/drivers/heat/driver.py17:48
imdigitaljimerr https://review.openstack.org/#/c/497144/17:48
strigaziimdigitaljim: thanks17:48
strigaziand quickly one more17:48
imdigitaljimgreat callout for it =)17:48
strigaziThe keypait issue:17:49
strigaziThe keypair issue:17:49
strigazi#link https://storyboard.openstack.org/#!/story/200264817:49
strigaziif you offer magnum to others and they want you to do things for them it's pretty important17:50
strigaziThe idea is to create a keypait per stack17:50
strigaziThe idea is to create a keypair per stack17:50
strigaziSo when a client wants you to scale or fix their stack with stack update you should be able to do it.17:51
strigaziI'll push a patch for it as well.17:52
strigaziApologies for this meeting folks, we can try to make the next one more structured.17:53
colin-no complaints here :)17:54
imdigitaljimits complaints here as well17:54
imdigitaljims/its/no17:54
*** lpetrut has quit IRC17:54
strigazi:)17:54
strigaziDo tou need to bring up anything else?17:55
flwang1i need testing from imdigitaljim for https://review.openstack.org/57602917:56
flwang1or any other bless for this one17:56
cbrummNope, I think we've covered everything17:56
*** markguz has quit IRC17:56
imdigitaljimflwang: im on it by the end of this week17:56
imdigitaljimi should post about it early next week17:57
flwang1imdigitaljim: cool, thanks17:57
*** markguz has joined #openstack-containers17:57
cbrummThank you guys for hosting this again. It's great to get some solid time with you both17:57
strigaziLet's meet in 6 days and 23 hours again :)17:57
cbrummsounds like a plan17:57
flwang1good to discuss with you guys, very productive17:58
strigazicbrumm: imdigitaljim colin- flwang1 jslater thanks for joining, see you soon17:58
colin-ttyl17:59
flwang1ttyl17:59
cbrummHave a good evening/day17:59
strigazicheers!17:59
strigazi#endmeeting17:59
*** openstack changes topic to "OpenStack Containers Team"17:59
openstackMeeting ended Thu Jun 28 17:59:32 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:59
openstackMinutes:        http://eavesdrop.openstack.org/meetings/containers/2018/containers.2018-06-28-17.00.html17:59
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/containers/2018/containers.2018-06-28-17.00.txt17:59
openstackLog:            http://eavesdrop.openstack.org/meetings/containers/2018/containers.2018-06-28-17.00.log.html17:59
*** markguz has quit IRC18:01
*** jmlowe has joined #openstack-containers18:08
*** pcaruana has quit IRC18:14
*** yolanda_ has joined #openstack-containers18:25
*** yolanda has quit IRC18:27
*** markguz has joined #openstack-containers18:28
*** markguz has quit IRC18:38
*** markguz has joined #openstack-containers18:39
*** armaan has joined #openstack-containers18:40
*** markguz has quit IRC18:43
*** armaan has quit IRC18:45
*** armaan has joined #openstack-containers18:45
openstackgerritDavid Rabel proposed openstack/python-magnumclient master: osc: Don't pass parameters with null value  https://review.openstack.org/57884318:47
*** markguz_ is now known as markguz18:47
*** armaan has quit IRC18:50
*** chhavi__ has quit IRC18:58
*** itlinux has quit IRC19:29
*** itlinux has joined #openstack-containers19:31
*** georgem1 has quit IRC19:31
*** georgem1 has joined #openstack-containers19:31
*** georgem1 has quit IRC19:36
*** flwang1 has quit IRC19:58
*** georgem1 has joined #openstack-containers20:02
*** georgem1 has quit IRC20:04
*** georgem1 has joined #openstack-containers20:04
*** georgem1 has quit IRC20:04
*** zul has quit IRC20:10
*** itlinux has quit IRC20:28
*** itlinux has joined #openstack-containers20:42
openstackgerritJim Bach proposed openstack/magnum master: Enable magnum to allow load balancers to be in their own subnet  https://review.openstack.org/57757021:00
*** flwang1 has joined #openstack-containers21:04
imdigitaljimflwang1: just tested https://review.openstack.org/#/c/576029/421:16
imdigitaljimworks great21:16
imdigitaljimgood job!21:16
flwangimdigitaljim: fantastic, thanks a lot21:17
flwangplease update it as a comment to the patch, so that strigazi can be confident to merge it21:17
flwangas for the issue he mentioned in the review comments,  i have already opened a new task to track it, i will add a comment too21:18
flwangcheers21:18
itlinuxHi Flwang: I have checked looks like the script on step 5 is failing to get the cert21:25
itlinuxany tips on that..21:25
flwangfirstly, manually run the script to check the connection b/w your vm and your keystone/magnum21:26
flwangsecondly, let's see the result of step #1 :)21:27
*** serlex has quit IRC21:33
itlinuxok sounds good thanks21:37
*** rcernin has joined #openstack-containers21:42
flwangcool21:42
*** itlinux has quit IRC21:50
openstackgerritRick Cano proposed openstack/magnum master: Fixing coreos driver - WIP  https://review.openstack.org/57902621:58
*** lxkong has joined #openstack-containers22:04
*** hongbin has quit IRC22:16
*** linkmark has quit IRC22:38
*** threestrands has joined #openstack-containers23:05
*** threestrands has quit IRC23:05
*** threestrands has joined #openstack-containers23:05
*** yolanda__ has joined #openstack-containers23:07
*** yolanda_ has quit IRC23:11
*** yolanda__ has quit IRC23:18
*** threestrands has quit IRC23:38
*** masber has quit IRC23:48

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