Skip to content

tentacle: blk/kernel: bring "bdev_async_discard" config parameter back.#65609

Merged
ifed01 merged 1 commit intoceph:tentaclefrom
ifed01:wip-ifed-bool-async-discard-back-tent
Nov 5, 2025
Merged

tentacle: blk/kernel: bring "bdev_async_discard" config parameter back.#65609
ifed01 merged 1 commit intoceph:tentaclefrom
ifed01:wip-ifed-bool-async-discard-back-tent

Conversation

@ifed01
Copy link
Contributor

@ifed01 ifed01 commented Sep 19, 2025

To ensure backword compatibility for clusters with this parameter previously set to true.
Relates to: https://tracker.ceph.com/issues/70327

(cherry picked from commit 7b914cb)

Contribution Guidelines

  • To sign and title your commits, please refer to Submitting Patches to Ceph.

  • If you are submitting a fix for a stable branch (e.g. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.

  • When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an x between the brackets: [x]. Spaces and capitalization matter when checking off items this way.

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands

You must only issue one Jenkins command per-comment. Jenkins does not understand
comments with more than one command.

To ensure backword compatibility for clusters with this parameter
previously set to true.

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit 7b914cb)
Copy link
Contributor

@rzarzynski rzarzynski left a comment

Choose a reason for hiding this comment

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

The referenced commit (7b914cb) is from squid. It's non-orthodox but the PR is already merged.

@sseshasa
Copy link
Contributor

@ifed01
Copy link
Contributor Author

ifed01 commented Oct 27, 2025

jenkins test windows

@SrinivasaBharath
Copy link
Contributor

jenkins test make check

1 similar comment
@ifed01
Copy link
Contributor Author

ifed01 commented Oct 29, 2025

jenkins test make check

@SrinivasaBharath
Copy link
Contributor

@ifed01 - Please resolve the pending checks and proceed to merge the PR once they are successful.

@ifed01 ifed01 merged commit b973c0e into ceph:tentacle Nov 5, 2025
11 of 12 checks passed
@ifed01 ifed01 deleted the wip-ifed-bool-async-discard-back-tent branch November 5, 2025 14:32
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.

5 participants