09:04:23 <brainfunked> #startmeeting check-in_20161221
09:04:23 <zodbot> Meeting started Wed Dec 21 09:04:23 2016 UTC.  The chair is brainfunked. Information about MeetBot at http://wiki.debian.org/MeetBot.
09:04:23 <zodbot> Useful Commands: #action #agreed #halp #info #idea #link #topic.
09:04:23 <zodbot> The meeting name has been set to 'check-in_20161221'
09:04:37 <brainfunked> #topic list views: neha__, kaverma and noopq
09:05:09 <noopq> brainfunked, I am done working on the list views for hosts and clusters, I will commit the code shortly and update the list view spec
09:05:40 <noopq> the spec will have examples for the API response which can be used by neha__ and kaverma to build the UI
09:06:10 <brainfunked> #info noopq will be on leave tomorrow. He'll be unavailble starting today evening, till Friday afternoon. kaverma and neha__ need to ensure that they have all the necessary information from him to be able to continue with the work on the list views in his absence.
09:06:39 <brainfunked> #undo
09:06:39 <zodbot> Removing item from minutes: INFO by brainfunked at 09:06:10 : noopq will be on leave tomorrow. He'll be unavailble starting today evening, till Friday afternoon. kaverma and neha__ need to ensure that they have all the necessary information from him to be able to continue with the work on the list views in his absence.
09:06:51 <brainfunked> #info noopq will be on leave tomorrow. He'll be unavailble starting today evening, till Friday afternoon.
09:07:01 <neha__> I have updated the PR for all specs. I have started with the cluster list implementation with mock json..will update the response once received from ANup
09:07:22 <brainfunked> #action neha__ and kaverma need to ensure that they have all the necessary information from noopq to be able to continue with the work on the list views in his absence over the next couple of days.
09:07:51 <brainfunked> neha__, are there any concerns or blockers?
09:08:00 <neha__> noopq, I have written down in the spec which data we need from API
09:08:31 <noopq> neha__, let's get on a call after this with kaverma
09:08:44 <neha__> brainfunked, https://github.com/Tendrl/specifications/pull/81 This PR is still not reviewed
09:08:50 <neha__> noopq, ok
09:08:56 <brainfunked> neha__, checking
09:09:43 <brainfunked> noopq, could you take a quick look at that spec and confirm whether your updated APIs will be able to provide means to address this?
09:09:51 <noopq> neha__, I know its waiting on me to review, I can do it later, but we need to discuss some changes before that
09:09:57 <noopq> brainfunked, ok
09:11:18 <kaverma> brainfunked, Can i start with my update ?
09:11:44 <brainfunked> kaverma, need info pertaining to list views specifically at the moment
09:11:54 <brainfunked> kaverma, is that all you're working on enabling presently?
09:12:23 <kaverma> brainfunked, i am working on host-list-view
09:12:27 <kaverma> brainfunked, just started
09:13:00 <kaverma> brainfunked, and anup said .. he is going to give response format .. than i am not blocked with anything
09:13:03 <brainfunked> kaverma, are the components you were working on done?
09:13:09 <noopq> neha__, why are you concerned with the PR?
09:13:33 <noopq> https://github.com/Tendrl/specifications/pull/81
09:13:35 <kaverma> brainfunked, there is 2 chart components - donut and sparkline
09:13:40 <noopq> I should be converned about it
09:13:45 <noopq> *concerned
09:13:53 <kaverma> brainfunked, dont chart is done and got merged .. only sparkline is pending
09:14:18 <kaverma> brainfunked,  i sent patch for sparkline chart . you can also please have look once ..
09:14:28 <brainfunked> kaverma, ok, thanks
09:14:49 <noopq> neha__, the monitoring stats will be associated as a part of signle node/cluster response.
09:14:59 <neha__> noopq, this will give us the API response format for usage right?
09:15:09 <neha__> noopq, ok
09:15:18 <brainfunked> #action noopq to add details about the updated APIs to enable the list views on the specification and provide the response format so that the UI work can proceed accordingly
09:15:18 <noopq> neha__, NO!, you are never integrating with the monitoring api ever
09:16:02 <brainfunked> #accept kaverma has implemented the donut component, the sparkline component has a pull request that's pending reviews
09:16:10 <noopq> you will always get the details from tendrl-api associated to the respective node/cluster
09:16:17 <noopq> neha__, ^^
09:16:18 <neha__> noopq, ok
09:16:37 <brainfunked> ok, neha__, kaverma, noopq, get on a call and document and get clarity on this stuff like we did on Friday
09:16:45 <noopq> brainfunked, ack.
09:16:45 <brainfunked> noopq, document directly on the spec itself
09:17:11 <kaverma> brainfunked , sure .. we all sync up
09:18:18 <brainfunked> neha__, kaverma, the UI for the list views needs to be completed this week. For missing data points, noopq will be able to provide you the response format. Wherever the response doesn't provide the data, ensure that you can display N/A or some such on the UI, without breaking the layout. As soon as the data is provided by the API, without any changes to the UI code, it should start getting displayed in the appropriate place.
09:18:38 <brainfunked> noopq, get anbehl on the call too
09:18:45 <noopq> brainfunked, alright
09:19:01 <kaverma> brainfunked, ok
09:19:02 <brainfunked> ok, neha__, kaverma are there any updates other than list views?
09:19:18 <neha__> brainfunked, not as of now
09:19:21 <brainfunked> ack
09:19:26 <kaverma> brainfunked, no
09:19:33 <brainfunked> alright, moving on then, thanks for your updates
09:19:55 <brainfunked> #topic dahorak's updates
09:20:04 <dahorak> GitHub Link: https://github.com/Tendrl/usmqe-setup/pull/6
09:20:04 <dahorak> Progress: Pull request discussed with Martin
09:20:04 <dahorak> GitHub Link: https://github.com/Tendrl/usmqe-setup/pull/11 Gluster bricks and volume roles
09:20:04 <dahorak> GitHub Link: https://github.com/Tendrl/usmqe-setup/pull/12 Role qe-server: disable host_key_checking in ansible.cfg
09:20:04 <dahorak> GitHub Link: https://github.com/Tendrl/usmqe-setup/pull/13 Add qe_server_sshkey.yml playbook (used mainly in CentOS CI)
09:20:05 <dahorak> GitHub Link: https://github.com/Tendrl/usmqe-setup/pull/14 Configure firewall on test machines (workaround)
09:20:05 <dahorak> Progress: Created pull requests (QE infrastructure related)
09:20:05 <dahorak> GitHub Link: https://github.com/Tendrl/specifications/pull/61 Specification: Centos CI utilization
09:20:05 <dahorak> Progress: small update, otherwise I think most of the question were answered from QE perspective
09:20:06 <dahorak> GitHub Link: https://github.com/Tendrl/specifications/issues/53
09:20:06 <dahorak> Progress: continues discussion about some questions
09:20:07 <dahorak> Status: good/not blocked, PTO from tomorrow till 2nd Jan
09:24:12 <brainfunked> #accept dahorak continuing work on the centos ci specification, gluster bricks and volume roles pull request, along with other testing infrastructure related pull requests
09:24:27 <brainfunked> #info dahorak will be on PTO from tomorrow till 2nd Jan
09:24:30 <brainfunked> thanks for the update
09:24:39 <brainfunked> #topic gowtham's updates
09:24:48 <gowtham> Adding disk details in hardware inventory
09:24:48 <gowtham> git-issue: https://github.com/Tendrl/node_agent/issues/7
09:24:48 <gowtham> spec file: https://github.com/Tendrl/specifications/pull/21
09:24:48 <gowtham> code-PR: https://github.com/Tendrl/node_agent/pull/64
09:25:25 <gowtham> brainfunked, hwinfo giving some unique id, but not in uuid,we can use that
09:25:44 <brainfunked> gowtham, since the uuid isn't available for free disks, we need to use an id that's always available
09:25:55 <brainfunked> uuid can be recorded when available
09:25:57 <gowtham> nishanth, no not available
09:26:04 <gowtham> brainfunked, sry not available
09:26:25 <gowtham> brainfunked, but it gives some unique id for all
09:26:32 <brainfunked> gowtham, yes, so we could leverage that
09:26:48 <gowtham> brainfunked, no need to use that
09:26:50 <gowtham> ?
09:26:56 <brainfunked> gowtham, could you populate the summary of our conversation and your findings about hwinfo on the spec?
09:27:10 <brainfunked> gowtham, use the unique id that's always available, so if it's the id that hwinfo provides, we should use it
09:27:27 <brainfunked> gowtham, however, you need to investigate where this id actually comes from
09:27:30 <gowtham> brainfunked, ok
09:27:41 <brainfunked> gowtham, so check the hwinfo documentation or codebase to understand the origins for this id
09:27:50 <gowtham> brainfunked, ok and partitions have that id as parent id
09:28:06 <brainfunked> which is good, it allows linkage
09:28:13 <gowtham> brainfunked, so we can identify partitions comes under which parent
09:28:43 <gowtham> brainfunked, so i use lsblk and hwinfo both to get all possible fields
09:28:57 <brainfunked> gowtham, the specificaion is missing the etcd structure you'll be populating, please add that info
09:29:13 <gowtham> brainfunked, ok will add
09:29:18 <nishanth> gowtham, is there any fields which are not available with hwinfo?
09:29:40 <brainfunked> gowtham, you'll also need to add the strategy for hwinfo you'll use
09:29:43 <gowtham> brainfunked, if we use that unique id then we can store all disk in same key right?
09:29:51 <gowtham> brainfunked, ok
09:30:39 <gowtham> brainfunked, only problem rhel and eprl does not have hwinfo, we have to build
09:30:47 <nishanth> brainfunked, if we have unique-id do we really need to store as disks/used, disks/free
09:31:01 <brainfunked> let's take this to the spec
09:31:10 <Shu6h3ndu> nishanth, brainfunked, it should not be required.
09:31:17 <brainfunked> nishanth, I've already commented on the spec about how the etcd data structures should look like
09:31:35 <nishanth> brainfunked, ok I will take look
09:31:36 <brainfunked> gowtham, apart from this, did you work on anything else?
09:31:59 <gowtham> gowtham, ok i got that i have to store all disk using id and free and used key for reference
09:32:18 <gowtham> brainfunked, i am working on logging implementation
09:32:30 <gowtham> brainfunked, i have already sent sepc file for that
09:32:45 <brainfunked> ok, will review
09:32:50 <gowtham> brainfunked, ok
09:33:53 <brainfunked> #action gowtham to update the disk inventory specification https://github.com/Tendrl/specifications/pull/21 with details regarding hwinfo and how it can be used to gather disk data and track changes
09:35:19 <brainfunked> gowtham, I can't find your logging PR from the issue
09:35:29 <brainfunked> gowtham, please ensure that you link each PR to the appropriate spec issue
09:35:41 <brainfunked> gowtham, and also each issue created on the component repositories
09:35:45 <gowtham> brainfunked, https://github.com/Tendrl/specifications/pull/94
09:35:56 <gowtham> brainfunked, ok
09:36:12 <gowtham> brainfunked, i will link the pr in issue
09:36:34 <brainfunked> #action brainfunked Shu6h3ndu and nishanth to review logging implementation spec https://github.com/Tendrl/specifications/pull/94
09:36:47 <brainfunked> alright thanks for the update
09:36:59 <brainfunked> #topic mkudlej's updates
09:40:54 <brainfunked> #info mkudlej doesn't seem to be present, moving on
09:40:58 <brainfunked> #topic nishanth's updates
09:41:42 <nishanth> Issue: https://github.com/Tendrl/specifications/issues/55
09:41:42 <nishanth> Spec-PR:
09:41:42 <nishanth> Progress and status: Started working on this
09:41:52 <nishanth> Reviews
09:41:52 <nishanth> - https://github.com/Tendrl/specifications/pull/67
09:41:52 <nishanth> - https://github.com/Tendrl/specifications/pull/63
09:41:52 <nishanth> - https://github.com/Tendrl/specifications/pull/85
09:41:52 <nishanth> - https://github.com/Tendrl/specifications/pull/83
09:41:53 <nishanth> - https://github.com/Tendrl/specifications/pull/82
09:41:54 <nishanth> - https://github.com/Tendrl/specifications/pull/76
09:41:56 <nishanth> - https://github.com/Tendrl/specifications/pull/92
09:41:58 <nishanth> - https://github.com/Tendrl/specifications/pull/90
09:42:00 <nishanth> - https://github.com/Tendrl/specifications/pull/21
09:42:31 <nishanth> also started with code refactoring code reviews
09:42:42 <nishanth> Planing to complete that today
09:47:29 <mkudlej> brainfunked: During review of https://github.com/Tendrl/tendrl-api/issues/40
09:47:32 <mkudlej> I've found https://github.com/Tendrl/documentation/issues/70
09:47:35 <mkudlej> Also i think we should solve https://github.com/Tendrl/specifications/issues/53 and I'm waiting for Mrugesh's comment there.
09:49:24 <brainfunked> nishanth, apart from the service detection spec, which is under review, and the import cluster spec, which is pending on me, was there any other spec you were working on?
09:49:59 <nishanth> yeah, flight ops issues/55
09:50:17 <brainfunked> ok
09:50:32 <brainfunked> #accept nishanth working on in-flight operations updates specification https://github.com/Tendrl/specifications/issues/55
09:50:49 <brainfunked> # topic mkudlej's updates
09:51:02 <brainfunked> #topic mkudlej's updates
09:51:23 <brainfunked> #action brainfunked to comment on the centos ci discussion at https://github.com/Tendrl/specifications/issues/53
09:52:35 <brainfunked> mkudlej, https://github.com/Tendrl/tendrl-api/issues/40 should be deferred. There are expected to be changes there based on the ongoing discussions around the import cluster UX designs
09:53:03 <brainfunked> mkudlej, anything else that you're working on?
09:54:29 <mkudlej> brainfunked: that's all. Could you please write comment to #40 about changes?
09:55:38 <brainfunked> mkudlej, it needs to be tracked via the specification. I'll ask nishanth to link it to the spec, and the details will be on the spec itself for review, before being added to this issue
09:56:44 <brainfunked> #action nishanth to link the issue regarding import cluster on the api repository https://github.com/Tendrl/tendrl-api/issues/40 to the appropriate specification so that the details can be hashed out there before the issue is updated and taken up for implementation
09:57:02 <brainfunked> mkudlej, is there a specific order in which you're picking up issues for review?
09:57:16 <brainfunked> mkudlej, because we need to have the specifications being reviewed first
10:01:04 <brainfunked> I'll add this as information and move on
10:01:11 <mkudlej> brainfunked: I've found this incosistency during review of https://github.com/Tendrl/documentation/pull/62/files
10:01:28 <brainfunked> mkudlej, any response to my question?
10:04:28 <mkudlej> brainfunked: yes, I'll review also specifications
10:04:40 <brainfunked> thanks
10:05:24 <brainfunked> #accept mkudlej has raised an issue regarding the use of HTTP verbs in the API https://github.com/Tendrl/documentation/issues/70 during the review of the API documentation https://github.com/Tendrl/documentation/pull/62/
10:05:49 <brainfunked> #action noopq to respond to the issue regarding the use of HTTP verbs https://github.com/Tendrl/documentation/issues/70
10:06:11 <brainfunked> #info mkudlej has been requested to review specifications and has agreed to do so
10:06:22 <brainfunked> #topic Shu6h3ndu's updates
10:06:48 <Shu6h3ndu> Issue: https://github.com/Tendrl/specifications/issues/80
10:06:48 <Shu6h3ndu> Spec: https://github.com/Tendrl/specifications/pull/93
10:06:48 <Shu6h3ndu> Progress: Got few responses on mail chain with ceph-devel on usage of ceph df. Review in progress
10:06:48 <Shu6h3ndu> Status: NOT BLOCKED.
10:06:48 <Shu6h3ndu> Summary: As suggested wil break the spec in two 1. for ceph bridge 2. for monitoring (anmol to pick)
10:06:49 <Shu6h3ndu> Issue: https://github.com/Tendrl/specifications/issues/30 (Accomodate gluster get-state cli changes post gluster 3.9)
10:06:52 <Shu6h3ndu> Spec: https://github.com/Tendrl/specifications/pull/6 (Added specification for refactoring gluster get-state output)
10:06:57 <Shu6h3ndu> Progress: No major review comments. Ready to merge
10:06:59 <Shu6h3ndu> Status: WAITING FOR MERGE
10:07:03 <Shu6h3ndu> Issue: https://github.com/Tendrl/specifications/issues/31 (Refactor Tendrl framework sub modules)
10:07:05 <Shu6h3ndu> Spec: https://github.com/Tendrl/specifications/pull/7 (Added specifications for refactoring tendrl code)
10:07:08 <Shu6h3ndu> Progress: code change PR under review. Pushed ceph_integration changes on 19-dec-2016. ALL THE CODE CHANGES IN
10:07:11 <Shu6h3ndu> Status: BLOCKED ON REVIEW
10:07:13 <Shu6h3ndu> Issue: https://github.com/Tendrl/gluster_integration/issues/74 (Implement the pre and post runs for volume operations)
10:07:16 <Shu6h3ndu> Spec: NA (OLD ONE)
10:07:20 <Shu6h3ndu> Progress: Comments incorporated. Nishanth to review
10:07:22 <Shu6h3ndu> Status: BLOCKED ON REVIEW
10:07:24 <Shu6h3ndu> Issue: https://github.com/Tendrl/gluster_integration/issues/73 (Delete the volume entry from etcd as well while volume delete)
10:07:28 <brainfunked> Shu6h3ndu, I've reviewed the ceph pool usage spec
10:07:29 <Shu6h3ndu> Spec: NA (OLD ONE)
10:07:31 <Shu6h3ndu> Progress: Comments incorporated. Nishanth to review
10:07:35 <Shu6h3ndu> Status: BLOCKED ON REVIEW
10:07:37 <Shu6h3ndu> Reviews
10:07:39 <Shu6h3ndu> https://github.com/Tendrl/node_agent/pull/108
10:07:40 <brainfunked> ack, you've mentioned as such here
10:07:41 <Shu6h3ndu> https://github.com/Tendrl/tendrl-api/issues/31
10:07:43 <Shu6h3ndu> https://github.com/Tendrl/node_agent/issues/94
10:07:45 <Shu6h3ndu> https://github.com/Tendrl/specifications/pull/81
10:09:00 <brainfunked> #action nishanth to review pre and post runs for volume operations, https://github.com/Tendrl/gluster_integration/issues/74, and deleting volume entries from etcd on volume delete, https://github.com/Tendrl/gluster_integration/issues/73, pull requests
10:09:24 <brainfunked> #accept Shu6h3ndu working on the ceph pool usage spec, brainfunked has reviewed the same
10:09:52 <brainfunked> #action Shu6h3ndu to split the ceph pool usage spec into two, bridge and monitoring specific specs
10:10:35 <brainfunked> #accept gluster get-state change related specification is ready to be merged https://github.com/Tendrl/specifications/issues/30
10:10:41 <brainfunked> #topic t1m1's updates
10:10:54 <t1m1> https://github.com/TimothyAsirJeyasing/node_agent/commit/b32c8f70b60a1b501c9c129c83a431ca0f57c15e
10:10:58 <t1m1> sorry
10:11:07 <t1m1> Bug Fixes
10:11:08 <t1m1> sent PR to https://github.com/Tendrl/tendrl-api/pull/49 for
10:11:08 <t1m1> issues: https://github.com/Tendrl/tendrl-api/issues/45
10:11:08 <t1m1> and https://github.com/Tendrl/tendrl-api/issues/46
10:11:08 <t1m1> Updated the specification as per review comment
10:11:08 <t1m1> https://github.com/Tendrl/specifications/issues/57
10:11:12 <t1m1> Sent a PR: https://github.com/Tendrl/node_agent/pull/108
10:11:14 <t1m1> for specification: https://github.com/Tendrl/specifications/pull/73
10:11:16 <t1m1> Working on:
10:11:18 <t1m1> spec: https://github.com/Tendrl/specifications/pull/73
10:11:20 <t1m1> Bug-fixes: https://github.com/Tendrl/gluster_integration/issues/82
10:11:22 <t1m1> and https://github.com/Tendrl/node_agent/issues/105
10:13:23 <brainfunked> t1m1, please provide spec titles with the links
10:13:31 <t1m1> Sure
10:13:31 <brainfunked> t1m1, issue titles too
10:14:08 <t1m1> Working on: Specification for service detction modules: https://github.com/Tendrl/specifications/pull/73
10:14:50 <t1m1> Yesterday sent patch to update review comment for
10:15:04 <t1m1> spec :  Missing job validation as per latest changes: https://github.com/Tendrl/specifications/issues/57
10:15:58 <brainfunked> #accept t1m1 updating the specification for service detection modules, https://github.com/Tendrl/specifications/pull/73, and job validations https://github.com/Tendrl/specifications/issues/57
10:15:59 <t1m1> Also working on fixing RPMlint errors: bug (issue): https://github.com/Tendrl/gluster_integration/issues/82
10:16:05 <t1m1> and
10:16:37 <t1m1> No package available for pip and python-virtualenvwrapper error: bug (issue url):  https://github.com/Tendrl/node_agent/issues/105
10:16:53 <brainfunked> #accept t1m1 working on various bug fixes
10:18:11 <brainfunked> #action brainfunked, Shu6h3ndu to review the service detection spec https://github.com/Tendrl/specifications/pull/73
10:18:31 <brainfunked> #action brainfunked, nishanth and Shu6h3ndu to review the job validations spec https://github.com/Tendrl/specifications/issues/57
10:18:36 <brainfunked> thanks
10:18:46 <t1m1> no prob;
10:18:50 <brainfunked> #endmeeting