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

fix Abs for FabArray #1004

Merged
merged 1 commit into from
Jun 25, 2020
Merged

fix Abs for FabArray #1004

merged 1 commit into from
Jun 25, 2020

Conversation

WeiqunZhang
Copy link
Member

No description provided.

@WeiqunZhang WeiqunZhang requested a review from atmyers June 11, 2020 17:58
@WeiqunZhang
Copy link
Member Author

macos/AppleClang@11.0 Gfortran@9.3 seems to fail quite often. It seems that sometimes it fails to comile Fortran module. Maybe we should disable Fortran. @ax3l

@ax3l

This comment has been minimized.

@ax3l

This comment has been minimized.

@ax3l
Copy link
Member

ax3l commented Jun 11, 2020

Hm, that's only a warning...
Let's try to trigger it again, I did not save the complete log and it passes now. Generally, on all CI services, macOS is a bit more flaky. Maybe rerunning the check from the Checks tabs when this happens is okay if this is sporadic.

@WeiqunZhang
Copy link
Member Author

This happens again in #1024.

@WeiqunZhang
Copy link
Member Author

And after Ann pushed a new commit, it failed again. That was twice in a row. Is this a cmake issue?

@ax3l
Copy link
Member

ax3l commented Jun 16, 2020

Ouch, can you please save the log file before you restart next time?
In the "Checks" tabs, e.g. here, is a ... symbol that has the option "View raw logs". Maybe just download and add as a text file to the comment so we can find out what's going on.

I suspect it's either a network split, flaky macOS VM runner, VM filesystem issue, brew issue (dependency download for gfortran et al.), or CMake dependency race if we forget something... but that's all speculative until we can see the logs.

Generally speaking, if we are unlucky its just the flakyness of the VMs we run on. In my experience, 1:20 on bad days to 1:50 runners on Travis CI error due to network/VM issues. There is a lot of booting and downloading from third parties going on for partitioning the runners, it's way more than starting a local docker image. GitHub CI is significantly better on Linux than travis, but maybe the macOS images are flaky. Restarting or ignoring in such a case and checking for maintenance windows on the respective status pages would then be okay, as well as excluding (or ignoring states of) known unstable images if we cannot figure out what's off from the logs in the long term.

@WeiqunZhang
Copy link
Member Author

The log is still there. https://github.com/AMReX-Codes/amrex/runs/777512636?check_suite_focus=true

@ax3l
Copy link
Member

ax3l commented Jun 16, 2020

Oh awesome, thanks! I collect more in #1025.
Will take a quick look, I agree this looks a bit like a missing declared CMake dependency for Fortran modules in some way or something macOS specific with Fortran modules.

@atmyers atmyers merged commit 87c998a into AMReX-Codes:development Jun 25, 2020
@WeiqunZhang WeiqunZhang deleted the fix_abs branch July 13, 2020 23:17
dwillcox pushed a commit to dwillcox/amrex that referenced this pull request Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants