Wednesday, 2017-12-13

*** felipemonteiro_ has quit IRC00:02
openstackgerritPete Birley proposed openstack/openstack-helm-infra master: Makefile: Allow pulling of all images for a targeted chart  https://review.openstack.org/52754300:07
*** jaypipes has quit IRC00:28
*** openstackgerrit has quit IRC00:33
*** openstackgerrit has joined #openstack-helm00:45
openstackgerritGanesh Maharaj Mahalingam proposed openstack/openstack-helm-infra master: Enable ceph-mgr label on nodes to support luminous upgrade  https://review.openstack.org/52755200:45
openstackgerritGanesh Maharaj Mahalingam proposed openstack/openstack-helm master: Ceph: luminous fixes  https://review.openstack.org/50762800:46
*** duonghq has joined #openstack-helm01:06
*** julim has joined #openstack-helm01:52
*** julim_ has joined #openstack-helm01:54
*** julim has quit IRC01:57
*** gouthamr has joined #openstack-helm02:06
openstackgerritShu Yingya proposed openstack/openstack-helm master: Sahara: Add Sahara chart  https://review.openstack.org/47771802:21
*** julim has joined #openstack-helm02:53
*** julim_ has quit IRC02:57
*** julim has quit IRC03:59
*** openstackgerrit has quit IRC04:18
jayahnportdirect: are you still here?05:24
*** coolsvap has joined #openstack-helm05:31
*** openstackgerrit has joined #openstack-helm05:53
openstackgerritTin Lam proposed openstack/openstack-helm-infra master: WIP: Consolidate serviceaccounts and restrict rbac  https://review.openstack.org/52646405:53
*** felipemonteiro has quit IRC05:58
*** gouthamr has quit IRC06:48
*** duonghq has quit IRC07:41
*** duonghq has joined #openstack-helm07:41
openstackgerritSungil Im proposed openstack/openstack-helm-infra master: Fluent-logging helm chart  https://review.openstack.org/51954807:51
*** MarkBaker has quit IRC07:58
*** MarkBaker has joined #openstack-helm08:03
*** gmmaha has quit IRC09:20
*** korzen_ has quit IRC09:20
*** korzen_ has joined #openstack-helm09:21
*** gmmaha has joined #openstack-helm09:21
*** MarkBaker has quit IRC10:00
*** yamamoto has quit IRC10:13
*** duonghq has quit IRC10:23
*** yamamoto has joined #openstack-helm10:43
*** lunarlamp has quit IRC10:49
*** julim has joined #openstack-helm10:50
*** mariusv has joined #openstack-helm10:50
*** julim has quit IRC10:54
*** MarkBaker has joined #openstack-helm10:54
srwilkerso/12:57
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add example snapshot action to curator config  https://review.openstack.org/52753913:44
*** srwilkers_ has joined #openstack-helm13:45
srwilkershey jayahn or sungil -- just a quick comment re: the partial deploy conditional in https://review.openstack.org/#/c/519548/.  sorry, i know its late.  but i dont think this should be handled by a conditional check.  i think it should be handled via overrides for the label.  i can see instances where specific nodes would want a specific fluentbit configuration, and i think that use case and a blanket "put13:55
srwilkersfluentbit on every node" can both be handled with the standard label mechanism13:55
*** jaypipes has joined #openstack-helm13:57
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750514:21
jayahnsrwilkers: don't worry. my phone will go slience when needed. so you can freely throw message anytime.14:23
jayahnand..14:23
jayahnsrwilkers: I am looking into Sungil's PS.14:23
srwilkersoh dont say that, you might find yourself with plenty of late night IRC messages :D14:24
jayahnno problem there. :)14:24
jayahneither i will respond it and have a bit more late sleep.. or I will wake up with so many welcoming irc message.14:25
openstackgerritTin Lam proposed openstack/openstack-helm-infra master: WIP: Consolidate serviceaccounts and restrict rbac  https://review.openstack.org/52646414:25
jayahnokay. my understanding is.. Sungil totaly agrees with you on node selector mechanism. you are right there. Only difference is "when we put fluent-bit in every nodes".14:27
jayahnby definition, not having node selector means putting everywhere. so Sunil think it will be good to have that a mechansim to apply that specific case "apply to all".14:28
* jayahn my english is breaking.... 14:29
srwilkersif it is, i didnt notice :)14:29
srwilkersi understand the thought there.  im just wondering if the same cant be achieved by giving some arbitrary label to every single node without introducing a new conditional check to the chart14:30
jayahnhe just introduced one additional key called "fluentbit.partial_deploy" in values.yaml. and when you set that as true, it will follow original node selector mechanism.14:32
jayahnset as false, it will skip node selector and deploy to all.14:32
jayahni think it is just matter of style. :)14:33
jayahnwe can have a label applying to everynode. that will do the exactly same thing.14:33
jayahnwithough sunil's logic.14:33
jayahnso Sunil thought "applying every node" as a sort of  corner case, and put a specific mechanism to address that.14:34
jayahnnot a big issue. we can hear you opinion as well.14:34
jayahnI think Sungil will put some reply on your feedback early tomorrow. if you can tell your preference, I will make sure Sunil will hear that.14:35
srwilkersill throw it in the patchset :)  i wont keep you up any longer than i have14:36
srwilkersill chew on my opinion a bit more14:36
jayahnI will also think a bit more to sort out my opinion. I am kind of leaning toward yours...14:37
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750514:39
*** openstack has quit IRC14:39
*** openstack has joined #openstack-helm14:40
*** ChanServ sets mode: +o openstack14:40
*** openstack has quit IRC14:43
*** openstack has joined #openstack-helm14:44
*** ChanServ sets mode: +o openstack14:44
*** openstack has quit IRC14:44
*** openstack has joined #openstack-helm14:45
*** ChanServ sets mode: +o openstack14:45
*** felipemonteiro has joined #openstack-helm14:47
*** felipemonteiro_ has joined #openstack-helm14:48
jayahnsrwilkers: i put mine in the ps. :)14:48
srwilkersawesome :) im grabbing a coffee refill then going to put mine down14:48
* jayahn i am realzing I am often blocking "coffee refilling activity" of people there. yesterday - pete, today - steve. 14:51
*** felipemonteiro has quit IRC14:52
srwilkers:)14:53
*** marst has joined #openstack-helm14:56
srwilkersjust replied -- we can definitely handle that in the gate.  all nodes in the gate currently get the "openstack-control-plane":"enabled" label14:57
srwilkersbut this could just as well be something else if necessary14:57
* srwilkers realizes this would be a great time to have that gate documentation he mentioned finished14:58
*** gouthamr has joined #openstack-helm15:06
*** trantor has joined #openstack-helm15:08
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750515:15
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add example snapshot action to curator config  https://review.openstack.org/52753915:17
openstackgerritSamuel Gil Jonesi proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670415:21
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add example snapshot action to curator config  https://review.openstack.org/52753915:21
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add example snapshot action to curator config  https://review.openstack.org/52753915:34
openstackgerritSamuel Gil Jonesi proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670415:35
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add example snapshot action to curator config  https://review.openstack.org/52753915:41
*** julim has joined #openstack-helm15:51
*** julim has quit IRC15:55
*** trantor has quit IRC15:59
*** Neptu_ has joined #openstack-helm16:00
*** MarkBaker has quit IRC16:00
openstackgerritGabe Maurer proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670416:04
*** MarkBaker has joined #openstack-helm16:06
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750516:10
*** jaypipes is now known as jay_tampa16:10
openstackgerritPete Birley proposed openstack/openstack-helm-infra master: Support IP addresses as hosts within keystone_endpoint_uri_lookup  https://review.openstack.org/52727916:45
openstackgerritSamuel Gil Jonesi proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670416:46
openstackgerritSamuel Gil Jonesi proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670416:50
openstackgerritGabe Maurer proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670417:08
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750517:17
openstackgerritSamuel Gil Jonesi proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670417:24
openstackgerritPete Birley proposed openstack/openstack-helm master: Ceph: luminous fixes  https://review.openstack.org/50762817:38
openstackgerritGanesh Maharaj Mahalingam proposed openstack/openstack-helm-infra master: Enable ceph-mgr label on nodes to support luminous upgrade  https://review.openstack.org/52755217:50
gmmahaportdirect: the new ceph luminous change requires kube-dns IP to be part of the nameservers. without that mgr is unable to talk to the mon to register itself causing the crash17:56
gmmahahttps://review.openstack.org/#/c/527552/ change is probably horrible in adding it to the kubelet-reslov.conf but just testing the theory here..17:57
portdirectif that change is required - then the dnspolicy in the mgr is wrong17:57
portdirectI'm just spinning up a new cluster now to check17:58
gmmahaportdirect: thanks.. new to all these & learning as i stumble. Bear with me while i make all these stupid changes18:01
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750518:02
*** felipemonteiro_ has quit IRC18:03
openstackgerritPete Birley proposed openstack/openstack-helm master: Ceph: luminous fixes  https://review.openstack.org/50762818:06
portdirectgmmaha: not at all :)18:06
gmmahaportdirect: got it.. mgr was using the default dnsPolicy18:06
portdirecti think we should be good with the one change I added18:06
gmmahait had to be changed..18:06
gmmahalet me go ahead and revert the infra patch18:07
portdirectwith that i think we are in business :D18:08
gmmahaportdirect: is the makefile change necessary for that?18:08
portdirectno18:08
portdirectI've just run the gate through here - and all looks good :D18:08
portdirectback in a bit - got to run to a meeting.18:08
openstackgerritGanesh Maharaj Mahalingam proposed openstack/openstack-helm-infra master: Enable ceph-mgr label on nodes to support luminous  https://review.openstack.org/52755218:09
*** yamamoto has quit IRC18:45
*** yamamoto has joined #openstack-helm19:01
*** felipemonteiro has joined #openstack-helm19:02
*** felipemonteiro_ has joined #openstack-helm19:04
*** yamamoto has quit IRC19:06
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750519:06
*** felipemonteiro has quit IRC19:08
*** MarkBaker has quit IRC19:09
*** MarkBaker has joined #openstack-helm19:25
openstackgerritPete Birley proposed openstack/openstack-helm master: Ceph: luminous fixes  https://review.openstack.org/50762819:27
*** coolsvap has quit IRC19:31
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750519:37
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Fix elasticsearch curator file entry in configmap  https://review.openstack.org/52520019:44
openstackgerritMerged openstack/openstack-helm-infra master: Support IP addresses as hosts within keystone_endpoint_uri_lookup  https://review.openstack.org/52727919:45
openstackgerritGabe Maurer proposed openstack/openstack-helm master: Added uninstall information that will allow a clean development environment for testing or reinstalling OpenStack-Helm. This includes stopping the Kubelet and removing the openstack-helm directory.  https://review.openstack.org/52749620:07
*** yamamoto has joined #openstack-helm20:09
*** yamamoto has quit IRC20:18
openstackgerritMerged openstack/openstack-helm-infra master: Enable ceph-mgr label on nodes to support luminous  https://review.openstack.org/52755220:25
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750520:32
openstackgerritGabe Maurer proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670420:38
*** gouthamr has quit IRC20:45
openstackgerritSamuel Gil Jonesi proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670420:47
openstackgerritPete Birley proposed openstack/openstack-helm master: Ceph: luminous fixes  https://review.openstack.org/50762820:52
openstackgerritSamuel Gil Jonesi proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670420:56
*** japestinho has quit IRC21:02
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750521:08
openstackgerritGabe Maurer proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670421:09
openstackgerritdave kormann proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750521:30
openstackgerritSamuel Gil Jonesi proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670421:34
*** MarkBaker has quit IRC21:47
openstackgerritGabe Maurer proposed openstack/openstack-helm master: WIP: Updating all in one document  https://review.openstack.org/52670421:54
openstackgerritGanesh Maharaj Mahalingam proposed openstack/openstack-helm master: Ceph OSD disk targeting  https://review.openstack.org/52750522:13
*** marst has quit IRC22:24
gmmahasrwilkers: for patch 507628, is it alright that the ovs test failed. I was starting to look into why that happened22:33
gmmahai see that is non-voting, but wasn't sure if that is cause it is being phased-out or phased-in22:33
srwilkersgmmaha: yeah, it's fine.  the OVS gate has been a bit wonky for awhile22:35
portdirectwe often see issue with the current tests in the rally tests - as they push the gate vm's pretty hard22:35
srwilkers^22:35
portdirectthis will be fixed in the v3 gate22:35
gmmahaaaah22:35
portdirectso we can move them back to voting22:35
gmmahagotcha. thanks22:35
gmmahathanks srwilkers portdirect22:36
srwilkersnp :)22:36
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add example snapshot action to curator config  https://review.openstack.org/52753922:38
openstackgerritTin Lam proposed openstack/openstack-helm-infra master: Consolidate serviceaccounts and restrict rbac  https://review.openstack.org/52646422:41
SamYaplesrwilkers: portdirect are you all doing ovs on ovs? or ovs on calico?22:55
SamYapleas in ovs on weave or something22:55
srwilkersovs on ovs on ovs -- the new tripleo22:55
SamYapleso if itsovs on weave... you can kinda sometimes get it to work, but i can assure you its not doing what you think22:56
SamYapleyou cant do ovs on ovs as a general statement22:56
osh-chatbot4<portdirect> ovs and calico22:56
SamYaplegot it. thats perfect22:56
osh-chatbot4<portdirect> not one on the other22:56
SamYaplethe perfect combo22:56
SamYaplewell "on" as in calico for k8s and ovs for openstac22:56
osh-chatbot4<portdirect> yup22:56
osh-chatbot4<portdirect> all ovs runs in host network22:57
osh-chatbot4<portdirect> may do some fun with ovn, where some of the dbs run in calico22:57
SamYapleso the thing is, ovs running in host network doesnt really matter here22:58
SamYapleits theshared kernel module thats the issue22:58
osh-chatbot4<portdirect> yarp22:58
SamYaplebasically what im saying is you need to share the same DB backend, you *can* do ovn for k8s and openstack, its just a bit trickier22:58
SamYaplejust curious22:58
SamYapletrying to head off disaster!22:58
osh-chatbot4<portdirect> I have done that in the past22:59
osh-chatbot4<portdirect> but that was a year ago22:59
osh-chatbot4<portdirect> had to make some patches to the ovn-neutron plugin - as it assumed exclusivity22:59
SamYapleoh look who is here mr "i already did that"22:59
osh-chatbot4<portdirect> i think now it may be a lot cleaner22:59
osh-chatbot4<portdirect> if they have got the "underlay" mode sorted - as running neutron for essentually ipam was a pain.23:00
SamYaplewe just ripped out neutron+ipam from our env23:00
SamYaplebut that was not ipams fault23:01
SamYaplemostly23:01
*** felipemonteiro_ has quit IRC23:01
*** Neptu_ has quit IRC23:11
*** jay_tampa is now known as jaypipes23:13
*** Neptu_ has joined #openstack-helm23:20
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Add example snapshot action to curator config  https://review.openstack.org/52753923:23
openstackgerritMerged openstack/openstack-helm master: Ceph: luminous fixes  https://review.openstack.org/50762823:24
openstackgerritTin Lam proposed openstack/openstack-helm-infra master: Consolidate serviceaccounts and restrict rbac  https://review.openstack.org/52646423:25
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Add Grafana chart to OSH infra  https://review.openstack.org/52360523:40
*** julim has joined #openstack-helm23:44
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Add Grafana chart to OSH infra  https://review.openstack.org/52360523:47
*** julim has quit IRC23:49

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