v1 is currently identical to v1beta1. Upstream we have automation that
makes sure the two protos are identical (apart from their package).
Signed-off-by: Nic Cope <nicc@rk0n.org>
Notably this includes a handy new linter that warns when you don't use
GetFoo methods to access protobuf fields.
Signed-off-by: Nic Cope <nicc@rk0n.org>
This way even if the RunFunctionRequest doesn't include a desired XR,
GetDesiredComposedResource will return a datastructure that you can
mutate and return without fear of writing to a nil map.
Signed-off-by: Nic Cope <nicc@rk0n.org>
This makes it more convenient and discoverable to get and set arbitrary
unstructured data from a composed or composite resource.
Signed-off-by: Nic Cope <nicc@rk0n.org>
This makes it possible to mutate struct values while iterating over the
map of composed resources. I found myself naturally wanting to do this
while writing a Function. e.g.:
desired, _ = request.GetDesiredComposedResources(req)
for _, dr := range desired {
dr.Ready = resource.ReadyTrue
}
_ = response.SetDesiredComposedResources(desired)
I haven't made the same change for observed resources, since they're
supposed to be read-only to a Function.
Signed-off-by: Nic Cope <nicc@rk0n.org>
These just served to obfuscate what the types actually were - a map of
resource name to observed/desired composed resource structs.
Signed-off-by: Nic Cope <nicc@rk0n.org>
Nic Cope
Clément Blaise
Philippe Scorsolini