Tuesday, 2019-02-05

*** hongbin has quit IRC00:07
*** sdake has quit IRC00:25
*** sdake has joined #openstack-containers00:30
*** sdake has quit IRC00:30
*** N3l1x has joined #openstack-containers00:32
*** Nel1x has quit IRC00:32
*** flwang has quit IRC00:42
*** sdake has joined #openstack-containers00:56
*** sdake has quit IRC01:03
*** sdake has joined #openstack-containers01:19
*** sdake has quit IRC01:27
*** sdake has joined #openstack-containers01:35
*** flwang has joined #openstack-containers01:48
*** ricolin has joined #openstack-containers01:50
*** sdake has quit IRC01:52
*** ricolin has quit IRC02:03
*** sdake has joined #openstack-containers02:20
*** hongbin has joined #openstack-containers03:00
*** jento has quit IRC03:05
*** sdake has quit IRC03:17
*** sdake has joined #openstack-containers03:19
*** ykarel|away has joined #openstack-containers03:23
*** hongbin has quit IRC03:34
*** sdake has quit IRC03:35
*** sdake has joined #openstack-containers03:58
*** sdake has quit IRC04:16
*** udesale has joined #openstack-containers04:17
*** jento has joined #openstack-containers04:39
*** sdake has joined #openstack-containers04:45
*** lpetrut has joined #openstack-containers04:46
*** dims has quit IRC04:58
*** ykarel|away is now known as ykarel04:59
*** N3l1x has quit IRC05:19
*** lpetrut has quit IRC05:23
*** janki has joined #openstack-containers05:42
*** ykarel has quit IRC05:44
*** ykarel has joined #openstack-containers05:53
*** sdake has quit IRC06:03
*** dims has joined #openstack-containers06:48
*** dims has quit IRC06:52
*** dims has joined #openstack-containers06:54
*** dims has quit IRC06:59
*** dims has joined #openstack-containers07:02
*** pcaruana has joined #openstack-containers07:45
*** ricolin has joined #openstack-containers07:50
*** ricolin has quit IRC08:03
*** belmoreira has joined #openstack-containers08:09
*** lpetrut has joined #openstack-containers08:21
*** lpetrut has quit IRC08:21
*** lpetrut has joined #openstack-containers08:22
*** lpetrut has quit IRC08:28
*** ykarel is now known as ykarel|lunch08:44
*** ttsiouts has joined #openstack-containers08:45
*** ttsiouts has quit IRC08:55
*** ttsiouts has joined #openstack-containers09:08
openstackgerritMerged openstack/magnum stable/rocky: [k8s_fedora_atomic] Delete floating ip for load balancer  https://review.openstack.org/63462010:00
*** ykarel|lunch is now known as ykarel10:05
*** ign0tus has joined #openstack-containers10:11
*** yolanda has quit IRC10:14
openstackgerritSpyros Trigazis proposed openstack/magnum master: k8s_fedora: Deploy tiller  https://review.openstack.org/61233610:52
openstackgerritLingxian Kong proposed openstack/magnum stable/rocky: Support octavia-ingress-controller  https://review.openstack.org/63491110:59
*** ttsiouts has quit IRC11:12
*** ttsiouts has joined #openstack-containers11:12
*** ttsiouts has quit IRC11:16
*** udesale has quit IRC11:28
*** ramishra has quit IRC11:45
openstackgerritRicardo Rocha proposed openstack/magnum master: Allow overwriting labels on swarm mode creation  https://review.openstack.org/63450111:56
openstackgerritRicardo Rocha proposed openstack/magnum master: Allow overwriting labels on swarm mode creation  https://review.openstack.org/63450111:58
*** ramishra has joined #openstack-containers12:00
*** ttsiouts has joined #openstack-containers12:31
openstackgerritSpyros Trigazis proposed openstack/magnum master: [k8s] helm install metrics service  https://review.openstack.org/63239212:43
*** janki has quit IRC12:59
*** sdake has joined #openstack-containers13:05
*** jmlowe has quit IRC13:17
*** jmlowe has joined #openstack-containers13:34
*** sdake has quit IRC13:54
*** ykarel is now known as ykarel|away13:58
*** sdake has joined #openstack-containers14:03
*** sdake has quit IRC14:17
*** ttsiouts has quit IRC14:30
*** ttsiouts has joined #openstack-containers14:31
*** udesale has joined #openstack-containers14:32
*** sdake has joined #openstack-containers14:34
*** ttsiouts has quit IRC14:36
*** PabloMartinez has quit IRC14:42
*** sdake has quit IRC14:45
*** ttsiouts has joined #openstack-containers14:47
*** sdake has joined #openstack-containers14:49
openstackgerritRicardo Rocha proposed openstack/magnum master: Add hidden flag to cluster template  https://review.openstack.org/63494814:49
*** sdake has quit IRC14:56
*** ign0tus has quit IRC14:56
*** Nel1x has joined #openstack-containers14:59
*** sdake has joined #openstack-containers15:04
*** udesale has quit IRC15:06
openstackgerritDiogo Guerra proposed openstack/magnum master: k8s_fedora: Deploy tiller  https://review.openstack.org/61233615:06
openstackgerritDiogo Guerra proposed openstack/magnum master: [k8s] helm install metrics service  https://review.openstack.org/63239215:06
*** ign0tus has joined #openstack-containers15:11
openstackgerritRicardo Rocha proposed openstack/python-magnumclient master: Add hidden cmd flag to cluster template  https://review.openstack.org/63495515:22
*** hongbin has joined #openstack-containers15:51
*** ricolin has joined #openstack-containers15:58
*** ign0tus has quit IRC16:11
*** belmoreira has quit IRC16:19
*** ttsiouts has quit IRC16:33
*** ttsiouts has joined #openstack-containers16:33
openstackgerritRicardo Rocha proposed openstack/magnum master: Add hidden flag to cluster template  https://review.openstack.org/63494816:35
*** openstackgerrit has quit IRC16:35
brtknroh no... openstackgerrit has left?16:37
*** ttsiouts has quit IRC16:37
*** jmlowe has quit IRC16:44
*** ttsiouts has joined #openstack-containers16:44
*** ttsiouts has quit IRC16:48
*** ramishra has quit IRC17:15
*** flwang1 has joined #openstack-containers17:20
flwang1strigazi: today is NZ public holiday, i probably can't join the meeting17:20
*** jmlowe has joined #openstack-containers17:27
*** ricolin has quit IRC17:32
*** jmlowe has quit IRC17:36
*** ykarel|away has quit IRC17:38
*** ykarel|away has joined #openstack-containers17:42
colin-enjoy :)17:55
*** jmlowe has joined #openstack-containers17:57
*** ykarel|away has quit IRC18:03
*** dims has quit IRC18:06
*** sdake has quit IRC18:11
*** dims has joined #openstack-containers18:26
*** Nel1x has quit IRC18:29
*** itlinux has joined #openstack-containers18:49
*** sdake has joined #openstack-containers18:52
*** dims has quit IRC19:16
*** dims has joined #openstack-containers19:25
*** pcaruana has quit IRC19:27
*** openstackgerrit has joined #openstack-containers19:54
openstackgerritRicardo Rocha proposed openstack/magnum master: Add hidden flag to cluster template  https://review.openstack.org/63494819:54
*** jmlowe has quit IRC20:29
openstackgerritRicardo Rocha proposed openstack/magnum master: Add hidden flag to cluster template  https://review.openstack.org/63494820:48
strigaziflwang1: ack, thanks for the heads up, cheers20:48
openstackgerritSpyros Trigazis proposed openstack/magnum master: IGNORE: test gate master  https://review.openstack.org/38220220:51
openstackgerritFeilong Wang proposed openstack/magnum stable/rocky: Allow cluster template being renamed  https://review.openstack.org/63503520:55
strigazi#startmeeting containers21:01
openstackMeeting started Tue Feb  5 21:01:09 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.21:01
*** openstack changes topic to " (Meeting topic: containers)"21:01
openstackThe meeting name has been set to 'containers'21:01
strigazi#topic Roll Call21:01
eanderssono/21:01
*** openstack changes topic to "Roll Call (Meeting topic: containers)"21:01
strigazio/21:01
jakeyipo/21:01
strigaziHey eandersson and jakeyip, let's get into it. I needs some reviewers :)21:03
strigazi#topic Stories/Tasks21:03
*** openstack changes topic to "Stories/Tasks (Meeting topic: containers)"21:03
eanderssonThat's what we are here for! :D21:03
strigazi5 items, here it goes:21:04
strigazik8s_fedora: Deploy tiller https://review.openstack.org/#/c/612336/21:04
strigazi[k8s_fedora] Add heat-agent to worker nodes https://review.openstack.org/#/c/561858/21:04
strigazi[k8s] helm install metrics service https://review.openstack.org/#/c/632392/21:04
strigaziAllow cluster template being renamed https://review.openstack.org/#/c/609174/21:04
strigaziAdd hidden flag to cluster template https://review.openstack.org/#/c/634948/21:04
strigaziFirst two are mine, ready for review. I have mentioned them before.21:04
strigaziTiller in master nodes, and heat-agent in all nodes21:05
strigaziNext, colleague Diogo works on deploying the metrics server with helm in 63239221:05
strigaziAnd last two are useful for operators to advertise public CTs21:06
strigaziFeilong proposed to allow CTs being renamed, like docker.io/ubuntu:latest21:06
strigaziAnd ricardo has a complementary patch to add a hidden field in CTs.21:07
strigaziYou advertise a CT to users and when you want to put it away you hide like images.21:07
jakeyipwould love to help, actually looking at last two right now because that's something I was interested in :)21:07
strigaziclusters can use a hidden public CT, but users shopping for a CT can not list it.21:08
strigaziIf the know the uuid they can use it. (bad for them, kind of). We can add a check to not allow creation with hidden CTs. Not sure we want this or if it is that important.21:09
strigazinice jakeyip21:09
strigazijakeyip: eandersson any questions?21:09
jakeyipthere's actually a similar thing in glance but it's called community (?)21:10
openstackgerritSpyros Trigazis proposed openstack/magnum master: k8s_fedora: Deploy tiller  https://review.openstack.org/61233621:10
openstackgerritSpyros Trigazis proposed openstack/magnum master: [k8s] helm install metrics service  https://review.openstack.org/63239221:10
strigazijakeyip yes, we thought about it21:10
eanderssonsounds straightforward enough21:10
eanderssonanything in particular that is of higher priority?21:10
strigazibut we would have to deprecate public and make a new enum field. microversions etc21:10
jakeyiphonestly, I think this is easier to understand and less awkward, but just would like to know if making things similar across services was considered21:11
jakeyipall good then21:11
strigazi1st 634948 and 609174 the other three are important too but can be cherry-picked :)21:12
strigaziI mean downstream21:12
strigaziAdding helm, will simplify things a lot. We want all the CERN specific things to become a single chart that we just enable21:13
strigaziThe heat agent can wait a bit but it is the simplest :)21:13
jakeyipre: tiller, I wonder if anybody has done tiller in gitlab before? does this implementation break that?21:14
strigazijakeyip I had tried it. You can have many tillers21:14
strigazithat is why I add tiller in magnum-tiller21:14
strigazithe releases of helm live there21:15
strigaziThe only thing that would break if gitlab's tiller wants to deploy the same thing.21:15
strigaziHowever I added the option to select the namespace21:15
jakeyipI see. nice!21:16
strigaziSo you can install tiller in kube-system21:16
strigazionly things is you would have to distribute tiller certs21:16
strigazijakeyip: how you deploy with gitlab?21:17
jakeyipso you can install in kube-system and let gitlab detect it?21:17
strigazijakeyip: yes, it is the default but gitlab will need the certs to talk to tiller over tls. I haven't tried that21:18
strigazijakeyip: I'll give it a go.21:18
jakeyipstrigazi: a gitlab user can spin up their own cluster and attach the cluster to it their gitlab account21:18
jakeyipstrigazi: then they'll have the option to install tiller and it'll be able to pick up the jobs in CI/CD pipelines on a commit.21:19
strigazijakeyip: I'll give a go again, it is been some time. The option to insall tiller in magnum-tiller ns is totally compatible21:20
jakeyipI think it's fine if it's different NS21:20
*** schaney has joined #openstack-containers21:20
jakeyipthe annoying thing is that gitlab doesn't let you choose NS :P21:20
jakeyipyour implementation is superior21:21
strigazijakeyip: I implemtented all the best practices mentioned in the docs21:21
strigazijakeyip: only thing I couldn't avoid is make tiller sa very powerfull21:22
strigazijakeyip: the metrics server chart needs to create cluster level roles and bindings21:22
strigazijakeyip: it was unavoidable21:23
strigazijakeyip: eandersson have a look into this: https://github.com/stefanprodan/gitops-helm21:23
strigazibetter than pushing to the cluster. The cluster pulls instead when you push code.21:23
eanderssoncbrumm_, ^21:24
cbrumm_thanks @eandersson: I'll review later21:25
strigaziRicardo had implemented a similar method but this one is more mature21:25
strigazieandersson: this one https://gitlab.cern.ch/helm/plugins/crd-release/tree/crd21:26
eanderssoninteresting21:26
strigaziwe didn't know about the gitops thing before21:27
strigazieandersson: the only depressing thing is that it is the same pattern as our puppet deployment, soooo it might bite later? who knows :)21:28
*** itlinux has quit IRC21:29
jakeyipstrigazi: how's your puppet deployment like?21:30
strigazijakeyip: in short: you push to repo, puppet-master picks up the changes, puppet agent runs every 60mins (unless you tell it to run) and applies the config21:31
strigazijakeyip: so similarly, with gitops, you push the chart to repo, cluster listens to webhook (or cronjob) and it applies the chart21:32
strigazijakeyip: makes sense?21:33
jakeyipare you using puppet to update helm charts?21:33
strigazijakeyip: no21:34
jakeyipsorry, I mean to run the updated charts; update containers in your cluster21:34
strigazijakeyip: at the moment each team has each own method. Some do it with helm by hand21:36
strigaziothers with gitlab, but without the integration, just in gitlab ci the ci knows how to talk to the cluster21:37
*** sdake has quit IRC21:39
jakeyipstrigazi: which integration is this referring to?21:39
jakeyip(I think we can take this topic offline if necessary, I don't want to hold up meeting time)21:40
strigazieandersson: do you have anything to add? Do you need any further clarifications for the patches I mentioned before?21:41
strigazijakeyip: same ^^21:41
eanderssonI'll take some time towards the end of the week to go through them21:42
eanderssonand I'll let you know if I need anything else21:42
strigaziThanks21:43
jakeyipsame. I'm new so not that familiar with magnum code base, but I'll try to help.21:44
strigazithanks jakeyip21:45
openstackgerritMerged openstack/magnum master: Allow cluster template being renamed  https://review.openstack.org/60917421:46
strigazieandersson: jakeyip so Feilong was did it :)21:47
strigaziflwang1: you can also review Ricardo's patch, Allow cluster template being renamed https://review.openstack.org/#/c/609174/21:47
strigaziI think you will need that to not have a huge list of CTs21:48
flwang1strigazi: which one?21:48
strigazitwo line above21:48
strigaziwrong link21:49
strigazicorrect:21:49
strigaziAdd hidden flag to cluster template https://review.openstack.org/#/c/634948/21:49
flwang1strigazi: oh, yep, we had similar idea for glance public images21:50
flwang1it's quite annoying when the number is big21:50
strigaziflwang1: the idea comes from there. We use community images in our cloud to not have a massive list of images21:50
flwang1i will review that21:50
strigaziI explained above that we didn't use the same thing to not have to deprecate the public field.21:51
strigaziAlso IMO hidden makes more sense than community21:51
flwang1strigazi: yep, generally, user/admin just don't want to see them in the list21:52
strigaziWhen the cloud admin marks an image community, it is strange21:52
flwang1i agree21:54
flwang1personally, i like the 'hidden' idea21:54
strigazisweet \o/21:54
strigazilet's do this :)21:55
strigaziI need to review it actually first xD21:55
strigaziLet's end the meeting? Anything else to discuss?21:55
strigazisaid once21:57
strigazisaid twice21:57
strigaziSee you around!21:57
strigazi#endmeeting21:57
*** openstack changes topic to "OpenStack Containers Team"21:57
openstackMeeting ended Tue Feb  5 21:57:38 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)21:57
openstackMinutes:        http://eavesdrop.openstack.org/meetings/containers/2019/containers.2019-02-05-21.01.html21:57
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/containers/2019/containers.2019-02-05-21.01.txt21:57
openstackLog:            http://eavesdrop.openstack.org/meetings/containers/2019/containers.2019-02-05-21.01.log.html21:57
*** _fragatina has joined #openstack-containers22:06
*** _fragatina_ has joined #openstack-containers22:07
flwang1thank you, strigazi22:08
*** _fragatina has quit IRC22:10
*** sdake has joined #openstack-containers22:27
*** sdake has quit IRC22:34
*** sdake has joined #openstack-containers22:36
*** _fragatina_ has quit IRC22:43
openstackgerritRicardo Rocha proposed openstack/magnum master: Add hidden flag to cluster template  https://review.openstack.org/63494823:05
*** hongbin has quit IRC23:18

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