Skip to content

Fix bugfix checker#66120

Merged
Algunenano merged 5 commits intoClickHouse:masterfrom
Algunenano:fix_bugfix_checker
Jul 8, 2024
Merged

Fix bugfix checker#66120
Algunenano merged 5 commits intoClickHouse:masterfrom
Algunenano:fix_bugfix_checker

Conversation

@Algunenano
Copy link
Copy Markdown
Member

@Algunenano Algunenano commented Jul 5, 2024 •
edited
Loading

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

  • Fix bugfix checker

Documentation entry for user-facing changes

xxx

Marked as BugFix to launch validation. Should be changed to Not for changelog after everything is working
Closes #65946

CI Settings (Only check the boxes if you know what you are doing):

  • Allow: All Required Checks
  • Allow: Stateless tests
  • Allow: Stateful tests
  • Allow: Integration Tests
  • Allow: Performance tests
  • Allow: All Builds
  • Allow: batch 1, 2 for multi-batch jobs
  • Allow: batch 3, 4, 5, 6 for multi-batch jobs
  • Exclude: Style check
  • Exclude: Fast test
  • Exclude: All with ASAN
  • Exclude: All with TSAN, MSAN, UBSAN, Coverage
  • Exclude: All with aarch64, release, debug
  • Do not test
  • Woolen Wolfdog
  • Upload binaries for special builds
  • Disable merge-commit
  • Disable CI cache

@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-bugfix Pull request with bugfix, not backported by default label Jul 5, 2024
@robot-ch-test-poll1
Copy link
Copy Markdown
Contributor

robot-ch-test-poll1 commented Jul 5, 2024 •
edited by robot-clickhouse-ci-2
Loading

This is an automated comment for commit 18e2f0d with description of existing statuses. It's updated for the latest CI running

❌ Click here to open a full report in a separate page

Check nameDescriptionStatus
Bugfix validationChecks that either a new test (functional or integration) or there some changed tests that fail with the binary built on master branch❌ failure
Stateless testsRuns stateless functional tests for ClickHouse binaries built in various configurations -- release, debug, with sanitizers, etc❌ failure
Successful checks
Check nameDescriptionStatus
Flaky testsChecks if new added or modified tests are flaky by running them repeatedly, in parallel, with more randomization. Functional tests are run 100 times with address sanitizer, and additional randomization of thread scheduling. Integration tests are run up to 10 times. If at least once a new test has failed, or was too long, this check will be red. We don't allow flaky tests, read the doc✅ success
Push to DockerhubThe check for building and pushing the CI related docker images to docker hub✅ success
Style checkRuns a set of checks to keep the code style clean. If some of tests failed, see the related log from the report✅ success

@Algunenano Algunenano force-pushed the fix_bugfix_checker branch from 49fb1f0 to ccd0f78 Compare July 5, 2024 12:08
@Algunenano Algunenano force-pushed the fix_bugfix_checker branch from ccd0f78 to 9185022 Compare July 5, 2024 14:47
@Algunenano
Copy link
Copy Markdown
Member Author

Algunenano commented Jul 8, 2024

It runs (and fails succesfully): https://s3.amazonaws.com/clickhouse-test-reports/66120/18e2f0da9ffc193f26e807b5a6ba9a5f3111fa15/bugfix_validation.html

@Algunenano Algunenano added pr-not-for-changelog This PR should not be mentioned in the changelog and removed pr-bugfix Pull request with bugfix, not backported by default labels Jul 8, 2024
@Algunenano Algunenano added this pull request to the merge queue Jul 8, 2024
Merged via the queue into ClickHouse:master with commit 847a429 Jul 8, 2024
@Algunenano Algunenano deleted the fix_bugfix_checker branch July 8, 2024 15:52
@robot-clickhouse robot-clickhouse added the pr-synced-to-cloud The PR is synced to the cloud repo label Jul 8, 2024
@pamarcos pamarcos mentioned this pull request Jul 9, 2024
Closed
@joelynch joelynch mentioned this pull request Jul 11, 2024
Merged
20 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

"Bugfix Validation" task is broken

4 participants

@Algunenano @robot-ch-test-poll1 @robot-clickhouse @robot-ch-test-poll3