Skip to content

Define logger in upgrade_crc_check_chance_test.py#25

Closed
cclauss wants to merge 1 commit intoapache:masterfrom
cclauss:patch-1
Closed

Define logger in upgrade_crc_check_chance_test.py#25
cclauss wants to merge 1 commit intoapache:masterfrom
cclauss:patch-1

Conversation

@cclauss
Copy link
Copy Markdown

@cclauss cclauss commented May 6, 2018

Resolves undefined names on lines 105, 111, 127, 132, 135, 137, 140, 145, and 147

flake8 testing of https://github.com/apache/cassandra-dtest on Python 3.6.3

$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics

./auth_join_ring_false_test.py:188:13: F821 undefined name 'assert_regexp_matches'
            assert_regexp_matches(repr(cm._excinfo[1]), message)
            ^
./upgrade_crc_check_chance_test.py:101:9: F821 undefined name 'logger'
        logger.debug('Test completed successfully')
        ^
./upgrade_crc_check_chance_test.py:107:9: F821 undefined name 'logger'
        logger.debug(meta.options['compression_parameters'])
        ^
./upgrade_crc_check_chance_test.py:123:9: F821 undefined name 'logger'
        logger.debug('Upgrading node {node} to {tag}'.format(**format_args))
        ^
./upgrade_crc_check_chance_test.py:128:9: F821 undefined name 'logger'
        logger.debug('{node} stopped'.format(**format_args))
        ^
./upgrade_crc_check_chance_test.py:131:9: F821 undefined name 'logger'
        logger.debug('Updating version to tag {tag}'.format(**format_args))
        ^
./upgrade_crc_check_chance_test.py:133:9: F821 undefined name 'logger'
        logger.debug('Set new cassandra dir for {node}: {tag}'.format(**format_args))
        ^
./upgrade_crc_check_chance_test.py:136:9: F821 undefined name 'logger'
        logger.debug('Starting {node} on new version ({tag})'.format(**format_args))
        ^
./upgrade_crc_check_chance_test.py:141:9: F821 undefined name 'logger'
        logger.debug('Running upgradesstables')
        ^
./upgrade_crc_check_chance_test.py:143:9: F821 undefined name 'logger'
        logger.debug('Upgrade of {node} complete'.format(**format_args))
        ^
10    F821 undefined name 'assert_regexp_matches'
10

Resolves _undefined names_ on lines 105, 111, 127, 132, 135, 137, 140, 145, and 147

flake8 testing of https://github.com/apache/cassandra-dtest on Python 3.6.3

$ __flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics__
```
./auth_join_ring_false_test.py:188:13: F821 undefined name 'assert_regexp_matches'
            assert_regexp_matches(repr(cm._excinfo[1]), message)
            ^
./upgrade_crc_check_chance_test.py:101:9: F821 undefined name 'logger'
        logger.debug('Test completed successfully')
        ^
./upgrade_crc_check_chance_test.py:107:9: F821 undefined name 'logger'
        logger.debug(meta.options['compression_parameters'])
        ^
./upgrade_crc_check_chance_test.py:123:9: F821 undefined name 'logger'
        logger.debug('Upgrading node {node} to {tag}'.format(**format_args))
        ^
./upgrade_crc_check_chance_test.py:128:9: F821 undefined name 'logger'
        logger.debug('{node} stopped'.format(**format_args))
        ^
./upgrade_crc_check_chance_test.py:131:9: F821 undefined name 'logger'
        logger.debug('Updating version to tag {tag}'.format(**format_args))
        ^
./upgrade_crc_check_chance_test.py:133:9: F821 undefined name 'logger'
        logger.debug('Set new cassandra dir for {node}: {tag}'.format(**format_args))
        ^
./upgrade_crc_check_chance_test.py:136:9: F821 undefined name 'logger'
        logger.debug('Starting {node} on new version ({tag})'.format(**format_args))
        ^
./upgrade_crc_check_chance_test.py:141:9: F821 undefined name 'logger'
        logger.debug('Running upgradesstables')
        ^
./upgrade_crc_check_chance_test.py:143:9: F821 undefined name 'logger'
        logger.debug('Upgrade of {node} complete'.format(**format_args))
        ^
10    F821 undefined name 'assert_regexp_matches'
10
```
@cclauss cclauss closed this Nov 24, 2019
@cclauss cclauss deleted the patch-1 branch November 24, 2019 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant