| *** dave-mccowan has joined #openstack-containers | 00:29 | |
| *** dave-mccowan has quit IRC | 00:31 | |
| *** itlinux has joined #openstack-containers | 00:59 | |
| *** sapd1_x has joined #openstack-containers | 02:16 | |
| *** irclogbot_0 has quit IRC | 02:23 | |
| *** ricolin has joined #openstack-containers | 02:35 | |
| *** ykarel has joined #openstack-containers | 02:52 | |
| *** hongbin has joined #openstack-containers | 03:12 | |
| *** hongbin has quit IRC | 03:21 | |
| *** sapd1_x has quit IRC | 03:26 | |
| brtknr | flwang: what about this one :P https://review.openstack.org/#/c/629130/11 | 03:31 |
|---|---|---|
| flwang | brtknr: it's on my list, i prefer to wait until the dependencies in | 03:32 |
| brtknr | flwang: which dependencies? | 03:32 |
| brtknr | flwang: Ah the other review... i get it! | 03:33 |
| flwang | brtknr: i'm happy to get them in, is it urgent? | 03:34 |
| *** ttsiouts has quit IRC | 03:52 | |
| *** janki has joined #openstack-containers | 04:05 | |
| *** udesale has joined #openstack-containers | 04:08 | |
| *** itlinux has quit IRC | 04:15 | |
| openstackgerrit | Merged openstack/magnum stable/queens: [k8s] Cluster creation speedup https://review.openstack.org/631236 | 04:24 |
| openstackgerrit | Merged openstack/magnum stable/queens: [k8s] Add kubelet to the master nodes https://review.openstack.org/631474 | 04:24 |
| openstackgerrit | Merged openstack/magnum stable/queens: Bump k8s version up to v1.11.5 https://review.openstack.org/631535 | 04:24 |
| *** chhagarw has joined #openstack-containers | 04:45 | |
| *** ramishra has joined #openstack-containers | 04:46 | |
| *** ivve has quit IRC | 05:22 | |
| *** absubram has joined #openstack-containers | 05:23 | |
| *** absubram_ has joined #openstack-containers | 05:27 | |
| *** absubram has quit IRC | 05:28 | |
| *** absubram_ is now known as absubram | 05:28 | |
| *** ykarel_ has joined #openstack-containers | 05:43 | |
| *** rcernin has quit IRC | 05:43 | |
| *** ykarel has quit IRC | 05:45 | |
| *** ykarel_ is now known as ykarel | 05:51 | |
| *** rcernin has joined #openstack-containers | 05:59 | |
| *** sapd1_x has joined #openstack-containers | 06:18 | |
| *** ivve has joined #openstack-containers | 06:30 | |
| *** absubram has quit IRC | 06:45 | |
| *** mkuf_ has joined #openstack-containers | 06:51 | |
| *** mkuf_ is now known as mkuf | 06:51 | |
| *** ivve has quit IRC | 07:07 | |
| *** pcaruana has joined #openstack-containers | 07:19 | |
| *** belmoreira has joined #openstack-containers | 07:24 | |
| *** ivve has joined #openstack-containers | 07:38 | |
| *** mkuf has quit IRC | 08:04 | |
| *** ttsiouts has joined #openstack-containers | 08:06 | |
| *** ykarel is now known as ykarel|lunch | 08:21 | |
| *** ramishra has quit IRC | 08:35 | |
| *** ramishra has joined #openstack-containers | 08:35 | |
| *** gsimondon has joined #openstack-containers | 08:37 | |
| *** flwang1 has joined #openstack-containers | 08:57 | |
| flwang1 | strigazi: around for a catch up? | 08:57 |
| *** sapd1_x has quit IRC | 08:57 | |
| *** mkuf has joined #openstack-containers | 09:11 | |
| *** belmoreira has quit IRC | 09:15 | |
| openstackgerrit | Diogo Guerra proposed openstack/magnum master: [k8s] Install prometheus monitoring with helm https://review.openstack.org/629577 | 09:24 |
| *** ykarel|lunch is now known as ykarel | 09:27 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum master: k8s_fedora: Add ca_key before all deployments https://review.openstack.org/643904 | 09:50 |
| strigazi | flwang1: hello, can you review ^^ | 09:50 |
| flwang1 | strigazi: sure | 09:52 |
| flwang1 | that looks like a safe change, +2 | 09:54 |
| strigazi | flwang1: for fips, I'm ok with using conditions. I just wanted a note to warn users. https://review.openstack.org/#/c/641547/ I left a comment. The current PS doesn't work properly. | 09:56 |
| flwang1 | strigazi: yep, i was saying that before. it's ok to move, but we need a release note | 09:57 |
| flwang1 | i will ask lxkong to fix it | 09:57 |
| flwang1 | strigazi: i have uploaded a new patch set for resize api https://review.openstack.org/638572 and it works for 'update_in_progress' now | 09:58 |
| flwang1 | so consumer can trigger another api call when the cluster is in updating | 09:58 |
| strigazi | I was testing it on friday, seems to work. | 09:59 |
| strigazi | flwang1: Shall we add a microversion? | 09:59 |
| flwang1 | strigazi: good question | 09:59 |
| flwang1 | i'd like to add one so that we can do better version discovery in CA | 10:00 |
| strigazi | flwang1: it would be useful for the autoscaler to know | 10:00 |
| strigazi | yeap, that's it | 10:00 |
| flwang1 | will do | 10:00 |
| strigazi | the CA is merged by the way | 10:00 |
| flwang1 | yep, i saw that | 10:00 |
| flwang1 | strigazi: did you get a chance to review https://review.openstack.org/631378 ? | 10:00 |
| strigazi | no, I didn't | 10:01 |
| flwang1 | we could add another label "auto_scaling_enabled" | 10:01 |
| strigazi | question | 10:02 |
| flwang1 | to enable CA but without Draino | 10:02 |
| strigazi | if you don't add draino | 10:02 |
| flwang1 | listening | 10:02 |
| strigazi | what would the CA do when a node is not ready? | 10:02 |
| flwang1 | IIRC, CA has a param to cover the "NotReady" case | 10:02 |
| flwang1 | but Draino can do check more conditions | 10:03 |
| flwang1 | let me find the link | 10:03 |
| strigazi | flwang1: can you also have a look to "[k8s] Install prometheus monitoring with helm https://review.openstack.org/629577" | 10:04 |
| flwang1 | strigazi: sure, i'd like to take that one to shape our addon/helm policy | 10:04 |
| strigazi | +1 to we could add another label "auto_scaling_enabled" | 10:05 |
| flwang1 | strigazi: https://github.com/kubernetes/autoscaler/blob/master/cluster-autoscaler/main.go#L93 | 10:05 |
| *** alisanhaji has joined #openstack-containers | 10:06 | |
| flwang1 | i haven't tested that param because i haven't find a good way to make a node NotReady, do you know how? | 10:06 |
| strigazi | systemctl stop kubelet ;) | 10:08 |
| strigazi | or openstack server sttop | 10:08 |
| flwang1 | i don't think that work, IIRC | 10:08 |
| flwang1 | anyway, after rebase my auto healing patch | 10:09 |
| flwang1 | i will test that again | 10:09 |
| flwang1 | i need rename the cloud provider from openstack to magnum | 10:09 |
| flwang1 | though i think openstack-magnum is much better than magnum | 10:09 |
| strigazi | agree, I didn't want to argue and hold on naming | 10:10 |
| flwang1 | cause user can't see any relationship with openstack from the name 'magnum' | 10:10 |
| flwang1 | strigazi: me too | 10:10 |
| strigazi | if the openstack foundation wants that, to me it doesn't really mater | 10:10 |
| flwang1 | we can propose the change, when we propose the PR to leverage resize api | 10:10 |
| flwang1 | TBH, foundation should also consider magnum team's option | 10:11 |
| strigazi | personally, i would do it. we don't need controversy while we don't get any features | 10:11 |
| strigazi | For me 'magnum' is good enough | 10:12 |
| flwang1 | fair enough | 10:12 |
| flwang1 | im ok with that | 10:12 |
| strigazi | we do things that work ;) | 10:12 |
| strigazi | and our autoscaler works xD | 10:13 |
| flwang1 | strigazi: what's the status of your upgrade patch? anything i can help, we really need to speed up | 10:13 |
| strigazi | yes, | 10:13 |
| strigazi | how would you upgrade calico? | 10:13 |
| strigazi | just the tag? | 10:14 |
| strigazi | flannel doesn't have major versions | 10:14 |
| flwang1 | no | 10:14 |
| flwang1 | calico is a bit complicated | 10:14 |
| flwang1 | we need to upgrade calico to 3.x first | 10:14 |
| flwang1 | the yaml will change a lot | 10:14 |
| strigazi | Do yo want to that first? | 10:14 |
| flwang1 | no | 10:14 |
| strigazi | and then just change the tag? | 10:15 |
| flwang1 | i don't want that work block the upgrade work | 10:15 |
| flwang1 | at this moment, we(CC) does really care about the base OS upgrade and k8s upgrade | 10:15 |
| strigazi | ok, so just change of tag for now | 10:15 |
| flwang1 | if we can have them first, we're OK | 10:15 |
| flwang1 | (23:15:32) strigazi: ok, so just change of (k8s?) tag for now | 10:15 |
| strigazi | ok | 10:16 |
| flwang1 | stein will be released soon, we need to make sure the patch can be merged in stein | 10:16 |
| flwang1 | are you still confident that? | 10:16 |
| strigazi | yes | 10:17 |
| flwang1 | your last patch set is 17 days ago, do you have a new patchset now so that i can help testing? | 10:17 |
| flwang1 | that's the only risky feature i'm concerned for Stein | 10:17 |
| strigazi | will have in a bit | 10:18 |
| strigazi | risky??? | 10:18 |
| flwang1 | hah, i say 'risky' cause i haven't seen the code yet :) | 10:18 |
| strigazi | too much stress | 10:18 |
| flwang1 | haha | 10:18 |
| flwang1 | i'm sorry, my friend | 10:19 |
| strigazi | I'm working on it now? | 10:19 |
| strigazi | I'm working on it now | 10:19 |
| flwang1 | i should push you 1 year ago so that you don't get too much stress now ;) | 10:19 |
| strigazi | that's ok, can you help with the other reviews? | 10:20 |
| flwang1 | which one you think i should pay attention first? the helm one? | 10:20 |
| strigazi | and nodegroups too? | 10:20 |
| flwang1 | and node group? | 10:20 |
| strigazi | the functinality for current clusters is done | 10:21 |
| flwang1 | you mean node group? | 10:21 |
| strigazi | ready for review and if it is ok, merge | 10:21 |
| strigazi | yes | 10:21 |
| flwang1 | is the api ready? i saw it's still with WIP tag | 10:21 |
| strigazi | one sec | 10:22 |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: Add nodegroup representation https://review.openstack.org/604823 | 10:23 |
| strigazi | in this branch https://review.openstack.org/#/q/topic:magnum_nodegroups+(status:open+OR+status:merged) | 10:25 |
| strigazi | all paches that do not have WIP are complete | 10:25 |
| flwang1 | cool, i will start to review it tomorrow | 10:26 |
| strigazi | this one is also done https://review.openstack.org/604824 but it is missing tests | 10:26 |
| strigazi | ignore 604824 and 629274 for now | 10:27 |
| *** udesale has quit IRC | 10:28 | |
| strigazi | Will you change anything else in Support <ClusterID>/actions/resize API ? apart from microversion? | 10:28 |
| flwang1 | strigazi: no | 10:31 |
| flwang1 | in my testing, it works perfectly | 10:31 |
| flwang1 | i can add the microversion in a following patch if you think that's better | 10:31 |
| strigazi | what do you think? I think it is need by the CA, thoughts? | 10:32 |
| flwang1 | i'm working on that now, i think we just need one line like this @base.Controller.api_version("1.7") | 10:33 |
| strigazi | I think so | 10:34 |
| flwang1 | i can add it in the api ref patch | 10:35 |
| flwang1 | so that we can do a quick merge now for the api one | 10:35 |
| flwang1 | and this CURRENT_MAX_VER = '1.6' | 10:36 |
| flwang1 | we need to bump it to 1.7 | 10:36 |
| strigazi | yes | 10:36 |
| flwang1 | and recently when reviewing code, i just realized that we don't have api ref for fedoration api and we didn't add microversion for that | 10:37 |
| flwang1 | which we should | 10:37 |
| flwang1 | i think the work was done by a cern engineer as well? | 10:37 |
| strigazi | resize is waaaaaay more important :) but yes we should | 10:37 |
| strigazi | intern | 10:38 |
| flwang1 | ok | 10:38 |
| flwang1 | still in cern? | 10:38 |
| strigazi | nope | 10:38 |
| flwang1 | ok | 10:38 |
| flwang1 | now i'm going to add the micro version support | 10:38 |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: Support <ClusterID>/actions/resize API https://review.openstack.org/638572 | 10:46 |
| strigazi | flwang1: do we need a change in the client for that? | 10:48 |
| flwang1 | strigazi: yep, i already got a patch, will submit tomorrow | 10:48 |
| flwang1 | and i'm working on the gophercloud change as well | 10:48 |
| strigazi | Shall we add a couple more UTs for this https://review.openstack.org/#/c/638572/19/magnum/tests/unit/api/controllers/v1/test_cluster_actions.py | 10:50 |
| flwang1 | strigazi: we should, definitly | 10:52 |
| flwang1 | do you want me add it in this patch? | 10:52 |
| flwang1 | i can work on that when i work on the upgrade api patch ( in parallel) | 10:52 |
| strigazi | ok, I'm testing different combinations now | 10:53 |
| flwang1 | thanks | 10:54 |
| strigazi | flwang1: this breaks swarm: https://review.openstack.org/#/c/638572/19/magnum/drivers/heat/driver.py@195 | 10:56 |
| flwang1 | oops, why? | 10:57 |
| strigazi | swarm doesn't have minions_to_remove | 10:57 |
| strigazi | heat_params["minions_to_remove"] this will break when sent | 10:58 |
| flwang1 | what's the name in swarm driver? | 10:58 |
| flwang1 | i will think if we have a good way to get the name | 10:58 |
| strigazi | this is when I have added get_scale_params | 10:58 |
| flwang1 | i see | 10:59 |
| flwang1 | so what's your advice here? | 10:59 |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: [WIP]: Out of box Keystone auth https://review.openstack.org/643225 | 10:59 |
| strigazi | I'm looking at https://github.com/openstack/magnum/commit/3f773f1fd045a507c3962ae509fcd57352cdc9ae | 10:59 |
| strigazi | pass nodes_to_remove to get_scale_params | 11:00 |
| strigazi | ? | 11:00 |
| strigazi | I think it will work | 11:00 |
| flwang1 | scale_manager.get_removal_nodes(nodes_to_remove) ? | 11:02 |
| strigazi | no | 11:03 |
| strigazi | flwang1: add one more argument here: https://github.com/openstack/magnum/commit/3f773f1fd045a507c3962ae509fcd57352cdc9ae#diff-828ec8ab190ded59e8dfa1dbf0ad3cd1R78 | 11:03 |
| strigazi | if node_to_remove: | 11:03 |
| strigazi | # do what you need to do | 11:04 |
| strigazi | https://github.com/openstack/magnum/blob/3f773f1fd045a507c3962ae509fcd57352cdc9ae/magnum/drivers/mesos_ubuntu_v1/template_def.py#L78 | 11:04 |
| strigazi | https://github.com/openstack/magnum/blob/3f773f1fd045a507c3962ae509fcd57352cdc9ae/magnum/drivers/heat/k8s_template_def.py#L142 here for example: | 11:06 |
| flwang1 | strigazi: so i have to pass in 'nodes_to_remove' to get_scale_params ? | 11:06 |
| strigazi | http://paste.openstack.org/show/747953/ | 11:07 |
| strigazi | sorry, team lunch | 11:07 |
| strigazi | see you later | 11:07 |
| flwang1 | ok, i see | 11:07 |
| strigazi | strigazi: so i have to pass in 'nodes_to_remove' to get_scale_params ? yes | 11:07 |
| *** mkuf has quit IRC | 11:08 | |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: Support <ClusterID>/actions/resize API https://review.openstack.org/638572 | 11:16 |
| *** ttsiouts has quit IRC | 11:21 | |
| *** sapd1_x has joined #openstack-containers | 11:23 | |
| openstackgerrit | Merged openstack/magnum master: k8s_fedora: Add ca_key before all deployments https://review.openstack.org/643904 | 11:27 |
| *** mkuf has joined #openstack-containers | 11:29 | |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Add nodegroup representation https://review.openstack.org/604823 | 11:31 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Preparation for nodegroups https://review.openstack.org/642007 | 11:31 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Adapt conductor cluster handler https://review.openstack.org/642008 | 11:31 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Adapt existing drivers https://review.openstack.org/642009 | 11:31 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Adapt cluster object https://review.openstack.org/642010 | 11:31 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: [WIP] API changes for nodegroups https://review.openstack.org/604824 | 11:31 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Very [WIP] conductor changes https://review.openstack.org/629273 | 11:31 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Super uber [WIP] k8s_fedora_atomic_v2 https://review.openstack.org/629274 | 11:31 |
| brtknr | strigazi: flwang1: regarding OCCM, its semi-urgent... we are using a custom branch at the moment with OCCM enabled for queens but would be nice to use the mainline repo | 11:35 |
| brtknr | strigazi: Is ttsiouts about? | 11:36 |
| *** kaiokmo has quit IRC | 11:37 | |
| brtknr | Don't worry, I'll send him an email, but would be nice to have discussions here | 11:38 |
| brtknr | about nodegroups | 11:39 |
| *** kaiokmo has joined #openstack-containers | 11:39 | |
| *** kaiokmo has quit IRC | 11:39 | |
| *** kaiokmo has joined #openstack-containers | 11:40 | |
| brtknr | flwang1: strigazi: also since when did it become possible to request specific kube_version? Is it after Rocky? | 11:43 |
| brtknr | My suspect Stein. I tried adding kube_version label to a cluster and it didn't have any effect. This is on an external cloud running Rocky so it might be an older version | 11:44 |
| *** udesale has joined #openstack-containers | 11:45 | |
| brtknr | Oops, my bad, I now realise that `kube_tag` is the valid label | 11:46 |
| *** rcernin has quit IRC | 11:46 | |
| *** ttsiouts has joined #openstack-containers | 11:47 | |
| *** belmoreira has joined #openstack-containers | 11:47 | |
| strigazi | brtknr which branch? | 11:48 |
| brtknr | strigazi: I think rocky... I am not definite as this is external cloud, not one that we manage | 11:49 |
| strigazi | brtknr: you can also send to the ML | 11:50 |
| strigazi | brtknr: gerrit actually is the best | 11:50 |
| strigazi | brtknr: I didn't underdstand your question, what is semi-urgent? | 11:51 |
| brtknr | strigazi: Using gerrit for feedback is a great idea! | 11:58 |
| brtknr | strigazi: semi-urgent -- merging OCCM for queens | 11:58 |
| ttsiouts | brtknr: Hi! | 12:01 |
| ttsiouts | brtknr: it seems that I missed something in the first patch | 12:01 |
| *** udesale has quit IRC | 12:01 | |
| ttsiouts | brtknr: this is why you get the AttributeError | 12:02 |
| ttsiouts | brtknr: I'll update the patch | 12:02 |
| ttsiouts | brtknr: thanks again for testing | 12:02 |
| *** udesale has joined #openstack-containers | 12:02 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum stable/rocky: k8s_fedora: Add ca_key before all deployments https://review.openstack.org/643924 | 12:06 |
| brtknr | ttsiouts: my pleasure, look forward to doing more tests | 12:06 |
| brtknr | strigazi: Have you guys looked again at whether ostree upgrade to docker-ce allows k8s to run containers in privileged mode? | 12:08 |
| strigazi | brtknr: briefly, nothing obvious | 12:09 |
| strigazi | brtknr: I need to find some time to look into it. | 12:10 |
| brtknr | strigazi: I thought there was a story related to this on storyboard but cant find it... | 12:16 |
| *** udesale has quit IRC | 12:27 | |
| *** udesale has joined #openstack-containers | 12:28 | |
| *** zul has joined #openstack-containers | 12:52 | |
| *** pcaruana|afk| has joined #openstack-containers | 12:54 | |
| henriqueof | 'failed: rpc error: code = Unknown desc = rewrite resolv.conf failed for pod' is someone else having this error? | 12:55 |
| *** pcaruana has quit IRC | 12:56 | |
| *** ttsiouts has quit IRC | 13:36 | |
| *** sapd1_x has quit IRC | 13:38 | |
| *** ttsiouts has joined #openstack-containers | 13:41 | |
| *** absubram has joined #openstack-containers | 13:41 | |
| *** ttsiouts_ has joined #openstack-containers | 13:42 | |
| *** absubram_ has joined #openstack-containers | 13:42 | |
| *** absubram has quit IRC | 13:45 | |
| *** absubram_ is now known as absubram | 13:45 | |
| *** ttsiouts has quit IRC | 13:46 | |
| *** pcaruana|afk| has quit IRC | 13:48 | |
| *** sapd1_x has joined #openstack-containers | 13:50 | |
| *** alisanhaji has quit IRC | 14:02 | |
| *** mgariepy has quit IRC | 14:18 | |
| *** hongbin has joined #openstack-containers | 14:28 | |
| brtknr | henriqueof: where are you seeing this? | 14:31 |
| henriqueof | Kubelet logs, inside the pod there are invalid values for DNS thus there is no pod-to-pod communication. | 14:32 |
| henriqueof | I am givin' up on Magnum, two months plus trying. | 14:32 |
| *** pcaruana has joined #openstack-containers | 14:33 | |
| *** alisanhaji has joined #openstack-containers | 14:35 | |
| *** sapd1_x has quit IRC | 14:42 | |
| openstackgerrit | OpenStack Release Bot proposed openstack/python-magnumclient stable/stein: Update .gitreview for stable/stein https://review.openstack.org/644159 | 14:52 |
| openstackgerrit | OpenStack Release Bot proposed openstack/python-magnumclient stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein https://review.openstack.org/644160 | 14:52 |
| openstackgerrit | OpenStack Release Bot proposed openstack/python-magnumclient master: Update master for stable/stein https://review.openstack.org/644161 | 14:52 |
| brtknr | Does anyone else see this issue when trying to generate cluster config using python 3.6+? https://storyboard.openstack.org/#!/story/2005257 | 15:06 |
| strigazi | brtknr: this was fixed. where do you see it? | 15:08 |
| NobodyCam | Good Morning OpenStack Container Folks! | 15:08 |
| brtknr | strigazi: is this a server side problem? or client side? | 15:12 |
| brtknr | this is on my machine | 15:12 |
| brtknr | the server is rocky | 15:12 |
| brtknr | not the master branch | 15:13 |
| strigazi | client https://review.openstack.org/#/c/638077/4 | 15:16 |
| strigazi | server https://review.openstack.org/#/c/638336/4/magnum/conductor/handlers/common/cert_manager.py | 15:17 |
| strigazi | brtknr: can you try with --debug? | 15:17 |
| brtknr | Oh, so the patch needs to be applied to the server too? | 15:19 |
| brtknr | strigazi: here's the debug: http://paste.openstack.org/show/747973/ | 15:20 |
| strigazi | brtknr: looks like client to me | 15:22 |
| brtknr | strigazi: yes, thats what I thought, as the certificate appears in the return header | 15:23 |
| brtknr | This was using the pip package I installed... I guess the master is not incorporated into pypi | 15:23 |
| *** janki has quit IRC | 15:26 | |
| *** absubram has quit IRC | 15:29 | |
| *** absubram has joined #openstack-containers | 15:29 | |
| brtknr | strigazi: would be nice to issue a new release with this fix... | 15:30 |
| brtknr | 2.12.0 got release the same day just before the fix was merged: https://github.com/openstack/python-magnumclient/commits/master | 15:32 |
| strigazi | brtknr: coming https://review.openstack.org/#/c/642609/2 | 15:32 |
| brtknr | whats FFE? | 15:34 |
| brtknr | strigazi: ^ | 15:34 |
| strigazi | feature freeze something | 15:35 |
| openstackgerrit | Spyros Trigazis proposed openstack/python-magnumclient stable/stein: python3 fixes https://review.openstack.org/644248 | 15:37 |
| openstackgerrit | Spyros Trigazis proposed openstack/python-magnumclient stable/stein: Support health_status on client side https://review.openstack.org/644249 | 15:37 |
| strigazi | flwang1: ^^ for the client release | 15:37 |
| *** mgariepy has joined #openstack-containers | 15:38 | |
| strigazi | brtknr: ^^ | 15:38 |
| brtknr | ++1 | 15:40 |
| *** ykarel is now known as ykarel|away | 15:41 | |
| *** gsimondon has quit IRC | 15:49 | |
| brtknr | strigazi: after I attach a floating IP to the master and want to generate a new cluster config for that floating IP, how can I do that? | 16:05 |
| *** ttsiouts_ has quit IRC | 16:05 | |
| *** mkuf has quit IRC | 16:11 | |
| *** ttsiouts has joined #openstack-containers | 16:13 | |
| *** udesale has quit IRC | 16:21 | |
| strigazi | brtknr it won't work | 16:28 |
| strigazi | brtknr: the fip won't be inside the cert | 16:28 |
| strigazi | brtknr: this will fix it ^^ https://review.openstack.org/#/c/641547/ | 16:29 |
| brtknr | strigazi: yes I have already experienced it not working :) | 16:29 |
| *** henriqueof has quit IRC | 16:32 | |
| brtknr | strigazi: have you tried using podman instead of docker? do you think we'll be moving to use podman with fcos? | 16:39 |
| strigazi | brtknr: I don't think so, containerd, docker or cri-o | 16:40 |
| brtknr | strigazi: okay, just wondering | 16:41 |
| openstackgerrit | Vishal Manchanda proposed openstack/magnum-ui master: Add nodejs10 jobs on bionic. https://review.openstack.org/644290 | 16:54 |
| *** ttsiouts has quit IRC | 16:57 | |
| *** itlinux has joined #openstack-containers | 16:58 | |
| *** mgoddard has quit IRC | 17:02 | |
| *** cbrumm has quit IRC | 17:03 | |
| *** cbrumm has joined #openstack-containers | 17:08 | |
| *** ivve has quit IRC | 17:13 | |
| *** ykarel|away has quit IRC | 17:18 | |
| *** ramishra has quit IRC | 17:35 | |
| *** ricolin has quit IRC | 17:41 | |
| *** itlinux has quit IRC | 17:50 | |
| *** itlinux has joined #openstack-containers | 17:53 | |
| *** jmlowe has joined #openstack-containers | 18:03 | |
| *** itlinux has quit IRC | 18:10 | |
| *** flwang1 has quit IRC | 18:12 | |
| *** itlinux has joined #openstack-containers | 18:15 | |
| *** itlinux has quit IRC | 18:34 | |
| *** itlinux has joined #openstack-containers | 18:38 | |
| *** ivve has joined #openstack-containers | 18:45 | |
| *** chhagarw has quit IRC | 19:06 | |
| *** jmlowe has quit IRC | 19:13 | |
| *** itlinux has quit IRC | 19:15 | |
| *** itlinux has joined #openstack-containers | 19:24 | |
| lxkong | strigazi: hi, i know we could use 'contidions' in the template, but are you happy with the current approach? | 19:48 |
| lxkong | so we don't need to upgrade the template version | 19:49 |
| lxkong | and don't surprise the user who is using a very old Magnum version | 19:50 |
| *** itlinux has quit IRC | 19:53 | |
| *** itlinux has joined #openstack-containers | 19:57 | |
| eandersson | lxkong, I wouldn't mind a follow up patch where we update the template version and actually start using the new features | 20:12 |
| *** irclogbot_0 has joined #openstack-containers | 20:12 | |
| eandersson | the current apporach looks ok to me, but strigazi is way more informed on the floating ip topic | 20:13 |
| lxkong | eandersson: that's good suggestion, but let's get this job done first. | 20:14 |
| eandersson | yep for sure | 20:14 |
| *** jmlowe has joined #openstack-containers | 20:20 | |
| *** irclogbot_0 has quit IRC | 20:25 | |
| *** irclogbot_0 has joined #openstack-containers | 20:27 | |
| openstackgerrit | Merged openstack/python-magnumclient stable/stein: Support health_status on client side https://review.openstack.org/644249 | 20:37 |
| openstackgerrit | Merged openstack/python-magnumclient stable/stein: python3 fixes https://review.openstack.org/644248 | 20:37 |
| openstackgerrit | Merged openstack/python-magnumclient master: Update master for stable/stein https://review.openstack.org/644161 | 20:37 |
| *** alisanhaji has quit IRC | 20:38 | |
| *** dave-mccowan has joined #openstack-containers | 20:45 | |
| openstackgerrit | Merged openstack/python-magnumclient stable/stein: Update .gitreview for stable/stein https://review.openstack.org/644159 | 20:46 |
| openstackgerrit | Merged openstack/python-magnumclient stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein https://review.openstack.org/644160 | 20:46 |
| openstackgerrit | Merged openstack/python-magnumclient master: add python 3.7 unit test job https://review.openstack.org/637267 | 20:46 |
| *** pcaruana has quit IRC | 20:54 | |
| *** itlinux has quit IRC | 21:01 | |
| *** irclogbot_0 has quit IRC | 21:05 | |
| *** irclogbot_0 has joined #openstack-containers | 21:07 | |
| *** itlinux has joined #openstack-containers | 21:13 | |
| *** itlinux has quit IRC | 21:17 | |
| *** itlinux has joined #openstack-containers | 21:25 | |
| *** irclogbot_0 has quit IRC | 21:26 | |
| *** irclogbot_0 has joined #openstack-containers | 21:28 | |
| *** mrodriguez has joined #openstack-containers | 21:33 | |
| *** itlinux has quit IRC | 21:43 | |
| *** itlinux has joined #openstack-containers | 21:45 | |
| *** itlinux has quit IRC | 21:51 | |
| *** itlinux has joined #openstack-containers | 22:09 | |
| *** itlinux has quit IRC | 22:20 | |
| *** rcernin has joined #openstack-containers | 23:04 | |
| *** ivve has quit IRC | 23:09 | |
| *** hongbin has quit IRC | 23:19 | |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: Support <ClusterID>/actions/resize API https://review.openstack.org/638572 | 23:20 |
| openstackgerrit | Lingxian Kong proposed openstack/magnum master: Improve floating IP allocation https://review.openstack.org/641547 | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!