Skip to content

chore: bump aap-api-tests pipeline - #1572

Closed
ptoscano wants to merge 1 commit into
ansible:mainfrom
ptoscano:bump-tekton-test-pipeline
Closed

chore: bump aap-api-tests pipeline#1572
ptoscano wants to merge 1 commit into
ansible:mainfrom
ptoscano:bump-tekton-test-pipeline

Conversation

@ptoscano

@ptoscano ptoscano commented May 29, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated automated test pipeline configuration to reference the latest pipeline bundle version.

@ptoscano
ptoscano requested a review from a team as a code owner May 29, 2026 08:32
@coderabbitai

coderabbitai Bot commented May 29, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 42546433-54d5-44b7-872c-be15a794189c

📥 Commits

Reviewing files that changed from the base of the PR and between fd2e1f1 and 7f468de.

📒 Files selected for processing (1)
  • .tekton/run-atf-tests-pull-request.yaml

📝 Walkthrough

Walkthrough

The pull request updates the Tekton PipelineRun configuration to pin the AAP API tests pipeline to a new immutable container bundle digest, replacing the previous sha256 reference with an updated one.

Changes

Pipeline Bundle Digest Update

Layer / File(s) Summary
AAP API tests pipeline bundle reference
.tekton/run-atf-tests-pull-request.yaml
The bundle parameter for the aap-api-tests pipeline is updated to reference a new container image digest (sha256), pinning the PipelineRun to an updated immutable bundle.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning No pull request description was provided, but the template requires clear explanation of changes, purpose, rationale, and testing information. Add a description explaining what bundle digest was changed, why the pipeline needs updating, and how to verify the new pipeline bundle works correctly.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating the aap-api-tests pipeline bundle digest in the Tekton configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ptoscano

Copy link
Copy Markdown
Contributor Author

/run-atf-tests

@codecov-commenter

codecov-commenter commented May 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.17%. Comparing base (fd2e1f1) to head (7f468de).

@@           Coverage Diff           @@
##             main    #1572   +/-   ##
=======================================
  Coverage   92.17%   92.17%           
=======================================
  Files         244      244           
  Lines       11214    11214           
=======================================
  Hits        10337    10337           
  Misses        877      877           
Flag Coverage Δ
unit-int-tests-3.11 92.17% <ø> (ø)
unit-int-tests-3.12 92.17% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sonarqubecloud

Copy link
Copy Markdown

@aap-pde-ci-bot

Copy link
Copy Markdown

❌ API Test Results - FAILED

Summary

Metric Count
Total Tests 71
✅ Passed 31
❌ Failed 0
⚠️ Errors 10
⏭️ Skipped 30
⏱️ Duration 119.36s

Pass Rate: 43.7%

❌ Failed Tests

Test Class
test_credential_type_test opt.test-suite.tests.api.test_credential_types.TestCredentialTypes
test_create_decision_environment_duplicated_name opt.test-suite.tests.api.test_decision_environments.TestDecisionEnvironments
test_read_decision_environment opt.test-suite.tests.api.test_decision_environments.TestDecisionEnvironments
test_update_decision_environment opt.test-suite.tests.api.test_decision_environments.TestDecisionEnvironments
test_delete_decision_environment_not_in_use opt.test-suite.tests.api.test_decision_environments.TestDecisionEnvironments
test_delete_decision_environment_with_force_flag opt.test-suite.tests.api.test_decision_environments.TestDecisionEnvironments
test_list_decision_environments opt.test-suite.tests.api.test_decision_environments.TestDecisionEnvironments
test_filter_name_decision_environments opt.test-suite.tests.api.test_decision_environments.TestDecisionEnvironments
test_eda_credential_test opt.test-suite.tests.api.test_eda_credentials.TestEdaCredentials
test_update_project opt.test-suite.tests.api.test_projects.TestProjects

@ptoscano
ptoscano marked this pull request as draft May 29, 2026 13:34
@ptoscano

ptoscano commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

Replaced by the automation (finally!): #1574.

@ptoscano ptoscano closed this Jun 1, 2026
@ptoscano
ptoscano deleted the bump-tekton-test-pipeline branch June 1, 2026 09:07
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.

3 participants