Tuesday, 2020-05-12

*** k_mouza has joined #openstack-containers01:02
*** k_mouza has quit IRC01:10
*** hongbin has joined #openstack-containers03:00
*** k_mouza has joined #openstack-containers03:11
*** k_mouza has quit IRC03:16
*** ykarel|away is now known as ykarel04:41
*** hongbin has quit IRC05:04
*** udesale has joined #openstack-containers05:35
*** k_mouza has joined #openstack-containers06:11
*** k_mouza has quit IRC06:16
*** ttsiouts has joined #openstack-containers06:39
*** livelace has joined #openstack-containers06:55
*** belmoreira has joined #openstack-containers06:57
*** sapd1_x has joined #openstack-containers07:45
*** ykarel is now known as ykarel|lunch08:22
*** ttsiouts has quit IRC08:32
*** ttsiouts_ has joined #openstack-containers08:32
*** ttsiouts has joined #openstack-containers08:36
*** ttsiouts_ has quit IRC08:38
openstackgerritSpyros Trigazis proposed openstack/magnum master: Fix pep8 for ambiguous variable name  https://review.opendev.org/72711008:39
*** ttsiouts has quit IRC08:39
*** ttsiouts has joined #openstack-containers08:39
brtknrLarsErikP: sounds good :)08:44
*** livelace has quit IRC08:49
*** ykarel|lunch is now known as ykarel08:51
*** ttsiouts_ has joined #openstack-containers09:07
LarsErikPbrtknr: still some issues though.. I'll come back. Issues with flannel, and heat-container-agent throws errors on me09:09
*** ttsiouts has quit IRC09:09
LarsErikPrunning Atomic 29 btw; as CoreOS apparently is not supported in stein?09:10
*** mgariepy has quit IRC09:10
brtknrLarsErikP: no CoreOS support was added in Train, and required Train Heat09:10
brtknrI recommend upgrading as Fedora Atomic is no longer supported09:10
brtknri.e. no more security patches09:11
LarsErikPwell, we can't uprade our cloud to Train yet09:11
brtknrCan you cherry-pick?09:11
*** livelace has joined #openstack-containers09:11
LarsErikPunless magnum and heat can run train, with everything else is running stein?09:12
LarsErikPanyhow; this _is_ supposed to work with AtmoicHost in stein; right?09:12
brtknrYes its supposed to09:12
LarsErikPeven though atomichost is eol..09:12
brtknrWhat version of k8s are you trying to run09:12
LarsErikPthe default for stein09:12
brtknrhave you tried calico network plugin?09:13
brtknrwhat you share what labels you are using?09:14
brtknrIts been a while since I used magnum stein :)09:14
LarsErikPbrtknr: yep. Just have to take my lunch break here ;)09:14
LarsErikP(living in UTC+2)09:14
brtknrok no worries09:14
LarsErikPthanks for helping!09:14
brtknrnp09:14
brtknrhttps://wiki.openstack.org/wiki/Magnum#Compatibility_Matrix09:15
brtknrthis might be useful^09:15
*** mgariepy has joined #openstack-containers09:15
brtknrLarsErikP:09:15
*** ttsiouts_ has quit IRC09:18
*** ttsiouts has joined #openstack-containers09:19
*** k_mouza has joined #openstack-containers09:21
ttsioutsstrigazi, brtknr: are you guys around?09:25
brtknrmorning ttsiouts09:25
ttsioutsbrtknr: good morning!09:25
ttsioutsbrtknr: thanks for the review09:25
brtknrttsiouts: you're welcome :)09:26
brtknrthanks for the tests09:26
ttsioutsbrtknr: I am still thinking about the labels_added09:27
ttsioutsbrtknr: I'm not sure it should be a dict09:27
ttsioutsthe reason is that their values will be present in the labels field09:27
ttsioutsbrtknr: what do you think?09:28
brtknrI understand its duplication, however it requires the user to do a second lookup using the key from labels_added09:30
brtknralso slightly confusing to see a list of labels09:30
brtknri think that despite the duplication, its better to have all labels* fields presented as dicts09:31
*** jhesketh has joined #openstack-containers09:31
brtknralso looks better when you do openstack coe cluster show <cluster> -f yaml09:31
ttsioutsbrtknr: hmmm09:32
strigazittsiouts: how it was described in the SPEC?09:33
strigaziI forgot tbh09:33
ttsioutslabels_added: labels that do not exist in the parent labels and were added in the object09:34
brtknrthe spec is the explicit09:34
brtknr    * labels_overridden: labels that exist in both parent and object labels but09:35
brtknr                         have different value09:35
brtknr    * labels_added: labels that do not exist in the parent labels and were09:35
brtknr                    added in the object09:35
brtknr    * labels_skipped: labels that exist in the parent dict but do not exist in09:35
brtknr                      the object's labels. Specifically, this field will be09:35
brtknr                      used when the user did not provide the --merge-labels09:35
brtknr                      (used the current functionality) and did not provide some09:35
brtknr                      of the labels that exist in the parent.09:35
brtknrthe spec is not explicit :)09:35
ttsioutsbrtknr: indeed09:39
ttsioutsI could update the patch with it..09:41
*** k_mouza has quit IRC09:42
brtknrttsiouts: everything dict?09:42
*** k_mouza has joined #openstack-containers09:47
openstackgerritLingxian Kong proposed openstack/magnum master: [K8S] Delete all related load balancers before deleting cluster  https://review.opendev.org/71693009:47
openstackgerritLingxian Kong proposed openstack/magnum master: [K8S] Delete all related load balancers before deleting cluster  https://review.opendev.org/71693009:48
openstackgerritBharat Kunwar proposed openstack/magnum master: [k8s] Support Helm v3  https://review.opendev.org/72023409:51
ttsioutsbrtknr: yes everything dict..10:03
brtknr+1 great thanks10:04
strigazibrtknr: is there anything else though? Maybe we wait a bit more to see if more things come up? Feilong may want more changes.10:06
strigazittsiouts: ^^ what do you think?10:06
brtknrjust the things I mentioned on revie10:06
openstackgerritBharat Kunwar proposed openstack/magnum master: [k8s] Support Helm v3  https://review.opendev.org/72023410:07
brtknrstrigazi: btw helm3 is ready for review finally^10:07
strigaziyou mentioned you will do the metachart10:07
strigazibrtknr: has this changed?10:08
brtknryes10:08
strigaziok, someone from cern will do it10:08
brtknrthe review?10:09
*** livelace has quit IRC10:09
ttsioutsstrigazi, brtknr: I have no strong feelings for this. my main issue is that the values will be duplicated. meaning that they will be shown in the labels field..10:09
ttsioutsstrigazi, brtknr: if you guys think that it's ok I can update the patch with all the fields as dict.10:10
brtknri think its okay, so feilong can review the most up to date agreed patch10:10
*** udesale_ has joined #openstack-containers10:18
*** ttsiouts_ has joined #openstack-containers10:19
*** ttsiouts has quit IRC10:19
*** udesale has quit IRC10:21
*** pcaruana has quit IRC10:28
*** ttsiouts_ has quit IRC10:34
*** ttsiouts has joined #openstack-containers10:34
LarsErikPbrtknr: http://paste.ubuntu.com/p/mXQsJPv997/10:37
LarsErikPthe only labels I've changed is heat_container_agent_tag from stein-dev to stein-stable; and set keystone_auth_enabled to false10:38
LarsErikPmagnum services are running 8.0.0 from the Ubuntu cloud archives10:39
*** ttsiouts has quit IRC10:51
*** pcaruana has joined #openstack-containers10:53
*** ttsiouts has joined #openstack-containers11:00
*** ttsiouts_ has joined #openstack-containers11:03
*** ttsiouts has quit IRC11:04
LarsErikPbrtknr: now i'm getting this: Failed to pull image "docker.io/googlecontainer/cluster-proportional-autoscaler-amd64:1.1.2": rpc error: code = Unknown desc = repository docker.io/googlecontainer/cluster-proportional-autoscaler-amd64 not found: does not exist or no pull access11:12
brtknryes thats a known issue11:12
LarsErikPcool..? so then? /ignore ?11:12
brtknrLarsErikP:11:13
brtknrc0e51198a3116d2ddc81edba220b56ebea528fd811:13
brtknryou need this commit11:13
brtknror update the image manually11:13
brtknrI think the new stein release fixed this11:13
brtknrbut you are running 8.0.011:13
brtknryes the new stein release has the fix for this11:14
LarsErikPyeah; 8.0.0 is the only version in UCA..11:14
brtknri recommend running 8.2.011:14
LarsErikPthen we should recommend for the UCA guys to include it :P11:15
brtknryes go ahead :)11:15
LarsErikPhehe11:15
LarsErikPI know who to poke..11:15
*** livelace has joined #openstack-containers11:17
brtknrUCA doesnt seem to have any bugfix releases11:17
brtknrthats a big disadvantage11:17
LarsErikPso; 8.0.0 and kubernetes don't work..? easy as that :P11:17
openstackgerritTheodoros Tsioutsias proposed openstack/magnum master: Labels override  https://review.opendev.org/72022111:18
openstackgerritSpyros Trigazis proposed openstack/magnum master: Update nginx-ingress to v1.36.3 and 0.32.0 tag  https://review.opendev.org/72716211:39
*** obre has joined #openstack-containers11:58
openstackgerritTheodoros Tsioutsias proposed openstack/python-magnumclient master: Labels override  https://review.opendev.org/72022212:02
*** vishalmanchanda has quit IRC12:07
LarsErikPbrtknr: is that commit related to a bug in storyboard?12:08
*** sapd1_x has quit IRC12:08
LarsErikPalways helps when talking to #ubuntu-server guys :)12:08
LarsErikPnevermind; it's of course mentioned in the commit message..12:09
*** ttsiouts_ has quit IRC12:20
*** ttsiouts has joined #openstack-containers12:21
LarsErikPbrtknr: heh, tried to be clever and set container_infra_prefix to that gcr.io url. that failed badly :P12:21
*** ttsiouts_ has joined #openstack-containers12:31
*** ttsiouts has quit IRC12:31
LarsErikPbrtknr: hmm talked to coreycb over at #ubuntu-server now, but he asks questions I'm not able to answer..12:37
*** ttsiouts_ has quit IRC12:38
LarsErikPbrtknr: do you mind talking to him? *angel-emoji*12:38
*** ttsiouts has joined #openstack-containers12:38
LarsErikPbrtknr: https://paste.ubuntu.com/p/bT7WHX9KjT/12:38
*** belmoreira has quit IRC12:41
*** vishalmanchanda has joined #openstack-containers12:41
*** ttsiouts has quit IRC12:42
*** belmoreira has joined #openstack-containers12:44
*** pcaruana has quit IRC12:59
*** pcaruana has joined #openstack-containers13:02
*** livelace has quit IRC13:05
*** ttsiouts has joined #openstack-containers13:06
*** livelace has joined #openstack-containers13:08
*** ttsiouts has quit IRC13:21
*** ttsiouts has joined #openstack-containers13:21
*** ttsiouts has quit IRC13:25
*** ttsiouts has joined #openstack-containers13:26
*** spotz has joined #openstack-containers13:27
LarsErikPbrtknr: i "ninja-patched" the autoscaler patched here. Worked a charm. Still receving an error from heat-container-agent though..13:34
LarsErikPit runs; but throws [ERROR] Error running heat-config-notify. [1]13:34
LarsErikPseems to be a 404 from keystone.. huhm..13:35
brtknrLarsErikP: is it repeatable?13:35
LarsErikPhappens every time13:35
brtknrnot sure...13:36
brtknrit could be anything...13:36
LarsErikPhehe13:36
LarsErikPthink I have to dig through keystone logs to see what it tried to do13:37
*** openstackstatus has quit IRC13:53
*** openstackstatus has joined #openstack-containers13:53
*** ChanServ sets mode: +v openstackstatus13:54
LarsErikPbrtknr: hmm, I see in the json-file that heat-config-notify used, the "deploy_auth_url" is "https://<our-endpoint>:5000/"14:00
LarsErikPnotice, without /v314:00
LarsErikPwhere does that come from?14:00
*** sapd1_x has joined #openstack-containers14:12
*** ttsiouts has quit IRC14:16
brtknrumm14:20
brtknrhave you checked [keystone_authtoken]14:22
brtknrauth_version = v314:22
brtknrwww_authenticate_uri = http://192.168.7.1:5000/v314:22
brtknrin magnum.conf?14:22
*** ttsiouts has joined #openstack-containers14:22
brtknralso auth_url under [keystone_auth]14:22
*** mgariepy has quit IRC14:31
*** ttsiouts_ has joined #openstack-containers14:31
*** ttsiouts has quit IRC14:35
*** ttsiouts_ has quit IRC14:41
*** ttsiouts has joined #openstack-containers14:41
*** mgariepy has joined #openstack-containers14:44
*** ttsiouts has quit IRC14:46
*** livelace has quit IRC14:46
openstackgerritBharat Kunwar proposed openstack/magnum master: [k8s] Support Helm v3  https://review.opendev.org/72023414:48
*** rcernin has quit IRC14:48
openstackgerritBharat Kunwar proposed openstack/magnum master: [k8s] Support Helm v3  https://review.opendev.org/72023414:50
*** dasp_ has quit IRC15:14
*** dtomasgu_ has joined #openstack-containers15:16
dtomasgu_nick dioguerra15:16
*** dtomasgu_ is now known as dioguerra15:16
*** dasp has joined #openstack-containers15:16
*** belmoreira has quit IRC15:22
brtknrdioguerra: hi15:27
dioguerrahi !15:40
*** dasp has quit IRC15:41
dioguerrai just log on to get the etherpad url :D15:42
*** dasp has joined #openstack-containers15:42
*** livelace has joined #openstack-containers16:02
*** ttsiouts has joined #openstack-containers16:35
LarsErikPbrtknr: I'll check tomorrow. thanks16:36
*** udesale_ has quit IRC16:41
*** ttsiouts has quit IRC16:50
*** ttsiouts has joined #openstack-containers16:51
*** ykarel is now known as ykarel|away16:55
*** ttsiouts has quit IRC16:55
*** vishalmanchanda has quit IRC16:57
*** yolanda has quit IRC17:35
*** sapd1_x has quit IRC18:03
*** vishalmanchanda has joined #openstack-containers18:04
*** dioguerra has quit IRC18:33
*** livelace has quit IRC18:35
*** livelace has joined #openstack-containers18:50
*** k_mouza has quit IRC19:44
*** k_mouza has joined #openstack-containers19:51
*** k_mouza has quit IRC20:20
*** k_mouza has joined #openstack-containers20:35
*** vishalmanchanda has quit IRC20:37
*** k_mouza has quit IRC20:59
*** k_mouza has joined #openstack-containers21:00
*** k_mouza has quit IRC21:25
*** k_mouza has joined #openstack-containers21:28
*** k_mouza has quit IRC21:50
*** k_mouza has joined #openstack-containers21:57
openstackgerritMerged openstack/magnum-specs master: Change flag name  https://review.opendev.org/72694322:08
*** k_mouza has quit IRC22:21
*** k_mouza has joined #openstack-containers22:22
*** k_mouza has quit IRC22:35
*** k_mouza has joined #openstack-containers22:35
*** k_mouza has quit IRC22:45
openstackgerritMerged openstack/magnum master: Fix pep8 for ambiguous variable name  https://review.opendev.org/72711022:48
*** k_mouza has joined #openstack-containers22:50
openstackgerritFeilong Wang proposed openstack/magnum master: Update nginx-ingress to v1.36.3 and 0.32.0 tag  https://review.opendev.org/72716222:56
openstackgerritMerged openstack/magnum master: [K8S] Delete all related load balancers before deleting cluster  https://review.opendev.org/71693022:57
openstackgerritMerged openstack/magnum master: Update prometheus monitoring chart and images  https://review.opendev.org/72125923:01
*** rcernin has joined #openstack-containers23:19
*** k_mouza has quit IRC23:47
*** k_mouza has joined #openstack-containers23:50

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