)]}'
{"monasca_agent/collector/checks_d/vcenter.py":[{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"6cad7f62f5bad9a347942bd8fefccbd93e22a436","unresolved":false,"context_lines":[{"line_number":448,"context_line":"                           (managed_cluster, key, data.get(key)))"},{"line_number":449,"context_line":""},{"line_number":450,"context_line":"    def _get_dims(self, cluster, mor_id):"},{"line_number":451,"context_line":"        hostname \u003d mor_id + \".\" + self.vc_uuid"},{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_8e3c98e3","line":451,"range":{"start_line":451,"start_character":8,"end_line":451,"end_character":16},"updated":"2016-04-13 21:35:51.000000000","message":"Not sure why you are setting hostname to an entirely different value than for the rest of the metrics. This is not conforming to the conventions/precedents that have already been set regarding this dimension.","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"2e8adc20206d5d499d61932e6cc508d0d0d107df","unresolved":false,"context_lines":[{"line_number":448,"context_line":"                           (managed_cluster, key, data.get(key)))"},{"line_number":449,"context_line":""},{"line_number":450,"context_line":"    def _get_dims(self, cluster, mor_id):"},{"line_number":451,"context_line":"        hostname \u003d mor_id + \".\" + self.vc_uuid"},{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_a15a2328","line":451,"range":{"start_line":451,"start_character":8,"end_line":451,"end_character":16},"in_reply_to":"5a18252c_708f3b2a","updated":"2016-04-14 12:51:17.000000000","message":"OK, understand what is going on now. Thanks. Will discuss later with team and merge as appropriate.","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":20327,"name":"RenukaPrasadNS","email":"renuka.n-s@hpe.com","username":"nsren"},"change_message_id":"4e47f0df178ef6fbcb4cfc0b107a21d4a488c5b4","unresolved":false,"context_lines":[{"line_number":448,"context_line":"                           (managed_cluster, key, data.get(key)))"},{"line_number":449,"context_line":""},{"line_number":450,"context_line":"    def _get_dims(self, cluster, mor_id):"},{"line_number":451,"context_line":"        hostname \u003d mor_id + \".\" + self.vc_uuid"},{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_708f3b2a","line":451,"range":{"start_line":451,"start_character":8,"end_line":451,"end_character":16},"in_reply_to":"5a18252c_8e3c98e3","updated":"2016-04-14 12:41:08.000000000","message":"As this is remote monitoring plugin, the data which it collects is not related to host where the plugin is running.\nWe are conforming to the name same as set in nova when an ESX cluster is activated. If this is not set, then the ESX-Compute-Proxy\u0027s hostname is taken which is not exactly the hostname of the compute cluster, which it is monitoring.\nActivated - means enabled ESX cluster to act as compute node","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"6cad7f62f5bad9a347942bd8fefccbd93e22a436","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        hostname \u003d mor_id + \".\" + self.vc_uuid"},{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"},{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_312a097f","line":454,"range":{"start_line":454,"start_character":13,"end_line":454,"end_character":20},"updated":"2016-04-13 21:35:51.000000000","message":"Is cluster something that is specific to identifying something in vcenter?","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":20327,"name":"RenukaPrasadNS","email":"renuka.n-s@hpe.com","username":"nsren"},"change_message_id":"ac0f6897f1f758a8ca680d20dd3d95233f1d7966","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        hostname \u003d mor_id + \".\" + self.vc_uuid"},{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"},{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_29c3998a","line":454,"range":{"start_line":454,"start_character":13,"end_line":454,"end_character":20},"in_reply_to":"5a18252c_067a0dbd","updated":"2016-04-14 13:04:30.000000000","message":"Fine, no issue, we will remove this as well.","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":20327,"name":"RenukaPrasadNS","email":"renuka.n-s@hpe.com","username":"nsren"},"change_message_id":"4e47f0df178ef6fbcb4cfc0b107a21d4a488c5b4","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        hostname \u003d mor_id + \".\" + self.vc_uuid"},{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"},{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_b001d36c","line":454,"range":{"start_line":454,"start_character":13,"end_line":454,"end_character":20},"in_reply_to":"5a18252c_312a097f","updated":"2016-04-14 12:41:08.000000000","message":"Yes, it is the name of the ESX cluster which is activated. Similar to compute name for RHEL/hLinux","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"2e8adc20206d5d499d61932e6cc508d0d0d107df","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        hostname \u003d mor_id + \".\" + self.vc_uuid"},{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"},{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_067a0dbd","line":454,"range":{"start_line":454,"start_character":13,"end_line":454,"end_character":20},"in_reply_to":"5a18252c_b001d36c","updated":"2016-04-14 12:51:17.000000000","message":"The problem with this name is that this dimension is already used in Helion, which for upstream is not an issue, but it would be nice to select a dimension name that didn\u0027t collide. So, by setting it here you will override a dimension that Helion is using and this will be important in the near future when support for multi-region is enabled. There are a couple of other dimensions that are used in Helion too, such as control_plane, but this one will definitely collide, so I think we need a new name for this dimension.","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"6cad7f62f5bad9a347942bd8fefccbd93e22a436","unresolved":false,"context_lines":[{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"},{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"},{"line_number":458,"context_line":"            \"hostname\": hostname"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_ce0df03f","line":455,"range":{"start_line":455,"start_character":13,"end_line":455,"end_character":22},"updated":"2016-04-13 21:35:51.000000000","message":"I know this is already in the code, but why do you need a host_type\u003dcompute_node. I missed this in the initial review.","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"2e8adc20206d5d499d61932e6cc508d0d0d107df","unresolved":false,"context_lines":[{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"},{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"},{"line_number":458,"context_line":"            \"hostname\": hostname"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_66514136","line":455,"range":{"start_line":455,"start_character":13,"end_line":455,"end_character":22},"in_reply_to":"5a18252c_8b4c669f","updated":"2016-04-14 12:51:17.000000000","message":"Thanks","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":20327,"name":"RenukaPrasadNS","email":"renuka.n-s@hpe.com","username":"nsren"},"change_message_id":"4e47f0df178ef6fbcb4cfc0b107a21d4a488c5b4","unresolved":false,"context_lines":[{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"},{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"},{"line_number":458,"context_line":"            \"hostname\": hostname"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_8b4c669f","line":455,"range":{"start_line":455,"start_character":13,"end_line":455,"end_character":22},"in_reply_to":"5a18252c_ce0df03f","updated":"2016-04-14 12:41:08.000000000","message":"Added in CS9, not relevant now, will remove","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"6cad7f62f5bad9a347942bd8fefccbd93e22a436","unresolved":false,"context_lines":[{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"},{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"},{"line_number":458,"context_line":"            \"hostname\": hostname"},{"line_number":459,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_b10f392d","line":456,"range":{"start_line":456,"start_character":13,"end_line":456,"end_character":17},"updated":"2016-04-13 21:35:51.000000000","message":"Same comment as above, but why to you need to identify the role  as \"esx\"? There aren\u0027t any other metrics in the system that have a dimension of role, so I\u0027m now wondering how this is going to be used.","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"2e8adc20206d5d499d61932e6cc508d0d0d107df","unresolved":false,"context_lines":[{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"},{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"},{"line_number":458,"context_line":"            \"hostname\": hostname"},{"line_number":459,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_4657e525","line":456,"range":{"start_line":456,"start_character":13,"end_line":456,"end_character":17},"in_reply_to":"5a18252c_0b617605","updated":"2016-04-14 12:51:17.000000000","message":"Thanks","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":20327,"name":"RenukaPrasadNS","email":"renuka.n-s@hpe.com","username":"nsren"},"change_message_id":"4e47f0df178ef6fbcb4cfc0b107a21d4a488c5b4","unresolved":false,"context_lines":[{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"cluster\": cluster,"},{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"},{"line_number":458,"context_line":"            \"hostname\": hostname"},{"line_number":459,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_0b617605","line":456,"range":{"start_line":456,"start_character":13,"end_line":456,"end_character":17},"in_reply_to":"5a18252c_b10f392d","updated":"2016-04-14 12:41:08.000000000","message":"Added in CS9, not relevant now, will remove","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":15027,"name":"Michael Hoppal","email":"michael.jam.hoppal@hpe.com","username":"michael-hoppal"},"change_message_id":"371496075d059a628693a5d54be8f1e029855a4b","unresolved":false,"context_lines":[{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"},{"line_number":458,"context_line":"            \"hostname\": hostname"},{"line_number":459,"context_line":"        }"},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_f1a6d151","line":458,"range":{"start_line":458,"start_character":13,"end_line":458,"end_character":21},"updated":"2016-04-13 21:59:28.000000000","message":"We set this at the top level when setting up the agent so overriding this at the collection level is not something we would like to do. This goes along with the cluster dimension above.","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":20327,"name":"RenukaPrasadNS","email":"renuka.n-s@hpe.com","username":"nsren"},"change_message_id":"4e47f0df178ef6fbcb4cfc0b107a21d4a488c5b4","unresolved":false,"context_lines":[{"line_number":455,"context_line":"            \"host_type\": \"compute_node\","},{"line_number":456,"context_line":"            \"role\": \"esx\","},{"line_number":457,"context_line":"            \"id\": cluster + \"-\" + self.vcenter_ip,"},{"line_number":458,"context_line":"            \"hostname\": hostname"},{"line_number":459,"context_line":"        }"},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_eb407a36","line":458,"range":{"start_line":458,"start_character":13,"end_line":458,"end_character":21},"in_reply_to":"5a18252c_f1a6d151","updated":"2016-04-14 12:41:08.000000000","message":"Explained above. This is done for a reason to support mapping between nova and monasca data.","commit_id":"eeb0e1f9152cb59fcc02c8b9e2657f92ecc7e884"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"fcb765a121d85d2f9b156850b58f53f422b5a9fb","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        hostname \u003d mor_id + \".\" + self.vc_uuid"},{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"hostname\": hostname"},{"line_number":455,"context_line":"        }"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a18252c_c0bf46a8","line":454,"range":{"start_line":454,"start_character":13,"end_line":454,"end_character":21},"updated":"2016-04-14 17:34:28.000000000","message":"We reviewed with the team and are getting closer, but we don\u0027t think this is quite done yet and have some further questions.\n\nWe believe setting the hostname to the remote host that is being monitoring is acceptable. However:\n\n1. The hostname should be a FQDN or IP address. The hostname that you are setting too here has the UUID in it and it doesn\u0027t look like it is a FQDN or IP address. \n\n2. Add a dimension of observer_host. There are several other metrics that set the dimension of hostname to the name of the remote host, such as the host_alive check. However, in that case, a dimension of observer_host is added.\n\n3. If you need UUID as part of the metric, which it looks like it is, would adding it as a dimension be a better approach.","commit_id":"7f2fc3cf686bbd8addab6dd16daab3895205806a"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"83f615af05b8131f58f57671372cd927cb3def91","unresolved":false,"context_lines":[{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"observer_host\": obs_host"},{"line_number":455,"context_line":"        }"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":"    @staticmethod"},{"line_number":458,"context_line":"    def parse_agent_config(agentConfig):"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a122d0e_79968d95","line":455,"range":{"start_line":455,"start_character":8,"end_line":455,"end_character":9},"updated":"2016-04-15 15:51:27.000000000","message":"The \"observer_host\", should be the FQDN or IP address of the host that the check is running on. Currently, your review has the \"observer_host\" set to the ESX Cluster ID. This is what I think is the preferred approach:\n\n1. Set \"observer_host\" to be the FQDN or IP address of the host that you are running on. I think you can just copy the existing \"hostname\" dimension over. If you don\u0027t have the FQDN or IP address of the remote host, then don\u0027t add \"observer_host.\n\n2. Set \"hostname\" to the FQDN or IP address of the remote host that you are checking. Note, I\u0027m assuming that the agent is running on a different system than the one that you are running on. If you don\u0027t have access to the FQDN or IP address of the remote host, then don\u0027t set this dimension.\n\n3. Create a new dimension, such as \"esx_cluster_id\", and set that as follows:\n\n        esx_cluster_id \u003d mor_id + \".\" + self.vc_uuid\n\nIn the best case scenario you have the FQDN or IP address of the remote host. The preferred response is as follows:\n\n        return {\n            \"vcenter_ip\": self.vcenter_ip,\n            \"observer_host\": ...,\n            \"hostname\": ...,\n            \"esx_cluster_id\": esx_cluster_id\n        }\n\nIf you don\u0027t have the FQDN or IP address of the remote host, then don\u0027t bother changing the \"observer_host and just return the following:\n\n        return {\n            \"vcenter_ip\": self.vcenter_ip,\n            \"esx_cluster_id\": esx_cluster_id\n        }","commit_id":"226c1089e2129f423a47abf982ff30200f56026c"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"43ea35e95bee1e8b9a5f03a168e14a0c9bf4165d","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        cluster_id \u003d mor_id + \".\" + self.vc_uuid"},{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"esxcluster_id\": cluster_id"},{"line_number":455,"context_line":"        }"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a122d0e_3fab558b","line":454,"range":{"start_line":454,"start_character":13,"end_line":454,"end_character":26},"updated":"2016-04-15 16:34:54.000000000","message":"Would be much more readable with an underscore between names. As in\n\n\"esx_cluster_id\"\n\nBTW, what does the \"vcenter_ip\" correspond too?","commit_id":"9f6333550097a458bb7e3105345ceca48e124d4b"},{"author":{"_account_id":20327,"name":"RenukaPrasadNS","email":"renuka.n-s@hpe.com","username":"nsren"},"change_message_id":"e0a98d1703a2a7dd197a495b5a10dc89e8f09b9d","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        cluster_id \u003d mor_id + \".\" + self.vc_uuid"},{"line_number":452,"context_line":"        return {"},{"line_number":453,"context_line":"            \"vcenter_ip\": self.vcenter_ip,"},{"line_number":454,"context_line":"            \"esxcluster_id\": cluster_id"},{"line_number":455,"context_line":"        }"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a122d0e_bf4925d3","line":454,"range":{"start_line":454,"start_character":13,"end_line":454,"end_character":26},"in_reply_to":"1a122d0e_3fab558b","updated":"2016-04-15 16:40:07.000000000","message":"thats ipaddress of the vcenter server.\nvCenter is a vm which manages clusters.","commit_id":"9f6333550097a458bb7e3105345ceca48e124d4b"}]}