resources: Adjust the remote Get cache so it does not get evicted on restarts

We fill improve this, soon.
This commit is contained in:
Bjørn Erik Pedersen 2021-11-30 16:54:01 +01:00
parent 8aa7257f65
commit 08a863e1e8
No known key found for this signature in database
GPG key ID: 330E6E2BD4859D8F

View file

@ -156,7 +156,10 @@ func (c *Client) FromRemote(uri string, options map[string]interface{}) (resourc
resourceID := helpers.HashString(uri, options)
return c.rs.ResourceCache.GetOrCreate(resources.ResourceCacheKey(resourceID), func() (resource.Resource, error) {
// This caches to memory and will, in server mode, not be evicted unless the resourceID changes
// or the server restarts.
// There is ongoing work to improve this.
return c.rs.ResourceCache.GetOrCreate(resourceID, func() (resource.Resource, error) {
method, reqBody, err := getMethodAndBody(options)
if err != nil {
return nil, errors.Wrapf(err, "failed to get method or body for resource %s", uri)