Skip to content
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

geo-DR: remove need to have ssh #16

Open
grypyrg opened this issue Aug 30, 2014 · 9 comments
Open

geo-DR: remove need to have ssh #16

grypyrg opened this issue Aug 30, 2014 · 9 comments

Comments

@grypyrg
Copy link
Contributor

grypyrg commented Aug 30, 2014

Why do we need ssh from every machine to every machine?

For geo-DR, this is used to fetch who the master is and it's binary log information.

Can't we just have a small daemon that runs in the cluster and serves those requests?
Or are there other ways through booth itself?

@y-trudeau
Copy link

We could use xinetd for example. That does introduce an extra dependency though, ssh is usually already installed. Do you have security concerns?

@grypyrg
Copy link
Contributor Author

grypyrg commented Sep 13, 2014

Yes, indeed. Many environments I work on do not allow root ssh automatically

@dotmanila
Copy link
Contributor

@y-trudeau
Copy link

we definitely could use remote connections but, in an environment that prohibit ssh, do you think allowing remote cib connection is not even more dangerous? I kind of like the limited capability of an xinetd frontend which just output data and allow not modification.

@dotmanila
Copy link
Contributor

That's a good point, I wonder though the portability of xinetd? Remote CIB connections just appeals to me as its builtin :-)

@y-trudeau
Copy link

remote cib access can be configured with readonly ACL, I'll look into this.

@grypyrg
Copy link
Contributor Author

grypyrg commented Oct 13, 2014

Rumor has it that @dotmanila has a non-root ssh implementation with sudo almost ready as intermediate measure :)

@y-trudeau
Copy link

Checking @dotmanila code right now

@y-trudeau y-trudeau self-assigned this Mar 2, 2015
@y-trudeau
Copy link

@dotmanila code has been merged in 1.0.0, I'll explore the possibility of using pacemaker directly with ACL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants