Skip to content

Commit

Permalink
fix(perf): update ES with append_scylla_yaml dict
Browse files Browse the repository at this point in the history
Since `append_scylla_yaml` is a dict and has value set, updating ES
fails with `failed to parse field [setup_details.append_scylla_yaml] of
type [text]` error.

Fix by converting it to `str` when generating `setup_details`.
  • Loading branch information
soyacz authored and vponomaryov committed Jul 1, 2024
1 parent fe2e27a commit 60cddb4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions sdcm/db_stats.py
Original file line number Diff line number Diff line change
Expand Up @@ -638,6 +638,7 @@ def get_setup_details(self):

setup_details['db_cluster_node_details'] = {}
setup_details['sysctl_output'] = []
setup_details['append_scylla_yaml'] = str(self.params.get('append_scylla_yaml') or '')
return setup_details

def get_test_details(self):
Expand Down
3 changes: 2 additions & 1 deletion unit_tests/test_scylla_yaml.py
Original file line number Diff line number Diff line change
Expand Up @@ -398,7 +398,8 @@ def test_scylla_yaml(self):
'virtual_dirty_soft_limit': None,
'volatile_system_keyspace_for_testing': None,
'workdir': None,
'write_request_timeout_in_ms': None
'write_request_timeout_in_ms': None,
'enable_tablets': None,
}
)

Expand Down

0 comments on commit 60cddb4

Please sign in to comment.