Monday, 2018-09-17

*** openstackgerrit has quit IRC00:09
*** sawblade6 has joined #openstack-helm00:59
*** sawblade6 has quit IRC01:04
*** sawblade6 has joined #openstack-helm01:19
*** sawblade6 has quit IRC01:23
*** chenyb4 has joined #openstack-helm01:24
*** sawblade6 has joined #openstack-helm01:31
*** sawblade6 has quit IRC01:35
*** sawblade6 has joined #openstack-helm01:42
*** sawblade6 has quit IRC01:50
*** SamYaple has quit IRC02:14
*** sawblade6 has joined #openstack-helm02:22
*** sawblade6 has quit IRC02:33
jayahnevrardjp: portdirect: cloud you please make changes on loci so that we can make rally image as we discussed (the one that does not follow global requirements)? I remember you said there was a way to do that. :)02:34
*** sawblade6 has joined #openstack-helm03:01
*** sawblade6 has quit IRC03:06
*** sawblade6 has joined #openstack-helm03:07
*** sawblade6 has quit IRC03:11
*** sawblade6 has joined #openstack-helm03:28
*** openstackgerrit has joined #openstack-helm03:36
openstackgerritTin Lam proposed openstack/openstack-helm-infra master: DNM, RFC: Add pod security policy for k8s cluster  https://review.openstack.org/60209403:36
*** sawblade6 has quit IRC03:41
*** sawblade6 has joined #openstack-helm03:56
*** sawblade6 has quit IRC04:02
*** jamesgu__ has joined #openstack-helm04:53
*** nguyenhai has joined #openstack-helm04:54
*** nguyenhai_ has quit IRC04:56
*** lemko has joined #openstack-helm05:39
openstackgerritTin Lam proposed openstack/openstack-helm-infra master: Add LDAP support for k8s-keystone-auth in gate  https://review.openstack.org/60141905:44
*** sawblade6 has joined #openstack-helm05:54
*** jamesgu__ has quit IRC05:56
*** jamesgu__ has joined #openstack-helm05:56
*** sawblade6 has quit IRC05:58
*** aojea has joined #openstack-helm06:27
*** sawblade6 has joined #openstack-helm06:50
*** sawblade6 has quit IRC06:55
*** tmorin has joined #openstack-helm07:03
*** mdih has joined #openstack-helm07:06
evrardjpjayahn: pending... I see a few issues with what I did -- will try to fix it today07:29
jayahnevrardjp: great! thanks a lot.07:30
*** sawblade6 has joined #openstack-helm07:34
*** sawblade6 has quit IRC07:38
*** lemko has quit IRC07:49
evrardjpjayahn: it's also very theoretical :)07:59
*** sawblade6 has joined #openstack-helm08:37
*** roman_g has joined #openstack-helm08:38
*** sawblade6 has quit IRC08:44
*** sawblade6 has joined #openstack-helm08:59
*** sawblade6 has quit IRC09:03
*** sawblade6 has joined #openstack-helm10:23
*** chenyb4 has quit IRC10:28
*** sawblade6 has quit IRC10:28
*** sawblade6 has joined #openstack-helm10:51
*** sawblade6 has quit IRC10:59
evrardjpjayahn: yeah after further checking it will be hard until we set up an argument for bypassing the constraints11:22
evrardjpjayahn: but I think it's not appropriate, as rally is now quite independant, and publishing its own dockerfile and docker images11:22
evrardjpalternatively passing PIP_PACKAGES should help in the case you built the wheel before hand or the wheel is published11:23
evrardjpwhich looks okay to me : https://pypi.org/project/rally/#files11:23
*** nicolasbock has joined #openstack-helm11:48
*** sawblade6 has joined #openstack-helm12:01
*** Lnic has quit IRC12:02
*** sawblade6 has quit IRC12:05
*** sawblade6 has joined #openstack-helm12:25
*** aaronsheffield has joined #openstack-helm12:29
*** sawblade6 has quit IRC12:30
*** Tomatosoup- has joined #openstack-helm12:40
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Fluentbit: Update version, config util template  https://review.openstack.org/60278412:45
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Fluentbit: Add kernel, kubelet, and dockerd logs  https://review.openstack.org/60278612:46
srwilkersmorning jayahn and evrardjp :)12:46
evrardjphey srwilkers12:46
srwilkershow was your trip home?12:46
evrardjpgood -- I even slept, so I didn't change gates yet12:47
evrardjp:p12:47
evrardjpI'd like to fix what I have in loci first12:47
srwilkershaha.  that's okay, the gates can wait :)12:47
srwilkerssleeps more important anyway.  i didnt realize how much i didnt sleep in denver until i slept almost all weekend away12:47
evrardjpsrwilkers: I will need to add suse eventually, so I'd rather not pile on top of this :p Every little help counts :)12:48
evrardjpsrwilkers: lucky you on the sleep :)12:48
jayahnsrwilkers: good evening for me. :)13:06
jayahnevrardjp: thanks for advice. if rally has its own dockerfile, better to use it. :)13:07
evrardjpjayahn: yeah xrally is becoming a thing now13:08
jayahnI probably try to add new loci image with different project, we are pretty interested in cyborg project. ;)13:08
srwilkersjayahn: that would be awesome :)13:10
jayahnskt has our own ai accelator (inference engine) based on xilinx. we are interested in "integration of xlinx with cy org" happening in stein timeframe. happy to make that happen also with openstack-helm.13:12
*** munimeha1_ has joined #openstack-helm13:14
srwilkersthat'd be really cool.  we'd be happy to support you however we can.  did you want to create an item in storyboard for that work so we can make sure other's can see your team wants to take that on? :)13:16
*** Lnic has joined #openstack-helm13:16
jayahnsrwilkers: sure. I can do that, although it will be few months away work. :)13:19
srwilkersthat's okay :)13:20
srwilkersfigured it'd be nice to have so if anyone else comes along, they can see someone's already wanting to work on it :)  could potentially help reduce duplicate work13:20
openstackgerritMateusz Urbanczyk proposed openstack/openstack-helm master: Ensure one ip address is taken from network interface  https://review.openstack.org/60312513:22
Tomatosoup-up: it's me in case you have any questions, or would like to propose alternative solution to the bug that was described in commit message13:25
Tomatosoup-I'm not sure if the greedy strategy "take first" is a good solution, but seems sufficient13:26
*** Tomatosoup- has quit IRC13:31
*** Tomatosoup- has joined #openstack-helm13:32
*** sthussey has joined #openstack-helm13:42
openstackgerritMerged openstack/openstack-helm-infra master: Gate: only restart network manager if required  https://review.openstack.org/60276613:49
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: Armada: Disable rabbitmq tests  https://review.openstack.org/59214713:54
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: Armada: Disable rabbitmq tests  https://review.openstack.org/59214713:54
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: Armada: Add Ocata Armada gate  https://review.openstack.org/59180813:55
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Gate: Trim dev-deploy gates  https://review.openstack.org/60283313:56
evrardjpthanks srwilkers :D13:57
srwilkersevrardjp: :)13:58
*** eeiden has joined #openstack-helm14:00
*** eeiden has quit IRC14:09
*** eeiden has joined #openstack-helm14:11
*** tmorin has quit IRC14:19
Lnichello ! is there some core reviewer who can merge my commit ? ( https://review.openstack.org/#/c/600019/1 ) code was already review14:22
*** tmorin has joined #openstack-helm14:22
openstackgerritMark Burnett proposed openstack/openstack-helm-infra master: Add calico-node dependency on calico-settings job  https://review.openstack.org/60084114:27
srwilkersone of the items we discussed at the PTG was the values ordering spec -- https://review.openstack.org/#/c/552485/.  would be great to get some feedback on this, while i work to get some sort of validation we can use to enforce whatever ordering we align on14:51
*** tmorin has quit IRC14:59
evrardjpsrwilkers: will you add a yaml validator for this?15:01
*** mdih has quit IRC15:01
*** roman_g has quit IRC15:01
evrardjpin linting15:01
srwilkersevrardjp: yeah15:02
*** dpascheka has joined #openstack-helm15:03
evrardjpsrwilkers: commented on the review15:04
evrardjpthanks15:04
*** roman_g has joined #openstack-helm15:04
evrardjpif you plan to add a linter from now on , I guess it makes sense to add it on the spec ;)15:05
evrardjpruamel.yaml can load yaml with comments and have ordered dicts easily :p15:05
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: Specification: Value File Ordering  https://review.openstack.org/55248515:07
srwilkersevrardjp: awesome.  just amended to spec to include a reference there15:08
evrardjpsrwilkers: please note that yamllint makes it super easy to use alphabetically ordered keys without code15:09
openstackgerritsai battina proposed openstack/openstack-helm-addons master: Initial commit to add mission-control, distribution, xray charts Also add enterprise artifactory chart  https://review.openstack.org/58837015:10
openstackgerritTin Lam proposed openstack/openstack-helm-infra master: Helm-Toolkit: Document kubernetes entrypoint macros  https://review.openstack.org/60234515:12
openstackgerritsai battina proposed openstack/openstack-helm-addons master: Initial commit to add mission-control, distribution, xray charts Also add enterprise artifactory chart  https://review.openstack.org/58837015:13
openstackgerritMerged openstack/openstack-helm-infra master: Prometheus: Fix Prometheus endpoints in apache config  https://review.openstack.org/59715115:19
srwilkersevrardjp: thanks, will note that.15:35
openstackgerritsai battina proposed openstack/openstack-helm-addons master: Initial commit to add mission-control, distribution, xray charts Also add enterprise artifactory chart  https://review.openstack.org/58837015:49
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: Armada: Add Ocata Armada gate  https://review.openstack.org/59180816:03
openstackgerritRoy Tang proposed openstack/openstack-helm master: Neutron mtu setting for sriov nic  https://review.openstack.org/60317916:26
*** aojea has quit IRC16:53
*** cfriesen has joined #openstack-helm16:56
openstackgerritMerged openstack/openstack-helm-infra master: Helm-Toolkit: Add doc comments for keystone env snippets  https://review.openstack.org/60234317:11
openstackgerritJohn Lawrence proposed openstack/openstack-helm-infra master: Monitor postgresql, Openstack virt resources, api, logs, pod and nodes status  https://review.openstack.org/57997617:18
openstackgerritJohn Lawrence proposed openstack/openstack-helm-infra master: Monitor postgresql, Openstack virt resources, api, logs, pod and nodes status  https://review.openstack.org/57997617:34
openstackgerritJohn Lawrence proposed openstack/openstack-helm-infra master: Monitor postgresql, Openstack virt resources, api, logs, pod and nodes status  https://review.openstack.org/57997617:47
*** gmmaha has quit IRC17:56
*** gmmaha has joined #openstack-helm18:00
*** openstackgerrit has quit IRC18:10
*** openstackgerrit has joined #openstack-helm18:33
openstackgerritHari proposed openstack/openstack-helm-addons master: Implemente: ranger helm charts  https://review.openstack.org/56979718:33
openstackgerritJean-Charles Lopez proposed openstack/openstack-helm-infra master: Cluster checking and validation tools  https://review.openstack.org/60251518:41
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Monitor postgresql, Openstack virt resources, api, logs, pod and nodes status  https://review.openstack.org/57997619:03
openstackgerritHari proposed openstack/openstack-helm-addons master: Implemente: ranger helm charts  https://review.openstack.org/56979719:19
openstackgerritRenis Makadia proposed openstack/openstack-helm-infra master: Implement Ceph helm test  https://review.openstack.org/59991919:20
openstackgerritRenis Makadia proposed openstack/openstack-helm-infra master: WIP Ceph Helm Tests: cluster checking  https://review.openstack.org/60251519:20
openstackgerritRenis Makadia proposed openstack/openstack-helm-infra master: WIP Ceph Helm Tests: cluster checking  https://review.openstack.org/60251519:30
*** aaronsheffield has quit IRC19:59
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add yamllint to zuul-linter playbook  https://review.openstack.org/60322220:08
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: Specification: Value File Ordering  https://review.openstack.org/55248520:10
*** lemko has joined #openstack-helm20:11
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add yamllint to zuul-linter playbook  https://review.openstack.org/60322220:19
openstackgerritPete Birley proposed openstack/openstack-helm master: WIP: move glance to uswgi  https://review.openstack.org/60322320:25
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add yamllint to zuul-linter playbook  https://review.openstack.org/60322220:30
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add yamllint to zuul-linter playbook  https://review.openstack.org/60322220:37
evrardjpsrwilkers: haha I love the fact we are bringing this yamllint, compliance - conformity!20:37
srwilkers:D20:37
evrardjpfyi releases is using this: https://github.com/openstack/releases/blob/master/yamllint.yml so we are already more strict.20:37
evrardjpsrwilkers: I will check how it is possible to not have to type those comments for top level ordering20:40
srwilkersevrardjp: that'd be awesome.  a quick google search led me to using comments for now, but would like a better solution20:41
evrardjpsrwilkers: yeah it looks like it all points to a custom validator which is kinda not as good as yamllint20:45
evrardjpand more work20:45
evrardjpand disabling key-ordering seems bad ;p20:46
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Gate: Gather previous pod logs for failed containers  https://review.openstack.org/60322920:51
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add yamllint to zuul-linter playbook  https://review.openstack.org/60322220:53
openstackgerritSteve Wilkerson proposed openstack/openstack-helm master: Update the cirros to 0.4.0  https://review.openstack.org/57071320:56
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add yamllint to zuul-linter playbook  https://review.openstack.org/60322220:58
openstackgerritRenis Makadia proposed openstack/openstack-helm-infra master: WIP Ceph Helm Tests: cluster checking  https://review.openstack.org/60251521:00
openstackgerritRenis Makadia proposed openstack/openstack-helm-infra master: WIP Ceph Helm Tests: cluster checking  https://review.openstack.org/60251521:01
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Monitor postgresql, Openstack virt resources, api, logs, pod and nodes status  https://review.openstack.org/57997621:02
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Add yamllint to zuul-linter playbook  https://review.openstack.org/60322221:07
*** munimeha1_ has quit IRC21:27
*** eeiden has quit IRC21:41
evrardjpsrwilkers: yamllint is quite hard wired and not very flexible:  https://github.com/adrienverge/yamllint/blob/e4e99f0aba5bc894712a720d8516db556e48acf1/yamllint/rules/__init__.py21:49
evrardjpI think the best would be to change the code to add a dynamic loading fonction there using endpoints but I am not sure if the author is responsive21:50
srwilkersevrardjp: bummer :( we’ll see what we can do then21:59
evrardjpI am afraid ppl will not be as willing as you to do this dance :)22:00
evrardjpI can try to fork and see how it goes22:00
evrardjpbut I already have much on my plate, I'd rather work on gating22:00
evrardjpthis is just a nice to have22:00
evrardjpagreed?22:00
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: Gate: Gather previous pod logs for failed containers  https://review.openstack.org/60322922:01
portdirect++22:03
openstackgerritJean-Charles Lopez proposed openstack/openstack-helm-infra master: WIP Ceph Helm Tests: cluster checking  https://review.openstack.org/60251522:13
srwilkersIt’s a nice to have certainly. Was hoping for an easy way to enforce our values structure. The alternative would be to alphabetize everything but the conf: key22:20
*** lemko has quit IRC22:21
srwilkersBut not a pressing matter at the moment22:21
evrardjpyou play with this more than me, I listen to your advice :)22:27
srwilkersSounds like a good topic for the meeting tomorrow portdirect ;)22:27
openstackgerritRenis Makadia proposed openstack/openstack-helm master: Ceph helm test execution  https://review.openstack.org/60324522:45
openstackgerritPete Birley proposed openstack/openstack-helm master: WIP: move services to uswgi  https://review.openstack.org/60322322:54
*** aagate has joined #openstack-helm23:14
*** sthussey has quit IRC23:49
*** aagate has quit IRC23:52
*** aagate has joined #openstack-helm23:58

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