Skip to content

osd: expose osdspec_affinity to osd_metadata#35588

Merged
tchaikov merged 1 commit intoceph:masterfrom
jschmid1:osdspec_affinity_to_metadata
Jun 20, 2020
Merged

osd: expose osdspec_affinity to osd_metadata#35588
tchaikov merged 1 commit intoceph:masterfrom
jschmid1:osdspec_affinity_to_metadata

Conversation

@jschmid1
Copy link
Contributor

@jschmid1 jschmid1 commented Jun 16, 2020 •
edited
Loading

Signed-off-by: Joshua Schmid jschmid@suse.de

Follow-up of #34835

Example out:

ceph osd metadata

{
    "id": 0,
[..snip..]
    "osd_objectstore": "bluestore",
    "osdspec_affinity": "test_affinity",   <- new
    "rotational": "0"
}

Fixes: https://tracker.ceph.com/issues/44755

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

osd: expose osdspec_affinity to osd_metadata
e96eacf 
Signed-off-by: Joshua Schmid <jschmid@suse.de>
@tchaikov
Copy link
Contributor

tchaikov commented Jun 20, 2020

http://pulpito.ceph.com/kchai-2020-06-19_01:26:25-rados-wip-kefu2-testing-2020-06-19-0046-distro-basic-smithi/

@tchaikov tchaikov merged commit 9a8b4fd into ceph:master Jun 20, 2020
@jschmid1
Copy link
Contributor Author

jschmid1 commented Jul 6, 2020

@tchaikov @jdurgin This patch should be backported to octopus. I've added the PR-ID to the tracker issue and assigned the octopus label to the backport dropdown. Are there any more steps required by me?

@jdurgin
Copy link
Member

jdurgin commented Jul 6, 2020

@jschmid1 that's all that's required for the backport team to pick it up. if it's an urgent fix you might want to create the backport PR yourself

@smithfarm smithfarm mentioned this pull request Jul 7, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jdurgin jdurgin jdurgin approved these changes

@sebastian-philipp sebastian-philipp Awaiting requested review from sebastian-philipp

Assignees

No one assigned

Labels

cephadm core needs-qa

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jschmid1 @tchaikov @jdurgin