)]}'
{"ceph/values.yaml":[{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"861b2578ba4727c40bf50a76791aee9120a3fb5f","unresolved":false,"context_lines":[{"line_number":335,"context_line":"# above for more info. For example:"},{"line_number":336,"context_line":"ceph_mgr_modules_config:"},{"line_number":337,"context_line":"  prometheus:"},{"line_number":338,"context_line":"    port: 9283"},{"line_number":339,"context_line":"#  dashboard:"},{"line_number":340,"context_line":"#    port: 7000"},{"line_number":341,"context_line":"#  localpool:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"7f96bb07_2172afe4","line":338,"updated":"2018-01-14 21:43:33.000000000","message":"Should we populate this dynamically based on scrape_port below, to avoid duplicate definitions?","commit_id":"b928a4a15111ba3a289e12131bdb0f0a1de51b5f"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"f4afb3c5c72ba761af16487ba030912c38e44cdd","unresolved":false,"context_lines":[{"line_number":335,"context_line":"# above for more info. For example:"},{"line_number":336,"context_line":"ceph_mgr_modules_config:"},{"line_number":337,"context_line":"  prometheus:"},{"line_number":338,"context_line":"    port: 9283"},{"line_number":339,"context_line":"#  dashboard:"},{"line_number":340,"context_line":"#    port: 7000"},{"line_number":341,"context_line":"#  localpool:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"7f96bb07_d537f7e2","line":338,"in_reply_to":"7f96bb07_215b8f54","updated":"2018-01-15 14:11:10.000000000","message":"Actually I dont believe we\u0027ll need this, unless we\u0027re looking to change the default port. Changing the scrape port key is fine, but should be handled separately as that\u0027s going to touch more than just ceph.","commit_id":"b928a4a15111ba3a289e12131bdb0f0a1de51b5f"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"13ebf3b1f94759a843c28d1f0d870a20aa8eec6c","unresolved":false,"context_lines":[{"line_number":335,"context_line":"# above for more info. For example:"},{"line_number":336,"context_line":"ceph_mgr_modules_config:"},{"line_number":337,"context_line":"  prometheus:"},{"line_number":338,"context_line":"    port: 9283"},{"line_number":339,"context_line":"#  dashboard:"},{"line_number":340,"context_line":"#    port: 7000"},{"line_number":341,"context_line":"#  localpool:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"7f96bb07_215b8f54","line":338,"in_reply_to":"7f96bb07_2172afe4","updated":"2018-01-14 21:46:10.000000000","message":"good point, can we also include the port in the endpoints beneath the \u0027port\u0027 key, ie `port.prometheus` rather than the arbitrary `scrape_port` at the root of the endpoint tree.","commit_id":"b928a4a15111ba3a289e12131bdb0f0a1de51b5f"}]}