Skip to content

Commit

Permalink
Enable MN test for PG 15 in CI
Browse files Browse the repository at this point in the history
In e343778 we disabled the MN test for PR. This leads to the problem
that a PR can break MN tests and get merged. This PR enables MN test for
one PG version to make the PR author aware of any MN-related problems.
  • Loading branch information
jnidzwetzki committed Nov 21, 2023
1 parent 7b0dda6 commit 7c9099f
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions .github/gh_matrix_builder.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,8 +79,9 @@ def build_debug_config(overrides):
"tsdb_build_args": "-DWARNINGS_AS_ERRORS=ON -DREQUIRE_ALL_TESTS=ON",
}
)
if not pull_request:
base_config["tsdb_build_args"] += " -DENABLE_MULTINODETESTS=ON"
# We use .get() here to be able to handle also configurations without PG versions
if not pull_request or overrides.get("pg") == PG15_LATEST:
base_config["tsdb_build_args"] += " -DENABLE_MULTINODE_TESTS=ON"
base_config.update(overrides)
return base_config

Expand All @@ -100,7 +101,7 @@ def build_release_config(overrides):
}
)
if not pull_request:
release_config["tsdb_build_args"] += " -DENABLE_MULTINODETESTS=ON"
release_config["tsdb_build_args"] += " -DENABLE_MULTINODE_TESTS=ON"
base_config.update(release_config)
base_config.update(overrides)
return base_config
Expand Down Expand Up @@ -157,7 +158,7 @@ def macos_config(overrides):
}
)
if not pull_request:
base_config["tsdb_build_args"] += " -DENABLE_MULTINODETESTS=ON"
base_config["tsdb_build_args"] += " -DENABLE_MULTINODE_TESTS=ON"
base_config.update(overrides)
return base_config

Expand Down Expand Up @@ -237,7 +238,7 @@ def macos_config(overrides):
"dist_gapfill_pushdown-13",
"transparent_decompress_chunk-13",
},
"tsdb_build_args": "-DWARNINGS_AS_ERRORS=ON -DASSERTIONS=ON -DPG_ISOLATION_REGRESS=OFF -DENABLE_MULTINODETESTS=ON",
"tsdb_build_args": "-DWARNINGS_AS_ERRORS=ON -DASSERTIONS=ON -DPG_ISOLATION_REGRESS=OFF -DENABLE_MULTINODE_TESTS=ON",
}
m["include"].append(build_debug_config(pg13_debug_earliest))

Expand Down

0 comments on commit 7c9099f

Please sign in to comment.