Skip to content

mgr/cephadm: annotate scheduling code with comments#35567

Merged
sebastian-philipp merged 1 commit intomasterfrom
cephadm_annotate_scheduler
Jun 17, 2020
Merged

mgr/cephadm: annotate scheduling code with comments#35567
sebastian-philipp merged 1 commit intomasterfrom
cephadm_annotate_scheduler

Conversation

@jschmid1
Copy link
Contributor

Also,

  • Moving the logic to find out which hosts will receive new daemons and which daemons will be removed from hosts to the HostAssignment class.
  • Adapt some variable names to clarify their purpose.

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

Signed-off-by: Joshua Schmid <jschmid@suse.de>
@tchaikov
Copy link
Contributor

hi @jschmid1 could you create PRs from your own remote repo in future? normaly, ceph/ceph are used for named branches like LTS releases, instead of for topic branches.

@sebastian-philipp sebastian-philipp merged commit 8acb692 into master Jun 17, 2020
@sebastian-philipp sebastian-philipp deleted the cephadm_annotate_scheduler branch June 17, 2020 14:28
@sebastian-philipp
Copy link
Contributor

sebastian-philipp commented Jun 17, 2020

@jschmid1 can you remove the cephadm_annotate_scheduler branch on ceph/ceph?

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.

3 participants