You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by da...@apache.org on 2024/03/22 13:41:47 UTC

(beam) branch master updated: add pythonpath to cloudcoverage tox environment (#30673)

This is an automated email from the ASF dual-hosted git repository.

damccorm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new 872afa0d7ab add pythonpath to cloudcoverage tox environment (#30673)
872afa0d7ab is described below

commit 872afa0d7ab52e52687939b42e90bd64aafcfa57
Author: Vlado Djerek <ra...@gmail.com>
AuthorDate: Fri Mar 22 14:41:40 2024 +0100

    add pythonpath to cloudcoverage tox environment (#30673)
    
    * add pythonpath to cloudcoverage tox environment
    
    * update codecov-action
    
    ---------
    
    Co-authored-by: Vlado Djerek <dj...@gmail.com>
---
 .github/workflows/beam_PreCommit_Python_Coverage.yml |  4 ++++
 sdks/python/tox.ini                                  | 10 +++-------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/beam_PreCommit_Python_Coverage.yml b/.github/workflows/beam_PreCommit_Python_Coverage.yml
index 16329b87ca1..35aaa426c97 100644
--- a/.github/workflows/beam_PreCommit_Python_Coverage.yml
+++ b/.github/workflows/beam_PreCommit_Python_Coverage.yml
@@ -90,6 +90,10 @@ jobs:
           gradle-command: :sdks:python:test-suites:tox:py38:preCommitPyCoverage
           arguments: |
             -PuseWheelDistribution
+      - uses: codecov/codecov-action@v4
+        with:
+          flags: python
+          directory: sdks/python
       - name: Archive Python Test Results
         uses: actions/upload-artifact@v4
         if: failure()
diff --git a/sdks/python/tox.ini b/sdks/python/tox.ini
index 917eb0bd522..86d12f31a18 100644
--- a/sdks/python/tox.ini
+++ b/sdks/python/tox.ini
@@ -96,17 +96,13 @@ deps =
   pytest-cov==3.0.0
 # Don't set TMPDIR to avoid "AF_UNIX path too long" errors in certain tests.
 setenv =
+  PYTHONPATH = {toxinidir}
 platform = linux
-passenv = GIT_*,BUILD_*,ghprb*,CHANGE_ID,BRANCH_NAME,JENKINS_*,CODECOV_*
+passenv = GIT_*,BUILD_*,ghprb*,CHANGE_ID,BRANCH_NAME,JENKINS_*,CODECOV_*,GITHUB_*
 extras = test,gcp,interactive,dataframe,aws
 commands =
-  -rm .coverage
-  curl -Os https://uploader.codecov.io/latest/linux/codecov
-  chmod +x codecov
   bash {toxinidir}/scripts/run_pytest.sh {envname} "{posargs}" "--cov-report=xml --cov=. --cov-append"
-  ./codecov -F python
-  -rm codecov
-
+  
 [testenv:py38-lint]
 # Don't set TMPDIR to avoid "AF_UNIX path too long" errors in pylint.
 setenv =