09:03:10 #startmeeting check-in_20161215 09:03:10 Meeting started Thu Dec 15 09:03:10 2016 UTC. The chair is brainfunked. Information about MeetBot at http://wiki.debian.org/MeetBot. 09:03:10 Useful Commands: #action #agreed #halp #info #idea #link #topic. 09:03:10 The meeting name has been set to 'check-in_20161215' 09:03:21 #topic brainfunked's updates 09:03:55 #info All of the action items on me from yesterday were reviews, which I'll get to as soon as this check-in finishes. 09:04:05 #topic anmol's updates 09:04:14 please find my update @ https://paste.fedoraproject.org/506679/14817925/ and also I am planning to be on leave from 20th to 24th of this month. So, I am planning to have my pending PRs review ready by tomorrow 09:05:19 somehow the site always takes a long time to load for me 09:06:37 anmol, you'll need to ensure that the implementation for all the alerting and monitoring related specifications, including test cases, is available for review by the 27th at the latest 09:07:09 brainfunked, PR for review by 27th looks feasible 09:07:26 brainfunked, as per current status 09:07:27 anmol that technically gives you just till Monday to get at least the initial commits in, with all the specifications finalised and merged 09:08:28 brainfunked, It would be helpful if the specifications review can be accelerated and if we can take them in by today EOD 09:08:45 brainfunked, Its just my request 09:08:50 anmol, yes, we'll do that, I know there are some pending on, I"ll finish them immediately after this meeting 09:09:03 brainfunked, ok thanks 09:09:21 anmol, 27th btw would be really close, since we'll need to have a developer tested/qe ready build by the 30th 09:10:11 brainfunked, you said 27th is deadline to have review ready PRs right 09:10:35 anmol, yes, based on your leaves. we need to have a build by the 30th 09:10:50 hence would be ideal if you could get PRs in for review by Monday EoD 09:11:00 that way initial reviews could be completed while you're on leave 09:11:07 brainfunked, ok 09:11:09 and you can make any changes to get another round of reviews on 27th 09:11:16 ok 09:12:00 I'll double accelerate the process from my side :) 09:12:35 #action brainfunked, noopq, nishanth and k4n0 to finish reviews (whichever applicable individually) for https://github.com/Tendrl/specifications/issues/42 https://github.com/Tendrl/specifications/issues/40 https://github.com/Tendrl/specifications/issues/62 and get them merged by EoD today 09:12:39 anmol, thank you :) 09:12:50 I have reviewed them all 09:12:56 mee too 09:13:14 #accept Alerting and monitoring work is progressing at a steady pace. Pending reviews and merges of the specifications to be finished today. 09:13:28 "whichever applicable individually" :) 09:13:39 #topic dahorak's updates 09:14:23 no update related to Tendrl (working on unrelated tasks) 09:16:29 #info No update from dahorak "no update related to Tendrl (working on unrelated tasks)" 09:16:41 #topic fbalak's updates 09:16:46 Issue: https://github.com/Tendrl/usmqe-setup/tree/tendrl-setup-ceph - Create playbook for installation of Tendrl with Ceph 09:16:48 Issue: https://github.com/Tendrl/documentation/pull/62 - Review Ceph api calls if they are according to documentation 09:16:49 Progress: Working on it 09:16:51 Status: Good, not blocked 09:17:01 anmol, btw, I think fbalak would need to review the API relates specifications too 09:17:21 brainfunked, ok I'll add him to the specs 09:17:28 * my specs :) 09:17:45 fbalak, would you be able to review anmol's specs on priority today, for the impact on the API? 09:18:31 brainfunked: I am not sure, first I need to succesfuly install tendrl with ceph 09:18:53 fbalak, spec just needs to be read through, nothing to do with actual testing 09:19:12 brainfunked: than ok 09:19:27 fbalak, spec points out how the API will behave and you'd be in the best position to point out if the changes proposed can be tested properly :) 09:19:47 brainfunked: ok :) 09:20:03 fbalak, I know I'm bringing this to you at the last minute, but it would be awesome if you could take this review up on priority so that we can merge it later today 09:20:07 fbalak, thanks :) 09:20:13 anmol, spec link please 09:20:36 brainfunked: searching... 09:21:09 https://github.com/Tendrl/specifications/pull/20 09:21:24 https://github.com/Tendrl/specifications/pull/63 09:21:32 #accept fbalak is working on ceph related playbooks to test the API and validating the import documentation in the process 09:22:10 #action fbalak to review the API changes proposed on https://github.com/Tendrl/specifications/pull/20 and https://github.com/Tendrl/specifications/pull/63 to ensure that they can be tested 09:22:18 fbalak, anmol, thanks 09:22:25 #topic gowtham's updates 09:22:25 and also https://github.com/Tendrl/specifications/pull/18/ 09:22:37 Adding disk details in hardware inventory 09:22:44 git-issue: https://github.com/Tendrl/node_agent/issues/7 09:22:51 spec file: https://github.com/Tendrl/specifications/pull/21 09:22:55 #action fbalak to review the API changes proposed on https://github.com/Tendrl/specifications/pull/18/ to ensure that they can be tested 09:23:13 code-PR: https://github.com/Tendrl/node_agent/pull/6 09:23:22 Adding Network details in hardware inventory 09:23:24 anmol, I obviously haven't reviewed these yet, but fbalak should be able to look up the API impact section directly in the spec, correct? 09:23:51 brainfunked, I'll help him with that oofline 09:23:51 git-issue: https://github.com/Tendrl/node_agent/issues/78 09:23:54 *offline 09:24:03 anmol, thanks 09:24:11 spc file: https://github.com/Tendrl/specifications/pull/19 09:24:18 code-PR: https://github.com/Tendrl/node_agent/pull/79 09:24:32 all changes are done 09:24:41 brainfunked, np :) 09:24:41 Please review it quickly, This patches required for import cluster, 09:24:41 please review and Aprove spec file 09:24:54 gowtham, node_agent issue #7 doesn't look like one you're working on :) 09:25:18 brainfunked, checking 09:26:08 sorry :) 09:26:09 78 09:26:15 git-issue: https://github.com/Tendrl/node_agent/issues/78 09:26:40 i have done all changes in these two patch 09:26:51 i have updated spec file also 09:27:18 my few patches got merged 09:27:21 1.Travis commit message check is modified 09:27:28 git-issue: https://github.com/Tendrl/performance_monitoring/issues/10, https://github.com/Tendrl/alerting/issues/6 09:27:34 PR: https://github.com/Tendrl/performance_monitoring/pull/12, https://github.com/Tendrl/alerting/pull/8 09:27:41 2.Logged EtcdKeyNotFound exception 09:27:47 git-issue: https://github.com/Tendrl/gluster_integration/issues/26 09:27:53 PR: https://github.com/Tendrl/gluster_integration/pull/78 09:27:57 status: good 09:28:04 i am not blocked 09:28:10 gowtham, is there any implementation against https://github.com/Tendrl/specifications/pull/19 "network details in hardware inventory" ? 09:28:25 yes i have sent pr 09:28:27 PR 09:28:36 https://github.com/Tendrl/node_agent/pull/79 09:28:48 gowtham, please start linking these on the issue itself 09:28:49 both are done 09:29:04 it's difficult to figure out where the component issues are located otherwise 09:29:37 oj 09:29:39 ok 09:30:11 brainfunked, Am i take configuration issue now, Because i can learn lot in that task :) 09:31:05 gowtham, would you be able to help with https://github.com/Tendrl/specifications/issues/46 ? 09:31:21 brainfunked, ok i will do it 09:31:22 :) 09:31:27 nishanth, that ok with you? ^ 09:31:39 nishanth, ties in with your import spec 09:32:45 #accept population of hardware inventory for each node via the node agent specifications should be reviewed and merged today, initial implementation underway 09:33:05 gowtham, thanks for the update 09:33:11 moving on while nishanth responds 09:33:12 brainfunked, thanks 09:33:21 #topic k4n0's updates 09:33:51 k4n0, you had a bunch of issues for review yesterday 09:33:59 Those are done 09:34:23 Also completed few more PR reviews, spec reviews, individual owners can check comments on github 09:34:54 I havent been able to work on https://github.com/Tendrl/specifications/issues/32 09:35:03 Working on it today 09:35:27 Thats all 09:35:41 k4n0, have you made progress on #35 to #39? 09:35:50 Nope 09:36:16 #action k4n0 to keep working on specifications #32, #35 to #39 09:36:53 k4n0, just in case, could you go through the action times for you from yesterday and verify that they're all done? https://meetbot.fedoraproject.org/tendrl-devel/2016-12-14/check-in_20161214.2016-12-14-09.02.html 09:37:04 just to ensure that no reviews have been missed out on 09:37:12 They are done, I went through that list 09:37:20 ok thanks 09:37:24 brainfunked, https://github.com/Tendrl/specifications/issues/46, I think t1m1 shown interest on this yesterday. Not sure he started on that? 09:37:55 nishanth, brainfunked yes strated working on 09:37:57 #accept k4n0 finished the pending reviews from yesterday 09:38:37 ok, in that case, there's another we need to tackle: https://github.com/Tendrl/specifications/issues/28 nishanth 09:38:48 gowtham, do you think you could take https://github.com/Tendrl/specifications/issues/28 up instead? 09:38:59 brainfunked, checking 09:39:30 brainfunked, i can take this 09:40:04 brainfunked, okay 09:40:20 gowtham, cool, so I'll assign this to you 09:40:27 brainfunked, ok 09:40:43 #action gowtham to take up the tagged logging specification https://github.com/Tendrl/specifications/issues/28 09:41:43 brainfunked, import has a dependency on two specs own by gowtham 09:41:45 gowtham, once this is done, you should able to go back to the centralised configuration spec 09:42:12 nishanth, yes, the inventory ones, they're pending reviews, I'll finish them and let's get them merged today 09:42:15 brainfunked, ok sure :) 09:42:30 thanks 09:42:47 k4n0, btw, could you use some help from gowtham on any of your specs? 09:44:02 #topic kaverma and neha's updates 09:44:07 Title: import cluster workflow using UI 09:44:20 Spec-PR: https://github.com/Tendrl/specifications/pull/64 09:44:28 Spec-Issue: https://github.com/Tendrl/specifications/issues/56 09:44:36 Github-PR: https://github.com/Tendrl/tendrl_frontend/pull/60 (under review) 09:44:42 Github-Issue: https://github.com/Tendrl/tendrl_frontend/issues/63 09:44:52 Progress: Discussed with nishanth regarding import cluster workflow and updated my spec PR accordingly . please review it. now going to work on my github-PR's comments. 09:45:00 Status: good . not blocked. 09:45:02 End 09:45:07 Issue - https://github.com/Tendrl/tendrl_frontend/issues/66 09:45:07 Specs for Host, cluster, pools, file shares are in progress. k4n0 will comment on the specification doc's PR regarding the data to be shown for these views. 09:45:07 Sent PR for table component creation - https://github.com/Tendrl/specifications/pull/67 09:45:18 End 09:45:24 neha, please send link for spec PR 09:45:55 k4n0, I am working on it..will send you in half an hour 09:46:00 ok 09:47:20 k4n0, what should I write under Data model impact section in specification 09:47:46 neha, send the WIP progress spec for review, I need context, will comment on that 09:47:51 #action brainfunked, nishanth and noopq need to review https://github.com/Tendrl/specifications/issues/56 09:48:02 #udo 09:48:04 #undo 09:48:04 Removing item from minutes: ACTION by brainfunked at 09:47:51 : brainfunked, nishanth and noopq need to review https://github.com/Tendrl/specifications/issues/56 09:48:10 k4n0, ok 09:48:19 #action brainfunked, nishanth and noopq need to review the "import cluster UI" spec https://github.com/Tendrl/specifications/issues/56 09:48:49 brainfunked, btw, any updates on the user stories by JeffA ? 09:48:51 #action kaverma needs to link to the UX designs for each of the steps mentioned in the "import cluster UI" spec https://github.com/Tendrl/specifications/pull/67 09:49:24 #action neha to send an update to the "table component creation" specification 09:49:49 #topic mbukatov's updates 09:52:18 I go on with the tendrl-api setup, 09:52:49 https://github.com/Tendrl/usmqe-setup/pull/6#commits-pushed-5a9941d 09:53:13 #action mbukatov needs to ask selinux people about configuration of booleans 09:53:30 not sure if that'll register :) 09:53:37 #action mbukatov needs to ask selinux people about configuration of booleans 09:53:42 #action mbukatov needs to close the review and updates for https://github.com/Tendrl/usmqe-setup/pull/6 09:53:54 #action mbukatov needs to close the review and updates for https://github.com/Tendrl/usmqe-setup/pull/6 09:54:07 what's that pull request trying to achieve btw? 09:54:15 and that's all, as I had 1/2 day pto yesterday 09:54:40 brainfunked: automate installation of various tendrl component via ansible playbooks/roles 09:55:01 ok thanks 09:55:01 brainfunked: following tendrl documentation 09:55:15 brainfunked, I can't see any comment on table component creation spec's PR - https://github.com/Tendrl/specifications/pull/67. What I need to update 09:55:44 #accept mbukatov working on automated installation of tendrl components via ansible playbooks by using tendrl documentation 09:56:30 * sankarshan would like to point out that this is a good place where any documentation gaps will be identified. 09:56:32 neha, there are no details in the specification, you need to provide examples of how such a component would be used and how it impacts the UI functionality 09:56:48 * brainfunked agrees with sankarshan 09:57:02 brainfunked, ok 09:57:43 #action expanding the previous action assigned to neha, to say that the specification needs to be updated to examples of how such a component would be used and how it impacts the UI functionality 09:57:57 #topic nishanth's updates 09:58:07 #info nishanth is away, has provided me with the updated 09:58:16 #accept nishanth has finished the pending reviews 09:58:59 #action brainfunked needs to review https://github.com/Tendrl/specifications/issues/54, nishanth would continue working on it 09:59:06 #topic noopq's updates 09:59:17 Spec: https://github.com/Tendrl/specifications/pull/58 is done and approved 09:59:34 Working on Issue: https://github.com/Tendrl/tendrl-api/issues/39 09:59:46 Will write a spec for required attributes for the UI 10:00:03 Will review requested specs and issues 10:00:06 Nothing blocking me 10:00:41 noopq, this spec wasn't in the priority list, IAC, check the Focus Areas board for priorities on the API related issues and let me know if they need to be changed 10:01:13 noopq, where's the issue on the specifications repo for this spec? 10:01:16 brainfunked, You mean the Dec 16 once? 10:01:19 one* 10:01:42 brainfunked, this? 10:01:57 noopq, link? 10:02:06 this is the spec https://github.com/Tendrl/specifications/pull/58 10:02:14 noopq, that's a PR, where's the issue? 10:02:15 this is the issue https://github.com/Tendrl/tendrl-api/issues/39 10:02:37 the board is prioritised using issues, PRs come in later, so need an issue that can be tagged and prioritised 10:02:48 and the PR is approved 10:03:02 ok, so you want a issue in specifications repo? 10:03:23 https://github.com/Tendrl/specifications/issues/33 https://github.com/Tendrl/specifications/issues/35 those two, for example, were already present for being picked up 10:03:28 noopq, yes, please 10:03:32 brainfunked, will do 10:03:36 and link it to the PR 10:03:43 Alright 10:04:14 noopq, in fact, if you think there are any other API related things that aren't covered by any other specs, please file issues for them 10:04:23 noopq, and if there's nothing else, you could take a look at https://github.com/Tendrl/specifications/issues/22 10:04:26 brainfunked, yes I have a couple 10:04:36 ok, file issues, let's prioritise them 10:04:37 the jobs list API too 10:05:32 but I think there is a issue which says we need some kind of tagging? 10:05:35 there was a specification for job updates 10:05:58 yes there is 10:06:07 https://github.com/Tendrl/specifications/issues/55 10:06:14 that needs to be prioritised 10:06:20 which would impact the API as well 10:06:34 I'll add it in the priorities for now, we can discuss this one tomorrow 10:06:52 brainfunked, ok 10:07:06 ok then 10:07:46 #action noopq to file issues for pending API work, evaluate existing issues, help prioritise the API specs and start work on them 10:08:01 #topic ndarshan's updates 10:08:14 sorry, jumped ahead and skipped him over 10:09:42 -----Github specification issue title and link----- 10:09:43 Introduce flows tied to Tendrl objects(https://github.com/Tendrl/specifications/issues/34) 10:09:43 Specification for adding tendrl-common utils (https://github.com/Tendrl/specifications/issues/72) 10:09:43 -----Issues Fixed----- 10:09:43 https://github.com/Tendrl/node_agent/issues/83 (this is a bug so no spec for this) 10:09:43 -----Summary of the update across the whole specification---- 10:09:45 Introduce flows tied to Tendrl objects (https://github.com/Tendrl/specifications/pull/10) --> spec got merged today. will start work on coding 10:09:48 Specification for adding tendrl-common utils (https://github.com/Tendrl/specifications/pull/69) --> Sent spec, was reviewed and merged. Will work on the code 10:09:51 Posted a PR for node_agent issue #83 : https://github.com/Tendrl/node_agent/issues/104 --> sent PR, merged and issue closed 10:09:54 -----Blockers----- 10:09:56 None 10:09:58 -----My Reviews----- 10:10:00 https://github.com/Tendrl/gluster_integration/pull/63#pullrequestreview-12917007 10:12:02 #action brainfunked and nishanth need to review "common utils" spec https://github.com/Tendrl/tendrl-api/issues/39 10:12:29 ndarshan, what will you be picking up now? 10:13:13 I am working on the implementation(code) of the 2 specs mentioned above 10:13:24 ndarshan, ack, thanks 10:13:55 brainfunked, I will be on PTO from 19th to 26th. Have marked in tendrl calendar. 10:14:05 #accept ndarshan's specifications regarding object specific flows and common utils have been merged, he's working on the implementation 10:14:20 #info ndarshan will be on PTO from 19th to 26th December 10:14:43 #info anmol will be on PTO from 20th to 24th December 10:15:03 #topic Shu6h3ndu's updates 10:15:12 brainfunked, sure 10:15:21 Issue: https://github.com/Tendrl/specifications/issues/30 (Accomodate gluster get-state cli changes post gluster 3.9) 10:15:21 Spec: https://github.com/Tendrl/specifications/pull/6 (Added specification for refactoring gluster get-state output) 10:15:21 Progress: code change PR under review 10:15:22 Status: Rohan, Darshan, Nishanth to review the code 10:15:22 Issue: https://github.com/Tendrl/specifications/issues/31 (Refactor Tendrl framework sub modules) 10:15:24 Spec: https://github.com/Tendrl/specifications/pull/7 (Added specifications for refactoring tendrl code) 10:15:27 Progress: code change PR under review. Working on test failures and review comments 10:15:29 Status: NOT BLOCKED 10:15:33 Issue: https://github.com/Tendrl/gluster_integration/issues/74 (Implement the pre and post runs for volume operations) 10:15:36 Spec: NA (OLD ONE) 10:15:38 Progress: Review in progress 10:15:40 Status: BLOCKED ON REVIEWS 10:15:42 Issue: https://github.com/Tendrl/gluster_integration/issues/73 (Delete the volume entry from etcd as well while volume delete) 10:15:49 Spec: NA (OLD ONE) 10:15:51 Progress: Review in progress 10:15:53 Status: NOT BLOCKED 10:15:55 Finished spec reviews today 10:15:57 For https://github.com/Tendrl/specifications/issues/31 I will create individual issues for code coverage and attach with specs 10:16:46 overall working on test case failures post re-factor for common and node agent module 10:16:59 post this I would pick gluster and ceph integration for re-factor 10:17:25 Shu6h3ndu, is the refactoring still on track to be done by Monday? 10:18:01 brainfunked, hope. so if nothing major comes while solving test case failures. worst case another day or so might be required 10:18:19 Shu6h3ndu, ok 10:18:22 do we anticipate any major failures from test case? 10:18:28 sorry - had to ask 10:18:59 not major ones but coverage is pretty low after last refactor and would create other issues to solve them separately 10:19:20 ok. fair enough - thanks 10:19:21 refactoring with working code I am trying first :) 10:19:31 ack 10:21:16 #accept Shu6h3ndu working on refactoring and gluster get-state updates. Refactoring currently looks to be on track by Monday, barring any major revelations. 10:21:24 #topic t1m1's updates 10:21:35 Yesterday I sent a patch for a bug: https://github.com/Tendrl/gluster_integration/issues/87 10:21:35 Reviewed: https://github.com/Tendrl/tendrl_frontend/pull/59 10:21:35 Working on updating existing patches for bugs posted by mbukatov and 10:21:35 Started working on specification: https://github.com/Tendrl/specifications/issues/46 10:22:36 t1m1, which bugs? 10:23:27 1) https://github.com/Tendrl/node_agent/issues/97 10:23:46 2) https://github.com/Tendrl/node_agent/issues/89 10:24:03 3) https://github.com/Tendrl/node_agent/issues/88 10:25:02 ok, thanks 10:25:15 brainfunked, no prob; 10:25:23 #action t1m1 working on the "service detection modules" spec https://github.com/Tendrl/specifications/issues/46 10:25:35 #undo 10:25:35 Removing item from minutes: ACTION by brainfunked at 10:25:23 : t1m1 working on the "service detection modules" spec https://github.com/Tendrl/specifications/issues/46 10:25:47 #action t1m1 to send a pull request for the "service detection modules" spec https://github.com/Tendrl/specifications/issues/46 10:26:27 #accept t1m1 pushing fixes against issues raised by mbukatov, https://github.com/Tendrl/node_agent/issues/97 https://github.com/Tendrl/node_agent/issues/89 https://github.com/Tendrl/node_agent/issues/88. 10:27:03 #action some of the fixes t1m1 is pushing require to be turned into pull requests on the tendrl repositories 10:27:33 thanks for the updates everyone 10:27:36 #endmeeting