You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think we shouldn't append the .* to the filter (which used to be important when selecting by self-links, but not anymore) and let users set their own regex if they choose to.
@icco So AFAIK the resources in .all method used to be looked up by self-link attribute, making it necessary to match my_instance to https://www.googleapis.com/compute/v1/projects/my-project/zones/australia-southeast1-c/instances/my_instance. Otherwise I don't see much need with the leading .* in (:filter => "name eq .*#{identity}")
The problem of changing it to (:filter => "name eq .*#{identity}") is that if someone relied on old behavior, i.e. server matching my_server this will break their lookup.
So, if I'm reading it right the "name eq .*#{identity}" logic is not needed anymore.
I'm talking about the "Servers" for example:
fog-google/lib/fog/compute/google/models/servers.rb
Lines 34 to 38 in 8780575
I think we shouldn't append the
.*
to the filter (which used to be important when selecting by self-links, but not anymore) and let users set their own regex if they choose to.This will also get rid of #33
This is a breaking change so setting to milestone
2.0
@icco WDYT?
The text was updated successfully, but these errors were encountered: