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

update cob-stud installation #85

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

update cob-stud installation #85

wants to merge 3 commits into from

Conversation

ipa-nhg
Copy link
Member

@ipa-nhg ipa-nhg commented Feb 3, 2017

No description provided.

@@ -35,9 +35,12 @@ $ sudo sed -i 's/PermitRootLogin without-password/PermitRootLogin yes/g' /etc/ss

==== Gain root rights for admin user's
----
$ sudo adduser --home /home-local/Admin Admin --force-badname
Copy link
Member

Choose a reason for hiding this comment

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

why do you use Admin with capital A? we always use lower case names for accounts

Copy link
Member Author

Choose a reason for hiding this comment

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

yes with a capital A, admin is already a group.

Copy link
Member

Choose a reason for hiding this comment

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

how about stud-admin user, then?

@@ -35,9 +35,12 @@ $ sudo sed -i 's/PermitRootLogin without-password/PermitRootLogin yes/g' /etc/ss

==== Gain root rights for admin user's
----
$ sudo adduser --home /home-local/Admin Admin --force-badname
$ sudo adduser Admin sudo
$ sudo nano /etc/sudoers
##add to the list###
stud-admin ALL=(ALL) NOPASSWD: ALL
Copy link
Member

Choose a reason for hiding this comment

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

does this mean we can delete stud-admin?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, stud-admin has still sudo rights, but it is a pool account and not a local one. This user is also the administrator for cob-kitchen-svr

@@ -11,21 +11,12 @@ fi

client_list="
cob-stud-1
cob-stud-2
cob.stud-2
Copy link
Member

Choose a reason for hiding this comment

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

revert

@@ -0,0 +1,33 @@
#!/bin/bash
Copy link
Member

Choose a reason for hiding this comment

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

is this well tested? can this be used for the robot pcs too?

Copy link
Member Author

Choose a reason for hiding this comment

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

tested and adapted for the robots! ipa320/setup_cob4#51

Copy link
Member

Choose a reason for hiding this comment

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

please consider updated scripts for robot as proposed in ipa320/setup_cob4#84

is it possible to make the pcs list configurable from outside of the script and use the functional part for both robots and cob-stud-pcs.... ➡️ harmonization and less maintenance...

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

Successfully merging this pull request may close these issues.

3 participants