fallback to skip volume mount and log warning when path cannot be resolved, as /sys, /root, /var/lib/docker
in e2e TestMultipleContainersInPod directory /data replaced with /data-dir
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
* adds a label for initContainers, test and user-guide
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
* solved suggestion version from 2 to 3, coments
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
* test e2e
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
* add target to kompose build
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
* chore(deps)(deps): bump golang.org/x/tools from 0.16.1 to 0.19.0 (#1836)
Bumps [golang.org/x/tools](https://github.com/golang/tools) from 0.16.1 to 0.19.0.
- [Release notes](https://github.com/golang/tools/releases)
- [Commits](https://github.com/golang/tools/compare/v0.16.1...v0.19.0)
---
updated-dependencies:
- dependency-name: golang.org/x/tools
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* changed var names contaers to containers
changed labels to kompose.init.container.name, kompose.init.container.image, kompose.init.container.command
fixed documentation, tests and e2e
also merged main with new commits to this branch
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
---------
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Kubernetes Prow Robot <20407524+k8s-ci-robot@users.noreply.github.com>
add tests and e2e
function isconfigfile check if we need to use configmap and mount in readonly
readonly not asigned because not fail e2e
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
* fix issue 1793 https://github.com/kubernetes/kompose/issues/1793
*add tests
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
formated k8utils_test.go
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
labels formatted as name"-"envName to match fixtures when performing the gitHub action
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
removed this piece code because apply it later, and it is redundant
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
Refactor test in k8sutils_test.go to extract the last 63 characters. This addresses the impact of the removed code that previously truncated the input with if len(envName) > 63 { envName = envName[len(envName)-63:] }
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
changed to name function to getUsableNameEnvFile
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
fix issue 1793
* fix issue 1793 https://github.com/kubernetes/kompose/issues/1793
*add tests
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
labels formatted as name"-"envName to match fixtures when performing the gitHub action
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
changed to name function to getUsableNameEnvFile
Signed-off-by: jose luis <2064537+sosan@users.noreply.github.com>
* fix issue https://github.com/kubernetes/kompose/issues/1683
* add tests, die if inputfiles not specified
* fix GetComposeFileDir function and use it
* use fixed GetComposeFileDir instead of filepath.Dir
* return error instead of fatal
* add pod configmap creation to openshift
* fix: env_file configmap name length
fix filename to configmap name transformer
* fix configmap names in tests
* add env-multiple fixture and test
* adjust to test
* disable env-multiple suite
#### What type of PR is this?
<!--
Add one of the following kinds:
/kind cleanup
/kind documentation
/kind feature
-->
/kind bug
#### What this PR does / why we need it:
When we generate the YAML, we should NOT add namespace by default,
namespace should only be added if it has been specified via the command
line.
#### Which issue(s) this PR fixes:
<!--
*Automatically closes linked issue when PR is merged.
Usage: `Fixes #<issue number>`, or `Fixes (paste link of issue)`.
-->
Fixes https://github.com/kubernetes/kompose/issues/1729
#### Special notes for your reviewer:
Signed-off-by: Charlie Drage <charlie@charliedrage.com>
#### What type of PR is this?
<!--
Add one of the following kinds:
/kind cleanup
/kind documentation
/kind feature
-->
/kind bug
Fixes a validation bug where if you do not provide a compose.yaml or
docker-compose.yaml it will nil point error out rather than have an
appropriate "file not found" output.
#### Which issue(s) this PR fixes:
<!--
*Automatically closes linked issue when PR is merged.
Usage: `Fixes #<issue number>`, or `Fixes (paste link of issue)`.
-->
Closes https://github.com/kubernetes/kompose/issues/1719
#### Special notes for your reviewer:
Signed-off-by: Charlie Drage <charlie@charliedrage.com>
