[
https://issues.apache.org/jira/browse/CASSANDRA-14059?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16277829#comment-16277829
]
Joel Knighton commented on CASSANDRA-14059:
-------------------------------------------
Follow-up commit here
https://github.com/jkni/cassandra-dtest/commit/179bf9f16a03394aee9715b745bfd4773c5b05ba
It uses string formatting instead of concatenation, so we'll automatically
stringify lists. I smoke tested this on the old set as well as manually on the
tests listed above. I'm running a full set of dtests as well, but this is very
likely no worse than the current situation if there's support for committing it
sooner. Otherwise, I'll commit once tests are clean.
> Root logging formatter broken in dtests
> ---------------------------------------
>
> Key: CASSANDRA-14059
> URL: https://issues.apache.org/jira/browse/CASSANDRA-14059
> Project: Cassandra
> Issue Type: Bug
> Components: Testing
> Reporter: Joel Knighton
> Assignee: Joel Knighton
> Priority: Minor
>
> Since the ccm dependency in dtest was bumped to {{3.1.0}} in
> {{7cc06a086f89ed76499837558ff263d84337acba}}, when dtests are run with
> --nologcapture, errors of the following form are printed:
> {code}
> Traceback (most recent call last):
> File "/usr/lib64/python2.7/logging/__init__.py", line 861, in emit
> msg = self.format(record)
> File "/usr/lib64/python2.7/logging/__init__.py", line 734, in format
> return fmt.format(record)
> File "/usr/lib64/python2.7/logging/__init__.py", line 469, in format
> s = self._fmt % record.__dict__
> KeyError: 'current_test'
> Logged from file dtest.py, line 485
> {code}
> This is because CCM no longer installs a basic root logger configuration,
> which is probably a more correct behavior than what it did prior to this
> change. Now, dtest installs its own basic root logger configuration which
> writes to 'dtest.log' using the formatter {{'%(asctime)s,%(msecs)d %(name)s
> %(current_test)s %(levelname)s %(message)s'}}. This means that anything
> logging a message must provide the current_test key in its extras map. The
> dtest {{debug}} and {{warning}} functions do this, but logging from
> dependencies doesn't, producing these {{KeyError}} s.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
