19:01:00 #startmeeting refstack 19:01:01 Meeting started Tue Apr 4 19:01:00 2017 UTC and is due to finish in 60 minutes. The chair is catherineD. Information about MeetBot at http://wiki.debian.org/MeetBot. 19:01:02 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 19:01:05 The meeting name has been set to 'refstack' 19:01:34 o/ 19:01:51 o/ 19:02:11 o/ 19:02:48 #link meeting agenda and notes, https://etherpad.openstack.org/p/refstack-meeting-17-04-04 19:03:00 hi everyone 19:03:11 hello! 19:03:47 Let's start 19:04:00 #topic Displaying RefStack documentation 19:04:33 #topic Hosting at RefStack 19:05:36 From last week Paul's mockup, I think we agreed to use docituls to generate html from rst ... 19:06:23 If everyone agrees ... I think pvaneck: will proceed to implementation? 19:06:43 yes, spec is fine. just waiting one the gate fix so we can merge it 19:06:52 ah, yes 19:06:59 great 19:07:02 yes 19:07:03 my bad, I can do that after meeting 19:07:16 o/ 19:07:16 mguiney: no it is not your issue ... 19:07:26 o/ 19:07:40 ah, my bad 19:07:44 we just need to merge https://review.openstack.org/#/c/452578/ 19:07:58 i thought you were referring to the error that came up in tests 19:08:02 I will merge https://review.openstack.org/#/c/452578/ after the meeting ... 19:08:05 which i can take a look at 19:08:51 mguiney: ... pvaneck: looked and had submited fixes for that ... in https://review.openstack.org/#/c/452578/ that we need to merge to fix the test problem 19:09:42 any other thoughts on this one ? 19:10:54 alright moving on .. 19:11:07 #topic Hosting at OpenStacl doc site 19:11:51 luzC: I think we just need to last patch to merge right ? 19:12:02 yes 19:12:45 the first patch was to add doc gate jobs to refstack... 19:12:51 'gate-{name}-docs-ubuntu-trusty' 19:13:00 'gate-{name}-docs-ubuntu-xenial' 19:13:06 o/ 19:13:16 o/ 19:13:46 that patch was merged yesterday .. so we should be OK 19:13:56 exactly 19:14:20 luzC: thanks for submitting all these patches 19:14:28 moving on ... 19:14:30 NP ;) 19:14:47 #topic Update existing certified data with the verified flag 19:15:03 #link Add a script to update the internal RefStack DB ( https://review.openstack.org/#/c/452518/ ) 19:15:29 I have reviewed and added comments .. 19:15:29 I saw the comments you posted, I am planning on working on those changes today 19:15:41 mguiney: thanks .. 19:16:01 I am planning on adding the csv parsing version as a flag 19:16:27 want to see whether the team agree with the comment .. that we should not depending on by using Google doc link as input .. 19:16:51 * mguiney nods 19:17:26 the upside of adding the option of using a csv instead is that you circumvent somewhat the need to set up a google api service account 19:17:28 +1 to catherineD comments 19:17:46 +1 19:17:54 +1 19:18:40 should we eliminate the Google option ? 19:18:53 did we agree that we'd be merging this into the repo? 19:19:27 Personally, I would like it to be merged in the repo under tools .. 19:19:28 I'd prefer to have only csv option 19:20:06 if order to keep it simple 19:20:17 because we could add mor option to update other field in the future (using RefStack API) 19:20:32 sslypushenko_: ++ 19:20:33 * mguiney nods 19:20:37 I agree with sslypushenko_ 19:20:38 that does make sense 19:21:36 luzC: pvaneck: hogepodge: how about you on only csv input? 19:21:47 I agree too 19:22:22 If we plan on housing this in the repo, then yes 19:22:23 In fact, I'd love to see a tool that auto convets our json scores docs to .csv so we can download them to our favorite spreadsheet tool. 19:22:49 Rockyg: yea .. that could be a new feature ... 19:23:25 It would make scoring much easier and help to get more prople (like me;) i9nvolved. 19:24:04 Rockyg: +1 to that 19:24:44 #agreed A CSV file will be used as the input for the verficiation field update script (no Google spreadsheetused as input). 19:24:52 Rockyg: noted 19:25:20 the google sheets option comes from my internal database of results, fwiw 19:26:19 but we can do a two step process, google -> csv -> refstack db, and vice versa 19:27:12 hogepodge: mguiney: yup, that way the script will be simple and concentrating on the DB update part 19:28:07 Now about update the DB fields by using RefStack APIs and not DB commands 19:28:27 the one issue that i was running into with converting google spreadsheet into csv and then parsing was that a lot of the modifications that can be made to make it more human readable make it very gross to parse 19:29:30 mguiney: we could help to see whether we can resolve that .. 19:29:42 newlines, for example, were really bad about breaking things 19:29:54 in terms of finding the end of entries 19:30:01 but yes, that sounds good 19:30:07 mguiney: I would still suggest reading the gdoc into memory, then we can parse that out to the csv 19:30:49 the google doc is an implementation detail on the osf side 19:30:54 can do. perhaps making the usual fixes and then reading into csv would be the best decision here 19:31:50 alright .. 19:32:08 any input about using RefStack API vs DB commands? 19:32:21 mguiney: yes, and where there are newlines we can create new rows, save some headaches 19:33:05 I figured, I'll just read in at a different point than I was before 19:33:24 say, after I split my lines and do other formatting fixes 19:34:03 I can absolutely use the RefStack api commands, I think I just overlooked converting those in the version I fixed 19:34:41 *pushed 19:35:01 mguiney: thanks please ping us anytime at #refstack if needed 19:35:11 * mguiney nods 19:35:22 thank you for the feedback and clarification :) 19:35:47 moving on ... 19:35:54 #topic Pending reviews 19:36:10 #link Fix py35 func tests ( https://review.openstack.org/#/c/452578/ ) 19:36:22 sslypushenko_: thx for merging that ... 19:36:50 catherineD: np 19:37:09 this should fix test failures on all tests 19:37:18 \o/ 19:37:24 awesome! 19:37:48 pvaneck: thx for submitting that 19:37:59 yep np 19:38:12 #link Fix to not run any test when the input test list is empty ( https://review.openstack.org/#/c/451599/ ) 19:38:34 luzC: thx for the review ... I will work on a new patch 19:38:47 thank you :) 19:38:54 #link Change doc references from DefCore to Interop Working Group ( https://review.openstack.org/#/c/390881/ ) 19:39:02 I guess we are still waiting ... 19:39:12 yes 19:39:30 #link Add scripts for running refstack-client in docker ( https://review.openstack.org/#/c/430701/ ) 19:39:43 it is still in my todo list 19:40:04 sslypushenko_: and mine too ... based on the fact that luzC: was able to pass the test 19:40:24 #topic Open discussion 19:40:46 I just need handle some corner cases.... 19:41:05 I am very please to nominate Luz for RefStack core 19:41:18 catherineD, I think the repos might have gotten their rename this weekend... 19:41:19 #link Nominate Luz for core ( http://lists.openstack.org/pipermail/openstack-dev/2017-April/114966.html ) 19:41:52 Rockyg: great then we will be able to merge our patch soon ... 19:42:16 +1 luzC core 19:42:23 I just +1 to the reply from Daniel Mellado. I seem to be losing some emails again.... 19:42:49 Rockyg: pvaneck: thx.. 19:42:59 +1 19:43:35 LuzC, congrats ) 19:43:45 thank you all for the nomination, I'll do my best :) 19:43:58 +1 that is awesome 19:44:17 luzC: well deserved ! 19:44:48 that is all the topic I have for today ... 19:44:58 any other thoughts? 19:45:12 It's really nice to see the team getting more energized. This is great. And thanks to all you guys here (and those who couldn't make it, but read the logs ;) 19:45:39 * catherineD Rockyg: only see Daniel's reply .. 19:46:25 Rockyg: yea this has been a 3 years journey !!! 19:47:03 anything else for today? 19:47:23 all good here 19:47:37 Alright .. I think we can close up for the day 19:47:47 thank you all! 19:47:51 bye! 19:47:59 have a good day, y'alL! 19:48:05 #endmeeting