crossplane-runtime
                                
                                 crossplane-runtime copied to clipboard
                                
                                    crossplane-runtime copied to clipboard
                            
                            
                            
                        Add EquateParameters cmp helper
What problem are you facing?
https://github.com/crossplaneio/stack-gcp/pull/151/files?file-filters%5B%5D=.go#diff-888e0ec46a21ebd2bbdc8d86d4553d9bR92
	return cmp.Equal(in, currentParams, cmpopts.IgnoreInterfaces(struct{ resource.AttributeReferencer }{}))
We frequently use the above cmpopt when determining whether managed resources are equivalent in the IsUpToDate method of each resource's controller. The readability of this option is a little opaque in how it relates to Crossplane.
How could Crossplane help solve your problem?
Perhaps we should add a function like our existing EquateErrors that equates managed resource parameters? This could just be an abstraction on IgnoreInterfaces to begin with.
Hi @negz, is this issue still valid ?? If yes, I would like to work on this.
I believe so! CC @muvaf who may have an opinion here.
Thanks, I will pick this up then.
Hi! I think resource.AttributeReferencer is remnant of the old approach we had taken back then. All reference-related functionality resides in pkg/reference package. In fact pkg/resource/reference.go can be deleted, @aditya109 this could be your first contribution 🙂
Thanks @muvaf, will keep it in mind. Excited to try working on this repo.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Crossplane does not currently have enough maintainers to address every issue and pull request. This issue has been automatically marked as stale because it has had no activity in the last 90 days. It will be closed in 14 days if no further activity occurs. Leaving a comment starting with /fresh will mark this issue as not stale.