Don't use f-strings during logging

[adswa]

I'm recording a comment of @mih in the chat because it applies to metalad:

logging should not use f-strings

As also outlined in the Design document:

A message can be a string or a tuple. In case of a tuple, the second item can contain values for %-expansion of the message string. Expansion is performed only immediately prior to actually outputting the message, hence string formatting runtime costs can be avoided this way, if a message is not actually shown.

So code like

            return dict(
                action="meta_add",
                status="error",
                path=str(add_parameter.result_path),
                message=f'value of "dataset-id" '
                        f'({add_parameter.dataset_id}) does not match '
                        f'ID of dataset at {dataset_path} ({dataset_id})')

should be adjusted to not use f-strings.