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

ducky: unmute fixed 5884 test #6653

Merged
merged 1 commit into from
Oct 17, 2022
Merged

ducky: unmute fixed 5884 test #6653

merged 1 commit into from
Oct 17, 2022

Conversation

rystsov
Copy link
Contributor

@rystsov rystsov commented Oct 6, 2022

Cover letter

Unmute fixed 5884 test

Backport Required

  • not a bug fix
  • issue does not exist in previous branches
  • papercut/not impactful enough to backport
  • v22.2.x
  • v22.1.x
  • v21.11.x

Release notes

  • none

@rystsov rystsov requested review from ztlpn and BenPope October 6, 2022 15:11
@dotnwat
Copy link
Member

dotnwat commented Oct 6, 2022

looks like john has a fix in the works for the timequery test failures

BenPope
BenPope approved these changes Oct 7, 2022
View reviewed changes
Copy link
Member

@BenPope BenPope left a comment

Choose a reason for hiding this comment

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

In the last 60 days:

OFAIL test: PartitionBalancerTest.test_full_nodes (14/1156 runs)
  failure at 2022-09-30T07:03:35.986Z: AssertionError()

The fix merged 7 days ago, so we'd expect maybe 1 or 2 failures since.

Happy to give it a go though.

@dotnwat
Copy link
Member

dotnwat commented Oct 10, 2022

/ci-repeat 5

@rystsov
Copy link
Contributor Author

rystsov commented Oct 17, 2022

Known issues not related to the change:

@piyushredpanda piyushredpanda merged commit a8d0c6c into redpanda-data:dev Oct 17, 2022
@jcsp
Copy link
Contributor

jcsp commented Oct 18, 2022

It looks like the unmute of this raced with something breaking it. The tests ran on the 10th but it was merged on the 18th

jcsp added a commit to jcsp/redpanda that referenced this pull request Oct 18, 2022
@jcsp
tests: mark PartitionBalancerTest.test_full_nodes ok_to_fail
fa19cb6 
This is too frequent of a failure to tolerate on the
tip of dev.

It was recently re-enabled in redpanda-data#6653

Related: redpanda-data#6810
jcsp added a commit to jcsp/redpanda that referenced this pull request Oct 18, 2022
@jcsp
tests: mark PartitionBalancerTest.test_full_nodes ok_to_fail
591aeab 
This is too frequent of a failure to tolerate on the
tip of dev.

It was recently re-enabled in redpanda-data#6653

Related: redpanda-data#6810
@jcsp jcsp mentioned this pull request Oct 18, 2022
Merged
6 tasks
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BenPope BenPope BenPope approved these changes

@ztlpn ztlpn ztlpn approved these changes

Assignees
No one assigned
Labels
area/tests ci-failure kind/enhance New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@rystsov @dotnwat @jcsp @BenPope @ztlpn @mmedenjak @piyushredpanda