From 0560f09767e31f3cf7f0070f39009c8c27d3ca31 Mon Sep 17 00:00:00 2001 From: Arko Dasgupta Date: Fri, 3 Feb 2023 10:20:34 -0800 Subject: [PATCH] fix function names for RL Infra The create and delete function names were incorrectly swapped. Signed-off-by: Arko Dasgupta --- internal/infrastructure/kubernetes/infra.go | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/internal/infrastructure/kubernetes/infra.go b/internal/infrastructure/kubernetes/infra.go index 3d297ea4e0..80a8d57ccc 100644 --- a/internal/infrastructure/kubernetes/infra.go +++ b/internal/infrastructure/kubernetes/infra.go @@ -91,24 +91,24 @@ func (i *Infra) CreateOrUpdateRateLimitInfra(ctx context.Context, infra *ir.Rate if infra == nil { return errors.New("ratelimit infra ir is nil") } - - if err := i.deleteRateLimitService(ctx, infra); err != nil { + if err := i.createOrUpdateRateLimitServiceAccount(ctx, infra); err != nil { return err } - if err := i.deleteRateLimitDeployment(ctx, infra); err != nil { + if err := i.createOrUpdateRateLimitConfigMap(ctx, infra); err != nil { return err } - if err := i.deleteRateLimitConfigMap(ctx, infra); err != nil { + if err := i.createOrUpdateRateLimitDeployment(ctx, infra); err != nil { return err } - if err := i.deleteRateLimitServiceAccount(ctx, infra); err != nil { + if err := i.createOrUpdateRateLimitService(ctx, infra); err != nil { return err } return nil + } // DeleteRateLimitInfra removes the managed kube infra, if it doesn't exist. @@ -116,22 +116,22 @@ func (i *Infra) DeleteRateLimitInfra(ctx context.Context, infra *ir.RateLimitInf if infra == nil { return errors.New("ratelimit infra ir is nil") } - if err := i.createOrUpdateRateLimitServiceAccount(ctx, infra); err != nil { + + if err := i.deleteRateLimitService(ctx, infra); err != nil { return err } - if err := i.createOrUpdateRateLimitConfigMap(ctx, infra); err != nil { + if err := i.deleteRateLimitDeployment(ctx, infra); err != nil { return err } - if err := i.createOrUpdateRateLimitDeployment(ctx, infra); err != nil { + if err := i.deleteRateLimitConfigMap(ctx, infra); err != nil { return err } - if err := i.createOrUpdateRateLimitService(ctx, infra); err != nil { + if err := i.deleteRateLimitServiceAccount(ctx, infra); err != nil { return err } return nil - }