Skip to content

Commit

Permalink
Deduplicate related images
Browse files Browse the repository at this point in the history
Signed-off-by: Fabricio Aguiar <[email protected]>
  • Loading branch information
fao89 committed Nov 24, 2023
1 parent 585abc0 commit ce564bc
Showing 1 changed file with 23 additions and 2 deletions.
25 changes: 23 additions & 2 deletions cmd/csv-merger/csv-merger.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ import (
"github.com/ghodss/yaml"
"github.com/imdario/mergo"
"github.com/openstack-k8s-operators/openstack-operator/pkg/util"
"golang.org/x/exp/maps"

semver "github.com/blang/semver/v4"
"github.com/operator-framework/api/pkg/lib/version"
Expand Down Expand Up @@ -163,6 +164,18 @@ func main() {
envVarList = append(envVarList, getEnvsFromFile(filename)...)
}
}
uniqueEnv := make(map[string]v1.EnvVar)
for _, container := range csvBase.Spec.InstallStrategy.StrategySpec.DeploymentSpecs[0].Spec.Template.Spec.Containers {
if container.Name == "manager" {
for _, env := range container.Env {
uniqueEnv[env.Name] = env
}
}
}
uniqueImages := make(map[string]csvv1alpha1.RelatedImage)
for _, img := range csvBase.Spec.RelatedImages {
uniqueImages[img.Name] = img
}
for _, csvFile := range csvs {
if csvFile != "" {
csvBytes, err := os.ReadFile(csvFile)
Expand All @@ -182,14 +195,22 @@ func main() {
for _, container := range csvStruct.Spec.InstallStrategy.StrategySpec.DeploymentSpecs[0].Spec.Template.Spec.Containers {
// Copy env vars from the Service Operator into the OpenStack Operator
if container.Name == "manager" {
envVarList = append(envVarList, container.Env...)
for _, env := range container.Env {
uniqueEnv[env.Name] = env
}
}
}
envVarList = maps.Values(uniqueEnv)

installStrategyBase.DeploymentSpecs = append(installStrategyBase.DeploymentSpecs, csvStruct.Spec.InstallStrategy.StrategySpec.DeploymentSpecs...)
installStrategyBase.ClusterPermissions = append(installStrategyBase.ClusterPermissions, csvStruct.Spec.InstallStrategy.StrategySpec.ClusterPermissions...)
installStrategyBase.Permissions = append(installStrategyBase.Permissions, csvStruct.Spec.InstallStrategy.StrategySpec.Permissions...)
csvNew.Spec.RelatedImages = append(csvNew.Spec.RelatedImages, csvStruct.Spec.RelatedImages...)
for _, img := range csvStruct.Spec.RelatedImages {
if img.Name != "manager" {
uniqueImages[img.Name] = img
}
}
csvNew.Spec.RelatedImages = maps.Values(uniqueImages)

for _, owned := range csvStruct.Spec.CustomResourceDefinitions.Owned {
csvNew.Spec.CustomResourceDefinitions.Owned = append(
Expand Down

0 comments on commit ce564bc

Please sign in to comment.