.github/workflows/go.yml

@@ -29,7 +29,7 @@ jobs:
       run: make bin
 
     - name: Upload a Build Artifact
-      uses: actions/upload-artifact@v3
+      uses: actions/upload-artifact@v4
       with:
         name: "kompose"
         path: "kompose"

.github/workflows/test.yml

@@ -2,7 +2,7 @@ name: Kompose CI
 on:
   push:
     branches:
-      - main
+      - main 
   pull_request:
 env:
   # Avoid noisy outputs like "tput: No value for $TERM and no -T specified"
@@ -27,23 +27,23 @@ jobs:
       - name: Perform cross compile
         if: ${{ matrix.cross_compile }}
         run: make cross
-  # docs:
+  docs:
-  #   name: Build docs and Coveralls integration
+    name: Build docs and Coveralls integration
-  #   runs-on: ubuntu-latest
+    runs-on: ubuntu-latest
-  #   needs: test
+    needs: test
-  #   steps:
+    steps:
-  #     - uses: actions/checkout@v4
+      - uses: actions/checkout@v4
-  #     - uses: actions/setup-go@v5
+      - uses: actions/setup-go@v5
-  #       with:
+        with:
-  #         go-version: ^1.21
+          go-version: ^1.21
-  #     - name: Install dyff
+      - name: Install dyff
-  #       run: go install github.com/homeport/dyff/cmd/dyff@v1.5.8
+        run: go install github.com/homeport/dyff/cmd/dyff@v1.5.8
-  #     - name: Create .coverprofile for each targeted directory by re:running tests
+      - name: Create .coverprofile for each targeted directory by re:running tests
-  #       run: make test
+        run: make test
-  #     - name: Collect all .coverprofile files and save it to one file gover.coverprofile
+      - name: Collect all .coverprofile files and save it to one file gover.coverprofile
-  #       run: gover
+        run: gover
-  #     - name: Send coverage
+      - name: Send coverage
-  #       run: goveralls -coverprofile=gover.coverprofile -service=github
+        run: goveralls -coverprofile=gover.coverprofile -service=github
-  #       env:
+        env:
-  #         # As per https://github.com/mattn/goveralls#github-actions
+          # As per https://github.com/mattn/goveralls#github-actions
-  #         COVERALLS_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          COVERALLS_TOKEN: ${{ secrets.GITHUB_TOKEN }}

pkg/app/app.go

@@ -261,7 +261,7 @@ func Convert(opt kobject.ConvertOptions) ([]runtime.Object, error) {
 	}
 
 	// Print output
-	err = kubernetes.PrintList(objects, opt, komposeObject)
+	err = kubernetes.PrintList(objects, opt)
 	if err != nil {
 		log.Fatalf(err.Error())
 	}

pkg/kobject/kobject.go

@@ -233,8 +233,6 @@ type Volumes struct {
 	PVCName       string // name of PVC
 	PVCSize       string // PVC size
 	SelectorValue string // Value of the label selector
-	VolumeType    string // Type of volume (e.g., "secret")
-	SecretName    string // Name of Kubernetes Secret (if VolumeType is "secret")
 }
 
 // Placement holds the placement struct of container

pkg/loader/compose/compose.go

@@ -839,14 +839,12 @@ func handleVolume(komposeObject *kobject.KomposeObject, volumes *types.Volumes)
 			errors.Wrap(err, "could not retrieve vvolume")
 		}
 		for volName, vol := range vols {
-			size, selector, volumeType, secretName := getVolumeLabels(vol.VolumeName, volumes)
+			size, selector := getVolumeLabels(vol.VolumeName, volumes)
-			if len(size) > 0 || len(selector) > 0 || len(volumeType) > 0 || len(secretName) > 0 {
+			if len(size) > 0 || len(selector) > 0 {
 				// We can't assign value to struct field in map while iterating over it, so temporary variable `temp` is used here
 				var temp = vols[volName]
 				temp.PVCSize = size
 				temp.SelectorValue = selector
-				temp.VolumeType = volumeType
-				temp.SecretName = secretName
 				vols[volName] = temp
 			}
 		}
@@ -951,30 +949,20 @@ func getVol(toFind kobject.Volumes, Vols []kobject.Volumes) (bool, kobject.Volum
 	return false, kobject.Volumes{}
 }
 
-func getVolumeLabels(name string, volumes *types.Volumes) (string, string, string, string) {
+func getVolumeLabels(name string, volumes *types.Volumes) (string, string) {
-	size, selector, volumeType, secretName := "", "", "", ""
+	size, selector := "", ""
 
 	if volume, ok := (*volumes)[name]; ok {
-		log.Debugf("Getting labels for volume %s, labels: %v", name, volume.Labels)
 		for key, value := range volume.Labels {
 			if key == "kompose.volume.size" {
 				size = value
 			} else if key == "kompose.volume.selector" {
 				selector = value
-			} else if key == "kompose.volume.type" {
-				volumeType = value
-			} else if key == "kompose.volume.secret-name" {
-				secretName = value
 			}
 		}
-		if volumeType != "" {
-			log.Infof("Volume %s has type %s, secret name: %s", name, volumeType, secretName)
-		}
-	} else {
-		log.Debugf("Volume %s not found in volumes map", name)
 	}
 
-	return size, selector, volumeType, secretName
+	return size, selector
 }
 
 // getGroupAdd will return group in int64 format

pkg/loader/compose/utils.go

@@ -94,12 +94,6 @@ const (
 	LabelInitContainerImage = "kompose.init.containers.image"
 	// LabelInitContainerCommand defines commands
 	LabelInitContainerCommand = "kompose.init.containers.command"
-	// LabelInitContainerService defines service(s) to use as init containers (comma-separated)
-	LabelInitContainerService = "kompose.init.containers.service"
-	// LabelVolumeType defines the type of volume (e.g., "secret" for Kubernetes Secret volumes)
-	LabelVolumeType = "kompose.volume.type"
-	// LabelVolumeSecretName defines the name of the Kubernetes Secret to use
-	LabelVolumeSecretName = "kompose.volume.secret-name"
 	// LabelHpaMinReplicas defines min pod replicas
 	LabelHpaMinReplicas = "kompose.hpa.replicas.min"
 	// LabelHpaMaxReplicas defines max pod replicas

pkg/transformer/kubernetes/k8sutils.go

@@ -82,88 +82,10 @@ type DeploymentMapping struct {
 	TargetDeploymentName string
 }
 
-// ServiceValues holds the extracted values for templating
-type ServiceValues struct {
-	Image struct {
-		Repository string
-		Tag        string
-		PullPolicy string
-	}
-	Env map[string]string
-}
-
-// splitImage splits "repo:tag" or "repo" into repository and tag
-func splitImage(image string) (string, string) {
-	parts := strings.Split(image, ":")
-	if len(parts) == 2 {
-		return parts[0], parts[1]
-	}
-	// No tag specified, use "latest"
-	return image, "latest"
-}
-
-// unquoteHelmTemplates removes quotes around Helm template syntax that yaml.Marshal adds
-func unquoteHelmTemplates(yamlBytes []byte) []byte {
-	yamlStr := string(yamlBytes)
-	lines := strings.Split(yamlStr, "\n")
-
-	for i, line := range lines {
-		// If line contains Helm template syntax {{...}}
-		if strings.Contains(line, "{{") && strings.Contains(line, "}}") {
-			// Remove outer quotes only - find the first and last quote on the line
-			// that enclose the entire value (after the colon for YAML)
-			if idx := strings.Index(line, ": "); idx != -1 {
-				prefix := line[:idx+2] // Keep "key: "
-				value := line[idx+2:]  // Get the value part
-
-				// Remove surrounding quotes from value if present
-				if (strings.HasPrefix(value, `"`) && strings.HasSuffix(value, `"`)) ||
-					(strings.HasPrefix(value, `'`) && strings.HasSuffix(value, `'`)) {
-					value = value[1 : len(value)-1]
-				}
-				lines[i] = prefix + value
-			}
-		}
-	}
-
-	return []byte(strings.Join(lines, "\n"))
-}
-
-// extractValuesFromKomposeObject extracts values from KomposeObject for values.yaml
-func extractValuesFromKomposeObject(komposeObject kobject.KomposeObject) map[string]ServiceValues {
-	values := make(map[string]ServiceValues)
-
-	for serviceName, service := range komposeObject.ServiceConfigs {
-		svcValues := ServiceValues{}
-
-		// Extract image
-		repo, tag := splitImage(service.Image)
-		svcValues.Image.Repository = repo
-		svcValues.Image.Tag = tag
-
-		// Extract pull policy
-		pullPolicy := service.ImagePullPolicy
-		if pullPolicy == "" {
-			pullPolicy = "IfNotPresent"
-		}
-		svcValues.Image.PullPolicy = pullPolicy
-
-		// Extract env vars
-		svcValues.Env = make(map[string]string)
-		for _, envVar := range service.Environment {
-			svcValues.Env[envVar.Name] = envVar.Value
-		}
-
-		values[serviceName] = svcValues
-	}
-
-	return values
-}
-
 /**
  * Generate Helm Chart configuration
  */
-func generateHelm(dirName string, values map[string]ServiceValues) error {
+func generateHelm(dirName string) error {
 	type ChartDetails struct {
 		Name string
 	}
@@ -219,61 +141,10 @@ home:
 		return err
 	}
 
-	/* Create the values.yaml file */
-	if len(values) > 0 {
-		valuesYAML, err := generateValuesYAML(values)
-		if err != nil {
-			return errors.Wrap(err, "Failed to generate values.yaml")
-		}
-		err = os.WriteFile(dirName+string(os.PathSeparator)+"values.yaml", valuesYAML, 0644)
-		if err != nil {
-			return err
-		}
-	}
-
 	log.Infof("chart created in %q\n", dirName+string(os.PathSeparator))
 	return nil
 }
 
-// generateValuesYAML creates values.yaml content from extracted values
-func generateValuesYAML(values map[string]ServiceValues) ([]byte, error) {
-	// Build hierarchical structure: serviceName -> image -> {repository, tag, pullPolicy}
-	valuesMap := make(map[string]interface{})
-
-	// Sort service names for consistent output
-	serviceNames := make([]string, 0, len(values))
-	for name := range values {
-		serviceNames = append(serviceNames, name)
-	}
-	sort.Strings(serviceNames)
-
-	for _, serviceName := range serviceNames {
-		svcValues := values[serviceName]
-		serviceMap := map[string]interface{}{
-			"image": map[string]string{
-				"repository": svcValues.Image.Repository,
-				"tag":        svcValues.Image.Tag,
-				"pullPolicy": svcValues.Image.PullPolicy,
-			},
-		}
-
-		// Add env vars if present
-		if len(svcValues.Env) > 0 {
-			serviceMap["env"] = svcValues.Env
-		}
-
-		valuesMap[serviceName] = serviceMap
-	}
-
-	// Use marshalWithIndent for consistent 2-space indentation
-	yamlBytes, err := marshalWithIndent(valuesMap, 2)
-	if err != nil {
-		return nil, err
-	}
-
-	return yamlBytes, nil
-}
-
 // Check if given path is a directory
 func isDir(name string) (bool, error) {
 	// Open file to get stat later
@@ -312,7 +183,7 @@ func getDirName(opt kobject.ConvertOptions) string {
 }
 
 // PrintList will take the data converted and decide on the commandline attributes given
-func PrintList(objects []runtime.Object, opt kobject.ConvertOptions, komposeObject kobject.KomposeObject) error {
+func PrintList(objects []runtime.Object, opt kobject.ConvertOptions) error {
 	var f *os.File
 	dirName := getDirName(opt)
 	log.Debugf("Target Dir: %s", dirName)
@@ -344,13 +215,6 @@ func PrintList(objects []runtime.Object, opt kobject.ConvertOptions, komposeObje
 	}
 
 	var files []string
-	var imageValues map[string]ServiceValues
-
-	// Extract values from KomposeObject for values.yaml
-	if opt.CreateChart {
-		imageValues = extractValuesFromKomposeObject(komposeObject)
-	}
-
 	// if asked to print to stdout or to put in single file
 	// we will create a list
 	if opt.ToStdout || f != nil {
@@ -368,12 +232,6 @@ func PrintList(objects []runtime.Object, opt kobject.ConvertOptions, komposeObje
 			if err != nil {
 				return fmt.Errorf("error in marshalling the List: %v", err)
 			}
-
-			// Unquote Helm templates if generating chart
-			if opt.CreateChart {
-				data = unquoteHelmTemplates(data)
-			}
-
 			// this part add --- which unifies the file
 			data = []byte(fmt.Sprintf("---\n%s", data))
 			printVal, err := transformer.Print("", dirName, "", data, opt.ToStdout, opt.GenerateJSON, f, opt.Provider)
@@ -407,7 +265,6 @@ func PrintList(objects []runtime.Object, opt kobject.ConvertOptions, komposeObje
 			var typeMeta metav1.TypeMeta
 			var objectMeta metav1.ObjectMeta
 
-			// Get object metadata first for templating
 			if us, ok := v.(*unstructured.Unstructured); ok {
 				typeMeta = metav1.TypeMeta{
 					Kind:       us.GetKind(),
@@ -418,13 +275,13 @@ func PrintList(objects []runtime.Object, opt kobject.ConvertOptions, komposeObje
 				}
 			} else {
 				val := reflect.ValueOf(v).Elem()
+				// Use reflect to access TypeMeta struct inside runtime.Object.
+				// cast it to correct type - metav1.TypeMeta
 				typeMeta = val.FieldByName("TypeMeta").Interface().(metav1.TypeMeta)
-				objectMeta = val.FieldByName("ObjectMeta").Interface().(metav1.ObjectMeta)
-			}
 
-			// Unquote Helm templates if generating chart
+				// Use reflect to access ObjectMeta struct inside runtime.Object.
-			if opt.CreateChart {
+				// cast it to correct type - api.ObjectMeta
-				data = unquoteHelmTemplates(data)
+				objectMeta = val.FieldByName("ObjectMeta").Interface().(metav1.ObjectMeta)
 			}
 
 			file, err = transformer.Print(objectMeta.Name, finalDirName, strings.ToLower(typeMeta.Kind), data, opt.ToStdout, opt.GenerateJSON, f, opt.Provider)
@@ -436,7 +293,7 @@ func PrintList(objects []runtime.Object, opt kobject.ConvertOptions, komposeObje
 		}
 	}
 	if opt.CreateChart {
-		err = generateHelm(dirName, imageValues)
+		err = generateHelm(dirName)
 		if err != nil {
 			return errors.Wrap(err, "generateHelm failed")
 		}
@@ -689,7 +546,7 @@ func (k *Kubernetes) UpdateKubernetesObjectsMultipleContainers(name string, serv
 }
 
 // UpdateKubernetesObjects loads configurations to k8s objects
-func (k *Kubernetes) UpdateKubernetesObjects(name string, service kobject.ServiceConfig, opt kobject.ConvertOptions, objects *[]runtime.Object, komposeObject *kobject.KomposeObject) error {
+func (k *Kubernetes) UpdateKubernetesObjects(name string, service kobject.ServiceConfig, opt kobject.ConvertOptions, objects *[]runtime.Object) error {
 	// Configure the environment variables.
 	envs, envsFrom, err := ConfigEnvs(service, opt)
 	if err != nil {
@@ -729,101 +586,6 @@ func (k *Kubernetes) UpdateKubernetesObjects(name string, service kobject.Servic
 	// Configure annotations
 	annotations := transformer.ConfigAnnotations(service)
 
-	// Process init container services if specified
-	var initContainers []api.Container
-	if initServicesStr, ok := service.Labels[compose.LabelInitContainerService]; ok && initServicesStr != "" && komposeObject != nil {
-		initServiceNames := strings.Split(initServicesStr, ",")
-		for _, initSvcName := range initServiceNames {
-			initSvcName = strings.TrimSpace(initSvcName)
-			if initSvcName == "" {
-				continue
-			}
-
-			// Find the init container service in the kompose object
-			initService, exists := komposeObject.ServiceConfigs[initSvcName]
-			if !exists {
-				log.Warnf("Init container service %s not found for service %s", initSvcName, name)
-				continue
-			}
-
-			log.Infof("Adding init container %s to service %s", initSvcName, name)
-
-			// Configure init container environment variables
-			initEnvs, initEnvsFrom, err := ConfigEnvs(initService, opt)
-			if err != nil {
-				return errors.Wrapf(err, "Unable to load env variables for init container %s", initSvcName)
-			}
-
-			// Configure init container volumes
-			initVolumeMounts, initVolumes, initPvc, initCms, err := k.ConfigVolumes(initSvcName, initService)
-			if err != nil {
-				return errors.Wrapf(err, "k.ConfigVolumes failed for init container %s", initSvcName)
-			}
-
-			// Add init container volumes to the main volumes list (deduplicate by name)
-			existingVolumeNames := make(map[string]bool)
-			for _, v := range volumes {
-				existingVolumeNames[v.Name] = true
-			}
-			for _, initVol := range initVolumes {
-				if !existingVolumeNames[initVol.Name] {
-					volumes = append(volumes, initVol)
-					existingVolumeNames[initVol.Name] = true
-				}
-			}
-
-			// Add init container PVCs and ConfigMaps to objects (deduplicate by name)
-			// Build a set of existing object names
-			existingObjectNames := make(map[string]bool)
-			for _, obj := range *objects {
-				switch o := obj.(type) {
-				case *api.PersistentVolumeClaim:
-					existingObjectNames[o.Name] = true
-				case *api.ConfigMap:
-					existingObjectNames[o.Name] = true
-				}
-			}
-
-			if initPvc != nil {
-				for _, p := range initPvc {
-					if !existingObjectNames[p.Name] {
-						*objects = append(*objects, p)
-						existingObjectNames[p.Name] = true
-					}
-				}
-			}
-			for _, c := range initCms {
-				if !existingObjectNames[c.Name] {
-					*objects = append(*objects, c)
-					existingObjectNames[c.Name] = true
-				}
-			}
-
-			// Get init container image
-			initImage := initService.Image
-			if initImage == "" {
-				initImage = initSvcName
-			}
-			if opt.CreateChart {
-				initImage = "{{ " + helmValuesPath(initSvcName, "image", "repository") + " }}:{{ " + helmValuesPath(initSvcName, "image", "tag") + " }}"
-			}
-
-			// Create init container spec
-			initContainer := api.Container{
-				Name:         FormatContainerName(initSvcName),
-				Image:        initImage,
-				Command:      initService.Command,
-				Args:         GetContainerArgs(initService),
-				Env:          initEnvs,
-				EnvFrom:      initEnvsFrom,
-				VolumeMounts: initVolumeMounts,
-				WorkingDir:   initService.WorkingDir,
-			}
-
-			initContainers = append(initContainers, initContainer)
-		}
-	}
-
 	// fillTemplate fills the pod template with the value calculated from config
 	fillTemplate := func(template *api.PodTemplateSpec) error {
 		template.Spec.Containers[0].Name = GetContainerName(service)
@@ -960,11 +722,6 @@ func (k *Kubernetes) UpdateKubernetesObjects(name string, service kobject.Servic
 			template.Spec.ServiceAccountName = serviceAccountName
 		}
 		fillInitContainers(template, service)
-
-		// Add init containers from referenced services
-		if len(initContainers) > 0 {
-			template.Spec.InitContainers = append(template.Spec.InitContainers, initContainers...)
-		}
 		return nil
 	}
 

pkg/transformer/kubernetes/kubernetes.go

@@ -57,22 +57,11 @@ type Kubernetes struct {
 	Opt kobject.ConvertOptions
 }
 
-// helmValuesPath generates a Helm template path using index notation
-// This works for all service names including those with hyphens
-func helmValuesPath(parts ...string) string {
-	quotedParts := make([]string, len(parts)+1)
-	quotedParts[0] = ".Values"
-	for i, part := range parts {
-		quotedParts[i+1] = `"` + part + `"`
-	}
-	return "index " + strings.Join(quotedParts, " ")
-}
-
 // PVCRequestSize (Persistent Volume Claim) has default size
 const PVCRequestSize = "100Mi"
 
 // ValidVolumeSet has the different types of valid volumes
-var ValidVolumeSet = map[string]struct{}{"emptyDir": {}, "hostPath": {}, "configMap": {}, "persistentVolumeClaim": {}, "secret": {}}
+var ValidVolumeSet = map[string]struct{}{"emptyDir": {}, "hostPath": {}, "configMap": {}, "persistentVolumeClaim": {}}
 
 const (
 	// DeploymentController is controller type for Deployment
@@ -120,17 +109,10 @@ func (k *Kubernetes) CheckUnsupportedKey(komposeObject *kobject.KomposeObject, u
 }
 
 // InitPodSpec creates the pod specification
-func (k *Kubernetes) InitPodSpec(name string, service kobject.ServiceConfig, opt kobject.ConvertOptions) api.PodSpec {
+func (k *Kubernetes) InitPodSpec(name string, image string, pullSecret string) api.PodSpec {
-	image := service.Image
 	if image == "" {
 		image = name
 	}
-
-	// Inject Helm template for chart generation
-	if opt.CreateChart {
-		image = "{{ " + helmValuesPath(name, "image", "repository") + " }}:{{ " + helmValuesPath(name, "image", "tag") + " }}"
-	}
-
 	pod := api.PodSpec{
 		Containers: []api.Container{
 			{
@@ -139,11 +121,10 @@ func (k *Kubernetes) InitPodSpec(name string, service kobject.ServiceConfig, opt
 			},
 		},
 	}
-
+	if pullSecret != "" {
-	if service.ImagePullSecret != "" {
 		pod.ImagePullSecrets = []api.LocalObjectReference{
 			{
-				Name: service.ImagePullSecret,
+				Name: pullSecret,
 			},
 		}
 	}
@@ -151,7 +132,7 @@ func (k *Kubernetes) InitPodSpec(name string, service kobject.ServiceConfig, opt
 }
 
 // InitPodSpecWithConfigMap creates the pod specification
-func (k *Kubernetes) InitPodSpecWithConfigMap(name string, service kobject.ServiceConfig, opt kobject.ConvertOptions) api.PodSpec {
+func (k *Kubernetes) InitPodSpecWithConfigMap(name string, image string, service kobject.ServiceConfig) api.PodSpec {
 	var volumeMounts []api.VolumeMount
 	var volumes []api.Volume
 
@@ -196,16 +177,6 @@ func (k *Kubernetes) InitPodSpecWithConfigMap(name string, service kobject.Servi
 		volumes = append(volumes, cmVol)
 	}
 
-	image := service.Image
-	if image == "" {
-		image = name
-	}
-
-	// Inject Helm template for chart generation
-	if opt.CreateChart {
-		image = "{{ " + helmValuesPath(name, "image", "repository") + " }}:{{ " + helmValuesPath(name, "image", "tag") + " }}"
-	}
-
 	pod := api.PodSpec{
 		Containers: []api.Container{
 			{
@@ -358,10 +329,6 @@ func (k *Kubernetes) IntiConfigMapFromFileOrDir(name, cmName, filePath string, s
 		configMap.Annotations = map[string]string{
 			"use-subpath": "true",
 		}
-		// Check if the file is executable and store that info
-		if mode&0111 != 0 { // Check if any execute bit is set
-			configMap.Annotations["executable"] = "true"
-		}
 	}
 
 	return configMap, nil
@@ -445,12 +412,12 @@ func (k *Kubernetes) InitConfigMapFromFile(name string, service kobject.ServiceC
 }
 
 // InitD initializes Kubernetes Deployment object
-func (k *Kubernetes) InitD(name string, service kobject.ServiceConfig, replicas int, opt kobject.ConvertOptions) *appsv1.Deployment {
+func (k *Kubernetes) InitD(name string, service kobject.ServiceConfig, replicas int) *appsv1.Deployment {
 	var podSpec api.PodSpec
 	if len(service.Configs) > 0 {
-		podSpec = k.InitPodSpecWithConfigMap(name, service, opt)
+		podSpec = k.InitPodSpecWithConfigMap(name, service.Image, service)
 	} else {
-		podSpec = k.InitPodSpec(name, service, opt)
+		podSpec = k.InitPodSpec(name, service.Image, service.ImagePullSecret)
 	}
 
 	rp := int32(replicas)
@@ -501,7 +468,7 @@ func (k *Kubernetes) InitD(name string, service kobject.ServiceConfig, replicas
 }
 
 // InitDS initializes Kubernetes DaemonSet object
-func (k *Kubernetes) InitDS(name string, service kobject.ServiceConfig, opt kobject.ConvertOptions) *appsv1.DaemonSet {
+func (k *Kubernetes) InitDS(name string, service kobject.ServiceConfig) *appsv1.DaemonSet {
 	ds := &appsv1.DaemonSet{
 		TypeMeta: metav1.TypeMeta{
 			Kind:       "DaemonSet",
@@ -516,7 +483,7 @@ func (k *Kubernetes) InitDS(name string, service kobject.ServiceConfig, opt kobj
 				MatchLabels: transformer.ConfigLabels(name),
 			},
 			Template: api.PodTemplateSpec{
-				Spec: k.InitPodSpec(name, service, opt),
+				Spec: k.InitPodSpec(name, service.Image, service.ImagePullSecret),
 			},
 		},
 	}
@@ -524,12 +491,12 @@ func (k *Kubernetes) InitDS(name string, service kobject.ServiceConfig, opt kobj
 }
 
 // InitSS method initialize a stateful set
-func (k *Kubernetes) InitSS(name string, service kobject.ServiceConfig, replicas int, opt kobject.ConvertOptions) *appsv1.StatefulSet {
+func (k *Kubernetes) InitSS(name string, service kobject.ServiceConfig, replicas int) *appsv1.StatefulSet {
 	var podSpec api.PodSpec
 	if len(service.Configs) > 0 {
-		podSpec = k.InitPodSpecWithConfigMap(name, service, opt)
+		podSpec = k.InitPodSpecWithConfigMap(name, service.Image, service)
 	} else {
-		podSpec = k.InitPodSpec(name, service, opt)
+		podSpec = k.InitPodSpec(name, service.Image, service.ImagePullSecret)
 	}
 	rp := int32(replicas)
 	ds := &appsv1.StatefulSet{
@@ -556,7 +523,7 @@ func (k *Kubernetes) InitSS(name string, service kobject.ServiceConfig, replicas
 }
 
 // InitCJ initializes Kubernetes CronJob object
-func (k *Kubernetes) InitCJ(name string, service kobject.ServiceConfig, schedule string, concurrencyPolicy batchv1.ConcurrencyPolicy, backoffLimit *int32, opt kobject.ConvertOptions) *batchv1.CronJob {
+func (k *Kubernetes) InitCJ(name string, service kobject.ServiceConfig, schedule string, concurrencyPolicy batchv1.ConcurrencyPolicy, backoffLimit *int32) *batchv1.CronJob {
 	cj := &batchv1.CronJob{
 		TypeMeta: metav1.TypeMeta{
 			Kind:       "CronJob",
@@ -573,7 +540,7 @@ func (k *Kubernetes) InitCJ(name string, service kobject.ServiceConfig, schedule
 				Spec: batchv1.JobSpec{
 					BackoffLimit: backoffLimit,
 					Template: api.PodTemplateSpec{
-						Spec: k.InitPodSpec(name, service, opt),
+						Spec: k.InitPodSpec(name, service.Image, service.ImagePullSecret),
 					},
 				},
 			},
@@ -1022,15 +989,13 @@ func (k *Kubernetes) ConfigVolumes(name string, service kobject.ServiceConfig) (
 	}
 
 	// Override volume type if specified in service labels.
-	var useSecret bool
+	if vt, ok := service.Labels["kompose.volume.type"]; ok {
-	if vt, ok := service.Labels[compose.LabelVolumeType]; ok {
 		if _, okk := ValidVolumeSet[vt]; !okk {
-			return nil, nil, nil, nil, fmt.Errorf("invalid volume type %s specified in label '%s' in service %s", vt, compose.LabelVolumeType, service.Name)
+			return nil, nil, nil, nil, fmt.Errorf("invalid volume type %s specified in label 'kompose.volume.type' in service %s", vt, service.Name)
 		}
 		useEmptyVolumes = vt == "emptyDir"
 		useHostPath = vt == "hostPath"
 		useConfigMap = vt == "configMap"
-		useSecret = vt == "secret"
 	}
 
 	// config volumes from secret if present
@@ -1044,52 +1009,23 @@ func (k *Kubernetes) ConfigVolumes(name string, service kobject.ServiceConfig) (
 	for _, volume := range service.Volumes {
 		// check if ro/rw mode is defined, default rw
 		readonly := len(volume.Mode) > 0 && (volume.Mode == "ro" || volume.Mode == "rox")
-
+		mountHost := volume.Host
-		// Reset volume type flags for each volume (service-level flags don't apply to individual volumes)
+		if mountHost == "" {
-		volumeUseSecret := useSecret
+			mountHost = volume.MountPath
-		volumeUseConfigMap := useConfigMap
+		}
-		volumeUseHostPath := useHostPath
+		// return useconfigmap and readonly,
-		volumeUseEmpty := useEmptyVolumes
+		// not used asigned readonly because dont break e2e
-
+		useConfigMap, _, skip = isConfigFile(mountHost)
-		// Check volume type from volume labels first, then fall back to file/path detection
+		if skip {
-		if volume.VolumeType == "secret" {
+			log.Warnf("Skip file in path %s ", volume.Host)
-			// Explicit secret volume from volume labels
+			continue
-			volumeUseSecret = true
-			volumeUseConfigMap = false
-			volumeUseHostPath = false
-			volumeUseEmpty = false
-		} else if volume.Host != "" {
-			// Bind mount: detect if it's a config file, otherwise treat as hostPath
-			volumeUseConfigMap, _, skip = isConfigFile(volume.Host)
-			if skip {
-				log.Warnf("Skip file in path %s ", volume.Host)
-				continue
-			}
-			if volumeUseConfigMap {
-				// It's a config file, use ConfigMap
-				volumeUseSecret = false
-				volumeUseHostPath = false
-				volumeUseEmpty = false
-			} else {
-				// It's a regular bind mount, use hostPath
-				volumeUseConfigMap = false
-				volumeUseSecret = false
-				volumeUseHostPath = true
-				volumeUseEmpty = false
-			}
-		} else {
-			// Named volume with no host path
-			// Keep volumeUseSecret if set, otherwise will fall through to PVC
-			volumeUseConfigMap = false
-			volumeUseHostPath = false
-			volumeUseEmpty = false
 		}
 		if volume.VolumeName == "" {
-			if volumeUseEmpty {
+			if useEmptyVolumes {
 				volumeName = strings.Replace(volume.PVCName, "claim", "empty", 1)
-			} else if volumeUseHostPath {
+			} else if useHostPath {
 				volumeName = strings.Replace(volume.PVCName, "claim", "hostpath", 1)
-			} else if volumeUseConfigMap {
+			} else if useConfigMap {
 				volumeName = strings.Replace(volume.PVCName, "claim", "cm", 1)
 			} else {
 				volumeName = volume.PVCName
@@ -1111,15 +1047,15 @@ func (k *Kubernetes) ConfigVolumes(name string, service kobject.ServiceConfig) (
 		// For PVC we will also create a PVC object and add to list
 		var volsource *api.VolumeSource
 
-		if volumeUseEmpty {
+		if useEmptyVolumes {
 			volsource = k.ConfigEmptyVolumeSource("volume")
-		} else if volumeUseHostPath {
+		} else if useHostPath {
 			source, err := k.ConfigHostPathVolumeSource(volume.Host)
 			if err != nil {
 				return nil, nil, nil, nil, errors.Wrap(err, "k.ConfigHostPathVolumeSource failed")
 			}
 			volsource = source
-		} else if volumeUseConfigMap {
+		} else if useConfigMap {
 			log.Debugf("Use configmap volume")
 			cm, err := k.IntiConfigMapFromFileOrDir(name, volumeName, volume.Host, service)
 			if err != nil {
@@ -1131,25 +1067,6 @@ func (k *Kubernetes) ConfigVolumes(name string, service kobject.ServiceConfig) (
 			if useSubPathMount(cm) {
 				volMount.SubPath = volsource.ConfigMap.Items[0].Path
 			}
-		} else if volumeUseSecret {
-			// Get secret name from volume config or service label, or default to volume name
-			secretName := volume.SecretName
-			if secretName == "" {
-				secretName = service.Labels[compose.LabelVolumeSecretName]
-			}
-			if secretName == "" {
-				secretName = volumeName
-			}
-			log.Infof("Creating Secret volume %s with secret %s", volumeName, secretName)
-
-			// Create secret volume source (all secrets are optional by default)
-			optional := true
-			volsource = &api.VolumeSource{
-				Secret: &api.SecretVolumeSource{
-					SecretName: secretName,
-					Optional:   &optional,
-				},
-			}
 		} else {
 			volsource = k.ConfigPVCVolumeSource(volumeName, readonly)
 			if volume.VFrom == "" {
@@ -1219,13 +1136,6 @@ func (k *Kubernetes) ConfigEmptyVolumeSource(key string) *api.VolumeSource {
 func (k *Kubernetes) ConfigConfigMapVolumeSource(cmName string, targetPath string, cm *api.ConfigMap) *api.VolumeSource {
 	s := api.ConfigMapVolumeSource{}
 	s.Name = cmName
-
-	// Set default mode to 0755 if the file is executable
-	if cm.Annotations != nil && cm.Annotations["executable"] == "true" {
-		mode := int32(0755)
-		s.DefaultMode = &mode
-	}
-
 	if useSubPathMount(cm) {
 		var keys []string
 		for k := range cm.Data {
@@ -1315,18 +1225,12 @@ func ConfigEnvs(service kobject.ServiceConfig, opt kobject.ConvertOptions) ([]ap
 	// Load up the environment variables
 	for _, v := range service.Environment {
 		if !keysFromEnvFile[v.Name] {
-			value := v.Value
+			if strings.Contains(v.Value, "run/secrets") {
-			if opt.CreateChart {
+				v.Value = FormatResourceName(v.Value)
-				// Inject Helm template syntax for chart generation
-				value = "{{ " + helmValuesPath(service.Name, "env", v.Name) + " | quote }}"
-			} else {
-				if strings.Contains(value, "run/secrets") {
-					value = FormatResourceName(value)
-				}
 			}
 			envs = append(envs, api.EnvVar{
 				Name:  v.Name,
-				Value: value,
+				Value: v.Value,
 			})
 		}
 	}
@@ -1443,15 +1347,15 @@ func (k *Kubernetes) CreateWorkloadAndConfigMapObjects(name string, service kobj
 	}
 
 	if opt.CreateD || opt.Controller == DeploymentController {
-		objects = append(objects, k.InitD(name, service, replica, opt))
+		objects = append(objects, k.InitD(name, service, replica))
 	}
 
 	if opt.CreateDS || opt.Controller == DaemonSetController {
-		objects = append(objects, k.InitDS(name, service, opt))
+		objects = append(objects, k.InitDS(name, service))
 	}
 
 	if opt.Controller == StatefulStateController {
-		objects = append(objects, k.InitSS(name, service, replica, opt))
+		objects = append(objects, k.InitSS(name, service, replica))
 	}
 
 	envConfigMaps := k.PargeEnvFiletoConfigMaps(name, service, opt)
@@ -1488,7 +1392,7 @@ func (k *Kubernetes) createConfigMapFromComposeConfig(name string, service kobje
 }
 
 // InitPod initializes Kubernetes Pod object
-func (k *Kubernetes) InitPod(name string, service kobject.ServiceConfig, opt kobject.ConvertOptions) *api.Pod {
+func (k *Kubernetes) InitPod(name string, service kobject.ServiceConfig) *api.Pod {
 	pod := api.Pod{
 		TypeMeta: metav1.TypeMeta{
 			Kind:       "Pod",
@@ -1499,7 +1403,7 @@ func (k *Kubernetes) InitPod(name string, service kobject.ServiceConfig, opt kob
 			Labels:      transformer.ConfigLabels(name),
 			Annotations: transformer.ConfigAnnotations(service),
 		},
-		Spec: k.InitPodSpec(name, service, opt),
+		Spec: k.InitPodSpec(name, service.Image, service.ImagePullSecret),
 	}
 	return &pod
 }
@@ -1767,22 +1671,6 @@ func (k *Kubernetes) Transform(komposeObject kobject.KomposeObject, opt kobject.
 			allobjects = append(allobjects, objects...)
 		}
 	}
-	// Track services used as init containers (to skip creating deployments for them)
-	initContainerServices := make(map[string]bool)
-	for _, service := range komposeObject.ServiceConfigs {
-		if initServicesStr, ok := service.Labels[compose.LabelInitContainerService]; ok && initServicesStr != "" {
-			// Parse comma-separated list of init container service names
-			initServices := strings.Split(initServicesStr, ",")
-			for _, initSvc := range initServices {
-				initSvc = strings.TrimSpace(initSvc)
-				if initSvc != "" {
-					initContainerServices[initSvc] = true
-					log.Infof("Service %s will be used as init container for service %s", initSvc, service.Name)
-				}
-			}
-		}
-	}
-
 	sortedKeys := SortedKeys(komposeObject.ServiceConfigs)
 	for _, name := range sortedKeys {
 		service := komposeObject.ServiceConfigs[name]
@@ -1792,12 +1680,6 @@ func (k *Kubernetes) Transform(komposeObject kobject.KomposeObject, opt kobject.
 			continue
 		}
 
-		// Skip services that are used as init containers for other services
-		if initContainerServices[name] {
-			log.Infof("Skipping deployment creation for service %s (used as init container)", name)
-			continue
-		}
-
 		var objects []runtime.Object
 
 		service.WithKomposeAnnotation = opt.WithKomposeAnnotation
@@ -1810,10 +1692,10 @@ func (k *Kubernetes) Transform(komposeObject kobject.KomposeObject, opt kobject.
 		if (service.Restart == "no" || service.Restart == "on-failure") && !opt.IsPodController() {
 			if service.CronJobSchedule != "" {
 				log.Infof("Create kubernetes pod instead of pod controller due to restart policy: %s", service.Restart)
-				cronJob := k.InitCJ(name, service, service.CronJobSchedule, service.CronJobConcurrencyPolicy, service.CronJobBackoffLimit, opt)
+				cronJob := k.InitCJ(name, service, service.CronJobSchedule, service.CronJobConcurrencyPolicy, service.CronJobBackoffLimit)
 				objects = append(objects, cronJob)
 			} else {
-				pod := k.InitPod(name, service, opt)
+				pod := k.InitPod(name, service)
 				objects = append(objects, pod)
 			}
 			envConfigMaps := k.PargeEnvFiletoConfigMaps(name, service, opt)
@@ -1825,7 +1707,7 @@ func (k *Kubernetes) Transform(komposeObject kobject.KomposeObject, opt kobject.
 			service.ServiceType = "Headless"
 		}
 		k.configKubeServiceAndIngressForService(service, name, &objects)
-		err := k.UpdateKubernetesObjects(name, service, opt, &objects, &komposeObject)
+		err := k.UpdateKubernetesObjects(name, service, opt, &objects)
 		if err != nil {
 			return nil, errors.Wrap(err, "Error transforming Kubernetes objects")
 		}

pkg/transformer/kubernetes/kubernetes_test.go

@@ -611,9 +611,7 @@ func TestRestartOnFailure(t *testing.T) {
 func TestInitPodSpec(t *testing.T) {
 	name := "foo"
 	k := Kubernetes{}
-	service := newServiceConfig()
+	result := k.InitPodSpec(name, newServiceConfig().Image, "")
-	opt := kobject.ConvertOptions{}
-	result := k.InitPodSpec(name, service, opt)
 	if result.Containers[0].Name != "foo" && result.Containers[0].Image != "image" {
 		t.Fatalf("Pod object not found")
 	}

pkg/transformer/openshift/openshift.go

@@ -151,7 +151,7 @@ func initBuildConfig(name string, service kobject.ServiceConfig, repo string, br
 }
 
 // initDeploymentConfig initializes OpenShifts DeploymentConfig object
-func (o *OpenShift) initDeploymentConfig(name string, service kobject.ServiceConfig, replicas int, opt kobject.ConvertOptions) *deployapi.DeploymentConfig {
+func (o *OpenShift) initDeploymentConfig(name string, service kobject.ServiceConfig, replicas int) *deployapi.DeploymentConfig {
 	containerName := []string{name}
 
 	// Properly add tags to the image name
@@ -164,14 +164,9 @@ func (o *OpenShift) initDeploymentConfig(name string, service kobject.ServiceCon
 
 	var podSpec corev1.PodSpec
 	if len(service.Configs) > 0 {
-		podSpec = o.InitPodSpecWithConfigMap(name, service, opt)
+		podSpec = o.InitPodSpecWithConfigMap(name, " ", service)
 	} else {
-		podSpec = o.InitPodSpec(name, service, opt)
+		podSpec = o.InitPodSpec(name, " ", "")
-	}
-
-	// OpenShift: Set image to a space - actual image comes from ImageStream trigger
-	for i := range podSpec.Containers {
-		podSpec.Containers[i].Image = " "
 	}
 
 	dc := &deployapi.DeploymentConfig{
@@ -338,10 +333,10 @@ func (o *OpenShift) Transform(komposeObject kobject.KomposeObject, opt kobject.C
 			}
 
 			if service.CronJobSchedule != "" {
-				cronJob := o.InitCJ(name, service, service.CronJobSchedule, service.CronJobConcurrencyPolicy, service.CronJobBackoffLimit, opt)
+				cronJob := o.InitCJ(name, service, service.CronJobSchedule, service.CronJobConcurrencyPolicy, service.CronJobBackoffLimit)
 				objects = append(objects, cronJob)
 			} else {
-				pod := o.InitPod(name, service, opt)
+				pod := o.InitPod(name, service)
 				objects = append(objects, pod)
 			}
 
@@ -351,7 +346,7 @@ func (o *OpenShift) Transform(komposeObject kobject.KomposeObject, opt kobject.C
 			objects = o.CreateWorkloadAndConfigMapObjects(name, service, opt)
 
 			if opt.CreateDeploymentConfig {
-				objects = append(objects, o.initDeploymentConfig(name, service, replica, opt)) // OpenShift DeploymentConfigs
+				objects = append(objects, o.initDeploymentConfig(name, service, replica)) // OpenShift DeploymentConfigs
 				// create ImageStream after deployment (creating IS will trigger new deployment)
 				objects = append(objects, o.initImageStream(name, service, opt))
 			}
@@ -431,7 +426,7 @@ func (o *OpenShift) Transform(komposeObject kobject.KomposeObject, opt kobject.C
 			}
 		}
 
-		err := o.UpdateKubernetesObjects(name, service, opt, &objects, &komposeObject)
+		err := o.UpdateKubernetesObjects(name, service, opt, &objects)
 		if err != nil {
 			return nil, errors.Wrap(err, "Error transforming Kubernetes objects")
 		}

pkg/transformer/openshift/openshift_test.go

@@ -74,12 +74,9 @@ func TestOpenShiftUpdateKubernetesObjects(t *testing.T) {
 	o := OpenShift{}
 	serviceConfig := newServiceConfig()
 	opt := kobject.ConvertOptions{}
-	komposeObject := kobject.KomposeObject{
-		ServiceConfigs: map[string]kobject.ServiceConfig{"foobar": serviceConfig},
-	}
 
-	object = append(object, o.initDeploymentConfig("foobar", serviceConfig, 3, opt))
+	object = append(object, o.initDeploymentConfig("foobar", serviceConfig, 3))
-	o.UpdateKubernetesObjects("foobar", serviceConfig, opt, &object, &komposeObject)
+	o.UpdateKubernetesObjects("foobar", serviceConfig, opt, &object)
 
 	for _, obj := range object {
 		switch tobj := obj.(type) {
@@ -98,8 +95,7 @@ func TestOpenShiftUpdateKubernetesObjects(t *testing.T) {
 
 func TestInitDeploymentConfig(t *testing.T) {
 	o := OpenShift{}
-	opt := kobject.ConvertOptions{}
+	spec := o.initDeploymentConfig("foobar", newServiceConfig(), 1)
-	spec := o.initDeploymentConfig("foobar", newServiceConfig(), 1, opt)
 
 	// Check that "foobar" is used correctly as a name
 	if spec.Spec.Template.Spec.Containers[0].Name != "foobar" {