[ https://issues.apache.org/jira/browse/AMBARI-22016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16215157#comment-16215157 ]
Jonathan Hurley commented on AMBARI-22016: ------------------------------------------ I just did - it was at the bottom of my list of reviews which is why I never saw it. Once you get the necessary +1's, please let the reviewers know you don't have commit rights so we can do it for you. Otherwise, it gets lost in the mix ... > changeToSecureUid.sh might collide with existing user if it is not a local > user but a network managed user > ---------------------------------------------------------------------------------------------------------- > > Key: AMBARI-22016 > URL: https://issues.apache.org/jira/browse/AMBARI-22016 > Project: Ambari > Issue Type: Bug > Components: stacks > Affects Versions: 2.5.2 > Environment: Using Centos as the test bed > Reporter: Aman Poonia > Assignee: Aman Poonia > Fix For: trunk > > Attachments: AMBARI-22016.patch, AMBARI-22016_branch-2.6.patch > > > changeToSecureUid.sh might collide with existing user if it is not a local > user but a network managed user > Currently we are doing > {code:java} > function find_available_uid() { > for ((i=1001; i<=2000; i++)) > do > grep -q $i /etc/passwd > if [ "$?" -ne 0 ] > then > newUid=$i > break > fi > done > } > {code} > This will only check local user. Network manged users are not directly > present inside passwd file. -- This message was sent by Atlassian JIRA (v6.4.14#64029)