Skip to content

ci: add duration parameter for ci workflow. - #142

Open
OscarDDD wants to merge 1 commit into
mainfrom
Add-execution-time-for-Integration-Tests
Open

OscarDDD wants to merge 1 commit into
mainfrom
Add-execution-time-for-Integration-Tests

Conversation

@OscarDDD

@OscarDDD OscarDDD commented Sep 14, 2026

Copy link
Copy Markdown

Context

Closes SAP/ai-sdk-python-backlog#35.

What this PR does and why it is needed

The integration tests (gen) pipeline can take over an hour, making it hard to identify which individual tests are the bottleneck. This PR adds --durations=0 to the pytest command in the integration-tests-gen CI job, which reports the execution time of every individual test at the end of the pipeline run (sorted slowest-first). This makes it immediately visible which tests are slow, enabling informed decisions on how to optimize the pipeline.

Definition of Done

  • [ ☑️] Code is tested (Unit, Integration, E2E) - the flag is a built-in pytest feature, verified locally against the real integration test suite; timing output confirmed in the run results
  • [ ☑️] Error handling created / updated & covered by the tests above — N/A, no error handling changes needed for a reporting flag
  • [ ☑️] Documentation updated - N/A, this is a CI-only change with no public API impact
  • [ ☑️] Aligned with JS/TS and Java SDK — N/A, CI-internal change
  • [ ☑️] Release notes — N/A, CI-internal change, no user-facing impact

Example output

Screenshot 2026-09-14 at 09 25 18

@OscarDDD OscarDDD changed the title Add duration parameter for ci workflow. ci: add duration parameter for ci workflow. Sep 14, 2026
@OscarDDD
OscarDDD marked this pull request as ready for review September 14, 2026 07:29
@OscarDDD
OscarDDD requested a review from alpkom as a code owner September 14, 2026 07:29
@yamaceay

Copy link
Copy Markdown
Contributor

lgtm @alpkom

@OscarDDD OscarDDD self-assigned this Sep 14, 2026
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