[
https://issues.apache.org/jira/browse/TRAFODION-2518?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15897710#comment-15897710
]
ASF GitHub Bot commented on TRAFODION-2518:
-------------------------------------------
Github user svarnau commented on a diff in the pull request:
https://github.com/apache/incubator-trafodion/pull/991#discussion_r104473719
--- Diff: install/python-installer/db_uninstall.py ---
@@ -96,13 +96,13 @@ def main():
remotes = [Remote(node, pwd=pwd) for node in nodes]
# stop trafodion on the first node
- remotes[0].execute('sudo su %s -l -c ckillall' % TRAF_USER,
chkerr=False)
+ #remotes[0].execute('sudo su %s -l -c ckillall' % TRAF_USER,
chkerr=False)
# remove trafodion userid and group on all trafodion nodes, together
with folders
for remote in remotes:
info('Remove Trafodion on node [%s] ...' % remote.host)
remote.execute('ps -ef|grep ^%s|awk \'{print $2}\'|xargs sudo kill
-9' % TRAF_USER, chkerr=False)
- remote.execute('ps -ef|grep krb5check|awk \'{print $2}\'|xargs
sudo kill -9', chkerr=False) # traf kerberos service
+ remote.execute('trafid=`getent passwd %s|awk -F: \'{print $3}\'`;
if [[ -n $trafid ]]; then ps -ef|grep ^$trafid|awk \'{print $2}\'|xargs sudo
kill -9; fi' % TRAF_USER, chkerr=False)
remote.execute('sudo -n /usr/sbin/userdel -rf %s' % TRAF_USER,
chkerr=False)
--- End diff --
on lines 104,105, instead of doing ps -ef and grep for TRAF_USER, wouldn't
it be better to do
'ps -f -u %s' % TRAF_USER
> pyinstaller - remove unused db_admin settings.
> ----------------------------------------------
>
> Key: TRAFODION-2518
> URL: https://issues.apache.org/jira/browse/TRAFODION-2518
> Project: Apache Trafodion
> Issue Type: Bug
> Reporter: Eason Zhang
> Assignee: Eason Zhang
>
> clean up unused db_admin settings since there's no DB__ADMIN user in
> Trafodion.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
