Tuesday, 2018-02-13

*** mjturek has quit IRC00:47
*** slaweq has joined #openstack-meeting-501:03
*** wxy has joined #openstack-meeting-501:06
*** slaweq has quit IRC01:08
*** yamahata has quit IRC01:37
*** yamahata has joined #openstack-meeting-502:26
*** ricolin has joined #openstack-meeting-502:52
*** felipemonteiro has joined #openstack-meeting-502:59
*** felipemonteiro_ has joined #openstack-meeting-503:00
*** felipemonteiro has quit IRC03:04
*** felipemonteiro_ has quit IRC03:18
*** felipemonteiro__ has joined #openstack-meeting-503:18
*** felipemonteiro__ has quit IRC03:26
*** yamamoto has joined #openstack-meeting-504:44
*** slaweq has joined #openstack-meeting-505:05
*** ricolin has quit IRC05:09
*** slaweq has quit IRC05:10
*** slaweq has joined #openstack-meeting-506:01
*** slaweq has quit IRC06:02
*** yamamoto has quit IRC06:41
*** yamamoto has joined #openstack-meeting-506:43
*** yamamoto has quit IRC06:50
*** yamamoto has joined #openstack-meeting-506:55
*** slaweq has joined #openstack-meeting-507:49
*** slaweq_ has joined #openstack-meeting-508:03
*** slaweq_ has quit IRC08:08
*** yamahata has quit IRC08:23
*** yamamoto has quit IRC08:35
*** yamamoto has joined #openstack-meeting-508:39
*** slaweq_ has joined #openstack-meeting-508:55
*** slaweq_ has quit IRC08:59
*** ricolin has joined #openstack-meeting-509:12
kazshtest09:23
kei-ichitest09:24
*** kei-ichi has quit IRC09:33
*** kei-ichi has joined #openstack-meeting-509:34
*** slaweq_ has joined #openstack-meeting-509:36
*** slaweq_ has quit IRC09:41
*** derekh has joined #openstack-meeting-509:54
*** yamamoto has quit IRC09:58
*** yamamoto has joined #openstack-meeting-510:15
*** yamamoto has quit IRC10:22
*** ricolin has quit IRC10:56
*** yamamoto has joined #openstack-meeting-511:13
*** derekh has quit IRC11:23
*** derekh has joined #openstack-meeting-511:23
*** portdirect has quit IRC12:03
*** portdirect has joined #openstack-meeting-512:03
*** jbryce has quit IRC12:11
*** jbryce has joined #openstack-meeting-512:11
*** slaweq_ has joined #openstack-meeting-513:01
*** slaweq_ has quit IRC13:05
*** yamamoto has quit IRC13:41
*** slaweq_ has joined #openstack-meeting-513:45
*** slaweq_ has quit IRC13:50
*** mjturek has joined #openstack-meeting-514:15
*** yamahata has joined #openstack-meeting-514:20
*** yamamoto has joined #openstack-meeting-514:27
*** iyamahat has joined #openstack-meeting-514:46
*** slaweq_ has joined #openstack-meeting-514:47
*** roman_g has joined #openstack-meeting-514:52
*** slaweq_ has quit IRC14:52
*** powerds0111 has quit IRC14:55
*** powerds0111 has joined #openstack-meeting-514:56
*** ying_zuo has quit IRC14:58
*** d|k has joined #openstack-meeting-514:58
*** ying_zuo has joined #openstack-meeting-514:58
mattmceuen#startmeeting openstack-helm15:00
openstackMeeting started Tue Feb 13 15:00:13 2018 UTC and is due to finish in 60 minutes.  The chair is mattmceuen. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: openstack-helm)"15:00
openstackThe meeting name has been set to 'openstack_helm'15:00
mattmceuen#topic rollcall15:00
*** openstack changes topic to "rollcall (Meeting topic: openstack-helm)"15:00
d|ko/15:00
mattmceuenhello everyboty15:00
mattmceueneverybody rather15:00
d|k<- not a bot, honest.15:00
mattmceuenoff to a great start here spelling-wise15:00
mattmceuenI'm not a bot, a bot could spell :)15:00
portdirect*smelling15:00
srwilkerso/15:01
*** Sam_Jonesi has joined #openstack-meeting-515:01
roman_go/15:01
mattmceuenHere's our agenda for today, please add in anything addn'l you'd like to discuss https://etherpad.openstack.org/p/openstack-helm-meeting-2018-02-1315:01
mattmceuen#topic Office Hours reminder15:02
*** openstack changes topic to "Office Hours reminder (Meeting topic: openstack-helm)"15:02
mattmceuenJust a quick reminder -- we just finished a weekly office hour session, and we have one more in seven hours :)15:02
mattmceuenWe currently have three per week and they're on our team wiki.  Thanks to everyone who helps facilitate (and who asks questions)15:03
mattmceuendo we have jayahn today by chance?15:03
*** gagehugo has joined #openstack-meeting-515:04
mattmceuenHe was going to try to find an additional time that worked for SK folks, I'll circle back w/ him later15:04
mattmceuen#topic Mascot voting results15:05
*** openstack changes topic to "Mascot voting results (Meeting topic: openstack-helm)"15:05
mattmceuenhttps://etherpad.openstack.org/p/openstack-helm-mascot15:05
*** gmmaha has joined #openstack-meeting-515:05
mattmceuenI must say that the Ratel made quite a showing!15:05
mattmceuenIt tied "everything else put together"15:05
portdirecthttp://i0.kym-cdn.com/photos/images/newsfeed/000/140/324/honey-badger-it-just-takes-what-it-wants-t-shirts_design.png15:06
mattmceuenWe'd like to maintain consensus on top of voting supremecy if possible.  So: speak now or forever hold your peace with the Ratel as our Mascot?15:07
* mattmceuen voted for Sea Otter, but is happy to adopt the inspirational Ratel15:07
srwilkersinspirational indeed15:08
srwilkersim inspired to learn to run backwards too15:09
mattmceuenSince I believe the majority of folks who voted for alternative candidates are not present, let us make the Ratel our Mascot-Elect, and revisit one final time to affirm consensus.15:09
mattmceuenAny more thoughts on this topic15:10
mattmceuen#topic Values YAML ordering15:10
*** openstack changes topic to "Values YAML ordering (Meeting topic: openstack-helm)"15:10
mattmceuenI know we've discussed this in various forums with various folks, but I wanted to discuss here before moving on it15:11
mattmceuenSteve has a PS out: https://review.openstack.org/#/c/542279/15:11
portdirectI like this, but i think each chart should be hit seperately15:12
mattmceuenThe idea is that we'd like to have a standard ordering to our values YAMLs across charts, and  the most objective way to to this is is simply to alphabetize all the keys15:12
portdirectand the sort should aslo go through all the keys, not just top level15:12
portdirect*also15:12
srwilkersportdirect: that's sane15:12
mattmceuenIs there a tool that would do this for us (or would it be easier to make a tool vs by hand)?15:13
portdirectFor this to have value, can we also add this to the linting gate15:13
srwilkersmattmceuen: depends on whatever text editor youre using15:13
srwilkersportdirect: ++15:13
mattmceuenYeah, and if it can be linted it can be printed!15:13
*** lamt_ has joined #openstack-meeting-515:13
portdirectatom has a load of plugins, though they all strip comments15:13
portdirectwhich we should keep15:13
portdirectso doing this the 1st time will be labourious15:14
portdirectbut a great bit of low hanging fruit for a new contrib to get under there belt15:14
mattmceuenSo I am not convinced on this approach.  I think we are sacrificing readability for consistency.15:14
portdirect*their15:14
mattmceuenI have heard a lot from portdirect and srwilkers on this and I'm happy to bow to team concensus15:15
portdirectfair, though in the interests of democracy, we are but two voices.15:15
mattmceuenCan you guys please pitch the benefits and why it's worth the readability drop?15:15
d|ksort of a strawman reaction: i think alphabetical organization makes some sense (and a standard ordering makes a TON of sense), but might it on the other hand be useful to have the things users might be most likely to edit at the top?15:15
alanmeadowsAlso, are there other simple consistencies beyond this we’d like to enforce at the same time?15:15
srwilkersthats a fair point, but that quickly becomes subjective djk15:16
d|kstrooth15:16
alanmeadowsEven outside of values15:16
portdirectalanmeadows: yes15:16
portdirectbut this is just one step on the road to helm8 ;)15:16
srwilkersand id like to have something consistent we can easily enforce, else we'll start seeing drift between the charts i think15:16
portdirectits already happened15:16
portdirectand we have a large number of charts, and contribs now15:17
srwilkersknowing where to find something chart to chart makes working with them easier, instead of having to run find on whatever key youre looking for between charts15:17
d|ki guess my point could be better addressed by just ensuring that values/charts are well-commented.15:17
roman_gAttributes are not in alphabetic order, for ex. Here https://review.openstack.org/#/c/542279/7/calico/values.yaml15:17
portdirectthat without some formal schema we will start to drown in minor deltas15:17
roman_gline 70: CALICO_NETWORKING_BACKEND and line 72 ETCD_CA_CERT_FILE and line 78 CALICO_DISABLE_FILE_LOGGING15:18
srwilkersroman_g: right.  we're proposing to order them alphabetically15:18
mattmceuenRevisiting the idea of "top level keys alphabetized" vs "all keys alphabetized" -- do we 90% of what you're after srwilkers with just the top level, while preserving most of the readability?15:18
srwilkersi think we need to order everything, as portdirect mentioned15:19
mattmceuenYep that was not my question though15:19
mattmceuenYou'd brought up the problem of having to Ctrl+F as a pain point -- wondering whether you'd have to Ctrl+F substantially less if we just organized the top level?15:20
srwilkerswe'd have to ctrl+f less if everything was ordered, id say15:21
srwilkerstop keys and subkeys15:21
portdirectsome of our charts have 15K lines in the values15:21
mattmceuenLet's do this please -- can we have something concrete to look at here -- maybe do the full depth alpha on your PS, and then let's look at it next team meeting?15:21
roman_gI would make it all alphabetical in distro, but not would enforce it for users.15:22
mattmceuen(^srwilkers)15:22
srwilkersmattmceuen: sure15:22
roman_ghttp://yamllint.readthedocs.io/en/latest/rules.html#module-yamllint.rules.key_ordering yawl alphabetical linter15:22
roman_g*yaml15:22
srwilkersill hit the prometheus chart as an example15:22
mattmceuenoo good find roman_g15:22
mattmceuenthanks srwilkers15:23
mattmceuen#action Revisit YAML value ordering next time with prometheus as example15:23
alanmeadowsroman_g: it has to be enforced on commit15:23
alanmeadowsor there is no point to the refactoring now15:23
roman_galanmeadows, I agree, it must be done on commit. But let's not enforce alphabetical order for users.15:24
srwilkersroman_g: not sure i follow15:24
alanmeadowsin fact I see a number of things from yamllint should be implemented15:24
mattmceuenby users do you mean e.g. overrides roman_g?15:24
roman_gyes.15:25
roman_gSomeone takes stock yaml, and modifies it. It should be applied w/o any problems, without warnings about non-alphabetical order of keys15:25
mattmceuenagree - overrides are often going to be pets, and when not, should be up to the user to enforce what they want15:26
srwilkersroman_g: that's outside the scope of osh -- whatever a user applies as overrides get's validated by helm, not the charts themselves15:26
mattmceuenall linting should be done at commit time, not runtime15:26
mattmceuenAlright - anything else before we move on?15:27
mattmceuen#topic Ceph changes merged15:27
*** openstack changes topic to "Ceph changes merged (Meeting topic: openstack-helm)"15:27
* mattmceuen woooooooooooooooooooooooooooooooooooooooooooooo!!15:27
d|kwhat. he. said.15:28
*** mattmceuen has quit IRC15:28
portdirectnext step is gonna be to split the chart out15:28
*** mattmceuen has joined #openstack-meeting-515:28
portdirectand see how that effects upgrades15:28
mattmceuenBig thanks to portdirect for getting this PS past the finish line, and to all the folks whose work this builds on (d|k renis ganesh and anyone else I missed)15:29
d|kportdirect: is there a plan for supporting device naming by [i]scsi IDs, or should we just use the fact that there are symlinks for those?15:29
d|kportdirect: also, you REWL for dragging this thing to the finish.15:29
portdirectd|k: yes to scsi15:30
d|kkewl15:31
alanmeadowsalso what about scrubbing, i.e. clean redeploy15:31
portdirectcan you add your awesome work in here?15:31
gmmahaportdirect: split the chart out meaning, just the ceph piece or helm-toolkit also goes along i presume?15:31
alanmeadowsa script is a decent half measure, but long-term?15:31
portdirectalanmeadows: do we want to support that?15:31
alanmeadowsat the very least is our script a tools/ helper?15:32
portdirecti suspect we do, but want consensus before people say portdirect you killed my cat pic collection.15:32
d|ki'd be happy to put up a change with my little physical-to-logical mapping thing and then that can be cleaned up/improved15:32
portdirectd|k: sounds great15:32
alanmeadowsIt shouldn't be super easy to tap into but it should be there, ephemeral clouds are the wave of the future15:33
portdirectalanmeadows: I could add a oldschool "REALLY_ZAP_OSDS" flag?15:33
alanmeadowsI think thats fine, I think it may also go along with maybe what we would do for PVCs on statefulsets15:33
portdirectroger, wfm15:33
alanmeadowsREALLY_OBLITERATE_ALL_MY_DATA_ON_PURGE15:33
portdirectgmmaha: the plan is to try splitting the mons, osds, and clients into seperate charts15:34
portdirectto allow them to be upgraded in sequence15:34
gmmahaportdirect: aaah.. that sounds like fun.. feel free to rope me in on the work15:35
portdirectgmmaha: will do - the split will be easy, the validation less so :)15:36
mattmceuenyou have the best definition of the word "fun" gmmaha :-D15:36
gmmahaportdirect: true dat... (now how all can we make this fail)15:36
gmmahamattmceuen: :D15:36
mattmceuenthat is the real fun part... torture the thing15:36
mattmceuenok - next topic15:37
gmmahaportdirect: is this split planned afgter we move it to ceph-helm or prior?15:37
gmmahasorry mattmceuen15:37
mattmceuend'oh15:37
gmmahaslow typer here15:37
mattmceuenNope my bad go for it15:37
portdirectgmmaha: the last step before moving15:37
portdirectthe day is approaching fast now15:37
gmmahaportdirect: cool. guess it's time to cue t15:38
gmmahathe music and get coffee15:38
gmmahathanks portdirect mattmceuen :)15:38
*** slaweq_ has joined #openstack-meeting-515:38
mattmceuenthanks gmmaha!15:38
mattmceuen#topic Let's get the PS in good shape!15:39
*** openstack changes topic to "Let's get the PS in good shape! (Meeting topic: openstack-helm)"15:39
mattmceuenuno momento - copy paste fail15:39
mattmceuen    https://review.openstack.org/#/c/538419/ - Add template for Fluent logging index - 1 week15:39
mattmceuen    https://review.openstack.org/#/c/537800/ - Change the check method for the ceph ods version - 2 week15:39
mattmceuen    https://review.openstack.org/#/c/539477/ - Change osd pool default size set 1 - 1 week15:39
mattmceuen    https://review.openstack.org/#/c/539019/ - Managing generic loopback devices for testing15:39
mattmceuen    https://review.openstack.org/#/c/528488/ - LMA documentation15:39
mattmceuen    https://review.openstack.org/#/c/540938/ - Deep merge functionality for Helm-Toolkit15:39
mattmceuenSome of these have been languishing as we've been busy with some big efforts15:40
mattmceuenBut let's try to get some solid feedback and/or merges on these guys in the next 2 days please15:40
mattmceuenalso     https://review.openstack.org/#/c/535699/ - Use ceph-admin-keyring to execute storage_init script in Cinder.15:41
portdirectwould also be awesome to expand the active reviewers15:41
srwilkersportdirect: ++15:41
mattmceuen+115:41
portdirectasl currently 60% is done by three people: http://stackalytics.com/?module=openstack-helm15:41
mattmceuenDoes anyone have any suggestions for how we can keep reviews moving?  Lessons learned from other projects?15:42
mattmceuen"one simple trick"?15:42
*** slaweq_ has quit IRC15:42
portdirectpointing out that a +/-1 is worth as much as a +/-2 is a big part of it i think15:43
srwilkersit'd be great to see more active reviewers over the next cycle so we can revisit core team makeup and membership15:43
portdirectwe really need more cores15:43
srwilkers^15:43
mattmceuenyep15:43
portdirectbut to get to that point people need to put the work in.15:43
mattmceuen+/-2s are a lot easier for cores to give once other team members have given strong reviews and +/-1s  -- agree portdirect15:44
portdirectits how I was brought up in OSS15:45
portdirectget stuck in :D15:45
srwilkers:p15:45
mattmceuenwell you did alright :)15:45
portdirect(and grow thick, loos skin)15:45
portdirect^^ Like a certain mascot15:45
mattmceuenthat's deep, man, deep15:46
mattmceuenAlrighty15:46
mattmceuen#topic Roundtable15:46
*** openstack changes topic to "Roundtable (Meeting topic: openstack-helm)"15:46
mattmceuenAny topics you all would like to discuss next time that might need some prep work, or things to bring up today?15:47
srwilkersim good i think15:47
mattmceuen#action The hardy Ratel is Mascot-Elect15:47
mattmceuenbtw I learned it's pronounced RAY-tel15:48
mattmceuenthat's all I got15:48
mattmceuengood meet peeps -- thanks for all your hard work.15:48
srwilkersfunny, i thought it was pronounced "honey badger"15:48
mattmceuenUHHHHWWHAAAAAT????15:48
mattmceuen#endmeeting15:49
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"15:49
openstackMeeting ended Tue Feb 13 15:49:04 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:49
openstackMinutes:        http://eavesdrop.openstack.org/meetings/openstack_helm/2018/openstack_helm.2018-02-13-15.00.html15:49
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/openstack_helm/2018/openstack_helm.2018-02-13-15.00.txt15:49
*** gmmaha has left #openstack-meeting-515:49
openstackLog:            http://eavesdrop.openstack.org/meetings/openstack_helm/2018/openstack_helm.2018-02-13-15.00.log.html15:49
*** d|k has left #openstack-meeting-515:49
*** Sam_Jonesi has quit IRC15:54
*** gagehugo has left #openstack-meeting-516:07
*** felipemonteiro has joined #openstack-meeting-516:09
*** roman_g has quit IRC16:09
*** felipemonteiro_ has joined #openstack-meeting-516:10
*** felipemonteiro has quit IRC16:15
*** slaweq_ has joined #openstack-meeting-516:24
*** coboluxx has quit IRC16:26
*** coboluxx has joined #openstack-meeting-516:26
*** slaweq_ has quit IRC16:29
*** stokvis has joined #openstack-meeting-517:02
*** stokvis has joined #openstack-meeting-517:02
*** slaweq has quit IRC17:02
*** slaweq has joined #openstack-meeting-517:03
*** slaweq has quit IRC17:07
*** iyamahat has quit IRC17:15
*** yamahata has quit IRC17:17
*** yamamoto has quit IRC17:21
*** yamamoto has joined #openstack-meeting-517:24
*** yamamoto has quit IRC17:24
*** iyamahat has joined #openstack-meeting-517:33
*** yamamoto has joined #openstack-meeting-517:40
*** yamamoto has quit IRC17:45
*** slaweq has joined #openstack-meeting-517:45
*** markvoelker has joined #openstack-meeting-517:46
*** slaweq has quit IRC17:50
*** yamahata has joined #openstack-meeting-517:51
*** derekh has quit IRC18:02
*** yamamoto has joined #openstack-meeting-518:06
*** yamamoto has quit IRC18:06
*** slaweq has joined #openstack-meeting-518:25
*** slaweq has quit IRC18:26
*** slaweq has joined #openstack-meeting-518:26
*** slaweq has quit IRC18:39
*** slaweq has joined #openstack-meeting-518:40
*** slaweq has quit IRC18:45
*** yamamoto has joined #openstack-meeting-519:06
*** yamamoto has quit IRC19:15
*** slaweq has joined #openstack-meeting-519:45
*** iyamahat_ has joined #openstack-meeting-519:45
*** iyamahat has quit IRC19:48
*** slaweq has quit IRC19:49
*** MikeG451_ has joined #openstack-meeting-521:03
*** pabelanger has quit IRC21:05
*** pabelanger has joined #openstack-meeting-521:05
*** iyamahat__ has joined #openstack-meeting-521:09
*** slaweq has joined #openstack-meeting-521:11
*** iyamahat_ has quit IRC21:11
*** felipemonteiro_ has quit IRC21:22
*** ttx has quit IRC21:30
*** slaweq_ has joined #openstack-meeting-522:06
*** lamt_ has quit IRC22:31
*** lamt_ has joined #openstack-meeting-522:31
*** felipemonteiro has joined #openstack-meeting-522:40
*** slaweq_ has quit IRC22:46
*** slaweq_ has joined #openstack-meeting-522:47
*** slaweq_ has quit IRC22:51
*** MikeG451_ has quit IRC22:53
*** lamt_ has quit IRC22:58
*** slaweq has quit IRC23:45
*** felipemonteiro has quit IRC23:50

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