-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
now you can use a network link name with a hyphen #1045
Conversation
Have you been able to test and validate that? |
@leseb sure; though I've gotten stuck at another issue ( #760 ) see my a little weak proof:
other settings are the same with the issue thread ( #1038 ) |
How many mons do you have? Only 2? |
@leseb I have 2 monitors and they seem to be up and running:
|
2 doesn't make a quorum. Drop down to 1 or go with 3 at least. |
@leseb I could run a ceph cluster with only 2 monitors when I used ceph-deploy; what makes you say so? I don't have enough resources to deploy 3 monitors; nor do I want to have only 1 monitor risking the availability. |
No no no. Monitors have quorum that can only be resolved by an odd number On Tuesday, 25 October 2016, Nicholas Gim [email protected] wrote:
Cheers –––––– "Always give 100%. Unless you're giving blood." Mail: [email protected] |
@leseb alright, thanks a lot for the information. I've reduced it to 1 from 2; the monitor is up and running ( http://docs.ceph.com/docs/jewel/rados/configuration/mon-config-ref/#initial-members ) now I get another error message for ceph-osds this time ( related pull request : #1035 (comment) )
|
@leseb I got through the running of 'ansible-playbook site.yml' by negating the pull request #1035 (comment) and I presume that my pull request would help work; though I'm entirely new to Jinja2 and the other stuff so not very sure about the safety of my code.
|
Alright thanks for the comments, the code looks good to me. Thanks! |
test this please |
LGTM, @font do you mind testing this against a containerized deployment? |
@leseb It should work for containerized deployments as this change only applies when |
@leseb Tests on containerized deployment looks good. |
@font thanks a lot! |
#1038