-
Notifications
You must be signed in to change notification settings - Fork 35
CI: test all PnetCDF test programs #874
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
base: main
Are you sure you want to change the base?
Conversation
3ca71ff
to
7f2d91d
Compare
Note this CI failed due to a bug in #873 |
run: | | ||
cd ${GITHUB_WORKSPACE}/PnetCDF/pnetcdf-${PNETCDF_VERSION} | ||
make -s LIBTOOLFLAGS=--silent V=1 -j 8 tests > qout 2>&1 | ||
export LD_PRELOAD=${GITHUB_WORKSPACE}/darshan_install/lib/libdarshan.so |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add simple logic here to test for existence of expected number of Darshan logs for these tests or something? You can see in the workflow output that no log files are created because we never set the DARSHAN_LOGPATH env variable. I think the least we could do is assert that the expected number of logs was generated (and that the log path is correctly set so Darshan can write them).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setting DARSHAN_LOGPATH has been added.
How does Darshah check the number of log files?
Note the number of PnetCDF test programs can increase in new releases.
run: | | ||
cd ${GITHUB_WORKSPACE}/PnetCDF/pnetcdf-${PNETCDF_VERSION} | ||
export LD_PRELOAD=${GITHUB_WORKSPACE}/darshan_install/lib/libdarshan.so | ||
make ptest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check log files are correctly generated here, too? Later on, we could add a script that checks more in-depth details as part of the workflow, but something simple is fine for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How does one check whether a Darshan log file is correct or not?
Download a PnetCDF official release and run its make check and make ptest when Darshan library is pre-loaded.
4a2491d
to
399cf1b
Compare
Download PnetCDF release and run its
make check
andmake ptest
when Darshan library is pre-loaded.