{"action":"created","comment":{"url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/comments/53968833","id":53968833,"diff_hunk":"@@ -0,0 +1,39 @@\n+##############################################################################\n+#\n+# Copyright (C) Zenoss, Inc. 2016, all rights reserved.\n+#\n+# This content is made available according to terms specified in\n+# License.zenoss under the directory where your Zenoss product is installed.\n+#\n+##############################################################################\n+\n+from Products.Zuul.infos.component.filesystem import FileSystemInfo as BaseFileSystemInfo\n+from Products.Zuul.decorators import info\n+from zope.interface import implements\n+from ZenPacks.zenoss.LinuxMonitor.interfaces import IHardDiskInfo\n+from Products.Zuul.infos import ProxyProperty\n+from Products.Zuul.infos.component import ComponentInfo\n+\n+\n+class HardDiskInfo(ComponentInfo):\n+    implements(IHardDiskInfo)\n+\n+    size = ProxyProperty('size')\n+\n+    @property\n+    @info\n+    def physicalvolumes(self):","path":"ZenPacks/zenoss/LinuxMonitor/info.py","position":25,"original_position":25,"commit_id":"d3cf879a7f16054d710ea93e6855adca59bc5188","original_commit_id":"d3cf879a7f16054d710ea93e6855adca59bc5188","user":{"login":"cluther","id":86579,"avatar_url":"https://avatars.githubusercontent.com/u/86579?v=3","gravatar_id":"","url":"https://api.github.com/users/cluther","html_url":"https://github.com/cluther","followers_url":"https://api.github.com/users/cluther/followers","following_url":"https://api.github.com/users/cluther/following{/other_user}","gists_url":"https://api.github.com/users/cluther/gists{/gist_id}","starred_url":"https://api.github.com/users/cluther/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cluther/subscriptions","organizations_url":"https://api.github.com/users/cluther/orgs","repos_url":"https://api.github.com/users/cluther/repos","events_url":"https://api.github.com/users/cluther/events{/privacy}","received_events_url":"https://api.github.com/users/cluther/received_events","type":"User","site_admin":false},"body":"HardDiskInfo.physicalvolumes should be removed. Having the count for to-many relationships is enough. Including a list of the actual objects causes too much to be serialized to for calls by the web interface.","created_at":"2016-02-24T16:53:48Z","updated_at":"2016-02-24T16:53:48Z","html_url":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor/pull/35#discussion_r53968833","pull_request_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/35","_links":{"self":{"href":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/comments/53968833"},"html":{"href":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor/pull/35#discussion_r53968833"},"pull_request":{"href":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/35"}}},"pull_request":{"url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/35","id":59835459,"html_url":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor/pull/35","diff_url":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor/pull/35.diff","patch_url":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor/pull/35.patch","issue_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/issues/35","number":35,"state":"open","locked":false,"title":"Feature/monitoring","user":{"login":"dbouchillon","id":8355179,"avatar_url":"https://avatars.githubusercontent.com/u/8355179?v=3","gravatar_id":"","url":"https://api.github.com/users/dbouchillon","html_url":"https://github.com/dbouchillon","followers_url":"https://api.github.com/users/dbouchillon/followers","following_url":"https://api.github.com/users/dbouchillon/following{/other_user}","gists_url":"https://api.github.com/users/dbouchillon/gists{/gist_id}","starred_url":"https://api.github.com/users/dbouchillon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dbouchillon/subscriptions","organizations_url":"https://api.github.com/users/dbouchillon/orgs","repos_url":"https://api.github.com/users/dbouchillon/repos","events_url":"https://api.github.com/users/dbouchillon/events{/privacy}","received_events_url":"https://api.github.com/users/dbouchillon/received_events","type":"User","site_admin":false},"body":"For Chet review\r\nThis adds in the HardDisk component for lvm.  Subclassed ZenModel.FileSystem to add in logical volume relationship.  updated impact diagram and yuml.  added monitoring for PV, LV, VG.","created_at":"2016-02-18T17:39:22Z","updated_at":"2016-02-24T16:53:48Z","closed_at":null,"merged_at":null,"merge_commit_sha":"e6aa6505b2a84676ef4d859ecc3aae64dfa0fafb","assignee":{"login":"cluther","id":86579,"avatar_url":"https://avatars.githubusercontent.com/u/86579?v=3","gravatar_id":"","url":"https://api.github.com/users/cluther","html_url":"https://github.com/cluther","followers_url":"https://api.github.com/users/cluther/followers","following_url":"https://api.github.com/users/cluther/following{/other_user}","gists_url":"https://api.github.com/users/cluther/gists{/gist_id}","starred_url":"https://api.github.com/users/cluther/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cluther/subscriptions","organizations_url":"https://api.github.com/users/cluther/orgs","repos_url":"https://api.github.com/users/cluther/repos","events_url":"https://api.github.com/users/cluther/events{/privacy}","received_events_url":"https://api.github.com/users/cluther/received_events","type":"User","site_admin":false},"milestone":null,"commits_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/35/commits","review_comments_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/35/comments","review_comment_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/comments{/number}","comments_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/issues/35/comments","statuses_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/statuses/d3cf879a7f16054d710ea93e6855adca59bc5188","head":{"label":"zenoss:feature/monitoring","ref":"feature/monitoring","sha":"d3cf879a7f16054d710ea93e6855adca59bc5188","user":{"login":"zenoss","id":243767,"avatar_url":"https://avatars.githubusercontent.com/u/243767?v=3","gravatar_id":"","url":"https://api.github.com/users/zenoss","html_url":"https://github.com/zenoss","followers_url":"https://api.github.com/users/zenoss/followers","following_url":"https://api.github.com/users/zenoss/following{/other_user}","gists_url":"https://api.github.com/users/zenoss/gists{/gist_id}","starred_url":"https://api.github.com/users/zenoss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zenoss/subscriptions","organizations_url":"https://api.github.com/users/zenoss/orgs","repos_url":"https://api.github.com/users/zenoss/repos","events_url":"https://api.github.com/users/zenoss/events{/privacy}","received_events_url":"https://api.github.com/users/zenoss/received_events","type":"Organization","site_admin":false},"repo":{"id":10365871,"name":"ZenPacks.zenoss.LinuxMonitor","full_name":"zenoss/ZenPacks.zenoss.LinuxMonitor","owner":{"login":"zenoss","id":243767,"avatar_url":"https://avatars.githubusercontent.com/u/243767?v=3","gravatar_id":"","url":"https://api.github.com/users/zenoss","html_url":"https://github.com/zenoss","followers_url":"https://api.github.com/users/zenoss/followers","following_url":"https://api.github.com/users/zenoss/following{/other_user}","gists_url":"https://api.github.com/users/zenoss/gists{/gist_id}","starred_url":"https://api.github.com/users/zenoss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zenoss/subscriptions","organizations_url":"https://api.github.com/users/zenoss/orgs","repos_url":"https://api.github.com/users/zenoss/repos","events_url":"https://api.github.com/users/zenoss/events{/privacy}","received_events_url":"https://api.github.com/users/zenoss/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor","description":"Zenoss - ZenPacks.zenoss.LinuxMonitor module","fork":false,"url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor","forks_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/forks","keys_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/keys{/key_id}","collaborators_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/teams","hooks_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/hooks","issue_events_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/issues/events{/number}","events_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/events","assignees_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/assignees{/user}","branches_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/branches{/branch}","tags_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/tags","blobs_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/git/refs{/sha}","trees_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/git/trees{/sha}","statuses_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/statuses/{sha}","languages_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/languages","stargazers_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/stargazers","contributors_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/contributors","subscribers_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/subscribers","subscription_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/subscription","commits_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/commits{/sha}","git_commits_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/git/commits{/sha}","comments_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/comments{/number}","issue_comment_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/issues/comments{/number}","contents_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/contents/{+path}","compare_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/compare/{base}...{head}","merges_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/merges","archive_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/downloads","issues_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/issues{/number}","pulls_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls{/number}","milestones_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/milestones{/number}","notifications_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/labels{/name}","releases_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/releases{/id}","deployments_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/deployments","created_at":"2013-05-29T18:03:23Z","updated_at":"2016-01-15T19:38:52Z","pushed_at":"2016-02-24T14:48:46Z","git_url":"git://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor.git","ssh_url":"git@github.com:zenoss/ZenPacks.zenoss.LinuxMonitor.git","clone_url":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor.git","svn_url":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor","homepage":"http://www.zenoss.com","size":379,"stargazers_count":2,"watchers_count":2,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":1,"forks":3,"open_issues":1,"watchers":2,"default_branch":"develop"}},"base":{"label":"zenoss:develop","ref":"develop","sha":"1e69fa8544f5fdf4654e3dab14708bf5b0d8dbdf","user":{"login":"zenoss","id":243767,"avatar_url":"https://avatars.githubusercontent.com/u/243767?v=3","gravatar_id":"","url":"https://api.github.com/users/zenoss","html_url":"https://github.com/zenoss","followers_url":"https://api.github.com/users/zenoss/followers","following_url":"https://api.github.com/users/zenoss/following{/other_user}","gists_url":"https://api.github.com/users/zenoss/gists{/gist_id}","starred_url":"https://api.github.com/users/zenoss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zenoss/subscriptions","organizations_url":"https://api.github.com/users/zenoss/orgs","repos_url":"https://api.github.com/users/zenoss/repos","events_url":"https://api.github.com/users/zenoss/events{/privacy}","received_events_url":"https://api.github.com/users/zenoss/received_events","type":"Organization","site_admin":false},"repo":{"id":10365871,"name":"ZenPacks.zenoss.LinuxMonitor","full_name":"zenoss/ZenPacks.zenoss.LinuxMonitor","owner":{"login":"zenoss","id":243767,"avatar_url":"https://avatars.githubusercontent.com/u/243767?v=3","gravatar_id":"","url":"https://api.github.com/users/zenoss","html_url":"https://github.com/zenoss","followers_url":"https://api.github.com/users/zenoss/followers","following_url":"https://api.github.com/users/zenoss/following{/other_user}","gists_url":"https://api.github.com/users/zenoss/gists{/gist_id}","starred_url":"https://api.github.com/users/zenoss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zenoss/subscriptions","organizations_url":"https://api.github.com/users/zenoss/orgs","repos_url":"https://api.github.com/users/zenoss/repos","events_url":"https://api.github.com/users/zenoss/events{/privacy}","received_events_url":"https://api.github.com/users/zenoss/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor","description":"Zenoss - ZenPacks.zenoss.LinuxMonitor module","fork":false,"url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor","forks_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/forks","keys_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/keys{/key_id}","collaborators_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/teams","hooks_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/hooks","issue_events_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/issues/events{/number}","events_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/events","assignees_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/assignees{/user}","branches_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/branches{/branch}","tags_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/tags","blobs_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/git/refs{/sha}","trees_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/git/trees{/sha}","statuses_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/statuses/{sha}","languages_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/languages","stargazers_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/stargazers","contributors_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/contributors","subscribers_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/subscribers","subscription_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/subscription","commits_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/commits{/sha}","git_commits_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/git/commits{/sha}","comments_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/comments{/number}","issue_comment_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/issues/comments{/number}","contents_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/contents/{+path}","compare_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/compare/{base}...{head}","merges_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/merges","archive_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/downloads","issues_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/issues{/number}","pulls_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls{/number}","milestones_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/milestones{/number}","notifications_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/labels{/name}","releases_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/releases{/id}","deployments_url":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/deployments","created_at":"2013-05-29T18:03:23Z","updated_at":"2016-01-15T19:38:52Z","pushed_at":"2016-02-24T14:48:46Z","git_url":"git://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor.git","ssh_url":"git@github.com:zenoss/ZenPacks.zenoss.LinuxMonitor.git","clone_url":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor.git","svn_url":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor","homepage":"http://www.zenoss.com","size":379,"stargazers_count":2,"watchers_count":2,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":1,"forks":3,"open_issues":1,"watchers":2,"default_branch":"develop"}},"_links":{"self":{"href":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/35"},"html":{"href":"https://github.com/zenoss/ZenPacks.zenoss.LinuxMonitor/pull/35"},"issue":{"href":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/issues/35"},"comments":{"href":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/issues/35/comments"},"review_comments":{"href":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/35/comments"},"review_comment":{"href":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/pulls/35/commits"},"statuses":{"href":"https://api.github.com/repos/zenoss/ZenPacks.zenoss.LinuxMonitor/statuses/d3cf879a7f16054d710ea93e6855adca59bc5188"}}}}