[GitHub] [openwhisk] codecov-commenter commented on pull request #5432: Compare invocation namespaces when handling a cycle and recovering a queue

2023-08-07 Thread via GitHub
codecov-commenter commented on PR #5432: URL: https://github.com/apache/openwhisk/pull/5432#issuecomment-1668670055 ## [Codecov](https://app.codecov.io/gh/apache/openwhisk/pull/5432?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report >

[GitHub] [openwhisk] bdoyle0182 commented on pull request #5432: Compare invocation namespaces when handling a cycle and recovering a queue

2023-08-07 Thread via GitHub
bdoyle0182 commented on PR #5432: URL: https://github.com/apache/openwhisk/pull/5432#issuecomment-1668656637 Is there anything that can be unit tested here? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [openwhisk] style95 commented on pull request #5432: Compare invocation namespaces when handling a cycle and recovering a queue

2023-08-07 Thread via GitHub
style95 commented on PR #5432: URL: https://github.com/apache/openwhisk/pull/5432#issuecomment-1668663957 > Is there anything that can be unit tested here? I tried to add a unit test but it seems it's not easy to reproduce the situation based on a unit test. -- This is an

[GitHub] [openwhisk] bdoyle0182 commented on pull request #5432: Compare invocation namespaces when handling a cycle and recovering a queue

2023-08-07 Thread via GitHub
bdoyle0182 commented on PR #5432: URL: https://github.com/apache/openwhisk/pull/5432#issuecomment-1668675872 > > Is there anything that can be unit tested here? > > I tried to add a unit test but it seems it's not easy to reproduce the situation based on a unit test. LGTM

[GitHub] [openwhisk] style95 opened a new pull request, #5432: Compare invocation namespaces when handling a cycle and recovering a queue

2023-08-07 Thread via GitHub
style95 opened a new pull request, #5432: URL: https://github.com/apache/openwhisk/pull/5432 ## Description This is to fix a bug in that ETCD data for non-existent containers persist forever. This bug happens when a shared action is invoked. When the queue manager receives an

[GitHub] [openwhisk] jrudolph commented on issue #5431: Memory leak in `akka.actor.LocalActorRef`

2023-08-07 Thread via GitHub
jrudolph commented on issue #5431: URL: https://github.com/apache/openwhisk/issues/5431#issuecomment-1668414786 Since the strings are all IP addresses and it is below the stream materializer, this could be incoming connections that are hanging / not cleaned up (without knowing anything

[GitHub] [openwhisk] joni-jones commented on issue #5431: Memory leak in `akka.actor.LocalActorRef`

2023-08-07 Thread via GitHub
joni-jones commented on issue #5431: URL: https://github.com/apache/openwhisk/issues/5431#issuecomment-1668559315 @jrudolph I'm looking at these graphs and strings with IPs having 0% in comparison to `RedBlackTreeMap` allocation. But I'm still looking if it could be an issue. -- This is

[GitHub] [openwhisk] style95 commented on pull request #5432: Compare invocation namespaces when handling a cycle and recovering a queue

2023-08-07 Thread via GitHub
style95 commented on PR #5432: URL: https://github.com/apache/openwhisk/pull/5432#issuecomment-1668928141 It seems there is a problem in multi-runtime tests. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL