Skip to content
This repository was archived by the owner on Mar 16, 2026. It is now read-only.

consistently remove domain from short hostname#72

Merged
alfredodeza merged 2 commits intoceph:masterfrom
angdraug:6269
Sep 11, 2013
Merged

consistently remove domain from short hostname#72
alfredodeza merged 2 commits intoceph:masterfrom
angdraug:6269

Conversation

@angdraug
Copy link
Copy Markdown
Contributor

This resolves the unnecessary warning thrown by ceph-deploy mon create when host:ip notation is used to specify the mon and the remote host reports an fqdn as its hostname:
http://tracker.ceph.com/issues/6269
The hostname handling code is extracted into remote_shortname() method in ceph_deploy.misc and reused across ceph-deploy to improve consistency.

Signed-off-by: Dmitry Borodaenko <angdraug@gmail.com>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this looks like two tests :)

Signed-off-by: Dmitry Borodaenko <angdraug@gmail.com>
@ceph-jenkins
Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

alfredodeza added a commit that referenced this pull request Sep 11, 2013
consistently remove domain from short hostname
@alfredodeza alfredodeza merged commit c7756f8 into ceph:master Sep 11, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants