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

TravisCI: add centos build. #265

Merged
merged 1 commit into from
Aug 9, 2020
Merged

Conversation

fghoussen
Copy link
Collaborator

Pull request purpose

centos build

@fghoussen fghoussen force-pushed the centos branch 6 times, most recently from 92dd238 to db8c3d0 Compare June 5, 2020 20:51
@fghoussen
Copy link
Collaborator Author

got similar pb on centos. no idea why ?! @dschwoerer : any clue ?

@dschwoerer
Copy link
Contributor

It seems you are missing the *-devel files, namely openmpi-devel, blas-devel, lapack-devel

@fghoussen
Copy link
Collaborator Author

fghoussen commented Jun 7, 2020

already tried that... but this was before upgrade. let's see

@dschwoerer
Copy link
Contributor

Just wanted to note some additional repositories might be required, for centos 8: dnf config-manager --set-enabled PowerTools is required for epel.
https://fedoraproject.org/wiki/EPEL#How_can_I_use_these_extra_packages.3F

@fghoussen
Copy link
Collaborator Author

not used to centos. trying tricks provided by collegues. will give up if dead end.

@fghoussen
Copy link
Collaborator Author

fghoussen commented Jun 8, 2020

openmpi KO, mpich OK. @dschwoerer : thanks !

@coveralls
Copy link

coveralls commented Jun 8, 2020

Pull Request Test Coverage Report for Build 1311

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 70.611%

Totals Coverage Status
Change from base Build 1309: 0.0%
Covered Lines: 16513
Relevant Lines: 23386

💛 - Coveralls

@sylvestre
Copy link
Contributor

Excellent, you could probably add this to the release notes?
CI: Support for centos added in automation

@fghoussen
Copy link
Collaborator Author

I feel always using same compilersgcc and openmpi which breaks here... Try to add another build with autotools + clang ?...

@fghoussen
Copy link
Collaborator Author

openmpi sucks on centos... no better than this

@dschwoerer
Copy link
Contributor

What issues are you having with openmpi on centos?

@fghoussen
Copy link
Collaborator Author

@dschwoerer : tests crash. don't know why. added error log tail in hope to get traces

@fghoussen
Copy link
Collaborator Author

fghoussen commented Jun 11, 2020

mpirun has detected an attempt to run as root. Running as root is strongly discouraged as any mistake (e.g., in defining TMPDIR) or bug can result in catastrophic damage to the OS file system, leaving your system in an unusable state. We strongly suggest that you run mpirun as a non-root user. You can override this protection by adding the --allow-run-as-root option

Oh men ! Already hit this problem : openmpi crashes because it's doesn't like been ran by root and you must be root in docker. openmpi could be used with that kind of stuff

sed -e 's/LOG_FLAGS = /LOG_FLAGS = --allow-run-as-root --oversubscribe /' -i PARPACK/EXAMPLES/MPI/Makefile.am && \
on CMakeLists.txt. Something like sed -e 's/mpirun /mpirun --allow-run-as-root --oversubscribe /' -i CMakeLists.txt should work before cmake ...

@dschwoerer
Copy link
Contributor

That is however not the only problem, see #268 for more issues.
https://travis-ci.org/github/opencollab/arpack-ng/jobs/697378825

Unfortunately I don't know what causes this.

@fghoussen
Copy link
Collaborator Author

@fghoussen
Copy link
Collaborator Author

BTW, as openmpi is broken, reset-hard to mpich.

@sylvestre sylvestre merged commit 4f993fd into opencollab:master Aug 9, 2020
@fghoussen fghoussen deleted the centos branch August 9, 2020 15:59
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.

4 participants