Skip to content

Feature/cspl 4378 test timeouts#1822

Merged
kasiakoziol merged 30 commits intodevelopfrom
feature/CSPL-4378-test-timeouts
Apr 16, 2026
Merged

Feature/cspl 4378 test timeouts#1822
kasiakoziol merged 30 commits intodevelopfrom
feature/CSPL-4378-test-timeouts

Conversation

@kasiakoziol
Copy link
Copy Markdown
Collaborator

@kasiakoziol kasiakoziol commented Apr 7, 2026

Description

It sets timeouts for smoke and integration tests on test case, test suite and pipeline level.

Key Changes

Test case:

  • short: 15 minutes
  • medium: 45 minutes
  • long: 100 minutes

Test suite:

  • short: 30 minutes
  • medium: 90 minutes
  • long: 200 minutes

Pipeline: 240 minutes

Testing and Verification

Automated tests

Related Issues

Jira: https://splunk.atlassian.net/browse/CSPL-4378

PR Checklist

  • Code changes adhere to the project's coding standards.
  • Relevant unit and integration tests are included.
  • Documentation has been updated accordingly.
  • All tests pass locally.
  • The PR description follows the project's guidelines.

kasiakoziol and others added 11 commits March 16, 2026 07:05
* CSPL-4397 Remove time dependency from tests

* CSPL-4397 Removing test reliance on time

* CSPL-4397 Adjusting timeouts

* CSPL-4354 Fixing time dependencies in unit tests and fixing timeouts in integ tests

* CSPL-4397 Move to using watch

* CSPL-4397 Refactor pod reset detection to use UID-based tracking instead of time-based comparison, add fail-fast test prerequisites validation, and improve code clarity with variable renaming

* CSPL-4397 Refactoring

* CSPL-4397 Addressing comments
…1768)

* CSPL-4602 Moving duplicated code for LM tests to a shared codebase

* CSPL-4602 Addinf license manager tests to workflows

* CSPL-4602 Address comments

* CSPL-4602 Moving LM shared test utils to licensemanager package

* CSPL-4602 Addressing base branch changes
* CSPL-4377 Get rid of not expected panic in tests

* CSPL-4377 Check error for app disablement in tests
* Enhance error handling in WaitforPhaseChange and fix variable name in NewTestCaseEnv

* Updated WaitforPhaseChange to log an error if the phase transition is not observed within the timeout.
* Corrected the variable name check in NewTestCaseEnv to use the provided name instead of an uninitialized envName.
* Changed Kind from "ClusterManager" to "ClusterMaster" in ClusterMaster struct.

* Update error logging in WaitforPhaseChange to use Error level for timeout observations
@kasiakoziol kasiakoziol marked this pull request as ready for review April 13, 2026 06:32
Comment thread .github/workflows/arm-AL2023-build-test-push-workflow-AL2023.yml
Comment thread test/appframework_aws/c3/appframework_aws_test.go Outdated
Comment thread test/testenv/testcaseenv.go Outdated
Comment thread test/testenv/testenv.go Outdated
Comment thread test/testenv/testenv.go Outdated
Comment thread test/testenv/deployment.go Outdated
Base automatically changed from feature/test-refactoring to develop April 14, 2026 17:57
@kasiakoziol kasiakoziol merged commit f4484a2 into develop Apr 16, 2026
80 of 85 checks passed
@kasiakoziol kasiakoziol deleted the feature/CSPL-4378-test-timeouts branch April 16, 2026 05:13
@github-actions github-actions bot locked and limited conversation to collaborators Apr 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants