Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23840 )
Change subject: IMPALA-14667: Remove reference to pytest.config ...................................................................... IMPALA-14667: Remove reference to pytest.config pytest.config global variable has been removed since pytest 5.0. https://docs.pytest.org/en/stable/deprecations.html#pytest-config-global This patch remove direct reference to pytest.config. Instead, it hook pytest config object to ImpalaTestClusterProperties singleton at pytest_configure() method at conftest.py. Added pytest_config() classmethod in BaseTestSuite as a shortcut to access ImpalaTestClusterProperties.get_instance().pytest_config(). Testing: - Pass core tests. Change-Id: Ieb09a9c8c1f474c355188b46a0ea913da6c2a38b Reviewed-on: http://gerrit.cloudera.org:8080/23840 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M tests/catalog_service/test_catalog_service_client.py M tests/common/base_test_suite.py M tests/common/custom_cluster_test_suite.py M tests/common/environ.py M tests/common/impala_test_suite.py M tests/common/kudu_test_suite.py M tests/common/skip.py M tests/common/test_dimensions.py M tests/conftest.py M tests/custom_cluster/test_event_processing_perf.py M tests/custom_cluster/test_insert_behaviour.py M tests/custom_cluster/test_kudu.py M tests/custom_cluster/test_saml2_sso.py M tests/metadata/test_partition_metadata.py M tests/query_test/test_kudu.py M tests/run-tests.py M tests/shell/util.py 17 files changed, 165 insertions(+), 113 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/23840 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ieb09a9c8c1f474c355188b46a0ea913da6c2a38b Gerrit-Change-Number: 23840 Gerrit-PatchSet: 4 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
