-
Notifications
You must be signed in to change notification settings - Fork 21
PR for all MapR docker containers #21
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
base: master
Are you sure you want to change the base?
Changes from 1 commit
bf36aea
a95013c
bdc12e6
be1582f
3c9fe45
f5602e0
e35f692
3004ec0
702f0af
c2ddb51
e2852a7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -27,6 +27,7 @@ RUN yum update -y \ | |
| # CONFIGURE SSH | ||
| && chkconfig sshd on \ | ||
| && grep -rl '#Port 22' /etc/ssh/sshd_config | xargs sed -i 's/#Port 22/Port 22/g' \ | ||
| && service sshd start \ | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. why would that be needed? This is during image build, the newly started container will have sshd down anyway (except for cases when supervisor.d will spin it up) |
||
|
|
||
| # INSTALL MAPR | ||
| && yum install -y mapr-fileserver mapr-nfs mapr-nodemanager mapr-cldb \ | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. squash into a single yum install invocation |
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is that needed? The other containers expose sshd as well and they don't seem to be sed-ing the 22 Port in config anywhere AFAIR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its not working without that , other containers will all have same problems when some one will ssh hadoop-master from outside.