Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

k8s_cp - fix issue when directory contains space in its name #552

Merged
merged 2 commits into from
Dec 15, 2022
Merged

k8s_cp - fix issue when directory contains space in its name #552

merged 2 commits into from
Dec 15, 2022

Conversation

abikouo
Copy link
Contributor

@abikouo abikouo commented Dec 12, 2022 •
edited
Loading

Depends-On: #549

SUMMARY

There is a remaining issue not addressed by #512 when copying directory from Pod to local filesystem, if the directory contains space into its name, the directory was not copied

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

k8s_cp

ADDITIONAL INFORMATION

@softwarefactory-project-zuul
Copy link

Build failed.

✔️ build-ansible-collection SUCCESS in 3m 47s
❌ ansible-test-sanity-docker-devel FAILURE in 8m 08s (non-voting)
❌ ansible-test-sanity-docker-milestone FAILURE in 7m 47s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 10m 24s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 14s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 14s
✔️ ansible-test-units-kubernetes-core-python38 SUCCESS in 6m 23s
✔️ ansible-test-splitter SUCCESS in 2m 24s
❌ integration-kubernetes.core-devel-1 FAILURE in 41m 04s (non-voting)
✔️ integration-kubernetes.core-devel-2 SUCCESS in 37m 50s (non-voting)
⚠️ integration-kubernetes.core-devel-3 SKIPPED (non-voting)
❌ integration-kubernetes.core-milestone-1 FAILURE in 39m 21s
✔️ integration-kubernetes.core-milestone-2 SUCCESS in 37m 52s
⚠️ integration-kubernetes.core-milestone-3 SKIPPED
❌ integration-kubernetes.core-2.9-1 FAILURE in 42m 10s
❌ integration-kubernetes.core-2.9-2 FAILURE in 41m 45s
⚠️ integration-kubernetes.core-2.9-3 SKIPPED
❌ integration-kubernetes.core-2.10-1 FAILURE in 42m 04s
✔️ integration-kubernetes.core-2.10-2 SUCCESS in 37m 31s
⚠️ integration-kubernetes.core-2.10-3 SKIPPED
❌ integration-kubernetes.core-with-turbo-1 FAILURE in 41m 28s
✔️ integration-kubernetes.core-with-turbo-2 SUCCESS in 40m 11s
⚠️ integration-kubernetes.core-with-turbo-3 SKIPPED
✔️ ansible-tox-linters SUCCESS in 4m 45s
✔️ ansible-test-changelog SUCCESS in 2m 09s
✔️ ansible-galaxy-importer SUCCESS in 3m 55s

@abikouo
Copy link
Contributor Author

abikouo commented Dec 13, 2022

recheck

@softwarefactory-project-zuul
Copy link

Build failed.

✔️ build-ansible-collection SUCCESS in 3m 16s
❌ ansible-test-sanity-docker-devel FAILURE in 11m 32s (non-voting)
❌ ansible-test-sanity-docker-milestone FAILURE in 7m 54s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 10m 24s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 00s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 8m 46s
✔️ ansible-test-units-kubernetes-core-python38 SUCCESS in 6m 15s
✔️ ansible-test-splitter SUCCESS in 2m 24s
✔️ integration-kubernetes.core-devel-1 SUCCESS in 34m 07s (non-voting)
✔️ integration-kubernetes.core-devel-2 SUCCESS in 36m 55s (non-voting)
⚠️ integration-kubernetes.core-devel-3 SKIPPED (non-voting)
❌ integration-kubernetes.core-milestone-1 FAILURE in 33m 36s
✔️ integration-kubernetes.core-milestone-2 SUCCESS in 36m 24s
⚠️ integration-kubernetes.core-milestone-3 SKIPPED
✔️ integration-kubernetes.core-2.9-1 SUCCESS in 35m 01s
✔️ integration-kubernetes.core-2.9-2 SUCCESS in 38m 34s
⚠️ integration-kubernetes.core-2.9-3 SKIPPED
✔️ integration-kubernetes.core-2.10-1 SUCCESS in 35m 16s
✔️ integration-kubernetes.core-2.10-2 SUCCESS in 37m 01s
⚠️ integration-kubernetes.core-2.10-3 SKIPPED
✔️ integration-kubernetes.core-with-turbo-1 SUCCESS in 34m 44s
✔️ integration-kubernetes.core-with-turbo-2 SUCCESS in 36m 18s
⚠️ integration-kubernetes.core-with-turbo-3 SKIPPED
✔️ ansible-tox-linters SUCCESS in 4m 49s
✔️ ansible-test-changelog SUCCESS in 2m 12s
✔️ ansible-galaxy-importer SUCCESS in 3m 40s

@abikouo
Copy link
Contributor Author

abikouo commented Dec 13, 2022

recheck

@abikouo abikouo closed this Dec 13, 2022
@abikouo abikouo reopened this Dec 13, 2022
@softwarefactory-project-zuul
Copy link

Build succeeded.

✔️ build-ansible-collection SUCCESS in 3m 17s
❌ ansible-test-sanity-docker-devel FAILURE in 13m 48s (non-voting)
❌ ansible-test-sanity-docker-milestone FAILURE in 25m 46s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 15m 20s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 28m 38s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 7m 53s
✔️ ansible-test-units-kubernetes-core-python38 SUCCESS in 6m 13s
✔️ ansible-test-splitter SUCCESS in 2m 23s
❌ integration-kubernetes.core-devel-1 FAILURE in 34m 42s (non-voting)
✔️ integration-kubernetes.core-devel-2 SUCCESS in 37m 45s (non-voting)
⚠️ integration-kubernetes.core-devel-3 SKIPPED (non-voting)
✔️ integration-kubernetes.core-milestone-1 SUCCESS in 35m 46s
✔️ integration-kubernetes.core-milestone-2 SUCCESS in 36m 45s
⚠️ integration-kubernetes.core-milestone-3 SKIPPED
✔️ integration-kubernetes.core-2.9-1 SUCCESS in 34m 34s
✔️ integration-kubernetes.core-2.9-2 SUCCESS in 37m 22s
⚠️ integration-kubernetes.core-2.9-3 SKIPPED
✔️ integration-kubernetes.core-2.10-1 SUCCESS in 37m 33s
✔️ integration-kubernetes.core-2.10-2 SUCCESS in 36m 16s
⚠️ integration-kubernetes.core-2.10-3 SKIPPED
✔️ integration-kubernetes.core-with-turbo-1 SUCCESS in 34m 32s
✔️ integration-kubernetes.core-with-turbo-2 SUCCESS in 38m 30s
⚠️ integration-kubernetes.core-with-turbo-3 SKIPPED
✔️ ansible-tox-linters SUCCESS in 4m 39s
✔️ ansible-test-changelog SUCCESS in 2m 11s
✔️ ansible-galaxy-importer SUCCESS in 4m 04s

@softwarefactory-project-zuul
Copy link

Build succeeded.

✔️ build-ansible-collection SUCCESS in 3m 17s
❌ ansible-test-sanity-docker-devel FAILURE in 7m 57s (non-voting)
❌ ansible-test-sanity-docker-milestone FAILURE in 7m 29s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 10m 14s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 50s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 7m 56s
✔️ ansible-test-units-kubernetes-core-python38 SUCCESS in 6m 07s
✔️ ansible-test-splitter SUCCESS in 2m 25s
✔️ integration-kubernetes.core-devel-1 SUCCESS in 35m 34s (non-voting)
✔️ integration-kubernetes.core-devel-2 SUCCESS in 36m 42s (non-voting)
⚠️ integration-kubernetes.core-devel-3 SKIPPED (non-voting)
✔️ integration-kubernetes.core-milestone-1 SUCCESS in 35m 31s
✔️ integration-kubernetes.core-milestone-2 SUCCESS in 37m 12s
⚠️ integration-kubernetes.core-milestone-3 SKIPPED
✔️ integration-kubernetes.core-2.9-1 SUCCESS in 34m 56s
✔️ integration-kubernetes.core-2.9-2 SUCCESS in 37m 28s
⚠️ integration-kubernetes.core-2.9-3 SKIPPED
✔️ integration-kubernetes.core-2.10-1 SUCCESS in 34m 52s
✔️ integration-kubernetes.core-2.10-2 SUCCESS in 36m 35s
⚠️ integration-kubernetes.core-2.10-3 SKIPPED
✔️ integration-kubernetes.core-with-turbo-1 SUCCESS in 34m 14s
✔️ integration-kubernetes.core-with-turbo-2 SUCCESS in 39m 22s
⚠️ integration-kubernetes.core-with-turbo-3 SKIPPED
✔️ ansible-tox-linters SUCCESS in 5m 02s
✔️ ansible-test-changelog SUCCESS in 2m 10s
✔️ ansible-galaxy-importer SUCCESS in 4m 00s

@abikouo abikouo added mergeit and removed mergeit labels Dec 13, 2022
@abikouo abikouo closed this Dec 14, 2022
@abikouo abikouo reopened this Dec 14, 2022
@softwarefactory-project-zuul
Copy link

Build succeeded.

✔️ build-ansible-collection SUCCESS in 3m 18s
❌ ansible-test-sanity-docker-devel FAILURE in 10m 17s (non-voting)
❌ ansible-test-sanity-docker-milestone FAILURE in 7m 41s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 11m 09s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 08s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 8m 22s
✔️ ansible-test-units-kubernetes-core-python38 SUCCESS in 5m 48s
✔️ ansible-test-splitter SUCCESS in 2m 34s
✔️ integration-kubernetes.core-devel-1 SUCCESS in 34m 14s (non-voting)
✔️ integration-kubernetes.core-devel-2 SUCCESS in 38m 00s (non-voting)
⚠️ integration-kubernetes.core-devel-3 SKIPPED (non-voting)
✔️ integration-kubernetes.core-milestone-1 SUCCESS in 34m 19s
✔️ integration-kubernetes.core-milestone-2 SUCCESS in 37m 23s
⚠️ integration-kubernetes.core-milestone-3 SKIPPED
✔️ integration-kubernetes.core-2.9-1 SUCCESS in 34m 53s
✔️ integration-kubernetes.core-2.9-2 SUCCESS in 37m 35s
⚠️ integration-kubernetes.core-2.9-3 SKIPPED
✔️ integration-kubernetes.core-2.10-1 SUCCESS in 35m 24s
✔️ integration-kubernetes.core-2.10-2 SUCCESS in 36m 20s
⚠️ integration-kubernetes.core-2.10-3 SKIPPED
✔️ integration-kubernetes.core-with-turbo-1 SUCCESS in 34m 19s
✔️ integration-kubernetes.core-with-turbo-2 SUCCESS in 37m 56s
⚠️ integration-kubernetes.core-with-turbo-3 SKIPPED
✔️ ansible-tox-linters SUCCESS in 4m 43s
✔️ ansible-test-changelog SUCCESS in 2m 11s
✔️ ansible-galaxy-importer SUCCESS in 3m 54s

@abikouo abikouo added mergeit and removed mergeit labels Dec 14, 2022
@softwarefactory-project-zuul
Copy link

Build succeeded (gate pipeline).

✔️ build-ansible-collection SUCCESS in 3m 13s
❌ ansible-test-sanity-docker-devel FAILURE in 7m 35s (non-voting)
❌ ansible-test-sanity-docker-milestone FAILURE in 7m 46s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 10m 12s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 11m 32s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 8m 44s
✔️ ansible-test-units-kubernetes-core-python38 SUCCESS in 7m 11s
✔️ ansible-test-splitter SUCCESS in 2m 27s
✔️ integration-kubernetes.core-devel-1 SUCCESS in 35m 23s (non-voting)
✔️ integration-kubernetes.core-devel-2 SUCCESS in 37m 19s (non-voting)
⚠️ integration-kubernetes.core-devel-3 SKIPPED (non-voting)
✔️ integration-kubernetes.core-milestone-1 SUCCESS in 34m 40s
✔️ integration-kubernetes.core-milestone-2 SUCCESS in 37m 27s
⚠️ integration-kubernetes.core-milestone-3 SKIPPED
✔️ integration-kubernetes.core-2.9-1 SUCCESS in 39m 52s
✔️ integration-kubernetes.core-2.9-2 SUCCESS in 38m 09s
⚠️ integration-kubernetes.core-2.9-3 SKIPPED
✔️ integration-kubernetes.core-2.10-1 SUCCESS in 39m 27s
✔️ integration-kubernetes.core-2.10-2 SUCCESS in 37m 09s
⚠️ integration-kubernetes.core-2.10-3 SKIPPED
✔️ integration-kubernetes.core-with-turbo-1 SUCCESS in 35m 29s
✔️ integration-kubernetes.core-with-turbo-2 SUCCESS in 39m 40s
⚠️ integration-kubernetes.core-with-turbo-3 SKIPPED
✔️ ansible-tox-linters SUCCESS in 4m 43s
✔️ ansible-test-changelog SUCCESS in 2m 13s
✔️ ansible-galaxy-importer SUCCESS in 3m 51s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 42ee210 into ansible-collections:main Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gravesm gravesm gravesm approved these changes

Assignees
No one assigned
Labels
mergeit
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@abikouo @gravesm