Monday, 2019-12-02

*** xinliang has joined #openstack-containers00:32
flwangcosmicsound: we(Catalyst Cloud) are using Ubuntu, but I know some other companies are using CentOS01:06
*** spsurya has joined #openstack-containers01:28
cosmicsoundI tried now ubuntu01:35
cosmicsoundAnd my magnum keeps on failing for now01:36
cosmicsound2019-12-02 01:22:38Z [kubernetes-ilbbpahqslad.kube_minions]: CREATE_FAILED  CREATE aborted (Task create from ResourceGroup "kube_minions" Stack "kubernetes-ilbbpahqslad" [abee1847-8156-45a2-989a-22985ac9f0d1] Timed out)01:36
cosmicsoundThis is not much to debug on01:36
cosmicsoundwould like to give centos a try01:36
cosmicsoundany reasons why your company choosed ubuntu over centos?01:37
cosmicsoundi know the community is bigger in terms of cloud enablement @ ubuntu01:37
*** dave-mccowan has joined #openstack-containers01:45
flwangi think it's because we are already on ubuntu before we got openstack01:52
*** ramishra has joined #openstack-containers03:36
*** ricolin has joined #openstack-containers03:47
*** ykarel has joined #openstack-containers04:20
*** udesale has joined #openstack-containers04:37
*** dave-mccowan has quit IRC04:40
openstackgerritSimon Merrick proposed openstack/magnum-ui master: Add ui for resizing clusters  https://review.opendev.org/69681404:43
*** xinliang has quit IRC04:44
*** ramishra has quit IRC05:20
*** ramishra has joined #openstack-containers05:39
*** elenalindq has joined #openstack-containers05:45
*** pcaruana has joined #openstack-containers06:32
*** ykarel is now known as ykarel|lunch07:43
*** ykarel|lunch is now known as ykarel08:20
openstackgerritDiogo Guerra proposed openstack/magnum master: [k8s] Update metrics-server  https://review.opendev.org/63793908:40
openstackgerritDiogo Guerra proposed openstack/magnum master: [k8s] Add heapster_enabled label  https://review.opendev.org/69683208:40
openstackgerritDiogo Guerra proposed openstack/magnum master: [k8s] Deprecate heapster  https://review.opendev.org/69683308:40
dioguerrabrtknr: I Added the heapster_enabled label and changed the default k8s metrics provider to the metrics-server. Altho in different commits...08:43
dioguerraMy devstack is bugging again... so i wanted to postpone. But since i did this a while ago, i just pushed upstream08:44
*** trident has quit IRC09:07
*** trident has joined #openstack-containers09:09
brtknrdioguerra: thank you :)09:39
andreincosmicsound: your issue sounds similar to what I experienced a while back. do you have "enable_cluster_user_trust: True" in globals.yml?09:45
*** elenalindq has quit IRC09:48
andreinhi flwang10:07
*** ivve has joined #openstack-containers10:08
*** lpetrut has joined #openstack-containers10:13
andreinhi brtknr10:28
openstackgerritAndrei Nistor proposed openstack/magnum master: Change k8s-keystone-auth docker repo  https://review.opendev.org/69670610:37
brtknrhi andrein10:45
brtknrstrigazi: are we okay to merge this? https://review.opendev.org/#/c/695709/10:57
brtknrstrigazi: also this: https://review.opendev.org/#/c/695709/310:57
*** ivve has quit IRC11:26
andreinbrtknr: wanted to ask for your feedback on the release notes on the k8s-keystone-auth repo change11:54
brtknrandrein: sure whats up11:59
andreinhttps://review.opendev.org/69670612:00
andreinbrtknr: ^^12:00
brtknrandrein: LGTM :)12:00
andreinthanks!12:01
*** ivve has joined #openstack-containers12:19
*** udesale has quit IRC12:36
*** udesale has joined #openstack-containers12:37
tridentbrtknr: Do you have any suggestions on the road forward regarding https://review.opendev.org/#/c/696253/ ? We probably have to revert that patch from stable/rocky. Is there after that any way to get the working code (for example my suggestion of moving the label loop to a separate part later in the process) merged? I tried to apply the patch I think is related to the differences in behaviour from rocky to stein (12:48
trident https://github.com/openstack/magnum/commit/2ab874a5be951a6eba4f9d4f54c106bc0c53d9b1#diff-75661c66ec3a574f251543009c12d86f ) but it didn't apply cleanly and I have not yet investigated why.12:48
openstackgerritAndrei Nistor proposed openstack/magnum master: Change k8s-keystone-auth docker repo  https://review.opendev.org/69670613:05
openstackgerritAndrei Nistor proposed openstack/magnum master: Change k8s-keystone-auth docker repo  https://review.opendev.org/69670613:09
*** dave-mccowan has joined #openstack-containers13:15
*** dave-mccowan has quit IRC13:21
brtknrtrident: i doubt that patch is relevant to be honest13:27
tridentThe reason I thought so was that it seem to change the order stuff is done by removing the cloud-init scripts to set up flannel which are the only parts that are run after enable-services-master.sh. So in fact it seems like enable-services-master.sh is the last script being run after that patch is applied. If I interpret correctly?13:30
tridentbrtknr: If I am correct, in stein it's quite okay that enable-services-master.sh blocks in that loop while waiting for flannel to be set up, which happens in another context which will not be blocked by enable-services-master.sh.13:31
brtknrtrident: i think the way that things are deployed in stein is different to rocky13:35
brtknrin  rocky, it is a SoftwareConfig13:36
brtknrwhereas in stein+ it is deployed as a software deployment13:36
brtknrtrident: can you try with this modification: https://seashells.io/v/yNAf5uuU13:40
brtknrand see if that works for you?13:40
tridentbrtknr: Will try, thanks!13:41
tridentbrtknr: Will take 15-20 minutes or so.13:43
brtknrtrident: also have you tried spawning the cluser with calico plugin ?13:45
tridentbrtknr: I have not tried with calico plugin. The modification you suggested above seem to work perfectly though. Both with tags 1.14.8 and 1.15.5.13:56
brtknrtrident: excellent13:58
tridentbrtknr: Is that a patch we could get merged into stable/rocky?13:59
brtknrtrident: lets see, its not something we do often14:01
openstackgerritBharat Kunwar proposed openstack/magnum stable/rocky: fix: Deploy enable_service last (rocky only)  https://review.opendev.org/69688414:01
brtknrtrident: ^14:03
brtknrlets see what strigazi and flwang have to say about this, i have never seen a patch merge directly to a stable branch14:03
brtknrbut in this case, the fix is only relevant to rocky14:04
tridentbrtknr: I actually read up a bit on it, and according to policy it seems to be okay at least: "Whether the fix is already on master and all consequent stable branches: a change must be a backport of a change already merged onto master, unless the change simply does not make sense on master."14:05
brtknrtrident: ah, thanks for checking that14:07
tridentbrtknr: That's by the way from https://docs.openstack.org/project-team-guide/stable-branches.html14:08
openstackgerritBharat Kunwar proposed openstack/magnum stable/rocky: fix: Deploy enable_service last (rocky only)  https://review.opendev.org/69688414:08
brtknrtrident: could you comment on the patch saying this addresses your issue14:09
brtknronce this gets merged, we can cut a new release14:09
openstackgerritBharat Kunwar proposed openstack/magnum stable/train: bug: Use configured heat-container-agent tag  https://review.opendev.org/69688714:13
tridentbrtknr: Absolutely, will comment!14:15
openstackgerritAndrei Nistor proposed openstack/magnum master: Change k8s-keystone-auth docker repo  https://review.opendev.org/69670614:15
brtknrandrein: you also need to update the k8s_keystone_tag in kubecluster.yaml14:17
*** ykarel is now known as ykarel|afk14:17
brtknrand please update the commit message saying you have bumped up the version for ussuri14:17
tridentbrtknr: Commented. Thanks very much for working with us on this!14:20
brtknrtrident: happy to help but if you want to run 1.16.x, there is no other option but to run magnum train14:21
brtknrunless you are happy to backport everything from master14:21
openstackgerritAndrei Nistor proposed openstack/magnum master: Change k8s-keystone-auth docker repo  https://review.opendev.org/69670614:22
andreinbrtknr: updated the tag, sorry I missed that14:22
brtknrandrein: thanks, looking good now14:24
tridentbrtknr: Yeah, we are aware of that and upgrading going forward is planned for the whole environments - just don't know exactly when we will get there. With a whole bunch of environments and most of them under different compliance requirements it's not always a simple a quick process unfortunately.14:30
brtknrtrident: i am glad you are able to support 1.15.5 though...14:31
brtknri didn't think rocky had the required changes to support this14:31
tridentbrtknr: With the heat_container_agent_tag=train-stable label it seem to work as far as I have tested at least.14:32
*** munimeha1 has joined #openstack-containers14:35
brtknrtrident: if you could test with calico plugin too and report back, that would be great, i am struggling to get my devstack environment up today for some reason14:38
brtknrtrident: we want to make sure we havent inadvertly broken something else as a result of this change14:38
tridentbrtknr: I'll try that as well!14:39
brtknrtrident: thanks14:39
*** spiette has joined #openstack-containers15:18
*** ykarel|afk is now known as ykarel|away15:54
*** ykarel|away has quit IRC16:02
tridentbrtknr: Okay, lets see. With calico plugin both 1.14.8 and 1.15.5 fails. They both go to CREATE_COMPLETE but in in 1.15.5 no calico-node-* gets spun up, only calico-kube-controllers and the rest of the pods hangs in ContainerCreating. However, without  https://review.opendev.org/696884 it fails in seemingly the exact same way (which makes sense, as all three scripts enable-services-master is moved after in that patch starts with "if [16:03
trident "$NETWORK_DRIVER" != "flannel" ]; then exit 0".16:03
tridentbrtknr: Without  https://review.opendev.org/#/c/696253/ on the other hand I don't even get that far. Most pods are just hanging in status Pending as nodes are tainted.16:04
tridentbrtknr: So my assessment is that those issues doesn't have anything to do with the fixes in question but is a separate issue that should probably be addressed separately?16:05
brtknrtrident: thanks for reporting back, i think i know why this is breaking16:06
brtknrtrident: are you using fedora atomic 29?16:08
tridentbrtknr: Yes.16:09
*** elenalindq has joined #openstack-containers16:21
*** ivve has quit IRC16:26
*** ykarel|away has joined #openstack-containers16:35
*** lbragsta_ has joined #openstack-containers16:36
*** udesale has quit IRC16:41
openstackgerritBharat Kunwar proposed openstack/magnum stable/rocky: fix: Deploy enable_service last (rocky only)  https://review.opendev.org/69688416:41
openstackgerritBharat Kunwar proposed openstack/magnum stable/rocky: k8s: Clear cni configuration  https://review.opendev.org/69691516:41
openstackgerritBharat Kunwar proposed openstack/magnum stable/rocky: k8s_fedora: Set rp_filter=1 for calico  https://review.opendev.org/69691616:41
brtknrtrident: please try again with the tree above16:41
brtknrthat ought to work16:41
tridentThanks, will try!16:43
*** trident has quit IRC16:51
*** trident has joined #openstack-containers16:53
*** df_sbr has joined #openstack-containers16:54
*** KeithMnemonic has joined #openstack-containers17:18
*** ykarel has joined #openstack-containers17:32
tridentbrtknr: Hmm.. This time the master didn't even get the kubectl command installed. So obviously it blocks on the loop trying to do the kubectl patch.17:34
*** ykarel|away has quit IRC17:34
tridentbrtknr: It seems to be writing stuff to /srv/magnum/kubernetes/install-kubernetes.sh including the atomic install commands. Then that script is never run.17:42
*** lbragsta_ has quit IRC17:44
*** ivve has joined #openstack-containers18:11
*** ricolin has quit IRC18:22
*** spsurya has quit IRC18:25
*** jmlowe has quit IRC18:39
*** jmlowe has joined #openstack-containers18:56
*** lpetrut has quit IRC19:00
*** ykarel has quit IRC19:07
*** jmlowe has quit IRC19:48
*** andrein_ has joined #openstack-containers19:54
*** kklimonda_ has joined #openstack-containers19:55
*** jrosser_ has joined #openstack-containers19:55
*** dioguerr1 has joined #openstack-containers19:57
*** pcaruana has quit IRC20:02
*** dioguerra has quit IRC20:02
*** andrein has quit IRC20:02
*** Jeffrey4l_ has quit IRC20:02
*** johanssone has quit IRC20:02
*** jrosser has quit IRC20:02
*** kklimonda has quit IRC20:02
*** andrein_ is now known as andrein20:02
*** kklimonda_ is now known as kklimonda20:02
*** jrosser_ is now known as jrosser20:02
*** Jeffrey4l has joined #openstack-containers20:02
*** johanssone has joined #openstack-containers20:03
*** pcaruana has joined #openstack-containers20:05
*** jmlowe has joined #openstack-containers20:14
*** munimeha1 has quit IRC20:20
*** rcernin has joined #openstack-containers21:57
*** rcernin has quit IRC21:59
*** elenalindq has quit IRC22:01
*** pcaruana has quit IRC22:11
*** rcernin has joined #openstack-containers22:57
*** dioguerr1 has quit IRC23:06
*** dioguerra has joined #openstack-containers23:06
brtknrflwang: Really sorry I still haven’t tested your patch because I’ve been failing to get a working devstack environment up and running23:15
flwangbrtknr: that's ok :)23:58

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