bernardodemarco commented on PR #11016: URL: https://github.com/apache/cloudstack/pull/11016#issuecomment-3052399701
@nvazquez, thanks for testing it! > Have you faced this case before? I was able to reproduce a similar scenario. I kept a console session active and simulated a CPVM downtime, by manually stopping it. In this scenario, the Management Server does not recognize that the session has been removed (removed column remains indefinitely as `NULL`). > I don't think it is an issue with this PR but perhaps there could be a mechanism to remove this kind of 'incorrect' console sessions from the API response Yes, this inconsistency would be out of scope of this PR. The `listConsoleSessions` API simply lists the sessions based on the metadata persisted in the DB. From my understanding, the only attributes that it can use to classify a session as active are the `acquired` and `removed` ones. Thus, I do not see a workaround for this besides fixing the root issue, that is the Management Server not recognizing when a session is released in certain scenarios. I will open a segregate issue for us to handle this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
