[
https://issues.apache.org/jira/browse/AMBARI-3615?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13808487#comment-13808487
]
Trevor McKay commented on AMBARI-3615:
--------------------------------------
I was bitten by this today (I must have pulled before the revert. The change
breaks killallprocesses(), and if that is fixed to parse pid files, then
ambari-agent kills itself on startup after the daemonize call :) )
Can you explain the issues? Is this for legacy code support?
the current killallprocesses() ends up looking in the installprefix dir, but it
only parses pid files that have names like NUMBER.pid. Anything else ending in
.pid will break it.
If this is for legacy, I think we need some comments. It's not clear to me
just from looking at the code that something is not broken.
> Ambari agent creates empty folder /var/ambari-agent
> ---------------------------------------------------
>
> Key: AMBARI-3615
> URL: https://issues.apache.org/jira/browse/AMBARI-3615
> Project: Ambari
> Issue Type: Bug
> Components: agent
> Affects Versions: 1.4.1
> Reporter: Dmytro Sen
> Assignee: Dmytro Sen
> Fix For: 1.4.2
>
> Attachments: AMBARI-3615_branch_1.4.2.patch, AMBARI-3615.patch
>
>
> Ambari agent creates an empty directory /var/ambari-agent during installation.
> This directory isn't needed, /var/run/ambari-agent is used instead.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
