09:02:38 <brainfunked> #startmeeting check-in_20161214 09:02:38 <zodbot> Meeting started Wed Dec 14 09:02:38 2016 UTC. The chair is brainfunked. Information about MeetBot at http://wiki.debian.org/MeetBot. 09:02:38 <zodbot> Useful Commands: #action #agreed #halp #info #idea #link #topic. 09:02:38 <zodbot> The meeting name has been set to 'check-in_20161214' 09:03:12 <brainfunked> #info First time using this meeting bot, so may be a little slow in driving the meeting today. 09:03:20 <brainfunked> #topic anmol's updates 09:03:25 <anmol> Spec Progress: 09:03:26 <anmol> - Issue: https://github.com/Tendrl/specifications/issues/42 - Add a monitoring provisioning flow on node agent 09:03:26 <anmol> Spec-PR: 09:03:26 <anmol> https://github.com/Tendrl/specifications/pull/20 -- Add a monitoring provisioning flow 09:03:26 <anmol> Progress : Almost approved 09:03:27 <anmol> @nthomas-redhat('Looks good to me. I will approve if the APIs are reviewed by @anivargi') 09:03:29 <anmol> @shtripat('Looks good to me. Anup to review api changes') 09:03:31 <anmol> @anivargi 09:03:33 <anmol> Status: Awaiting reviews from @brainfunked @r0h4n 09:03:35 <anmol> - Issue: https://github.com/Tendrl/specifications/issues/40 - Pluggability of different supported alert notifying means 09:03:38 <anmol> Spec PR: 09:03:40 <anmol> https://github.com/Tendrl/specifications/pull/18 -- Pluggability of different supported alert notifying means 09:03:43 <anmol> Progress: 09:03:45 <anmol> Created and Linked issues in spec's "Work Items" section 09:03:47 <anmol> Approved by: 09:03:49 <anmol> @nthomas-redhat('Looks good to me. Please get the APIs reviewed by @anivargi') 09:03:51 <anmol> @r0h4n 09:03:55 <anmol> @shtripat('Looks good to me as long as all the missing points added') 09:03:57 <anmol> Status: Awaiting reviews from @anivargi @brainfunked @r0h4n and @shtripat. 09:03:59 <anmol> - Issue: https://github.com/Tendrl/specifications/issues/62 -- Provide a way to expose the time series data in time-series db to the Tendrl/tendrl-api 09:04:02 <anmol> Spec PR: 09:04:04 <anmol> Created https://github.com/Tendrl/specifications/pull/63 -- Specification for exposing apis to access time-series data 09:04:07 <anmol> Progress: 09:04:09 <anmol> Spec PR raised with intial thoughts. Need to raise issues on impacted components and link back in spec. 09:04:12 <anmol> Status: Incoporating comments from Shubhendu. Awaiting tendrl-api impact from @anivargi 09:04:14 <anmol> Progress on github components : 09:04:16 <anmol> - https://github.com/Tendrl/node_agent/issues/70 -- Expose a way for node_agent to receive events and push the same to etcd and add generic atom to check service status 09:04:19 <anmol> -- Spec Details: 09:04:21 <anmol> Issue: https://github.com/Tendrl/specifications/issues/40 - Pluggability of different supported alert notifying means 09:04:26 <anmol> PR: https://github.com/Tendrl/specifications/pull/18 -- Pluggability of different supported alert notifying means 09:04:29 <anmol> -- Progress : 09:04:31 <anmol> PR : https://github.com/Tendrl/node_agent/pull/67 09:04:33 <anmol> 09:04:45 <anmol> looks like my update is not coming completely here 09:04:50 <anmol> so please find it in https://paste.fedoraproject.org/506075/17062701/ 09:05:00 <brainfunked> ok, let me go through it 09:07:20 <brainfunked> #action brainfunked and k4n0 to review https://github.com/Tendrl/specifications/issues/42 https://github.com/Tendrl/specifications/issues/40 09:07:53 <brainfunked> #action noopq shubhendu to review https://github.com/Tendrl/specifications/pull/18 09:08:17 <noopq> brainfunked, I just reviewed it 09:08:28 <brainfunked> alright 09:09:03 <shubhendu> brainfunked, Looks good to me https://github.com/Tendrl/specifications/pull/18 09:09:20 <brainfunked> #undo 09:09:20 <zodbot> Removing item from minutes: ACTION by brainfunked at 09:07:53 : noopq shubhendu to review https://github.com/Tendrl/specifications/pull/18 09:09:34 <brainfunked> #info noopq and shubhendu have reviewed https://github.com/Tendrl/specifications/pull/18 09:10:27 <brainfunked> #action brainfunked, k4n0, nishanth to review https://github.com/Tendrl/node_agent/pull/67 09:11:07 <brainfunked> anmol, should I finish my reviews tomorrow, would it block your work? 09:11:54 <anmol> brainfunked, I have to work on 2 specs yet 1 WIP raised already and other monitoring doc I have to start on it 09:12:13 <anmol> brainfunked, also I can start implementing the alerting part so not blocked as such 09:12:24 <brainfunked> anmol, alright, then I'll do the reviews tomorrow 09:12:30 <nishanth> brainfunked, I will try to finish the review tomorrow, but more priority to the specs 09:12:31 <brainfunked> noopq, have you reviewed https://github.com/Tendrl/performance_monitoring/pull/2 09:12:59 <noopq> brainfunked, I will do it today 09:13:28 <brainfunked> #accept brainfunked and nishanth to complete the reviews on anmol's specs tomorrow 09:13:54 <brainfunked> anmol, how does this #2 pr relate to the api spec? 09:14:07 <brainfunked> anmol, or they don't because this is for monitoring and the other is for alerting? 09:14:42 <anmol> brainfunked, https://github.com/Tendrl/performance_monitoring/pull/2 adds apis also 09:14:55 <anmol> brainfunked, apis to access time series data 09:15:48 <brainfunked> anmol, so this is a PR against https://github.com/Tendrl/specifications/issues/62 right? 09:16:57 <anmol> brainfunked, the spec PR is for the issue that you specified 09:17:11 <anmol> the issue you specified is a issue on spec repo 09:17:26 <brainfunked> anmol, yes, but isn't the monitoring codebase pr the implementation of this spec? 09:17:52 <anmol> yes it is 09:18:00 <anmol> it is an implementation of 2 specs 09:18:12 <brainfunked> anmol, please make a comment on that code pr and link it to the issue on the specs repo, #62 09:18:18 <anmol> ok 09:18:25 <brainfunked> ok, looks good 09:18:40 <brainfunked> I think the dependency is mostly on us doing the reviews and finalising the specs 09:18:52 <anmol> the code repo links to spec issue or spec? 09:19:15 <brainfunked> #action brainfunked shubhendu nishanth k4n0 and noopq to prioritise reviewing the specs by anmol so that they can be merged tomorrow 09:19:18 <anmol> I thought it was spec linking to spec issue and code pr linking to component issue and spec 09:19:24 <brainfunked> anmol, link to the issue on the spec repo 09:19:28 <anmol> ok 09:19:40 <shubhendu> brainfunked, ack 09:20:16 <brainfunked> #info the issues on the specifications repository are the single origin point against which all the pull requests and issues need to be linked 09:20:46 <brainfunked> #accept anmol's update indicates that the alerting and monitoring work is on track 09:20:58 <brainfunked> #topic dahorak's updates 09:21:13 <dahorak> - Issue: https://github.com/Tendrl/specifications/issues/53 -- CentOS CI Pipeline 09:21:13 <dahorak> Spec-PR: https://github.com/Tendrl/specifications/pull/61 -- Specification: Centos CI utilization 09:21:13 <dahorak> Progress: First proposal done, few notes/proposals raised, working on them. 09:21:13 <dahorak> Otherwise working on different (Tendrl unrelated) tasks (from yesterday till probably tomorrow). 09:22:57 <brainfunked> #info shubhendu has already made some comments on https://github.com/Tendrl/specifications/pull/61, mbukatov has made comments on https://github.com/Tendrl/specifications/issues/53 09:23:13 <brainfunked> #action brainfunked and nishanth to review https://github.com/Tendrl/specifications/pull/61 and respond to https://github.com/Tendrl/specifications/issues/53 09:23:40 <brainfunked> #accept dahorak's initial commit regarding the centos ci implementation 09:23:44 <brainfunked> thanks, moving on 09:23:50 <brainfunked> #topic fbalak's updates 09:25:47 <fbalak> - Issue: https://github.com/Tendrl/usmqe-tests/pull/6 09:27:06 <brainfunked> fbalak, title, progress and status please? 09:27:48 <brainfunked> also, has noopq incorporated the changes that are preventing https://github.com/Tendrl/tendrl-api/issues/23 and https://github.com/Tendrl/documentation/pull/62 from being merged? 09:27:53 <brainfunked> fbalak, ^ 09:28:46 <fbalak> Issue: Merging pull request for Tendrl-tests 09:29:02 <noopq> brainfunked, I have already updated the API doc 09:29:24 <noopq> and the doc is upto date as to what was available in 1.1 09:29:30 <fbalak> Progress: Working on it and reviewing https://github.com/Tendrl/documentation/pull/62 (still need to check ceph part of the api) 09:29:46 <fbalak> Status: good 09:30:24 <brainfunked> fbalak, can we get a spec for https://github.com/Tendrl/usmqe-tests/pull/6? 09:30:41 <mbukatov> brainfunked: no 09:30:54 <brainfunked> mbukatov, explanation? 09:31:17 <mbukatov> brainfunked: I mean to say that we are not going to provide a specification file for changes in usmqe repositories 09:31:40 <mbukatov> brainfunked: unless that change requires cooperation with other tenrl people 09:33:05 <brainfunked> #action mbukatov: "we are not going to provide a specification file for changes in usmqe repositories, unless that change requires cooperation with other tenrl people". This needs to be followed up on in a separate conversation. 09:33:30 <brainfunked> #action fbalak to finish reviewing https://github.com/Tendrl/documentation/pull/62 09:34:16 <brainfunked> thanks for the update 09:34:21 <brainfunked> #topic gowtham's updates 09:34:33 <gowtham> Adding disk details in hardware inventory 09:34:39 <gowtham> git-issue: https://github.com/Tendrl/node_agent/issues/78 09:34:49 <gowtham> spec file: https://github.com/Tendrl/specifications/pull/21 09:34:55 <gowtham> As per darshan comment i have modified spec file and implementation done 09:34:55 <gowtham> i am just testing 09:35:05 <gowtham> Adding Network details in hardware inventory 09:35:05 <gowtham> git-issue: https://github.com/Tendrl/node_agent/issues/78 09:35:06 <gowtham> spc file: https://github.com/Tendrl/specifications/pull/19 09:35:14 <gowtham> PR under review 09:35:22 <gowtham> Separate out per-component configuration files 09:35:30 <gowtham> git-issue: 09:35:30 <gowtham> https://github.com/Tendrl/gluster_integration/issues/81 09:35:31 <gowtham> https://github.com/Tendrl/ceph_integration/issues/54 09:35:31 <gowtham> https://github.com/Tendrl/node_agent/issues/91 09:35:31 <gowtham> https://github.com/Tendrl/common/issues/76 09:35:31 <gowtham> https://github.com/Tendrl/performance_monitoring/issues/11 09:35:33 <gowtham> https://github.com/Tendrl/alerting/issues/7 09:35:38 <gowtham> spec file: https://github.com/Tendrl/specifications/pull/11 09:35:52 <gowtham> updated spec file and Started to implement 09:35:59 <gowtham> Logged EtcdKeyNotFound exception 09:36:07 <gowtham> git-issue: 09:36:07 <gowtham> https://github.com/Tendrl/node_agent/issues/94 09:36:07 <gowtham> https://github.com/Tendrl/ceph_integration/issues/14 09:36:07 <gowtham> https://github.com/Tendrl/gluster_integration/issues/26 09:36:14 <gowtham> PR: 09:36:14 <gowtham> https://github.com/Tendrl/node_agent/pull/95 09:36:14 <gowtham> https://github.com/Tendrl/ceph_integration/pull/51 09:36:14 <gowtham> https://github.com/Tendrl/gluster_integration/pull/78 09:36:18 <brainfunked> gowtham, implementation start on what? 09:37:13 <gowtham> i have stared to implement a task "separating configuration per module wise" 09:37:49 <gowtham> i have completely understand how to implement it and i have updated the spec file 09:38:03 <gowtham> Travis commit message check is modified 09:38:03 <gowtham> git-issue: 09:38:03 <gowtham> https://github.com/Tendrl/alerting/issues/6 09:38:04 <gowtham> https://github.com/Tendrl/performance_monitoring/issues/10 09:38:04 <gowtham> PR : 09:38:04 <gowtham> https://github.com/Tendrl/alerting/pull/8 09:38:06 <gowtham> https://github.com/Tendrl/performance_monitoring/pull/12 09:38:14 <gowtham> Added base test file for alerting and performance monitoring 09:38:25 <gowtham> status : good 09:38:25 <gowtham> I am not blocked 09:39:20 <gowtham> can you please review the config task spec file 09:39:34 <gowtham> i have to know my understanding is correct or not 09:39:48 <brainfunked> gowtham, as I'd mentioned in the both of the earlier check-ins this week, the configuration specification isn't high priority. The specification was to be finished but not implemented yet. 09:40:27 <gowtham> ya i am mainly working on hardware inventory 09:40:42 <gowtham> i am just modified spec file thats all 09:40:53 <brainfunked> gowtham, ok, I thought implementation meant code 09:41:07 <gowtham> ok 09:42:14 <brainfunked> does https://github.com/Tendrl/alerting/issues/6 affect only the alerting repository? gowtham, anmol 09:42:49 <gowtham> no i sent patch for all the repo 09:43:02 <gowtham> it affect all the repo 09:43:05 <brainfunked> ok 09:43:20 <brainfunked> you've only linked to the issues/prs specifc to the two repos 09:43:26 <brainfunked> has this been merged in other repos? 09:43:56 <gowtham> yes it merged 09:43:58 <brainfunked> ok 09:44:11 <gowtham> why it not merged in alerting means 09:44:27 <gowtham> travis required base test file 09:44:39 <gowtham> i have updated the patch with base test file also 09:44:59 <gowtham> now it ready to merge 09:45:10 <brainfunked> #accept gowtham to continue the progress on the hardware inventory specifications: https://github.com/Tendrl/specifications/pull/21 https://github.com/Tendrl/specifications/pull/19 09:45:16 <brainfunked> gowtham, ack, thanks 09:45:22 <brainfunked> gowtham, blocked on anything or anyone? 09:45:28 <gowtham> no i am not blocked 09:45:35 <brainfunked> cool, moving on then 09:45:44 <brainfunked> #topic k4n0's updates 09:46:00 <k4n0> Issue: https://github.com/Tendrl/specifications/issues/32 09:46:16 <k4n0> Spec-PR: https://github.com/Tendrl/specifications/pull/8 09:46:41 <k4n0> Update: Need to freeze the spec and finish implementation for all Tendrl components 09:46:48 <k4n0> Status: Not blocked 09:47:23 <k4n0> I have been reviewing specs on a higher priority to be able to provide better time estimates on each spec 09:47:47 <k4n0> Will finish doing those reviews today, and continue to my earlier priority 09:47:57 <k4n0> questions? 09:48:37 <brainfunked> the specs #35 to #39 all show up as in progress for the specification 09:48:45 <brainfunked> are they on track? there seem to be no commits yet 09:49:08 <brainfunked> k4n0, ^ 09:49:17 <k4n0> They will be done till Friday, yes on track 09:49:28 <brainfunked> alright 09:50:13 <brainfunked> #action k4n0 to finish reviewing others' in progress specifications and then continue on #32, #35-#39 09:50:45 <brainfunked> #accept k4n0's working on reviews and specifications that affect the core framework functionality 09:50:50 <brainfunked> thanks for the update 09:51:03 <brainfunked> #topic kaverma and neha__'s updates 09:51:08 <kaverma> Title: import cluster workflow using UI 09:51:17 <kaverma> Spec-PR: https://github.com/Tendrl/specifications/pull/64 09:51:28 <kaverma> Spec-Issue: https://github.com/Tendrl/specifications/issues/56 (also added Linked for Jira Tickets ) 09:51:36 <kaverma> Github-PR: https://github.com/Tendrl/tendrl_frontend/pull/60 (under review) 09:51:43 <kaverma> Github-Issue: https://github.com/Tendrl/tendrl_frontend/issues/63 09:51:51 <kaverma> Progress: almost done except Table component integration .neha working on it (Table component creation - https://github.com/Tendrl/tendrl_frontend/issues/65). 09:51:51 <kaverma> and also done with server setup for import cluster workflow demo 09:51:57 <kaverma> Status: good . not blocked. 09:52:00 <kaverma> End 09:52:11 <neha__> Issue: https://github.com/Tendrl/tendrl_frontend/issues/65 09:52:12 <neha__> Specifications: https://github.com/Tendrl/specifications/issues/65 09:52:12 <neha__> Status: Started 09:52:35 <neha__> Issue : https://github.com/Tendrl/tendrl_frontend/issues/64 09:52:35 <neha__> Spec: https://github.com/Tendrl/specifications/issues/56 09:52:35 <neha__> Status: In progress. Need Kamlesh help. 09:52:57 <neha__> End 09:53:28 <brainfunked> there's no specification for #65, and yet seems like the implemenation is almost working? kaverma neha__ 09:53:53 <brainfunked> in addition, I've checked the import cluster UI spec, #64, it doesn't have any details whatsoever 09:54:28 <neha__> I working on specification for #65 09:54:33 <kaverma> brainfunked, i will update the pr #64. 09:55:25 <brainfunked> I have a feeling that you need to collaborate with nishanth on the import cluster specification 09:55:37 <brainfunked> it goes hand in hand with the backend implementation spec he's working on for the same 09:56:04 <kaverma> brainfunked, yes . i will sink with him and update the pr accordingly 09:56:11 <kaverma> *sync 09:56:28 <brainfunked> nishanth, do you agree? 09:56:57 <nishanth> brainfunked, ack, I will work with them 09:57:07 <brainfunked> alright, thanks 09:57:32 <brainfunked> #action neha__ and kaverma to stop implementation and finish the specifications in detail on priority. kaverma to work with nishanth on the import cluster specification. 09:58:01 <brainfunked> #topic ltrilety's updates 09:59:59 <ltrilety> no update, actually I am on sickness leave for now 10:00:39 <brainfunked> ltrilety, alright, thanks 10:00:57 <brainfunked> #info ltrilety has been on sick leave, hence no current updates 10:01:08 <brainfunked> #topic mbukatov's updates 10:01:59 <mbukatov> I'm still working on review/update of ansible code for tendrl-api which is part of https://github.com/Tendrl/usmqe-setup/pull/6 10:02:24 <mbukatov> this takes me longer than expected, because I had no prior experience with ruby tools 10:03:09 <mbukatov> besides that, I discussed CentOS CI with dahorak, provided some suggestion in https://github.com/dahorak/specifications/pull/1 10:04:21 <mbukatov> that's all 10:04:21 <brainfunked> #accept mbukatov working on review/update of the ansible code for tendrl api and contributing to the centos ci specification. 10:04:24 <brainfunked> thanks 10:04:33 <brainfunked> #topic ndarshan's updates 10:05:13 <ndarshan> -----Github specification issue title and link----- 10:05:13 <ndarshan> Refactor Tendrl framework sub modules(https://github.com/Tendrl/specifications/issues/31), 10:05:13 <ndarshan> Introduce flows tied to Tendrl objects(https://github.com/Tendrl/specifications/issues/34) 10:05:13 <ndarshan> -----Issues linked to this specification from component repositories----- 10:05:13 <ndarshan> https://github.com/Tendrl/specifications/issues/34 10:05:14 <ndarshan> https://github.com/Tendrl/common/issues/80 10:05:15 <ndarshan> https://github.com/Tendrl/node_agent/issues/83 (this is a bug so no spec for this) 10:05:17 <ndarshan> -----Summary of the update across the whole specification---- 10:05:19 <ndarshan> Received review comments for the above 2 items addressed the comments, will address further review suggestions If any. 10:05:22 <ndarshan> Will post a PR for node_agent issue #83(mentioned above) by evening currently WIP 10:05:24 <ndarshan> -----Minor Blockers----- 10:05:26 <ndarshan> https://github.com/Tendrl/common/pull/79 ---> shubhendu has reviewed, waiting for others 10:05:28 <ndarshan> https://github.com/Tendrl/specifications/pull/10 ---> mrugesh please review this, others have reviewed. If approved will start on implementation 10:05:31 <ndarshan> -----My Reviews----- 10:05:35 <ndarshan> https://github.com/Tendrl/node_agent/pull/100#pullrequestreview-12648978 10:05:37 <ndarshan> https://github.com/Tendrl/specifications/pull/19 10:06:49 <brainfunked> ndarshan, shubhendu, do you have any ETA for the refactoring? 10:07:10 <brainfunked> #action brainfunked to review https://github.com/Tendrl/specifications/pull/10 on priority 10:08:13 <shubhendu> brainfunked, common and node_agent changes are reviewed and comments incorporated already. I am working on test cases changes. Hopefully by friday or early next week can be closed for ceph and gluster integration as well 10:08:19 <brainfunked> ndarshan, https://github.com/Tendrl/common/pull/79 really needs a specification to say what kind of utilities are required and how they'll be used 10:08:33 <brainfunked> shubhendu, ack, thanks 10:08:40 <shubhendu> brainfunked, would request anmol to do the same changes later for alerting and monitoring as the projects are not yet merged 10:09:06 <anmol> shubhendu, ok 10:09:29 <ndarshan> brainfunked, Okay will work on adding specification 10:09:31 <brainfunked> #info the refactoring is showing progress. common and node_agent changes have been reviewed and merged. ETA for ceph and gluster integration changes is Monday. 10:09:56 <shubhendu> brainfunked, its merged for common and node agent. only spec PRs merged 10:10:21 <brainfunked> #action ndarshan to add a specification describing the nature and use of the utilites being added as part of https://github.com/Tendrl/common/pull/79. This specification needs to be prioritised for review and merge before the actuall pull request is approved. 10:10:37 <shubhendu> brainfunked, code PRs still under review and Monday is something I am targetting 10:10:43 <brainfunked> shubhendu, oh ok 10:11:12 <brainfunked> #info correction on the earlier refactoring status. Specifications have been merged, code is being worked upon. ETA is still Monday. 10:11:38 <brainfunked> ndarshan, please take up the spec on priority, let's have it merged tomorrow 10:11:41 <shubhendu> brainfunked, hopefully would be able to do it :) 10:11:56 <brainfunked> shubhendu, "Estimated" :) 10:12:09 <ndarshan> brainfunked, will post Spec before EOD 10:12:37 <brainfunked> ndarshan, thanks, please tag people who need to review it 10:13:08 <brainfunked> ndarshan, I can't find the spec for https://github.com/Tendrl/specifications/issues/34 either 10:14:36 <ndarshan> brainfunked, thats the spec I wanted you to review(you just added an AI on yourself to review https://github.com/Tendrl/specifications/pull/10) 10:14:50 <brainfunked> ok, I'll link it up 10:15:30 <brainfunked> ndarshan, I could find the commits, but not the PR. Added a comment to explicitly link the PR. 10:15:41 <brainfunked> alright, thanks for your update 10:16:41 <brainfunked> #accept ndarshan to add specification for the common utilities and has made progress on the refactoring and object specific flows. 10:17:32 <brainfunked> shubhendu, the changes you need anmol to make, have they been spec'd out so that he can implement them directly? 10:18:02 <brainfunked> and I'm going to jump ahead and come back, since shubhendu has already been involved in the conversation 10:18:06 <brainfunked> #topic shubhendu's updates 10:18:10 <shubhendu> brainfunked, its part of my spec only and issues are created for specific projects already 10:18:17 <shubhendu> brainfunked, will go with updates now 10:18:21 <brainfunked> ok 10:18:30 <shubhendu> Issue: https://github.com/Tendrl/specifications/issues/30 (Accomodate gluster get-state cli changes post gluster 3.9) 10:18:30 <shubhendu> Spec: https://github.com/Tendrl/specifications/pull/6 (Added specification for refactoring gluster get-state output) 10:18:30 <shubhendu> Progress: SPEC PR MERGED. code change PR under review 10:18:30 <shubhendu> Status: NOT BLOCKED 10:18:30 <shubhendu> Issue: https://github.com/Tendrl/specifications/issues/31 (Refactor Tendrl framework sub modules) 10:18:32 <shubhendu> Spec: https://github.com/Tendrl/specifications/pull/7 (Added specifications for refactoring tendrl code) 10:18:35 <shubhendu> Progress: SPEC PR MERGED. code change PR under review. Working on test cases 10:18:39 <shubhendu> Status: NOT BLOCKED 10:18:41 <shubhendu> Issue: https://github.com/Tendrl/gluster_integration/issues/74 (Implement the pre and post runs for volume operations) 10:18:44 <shubhendu> Spec: NA (OLD ONE) 10:18:48 <shubhendu> Progress: Review in progress 10:18:50 <shubhendu> Status: BLOCKED ON REVIEWS 10:18:54 <shubhendu> Issue: https://github.com/Tendrl/gluster_integration/issues/73 (Delete the volume entry from etcd as well while volume delete) 10:18:57 <shubhendu> Spec: NA (OLD ONE) 10:18:59 <shubhendu> Progress: Review in progress 10:19:01 <shubhendu> Status: BLOCKED ON REVIEWS 10:19:03 <shubhendu> Reviewing spec PRs on priority. 10:19:05 <shubhendu> I have removed individual issues references from above and retained only spec issue link ^^^ 10:20:14 <brainfunked> :) 10:20:26 <brainfunked> blocked on reviews from? 10:20:26 <shubhendu> brainfunked, looks crisp now ;) 10:20:45 <shubhendu> brainfunked, two old ones rohan and other have to review 10:21:01 <shubhendu> brainfunked, they are more of bug fixes and issues are linked with them 10:21:22 <brainfunked> ok 10:21:42 <brainfunked> rohan and who else? I'll add an action item 10:22:16 <shubhendu> brainfunked, may be darshan and nishanth I would say 10:22:21 <brainfunked> ok 10:22:30 <shubhendu> brainfunked, these are smaller fixes 10:22:31 <shubhendu> :) 10:22:42 <brainfunked> #action k4n0, nishanth and ndarshan to review https://github.com/Tendrl/gluster_integration/issues/74 https://github.com/Tendrl/gluster_integration/issues/73 10:23:17 <brainfunked> alright, so in general, I guess the above mentioned progress on the refactoring is the major update for you, along wih the merged specs 10:23:30 <shubhendu> brainfunked, thats correct 10:23:34 <brainfunked> awesome, thanks 10:24:51 <brainfunked> #accept shubhendu has had most of the important blockers from yesterday (reviews) resolved and specifications have been merged for refactoring and gluster get-state changes. He's now implementing these specifications. 10:25:10 <brainfunked> #topic nishanth's updates 10:25:12 <shubhendu> brainfunked, cool 10:25:31 <nishanth> Issue: https://github.com/Tendrl/specifications/issues/12 10:25:31 <nishanth> Spec-PR:https://github.com/Tendrl/specifications/pull/24 10:25:31 <nishanth> Progress and status: Merged the spec 10:25:31 <nishanth> Issue: https://github.com/Tendrl/specifications/issues/54 10:25:32 <nishanth> Spec-PR: https://github.com/Tendrl/specifications/pull/60 10:25:32 <nishanth> Progress and status: Spec updated. Few more things to be added 10:25:43 <nishanth> Reviews 10:25:43 <nishanth> - https://github.com/Tendrl/specifications/pull/21 10:25:43 <nishanth> - https://github.com/Tendrl/specifications/pull/18 10:25:43 <nishanth> - https://github.com/Tendrl/specifications/pull/20 10:25:44 <nishanth> - https://github.com/Tendrl/specifications/pull/6 10:25:46 <nishanth> - https://github.com/Tendrl/specifications/pull/7 10:25:48 <nishanth> - https://github.com/Tendrl/specifications/pull/10 10:26:26 <brainfunked> awesome, thanks 10:26:35 <brainfunked> #action brainfunked to review https://github.com/Tendrl/specifications/issues/54 10:26:52 <brainfunked> regarding the repository name changes, when do we execute them? 10:27:20 <nishanth> probably after the refactoring patch is in 10:27:41 <brainfunked> nishanth, have you pushed the name changes spec btw? 10:27:54 <brainfunked> the linked commit from your branch shows up, but not the pull request 10:28:22 <nishanth> I think the pull request is merged, let me check .. 10:28:43 <brainfunked> it is 10:28:55 <nishanth> yeah 10:29:10 <brainfunked> alright, thanks 10:29:50 <brainfunked> ok, so today will you be prioritising the reviews on the specifications? 10:30:30 <nishanth> yep 10:30:33 <brainfunked> ack 10:31:07 <brainfunked> #accept nishanth has updated the import cluster specification and will be targeting reviews for other specifications so that they can be merged 10:31:27 <brainfunked> #info noopq_away is away at the moment, so we'll need to circle back to him 10:31:39 <brainfunked> #topic t1m1's updates 10:31:42 <t1m1> I have sent spec patch for https://github.com/Tendrl/specifications/issues/57 in https://github.com/Tendrl/specifications/issues/66 10:31:56 <t1m1> just now 10:32:36 <brainfunked> whom do you want this to be reviewed by 10:33:06 <t1m1> k4n0 10:33:14 <brainfunked> ok 10:33:27 <brainfunked> #action k4n0 to review https://github.com/Tendrl/specifications/pull/66 10:33:39 <t1m1> and shubhendu / nishanth 10:33:56 <brainfunked> #action shubhendu and nishanth to review https://github.com/Tendrl/specifications/issues/66 10:34:09 <shubhendu> brainfunked, ack 10:34:10 <brainfunked> t1m1, what will you be working on while this is being reviewed? 10:35:26 <t1m1> i want to take one more bug from tendrl-api : https://github.com/Tendrl/tendrl-api/issues/37 (unification of etcd.yml) or 10:36:12 <brainfunked> t1m1, what about https://github.com/Tendrl/node_agent/issues/97 and https://github.com/Tendrl/node_agent/issues/99, which you said you were working on yesterday/ 10:36:23 <t1m1> yes i am working on that bugs also 10:37:57 <t1m1> I will pick up new bugs once its completed in the mean time 10:38:18 <brainfunked> t1m1, https://github.com/Tendrl/tendrl-api/issues/37 is for noopq_away to look at 10:38:22 <t1m1> also i am willing to take 10:38:33 <t1m1> https://github.com/Tendrl/specifications/issues/46 10:40:29 <brainfunked> you'll need to collaborate with nishanth on https://github.com/Tendrl/specifications/issues/46 10:40:42 <t1m1> ok 10:41:22 <brainfunked> t1m1, but before that, would you mind addressing the bugs mbukatov had created, excluding https://github.com/Tendrl/tendrl-api/issues/37 10:41:55 <brainfunked> t1m1, the list of bugs is in the summary of yesterday's check-in I had sent to the devel list 10:42:36 <t1m1> sure, i will go thur it and pick up free bugs 10:43:08 <brainfunked> thanks 10:43:35 <brainfunked> #action t1m1 to work on bugs created by mbukatov, listed in yesterday's summary for the check-in on the mailing list 10:43:57 <brainfunked> #accept t1m1 has committed a specification for validation, which needs to be reviewed 10:44:22 <brainfunked> #info noopq_away is still away, and we have no updates from him 10:44:29 <brainfunked> thanks everyone for the updates 10:44:31 <brainfunked> #endmeeting