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.

Sign up for GitHub

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

Jump to bottom

Re-factor of Travis-CI usage #909

Merged
merged 1 commit into from
Apr 17, 2019
Merged

Re-factor of Travis-CI usage #909

merged 1 commit into from
Apr 17, 2019

Conversation

adcroft
Copy link
Collaborator

@adcroft adcroft commented Apr 16, 2019

  • Uses stages to parallelize testing process
  • Adds a doxygen job that tests for doxygen errors
  • Reduces dependency on other repositories
    • No longer uses scripts from MOM6-examples/tools/tests/Travis-MOM6/
    • Still uses configurations from MOM6-examples/ocean_only/
  • Adds test building/using a debug executable
  • Adds a .testing/ directory
  • Uses the latest xenial image on Travis (with openmpi)
  • Example of new report is at https://travis-ci.org/adcroft/MOM6/builds/520870218

Todo:

  • Add code coverage (needs more experiments)
  • Break dependence on MOM6-examples (long term project)
  • Add other portable tests (e.g. parameter scaling, rotation, etc)

@adcroft
Re-factor of Travis-CI usage
69e6d30 
- Uses stages to parallelize testing process
- Adds a doxygen job that tests for doxygen errors
- Reduces dependency on other repositories
  - No longer uses scripst from MOM6-examples/tools/tests/Travis-MOM6/
  - Still uses configurations from MOM6-examples/ocean_only/
- Adds test building/using a debug executable
- Adds a .testing/ directory
- Uses the latest xenial image on Travis (with openmpi)

Todo:
- Add code coverage (needs more experiments)
- Break dependence on MOM6-examples (long term project)
- Add other portable tests (e.g. parameter scaling, rotation, etc)
@adcroft adcroft requested a review from marshallward April 16, 2019 17:40
marshallward
marshallward approved these changes Apr 17, 2019
View reviewed changes
Copy link
Collaborator

@marshallward marshallward left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Obviously this is mostly educational for me at this stage, and hard to offer much constructive feedback. But it seems good to me, and great to move more of the test framework over to Travis.

@marshallward marshallward merged commit 91c8083 into mom-ocean:dev/gfdl Apr 17, 2019
@marshallward
Copy link
Collaborator

Looks like I also have permissions to do PR merges.

@adcroft adcroft mentioned this pull request Apr 22, 2019
Merged
@adcroft adcroft deleted the travis-stages branch May 16, 2019 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marshallward marshallward marshallward approved these changes

Assignees

@Hallberg-NOAA Hallberg-NOAA

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adcroft @marshallward @Hallberg-NOAA