Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Testing GitHub actions #29

Closed
wants to merge 105 commits into from
Closed

Testing GitHub actions #29

wants to merge 105 commits into from

Conversation

jameslamb
Copy link
Owner

No description provided.

@jameslamb jameslamb force-pushed the testing-github-actions branch 2 times, most recently from 42e4752 to 85df137 Compare May 26, 2020 02:59
@jameslamb
Copy link
Owner Author

@jameslamb jameslamb force-pushed the testing-github-actions branch from b5a34f8 to f2cf809 Compare May 27, 2020 03:03
@jameslamb jameslamb force-pushed the testing-github-actions branch from 4e21788 to 52ac67a Compare May 27, 2020 03:25
@jameslamb
Copy link
Owner Author

copied from microsoft#3119 (comment) for documentation purposes


Ok I have some evidence of what I suspected to be true:

the way GitHub Actions powershell handles treatment of stderr is nondeterministic

I just pushed a commit that redirects the output of Rscript -e "install.packages(..." to $null. This is an extreme example of suppressing stderr.

Then I pushed 10 empty commits.

commit build MINGW build MSVC build
link link
link link ❌ (ERR1)
link link ❌ (ERR2)
link link ❌ (ERR1) ❌ (ERR2)
link link ❌ (ERR1)
link link ❌ (ERR1)
link link
link link ❌ (ERR1)
link link ❌ (ERR1)
link link ❌ (ERR1)

I really really think these "errors" are just "some code writing benign messages to stderr", and not actual failures of the calls. The two types of errors I encountered are shown below. The most common is what I've called ERR1, which comes from the use of message() inside install.packages(). I have not had success suppressing this output by using install.packages(..., quiet = TRUE, verbose = FALSE).

ERR1

image

ERR2

image

Given this finding, I'm sad to say that at this point, we should not move any of the Windows CI builds too GitHub Actions. It is too unreliable.

I'm planning to freeze this PR and use it in a post on the Github Actions Community Forum. If that post returns any useful information, or GitHub Support ever responds to the ticket I submitted, maybe we'll be able to add GitHub Actions jobs for Windows in the future.

@jameslamb jameslamb closed this Jun 7, 2020
@jameslamb jameslamb deleted the testing-github-actions branch June 7, 2020 19:29
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant