Skip to content

Remove test_disks_app_interactive from parallel integration tests execution#87194

Merged
divanik merged 5 commits intomasterfrom
divanik/remove_disks_app_interactive_from_parallel_execution
Sep 17, 2025
Merged

Remove test_disks_app_interactive from parallel integration tests execution#87194
divanik merged 5 commits intomasterfrom
divanik/remove_disks_app_interactive_from_parallel_execution

Conversation

@divanik
Copy link
Copy Markdown
Member

@divanik divanik commented Sep 16, 2025

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

  • Documentation is written (mandatory for new features)

Closes #87183

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Sep 16, 2025

Workflow [PR], commit [750d65f]

Summary:

job_name test_name status info comment
Integration tests (amd_asan, old analyzer, 6/6) failure
test_throttling/test.py::test_local_read_throttling_reload FAIL

- test_storage_iceberg_no_spark/
- test_storage_iceberg_with_spark/
- test_storage_iceberg_schema_evolution/
- test_storage_iceberg_with_spark/
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Noticed that it is not in alphabatical order

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Sep 16, 2025

initialized_disks = []
unitialized_disks = []

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

To launch flaky check

@divanik
Copy link
Copy Markdown
Member Author

divanik commented Sep 17, 2025

Ok, let me try fixing test_storage_iceberg_disks as well with the same approach

@divanik
Copy link
Copy Markdown
Member Author

divanik commented Sep 17, 2025

@divanik divanik added this pull request to the merge queue Sep 17, 2025
Merged via the queue into master with commit 7bf70c2 Sep 17, 2025
121 of 123 checks passed
@divanik divanik deleted the divanik/remove_disks_app_interactive_from_parallel_execution branch September 17, 2025 13:51
@robot-clickhouse robot-clickhouse added the pr-synced-to-cloud The PR is synced to the cloud repo label Sep 17, 2025
@maxknv
Copy link
Copy Markdown
Member

maxknv commented Sep 17, 2025

Do you have an idea why this test is flaky with parallel run? It would be nice to have some explanation why we have one or another test in this list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test_disks_app_interactive/test.py are flaky/broken

4 participants