Skip to content

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

Merged
ifed01 merged 1 commit intoceph:mainfrom
ifed01:wip-ifed-bool-async-discard-back
Oct 27, 2025
Merged

blk/kernel: bring "bdev_async_discard" config parameter back.#65608
ifed01 merged 1 commit intoceph:mainfrom
ifed01:wip-ifed-bool-async-discard-back

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)
@github-actions
Copy link

Config Diff Tool Output

+ added: bdev_async_discard (global.yaml.in)

The above configuration changes are found in the PR. Please update the relevant release documentation if necessary.
Ignore this comment if docs are already updated. To make the "Check ceph config changes" CI check pass, please comment /config check ok and re-run the test.

@ifed01
Copy link
Contributor Author

ifed01 commented Sep 22, 2025

jenkins test make check arm64

1 similar comment
@ifed01
Copy link
Contributor Author

ifed01 commented Sep 23, 2025

jenkins test make check arm64

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 commit description carries:

(cherry picked from commit https://github.com/ceph/ceph/commit/7b914cb49d50241b2ed7811d8660fce27a80ae39)

Is it really necessary here?

@rzarzynski
Copy link
Contributor

Otherwise LGTM.

@ljflores
Copy link
Member

@ifed01 ifed01 merged commit f3d801a into ceph:main Oct 27, 2025
17 of 26 checks passed
@ifed01 ifed01 deleted the wip-ifed-bool-async-discard-back branch October 27, 2025 09:17
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