From d1eef45f8cc34e30af56f93ea96ba7ae604fd096 Mon Sep 17 00:00:00 2001 From: Shane Bryzak Date: Mon, 11 Mar 2024 16:38:42 +1000 Subject: [PATCH] review comments --- pkg/signup/service/signup_service.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/signup/service/signup_service.go b/pkg/signup/service/signup_service.go index 08e1700c..2622ba3d 100644 --- a/pkg/signup/service/signup_service.go +++ b/pkg/signup/service/signup_service.go @@ -450,7 +450,7 @@ func (s *ServiceImpl) DoGetSignup(ctx *gin.Context, provider ResourceProvider, u // If UserSignup status is complete as active // Retrieve MasterUserRecord resource from the host cluster and use its status - mur, err := provider.GetMasterUserRecord(userSignup.Status.CompliantUsername) + mur, err := s.Services().InformerService().GetMasterUserRecord(userSignup.Status.CompliantUsername) if err != nil { return nil, errs.Wrap(err, fmt.Sprintf("error when retrieving MasterUserRecord for completed UserSignup %s", userSignup.GetName())) } @@ -473,7 +473,7 @@ func (s *ServiceImpl) DoGetSignup(ctx *gin.Context, provider ResourceProvider, u // #### The rest of this code only makes sense when there is a ProvisionedTime set, so we include it in the same block #### // Lookup the user's tier - tier, err := provider.GetUserTier(mur.Spec.TierName) + tier, err := s.Services().InformerService().GetUserTier(mur.Spec.TierName) if err != nil { return nil, errs.Wrap(err, fmt.Sprintf("error when retrieving UserTier [%s] for MasterUserRecord [%s]", mur.Spec.TierName, mur.GetName())) }