Skip to content

Commit

Permalink
[#24029] Update the callhome diagnostics not to send gflags details.
Browse files Browse the repository at this point in the history
Summary: Currently, collecting gflags information is redundant data between yugabyted and YBDB callhome. It was decided to not send gflags information from yugabyted, instead correlated the data using cluster_uuid, node_uuid, server_type for fetching the gflags details.

Test Plan: manual test

Reviewers: sgarg-yb

Reviewed By: sgarg-yb

Subscribers: yugabyted-dev

Differential Revision: https://phorge.dev.yugabyte.com/D38263
  • Loading branch information
nchandrappa committed Sep 20, 2024
1 parent a50a730 commit 0c84dbe
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions bin/yugabyted
Original file line number Diff line number Diff line change
Expand Up @@ -8283,8 +8283,8 @@ class Diagnostics(object):
payload = {
"data_dir_size": self.get_dir_size(self.configs.saved_data.get("data_dir")),
"num_cpus": multiprocessing.cpu_count(),
"master_flags": self.configs.saved_data.get("master_flags"),
"tserver_flags": self.configs.saved_data.get("tserver_flags"),
# "master_flags": self.configs.saved_data.get("master_flags"),
# "tserver_flags": self.configs.saved_data.get("tserver_flags"),
"is_docker" : str(os.path.exists("/.dockerenv"))
}
if Diagnostics.first_install is not None:
Expand Down

0 comments on commit 0c84dbe

Please sign in to comment.