Untitled

Similalry can you also remove hard coding from this?
def collect_db_logs(self, instance_id, workflow_path, test_start_timestamp):
        """
        Gather rds db diagnostics logs
        """
        credentials = self.get_pas_credentials(PAS_ADMIN_OPS_ROLE)
        cmds = self.get_rds_s3_file_transfer_template(
            credentials["AccessKeyId"], credentials["SecretAccessKey"]
        )
        cmds.append(
            "aws s3 cp "
            + f"{self.base_s3_location}/aurora-db-logs/test-gather-aurora-db-logs.sh /tmp "
        )
        cmds.append("sudo chmod +x /tmp/test-gather-aurora-db-logs.sh")

        if test_start_timestamp:
            test_start_timestamp = test_start_timestamp.strftime("%Y-%m-%d %H:%M:%S")
        cmds.append(
            f"sudo bash /tmp/test-gather-aurora-db-logs.sh -t '{test_start_timestamp}' -z aurora-perf-analysis-rdsdbdiagnostic -x /rdsdbdata/tmp/seed -x /rdsdbdata/db/pg_stat_tmp -x /rdsdbdata/tmp_lctn_on -x /rdsdbdata/tmp/mysql/mysql.ibd -x /rdsdbdata/tmp/pas /rdsdbdata"
        )
        cmds.append(
            "aws s3 cp /tmp/aurora-perf-analysis-rdsdbdiagnostic.tar.gz "
            + f"{get_pas_logs_s3_dir_for_instance(workflow_path, instance_id[0].strip())}"
        )
        self.__ssm_client.run_cmd(
            instance_id[1],
            cmds,
        )