[ 
https://issues.apache.org/jira/browse/ARTEMIS-4630?focusedWorklogId=903461&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-903461
 ]

ASF GitHub Bot logged work on ARTEMIS-4630:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 03/Feb/24 03:13
            Start Date: 03/Feb/24 03:13
    Worklog Time Spent: 10m 
      Work Description: jbertram commented on code in PR #4801:
URL: https://github.com/apache/activemq-artemis/pull/4801#discussion_r1476928352


##########
artemis-cli/src/main/resources/org/apache/activemq/artemis/cli/commands/bin/artemis-service:
##########
@@ -61,7 +61,7 @@ status() {
     pid=`cat "${PID_FILE}"`
     # check to see if it's gone...
     ps -p ${pid} > /dev/null
-    if [ $? -eq 0 ] ; then
+    if [ $? -eq 0 ] && [ "`pwdx ${pid} | awk '{print $2}'`" == 
"$ARTEMIS_INSTANCE" ]; then

Review Comment:
   Unfortunately it's still broken. I still get the same error as before. 
You're using `==` when I believe you should be using `=`.





Issue Time Tracking
-------------------

    Worklog Id:     (was: 903461)
    Time Spent: 40m  (was: 0.5h)

> AMQ Service status is not validating if the pid is from Artemis
> ---------------------------------------------------------------
>
>                 Key: ARTEMIS-4630
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-4630
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>          Components: Configuration
>    Affects Versions: 2.32.0
>            Reporter: Angelo Galvão
>            Priority: Major
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> In a rare condition, an old artemis.pid file is pointing to an existing 
> process that is not Artemis, causing the provisioning tools to believe that 
> Artemis process is running.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to