Skip to content

Commit 2e21b43

Browse files
committed
Temporarily focus CI on process-root coverage
- Make non-coverage workflows manual-only while iterating on the native-symbol PR. - Keep coverage active for push and pull_request but run only the process-root integration test. - Preserve focused Python and C++ Codecov uploads for the container native-symbol regression. Signed-off-by: Pramod Kumbhar <prkumbhar@nvidia.com>
1 parent 09f8954 commit 2e21b43

6 files changed

Lines changed: 14 additions & 42 deletions

File tree

.github/workflows/build_wheels.yml

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,7 @@ name: Wheels
33
permissions: {}
44

55
on:
6-
push:
7-
pull_request:
8-
release:
9-
types:
10-
- published
11-
schedule:
12-
# At 12:00 on every day-of-month
13-
- cron: "0 12 */1 * *"
6+
workflow_dispatch:
147

158
concurrency:
169
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}

.github/workflows/coverage.yml

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -5,17 +5,10 @@ permissions:
55

66
on:
77
push:
8-
branches:
9-
- main
108
pull_request:
119
branches:
1210
- main
13-
release:
14-
types:
15-
- published
16-
schedule:
17-
# At 12:00 on every day-of-month
18-
- cron: "0 12 */1 * *"
11+
workflow_dispatch:
1912

2013
concurrency:
2114
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
@@ -45,6 +38,7 @@ jobs:
4538
ninja-build \
4639
libdw-dev \
4740
libelf-dev \
41+
util-linux \
4842
python3.10-dev \
4943
python3.10-dbg
5044
- name: Install Python dependencies
@@ -54,26 +48,26 @@ jobs:
5448
- name: Disable ptrace security restrictions
5549
run: |
5650
echo 0 | sudo tee /proc/sys/kernel/yama/ptrace_scope
57-
- name: Require process-root native-symbol test support
51+
- name: Prepare process-root native-symbol test support
5852
run: |
5953
sudo sysctl -w kernel.unprivileged_userns_clone=1 || true
6054
sudo sysctl -w user.max_user_namespaces=15000 || true
6155
sudo sysctl -w kernel.apparmor_restrict_unprivileged_userns=0 || true
6256
unshare --user --map-root-user --mount --propagation private true
63-
PYSTACK_REQUIRE_PROCESS_ROOT_TEST=1 \
64-
python3 -m pytest -vvv -s tests/integration/test_native_process_root.py
6557
- name: Compute Python coverage
6658
run: |
67-
python3 -m pytest -vvv --log-cli-level=info -s --color=yes \
59+
PYSTACK_REQUIRE_PROCESS_ROOT_TEST=1 \
60+
python3 -m pytest -vvv --log-cli-level=info -s --color=yes \
6861
--cov=pystack --cov=tests --cov-config=pyproject.toml --cov-report=term \
69-
--cov-append tests --cov-fail-under=85
62+
--cov-append tests/integration/test_native_process_root.py
7063
python3 -m coverage lcov -i -o pycoverage.lcov
7164
genhtml *coverage.lcov --branch-coverage --output-directory pystack-coverage
7265
- name: Compute C++ coverage
7366
run: |
7467
rm -rf build
7568
CFLAGS="-O0 -pg --coverage" CXXFLAGS="-O0 -pg --coverage" SKBUILD_BUILD_DIR=build pip install -e . --no-build-isolation
76-
python3 -m pytest tests -v
69+
PYSTACK_REQUIRE_PROCESS_ROOT_TEST=1 \
70+
python3 -m pytest -vvv -s tests/integration/test_native_process_root.py
7771
find build -name "*.gcda" -o -name "*.gcno" | head -5
7872
lcov --capture --directory build --output-file cppcoverage.lcov
7973
lcov --extract cppcoverage.lcov '*/src/pystack/_pystack/*' --output-file cppcoverage.lcov

.github/workflows/docs.yml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,7 @@ name: Publish docs
33
permissions: {}
44

55
on:
6-
push:
7-
branches:
8-
- main
6+
workflow_dispatch:
97

108
jobs:
119
publish_docs:

.github/workflows/lint_and_docs.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@ name: Lint and docs
22

33
permissions: {}
44

5-
on: [push, pull_request]
5+
on:
6+
workflow_dispatch:
67

78
jobs:
89
lint_and_docs:

.github/workflows/news-check.yml

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,7 @@ name: News entry check
33
permissions: {}
44

55
on:
6-
pull_request:
7-
paths:
8-
- "src/**"
9-
types:
10-
- "opened"
11-
- "reopened"
12-
- "synchronize"
13-
- "labeled"
14-
- "unlabeled"
6+
workflow_dispatch:
157

168
jobs:
179
news_entry_check:

.github/workflows/sanity-check.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,7 @@ name: Sanity check
33
permissions: {}
44

55
on:
6-
pull_request:
7-
types:
8-
- "opened"
9-
- "reopened"
10-
- "synchronize"
11-
- "labeled"
12-
- "unlabeled"
6+
workflow_dispatch:
137

148
jobs:
159
commits_check_job:

0 commit comments

Comments
 (0)