{"action":"closed","issue":{"url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform/issues/193","repository_url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform","labels_url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform/issues/193/labels{/name}","comments_url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform/issues/193/comments","events_url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform/issues/193/events","html_url":"https://github.com/Terracotta-OSS/terracotta-platform/issues/193","id":191126857,"number":193,"title":"[mnm] Fixing Management Entity, TMS Entity and Monitoring Service implementation and responsability, especially around management call usage","user":{"login":"mathieucarbou","id":61346,"avatar_url":"https://avatars.githubusercontent.com/u/61346?v=3","gravatar_id":"","url":"https://api.github.com/users/mathieucarbou","html_url":"https://github.com/mathieucarbou","followers_url":"https://api.github.com/users/mathieucarbou/followers","following_url":"https://api.github.com/users/mathieucarbou/following{/other_user}","gists_url":"https://api.github.com/users/mathieucarbou/gists{/gist_id}","starred_url":"https://api.github.com/users/mathieucarbou/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mathieucarbou/subscriptions","organizations_url":"https://api.github.com/users/mathieucarbou/orgs","repos_url":"https://api.github.com/users/mathieucarbou/repos","events_url":"https://api.github.com/users/mathieucarbou/events{/privacy}","received_events_url":"https://api.github.com/users/mathieucarbou/received_events","type":"User","site_admin":false},"labels":[{"id":272986261,"url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform/labels/bug","name":"bug","color":"fc2929","default":true},{"id":399780661,"url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform/labels/clustering","name":"clustering","color":"d4c5f9","default":false},{"id":291085349,"url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform/labels/module:%20management","name":"module: management","color":"bfe5bf","default":false},{"id":291084887,"url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform/labels/under%20review","name":"under review","color":"006b75","default":false}],"state":"closed","locked":false,"assignee":{"login":"mathieucarbou","id":61346,"avatar_url":"https://avatars.githubusercontent.com/u/61346?v=3","gravatar_id":"","url":"https://api.github.com/users/mathieucarbou","html_url":"https://github.com/mathieucarbou","followers_url":"https://api.github.com/users/mathieucarbou/followers","following_url":"https://api.github.com/users/mathieucarbou/following{/other_user}","gists_url":"https://api.github.com/users/mathieucarbou/gists{/gist_id}","starred_url":"https://api.github.com/users/mathieucarbou/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mathieucarbou/subscriptions","organizations_url":"https://api.github.com/users/mathieucarbou/orgs","repos_url":"https://api.github.com/users/mathieucarbou/repos","events_url":"https://api.github.com/users/mathieucarbou/events{/privacy}","received_events_url":"https://api.github.com/users/mathieucarbou/received_events","type":"User","site_admin":false},"assignees":[{"login":"mathieucarbou","id":61346,"avatar_url":"https://avatars.githubusercontent.com/u/61346?v=3","gravatar_id":"","url":"https://api.github.com/users/mathieucarbou","html_url":"https://github.com/mathieucarbou","followers_url":"https://api.github.com/users/mathieucarbou/followers","following_url":"https://api.github.com/users/mathieucarbou/following{/other_user}","gists_url":"https://api.github.com/users/mathieucarbou/gists{/gist_id}","starred_url":"https://api.github.com/users/mathieucarbou/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mathieucarbou/subscriptions","organizations_url":"https://api.github.com/users/mathieucarbou/orgs","repos_url":"https://api.github.com/users/mathieucarbou/repos","events_url":"https://api.github.com/users/mathieucarbou/events{/privacy}","received_events_url":"https://api.github.com/users/mathieucarbou/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform/milestones/8","html_url":"https://github.com/Terracotta-OSS/terracotta-platform/milestones/BF-4939","labels_url":"https://api.github.com/repos/Terracotta-OSS/terracotta-platform/milestones/8/labels","id":2095544,"number":8,"title":"BF-4939","description":"","creator":{"login":"mathieucarbou","id":61346,"avatar_url":"https://avatars.githubusercontent.com/u/61346?v=3","gravatar_id":"","url":"https://api.github.com/users/mathieucarbou","html_url":"https://github.com/mathieucarbou","followers_url":"https://api.github.com/users/mathieucarbou/followers","following_url":"https://api.github.com/users/mathieucarbou/following{/other_user}","gists_url":"https://api.github.com/users/mathieucarbou/gists{/gist_id}","starred_url":"https://api.github.com/users/mathieucarbou/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mathieucarbou/subscriptions","organizations_url":"https://api.github.com/users/mathieucarbou/orgs","repos_url":"https://api.github.com/users/mathieucarbou/repos","events_url":"https://api.github.com/users/mathieucarbou/events{/privacy}","received_events_url":"https://api.github.com/users/mathieucarbou/received_events","type":"User","site_admin":false},"open_issues":2,"closed_issues":8,"state":"open","created_at":"2016-10-26T18:14:35Z","updated_at":"2016-12-02T22:49:23Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2016-11-22T21:18:54Z","updated_at":"2016-12-02T22:49:23Z","closed_at":"2016-12-02T22:49:23Z","body":" `ManagementAgent` that __only__ allows exposing registry and client tags (subset of the current management agent)\r\n\r\n- `TmsAgent` that will allows to trigger management calls, read cluster messages and topology and receives management call results\r\n\r\n- A ManagementCall object (like in TMS) that represents the call that we are waiting for (sort of completable future). We have that in TMS, needs to bring it in the client API.\r\n\r\nThis works is to fix:\r\n\r\n- security issues: any client can currently do a mangement call to any other client\r\n- have the same method call for both server and client management calls\r\n- be async all the way (client, active server, passive servers)\r\n\r\nAlso, split the big classes in monitoring service in smaller services:\r\n\r\n- ManagemetService: used for tms entity\r\n- ConsumerMangementRegistry\r\n- PlatformMangementRegistry\r\n- SharedMangementRegistry\r\n- ClientMonitoringService\r\n- ActiveEntityMonitoringService\r\n- PassiveEntityMonitoringService\r\n\r\nThis split will allow to implement management call correctly, because management call implementation depends whether the target is a client, an active entity or a passive entity."}}
{"actor":{"display_login":"anthonydahanne"}}