-
Notifications
You must be signed in to change notification settings - Fork 147
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
Add tests for ccpp_prebuild step #972
Merged
Merged
Changes from all commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
6a726ba
Initial commit
2437545
update CI
a3a049e
update CI
b26eca1
update CI
8cec93b
update CI
aa2664d
update CI
6a69e7c
update CI for fv3
3db6215
update CI for fv3
7aa47f9
update CI for fv3
bb140fc
update CI for fv3
401122b
update CI for fv3
ef273f5
update CI for fv3
b71284a
update CI for fv3
cd3d206
update to CI
ded8ce9
update to CI
b5767ca
update to CI
0385133
update to CI
f21feec
update to CI
a3e74bf
update to CI
b2f5c4d
update to CI
cdbf2f5
update to CI
1aff1b7
update to CI
6120509
update to CI
ecfe481
update to CI
142c799
update to CI
3a0641e
update to CI
716900f
update to CI
a11d566
update to CI
9d93c0c
update to CI
071cb33
update to CI
8855fd5
update to CI
6e0ee28
update to CI
1eb1740
update to CI
8a4ea15
update to CI
defc268
update to CI
6358b56
update to CI
c2d6feb
update to CI
bb3cbd9
update to CI
ed86940
update to CI
e66d159
update to CI
b2cd9aa
update to CI
8ba4760
update to CI
61f9670
Housekeeping
dustinswales ac43a97
Housekeeping
dustinswales 4f789c6
Update CI
dustinswales e84694e
Update CI
dustinswales 5e02827
Update CI
dustinswales e64a754
Update CI
dustinswales 3098318
Update CI
dustinswales 4ba2ba4
Update CI
dustinswales ba2dbbd
Update CI
dustinswales 1fb225b
Update CI
dustinswales 66212c0
Update CI
dustinswales b3f68a0
Update CI
dustinswales 07add40
Update CI
dustinswales File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,58 @@ | ||
name: CI test to run FV3 ccpp_prebuild step | ||
|
||
on: [push, pull_request] | ||
|
||
jobs: | ||
build-linux: | ||
|
||
# The type of runner that the job will run on | ||
runs-on: ubuntu-latest | ||
strategy: | ||
max-parallel: 5 | ||
|
||
steps: | ||
- name: Checkout current ccpp-physics code | ||
uses: actions/checkout@v3 | ||
with: | ||
ref: ${{ github.event.pull_request.head.sha }} | ||
|
||
- name: Store remote-URL for current ccpp-physics code | ||
run: echo "GIT_REMOTE_URL=`git remote get-url origin`" >> $GITHUB_ENV | ||
|
||
- name: Store branch name for current ccpp-physics code | ||
run: echo "GIT_REMOTE_BRANCH=`git rev-parse --abbrev-ref HEAD`" >> $GITHUB_ENV | ||
|
||
- name: Store hash for HEAD of current ccpp-physics code | ||
run: echo "GIT_REMOTE_HASH=`git rev-parse HEAD`" >> $GITHUB_ENV | ||
|
||
- name: Checkout latest fv3atm | ||
run: git clone https://github.com/NOAA-EMC/fv3atm.git | ||
|
||
- name: Initialize submodules | ||
run: | | ||
cd /home/runner/work/ccpp-physics/ccpp-physics/fv3atm | ||
git submodule update --init --recursive | ||
|
||
- name: Update ccpp-physics hash in fv3atm | ||
if: github.event.pull_request == false | ||
run: | | ||
cd /home/runner/work/ccpp-physics/ccpp-physics/fv3atm/ccpp/physics | ||
git remote add remote_local $GIT_REMOTE_URL | ||
git fetch remote_local $GIT_REMOTE_BRANCH | ||
git checkout remote_local/$GIT_REMOTE_BRANCH | ||
|
||
- name: Set up Python 3.8.5 | ||
uses: actions/setup-python@v3 | ||
with: | ||
python-version: 3.8.5 | ||
|
||
- name: Add conda to system path | ||
run: | | ||
# $CONDA is an environment variable pointing to the root of the miniconda directory | ||
echo $CONDA/bin >> $GITHUB_PATH | ||
|
||
- name: Run ccpp_prebuild.py | ||
run: | | ||
cd /home/runner/work/ccpp-physics/ccpp-physics/fv3atm/ccpp/ | ||
mkdir -p /home/runner/work/ccpp-physics/ccpp-physics/fv3atm/bin/ccpp/physics/physics/ | ||
./framework/scripts/ccpp_prebuild.py --config config/ccpp_prebuild_config.py | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,63 @@ | ||
name: CI test to run SCM ccpp_prebuild step | ||
|
||
on: [push, pull_request] | ||
|
||
jobs: | ||
build-linux: | ||
|
||
# The type of runner that the job will run on | ||
runs-on: ubuntu-latest | ||
strategy: | ||
max-parallel: 5 | ||
|
||
steps: | ||
|
||
- name: Checkout current ccpp-physics code | ||
uses: actions/checkout@v3 | ||
|
||
- name: Store remote-URL for current ccpp-physics code | ||
run: echo "GIT_REMOTE_URL=`git remote get-url origin`" >> $GITHUB_ENV | ||
|
||
- name: Store branch name for current ccpp-physics code | ||
run: echo "GIT_REMOTE_BRANCH=`git rev-parse --abbrev-ref HEAD`" >> $GITHUB_ENV | ||
|
||
- name: Store hash for HEAD of current ccpp-physics code | ||
run: echo "GIT_REMOTE_HASH=`git rev-parse HEAD`" >> $GITHUB_ENV | ||
|
||
- name: Checkout latest ccpp-scm code | ||
run: git clone https://github.com/NCAR/ccpp-scm.git | ||
|
||
- name: Initialize submodules | ||
run: | | ||
cd /home/runner/work/ccpp-physics/ccpp-physics/ccpp-scm | ||
git submodule update --init --recursive | ||
|
||
- name: Update ccpp-physics hash in ccpp-scm | ||
if: github.event.pull_request == false | ||
run: | | ||
cd /home/runner/work/ccpp-physics/ccpp-physics/ccpp-scm/ccpp/physics | ||
echo $GIT_REMOTE_URL | ||
echo $GIT_REMOTE_BRANCH | ||
echo ${{github.repository}} | ||
echo ${{ github.event.pull_request.head.sha }} | ||
echo $GITHUB_SHA | ||
git remote add remote_local $GIT_REMOTE_URL | ||
git fetch remote_local $GIT_REMOTE_BRANCH | ||
git checkout remote_local/$GIT_REMOTE_BRANCH | ||
|
||
- name: Set up Python 3.8.5 | ||
uses: actions/setup-python@v3 | ||
with: | ||
python-version: 3.8.5 | ||
|
||
- name: Add conda to system path | ||
run: | | ||
# $CONDA is an environment variable pointing to the root of the miniconda directory | ||
echo $CONDA/bin >> $GITHUB_PATH | ||
|
||
- name: Run ccpp_prebuild.py | ||
run: | | ||
cd /home/runner/work/ccpp-physics/ccpp-physics/ccpp-scm/ | ||
git status | ||
mkdir -p /home/runner/work/ccpp-physics/ccpp-physics/ccpp-scm/scm/bin/ccpp/physics/physics/ | ||
./ccpp/framework/scripts/ccpp_prebuild.py --config ccpp/config/ccpp_prebuild_config.py |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
For the first pass at this, I don't see a problem with parsing all suites, but we may want to define a shorter list to parse at some point to make these tests run faster?
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.
That sounds like a good idea. I'll add a note for us to discuss this.