Skip to content

Commit

Permalink
Set Test Colo DB Pinning to the Intended Range (#312)
Browse files Browse the repository at this point in the history
Set DB CPU pinning to the intended range for test

[ committed by @MattToast ]
[ reviewed by @mellis13 ]
  • Loading branch information
MattToast authored Jul 3, 2023
1 parent 200bc01 commit 288ed40
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/build_docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,12 +55,12 @@ jobs:
cp -r doc-branch/docs/* ./docs/
rm -rf doc-branch
- name: build documentation with docker
- name: Build documentation with docker
run: make docks

- name: Commit files
run: |
git add -A
git add -Af
git config --local user.email "action@github.com"
git config --local user.name "GitHub Action"
git commit -m "Update develop documentation"
Expand Down
4 changes: 2 additions & 2 deletions tests/on_wlm/test_colocated_model.py
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ def test_colocated_model_pinning_range(fileutils, coloutils, db_type):

db_args = {
"db_cpus": 4,
"custom_pinning": range(5)
"custom_pinning": range(4)
}

colo_model = coloutils.setup_test_colo(
Expand Down Expand Up @@ -174,4 +174,4 @@ def test_colocated_model_pinning_mixed(fileutils, coloutils, db_type):
assert colo_model.run_settings.colocated_db_settings["custom_pinning"] == "0,1,3"
exp.start(colo_model, block=True)
statuses = exp.get_status(colo_model)
assert all([stat == status.STATUS_COMPLETED for stat in statuses])
assert all([stat == status.STATUS_COMPLETED for stat in statuses])

0 comments on commit 288ed40

Please sign in to comment.