Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: use deterministic sort for identifier field (#93)
@kumargauravsharma noticed that when generating the User resource package in the elasticache-controller, sometimes he saw the Identifier field be User.Spec.UserID. Other times it was User.Spec.UserName. Turns out in the `pkg/model.CRD:SpecIdentifierField()` method, we were looping over the `CRD.SpecFields` map keys (which are not ordered) instead of a sorted list of spec field names, which is returned by the `CRD.SpecFieldNames()` method. By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
- Loading branch information