Skip to content

Commit

Permalink
Exclude cluster benchmark on linux and fixed bug
Browse files Browse the repository at this point in the history
  • Loading branch information
darrenge committed Sep 6, 2024
1 parent 7e2368c commit 9a22ced
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
5 changes: 5 additions & 0 deletions .github/workflows/ci-bdnbenchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,11 @@ jobs:
framework: [ 'net8.0' ]
configuration: [ 'Release' ]
test: [ 'CI_BDN_Config_RespParseStress.json', 'CI_BDN_Config_RespTsavoriteStress.json', 'CI_BDN_Config_RespClusterBench.json', 'CI_BDN_Config_RespClusterMigrateBench.json', 'CI_BDN_Config_Lua.json' ]
exclude:
- os: ubuntu-latest
test: 'CI_BDN_Config_RespClusterBench.json'
- os: ubuntu-latest
test: 'CI_BDN_Config_RespClusterMigrateBench.json'
steps:
- name: Check out code
uses: actions/checkout@v4
Expand Down
1 change: 1 addition & 0 deletions test/BDNPerfTests/run_bdnperftest.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,7 @@ param ($ResultsLine, $columnNum)
$columns = $ResultsLine.Trim('|').Split('|')
$column = $columns | ForEach-Object { $_.Trim() }
$foundValue = $column[$columnNum].Trim(' us')
$foundValue = $column[$columnNum].Trim(' ns')

# Just double check found results is not NA
if ($foundResultsValue -eq "NA") {
Expand Down

0 comments on commit 9a22ced

Please sign in to comment.