manifest_to_docker_images fails as path is not splittable

[roytev]

python-BaseException Traceback (most recent call last): File "/artifactory-cleanup/artifactory_cleanup/artifactorycleanup.py", line 61, in cleanup artifacts_to_remove = policy.filter(artifacts) ^^^^^^^^^^^^^^^^^^^^^^^^ File "/artifactory-cleanup/artifactory_cleanup/rules/base.py", line 286, in filter artifacts = rule.filter(artifacts) ^^^^^^^^^^^^^^^^^^^^^^ File "/artifactory-cleanup/artifactory_cleanup/rules/docker.py", line 84, in filter artifacts = self._manifest_to_docker_images(artifacts) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/artifactory-cleanup/artifactory_cleanup/rules/docker.py", line 49, in _manifest_to_docker_images artifact["path"], docker_tag = artifact["path"].rsplit("/", 1) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ValueError: not enough values to unpack (expected 2, got 1)

[jheinitz]

Same issue here. any chance to get the PR merged?

[allburov]

Sure, after one adds fallback logic!

https://github.com/devopshq/artifactory-cleanup/pull/140/files#r1543990473

idk why the path doesn't have repo/docker format in some cases, but let's add a fallback:

1. Try to split
2. If # of elements == 1 - use the new logic