| *** rcernin has joined #openstack-containers | 00:11 | |
| *** rcernin_ has quit IRC | 00:12 | |
| *** rcernin has quit IRC | 00:13 | |
| *** rcernin has joined #openstack-containers | 00:14 | |
| *** chhagarw has joined #openstack-containers | 00:47 | |
| *** chhagarw has quit IRC | 00:51 | |
| *** hongbin has joined #openstack-containers | 01:27 | |
| *** hongbin has quit IRC | 01:36 | |
| *** hongbin has joined #openstack-containers | 01:37 | |
| *** hongbin_ has joined #openstack-containers | 01:41 | |
| *** hongbin has quit IRC | 01:43 | |
| *** canori01 has quit IRC | 01:54 | |
| *** ricolin has joined #openstack-containers | 02:04 | |
| *** hongbin has joined #openstack-containers | 02:05 | |
| *** hongbin_ has quit IRC | 02:07 | |
| *** dave-mccowan has quit IRC | 02:57 | |
| *** munimeha1 has quit IRC | 03:01 | |
| *** ramishra has joined #openstack-containers | 03:12 | |
| *** lxkong has quit IRC | 03:15 | |
| *** lxkong has joined #openstack-containers | 03:15 | |
| *** hongbin has quit IRC | 03:52 | |
| *** chhagarw has joined #openstack-containers | 03:58 | |
| *** udesale has joined #openstack-containers | 03:58 | |
| *** janki has joined #openstack-containers | 04:59 | |
| *** ttsiouts has quit IRC | 05:42 | |
| *** ttsiouts has joined #openstack-containers | 05:43 | |
| *** ttsiouts has quit IRC | 05:47 | |
| *** spsurya has joined #openstack-containers | 05:48 | |
| *** ramishra_ has joined #openstack-containers | 06:40 | |
| *** ramishra has quit IRC | 06:43 | |
| *** pcaruana has joined #openstack-containers | 06:56 | |
| *** rcernin has quit IRC | 07:06 | |
| *** ramishra_ is now known as ramishra | 07:33 | |
| *** mattgo has joined #openstack-containers | 07:39 | |
| *** pvradu has joined #openstack-containers | 08:14 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum master: k8s_fedora: Deploy tiller https://review.openstack.org/612336 | 08:15 |
|---|---|---|
| *** ttsiouts has joined #openstack-containers | 08:17 | |
| *** salmankhan has joined #openstack-containers | 09:38 | |
| *** udesale has quit IRC | 09:51 | |
| *** udesale has joined #openstack-containers | 09:52 | |
| *** pvradu has quit IRC | 10:00 | |
| *** pvradu has joined #openstack-containers | 10:00 | |
| *** pvradu_ has joined #openstack-containers | 10:05 | |
| *** serlex has joined #openstack-containers | 10:05 | |
| *** pvradu has quit IRC | 10:09 | |
| *** ianychoi has quit IRC | 10:22 | |
| *** ianychoi has joined #openstack-containers | 10:25 | |
| *** flwang1 has joined #openstack-containers | 11:00 | |
| flwang1 | strigazi: any chance you're around? | 11:00 |
| strigazi | flwang1: yes but I was leaving. Is it quick? | 11:01 |
| flwang1 | strigazi: yep | 11:01 |
| strigazi | flwang1: tell me | 11:01 |
| flwang1 | should the workder node name be resolved in pod? | 11:01 |
| flwang1 | for example, in a pod based on alpine, i run exec -it pod-name sh | 11:02 |
| flwang1 | then run 'nslookup <worker-node-name> <dns_nameserver>' | 11:02 |
| strigazi | flwang1: if the node names are resolvable the cloud, yes | 11:04 |
| strigazi | coredns uses the vm dns | 11:04 |
| strigazi | the vm dns should be able to resolve the node names | 11:04 |
| strigazi | makes sense? | 11:04 |
| *** ttsiouts has quit IRC | 11:04 | |
| flwang1 | that said, if it's using 8.8.8.8, then the worker node can't be resolved? | 11:05 |
| strigazi | I guess not | 11:06 |
| flwang1 | ok, we can discuss when you back | 11:07 |
| strigazi | if login in the master can you lookup the worker? | 11:07 |
| strigazi | if you login in the master can you lookup the worker? | 11:07 |
| strigazi | by name I mean | 11:07 |
| * strigazi will be back | 11:08 | |
| *** ricolin has quit IRC | 11:13 | |
| *** udesale has quit IRC | 11:14 | |
| *** mattgo has quit IRC | 11:31 | |
| *** ricolin has joined #openstack-containers | 11:37 | |
| *** janki has quit IRC | 11:41 | |
| *** ttsiouts has joined #openstack-containers | 11:43 | |
| strigazi | I'm back | 11:45 |
| strigazi | flwang1: ping | 11:46 |
| flwang1 | hi | 11:46 |
| *** ttsiouts has quit IRC | 11:48 | |
| *** mattgo has joined #openstack-containers | 11:49 | |
| *** ttsiouts has joined #openstack-containers | 12:02 | |
| *** dave-mccowan has joined #openstack-containers | 12:07 | |
| *** janki has joined #openstack-containers | 12:14 | |
| *** mattgo has quit IRC | 12:16 | |
| *** janki has quit IRC | 12:25 | |
| *** janki has joined #openstack-containers | 12:25 | |
| *** janki has quit IRC | 12:27 | |
| *** tobberydberg has quit IRC | 12:30 | |
| *** udesale has joined #openstack-containers | 12:31 | |
| *** mattgo has joined #openstack-containers | 12:32 | |
| *** janki has joined #openstack-containers | 12:34 | |
| *** jchhatbar has joined #openstack-containers | 12:38 | |
| *** janki has quit IRC | 12:40 | |
| *** jchhatbar is now known as janki | 12:50 | |
| *** livelace has joined #openstack-containers | 13:22 | |
| *** serlex has quit IRC | 13:37 | |
| *** livelace has quit IRC | 13:48 | |
| *** hongbin has joined #openstack-containers | 14:00 | |
| *** mattgo has quit IRC | 14:01 | |
| *** markguz_ has joined #openstack-containers | 14:05 | |
| *** ramishra has quit IRC | 14:06 | |
| *** markguz_ has quit IRC | 14:10 | |
| *** janki has quit IRC | 14:21 | |
| *** mattgo has joined #openstack-containers | 14:34 | |
| *** canori01 has joined #openstack-containers | 14:35 | |
| tobias-urdin | strigazi: backports for the recent fix https://review.openstack.org/#/q/status:open+project:openstack/magnum+topic:swarm-mode-f27-fix | 15:07 |
| tobias-urdin | let me know if we dont want to backport all of them, i need rocky though, thx! | 15:07 |
| strigazi | tobias-urdin: ack | 15:15 |
| strigazi | tobias-urdin: isn't ocata eol? | 15:15 |
| tobias-urdin | newton is eol, ocata is in extended maintenance | 15:17 |
| strigazi | ok | 15:18 |
| tobias-urdin | ocata "estimated 2018-08-27" according to releases.o.org | 15:18 |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum master: Add heat_container_agent_tag https://review.openstack.org/612727 | 15:20 |
| strigazi | eandersson: ^^ | 15:20 |
| *** ianychoi_ has joined #openstack-containers | 15:36 | |
| *** ianychoi has quit IRC | 15:40 | |
| *** ivve has joined #openstack-containers | 15:42 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum master: Add heat_container_agent_tag https://review.openstack.org/612727 | 15:43 |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum master: Add heat_container_agent_tag label https://review.openstack.org/612727 | 15:44 |
| *** mattgo has quit IRC | 15:47 | |
| *** ricolin has quit IRC | 15:51 | |
| *** udesale has quit IRC | 16:06 | |
| eandersson | Thanks looks a lot better strigazi | 16:12 |
| *** ttsiouts has quit IRC | 16:16 | |
| *** ttsiouts has joined #openstack-containers | 16:17 | |
| *** ttsiouts has quit IRC | 16:21 | |
| openstackgerrit | Julia Kreger proposed openstack/magnum master: Minor fixes to re-align with Ironic https://review.openstack.org/612748 | 16:35 |
| *** pvradu_ has quit IRC | 16:37 | |
| *** dtruong has quit IRC | 16:56 | |
| *** dtruong has joined #openstack-containers | 16:56 | |
| *** dtruong has quit IRC | 17:02 | |
| openstackgerrit | Merged openstack/magnum master: Make master node schedulable with taints https://review.openstack.org/608627 | 17:02 |
| *** dtruong has joined #openstack-containers | 17:03 | |
| *** pvradu has joined #openstack-containers | 17:04 | |
| *** pvradu has quit IRC | 17:09 | |
| *** salmankhan has quit IRC | 17:20 | |
| *** pvradu has joined #openstack-containers | 17:42 | |
| *** pvradu has quit IRC | 17:53 | |
| *** irclogbot_1 has joined #openstack-containers | 18:35 | |
| *** flwang1 has quit IRC | 18:50 | |
| *** chhagarw has quit IRC | 18:57 | |
| *** salmankhan has joined #openstack-containers | 19:09 | |
| *** salmankhan has quit IRC | 19:13 | |
| *** pcaruana has quit IRC | 19:23 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum stable/rocky: Make master node schedulable with taints https://review.openstack.org/612790 | 19:27 |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum stable/rocky: Add prometheus-monitoring namespace https://review.openstack.org/612791 | 19:34 |
| brtknr | strigazi: is there a meeting later? | 20:03 |
| strigazi | yes https://wiki.openstack.org/wiki/Meetings/Containers#Agenda_for_2018-10-23_2100_UTC | 20:04 |
| strigazi | I got you patch in the agenda | 20:05 |
| strigazi | lgtm | 20:05 |
| *** openstackgerrit has quit IRC | 20:06 | |
| *** openstackgerrit has joined #openstack-containers | 20:37 | |
| openstackgerrit | Merged openstack/magnum master: Add prometheus-monitoring namespace https://review.openstack.org/600905 | 20:37 |
| *** ivve has quit IRC | 20:42 | |
| *** schaney has joined #openstack-containers | 20:46 | |
| *** flwang has joined #openstack-containers | 20:47 | |
| flwang | strigazi: meeting in 13 mins? | 20:47 |
| openstackgerrit | Feilong Wang proposed openstack/magnum stable/rocky: Add prometheus-monitoring namespace https://review.openstack.org/612791 | 20:54 |
| *** ttsiouts has joined #openstack-containers | 20:56 | |
| strigazi | meeting in 2' | 20:57 |
| strigazi | #startmeeting containers | 21:00 |
| openstack | Meeting started Tue Oct 23 21:00:11 2018 UTC and is due to finish in 60 minutes. The chair is strigazi. Information about MeetBot at http://wiki.debian.org/MeetBot. | 21:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 21:00 |
| *** openstack changes topic to " (Meeting topic: containers)" | 21:00 | |
| openstack | The meeting name has been set to 'containers' | 21:00 |
| strigazi | #topic Roll Call | 21:00 |
| *** openstack changes topic to "Roll Call (Meeting topic: containers)" | 21:00 | |
| strigazi | o/ | 21:00 |
| ttsiouts | o/ | 21:00 |
| flwang | o/ | 21:01 |
| imdigitaljim | o/ | 21:02 |
| strigazi | brtknr: you here? | 21:03 |
| eandersson | o/ | 21:03 |
| *** salmankhan has joined #openstack-containers | 21:03 | |
| strigazi | Thanks for joining the meeting ttsioutsflwang imdigitaljim eandersson | 21:03 |
| strigazi | Agenda: | 21:03 |
| strigazi | #link https://wiki.openstack.org/wiki/Meetings/Containers#Agenda_for_2018-10-23_2100_UTC | 21:04 |
| strigazi | it has some items | 21:04 |
| strigazi | #topic Stories/Tasks | 21:04 |
| *** openstack changes topic to "Stories/Tasks (Meeting topic: containers)" | 21:04 | |
| strigazi | 1. node groups https://review.openstack.org/#/c/607363/ | 21:04 |
| strigazi | I think we are pretty close to the final state of the spec | 21:05 |
| strigazi | please take a look | 21:05 |
| ttsiouts | strigazi: tomorrow I will push again | 21:05 |
| ttsiouts | adapting ricardo's comments | 21:06 |
| strigazi | oh I thought you pushed today, ok so take a looks guys take a look tmr as well :) | 21:06 |
| ttsiouts | :) tmr better | 21:07 |
| strigazi | @all do you want to discuss anything about nodegroups today? | 21:07 |
| strigazi | questions about nodegroups? | 21:07 |
| strigazi | ok, next then | 21:08 |
| schaney | o/ sorry for lateness, but yes | 21:08 |
| strigazi | schaney: hello. you have smth about nodegroups? | 21:09 |
| schaney | any mechanism to interact with NGs individually? | 21:09 |
| schaney | opposed to the top level cluster or stack | 21:09 |
| strigazi | the api will be like: | 21:09 |
| ttsiouts | schaney: do you mean updating a specific nodegroup? | 21:10 |
| schaney | yes for scaling or the like | 21:10 |
| strigazi | cluster/<cluster-identity>/nodegroup/<nodegroup-identity> | 21:10 |
| strigazi | so PATCH cluster/<cluster-identity>/nodegroup/<nodegroup-identity> | 21:10 |
| ttsiouts | https://review.openstack.org/#/c/607363/2/specs/stein/magnum-nodegroups.rst@117 | 21:10 |
| colin- | sorry i'm late! | 21:11 |
| strigazi | colin-: welcome | 21:11 |
| schaney | oh gotcha, ill have to dig into the work a bit, under the hood magnum just targets the name of the node group represented by the heat parameter though? | 21:12 |
| flwang | ttsiouts: the node groups is basically the same thing like node pool in GKE, right? | 21:13 |
| ttsiouts | schaney: that's the plan | 21:13 |
| ttsiouts | flwang: exactly | 21:13 |
| flwang | ttsiouts: cool | 21:14 |
| schaney | i see, did there happen to be any work with the "random node scale down" when magnum shrinks the cluster? | 21:14 |
| flwang | ttsiouts: i will review the spec first | 21:14 |
| schaney | from the API route it would seem so? | 21:14 |
| flwang | i think we probably better read the spec first and put comments in the code | 21:14 |
| flwang | instead of discussing design details here | 21:15 |
| schaney | good call | 21:15 |
| ttsiouts | schaney: we want to add a CLI for removing specific nodes from the cluster | 21:16 |
| ttsiouts | but this will come further down the way | 21:16 |
| strigazi | schaney: this won't be covered by this spec, but we should track it somewhere else | 21:16 |
| schaney | gotcha, thanks for the info | 21:16 |
| ttsiouts | flwang: thanks! | 21:16 |
| ttsiouts | flwang: tmr it will be more complete | 21:17 |
| imdigitaljim | were looking at modifying our driver to perhaps consume senlin clusters for each group | 21:17 |
| imdigitaljim | masters, minions-group-1, ... minions-group-n | 21:17 |
| flwang | imdigitaljim: is it a hard depedency? | 21:17 |
| imdigitaljim | would all have a senlin profile/cluster in heat | 21:17 |
| flwang | i mean for senlin | 21:17 |
| strigazi | imdigitaljim: that could be done, that is why we have drivers | 21:18 |
| strigazi | flwang: should be optional | 21:18 |
| strigazi | like alternative | 21:18 |
| imdigitaljim | it would probably be the driver would take on a senlin dependency (not magnum as a whole) | 21:18 |
| imdigitaljim | just like octavia or not | 21:18 |
| strigazi | when the cluster drivers where proposed senlin and ansble were the reasoning behind it | 21:19 |
| imdigitaljim | we're more focused on autoscaling/autohealing rather than cli manual scaling | 21:19 |
| imdigitaljim | the senlin PTL is here and is actively talking to the heat PTL on managing the senlin resources in heat so we might be able to inhouse create a better opportunity for senlin + heat + magnum | 21:20 |
| strigazi | imdigitaljim: here, like in this meeting? | 21:20 |
| imdigitaljim | no | 21:20 |
| imdigitaljim | sorry i just mean he works at blizzard | 21:21 |
| strigazi | This plan is compatible with nodegroups and nodegroups make it actually easier | 21:22 |
| imdigitaljim | we think so | 21:22 |
| strigazi | I'm not aware of it in detail, but it sounds doable | 21:23 |
| schaney | Senlin would work well with the NG layout, one thing to note is Senlin's dedicated API | 21:23 |
| imdigitaljim | yeah we arent either but we'll be working out over the next couple weeks | 21:23 |
| imdigitaljim | and see if its within reason on feasibility | 21:23 |
| eandersson | The Senlin PTL will be in Berlin btw | 21:23 |
| imdigitaljim | ^ | 21:23 |
| strigazi | cool | 21:24 |
| cbrumm__ | Its honestly too early to be talking about, there's a lot of heat/senlin ground work to do first | 21:24 |
| cbrumm__ | But hey, it's a thing we're thinking about. | 21:24 |
| strigazi | fair enough | 21:24 |
| strigazi | shall we move on? | 21:25 |
| cbrumm__ | yes | 21:26 |
| strigazi | I pushed two patches we were talking about, one is: | 21:26 |
| strigazi | Add heat_container_agent_tag label https://review.openstack.org/#/c/612727 | 21:26 |
| strigazi | we discussed with flwang and eandersson already, others have a look too | 21:27 |
| strigazi | the tag of the heat-agent was hardcoded this makes it a label. | 21:27 |
| strigazi | the 2nd one needs some discussion, it is | 21:28 |
| strigazi | deploy tiller by default https://review.openstack.org/#/c/612336/ | 21:28 |
| strigazi | Shall we have it in by default or optional? | 21:29 |
| flwang | strigazi: any potential issue if we enable it by default? | 21:30 |
| strigazi | and then next steps are, with tls or without? a tiller per ns or with the cluster-role? | 21:30 |
| strigazi | flwang: the user might want a diffrent tiller config | 21:31 |
| flwang | strigazi: that's the problem i think | 21:31 |
| strigazi | flwang: other than that, tiller will be there silent | 21:31 |
| flwang | we have seen similar 'issue' with other enabling, like the keystone auth integration | 21:32 |
| strigazi | flwang: so you are in-favor of optional | 21:32 |
| flwang | a new enabling feature may introduce a bunch of config | 21:32 |
| flwang | but now in mangum, we don't have a good way to maintain those config | 21:32 |
| flwang | labels are too flexible and lose | 21:33 |
| flwang | i prefer optional | 21:33 |
| flwang | based on the feedback we got so far, most of customer just want a vanilla k8s cluster | 21:33 |
| flwang | if they want something, they can DIY | 21:34 |
| strigazi | what vanilla means? api, sch, cm, kubelet, proxy, dns, cni | 21:34 |
| flwang | and i agree it's because we(catalyst cloud) are public cloud and our customers' requirements are vary | 21:34 |
| flwang | vanilla means a pure cluster, not too much plugins/addons | 21:34 |
| cbrumm__ | flwang: We've been getting some similar feedback from power users too | 21:35 |
| flwang | for private cloud, the thing maybe different | 21:35 |
| cbrumm__ | but I think that's expected from power users that are used to DYI | 21:35 |
| flwang | most of the customers of k8s know how to play with it | 21:35 |
| flwang | what they want is just a stable k8s cluster with good HA and integration with the underhood cloud provider | 21:36 |
| flwang | cbrumm__: what do you mean 'power users'? | 21:36 |
| strigazi | so optional | 21:37 |
| cbrumm__ | people who've used k8s before that was not a managed service | 21:37 |
| flwang | cbrumm__: ok, i see, thx | 21:37 |
| strigazi | flwang: any argument against having this optional? | 21:38 |
| *** spsurya has quit IRC | 21:38 | |
| flwang | strigazi: TBH, I would suggest we start to define a better addon architecture | 21:39 |
| imdigitaljim | ^ | 21:39 |
| flwang | like refactor the labels | 21:39 |
| imdigitaljim | thats one of the goals our driver plans to be solving | 21:39 |
| flwang | imdigitaljim: show me the code ;) | 21:39 |
| *** rtjure has quit IRC | 21:39 | |
| strigazi | so we don't add anything until we refactor? | 21:40 |
| flwang | i have heard about the v2 driver million times, i want to see the code :D | 21:40 |
| flwang | strigazi: i'm not saying that | 21:40 |
| imdigitaljim | :D when i get some free cycles and feel like its in a great uploading spot | 21:40 |
| flwang | i just say we should be more careful | 21:40 |
| strigazi | the current model is not unreasonable. we need to define careful and not make the service a framework | 21:42 |
| strigazi | I think for v1 which can be refactored only replaced/deprecated the model of addons is there | 21:43 |
| strigazi | s/can/can't/ | 21:44 |
| strigazi | labels for on/off and tags | 21:44 |
| imdigitaljim | imo we should look into a config file | 21:44 |
| imdigitaljim | kubernetes followed the same pattern when they realized there were too many flags | 21:45 |
| strigazi | config file to create the labels/fields or a config file to pass to the cluster | 21:45 |
| strigazi | ? | 21:45 |
| openstackgerrit | Merged openstack/magnum stable/queens: Provide a region to the K8S Fedora Atomic config https://review.openstack.org/612199 | 21:46 |
| imdigitaljim | openstack coe cluster template create --config myvalues.yaml | 21:46 |
| strigazi | these values are like labels? | 21:46 |
| imdigitaljim | could be everything | 21:46 |
| imdigitaljim | could be just labels | 21:46 |
| strigazi | code too? | 21:47 |
| strigazi | like to code? | 21:47 |
| imdigitaljim | ? | 21:47 |
| strigazi | or link to code? | 21:47 |
| imdigitaljim | instead of like --labels octavia_enabled=true, etc etc | 21:47 |
| imdigitaljim | it could be | 21:47 |
| imdigitaljim | [Loadbalancer} | 21:48 |
| imdigitaljim | [LoadBalancer] | 21:48 |
| strigazi | got it | 21:48 |
| imdigitaljim | octavia_enabled=true | 21:48 |
| imdigitaljim | but you could also do | 21:48 |
| imdigitaljim | [Network] | 21:48 |
| imdigitaljim | floating_ ... fixed_network= .. fixed_subnet= | 21:48 |
| flwang | yaml or the ini format | 21:48 |
| imdigitaljim | either or | 21:49 |
| imdigitaljim | any | 21:49 |
| imdigitaljim | json | 21:49 |
| flwang | yep | 21:49 |
| imdigitaljim | doesnt matter however we want to do it | 21:49 |
| flwang | agree | 21:49 |
| imdigitaljim | imo im a fan of that model | 21:49 |
| flwang | and with that case, we can publish sample config files | 21:49 |
| strigazi | flwang: what would cover your concern about the loose label design? | 21:49 |
| flwang | and user can decide how to combine the config | 21:49 |
| flwang | strigazi: yep, that's basically the arch in my mind | 21:50 |
| strigazi | #action strigazi to draft a spec and story for creating cluster with a config file. | 21:51 |
| flwang | strigazi: can we discuss this one https://github.com/kubernetes/cloud-provider-openstack/issues/280 next? | 21:51 |
| strigazi | I'll try to bring smth in the next meeting for this | 21:51 |
| flwang | strigazi: thanks | 21:51 |
| strigazi | before going into the bug of CPO | 21:52 |
| strigazi | @all take a look to the rest of the list of review in the agenda. they are ready to go in | 21:53 |
| strigazi | https://wiki.openstack.org/wiki/Meetings/Containers#Agenda_for_2018-10-23_2100_UTC | 21:53 |
| colin- | will do | 21:53 |
| strigazi | flwang: what might help a little is using the config drive before the metadata service | 21:54 |
| strigazi | colin-: thx | 21:54 |
| flwang | strigazi: does that need change in magnum? | 21:55 |
| flwang | imdigitaljim: did you ever see this issue https://github.com/kubernetes/cloud-provider-openstack/issues/280 ? | 21:55 |
| strigazi | imdigitaljim: eandersson colin- what is your experience with CPO | 21:55 |
| flwang | worker nodes are missing ips | 21:55 |
| imdigitaljim | i have not experienced this issue | 21:55 |
| strigazi | flwang: in the config of the CPO | 21:55 |
| flwang | imdigitaljim: probably because you're using v1.12? | 21:55 |
| imdigitaljim | we have an internal downstream with a few patches on CPO | 21:55 |
| imdigitaljim | were waiting on upstream commit permission for kubernetes org | 21:56 |
| imdigitaljim | (blizzard admin stuff) | 21:56 |
| imdigitaljim | we're on 1.12.1 correct | 21:56 |
| strigazi | patches regarding this bug? | 21:56 |
| imdigitaljim | no | 21:56 |
| imdigitaljim | UDP support, LBaaS naming, and a LBaaS edge case | 21:56 |
| flwang | https://github.com/kubernetes/kubernetes/pull/65226#issuecomment-431933545 | 21:56 |
| colin- | to jim, outloud just now i said "it's been better than starting from scratch"? :) | 21:57 |
| colin- | found it useful and it has definitely saved us some time, but as he said we've also found some gaps we want to address | 21:57 |
| flwang | it seems like a very common, high-chance problem | 21:57 |
| strigazi | colin-: you talk about k/cpo? | 21:58 |
| colin- | yes | 21:58 |
| flwang | strigazi: when you say 'config of CPO', does that mean we at least have to use cm+CPO mode? | 21:58 |
| strigazi | flwang: https://github.com/openstack/magnum/blob/master/magnum/drivers/common/templates/kubernetes/fragments/write-kube-os-config.sh#L12 | 21:59 |
| strigazi | it is adding [Metadata]search-order=configDriver,metadataService | 22:00 |
| flwang | strigazi: got it. but based on https://github.com/kubernetes/cloud-provider-openstack/issues/280#issuecomment-427416908 | 22:01 |
| flwang | does that mean we only need add this one [Metadata]search-order=configDriver,metadataService ? | 22:01 |
| strigazi | flwang: the code is... even if you disable set [Metadata]search-order=configDrive it still calls the metadataservice | 22:02 |
| strigazi | I tried with configdrive only and it was still doing calls to the APIs. | 22:03 |
| flwang | i'm confused | 22:03 |
| flwang | does that need any change in nova? | 22:03 |
| flwang | i mean nova config | 22:04 |
| strigazi | I'll end the meeting to be ~1 hour and we continue | 22:04 |
| flwang | cool, thanks | 22:04 |
| openstackgerrit | Merged openstack/magnum master: Minor fixes to re-align with Ironic https://review.openstack.org/612748 | 22:04 |
| strigazi | @all thanks for joining the meeting | 22:04 |
| strigazi | see you next week | 22:05 |
| strigazi | #endmeeting | 22:05 |
| *** openstack changes topic to "OpenStack Containers Team" | 22:05 | |
| openstack | Meeting ended Tue Oct 23 22:05:22 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 22:05 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/containers/2018/containers.2018-10-23-21.00.html | 22:05 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/containers/2018/containers.2018-10-23-21.00.txt | 22:05 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/containers/2018/containers.2018-10-23-21.00.log.html | 22:05 |
| *** ttsiouts has quit IRC | 22:06 | |
| *** ttsiouts has joined #openstack-containers | 22:06 | |
| strigazi | CPO is talking to the metadata service and uses the config drive if available too | 22:06 |
| flwang | ok | 22:07 |
| strigazi | flwang: you can set the order with that flag | 22:07 |
| flwang | but you just said it doesn't work? | 22:07 |
| flwang | ok | 22:07 |
| strigazi | yes, the docs claim that you can select only one | 22:07 |
| flwang | (11:02:25) strigazi: flwang: the code is... even if you disable set [Metadata]search-order=configDrive it still calls the metadataservice | 22:07 |
| flwang | does that mean 'CPO' will call both no matter what you set? | 22:08 |
| strigazi | I did set only the configDrive earlier today and then I was monitoing the nova api | 22:08 |
| flwang | then will the result from api call overwrite the result got from configdrive? | 22:08 |
| strigazi | I was receiving call from the CM and the kubelet | 22:08 |
| flwang | and then run into the problem again? | 22:08 |
| strigazi | I haven't looked in the code but i think it might be the same result | 22:09 |
| *** ttsiouts has quit IRC | 22:11 | |
| flwang | shit | 22:11 |
| flwang | i have asked in https://github.com/kubernetes/kubernetes/pull/65226 to backport this to v1.11 | 22:12 |
| flwang | and many people want that in v1.11 as well | 22:12 |
| strigazi | I think because of this https://github.com/kubernetes/kubernetes/blob/13705ac81e00f154434b5c66c1ad92ac84960d7f/pkg/cloudprovider/providers/openstack/openstack_volumes.go#L503 | 22:13 |
| strigazi | it always uses the metadata service | 22:13 |
| strigazi | it says in the comments: We're avoiding using cached metadata (or the configdrive), | 22:14 |
| strigazi | relying on the metadata service. | 22:14 |
| flwang | so though there is a config, the code just always skip it? | 22:15 |
| strigazi | that's my understanding | 22:16 |
| flwang | wait | 22:16 |
| flwang | why https://github.com/kubernetes/cloud-provider-openstack/issues/280#issuecomment-427416908 say it works | 22:16 |
| strigazi | I don't know I was trying to verify this comment | 22:17 |
| flwang | ok, i'm leaving comment to him | 22:18 |
| flwang | to ask more details | 22:18 |
| flwang | strigazi: thank you for all your time | 22:19 |
| flwang | always helpful | 22:19 |
| strigazi | almost helpful :) I tried :) | 22:19 |
| flwang | strigazi: have a good night, man | 22:20 |
| strigazi | he replied already | 22:21 |
| strigazi | thanks flwang , see you tmr/later today | 22:21 |
| flwang | thanks, interesting talk | 22:22 |
| strigazi | before going to sleep, quickly, about helm. what do you propose? opt-in works for me | 22:23 |
| *** rcernin has joined #openstack-containers | 22:24 | |
| flwang | i'd like to opt and then we can polish until we figure out a reasonable shape we all happy | 22:24 |
| flwang | is that OK for you? | 22:24 |
| flwang | but it's totally not a blocker | 22:25 |
| strigazi | ok, I'll add one more label to make this off by-default | 22:25 |
| flwang | i will need to talk about the flavor issue with you later | 22:25 |
| flwang | imdigitaljim: still around? | 22:26 |
| strigazi | ok then | 22:26 |
| strigazi | have a nice day | 22:26 |
| flwang | o/ | 22:26 |
| flwang | strigazi: btw | 22:27 |
| imdigitaljim | flwang: yes sorry | 22:27 |
| flwang | do you have a few moment? | 22:28 |
| imdigitaljim | yeah | 22:29 |
| imdigitaljim | flwang: whats up? | 22:31 |
| imdigitaljim | pm me as well if you want | 22:31 |
| *** threestrands has joined #openstack-containers | 23:02 | |
| *** hongbin has quit IRC | 23:10 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!