Skip to content

Commit

Permalink
[BACKPORT 2.20][#24029] Update the callhome diagnostics not to send g…
Browse files Browse the repository at this point in the history
…flags details.

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.
Original commit: 0c84dbe / D38263

Test Plan: manual test

Reviewers: sgarg-yb, djiang

Reviewed By: djiang

Subscribers: djiang, yugabyted-dev

Differential Revision: https://phorge.dev.yugabyte.com/D38278
  • Loading branch information
nchandrappa committed Sep 20, 2024
1 parent e1a3aca commit 4d24bbc
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 @@ -5349,8 +5349,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 4d24bbc

Please sign in to comment.