wojciech.kapcia@tigase.net opened 4 years ago
|
|
This comes as a result of seeing gigantic pep-sync stanza with user witch had hundreds of resources there... |
|
I've implemented "ghostbuster" and adjusted I still need to improve the synchronization of However, I'm not sure if we should sync |
|
I think that there is a problem with addressing:
|
|
@wojtek It should be fixed now in Tigase PubSub and in Tigase ACS PubSub. The main issue was with the serialization of entries to sync items between cluster nodes but PubSub was missing a method to retrieve service JID from the item. I've added a missing method and fixed the issue in ACS. |
|
Currently we have this version deployed:
It should contain the fix (https://github.com/tigase/tigase-acs-pubsub/commit/9dad8da78429e57762ef32d844919f5440e53ca7) yet the errors continue:
I checked the CloudWatch logs and it has entries like this:
and looking at the code it would mean that the actual JID stored in It seems it should be ok and fixed. After taking a look at the amount of errors it seems those are declining. Should we assume that the incorrect values were added to the collections and then during the upgrade those faulty values were synchronised back but with the pass of time those should be removed and the issue should resolve itself over time? Captura de pantalla 2021-05-28 a las 13.58.43.png Captura de pantalla 2021-05-28 a las 14.12.21.png |
|
Yes, you are correct. Invalid values were in the cache and were propagate to new cluster nodes when they joined the cluster. Now, as we do not have old/bad nodes, we need to wait for a new nodes to clean up those entries and everything should be back to normal. |
|
wojciech.kapcia@tigase.net added "Related" tigase/_server/server-core#1256 5 months ago
|
Type |
Task
|
Priority |
Normal
|
Assignee | |
Version |
Candidate for next minor release
|
Spent time |
7h
|
-
tigase/_server/tigase-acs-pubsub#22 You are not authorized to access this issue
As per our discussion in chat - it could be convenient to have a mechanism similar to MUC's ghostbuster for PEP.