Wednesday, 2020-05-20

« Tuesday, 2020-05-19 Index Thursday, 2020-05-21 »
*** k_mouza has joined #openstack-containers00:15
*** threestrands has joined #openstack-containers00:19
*** k_mouza has quit IRC00:20
openstackgerritgugug proposed openstack/magnum master: Fix pygments style  https://review.opendev.org/72945500:48
openstackgerritgugug proposed openstack/magnum master: Fix pygments style  https://review.opendev.org/72945500:53
*** k_mouza has joined #openstack-containers00:57
*** k_mouza has quit IRC01:13
*** sapd1_x has joined #openstack-containers03:08
*** k_mouza has joined #openstack-containers03:13
*** k_mouza has quit IRC03:17
*** ykarel|away is now known as ykarel04:24
*** udesale has joined #openstack-containers05:04
openstackgerritFeilong Wang proposed openstack/magnum-ui master: Fix cluster resize  https://review.opendev.org/72951005:24
openstackgerritFeilong Wang proposed openstack/magnum-ui master: Fix cluster resize  https://review.opendev.org/72951005:57
*** yolanda has joined #openstack-containers06:33
*** belmoreira has joined #openstack-containers06:50
*** born2bake has joined #openstack-containers07:13
*** ttsiouts has joined #openstack-containers07:15
*** vishalmanchanda has joined #openstack-containers07:25
openstackgerritOpenStack Proposal Bot proposed openstack/magnum-ui stable/ussuri: Imported Translations from Zanata  https://review.opendev.org/72952507:31
*** ttsiouts has quit IRC07:37
*** ttsiouts has joined #openstack-containers07:54
*** ramishra has joined #openstack-containers08:11
*** ramishra has quit IRC08:17
*** ramishra has joined #openstack-containers08:46
openstackgerritSpyros Trigazis proposed openstack/magnum-specs master: Add helm-config  https://review.opendev.org/72775608:47
*** flwang1 has joined #openstack-containers08:49
openstackgerritBharat Kunwar proposed openstack/magnum master: [k8s] Use Helm v3 by default  https://review.opendev.org/72023408:55
strigaziflwang1: brtknr: meeting?08:55
flwang1strigazi: i'm around08:56
brtknrstrigazi: flwang1: yes i'd like one08:56
flwang1pls put your topics on https://etherpad.opendev.org/p/magnum-weekly-meeting08:56
*** ykarel is now known as ykarel|lunch08:57
*** dioguerra has joined #openstack-containers08:58
flwang1#startmeeting magnum09:00
openstackMeeting started Wed May 20 09:00:08 2020 UTC and is due to finish in 60 minutes.  The chair is flwang1. Information about MeetBot at http://wiki.debian.org/MeetBot.09:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.09:00
*** openstack changes topic to " (Meeting topic: magnum)"09:00
openstackThe meeting name has been set to 'magnum'09:00
flwang1#topic roll call09:00
*** openstack changes topic to "roll call (Meeting topic: magnum)"09:00
flwang1o/09:00
strigazio/09:00
flwang1brtknr: ?09:01
brtknro/09:01
flwang1#topic PTG09:01
*** openstack changes topic to "PTG (Meeting topic: magnum)"09:01
jakeyip_hi :)09:01
flwang1jakeyip_: hi Jake, how are you09:02
flwang1did you guys saw our PTG time?09:02
jakeyip_no actually. any links?09:02
flwang13rd June   21UTC-23UTC     http://ptg.openstack.org/ptg.html09:02
jakeyip_thanks09:03
flwang1pls put the topics you'd like to discuss on PTG to this etherpad https://etherpad.opendev.org/p/magnum-victoria-ptg09:03
brtknro/ jakeyip_ long time no see09:04
jakeyip_o/ brtknr good to see all of you again :)09:04
flwang1jakeyip_: keen to upgrade to ussuri? :D09:04
strigazijakeyip_: o/09:04
jakeyip_sorry just been lurking, was busy with other things09:04
jakeyip_we are going to train only actually :P09:05
flwang1btw, guys, catalyst cloud just upgraded to stable/train this week09:05
flwang1and so far so good09:05
flwang1surely, we cherrypicked some features from master :)09:05
brtknrflwang1: i tried to sign up to CC again, its really hard to get an account...09:05
strigazijakeyip_: you want this https://github.com/openstack/magnum/blob/master/magnum/api/controllers/v1/cluster.py#L18509:05
jakeyip_nice. we were trying to move to Octavia/OVN, was a nightmare09:05
flwang1brtknr: it shouldn't be, can you send me an email and I can follow up?09:06
brtknrflwang1: sure will do09:06
flwang1brtknr: strigazi: any question for the ptg?09:07
brtknrflwang1: nope all good09:07
strigazijakeyip_: this is the design https://specs.openstack.org/openstack/magnum-specs/specs/ussuri/labels-override.html  :)09:07
strigaziflwang1: no Qs09:07
flwang1thanks09:07
strigaziflwang1: put the date and or link in etherpad?09:08
flwang1#topic helm config09:08
*** openstack changes topic to "helm config (Meeting topic: magnum)"09:08
flwang1strigazi: sure, will do09:08
strigaziflwang1: shall I start on helm-config?09:08
jakeyip_strigazi: oo nice the labels will be super useful for us09:08
flwang1strigazi: go ahead09:08
jakeyip_strigazi: sorry please go ahead09:08
strigaziI put the SPEC up after discussion within our team, dtomasgu will drive the implementation.09:09
strigaziI added example user workflow too09:09
*** ramishra has quit IRC09:09
strigaziThe "burning" issue is what will the datatype be.09:10
flwang1you mean in db?09:10
flwang1i saw you suggested Text09:11
strigaziAt cern we agreed on json/yaml but I'm not sure if it is possible what I added here:09:11
strigazihttps://review.opendev.org/#/c/727756/3/specs/victoria/helm-config.rst@10609:11
strigaziI added an alternative that we rejected downstream but I added there for completeness:09:12
strigazihttps://review.opendev.org/#/c/727756/3/specs/victoria/helm-config.rst@9509:12
openstackgerritSpyros Trigazis proposed openstack/magnum-specs master: Add helm-config  https://review.opendev.org/72775609:13
strigazithanks for the typo ^^09:13
flwang1ok, i will review it again09:13
strigaziAlso, could we merge these two: https://storyboard.openstack.org/#!/story/2003902 https://storyboard.openstack.org/#!/story/2004215 ?09:14
strigazibrtknr: ^^09:14
strigazidtomasgu: wants to them together (I though he would be here)09:14
strigazidtomasgu: wants to do them together (I though he would be here)09:14
brtknrsure09:14
brtknrthey're similar but slightly different things09:15
jakeyip_is it possible to copy a cluster template? or get a config from an existing one?09:15
brtknrbut if we can have one config file to rule them all, that sounds good tom e09:15
strigazi(I think there are not that similar, but I don't want to be the weird guy)09:15
dioguerraIm here09:15
brtknrone config file that has labels, helm_charts, helm_requirements, everything09:15
brtknram i correct in my understanding?09:15
strigaziIt will be a new config option that contains everything I think09:16
dioguerraThat was my original idea yes09:16
strigazimutually exclusive with commandline args09:16
brtknrthat sounds good to me :)09:16
brtknrI will stop using terraform after that09:16
dioguerraalthough splitting is not a bad option either because of the size. One file for the cluster other files for the rest09:16
brtknrsplitting?09:17
dioguerrahelm and cluster-configuration?09:17
brtknri dont think size should be an issue09:18
brtknrif we are parsing everything on client side?09:18
dioguerra:+1:09:18
brtknranyway this discussion requires another spec tbh09:19
*** xinliang has joined #openstack-containers09:19
* strigazi thinks it is, but he stopped caring about it09:19
brtknrthinks size is an issue or thinks this is discussion for another spec?09:20
strigaziIt is different work, one is to restructure/enhance the client, one is add new fields in the API and client09:21
strigazibut, let's merge to speed thnigs up09:21
flwang1not sure if it's in the scope of this spec, so please correct me if this is a silly question. with this helm-config, is it possible that we disable an addon for an existing cluster?09:22
flwang1or enable an addon for an existing cluster09:22
strigaziflwang1 it makes more doable09:22
strigaziflwang1 it makes it more doable09:22
strigaziwe can add one action and easily trigger a "helm upgrade"09:22
flwang1yep, that's my understanding09:23
flwang1i'd like to have it in the future09:23
strigazistill out of scope of this (if we want reasoanbly narrow scope)09:23
flwang1fair enough09:23
strigaziflwang1: brtknr, Note:09:23
strigaziat CERN we are using fluxCD which upgrades automatically helm deployments09:24
dioguerrabrtknr: i think enabling disabling with the values.yaml will be possible (bypassing labels)09:24
strigazithis is a answer to which question ^^09:25
jakeyip_sorry silly question, is this to provide a list of charts or 1 chart to a COE ?09:26
dioguerra11:22 flwang1, sorry wrong person09:26
strigazia meta-chart with subcharts #mindblown09:26
jakeyip_ok09:27
strigazijakeyip_: example https://gitlab.com/gitlab-org/charts/gitlab/-/blob/master/requirements.yaml#L1509:27
brtknrit will be interesting to see how labels and helm_values override each other :)09:27
strigazibrtknr: not really, pretty straigh foward. mutually exclusive09:27
strigazianything else on helm-config?09:28
brtknreg. nginx_tag passed via label vs tag passed via values.yaml09:28
strigaziagree to merge the work and the spec?09:28
brtknryes09:28
strigazibrtknr if you do this: 40109:29
brtknroh so make labels mutually exclusive with helm_values option?09:29
strigaziyes, to silly-proof09:29
dioguerrabrtknr: both default to the same and you do a XOR09:30
dioguerrawill see...09:31
*** ramishra has joined #openstack-containers09:31
brtknrok but some labels are not relevant to helm charts09:32
strigaziI'm not writing XOR anywhere in the SPEC. I will add a list the SPEC that if a meta-chart is used, it throughs an error.09:32
brtknrwhat about those?09:32
brtknror are we going to identify the labels which are relevant to charts and only exclude those?09:32
strigaziI will add the silly-proof sections in the spec.09:32
strigaziThat's what I said.09:32
brtknrok09:32
brtknrmy bad09:33
strigaziI will add new section that will refactor the client09:33
strigaziI will add a new section in the SPEC that will refactor the client09:33
strigaziok? flwang1 dtomasgu brtknr09:33
brtknrok09:33
flwang1strigazi: good for me09:33
flwang1#topic helm 309:34
*** openstack changes topic to "helm 3 (Meeting topic: magnum)"09:34
flwang1strigazi: sorry, anything else for helm-config?09:35
strigaziI'm good09:36
flwang1strigazi: thanks09:37
dioguerraok09:37
flwang1brtknr: anything else we need to discuss for helm 3?09:37
brtknrwell its ready to merge based on your feedback09:38
brtknrthats all09:38
flwang1brtknr: cool09:38
flwang1move on?09:39
flwang1#topic filter support for cluster template listing09:39
*** openstack changes topic to "filter support for cluster template listing (Meeting topic: magnum)"09:39
brtknrI have tested locally but feel free to test yourselves for your peace of mind09:39
flwang1brtknr: i have done that before, will do it again09:39
brtknrflwang1: thanks09:39
flwang1i'd like to introduce a basic filter funciton for template listing09:39
flwang1because i found it's really annoying to find public templates09:40
flwang1brtknr: strigazi: don't you guys have the same pain?09:40
dioguerrai do09:41
strigaziflwang1: in my projects I usually add my username in the name09:41
*** xinliang has quit IRC09:41
strigaziso convention by name09:42
strigazibut it's good to have09:42
dioguerrado you have something? im interested how this works because we wanted to add a "recommended" template functionality09:42
brtknri usually work with fewer than 10 templates in any given project so havent hit problems09:42
brtknrit would be good to deprecate old templates09:42
flwang1dioguerra: it's a common filter you an find it anywhere in other openstack services09:43
brtknrwouldnt needing to delete them09:43
jakeyip_sort of, how do you want it to look?09:43
flwang1openstack coe cluster template list --filter public=True,Hidden=False09:43
jakeyip_this problem is similar to glance public images, I guess?09:43
flwang1something like above09:43
flwang1jakeyip_: not really, i just want to do filter here09:44
brtknropenstack coe cluster template list --hidden --public would be nicer09:46
jakeyip_glance is similar? `glance image-list --visibility private --hidden False`09:46
flwang1brtknr: jakeyip_: i'm trying to propose a more general filter09:47
dioguerraactually, i like the 1option if it can be expanded to other elements09:47
flwang1i'm just taking public, hidden as a smaple09:47
brtknrwhat other things would need filtering?09:47
flwang1brtknr: e.g. network_driver=calico09:47
flwang1i haven't got a solid design, i just want to know you guys ideas if you are ok with the direction09:48
jakeyip_public and hidden alone will be great for us. when we upgrade CT we hid the old one and put a new one with same name. eventually I need to clean them up, so I need to get the hidden and public ones. right now I'm just grepping on the two fields, which I show using https://review.opendev.org/#/c/702322/09:49
jakeyip_flwang1:  have you tried displaying network_driver field, and grepping on that as a workaround?09:50
flwang1jakeyip_: does the fields works?09:51
brtknrwe should implement filtering schema for which there is precedence for in other projects, not try and invent our own09:51
flwang1brtknr: yep, that's my rough idea09:52
brtknrwith nova for example, you can do openstack server list --status ACTIVE09:52
brtknrwith glance, you can do openstack image list --public --shared --private09:52
flwang1anyway, i will think about this again later09:52
strigaziflwang1: SPEC ^^09:52
strigazi:)09:52
flwang1strigazi: sure09:52
flwang1will do09:53
flwang1anything else?09:53
strigazione thing09:53
jakeyip_flwang1: yeah think so. another thing is that openstack cli output can be json so you can do fancy things with jq too09:53
strigaziI will add again build of hyperkube in our CI.09:53
strigaziI will add again builds of hyperkube in our CI.09:54
flwang1strigazi: why?09:54
strigaziUpstream kubernetes thought it was very easy for us to use it and the dropped build starting from 1.1909:54
strigaziUpstream kubernetes thought it was very easy for us to use it and they dropped the build starting from 1.1909:54
* strigazi finds the link09:55
strigazithere you go https://github.com/kubernetes/kubernetes/pull/8867609:55
strigaziit lasted a few releases for us, it was good09:56
flwang1that's not a good idea :(09:57
flwang1strigazi: ok, pls add it09:57
brtknrargh what a pain09:57
flwang1and seems we have to09:57
*** jakeyip_ is now known as jakeyi[p09:57
*** jakeyi[p is now known as jakeyip09:57
brtknrcan we build a single hyperkube image or one per k8s service?09:57
strigaziposeidon typhoon does the same, they maintain a build09:58
strigazibrtknr: hyperkube was always one09:58
flwang1that's all for today?09:59
strigazii'm good09:59
flwang1strigazi: i got several patches +2 by brtknr, could you please take a look them?10:00
flwang1#endmeeting10:00
*** openstack changes topic to "OpenStack Containers Team | Meeting: every Wednesday @ 9AM UTC | Agenda: https://etherpad.openstack.org/p/magnum-weekly-meeting"10:00
openstackMeeting ended Wed May 20 10:00:29 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)10:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/magnum/2020/magnum.2020-05-20-09.00.html10:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/magnum/2020/magnum.2020-05-20-09.00.txt10:00
openstackLog:            http://eavesdrop.openstack.org/meetings/magnum/2020/magnum.2020-05-20-09.00.log.html10:00
flwang1thank you, team10:00
brtknrflwang1: can you please look at the clusterautoscaler patch10:00
flwang1brtknr: will do10:00
brtknri want to backport to train before cutting 9.4.0 release10:01
dioguerrasee ya10:02
*** dioguerra has quit IRC10:02
jakeyipflwang1: yeah fields work see https://paste.centos.org/view/a4d8b28b10:02
flwang1jakeyip: nice, thanks10:03
flwang1openstack coe cluster template list --fields public,hidden,network_driver -f json | jq -c '.[] | select(.public==true)'10:08
jakeyip:)10:09
*** flwang1 has quit IRC10:19
*** ykarel|lunch is now known as ykarel10:46
*** k_mouza has joined #openstack-containers10:48
openstackgerritMerged openstack/magnum master: [k8s] Deprecate in-tree Cinder  https://review.opendev.org/72813110:56
openstackgerritMerged openstack/magnum master: [k8s] Update Cluster Autoscaler ClusterRole  https://review.opendev.org/72553411:36
openstackgerritBharat Kunwar proposed openstack/magnum stable/ussuri: [k8s] Update Cluster Autoscaler ClusterRole  https://review.opendev.org/72956911:38
openstackgerritBharat Kunwar proposed openstack/magnum stable/train: [k8s] Update Cluster Autoscaler ClusterRole  https://review.opendev.org/72957011:38
*** sapd1_x has quit IRC11:41
*** ramishra has quit IRC11:58
*** k_mouza has quit IRC12:06
*** vishalmanchanda has quit IRC12:17
*** k_mouza has joined #openstack-containers12:20
*** udesale_ has joined #openstack-containers12:23
*** udesale has quit IRC12:25
*** ttsiouts has quit IRC12:27
*** ramishra has joined #openstack-containers13:03
*** andrein_ has joined #openstack-containers13:08
*** andrein has quit IRC13:10
*** andrein_ is now known as andrein13:10
*** ttsiouts has joined #openstack-containers13:19
*** ttsiouts has quit IRC13:24
*** ttsiouts has joined #openstack-containers13:27
*** sapd1_x has joined #openstack-containers13:34
*** vishalmanchanda has joined #openstack-containers13:41
*** ykarel is now known as ykarel|afk13:46
*** colin- has quit IRC13:59
*** ramishra has quit IRC14:03
*** colin- has joined #openstack-containers14:03
*** ykarel|afk is now known as ykarel14:03
*** ttsiouts has quit IRC14:15
*** KeithMnemonic has joined #openstack-containers14:19
*** threestrands has quit IRC14:34
*** belmoreira has quit IRC14:54
*** munimeha1 has joined #openstack-containers14:58
*** ttsiouts has joined #openstack-containers15:18
*** ttsiouts has quit IRC15:40
*** ttsiouts has joined #openstack-containers15:40
*** ttsiouts has quit IRC15:40
*** rcernin has quit IRC16:04
*** ykarel is now known as ykarel|away16:23
openstackgerritDiogo Guerra proposed openstack/magnum master: Scrape internal kubernetes components  https://review.opendev.org/72128416:24
openstackgerritDiogo Guerra proposed openstack/magnum master: Configurable prometheus monitoring persistent storage  https://review.opendev.org/72157216:24
openstackgerritDiogo Guerra proposed openstack/magnum master: Add persistency for grafana dashboards  https://review.opendev.org/72239616:24
openstackgerritDiogo Guerra proposed openstack/magnum master: Configure monitoring apps path based endpoints  https://review.opendev.org/72159216:24
*** sapd1_x has quit IRC16:52
*** sapd1_x has joined #openstack-containers17:04
*** udesale_ has quit IRC17:07
*** jmlowe has quit IRC17:23
*** jmlowe has joined #openstack-containers17:26
*** k_mouza has quit IRC17:56
*** munimeha1 has quit IRC18:18
*** sapd1_x has quit IRC18:26
*** k_mouza has joined #openstack-containers19:56
*** mgoddard has quit IRC19:59
*** k_mouza has quit IRC20:02
*** k_mouza has joined #openstack-containers20:22
*** k_mouza has quit IRC20:27
*** vishalmanchanda has quit IRC20:37
*** born2bake has quit IRC20:50
*** k_mouza has joined #openstack-containers22:23
*** k_mouza has quit IRC22:27
*** rcernin has joined #openstack-containers23:08
*** threestrands has joined #openstack-containers23:49
« Tuesday, 2020-05-19 Index Thursday, 2020-05-21 »

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!