FireXStuff / firexapp

BSD 3-Clause "New" or "Revised" License
7 stars 1 forks source link

Revoking the root task results in a leaked redis #3

Open mabouels opened 5 years ago

MarcDelahousse commented 5 years ago

We need to include a test for this, though it will need to be marked as none-coverage

MarcDelahousse commented 5 years ago

I wrote a flow_test to cover this case and I was not able to reproduce it.

mabouels commented 5 years ago
[2019-08-01 15:51:59 DEBUG] Running FireX self destruct
[2019-08-01 15:52:00 INFO] Revoking lingering tasks...
[2019-08-01 15:52:00 WARNING] Revoking microservices.worker_management_tasks.SpawnWaitReleaseChain
[2019-08-01 15:52:00 WARNING] Revoking microservices.worker_management_tasks.ExecuteChainOnWorkerFarm
[2019-08-01 15:52:00 WARNING] Revoking microservices.chaintasks.Nightly
[2019-08-01 15:52:00 WARNING] Revoking microservices.chaintasks.Regress
[2019-08-01 15:52:00 WARNING] Revoking microservices.core_tasks.WaitOnResults
[2019-08-01 15:52:00 INFO] Revoking lingering tasks...
[2019-08-01 15:52:00 WARNING] Revoking microservices.build_and_regress.MasterBuildAndRegress
[2019-08-01 15:52:00 DEBUG] Sending Celery shutdown
[2019-08-01 15:52:00 DEBUG] Waiting for broker to shut down
[2019-08-01 15:52:15 DEBUG] Warning! Broker was not shut down after 15 seconds.
MarcDelahousse commented 5 years ago

Celery fails to shut redis down. We'll need the celery logs