Update rocprofiler-compute workflows and CODEOWNERS (#221)

Update super repo codeowners- add Pratik to any doc changes for rocprofiler-compute.
Remove rocprofiler-compute-mi-rhel9.yml from super repo workflow folder- unused and out of date, coverage is already in RHEL8 workflow.
Remove CODEOWNERS, dependabot.yml, and all workflows from the projects/rocprofiler-compute/ project- confirmed these do not do anything anymore after being copied into super repo's main ./github/ section.

Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>

* Add a branch filter for push events

* Update .github/CODEOWNERS

Use reviewers team instead of using the individual profiles.

Co-authored-by: David Galiffi <David.Galiffi@amd.com>

Update owners section for rocprofiler-compute codeowners, fix spacing

Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>

---------

Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
Co-authored-by: David Galiffi <David.Galiffi@amd.com>
This commit is contained in:
cfallows-amd
2025-08-20 11:17:25 -04:00
committed by GitHub
parent f7338717ae
commit 8bc0063f2e
19 changed files with 19 additions and 804 deletions
+5 -5
View File
@@ -20,7 +20,7 @@
/projects/rocm-smi-lib/ @bill-shuzhou-liu @dmitrii-galantsev @charis-poag-amd @oliveiradan
/projects/rocminfo/ @dayatsin-amd @shwetagkhatri
/projects/rocprofiler/ @ammarwa @bgopesh
/projects/rocprofiler-compute/ @coleramos425 @feizheng10 @vedithal-amd @xuchen-amd @cfallows-amd @ywang103-amd @jamessiddeley-amd
/projects/rocprofiler-compute/ @ROCm/rocprof-compute-reviewer # Owners: @ROCm/rocprof-compute-owners, Reviewers: @ROCm/rocprof-compute-reviewer
/projects/rocprofiler-register/ @ammarwa @bgopesh
/projects/rocprofiler-sdk/ @jrmadsen @t-tye
/projects/rocprofiler-systems/ @ROCm/rocprof-sys @jrmadsen
@@ -99,10 +99,10 @@
/projects/rocprofiler/**/*.rst @ammarwa @bgopesh @ROCm/rocm-documentation
/projects/rocprofiler/doc/ @ammarwa @bgopesh @ROCm/rocm-documentation
/projects/rocprofiler-compute/**/*.md @coleramos425 @feizheng10 @vedithal-amd @xuchen-amd @cfallows-amd @ywang103-amd @jamessiddeley-amd @ROCm/rocm-documentation
/projects/rocprofiler-compute/**/*.rst @coleramos425 @feizheng10 @vedithal-amd @xuchen-amd @cfallows-amd @ywang103-amd @jamessiddeley-amd @ROCm/rocm-documentation
/projects/rocprofiler-compute/**/.readthedocs.yaml @coleramos425 @feizheng10 @vedithal-amd @xuchen-amd @cfallows-amd @ywang103-amd @jamessiddeley-amd @ROCm/rocm-documentation
/projects/rocprofiler-compute/docs/ @coleramos425 @feizheng10 @vedithal-amd @xuchen-amd @cfallows-amd @ywang103-amd @jamessiddeley-amd @ROCm/rocm-documentation
/projects/rocprofiler-compute/**/*.md @ROCm/rocprof-compute-reviewer @ROCm/rocm-documentation @prbasyal-amd
/projects/rocprofiler-compute/**/*.rst @ROCm/rocprof-compute-reviewer @ROCm/rocm-documentation @prbasyal-amd
/projects/rocprofiler-compute/**/.readthedocs.yaml @ROCm/rocprof-compute-reviewer @ROCm/rocm-documentation @prbasyal-amd
/projects/rocprofiler-compute/docs/ @ROCm/rocprof-compute-reviewer @ROCm/rocm-documentation @prbasyal-amd
/projects/rocprofiler-register/**/*.md @ammarwa @bgopesh @ROCm/rocm-documentation
/projects/rocprofiler-register/**/*.rst @ammarwa @bgopesh @ROCm/rocm-documentation
@@ -2,6 +2,8 @@ name: rocprofiler-compute Documentation
on:
push:
branches:
- develop
paths:
- 'projects/rocprofiler-compute/docs/archive/docs-2.x/**'
- 'projects/rocprofiler-compute/docs/archive/docs-1.x/**'
@@ -3,6 +3,8 @@ name: rocprofiler-compute Formatting
on:
push:
branches:
- develop
paths:
- 'projects/rocprofiler-compute/**'
pull_request:
@@ -1,128 +0,0 @@
name: rocprofiler-compute mi-rhel9
on:
push:
paths:
- 'projects/rocprofiler-compute/**'
# Allows manual execution
workflow_dispatch:
permissions:
contents: read
checks: write
pull-requests: write
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true
jobs:
profile:
strategy:
matrix:
version: [5.7.1, 6.0.2]
hardware: [mi100, mi200]
profiler: [default, rocprofv2]
exclude:
- profiler: rocprofv2
hardware: mi100
fail-fast: false
runs-on: [mi100, rhel9]
env:
PYTHONPATH: /home1/ciuser/rocprofiler-compute_deps
CI_VISIBLE_DEVICES: 1
name: ROCm v${{ matrix.version }} / ${{ matrix.hardware }} / ${{ matrix.profiler }}
steps:
- name: Checkout
uses: actions/checkout@v4
with:
sparse-checkout: projects/rocprofiler-compute
- name: Python Path
run: echo ${PYTHONPATH}
- name: Setup hardware-specific run details
working-directory: projects/rocprofiler-compute
run: |
if [ ${{ matrix.hardware }} == "mi100" ];then
echo "CI_QUEUE=ci" >> $GITHUB_ENV
echo "CI_ARCH=gfx908" >> $GITHUB_ENV
elif [ ${{ matrix.hardware }} == "mi200" ];then
echo "CI_QUEUE=mi2104x" >> $GITHUB_ENV
echo "CI_ARCH=gfx90a" >> $GITHUB_ENV
else
echo "Unsupported hardware"
exit 1
fi
- name: Setup profiling mode
working-directory: projects/rocprofiler-compute
run: |
if [ ${{ matrix.profiler }} == "rocprofv2" ];then
echo "ROCPROF=rocprofv2" >> $GITHUB_ENV
fi
- name: Install Python collateral (build and test)
working-directory: projects/rocprofiler-compute
run: |
pip3 install -t ${PYTHONPATH} -r requirements.txt
pip3 install -t ${PYTHONPATH} -r requirements-test.txt
- name: Load ROCm ${{ matrix.version}}
working-directory: projects/rocprofiler-compute
run: |
module load cmake
module load rocm/${{ matrix.version }}
echo $PATH > $GITHUB_PATH
echo "LD_LIBRARY_PATH=$LD_LIBRARY_PATH" >> "$GITHUB_ENV"
echo "ROCM_PATH=$ROCM_PATH" >> "$GITHUB_ENV"
- name: Check Environment
working-directory: projects/rocprofiler-compute
run: |
echo "PATH=$PATH"
echo "LD_LIBRARY_PATH=$LD_LIBRARY_PATH"
echo "ROCM_PATH=$ROCM_PATH"
echo "CI_QUEUE=$CI_QUEUE"
echo "CI_ARCH=$CI_ARCH"
echo "CI_VISIBLE_DEVICES=$CI_VISIBLE_DEVICES"
echo "ROCPROF=$ROCPROF"
- name: Configure
working-directory: projects/rocprofiler-compute
run: |
mkdir build
cd build
ml cmake
cmake -DENABLE_TESTS=ON -DCMAKE_HIP_ARCHITECTURES=$CI_ARCH -DENABLE_COVERAGE=ON -DPYTEST_NUMPROCS=8 ..
- name: Build tests and Run [profile] mode
working-directory: projects/rocprofiler-compute
run: |
cd build
make
srun -N 1 -J rocprof-compute -p $CI_QUEUE -t 00:20:00 ctest -j 4 --resource-spec-file ../tests/4gpus.json --verbose -L profile
- name: Run [analyze workloads] mode
working-directory: projects/rocprofiler-compute
if: '!cancelled()'
run: |
cd build
srun -N 1 -J rocprof-compute -p $CI_QUEUE -t 00:10:00 ctest --verbose -R test_analyze_workloads
- name: Run [analyze commands] mode
working-directory: projects/rocprofiler-compute
if: '!cancelled()'
run: |
cd build
srun -N 1 -J rocprof-compute -p $CI_QUEUE -t 00:10:00 ctest --verbose -R test_analyze_commands
- name: Publish Test Results
uses: EnricoMi/publish-unit-test-result-action/linux@v2
if: always()
with:
files: |
projects/rocprofiler-compute/tests/**/test_*.xml
- name: Summarize code coverage
if: always()
run: coverage report
- name: Upload code coverage
uses: zgosalvez/github-actions-report-lcov@v4
if: always()
with:
coverage-files: projects/rocprofiler-compute/tests/coverage.info
minimum-coverage: 35
artifact-name: code-coverage-report-rocm${{ matrix.version }}-${{ matrix.hardware }}-${{ matrix.profiler }}
github-token: ${{ secrets.GITHUB_TOKEN }}
update-comment: true
@@ -2,6 +2,8 @@ name: rocprofiler-compute Packaging
on:
push:
branches:
- develop
paths:
- 'projects/rocprofiler-compute/**'
tags:
@@ -71,4 +73,4 @@ jobs:
draft: False # toggle for debugging
files: |
projects/rocprofiler-computebuild/rocprofiler-compute-${{github.ref_name}}.tar.gz
name: ${{ env.RELEASE_NAME }}
name: ${{ env.RELEASE_NAME }}
@@ -5,6 +5,8 @@ name: rocprofiler-compute RHEL 8/9
# Controls when the workflow will run
on:
push:
branches:
- develop
paths:
- 'projects/rocprofiler-compute/**'
- '!projects/rocprofiler-compute/*.md'
@@ -74,4 +76,4 @@ jobs:
working-directory: projects/rocprofiler-compute
run: |
cd build
ctest --verbose -R test_analyze_workloads
ctest --verbose -R test_analyze_workloads
@@ -2,6 +2,8 @@ name: rocprofiler-compute tarball
on:
push:
branches:
- develop
paths:
- 'projects/rocprofiler-compute/**'
pull_request:
@@ -4,6 +4,8 @@ name: rocprofiler-compute Ubuntu 22.04
on:
push:
branches:
- develop
paths:
- 'projects/rocprofiler-compute/**'
- '!projects/rocprofiler-compute/*.md'