cassandra-pr mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cclauss <...@git.apache.org>
Subject [GitHub] cassandra-dtest pull request #25: Define logger in upgrade_crc_check_chance_...
Date Sun, 06 May 2018 08:44:18 GMT
GitHub user cclauss opened a pull request:

    https://github.com/apache/cassandra-dtest/pull/25

    Define logger in upgrade_crc_check_chance_test.py

    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
    ```

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/cclauss/cassandra-dtest patch-1

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cassandra-dtest/pull/25.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #25
    
----
commit a485db5bf8146fb38187ad42825ba09af54c24ec
Author: cclauss <cclauss@...>
Date:   2018-05-06T08:44:05Z

    Define logger in upgrade_crc_check_chance_test.py
    
    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
    ```

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org


Mime
View raw message