[Diamond (diamondpowell)]

Summary

• Record one aggregate timing result for each Azure Deployment, StatefulSet, and Job command.
• Preserve explicit failure status when one or more workload instances fail.
• Verify workload timing records are collected into results.json.
• Update CRUD examples to use python3 -m crud.main.

Timing contract

• Operation names remain create_pods, create_statefulset, and create_jobs.
• Each CLI command emits one aggregate row across all requested workload instances.
• Metadata includes workload type, node pool, requested count, replicas or completions, namespace, and successful workload count.

Validation

• test_operation.py: 27 tests passed
• test_azure_node_pool_crud.py: 24 tests passed
• test_main.py: 57 tests passed
• Pylint: 10.00/10
• git diff --check: clean

Pipeline validation

• Temporary validation configuration is isolated in commit 62c44efeb.
• End-to-end workload timing validation is pending.
• The temporary pipeline commit will be removed before the PR is marked ready for review.

[github-actions]

For reviewers only: reply /run-tf-integration to trigger the terraform integration pipeline before approving the PR.

[Copilot]

Copilot review overview

🟡 Changes recommended

Job metadata is mislabeled and temporary pipeline validation configuration remains committed.

Review effort: Balanced
Findings: 2

Open (2)

• Record Job completions under a Job-specific metadata key · New
• Remove temporary workload-timing validation configuration · New

What changed in this PR

Adds aggregate timing and failure reporting for Azure Kubernetes workload CRUD operations.

Changes:

• Records aggregate Deployment, StatefulSet, and Job timing metadata.
• Preserves explicit operation failures and expands result tests.
• Updates CRUD documentation and temporary pipeline validation.

File	Description
pipelines/​system/​new-pipeline-test.yml	Adds temporary workload timing validation.
modules/​python/​tests/​crud/​test_operation.py	Tests explicit failure preservation.
modules/​python/​tests/​crud/​test_main.py	Tests workload result collection.
modules/​python/​tests/​crud/​test_azure_node_pool_crud.py	Verifies aggregate timing records.
modules/​python/​crud/​README.md	Documents workload commands and module invocation.
modules/​python/​crud/​operation.py	Preserves explicitly assigned failure status.
modules/​python/​crud/​azure/​node_pool_crud.py	Records aggregate workload operation results.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Leonard Du (LeonardCareer)]

/run-tf-integration

[Leonard Du (LeonardCareer)]

/run-tf-integration

[Leonard Du (LeonardCareer)]

lgtm

[Leonard Du (LeonardCareer)]

The failed check is caused by a pre-existing CI bug in empty Terraform plan matrix handling. The Terraform tests passed, and this PR has no Terraform-related changes. I’ll fix the workflow separately. This PR is OK to merge.