From e401e522f1ac7f8ad1ebc81778b389116dc0b343 Mon Sep 17 00:00:00 2001 From: Pieszka Date: Tue, 9 Jul 2024 10:52:35 +0200 Subject: [PATCH 1/4] initial configuration imports and stuff naming changed and barebones dryRun corrected returning errors moved to step, simple unit tests step for generating RuntimeID two steps created but no tests yet just config instead of client refactor finished GenerateRuntimeID tests pass minimal tests logging yaml in dryRun mode --- cmd/broker/main.go | 20 +++- cmd/broker/provisioning.go | 11 +- go.mod | 39 +++---- go.sum | 88 +++++++-------- internal/broker/broker.go | 2 + internal/kim/config.go | 21 ++++ internal/kim/config_test.go | 28 +++++ .../create_runtime_resource_step.go | 102 ++++++++++++++++++ .../create_runtime_resource_step_test.go | 46 ++++++++ .../provisioning/generate_runtime_id_step.go | 80 ++++++++++++++ .../generate_runtime_id_step_test.go | 73 +++++++++++++ resources/keb/templates/deployment.yaml | 10 ++ resources/keb/values.yaml | 21 ++-- 13 files changed, 463 insertions(+), 78 deletions(-) create mode 100644 internal/kim/config.go create mode 100644 internal/kim/config_test.go create mode 100644 internal/process/provisioning/create_runtime_resource_step.go create mode 100644 internal/process/provisioning/create_runtime_resource_step_test.go create mode 100644 internal/process/provisioning/generate_runtime_id_step.go create mode 100644 internal/process/provisioning/generate_runtime_id_step_test.go diff --git a/cmd/broker/main.go b/cmd/broker/main.go index 71c6ba7833..d7aacbd5c5 100644 --- a/cmd/broker/main.go +++ b/cmd/broker/main.go @@ -237,11 +237,7 @@ func main() { health.NewServer(cfg.Host, cfg.StatusPort, logs).ServeAsync() go periodicProfile(logger, cfg.Profiler) - logs.Infof("Setting provisioner timeouts: provisioning=%s, deprovisioning=%s", cfg.Provisioner.ProvisioningTimeout, cfg.Provisioner.DeprovisioningTimeout) - logs.Infof("Setting staged manager configuration: provisioning=%s, deprovisioning=%s, update=%s", cfg.Provisioning, cfg.Deprovisioning, cfg.Update) - logs.Infof("InfrastructureManagerIntegrationDisabled: %v", cfg.InfrastructureManagerIntegrationDisabled) - logs.Infof("Archiving enabled: %v, dry run: %v", cfg.ArchiveEnabled, cfg.ArchiveDryRun) - logs.Infof("Cleaning enabled: %v, dry run: %v", cfg.CleaningEnabled, cfg.CleaningDryRun) + logConfiguration(logs, cfg) // create provisioner client provisionerClient := provisioner.NewProvisionerClient(cfg.Provisioner.URL, cfg.DumpProvisionerRequests, logs.WithField("service", "provisioner")) @@ -399,6 +395,20 @@ func main() { fatalOnError(http.ListenAndServe(cfg.Host+":"+cfg.Port, svr), logs) } +func logConfiguration(logs *logrus.Logger, cfg Config) { + logs.Infof("Setting provisioner timeouts: provisioning=%s, deprovisioning=%s", cfg.Provisioner.ProvisioningTimeout, cfg.Provisioner.DeprovisioningTimeout) + logs.Infof("Setting staged manager configuration: provisioning=%s, deprovisioning=%s, update=%s", cfg.Provisioning, cfg.Deprovisioning, cfg.Update) + logs.Infof("InfrastructureManagerIntegrationDisabled: %v", cfg.InfrastructureManagerIntegrationDisabled) + logs.Infof("Archiving enabled: %v, dry run: %v", cfg.ArchiveEnabled, cfg.ArchiveDryRun) + logs.Infof("Cleaning enabled: %v, dry run: %v", cfg.CleaningEnabled, cfg.CleaningDryRun) + logs.Infof("KIM enabled: %t, dry run: %t, view only CR: %t, plans: %s, KIM only plans: %s", + cfg.Broker.KimConfig.Enabled, + cfg.Broker.KimConfig.DryRun, + cfg.Broker.KimConfig.ViewOnly, + cfg.Broker.KimConfig.Plans, + cfg.Broker.KimConfig.KimOnlyPlans) +} + func createAPI(router *mux.Router, servicesConfig broker.ServicesConfig, planValidator broker.PlanValidator, cfg *Config, db storage.BrokerStorage, provisionQueue, deprovisionQueue, updateQueue *process.Queue, logger lager.Logger, logs logrus.FieldLogger, planDefaults broker.PlanDefaults, kcBuilder kubeconfig.KcBuilder) { suspensionCtxHandler := suspension.NewContextUpdateHandler(db.Operations(), provisionQueue, deprovisionQueue, logs) diff --git a/cmd/broker/provisioning.go b/cmd/broker/provisioning.go index 7392b73bd5..34ee1f1f21 100644 --- a/cmd/broker/provisioning.go +++ b/cmd/broker/provisioning.go @@ -2,7 +2,6 @@ package main import ( "context" - "github.com/kyma-project/kyma-environment-broker/common/hyperscaler" "github.com/kyma-project/kyma-environment-broker/internal/avs" "github.com/kyma-project/kyma-environment-broker/internal/process" @@ -79,6 +78,16 @@ func NewProvisioningProcessingQueue(ctx context.Context, provisionManager *proce stage: createRuntimeStageName, step: provisioning.NewCreateRuntimeWithoutKymaStep(db.Operations(), db.RuntimeStates(), db.Instances(), provisionerClient), }, + { + condition: provisioning.SkipForOwnClusterPlan, + stage: createRuntimeStageName, + step: provisioning.NewGenerateRuntimeIDStep(db.Operations(), db.RuntimeStates(), db.Instances()), + }, + { + condition: provisioning.SkipForOwnClusterPlan, + stage: createRuntimeStageName, + step: provisioning.NewCreateRuntimeResourceStep(db.Operations(), db.RuntimeStates(), db.Instances(), cfg.Broker.KimConfig), + }, { condition: provisioning.DoForOwnClusterPlanOnly, stage: createRuntimeStageName, diff --git a/go.mod b/go.mod index b3be1d0245..9017652a8b 100644 --- a/go.mod +++ b/go.mod @@ -1,6 +1,6 @@ module github.com/kyma-project/kyma-environment-broker -go 1.22.4 +go 1.22.5 require ( code.cloudfoundry.org/lager v2.0.0+incompatible @@ -22,6 +22,7 @@ require ( github.com/kyma-incubator/compass/components/director v0.0.0-20240329103248-7710e72be80a github.com/kyma-project/control-plane/components/provisioner v0.0.0-20240529135026-c4ef757de24f github.com/kyma-project/control-plane/components/schema-migrator v0.0.0-20240612080429-83a7c0eb13b8 + github.com/kyma-project/infrastructure-manager v0.0.0-20240705113756-2ea9b23b6154 github.com/lib/pq v1.10.9 github.com/matryer/is v1.4.1 github.com/pivotal-cf/brokerapi/v8 v8.2.3 @@ -31,16 +32,16 @@ require ( github.com/sirupsen/logrus v1.9.3 github.com/stretchr/testify v1.9.0 github.com/vrischmann/envconfig v1.3.0 - golang.org/x/exp v0.0.0-20240213143201-ec583247a57a + golang.org/x/exp v0.0.0-20240613232115-7f521ea00fb8 golang.org/x/oauth2 v0.21.0 golang.org/x/time v0.5.0 gopkg.in/yaml.v2 v2.4.0 gopkg.in/yaml.v3 v3.0.1 - k8s.io/api v0.30.1 + k8s.io/api v0.30.2 k8s.io/apiextensions-apiserver v0.30.1 - k8s.io/apimachinery v0.30.1 + k8s.io/apimachinery v0.30.2 k8s.io/client-go v11.0.1-0.20190409021438-1a26190bd76a+incompatible - sigs.k8s.io/controller-runtime v0.17.2 + sigs.k8s.io/controller-runtime v0.18.4 ) require ( @@ -53,24 +54,23 @@ require ( github.com/Azure/go-autorest/tracing v0.6.0 // indirect github.com/Masterminds/goutils v1.1.1 // indirect github.com/Masterminds/semver v1.5.0 // indirect - github.com/Microsoft/go-winio v0.6.1 // indirect + github.com/Microsoft/go-winio v0.6.2 // indirect github.com/agnivade/levenshtein v1.1.1 // indirect github.com/beorn7/perks v1.0.1 // indirect github.com/cespare/xxhash/v2 v2.2.0 // indirect - github.com/containerd/log v0.1.0 // indirect - github.com/davecgh/go-spew v1.1.1 // indirect + github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc // indirect github.com/distribution/reference v0.5.0 // indirect github.com/docker/go-units v0.5.0 // indirect github.com/emicklei/go-restful/v3 v3.11.2 // indirect github.com/evanphx/json-patch v5.9.0+incompatible // indirect github.com/evanphx/json-patch/v5 v5.9.0 // indirect github.com/felixge/httpsnoop v1.0.4 // indirect - github.com/go-logr/logr v1.4.1 // indirect + github.com/gardener/gardener v1.98.0 // indirect + github.com/go-logr/logr v1.4.2 // indirect github.com/go-logr/stdr v1.2.2 // indirect - github.com/go-logr/zapr v1.3.0 // indirect - github.com/go-openapi/jsonpointer v0.20.2 // indirect + github.com/go-openapi/jsonpointer v0.21.0 // indirect github.com/go-openapi/jsonreference v0.20.4 // indirect - github.com/go-openapi/swag v0.22.9 // indirect + github.com/go-openapi/swag v0.23.0 // indirect github.com/go-sql-driver/mysql v1.5.0 // indirect github.com/gogo/protobuf v1.3.2 // indirect github.com/golang-jwt/jwt/v4 v4.5.0 // indirect @@ -91,7 +91,6 @@ require ( github.com/mitchellh/mapstructure v1.5.0 // indirect github.com/mitchellh/reflectwalk v1.0.2 // indirect github.com/moby/docker-image-spec v1.3.1 // indirect - github.com/moby/term v0.5.0 // indirect github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd // indirect github.com/modern-go/reflect2 v1.0.2 // indirect github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 // indirect @@ -114,28 +113,24 @@ require ( github.com/xeipuuv/gojsonpointer v0.0.0-20190905194746-02993c407bfb // indirect github.com/xeipuuv/gojsonreference v0.0.0-20180127040603-bd5ef7bd5415 // indirect github.com/xeipuuv/gojsonschema v1.2.0 // indirect - go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.48.0 // indirect - go.opentelemetry.io/otel v1.23.1 // indirect + go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.49.0 // indirect + go.opentelemetry.io/otel v1.24.0 // indirect go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.23.1 // indirect - go.opentelemetry.io/otel/metric v1.23.1 // indirect + go.opentelemetry.io/otel/metric v1.24.0 // indirect go.opentelemetry.io/otel/sdk v1.23.1 // indirect - go.opentelemetry.io/otel/trace v1.23.1 // indirect + go.opentelemetry.io/otel/trace v1.24.0 // indirect go.uber.org/atomic v1.11.0 // indirect - go.uber.org/zap v1.27.0 // indirect golang.org/x/crypto v0.24.0 // indirect - golang.org/x/mod v0.18.0 // indirect golang.org/x/net v0.26.0 // indirect - golang.org/x/sync v0.7.0 // indirect golang.org/x/sys v0.21.0 // indirect golang.org/x/term v0.21.0 // indirect golang.org/x/text v0.16.0 // indirect - golang.org/x/tools v0.22.0 // indirect google.golang.org/protobuf v1.34.1 // indirect gopkg.in/inf.v0 v0.9.1 // indirect gotest.tools/v3 v3.1.0 // indirect k8s.io/klog/v2 v2.120.1 // indirect k8s.io/kube-openapi v0.0.0-20240228011516-70dd3763d340 // indirect - k8s.io/utils v0.0.0-20240102154912-e7106e64919e // indirect + k8s.io/utils v0.0.0-20240502163921-fe8a2dddb1d0 // indirect sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd // indirect sigs.k8s.io/structured-merge-diff/v4 v4.4.1 // indirect sigs.k8s.io/yaml v1.4.0 // indirect diff --git a/go.sum b/go.sum index 55dfe161fd..264b525512 100644 --- a/go.sum +++ b/go.sum @@ -35,8 +35,8 @@ github.com/Masterminds/semver v1.5.0 h1:H65muMkzWKEuNDnfl9d70GUjFniHKHRbFPGBuZ3Q github.com/Masterminds/semver v1.5.0/go.mod h1:MB6lktGJrhw8PrUyiEoblNEGEQ+RzHPF078ddwwvV3Y= github.com/Masterminds/sprig v2.22.0+incompatible h1:z4yfnGrZ7netVz+0EDJ0Wi+5VZCSYp4Z0m2dk6cEM60= github.com/Masterminds/sprig v2.22.0+incompatible/go.mod h1:y6hNFY5UBTIWBxnzTeuNhlNS5hqE0NB0E6fgfo2Br3o= -github.com/Microsoft/go-winio v0.6.1 h1:9/kr64B9VUZrLm5YYwbGtUJnMgqWVOdUAXu6Migciow= -github.com/Microsoft/go-winio v0.6.1/go.mod h1:LRdKpFKfdobln8UmuiYcKPot9D2v6svN5+sAH+4kjUM= +github.com/Microsoft/go-winio v0.6.2 h1:F2VQgta7ecxGYO8k3ZZz3RS8fVIXVxONVUPlNERoyfY= +github.com/Microsoft/go-winio v0.6.2/go.mod h1:yd8OoFMLzJbo9gZq8j5qaps8bJ9aShtEA8Ipt1oGCvU= github.com/PuerkitoBio/goquery v1.9.2 h1:4/wZksC3KgkQw7SQgkKotmKljk0M6V8TUvA8Wb4yPeE= github.com/PuerkitoBio/goquery v1.9.2/go.mod h1:GHPCaP0ODyyxqcNoFGYlAprUFH81NuRPd0GX3Zu2Mvk= github.com/agnivade/levenshtein v1.1.1 h1:QY8M92nrzkmr798gCo3kmMyqXFzdQVpxLlGPRBij0P8= @@ -47,6 +47,8 @@ github.com/alecthomas/units v0.0.0-20151022065526-2efee857e7cf/go.mod h1:ybxpYRF github.com/alecthomas/units v0.0.0-20190717042225-c3de453c63f4/go.mod h1:ybxpYRFXyAe+OPACYpWeL0wqObRcbAqCMya13uyzqw0= github.com/andreyvit/diff v0.0.0-20170406064948-c7f18ee00883 h1:bvNMNQO63//z+xNgfBlViaCIJKLlCJ6/fmUseuG0wVQ= github.com/andreyvit/diff v0.0.0-20170406064948-c7f18ee00883/go.mod h1:rCTlJbsFo29Kk6CurOXKm700vrz8f0KW0JNfpkRJY/8= +github.com/andybalholm/brotli v1.1.0 h1:eLKJA0d02Lf0mVpIDgYnqXcUn0GqVmEFny3VuID1U3M= +github.com/andybalholm/brotli v1.1.0/go.mod h1:sms7XGricyQI9K10gOSf56VKKWS4oLer58Q+mhRPtnY= github.com/andybalholm/cascadia v1.3.2 h1:3Xi6Dw5lHF15JtdcmAHD3i1+T8plmv7BQ/nsViSLyss= github.com/andybalholm/cascadia v1.3.2/go.mod h1:7gtRlve5FxPPgIgX36uWBX58OdBsSS6lUvCFb+h7KvU= github.com/arbovm/levenshtein v0.0.0-20160628152529-48b4e1c0c4d0 h1:jfIu9sQUG6Ig+0+Ap1h4unLjW6YQJpKZVmUzxsD4E/Q= @@ -64,8 +66,9 @@ github.com/containerd/log v0.1.0 h1:TCJt7ioM2cr/tfR8GPbGf9/VRAX8D2B4PjzCpfX540I= github.com/containerd/log v0.1.0/go.mod h1:VRRf09a7mHDIRezVKTRCrOq78v577GXq3bSa3EhrzVo= github.com/creack/pty v1.1.9/go.mod h1:oKZEueFk5CKHvIhNR5MUki03XCEU+Q6VDXinZuGJ33E= github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38= -github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c= github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38= +github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc h1:U9qPSI2PIWSS1VwoXQT9A3Wy9MM3WgvqSxFWenqJduM= +github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38= github.com/dgryski/trifles v0.0.0-20200323201526-dd97f9abfb48 h1:fRzb/w+pyskVMQ+UbP35JkH8yB7MYb4q/qhBarqZE6g= github.com/dgryski/trifles v0.0.0-20200323201526-dd97f9abfb48/go.mod h1:if7Fbed8SFyPtHLHbg49SI7NAdJiC5WIA09pe59rfAA= github.com/distribution/reference v0.5.0 h1:/FUIFXtfc/x2gpa5/VGfiGLuOIdYa1t65IKK2OFGvA0= @@ -88,8 +91,8 @@ github.com/felixge/httpsnoop v1.0.4 h1:NFTV2Zj1bL4mc9sqWACXbQFVBBg2W3GPvqp8/ESS2 github.com/felixge/httpsnoop v1.0.4/go.mod h1:m8KPJKqk1gH5J9DgRY2ASl2lWCfGKXixSwevea8zH2U= github.com/fsnotify/fsnotify v1.7.0 h1:8JEhPFa5W2WU7YfeZzPNqzMP6Lwt7L2715Ggo0nosvA= github.com/fsnotify/fsnotify v1.7.0/go.mod h1:40Bi/Hjc2AVfZrqy+aj+yEI+/bRxZnMJyTJwOpGvigM= -github.com/gardener/gardener v1.74.1 h1:K4b/vSmFxVd3QgICJtuQu3e+FJn/bMqQepdBtXKRbpk= -github.com/gardener/gardener v1.74.1/go.mod h1:uSkzPPoAEvdU1fvciTAsZFxPQ9vQpMbMFRJLMQgdfEQ= +github.com/gardener/gardener v1.98.0 h1:tu9ntrn6+rpJw8wscr8+TbDYr2pgQem9KbFw80nIXP8= +github.com/gardener/gardener v1.98.0/go.mod h1:EfvKbBF53a52Wz16/Qe8hhpTZTaJ/P+CRWRr35BVVq0= github.com/go-co-op/gocron v1.37.0 h1:ZYDJGtQ4OMhTLKOKMIch+/CY70Brbb1dGdooLEhh7b0= github.com/go-co-op/gocron v1.37.0/go.mod h1:3L/n6BkO7ABj+TrfSVXLRzsP26zmikL4ISkLQ0O8iNY= github.com/go-kit/kit v0.8.0/go.mod h1:xBxKIO96dXMWWy0MnWVtmwkA9/13aqxPnvrjFYMA2as= @@ -97,25 +100,26 @@ github.com/go-kit/kit v0.9.0/go.mod h1:xBxKIO96dXMWWy0MnWVtmwkA9/13aqxPnvrjFYMA2 github.com/go-logfmt/logfmt v0.3.0/go.mod h1:Qt1PoO58o5twSAckw1HlFXLmHsOX5/0LbT9GBnD5lWE= github.com/go-logfmt/logfmt v0.4.0/go.mod h1:3RMwSq7FuexP4Kalkev3ejPJsZTpXXBr9+V4qmtdjCk= github.com/go-logr/logr v1.2.2/go.mod h1:jdQByPbusPIv2/zmleS9BjJVeZ6kBagPoEUsqbVz/1A= -github.com/go-logr/logr v1.4.1 h1:pKouT5E8xu9zeFC39JXRDukb6JFQPXM5p5I91188VAQ= -github.com/go-logr/logr v1.4.1/go.mod h1:9T104GzyrTigFIr8wt5mBrctHMim0Nb2HLGrmQ40KvY= +github.com/go-logr/logr v1.4.2 h1:6pFjapn8bFcIbiKo3XT4j/BhANplGihG6tvd+8rYgrY= +github.com/go-logr/logr v1.4.2/go.mod h1:9T104GzyrTigFIr8wt5mBrctHMim0Nb2HLGrmQ40KvY= github.com/go-logr/stdr v1.2.2 h1:hSWxHoqTgW2S2qGc0LTAI563KZ5YKYRhT3MFKZMbjag= github.com/go-logr/stdr v1.2.2/go.mod h1:mMo/vtBO5dYbehREoey6XUKy/eSumjCCveDpRre4VKE= github.com/go-logr/zapr v1.3.0 h1:XGdV8XW8zdwFiwOA2Dryh1gj2KRQyOOoNmBy4EplIcQ= github.com/go-logr/zapr v1.3.0/go.mod h1:YKepepNBd1u/oyhd/yQmtjVXmm9uML4IXUgMOwR8/Gg= -github.com/go-openapi/jsonpointer v0.20.2 h1:mQc3nmndL8ZBzStEo3JYF8wzmeWffDH4VbXz58sAx6Q= -github.com/go-openapi/jsonpointer v0.20.2/go.mod h1:bHen+N0u1KEO3YlmqOjTT9Adn1RfD91Ar825/PuiRVs= +github.com/go-openapi/jsonpointer v0.21.0 h1:YgdVicSA9vH5RiHs9TZW5oyafXZFc6+2Vc1rr/O9oNQ= +github.com/go-openapi/jsonpointer v0.21.0/go.mod h1:IUyH9l/+uyhIYQ/PXVA41Rexl+kOkAPDdXEYns6fzUY= github.com/go-openapi/jsonreference v0.20.4 h1:bKlDxQxQJgwpUSgOENiMPzCTBVuc7vTdXSSgNeAhojU= github.com/go-openapi/jsonreference v0.20.4/go.mod h1:5pZJyJP2MnYCpoeoMAql78cCHauHj0V9Lhc506VOpw4= -github.com/go-openapi/swag v0.22.9 h1:XX2DssF+mQKM2DHsbgZK74y/zj4mo9I99+89xUmuZCE= -github.com/go-openapi/swag v0.22.9/go.mod h1:3/OXnFfnMAwBD099SwYRk7GD3xOrr1iL7d/XNLXVVwE= +github.com/go-openapi/swag v0.23.0 h1:vsEVJDUo2hPJ2tu0/Xc+4noaxyEffXNIs3cOULZ+GrE= +github.com/go-openapi/swag v0.23.0/go.mod h1:esZ8ITTYEsH1V2trKHjAN8Ai7xHb8RV+YSZ577vPjgQ= github.com/go-sql-driver/mysql v1.4.0/go.mod h1:zAC/RDZ24gD3HViQzih4MyKcchzm+sOG5ZlKdlhCg5w= github.com/go-sql-driver/mysql v1.4.1/go.mod h1:zAC/RDZ24gD3HViQzih4MyKcchzm+sOG5ZlKdlhCg5w= github.com/go-sql-driver/mysql v1.5.0 h1:ozyZYNQW3x3HtqT1jira07DN2PArx2v7/mN66gGcHOs= github.com/go-sql-driver/mysql v1.5.0/go.mod h1:DCzpHaOWr8IXmIStZouvnhqoel9Qv2LBy8hT2VhHyBg= github.com/go-stack/stack v1.8.0/go.mod h1:v0f6uXyyMGvRgIKkXu+yp6POWl0qKG85gN/melR3HDY= github.com/go-task/slim-sprig v0.0.0-20230315185526-52ccab3ef572 h1:tfuBGBXKqDEevZMzYi5KSi8KkcZtzBcTgAUUtapy0OI= -github.com/go-task/slim-sprig v0.0.0-20230315185526-52ccab3ef572/go.mod h1:9Pwr4B2jHnOSGXyyzV8ROjYa2ojvAY6HCGYYfMoC3Ls= +github.com/go-task/slim-sprig/v3 v3.0.0 h1:sUs3vkvUymDpBKi3qH1YSqBQk9+9D/8M2mN1vB6EwHI= +github.com/go-task/slim-sprig/v3 v3.0.0/go.mod h1:W848ghGpv3Qj3dhTPRyJypKRiqCdHZiAzKg9hl15HA8= github.com/gocraft/dbr v0.0.0-20190714181702-8114670a83bd h1:GlmMPhEpMWrNOyUaAMpRGy4zkb03eXuTb8TKXr3j0dQ= github.com/gocraft/dbr v0.0.0-20190714181702-8114670a83bd/go.mod h1:BK1nFI5Pp8XJg1sE7oMBzyW32LBuS2r25HlZPa6tXXs= github.com/gogo/protobuf v1.1.1/go.mod h1:r8qH/GZQm5c6nD/R0oafs1akxWv10x8SbQlK7atdtwQ= @@ -141,8 +145,8 @@ github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeN github.com/google/gofuzz v1.0.0/go.mod h1:dBl0BpW6vV/+mYPU4Po3pmUjxk6FQPldtuIdl/M65Eg= github.com/google/gofuzz v1.2.0 h1:xRy4A+RhZaiKjJ1bPfwQ8sedCA+YS2YcCHW6ec7JMi0= github.com/google/gofuzz v1.2.0/go.mod h1:dBl0BpW6vV/+mYPU4Po3pmUjxk6FQPldtuIdl/M65Eg= -github.com/google/pprof v0.0.0-20210720184732-4bb14d4b1be1 h1:K6RDEckDVWvDI9JAJYCmNdQXq6neHJOYx3V6jnqNEec= -github.com/google/pprof v0.0.0-20210720184732-4bb14d4b1be1/go.mod h1:kpwsk12EmLew5upagYY7GY0pfYCcupk39gWOCRROcvE= +github.com/google/pprof v0.0.0-20240424215950-a892ee059fd6 h1:k7nVchz72niMH6YLQNvHSdIE7iqsQxK1P41mySCvssg= +github.com/google/pprof v0.0.0-20240424215950-a892ee059fd6/go.mod h1:kf6iHlnVGwgKolg33glAes7Yg/8iWP8ukqeldJSO7jw= github.com/google/uuid v1.0.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo= github.com/google/uuid v1.4.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo= github.com/google/uuid v1.6.0 h1:NIvaJDMOsjHA8n1jAhLSgzrAzy1Hgr+hNrb57e+94F0= @@ -198,6 +202,8 @@ github.com/kyma-project/control-plane/components/provisioner v0.0.0-202405291350 github.com/kyma-project/control-plane/components/provisioner v0.0.0-20240529135026-c4ef757de24f/go.mod h1:X+h9AdPV8nPmhgTg82BpFKEFsSL7OGSGG4D8+G3giPM= github.com/kyma-project/control-plane/components/schema-migrator v0.0.0-20240612080429-83a7c0eb13b8 h1:btliPhcS1pF+AlQbmLjkV8GuPnTS1VbZ+m9DkPsxDCA= github.com/kyma-project/control-plane/components/schema-migrator v0.0.0-20240612080429-83a7c0eb13b8/go.mod h1:vABrhytVuZpchbdlIVdUDlhB/Q/3GIZld2JmdS2rZ6I= +github.com/kyma-project/infrastructure-manager v0.0.0-20240705113756-2ea9b23b6154 h1:1nE1nmUlhaQAaPA2/p6tcrx6pFJsCKqQzw46OoqI51o= +github.com/kyma-project/infrastructure-manager v0.0.0-20240705113756-2ea9b23b6154/go.mod h1:VIXFOtvzBYNj9RKB5K4do6loUxp9i9XNEvhqAQIZ5G8= github.com/lib/pq v1.0.0/go.mod h1:5WUZQaWbwv1U+lTReE5YruASi9Al49XbQIvNi/34Woo= github.com/lib/pq v1.1.1/go.mod h1:5WUZQaWbwv1U+lTReE5YruASi9Al49XbQIvNi/34Woo= github.com/lib/pq v1.10.9 h1:YXG7RB+JIjhP29X+OtkiDnYaXQwpS4JEWq7dtCCRUEw= @@ -241,10 +247,10 @@ github.com/onrik/logrus v0.11.0 h1:pu+BCaWL36t0yQaj/2UHK2erf88dwssAKOT51mxPUVs= github.com/onrik/logrus v0.11.0/go.mod h1:fO2vlZwIdti6PidD3gV5YKt9Lq5ptpnP293RAe1ITwk= github.com/onsi/ginkgo v1.16.5 h1:8xi0RTUf59SOSfEtZMvwTvXYMzG4gV23XVHOZiXNtnE= github.com/onsi/ginkgo v1.16.5/go.mod h1:+E8gABHa3K6zRBolWtd+ROzc/U5bkGt0FwiG042wbpU= -github.com/onsi/ginkgo/v2 v2.15.0 h1:79HwNRBAZHOEwrczrgSOPy+eFTTlIGELKy5as+ClttY= -github.com/onsi/ginkgo/v2 v2.15.0/go.mod h1:HlxMHtYF57y6Dpf+mc5529KKmSq9h2FpCF+/ZkwUxKM= -github.com/onsi/gomega v1.31.0 h1:54UJxxj6cPInHS3a35wm6BK/F9nHYueZ1NVujHDrnXE= -github.com/onsi/gomega v1.31.0/go.mod h1:DW9aCi7U6Yi40wNVAvT6kzFnEVEI5n3DloYBiKiT6zk= +github.com/onsi/ginkgo/v2 v2.19.0 h1:9Cnnf7UHo57Hy3k6/m5k3dRfGTMXGvxhHFvkDTCTpvA= +github.com/onsi/ginkgo/v2 v2.19.0/go.mod h1:rlwLi9PilAFJ8jCg9UE1QP6VBpd6/xj3SRC0d6TU0To= +github.com/onsi/gomega v1.33.1 h1:dsYjIxxSR755MDmKVsaFQTE22ChNBcuuTWgkUDSubOk= +github.com/onsi/gomega v1.33.1/go.mod h1:U4R44UsT+9eLIaYRB2a5qajjtQYn0hauxvRm16AVYg0= github.com/opencontainers/go-digest v1.0.0 h1:apOUWs51W5PlhuyGyz9FCeeBIOUDA/6nW8Oi/yOhh5U= github.com/opencontainers/go-digest v1.0.0/go.mod h1:0JzlMkj0TRzQZfJkVvzbP0HBR3IKzErnv2BNG4W4MAM= github.com/opencontainers/image-spec v1.1.0 h1:8SG7/vwALn54lVB/0yZ/MMwhFrPYtpEHQb2IpWsCzug= @@ -336,20 +342,20 @@ github.com/xeipuuv/gojsonschema v1.2.0/go.mod h1:anYRn/JVcOK2ZgGU+IjEV4nwlhoK5sQ github.com/yuin/goldmark v1.1.27/go.mod h1:3hX8gzYuyVAZsxl0MRgGTJEmQBFcNTphYh9decYSb74= github.com/yuin/goldmark v1.2.1/go.mod h1:3hX8gzYuyVAZsxl0MRgGTJEmQBFcNTphYh9decYSb74= github.com/yuin/goldmark v1.4.13/go.mod h1:6yULJ656Px+3vBD8DxQVa3kxgyrAnzto9xy5taEt/CY= -go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.48.0 h1:doUP+ExOpH3spVTLS0FcWGLnQrPct/hD/bCPbDRUEAU= -go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.48.0/go.mod h1:rdENBZMT2OE6Ne/KLwpiXudnAsbdrdBaqBvTN8M8BgA= -go.opentelemetry.io/otel v1.23.1 h1:Za4UzOqJYS+MUczKI320AtqZHZb7EqxO00jAHE0jmQY= -go.opentelemetry.io/otel v1.23.1/go.mod h1:Td0134eafDLcTS4y+zQ26GE8u3dEuRBiBCTUIRHaikA= +go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.49.0 h1:jq9TW8u3so/bN+JPT166wjOI6/vQPF6Xe7nMNIltagk= +go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.49.0/go.mod h1:p8pYQP+m5XfbZm9fxtSKAbM6oIllS7s2AfxrChvc7iw= +go.opentelemetry.io/otel v1.24.0 h1:0LAOdjNmQeSTzGBzduGe/rU4tZhMwL5rWgtp9Ku5Jfo= +go.opentelemetry.io/otel v1.24.0/go.mod h1:W7b9Ozg4nkF5tWI5zsXkaKKDjdVjpD4oAt9Qi/MArHo= go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.23.1 h1:o8iWeVFa1BcLtVEV0LzrCxV2/55tB3xLxADr6Kyoey4= go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.23.1/go.mod h1:SEVfdK4IoBnbT2FXNM/k8yC08MrfbhWk3U4ljM8B3HE= go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.23.1 h1:cfuy3bXmLJS7M1RZmAL6SuhGtKUp2KEsrm00OlAXkq4= go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.23.1/go.mod h1:22jr92C6KwlwItJmQzfixzQM3oyyuYLCfHiMY+rpsPU= -go.opentelemetry.io/otel/metric v1.23.1 h1:PQJmqJ9u2QaJLBOELl1cxIdPcpbwzbkjfEyelTl2rlo= -go.opentelemetry.io/otel/metric v1.23.1/go.mod h1:mpG2QPlAfnK8yNhNJAxDZruU9Y1/HubbC+KyH8FaCWI= +go.opentelemetry.io/otel/metric v1.24.0 h1:6EhoGWWK28x1fbpA4tYTOWBkPefTDQnb8WSGXlc88kI= +go.opentelemetry.io/otel/metric v1.24.0/go.mod h1:VYhLe1rFfxuTXLgj4CBiyz+9WYBA8pNGJgDcSFRKBco= go.opentelemetry.io/otel/sdk v1.23.1 h1:O7JmZw0h76if63LQdsBMKQDWNb5oEcOThG9IrxscV+E= go.opentelemetry.io/otel/sdk v1.23.1/go.mod h1:LzdEVR5am1uKOOwfBWFef2DCi1nu3SA8XQxx2IerWFk= -go.opentelemetry.io/otel/trace v1.23.1 h1:4LrmmEd8AU2rFvU1zegmvqW7+kWarxtNOPyeL6HmYY8= -go.opentelemetry.io/otel/trace v1.23.1/go.mod h1:4IpnpJFwr1mo/6HL8XIPJaE9y0+u1KcVmuW7dwFSVrI= +go.opentelemetry.io/otel/trace v1.24.0 h1:CsKnnL4dUAr/0llH9FKuc698G04IrpWV0MQA/Y1YELI= +go.opentelemetry.io/otel/trace v1.24.0/go.mod h1:HPc3Xr/cOApsBI154IU0OI0HJexz+aw5uPdbs3UCjNU= go.opentelemetry.io/proto/otlp v1.1.0 h1:2Di21piLrCqJ3U3eXGCTPHE9R8Nh+0uglSnOyxikMeI= go.opentelemetry.io/proto/otlp v1.1.0/go.mod h1:GpBHCBWiqvVLDqmHZsoMM3C5ySeKTC7ej/RNTae6MdY= go.uber.org/atomic v1.9.0/go.mod h1:fEN4uk6kAWBTFdckzkM89CLk9XfWZrxpCo0nPH17wJc= @@ -369,14 +375,12 @@ golang.org/x/crypto v0.6.0/go.mod h1:OFC/31mSvZgRz0V1QTNCzfAI1aIRzbiufJtkMIlEp58 golang.org/x/crypto v0.17.0/go.mod h1:gCAAfMLgwOJRpTjQ2zCCt2OcSfYMTeZVSRtQlPC7Nq4= golang.org/x/crypto v0.24.0 h1:mnl8DM0o513X8fdIkmyFE/5hTYxbwYOjDS/+rK6qpRI= golang.org/x/crypto v0.24.0/go.mod h1:Z1PMYSOR5nyMcyAVAIQSKCDwalqy85Aqn1x3Ws4L5DM= -golang.org/x/exp v0.0.0-20240213143201-ec583247a57a h1:HinSgX1tJRX3KsL//Gxynpw5CTOAIPhgL4W8PNiIpVE= -golang.org/x/exp v0.0.0-20240213143201-ec583247a57a/go.mod h1:CxmFvTBINI24O/j8iY7H1xHzx2i4OsyguNBmN/uPtqc= +golang.org/x/exp v0.0.0-20240613232115-7f521ea00fb8 h1:yixxcjnhBmY0nkL253HFVIm0JsFHwrHdT3Yh6szTnfY= +golang.org/x/exp v0.0.0-20240613232115-7f521ea00fb8/go.mod h1:jj3sYF3dwk5D+ghuXyeI3r5MFf+NT2An6/9dOA95KSI= golang.org/x/mod v0.2.0/go.mod h1:s0Qsj1ACt9ePp/hMypM3fl4fZqREWJwdYDEqhRiZZUA= golang.org/x/mod v0.3.0/go.mod h1:s0Qsj1ACt9ePp/hMypM3fl4fZqREWJwdYDEqhRiZZUA= golang.org/x/mod v0.6.0-dev.0.20220419223038-86c51ed26bb4/go.mod h1:jJ57K6gSWd91VN4djpZkiMVwK6gcyfeH4XE8wZrZaV4= golang.org/x/mod v0.8.0/go.mod h1:iBbtSCu2XBx23ZKBPSOrRkjjQPZFPuis4dIYUhu/chs= -golang.org/x/mod v0.18.0 h1:5+9lSbEzPSdWkH32vYPBwEpX8KwDbM52Ud9xBUvNlb0= -golang.org/x/mod v0.18.0/go.mod h1:hTbmBsO62+eylJbnUtE2MGJUyE7QWk4xUqPFrRgJ+7c= golang.org/x/net v0.0.0-20181114220301-adae6a3d119a/go.mod h1:mL1N/T3taQHkDXs73rZJwtUhF3w3ftmwwsq0BUmARs4= golang.org/x/net v0.0.0-20190404232315-eb5bcb51f2a3/go.mod h1:t9HGtf8HONx5eT2rtn7q6eTqICYqUVnKs3thJo3Qplg= golang.org/x/net v0.0.0-20190613194153-d28f0bde5980/go.mod h1:z5CRVTTTmAJ677TzLLGU+0bjPO0LkuOLi4/5GtJWs/s= @@ -399,8 +403,6 @@ golang.org/x/sync v0.0.0-20190911185100-cd5d95a43a6e/go.mod h1:RxMgew5VJxzue5/jJ golang.org/x/sync v0.0.0-20201020160332-67f06af15bc9/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM= golang.org/x/sync v0.0.0-20220722155255-886fb9371eb4/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM= golang.org/x/sync v0.1.0/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM= -golang.org/x/sync v0.7.0 h1:YsImfSBoP9QPYL0xyKJPq0gcaJdG3rInoqxTWbfQu9M= -golang.org/x/sync v0.7.0/go.mod h1:Czt+wKu1gCyEFDUtn0jG5QVvpJ6rzVqr5aXyt9drQfk= golang.org/x/sys v0.0.0-20180905080454-ebe1bf3edb33/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY= golang.org/x/sys v0.0.0-20181116152217-5ac8a444bdc5/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY= golang.org/x/sys v0.0.0-20190215142949-d0b11bdaac8a/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY= @@ -452,15 +454,15 @@ golang.org/x/xerrors v0.0.0-20190717185122-a985d3407aa7/go.mod h1:I/5z698sn9Ka8T golang.org/x/xerrors v0.0.0-20191011141410-1b5146add898/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0= golang.org/x/xerrors v0.0.0-20191204190536-9bdfabe68543/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0= golang.org/x/xerrors v0.0.0-20200804184101-5ec99f83aff1/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0= -gomodules.xyz/jsonpatch/v2 v2.2.0 h1:4pT439QV83L+G9FkcCriY6EkpcK6r6bK+A5FBUMI7qY= -gomodules.xyz/jsonpatch/v2 v2.2.0/go.mod h1:WXp+iVDkoLQqPudfQ9GBlwB2eZ5DKOnjQZCYdOS8GPY= -google.golang.org/genproto v0.0.0-20230803162519-f966b187b2e5 h1:L6iMMGrtzgHsWofoFcihmDEMYeDR9KN/ThbPWGrh++g= -google.golang.org/genproto/googleapis/api v0.0.0-20240102182953-50ed04b92917 h1:rcS6EyEaoCO52hQDupoSfrxI3R6C2Tq741is7X8OvnM= -google.golang.org/genproto/googleapis/api v0.0.0-20240102182953-50ed04b92917/go.mod h1:CmlNWB9lSezaYELKS5Ym1r44VrrbPUa7JTvw+6MbpJ0= -google.golang.org/genproto/googleapis/rpc v0.0.0-20240102182953-50ed04b92917 h1:6G8oQ016D88m1xAKljMlBOOGWDZkes4kMhgGFlf8WcQ= -google.golang.org/genproto/googleapis/rpc v0.0.0-20240102182953-50ed04b92917/go.mod h1:xtjpI3tXFPP051KaWnhvxkiubL/6dJ18vLVf7q2pTOU= -google.golang.org/grpc v1.61.0 h1:TOvOcuXn30kRao+gfcvsebNEa5iZIiLkisYEkf7R7o0= -google.golang.org/grpc v1.61.0/go.mod h1:VUbo7IFqmF1QtCAstipjG0GIoq49KvMe9+h1jFLBNJs= +gomodules.xyz/jsonpatch/v2 v2.4.0 h1:Ci3iUJyx9UeRx7CeFN8ARgGbkESwJK+KB9lLcWxY/Zw= +gomodules.xyz/jsonpatch/v2 v2.4.0/go.mod h1:AH3dM2RI6uoBZxn3LVrfvJ3E0/9dG4cSrbuBJT4moAY= +google.golang.org/genproto v0.0.0-20240213162025-012b6fc9bca9 h1:9+tzLLstTlPTRyJTh+ah5wIMsBW5c4tQwGTN3thOW9Y= +google.golang.org/genproto/googleapis/api v0.0.0-20240318140521-94a12d6c2237 h1:RFiFrvy37/mpSpdySBDrUdipW/dHwsRwh3J3+A9VgT4= +google.golang.org/genproto/googleapis/api v0.0.0-20240318140521-94a12d6c2237/go.mod h1:Z5Iiy3jtmioajWHDGFk7CeugTyHtPvMHA4UTmUkyalE= +google.golang.org/genproto/googleapis/rpc v0.0.0-20240314234333-6e1732d8331c h1:lfpJ/2rWPa/kJgxyyXM8PrNnfCzcmxJ265mADgwmvLI= +google.golang.org/genproto/googleapis/rpc v0.0.0-20240314234333-6e1732d8331c/go.mod h1:WtryC6hu0hhx87FDGxWCDptyssuo68sk10vYjF+T9fY= +google.golang.org/grpc v1.62.1 h1:B4n+nfKzOICUXMgyrNd19h/I9oH0L1pizfk1d4zSgTk= +google.golang.org/grpc v1.62.1/go.mod h1:IWTG0VlJLCh1SkC58F7np9ka9mx/WNkjl4PGJaiq+QE= google.golang.org/protobuf v1.34.1 h1:9ddQBjfCyZPOHPUiPxpYESBLc+T8P3E+Vo4IbKZgFWg= google.golang.org/protobuf v1.34.1/go.mod h1:c6P6GXX6sHbq/GpV6MGZEdwhWPcYBgnhAHhKbcUYpos= gopkg.in/alecthomas/kingpin.v2 v2.2.6/go.mod h1:FMv+mEhP44yOT+4EoQTLFTRgOQ1FBLkstjWtayDeSgw= @@ -502,8 +504,8 @@ k8s.io/klog/v2 v2.120.1 h1:QXU6cPEOIslTGvZaXvFWiP9VKyeet3sawzTOvdXb4Vw= k8s.io/klog/v2 v2.120.1/go.mod h1:3Jpz1GvMt720eyJH1ckRHK1EDfpxISzJ7I9OYgaDtPE= k8s.io/kube-openapi v0.0.0-20240228011516-70dd3763d340 h1:BZqlfIlq5YbRMFko6/PM7FjZpUb45WallggurYhKGag= k8s.io/kube-openapi v0.0.0-20240228011516-70dd3763d340/go.mod h1:yD4MZYeKMBwQKVht279WycxKyM84kkAx2DPrTXaeb98= -k8s.io/utils v0.0.0-20240102154912-e7106e64919e h1:eQ/4ljkx21sObifjzXwlPKpdGLrCfRziVtos3ofG/sQ= -k8s.io/utils v0.0.0-20240102154912-e7106e64919e/go.mod h1:OLgZIPagt7ERELqWJFomSt595RzquPNLL48iOWgYOg0= +k8s.io/utils v0.0.0-20240502163921-fe8a2dddb1d0 h1:jgGTlFYnhF1PM1Ax/lAlxUPE+KfCIXHaathvJg1C3ak= +k8s.io/utils v0.0.0-20240502163921-fe8a2dddb1d0/go.mod h1:OLgZIPagt7ERELqWJFomSt595RzquPNLL48iOWgYOg0= sigs.k8s.io/controller-runtime v0.14.6 h1:oxstGVvXGNnMvY7TAESYk+lzr6S3V5VFxQ6d92KcwQA= sigs.k8s.io/controller-runtime v0.14.6/go.mod h1:WqIdsAY6JBsjfc/CqO0CORmNtoCtE4S6qbPc9s68h+0= sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd h1:EDPBXCAspyGV4jQlpZSudPeMmr1bNJefnuqLsRAsHZo= diff --git a/internal/broker/broker.go b/internal/broker/broker.go index c636675663..0422b22c02 100644 --- a/internal/broker/broker.go +++ b/internal/broker/broker.go @@ -10,6 +10,7 @@ import ( "github.com/kyma-project/control-plane/components/provisioner/pkg/gqlschema" "github.com/kyma-project/kyma-environment-broker/internal" + kim "github.com/kyma-project/kyma-environment-broker/internal/kim" ) const ( @@ -49,6 +50,7 @@ type Config struct { EnableShootAndSeedSameRegion bool `envconfig:"default=false"` Binding BindingConfig + KimConfig kim.Config UseSmallerMachineTypes bool `envconfig:"default=false"` } diff --git a/internal/kim/config.go b/internal/kim/config.go new file mode 100644 index 0000000000..974744d49a --- /dev/null +++ b/internal/kim/config.go @@ -0,0 +1,21 @@ +package kim + +type Config struct { + Enabled bool `envconfig:"default=false"` + DryRun bool `envconfig:"default=true"` + ViewOnly bool `envconfig:"default=true"` + Plans []string `envconfig:"default=preview"` + KimOnlyPlans []string `envconfig:"default="` +} + +func (c *Config) IsEnabledForPlan(planName string) bool { + if c.Enabled == false { + return false + } + for _, plan := range c.Plans { + if plan == planName { + return true + } + } + return false +} diff --git a/internal/kim/config_test.go b/internal/kim/config_test.go new file mode 100644 index 0000000000..8a469aea36 --- /dev/null +++ b/internal/kim/config_test.go @@ -0,0 +1,28 @@ +package kim + +import ( + "github.com/stretchr/testify/assert" + "testing" +) + +func TestIsEnabled_KimDisabled(t *testing.T) { + config := &Config{ + Enabled: false, + Plans: []string{"gcp", "preview"}, + ViewOnly: false, + } + + assert.False(t, config.IsEnabledForPlan("gcp")) + assert.False(t, config.IsEnabledForPlan("preview")) +} + +func TestIsEnabled_KimEnabledForPreview(t *testing.T) { + config := &Config{ + Enabled: true, + Plans: []string{"preview"}, + ViewOnly: false, + } + + assert.False(t, config.IsEnabledForPlan("gcp")) + assert.True(t, config.IsEnabledForPlan("preview")) +} diff --git a/internal/process/provisioning/create_runtime_resource_step.go b/internal/process/provisioning/create_runtime_resource_step.go new file mode 100644 index 0000000000..42a42dde8a --- /dev/null +++ b/internal/process/provisioning/create_runtime_resource_step.go @@ -0,0 +1,102 @@ +package provisioning + +import ( + "fmt" + imv1 "github.com/kyma-project/infrastructure-manager/api/v1" + "github.com/kyma-project/kyma-environment-broker/internal/broker" + "github.com/kyma-project/kyma-environment-broker/internal/kim" + "gopkg.in/yaml.v3" + "time" + + "github.com/kyma-project/kyma-environment-broker/internal" + "github.com/kyma-project/kyma-environment-broker/internal/process" + "github.com/kyma-project/kyma-environment-broker/internal/storage" + "github.com/sirupsen/logrus" +) + +type CreateRuntimeResourceStep struct { + operationManager *process.OperationManager + instanceStorage storage.Instances + runtimeStateStorage storage.RuntimeStates + kimConfig kim.Config +} + +func NewCreateRuntimeResourceStep(os storage.Operations, runtimeStorage storage.RuntimeStates, is storage.Instances, kimConfig kim.Config) *CreateRuntimeResourceStep { + return &CreateRuntimeResourceStep{ + operationManager: process.NewOperationManager(os), + instanceStorage: is, + runtimeStateStorage: runtimeStorage, + kimConfig: kimConfig, + } +} + +func (s *CreateRuntimeResourceStep) Name() string { + return "Create_Runtime_Resource" +} + +func (s *CreateRuntimeResourceStep) Run(operation internal.Operation, log logrus.FieldLogger) (internal.Operation, time.Duration, error) { + if time.Since(operation.UpdatedAt) > CreateRuntimeTimeout { + log.Infof("operation has reached the time limit: updated operation time: %s", operation.UpdatedAt) + return s.operationManager.OperationFailed(operation, fmt.Sprintf("operation has reached the time limit: %s", CreateRuntimeTimeout), nil, log) + } + + if !s.kimConfig.IsEnabledForPlan(broker.PlanNamesMapping[operation.ProvisioningParameters.PlanID]) { + log.Infof("KIM is not enabled for plan %s, skipping", broker.PlanNamesMapping[operation.ProvisioningParameters.PlanID]) + } + + runtimeCR, err := s.createRuntimeResourceObject(operation) + if err != nil { + return s.operationManager.OperationFailed(operation, fmt.Sprintf("while creating Runtime CR object: %s", err), err, log) + } + + if s.kimConfig.DryRun { + yaml, err := EncodeRuntimeCR(runtimeCR) + if err != nil { + log.Infof("Runtime CR yaml:%s", yaml) + } else { + log.Errorf("failed to encode Runtime CR to yaml: %s", err) + } + } else { + err := s.CreateResource(runtimeCR) + if err != nil { + return s.operationManager.OperationFailed(operation, fmt.Sprintf("while creating Runtime CR object: %s", err), err, log) + + } + } + log.Info("Runtime CR creation process finished successfully") + return operation, 0, nil +} + +func (s *CreateRuntimeResourceStep) CreateResource(cr imv1.Runtime) error { + return nil +} + +//TODO remember - labels and annotations + +func (s *CreateRuntimeResourceStep) createRuntimeResourceObject(operation internal.Operation) (imv1.Runtime, error) { + runtime := imv1.Runtime{} + + //operation.InputCreator.SetProvisioningParameters(operation.ProvisioningParameters) + //operation.InputCreator.SetShootName(operation.ShootName) + //operation.InputCreator.SetShootDomain(operation.ShootDomain) + //operation.InputCreator.SetShootDNSProviders(operation.ShootDNSProviders) + //operation.InputCreator.SetLabel(brokerKeyPrefix+"instance_id", operation.InstanceID) + //operation.InputCreator.SetLabel(globalKeyPrefix+"subaccount_id", operation.ProvisioningParameters.ErsContext.SubAccountID) + //operation.InputCreator.SetLabel(grafanaURLLabel, fmt.Sprintf("https://grafana.%s", operation.ShootDomain)) + //request, err, := operation.InputCreator.CreateProvisionClusterInput() + //if err != nil { + // return imv1.Runtime{}, fmt.Errorf("while building input for provisioner: %w", err) + //} + //request.ClusterConfig.GardenerConfig.ShootNetworkingFilterDisabled = operation.ProvisioningParameters.ErsContext.DisableEnterprisePolicyFilter() + //request.ClusterConfig.GardenerConfig.EuAccess = &operation.InstanceDetails.EuAccess + + return runtime, nil +} + +func EncodeRuntimeCR(runtime imv1.Runtime) (string, error) { + result, err := yaml.Marshal(runtime) + if err != nil { + return "", err + } + return string(result), nil +} diff --git a/internal/process/provisioning/create_runtime_resource_step_test.go b/internal/process/provisioning/create_runtime_resource_step_test.go new file mode 100644 index 0000000000..466f01a660 --- /dev/null +++ b/internal/process/provisioning/create_runtime_resource_step_test.go @@ -0,0 +1,46 @@ +package provisioning + +import ( + "github.com/kyma-project/kyma-environment-broker/internal/fixture" + "github.com/kyma-project/kyma-environment-broker/internal/kim" + "testing" + + "github.com/kyma-project/kyma-environment-broker/internal/storage" + "github.com/sirupsen/logrus" + "github.com/stretchr/testify/assert" +) + +func TestCreateRuntimeResourceStep_HappyPath_YamlOnly(t *testing.T) { + // given + log := logrus.New() + memoryStorage := storage.NewMemoryStorage() + + preOperation := fixture.FixProvisioningOperation(operationID, instanceID) + err := memoryStorage.Operations().InsertOperation(preOperation) + assert.NoError(t, err) + + err = memoryStorage.Instances().Insert(fixInstance()) + assert.NoError(t, err) + + kimConfig := kim.Config{ + Enabled: true, + Plans: []string{"preview"}, + ViewOnly: false, + DryRun: true, + } + + step := NewCreateRuntimeResourceStep(memoryStorage.Operations(), memoryStorage.RuntimeStates(), memoryStorage.Instances(), kimConfig) + + // when + entry := log.WithFields(logrus.Fields{"step": "TEST"}) + _, repeat, err := step.Run(preOperation, entry) + + // then + assert.NoError(t, err) + assert.Zero(t, repeat) + + _, err = memoryStorage.Instances().GetByID(preOperation.InstanceID) + assert.NoError(t, err) + + //assert.YAMLEq(t, expectedKymaTemplate, op.KymaTemplate) +} diff --git a/internal/process/provisioning/generate_runtime_id_step.go b/internal/process/provisioning/generate_runtime_id_step.go new file mode 100644 index 0000000000..0e98dc81d4 --- /dev/null +++ b/internal/process/provisioning/generate_runtime_id_step.go @@ -0,0 +1,80 @@ +package provisioning + +import ( + "fmt" + "github.com/google/uuid" + + "time" + + "github.com/kyma-project/kyma-environment-broker/internal" + "github.com/kyma-project/kyma-environment-broker/internal/process" + "github.com/kyma-project/kyma-environment-broker/internal/storage" + "github.com/kyma-project/kyma-environment-broker/internal/storage/dberr" + "github.com/sirupsen/logrus" +) + +type GenerateRuntimeIDStep struct { + operationManager *process.OperationManager + instanceStorage storage.Instances + runtimeStateStorage storage.RuntimeStates +} + +func NewGenerateRuntimeIDStep(os storage.Operations, runtimeStorage storage.RuntimeStates, is storage.Instances) *GenerateRuntimeIDStep { + return &GenerateRuntimeIDStep{ + operationManager: process.NewOperationManager(os), + instanceStorage: is, + runtimeStateStorage: runtimeStorage, + } +} + +func (s *GenerateRuntimeIDStep) Name() string { + return "Generate_Runtime_ID" +} + +func (s *GenerateRuntimeIDStep) Run(operation internal.Operation, log logrus.FieldLogger) (internal.Operation, time.Duration, error) { + if operation.RuntimeID != "" { + log.Infof("RuntimeID already set %s, skipping", operation.RuntimeID) + return operation, 0, nil + } + + runtimeID := uuid.New().String() + + log.Infof("RuntimeID %s generated", runtimeID) + + repeatAfter := time.Duration(0) + operation, repeatAfter, _ = s.operationManager.UpdateOperation(operation, func(operation *internal.Operation) { + operation.RuntimeID = runtimeID + }, log) + if repeatAfter != 0 { + log.Errorf("cannot save RuntimeID in operation") + return operation, 5 * time.Second, nil + } + + err := s.updateInstance(operation.InstanceID, runtimeID) + + switch { + case err == nil: + case dberr.IsConflict(err): + err := s.updateInstance(operation.InstanceID, runtimeID) + if err != nil { + log.Errorf("cannot update instance: %s", err) + return operation, 1 * time.Minute, nil + } + } + + return operation, 0, nil +} + +func (s *GenerateRuntimeIDStep) updateInstance(id, runtimeID string) error { + instance, err := s.instanceStorage.GetByID(id) + if err != nil { + return fmt.Errorf("while getting instance: %w", err) + } + instance.RuntimeID = runtimeID + _, err = s.instanceStorage.Update(*instance) + if err != nil { + return fmt.Errorf("while updating instance: %w", err) + } + + return nil +} diff --git a/internal/process/provisioning/generate_runtime_id_step_test.go b/internal/process/provisioning/generate_runtime_id_step_test.go new file mode 100644 index 0000000000..3450b965e1 --- /dev/null +++ b/internal/process/provisioning/generate_runtime_id_step_test.go @@ -0,0 +1,73 @@ +package provisioning + +import ( + "github.com/kyma-project/kyma-environment-broker/internal/fixture" + "testing" + + "github.com/sirupsen/logrus" + "github.com/stretchr/testify/assert" + + "github.com/kyma-project/kyma-environment-broker/internal/storage" +) + +func TestNewGenerateRuntimeIDStep_LeaveRuntimeIDIfNotEmpty(t *testing.T) { + // given + log := logrus.New() + memoryStorage := storage.NewMemoryStorage() + + instance := fixInstance() + err := memoryStorage.Instances().Insert(instance) + assert.NoError(t, err) + + operation := fixture.FixProvisioningOperation(operationID, instanceID) + operation.RuntimeID = instance.RuntimeID + + err = memoryStorage.Operations().InsertOperation(operation) + assert.NoError(t, err) + + step := NewGenerateRuntimeIDStep(memoryStorage.Operations(), memoryStorage.RuntimeStates(), memoryStorage.Instances()) + + // when + entry := log.WithFields(logrus.Fields{"step": "TEST"}) + operation, repeat, err := step.Run(operation, entry) + + // then + assert.NoError(t, err) + assert.Zero(t, repeat) + + instanceAfter, err := memoryStorage.Instances().GetByID(operation.InstanceID) + assert.NoError(t, err) + assert.Equal(t, instance.RuntimeID, instanceAfter.RuntimeID) + assert.Equal(t, operation.RuntimeID, instanceAfter.RuntimeID) +} + +func TestNewGenerateRuntimeIDStep_LeaveCreateRuntimeIDIfEmpty(t *testing.T) { + // given + log := logrus.New() + memoryStorage := storage.NewMemoryStorage() + + operation := fixture.FixProvisioningOperation(operationID, instanceID) + operation.RuntimeID = "" + err := memoryStorage.Operations().InsertOperation(operation) + assert.NoError(t, err) + + instance := fixInstance() + instance.RuntimeID = "" + err = memoryStorage.Instances().Insert(instance) + assert.NoError(t, err) + + step := NewGenerateRuntimeIDStep(memoryStorage.Operations(), memoryStorage.RuntimeStates(), memoryStorage.Instances()) + + // when + entry := log.WithFields(logrus.Fields{"step": "TEST"}) + operation, repeat, err := step.Run(operation, entry) + + // then + assert.NoError(t, err) + assert.Zero(t, repeat) + + instanceAfter, err := memoryStorage.Instances().GetByID(operation.InstanceID) + assert.NoError(t, err) + assert.Equal(t, 36, len(instanceAfter.RuntimeID)) + assert.Equal(t, operation.RuntimeID, instanceAfter.RuntimeID) +} diff --git a/resources/keb/templates/deployment.yaml b/resources/keb/templates/deployment.yaml index 04a6e5b7bf..034ac5e9fa 100644 --- a/resources/keb/templates/deployment.yaml +++ b/resources/keb/templates/deployment.yaml @@ -390,6 +390,16 @@ spec: value: "{{ .Values.kymaResourceDeletionTimeout }}" - name: APP_BROKER_USE_SMALLER_MACHINE_TYPES value: "{{ .Values.useSmallerMachineTypes }}" + - name: APP_KIM_ENABLED + value: "{{ .Values.kim.enabled }}" + - name: APP_KIM_DRY_RUN + value: "{{ .Values.kim.dryRun }}" + - name: APP_KIM_VIEW_ONLY + value: "{{ .Values.kim.viewOnly }}" + - name: APP_KIM_PLANS + value: "{{ .Values.kim.plans }}" + - name: APP_KIM_KIM_ONLY_PLANS + value: "{{ .Values.kim.kimOnlyPlans }}" ports: - name: http containerPort: {{ .Values.broker.port }} diff --git a/resources/keb/values.yaml b/resources/keb/values.yaml index cb5fca2aeb..e1eb814570 100644 --- a/resources/keb/values.yaml +++ b/resources/keb/values.yaml @@ -100,10 +100,10 @@ deployment: pullPolicy: Always securityContext: runAsUser: 2000 - strategy: {} # Read more: https://kubernetes.io/docs/concepts/workloads/controllers/deployment/#strategy - nodeSelector: {} + strategy: { } # Read more: https://kubernetes.io/docs/concepts/workloads/controllers/deployment/#strategy + nodeSelector: { } -imagePullSecrets: [] +imagePullSecrets: [ ] namePrefix: "kcp" nameOverride: "kyma-environment-broker" fullnameOverride: "kcp-kyma-environment-broker" @@ -134,7 +134,7 @@ swagger: enabled: true resources: - {} + { } # We usually recommend not to specify default resources and to leave this as a conscious # choice for the user. This also increases chances charts run on environments with little # resources, such as Minikube. If you do want to specify resources, uncomment the following @@ -144,10 +144,10 @@ resources: # memory: 128Mi # requests: # cpu: 100m - # memory: 128Mi +# memory: 128Mi serviceAccount: - annotations: {} + annotations: { } infrastructureManager: disabled: "true" @@ -295,7 +295,7 @@ avs: trialParentId: "0" maintenanceModeDuringUpgrade: disabled: false # disable AvS maintenance mode during upgrade, false until SKR ZDM is done - alwaysDisabledGlobalAccountIDs: [] # list of GA IDs where maintenance mode is always disabled during upgrade + alwaysDisabledGlobalAccountIDs: [ ] # list of GA IDs where maintenance mode is always disabled during upgrade edp: authURL: "TBD" @@ -339,6 +339,13 @@ notification: url: "TBD" disabled: true +kim: + enabled: false + dryRun: true + viewOnly: true + plans: "preview" # comma separated list of plans where KIM will be used + kimOnlyPlans: "" # comma separated list of plans where KIM will be used exclusively + oidc: issuer: https://kymatest.accounts400.ondemand.com keysURL: https://kymatest.accounts400.ondemand.com/oauth2/certs From 561549be50a12b1ee51ab2e5a403422a08602e9f Mon Sep 17 00:00:00 2001 From: Pieszka Date: Wed, 10 Jul 2024 16:46:22 +0200 Subject: [PATCH 2/4] shootname added --- .../process/provisioning/create_runtime_resource_step.go | 6 ++++-- .../provisioning/create_runtime_resource_step_test.go | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/internal/process/provisioning/create_runtime_resource_step.go b/internal/process/provisioning/create_runtime_resource_step.go index 42a42dde8a..b0ff75e44d 100644 --- a/internal/process/provisioning/create_runtime_resource_step.go +++ b/internal/process/provisioning/create_runtime_resource_step.go @@ -42,6 +42,7 @@ func (s *CreateRuntimeResourceStep) Run(operation internal.Operation, log logrus if !s.kimConfig.IsEnabledForPlan(broker.PlanNamesMapping[operation.ProvisioningParameters.PlanID]) { log.Infof("KIM is not enabled for plan %s, skipping", broker.PlanNamesMapping[operation.ProvisioningParameters.PlanID]) + return operation, 0, nil } runtimeCR, err := s.createRuntimeResourceObject(operation) @@ -59,7 +60,7 @@ func (s *CreateRuntimeResourceStep) Run(operation internal.Operation, log logrus } else { err := s.CreateResource(runtimeCR) if err != nil { - return s.operationManager.OperationFailed(operation, fmt.Sprintf("while creating Runtime CR object: %s", err), err, log) + return s.operationManager.OperationFailed(operation, fmt.Sprintf("while creating Runtime CR resource: %s", err), err, log) } } @@ -75,6 +76,7 @@ func (s *CreateRuntimeResourceStep) CreateResource(cr imv1.Runtime) error { func (s *CreateRuntimeResourceStep) createRuntimeResourceObject(operation internal.Operation) (imv1.Runtime, error) { runtime := imv1.Runtime{} + runtime.Spec.Shoot.Name = operation.ShootName //operation.InputCreator.SetProvisioningParameters(operation.ProvisioningParameters) //operation.InputCreator.SetShootName(operation.ShootName) @@ -94,7 +96,7 @@ func (s *CreateRuntimeResourceStep) createRuntimeResourceObject(operation intern } func EncodeRuntimeCR(runtime imv1.Runtime) (string, error) { - result, err := yaml.Marshal(runtime) + result, err := yaml.Marshal(&runtime) if err != nil { return "", err } diff --git a/internal/process/provisioning/create_runtime_resource_step_test.go b/internal/process/provisioning/create_runtime_resource_step_test.go index 466f01a660..852ca32733 100644 --- a/internal/process/provisioning/create_runtime_resource_step_test.go +++ b/internal/process/provisioning/create_runtime_resource_step_test.go @@ -24,7 +24,7 @@ func TestCreateRuntimeResourceStep_HappyPath_YamlOnly(t *testing.T) { kimConfig := kim.Config{ Enabled: true, - Plans: []string{"preview"}, + Plans: []string{"azure"}, ViewOnly: false, DryRun: true, } From 2bcb0d1c08068b73cdd61e6dd9fa64590609139b Mon Sep 17 00:00:00 2001 From: Pieszka Date: Thu, 11 Jul 2024 08:42:37 +0200 Subject: [PATCH 3/4] stage 0 implementation --- .../create_runtime_resource_step.go | 38 ++++++------------- 1 file changed, 11 insertions(+), 27 deletions(-) diff --git a/internal/process/provisioning/create_runtime_resource_step.go b/internal/process/provisioning/create_runtime_resource_step.go index b0ff75e44d..42418f0fcd 100644 --- a/internal/process/provisioning/create_runtime_resource_step.go +++ b/internal/process/provisioning/create_runtime_resource_step.go @@ -51,52 +51,36 @@ func (s *CreateRuntimeResourceStep) Run(operation internal.Operation, log logrus } if s.kimConfig.DryRun { - yaml, err := EncodeRuntimeCR(runtimeCR) + yaml, err := RuntimeToYaml(runtimeCR) if err != nil { - log.Infof("Runtime CR yaml:%s", yaml) - } else { log.Errorf("failed to encode Runtime CR to yaml: %s", err) + } else { + log.Infof("Runtime CR yaml:%s", yaml) } } else { err := s.CreateResource(runtimeCR) if err != nil { return s.operationManager.OperationFailed(operation, fmt.Sprintf("while creating Runtime CR resource: %s", err), err, log) - } + log.Info("Runtime CR creation process finished successfully") } - log.Info("Runtime CR creation process finished successfully") return operation, 0, nil } -func (s *CreateRuntimeResourceStep) CreateResource(cr imv1.Runtime) error { +func (s *CreateRuntimeResourceStep) CreateResource(cr *imv1.Runtime) error { + logrus.Info("Creating Runtime CR - TO BE IMPLEMENTED") return nil } -//TODO remember - labels and annotations - -func (s *CreateRuntimeResourceStep) createRuntimeResourceObject(operation internal.Operation) (imv1.Runtime, error) { +func (s *CreateRuntimeResourceStep) createRuntimeResourceObject(operation internal.Operation) (*imv1.Runtime, error) { runtime := imv1.Runtime{} - runtime.Spec.Shoot.Name = operation.ShootName - - //operation.InputCreator.SetProvisioningParameters(operation.ProvisioningParameters) - //operation.InputCreator.SetShootName(operation.ShootName) - //operation.InputCreator.SetShootDomain(operation.ShootDomain) - //operation.InputCreator.SetShootDNSProviders(operation.ShootDNSProviders) - //operation.InputCreator.SetLabel(brokerKeyPrefix+"instance_id", operation.InstanceID) - //operation.InputCreator.SetLabel(globalKeyPrefix+"subaccount_id", operation.ProvisioningParameters.ErsContext.SubAccountID) - //operation.InputCreator.SetLabel(grafanaURLLabel, fmt.Sprintf("https://grafana.%s", operation.ShootDomain)) - //request, err, := operation.InputCreator.CreateProvisionClusterInput() - //if err != nil { - // return imv1.Runtime{}, fmt.Errorf("while building input for provisioner: %w", err) - //} - //request.ClusterConfig.GardenerConfig.ShootNetworkingFilterDisabled = operation.ProvisioningParameters.ErsContext.DisableEnterprisePolicyFilter() - //request.ClusterConfig.GardenerConfig.EuAccess = &operation.InstanceDetails.EuAccess + runtime.Spec.Shoot.Name = "shoot-name" - return runtime, nil + return &runtime, nil } -func EncodeRuntimeCR(runtime imv1.Runtime) (string, error) { - result, err := yaml.Marshal(&runtime) +func RuntimeToYaml(runtime *imv1.Runtime) (string, error) { + result, err := yaml.Marshal(runtime) if err != nil { return "", err } From a022b6c8992bce45f64e46d1514b0f1b26017178 Mon Sep 17 00:00:00 2001 From: Pieszka Date: Thu, 11 Jul 2024 08:52:05 +0200 Subject: [PATCH 4/4] make fix --- cmd/broker/provisioning.go | 1 + internal/kim/config_test.go | 3 ++- internal/process/provisioning/create_runtime_resource_step.go | 3 ++- .../process/provisioning/create_runtime_resource_step_test.go | 3 ++- internal/process/provisioning/generate_runtime_id_step.go | 1 + internal/process/provisioning/generate_runtime_id_step_test.go | 3 ++- 6 files changed, 10 insertions(+), 4 deletions(-) diff --git a/cmd/broker/provisioning.go b/cmd/broker/provisioning.go index 34ee1f1f21..d98e215bbe 100644 --- a/cmd/broker/provisioning.go +++ b/cmd/broker/provisioning.go @@ -2,6 +2,7 @@ package main import ( "context" + "github.com/kyma-project/kyma-environment-broker/common/hyperscaler" "github.com/kyma-project/kyma-environment-broker/internal/avs" "github.com/kyma-project/kyma-environment-broker/internal/process" diff --git a/internal/kim/config_test.go b/internal/kim/config_test.go index 8a469aea36..e8b467f698 100644 --- a/internal/kim/config_test.go +++ b/internal/kim/config_test.go @@ -1,8 +1,9 @@ package kim import ( - "github.com/stretchr/testify/assert" "testing" + + "github.com/stretchr/testify/assert" ) func TestIsEnabled_KimDisabled(t *testing.T) { diff --git a/internal/process/provisioning/create_runtime_resource_step.go b/internal/process/provisioning/create_runtime_resource_step.go index 42418f0fcd..76d2dea3a3 100644 --- a/internal/process/provisioning/create_runtime_resource_step.go +++ b/internal/process/provisioning/create_runtime_resource_step.go @@ -2,11 +2,12 @@ package provisioning import ( "fmt" + "time" + imv1 "github.com/kyma-project/infrastructure-manager/api/v1" "github.com/kyma-project/kyma-environment-broker/internal/broker" "github.com/kyma-project/kyma-environment-broker/internal/kim" "gopkg.in/yaml.v3" - "time" "github.com/kyma-project/kyma-environment-broker/internal" "github.com/kyma-project/kyma-environment-broker/internal/process" diff --git a/internal/process/provisioning/create_runtime_resource_step_test.go b/internal/process/provisioning/create_runtime_resource_step_test.go index 852ca32733..5bf293fa34 100644 --- a/internal/process/provisioning/create_runtime_resource_step_test.go +++ b/internal/process/provisioning/create_runtime_resource_step_test.go @@ -1,9 +1,10 @@ package provisioning import ( + "testing" + "github.com/kyma-project/kyma-environment-broker/internal/fixture" "github.com/kyma-project/kyma-environment-broker/internal/kim" - "testing" "github.com/kyma-project/kyma-environment-broker/internal/storage" "github.com/sirupsen/logrus" diff --git a/internal/process/provisioning/generate_runtime_id_step.go b/internal/process/provisioning/generate_runtime_id_step.go index 0e98dc81d4..e126fd62a8 100644 --- a/internal/process/provisioning/generate_runtime_id_step.go +++ b/internal/process/provisioning/generate_runtime_id_step.go @@ -2,6 +2,7 @@ package provisioning import ( "fmt" + "github.com/google/uuid" "time" diff --git a/internal/process/provisioning/generate_runtime_id_step_test.go b/internal/process/provisioning/generate_runtime_id_step_test.go index 3450b965e1..39a6c79923 100644 --- a/internal/process/provisioning/generate_runtime_id_step_test.go +++ b/internal/process/provisioning/generate_runtime_id_step_test.go @@ -1,9 +1,10 @@ package provisioning import ( - "github.com/kyma-project/kyma-environment-broker/internal/fixture" "testing" + "github.com/kyma-project/kyma-environment-broker/internal/fixture" + "github.com/sirupsen/logrus" "github.com/stretchr/testify/assert"