Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci: refactor operate integration test workflow #18559

Merged
merged 15 commits into from
May 16, 2024
Merged

ci: refactor operate integration test workflow #18559

merged 15 commits into from
May 16, 2024

Conversation

matt-whiteman
Copy link
Contributor

@matt-whiteman matt-whiteman commented May 15, 2024 •
edited

Description

  • Remove unused operate-backend.yml file
  • Refactor operate-ci-test-reusable to allow for input parameters for test type name, runner type, and maven command
  • Add separate unit test job to operate CI (runs on 4-core runner)
  • refactor operate maven profiles for maven-failsafe-plugin to consolidate configurations
  • optimize the operate integration test job to only run in the qa/integration-tests directory
  • add 20 minute timeout to operate test workflows

Related issues

closes #17653

@github-actions github-actions bot added the component/operate Related to the Operate component/team label May 15, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented May 15, 2024 •
edited

Operate Test Results

523 tests   - 275   521 ✅  - 274   6m 57s ⏱️ - 1m 24s
 88 suites  -  44     2 💤  -   1 
 88 files    -  44     0 ❌ ±  0 

Results for commit 52ca187. ± Comparison against base commit 9fad339.

This pull request removes 275 tests. 
io.camunda.operate.archiver.ElasticsearchArchiveRepositoryTest ‑ testDeleteDocuments
io.camunda.operate.archiver.ElasticsearchArchiveRepositoryTest ‑ testDeleteDocumentsError
io.camunda.operate.archiver.ElasticsearchArchiveRepositoryTest ‑ testGetBatchOperationsNextBatchEmptyBucket
io.camunda.operate.archiver.ElasticsearchArchiveRepositoryTest ‑ testReindexDocuments
io.camunda.operate.archiver.ElasticsearchArchiveRepositoryTest ‑ testReindexDocumentsError
io.camunda.operate.archiver.ElasticsearchArchiveRepositoryTest ‑ testSetIndexLifeCycle
io.camunda.operate.archiver.ElasticsearchArchiveRepositoryTest ‑ testSetIndexLifeCycleNoIndex
io.camunda.operate.archiver.OpensearchArchiverRepositoryTest ‑ testDeleteDocuments
io.camunda.operate.archiver.OpensearchArchiverRepositoryTest ‑ testGetBatchOperationsNextBatch
io.camunda.operate.archiver.OpensearchArchiverRepositoryTest ‑ testReindexDocuments
…

♻️ This comment has been updated with latest results.

@matt-whiteman matt-whiteman changed the title ci: change reusable test workflow to use input parameters ci: refactor operate integration test workflow May 15, 2024
@matt-whiteman matt-whiteman requested review from a team, Zelldon and cmur2 May 15, 2024 21:14
@matt-whiteman matt-whiteman marked this pull request as ready for review May 15, 2024 21:14
ralfpuchert
ralfpuchert reviewed May 16, 2024
View reviewed changes
.github/workflows/operate-ci-test-reusable.yml Outdated
required: true
type: string
test-type:
description: "Type of test, sed in the job name, stage name, and report artifact name"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is sed a typo?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup, I'll fix that

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented May 16, 2024 •
edited

Operate Unit Tests Results

275 tests   274 ✅  5m 52s ⏱️
 44 suites    1 💤
 44 files      0 ❌

Results for commit 56b0572.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented May 16, 2024 •
edited

Operate Integration Tests Results

523 tests   521 ✅  8m 17s ⏱️
 88 suites    2 💤
 88 files      0 ❌

Results for commit 56b0572.

♻️ This comment has been updated with latest results.

@matt-whiteman matt-whiteman added this pull request to the merge queue May 16, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 16, 2024
@matt-whiteman matt-whiteman added this pull request to the merge queue May 16, 2024
Merged via the queue into main with commit 9fb39f7 May 16, 2024
24 checks passed
@matt-whiteman matt-whiteman deleted the operate-gha-refactoring branch May 16, 2024 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ralfpuchert ralfpuchert ralfpuchert approved these changes

@Zelldon Zelldon Awaiting requested review from Zelldon

@cmur2 cmur2 Awaiting requested review from cmur2

Assignees
No one assigned
Labels
component/operate Related to the Operate component/team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create workflow for Operate unit tests
2 participants
@matt-whiteman @ralfpuchert