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

gurwls223 pushed a commit to branch branch-3.2
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.2 by this push:
     new efd33b7  [SPARK-36441][INFRA] Fix GA failure related to downloading 
lintr dependencies
efd33b7 is described below

commit efd33b72ca781ca49e44f5b483522a004ece7623
Author: Kousuke Saruta <[email protected]>
AuthorDate: Fri Aug 6 10:49:27 2021 +0900

    [SPARK-36441][INFRA] Fix GA failure related to downloading lintr 
dependencies
    
    ### What changes were proposed in this pull request?
    
    This PR fixes a GA failure which is related to downloading lintr 
dependencies.
    ```
     * installing *source* package ‘devtools’ ...
    ** package ‘devtools’ successfully unpacked and MD5 sums checked
    ** using staged installation
    ** R
    ** inst
    ** byte-compile and prepare package for lazy loading
    ** help
    *** installing help indices
    *** copying figures
    ** building package indices
    ** installing vignettes
    ** testing if installed package can be loaded from temporary location
    ** testing if installed package can be loaded from final location
    ** testing if installed package keeps a record of temporary installation 
path
    * DONE (devtools)
    
    The downloaded source packages are in
        ‘/tmp/Rtmpv53Ix4/downloaded_packages’
    Using bundled GitHub PAT. Please add your own PAT to the env var 
`GITHUB_PAT`
    Error: Failed to install 'unknown package' from GitHub:
      HTTP error 401.
      Bad credentials
    ```
    
    I re-triggered the GA job but it still fail with the same error.
    https://github.com/apache/spark/runs/3257853825
    
    The issue seems to happen when downloading lintr dependencies from GitHub. 
So, the solution is to change the way to download them.
    
    ### Why are the changes needed?
    
    To recover GA.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    GA itself.
    
    Closes #33660 from sarutak/fix-r-package-issue.
    
    Authored-by: Kousuke Saruta <[email protected]>
    Signed-off-by: Hyukjin Kwon <[email protected]>
    (cherry picked from commit e13bd586f1854df872f32ebd3c7686fc4013e06e)
    Signed-off-by: Hyukjin Kwon <[email protected]>
---
 .github/workflows/build_and_test.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/build_and_test.yml 
b/.github/workflows/build_and_test.yml
index 1d3d9c5..b518f87 100644
--- a/.github/workflows/build_and_test.yml
+++ b/.github/workflows/build_and_test.yml
@@ -363,7 +363,7 @@ jobs:
       run: |
         apt-get install -y libcurl4-openssl-dev libgit2-dev libssl-dev 
libxml2-dev
         Rscript -e "install.packages(c('devtools'), 
repos='https://cloud.r-project.org/')"
-        Rscript -e "devtools::install_github('jimhester/[email protected]')"
+        Rscript -e "devtools::install_version('lintr', version='2.0.1', 
repos='https://cloud.r-project.org')"
         ./R/install-dev.sh
     - name: Instll JavaScript linter dependencies
       run: |

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to