Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PWA LS Cache issues #60

Closed
apertome opened this issue Aug 1, 2018 · 8 comments
Closed

PWA LS Cache issues #60

apertome opened this issue Aug 1, 2018 · 8 comments
Assignees
Labels

Comments

@apertome
Copy link
Contributor

apertome commented Aug 1, 2018

Multiple PWA instances are seeing a large number of stale hosts in their local LS caches; it appears that records are expiring and getting renewed with a different ID, and PWA isn't matching them up with the old records.

PWA should be using the LS's client UUID field, but I need to review what the cache is doing, as I suspect it's not doing so.

@apertome apertome added the bug label Aug 1, 2018
@apertome apertome added this to the sprint-20180813 milestone Aug 1, 2018
@apertome apertome self-assigned this Aug 1, 2018
apertome added a commit that referenced this issue Aug 13, 2018
@apertome
Copy link
Contributor Author

While troubleshooting the stale hosts problem, I noticed that the cache ignores all hosts that fail reverse dns lookups. I fixed that as part of this issue.

@apertome apertome removed this from the sprint-20180910 milestone Sep 6, 2018
apertome added a commit that referenced this issue Sep 14, 2018
apertome added a commit that referenced this issue Sep 14, 2018
apertome added a commit that referenced this issue Sep 21, 2018
@apertome
Copy link
Contributor Author

There was no expiration of stale/old hosts happening; I have added that. Still working on the details of updating a host/service record.

@ShawnMcKee
Copy link

Any update? We are seeing MOST of our hosts in psconfig.opensciencegrid.org as STALE.

apertome added a commit that referenced this issue Oct 23, 2018
apertome added a commit that referenced this issue Oct 24, 2018
apertome added a commit that referenced this issue Oct 26, 2018
apertome added a commit that referenced this issue Oct 26, 2018
apertome added a commit that referenced this issue Oct 26, 2018
apertome added a commit that referenced this issue Oct 26, 2018
apertome added a commit that referenced this issue Oct 26, 2018
@apertome
Copy link
Contributor Author

I have a fix for this, testing in a few places

@apertome apertome added this to the sprint-20181105 milestone Oct 26, 2018
@apertome
Copy link
Contributor Author

apertome commented Nov 6, 2018

Fix seems to work, closing this

@apertome apertome closed this as completed Nov 6, 2018
@ShawnMcKee
Copy link

ShawnMcKee commented Nov 6, 2018 via email

@apertome
Copy link
Contributor Author

apertome commented Nov 6, 2018 via email

@apertome
Copy link
Contributor Author

apertome commented Nov 7, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants