Skip to content

[CP 1326] chore: exclude .claude, tests/pytests, and knowledge folders from mar…#519

Open
ci-penbot-01 wants to merge 1 commit intoROCm:mainfrom
ci-penbot-01:CP.O2O.pensando.gpu-operator.1326.rocm.gpu-operator.main
Open

[CP 1326] chore: exclude .claude, tests/pytests, and knowledge folders from mar…#519
ci-penbot-01 wants to merge 1 commit intoROCm:mainfrom
ci-penbot-01:CP.O2O.pensando.gpu-operator.1326.rocm.gpu-operator.main

Conversation

@ci-penbot-01
Copy link
Copy Markdown
Contributor

cp of pensando/gpu-operator#1326


Source PR Description (pensando/gpu-operator#1326):

…kdown linting

Add .claude, tests/pytests, and knowledge directories to markdownlint ignore patterns to prevent linting of test artifacts and AI-generated documentation.

Cherrypick triggered by: ACP-Automation

…… (#1326)

* chore: exclude .claude, tests/pytests, and knowledge folders from markdown linting

Add .claude, tests/pytests, and knowledge directories to markdownlint ignore patterns to prevent linting of test artifacts and AI-generated documentation.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* docs: remove extra blank line in developer-guide.md

Clean up formatting by removing an unnecessary blank line in the developer guide.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* chore: add OCI, gocheck, and teardown to spellcheck wordlist

Add technical terms to wordlist to resolve spellcheck failures in developer-guide.md.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
(cherry picked from commit 2fc32d2b31a16c76efa5f5727c5b211eabf1a831)
@ci-penbot-01
Copy link
Copy Markdown
Contributor Author

AI-Assisted Cherry-Pick

Source PR: #1326
Target Branch: main

The cherry-pick operation encountered merge conflicts which were resolved automatically using AI assistance.

Files with conflicts (resolved by AI):

  • .markdownlint-cli2.jsonc:5-17
  • docs/contributing/developer-guide.md:102-171
Original conflict in .markdownlint-cli2.jsonc
<<<<<<< HEAD
    "**/.git/**"
=======
    "**/.git/**",
    "**/ci-internal/**",
    "**/docs/cicd/**",
    "**/tests/e2e/**",
    "**/helm-charts-k8s/README.md",
    "**/internal-example/**",
    "**/.claude/**",
    "**/tests/pytests/**",
    "**/knowledge/**"
>>>>>>> 2fc32d2b... chore: exclude .claude, tests/pytests, and knowledge folders from mar… (#1326)
Original conflict in docs/contributing/developer-guide.md
<<<<<<< HEAD
=======
## GPU Operator E2E Tests

The `tests/k8s-e2e/` directory contains an e2e test suite that installs the GPU Operator via Helm and verifies metrics and health. Tests run against a live Kubernetes cluster.

### Prerequisites

- A running Kubernetes cluster with at least one AMD GPU node
- `kubectl` configured (`~/.kube/config` or a custom kubeconfig)
- Docker (to build the test runner image)

### Test runner image

```bash
docker build -t gpu-op-k8s-e2e:latest -f tests/k8s-e2e/Dockerfile.e2e tests/k8s-e2e/

Running tests

Full install + verify + teardown

Pass the helm chart as a local directory path (the helm-charts-k8s/ directory in the repository root) or an OCI/repo reference if publishing to a registry:

docker run --rm \
  -v /path/to/kubeconfig:/kubeconfig:ro \
  -v /path/to/gpu-operator/helm-charts-k8s:/helm-charts:ro \
  gpu-op-k8s-e2e:latest \
  -kubeconfig /kubeconfig \
  -operatorchart /helm-charts \
  -operatortag v1.5.0 \
  -test.timeout 60m

Verify only (pre-deployed cluster)

docker run --rm -v /path/to/kubeconfig:/kubeconfig:ro \
  gpu-op-k8s-e2e:latest \
  -kubeconfig /kubeconfig -existing \
  -check.f 'TestOp010|TestOp020|TestOp030|TestOp040|TestOp050|TestOp060|TestOp065|TestOp070' \
  -test.timeout 30m

Using make

# Full install+verify+teardown
make -C tests/k8s-e2e all KUBECONFIG=/path/to/kubeconfig OPERATOR_TAG=v1.5.0

# Verify only (pre-deployed)
make -C tests/k8s-e2e verify KUBECONFIG=/path/to/kubeconfig

Common flags

Flag Default Description
-kubeconfig ~/.kube/config Path to kubeconfig
-operatorchart OCI registry chart GPU Operator helm chart (OCI ref or local path)
-operatortag v1.4.1 GPU Operator chart version
-namespace kube-amd-gpu Kubernetes namespace
-existing false Skip install/teardown — verify only against pre-deployed cluster
-noteardown false Skip teardown after tests (leave operator installed)
-helmset (none) Extra helm --set override (repeatable)
-check.f (all) Regex filter for test names (gocheck syntax)
-test.timeout 30m Overall test timeout

2fc32d2b... chore: exclude .claude, tests/pytests, and knowledge folders from mar… (#1326)

</details>


*Cherry-pick triggered by: ACP-Automation*

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants