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.

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

storage: remove assertion on is_cloud_retention_active #23936

Merged
merged 1 commit into from
Nov 11, 2024

Conversation

ballard26
Copy link
Contributor

@ballard26 ballard26 commented Oct 28, 2024

This assertion can be triggered if a the configuration for a topic is changed between when space management sets the _cloud_gc_offset to when gc actually runs on the log. So to avoid crashing in this case the assertion is changed to an early return and a warning is logged instead.

Fixes CORE-8070

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v24.2.x
  • v24.1.x
  • v23.3.x

Release Notes

  • none

Copy link
Member

@dotnwat dotnwat left a comment

Choose a reason for hiding this comment

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

lgtm

@vbotbuildovich
Copy link
Collaborator

non flaky failures in https://buildkite.com/redpanda/redpanda/builds/57240#0192d53e-9405-4551-aaa8-13685ac95830:

"rptest.tests.partition_force_reconfiguration_test.PartitionForceReconfigurationTest.test_basic_reconfiguration.acks=-1.restart=True.controller_snapshots=False"

@vbotbuildovich
Copy link
Collaborator

Retry command for Build#57240

please wait until all jobs are finished before running the slash command

/ci-repeat 1
tests/rptest/tests/partition_force_reconfiguration_test.py::PartitionForceReconfigurationTest.test_basic_reconfiguration@{"acks":-1,"controller_snapshots":false,"restart":true}

@ballard26
Copy link
Contributor Author

/ci-repeat 4
tests/rptest/tests/partition_force_reconfiguration_test.py::PartitionForceReconfigurationTest.test_basic_reconfiguration@{"acks":-1,"controller_snapshots":false,"restart":true}

@vbotbuildovich
Copy link
Collaborator

the below tests from https://buildkite.com/redpanda/redpanda/builds/57345#0192db79-8dcd-43bc-9004-f2ee4667d1cf have failed and will be retried

storage_single_thread_rpunit
storage_e2e_single_thread_rpunit
gtest_raft_rpunit
partition_balancer_planner_test_rpunit

This assertion can be triggered if a the configuration for a topic is
changed between when space management sets the `_cloud_gc_offset` to
when gc actually runs on the log. So to avoid crashing in this case the
assertion is changed to an early return and a warning is logged instead.
@ballard26
Copy link
Contributor Author

The tests had previously failed either due to being flake-y or for infra issues. I've rebased this branch and am re-running the tests. Assuming all pass this should be good to merge.

@vbotbuildovich
Copy link
Collaborator

@ballard26 ballard26 merged commit d51bbc8 into redpanda-data:dev Nov 11, 2024
16 checks passed
@vbotbuildovich
Copy link
Collaborator

/backport v24.2.x

@vbotbuildovich
Copy link
Collaborator

/backport v24.1.x

@vbotbuildovich
Copy link
Collaborator

/backport v23.3.x

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

Successfully merging this pull request may close these issues.

4 participants