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

CI improvements #256

Merged

Conversation

scottwittenburg
Copy link
Collaborator

No description provided.

@scottwittenburg scottwittenburg force-pushed the more-image-improvements branch 29 times, most recently from 477222d to aabbf58 Compare September 4, 2017 17:57
Added mpi4py to build image and removed unwanted ohpc numpy package.

Use github api to retrieve real branch name, use that along with other
information to generate a meaningful build name.  To keep this from
happening too often, we only attempt to retrieve the branch name once
for each job during the first step, then communicate that name to
subsequent steps via an environment variable.  This required that we
stop using --login option when running the job steps from circleci,
as the lmod environment setup uses the same variable (BASH_ENV) to
store the path to the profile setup file that CircleCI sources for
each "run" in the job steps.

Use github api and the robot user oauth key (from CircleCI env) to post
an extra status message on each tested commit pointing to the CDash results.
By also adding an "update" step to the default build/test job and using
CTEST_UPDATE_VERSION_ONLY=TRUE, we can generate a deep link to the CDash
results specific to the appropriate commit.
@scottwittenburg scottwittenburg changed the title WIP: More image improvements CI improvements Sep 4, 2017
@chuckatkins chuckatkins merged commit 7511433 into ornladios:master Sep 5, 2017
@scottwittenburg scottwittenburg deleted the more-image-improvements branch September 21, 2017 16:03
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.

2 participants