From 5ad4414931f5da288e035eaef537a33fa3ba1174 Mon Sep 17 00:00:00 2001 From: Paddy Date: Wed, 2 May 2018 11:44:58 -0700 Subject: [PATCH] Don't reimplement policy translation. We already have this in spaner_database_iam, no need to reimplement it here. --- google/iam_spanner_instance.go | 18 ------------------ 1 file changed, 18 deletions(-) diff --git a/google/iam_spanner_instance.go b/google/iam_spanner_instance.go index c3eb4082..c0251206 100644 --- a/google/iam_spanner_instance.go +++ b/google/iam_spanner_instance.go @@ -110,21 +110,3 @@ func (u *SpannerInstanceIamUpdater) GetMutexKey() string { func (u *SpannerInstanceIamUpdater) DescribeResource() string { return fmt.Sprintf("Spanner Instance: %s/%s", u.project, u.instance) } - -func resourceManagerToSpannerPolicy(p *cloudresourcemanager.Policy) (*spanner.Policy, error) { - out := &spanner.Policy{} - err := Convert(p, out) - if err != nil { - return nil, errwrap.Wrapf("Cannot convert a resourcemanager policy to a spanner policy: {{err}}", err) - } - return out, nil -} - -func spannerToResourceManagerPolicy(p *spanner.Policy) (*cloudresourcemanager.Policy, error) { - out := &cloudresourcemanager.Policy{} - err := Convert(p, out) - if err != nil { - return nil, errwrap.Wrapf("Cannot convert a spanner policy to a resourcemanager policy: {{err}}", err) - } - return out, nil -}