container_import doesn't work with compressed tarballs

[pauldraper]

🐞 bug report

Affected Rule

container_import

Is this a regression?

No.

Description

Contrary to the documentation, container_import doesn't work with compressed tarballs.

🔬 Minimal Reproduction

Use a .tar.gz for container_load.

🔥 Exception or Error

Failed to load tarball into OCI format: failed to read docker image tarball from ... archive/tar: invalid tar header

🌍 Your Environment

Operating System:

Output of bazel version:

Rules_docker version:

Anything else relevant?

[marczych-zoox]

I'm seeing the same behavior with container_load whose docs also indicate that it should work with a compressed tarball.

[pjjw]

piling on with a +1.

[github-actions[bot]]

This issue has been automatically marked as stale because it has not had any activity for 180 days. It will be closed if no further activity occurs in 30 days. Collaborators can add an assignee to keep this open indefinitely. Thanks for your contributions to rules_docker!

[github-actions[bot]]

This issue was automatically closed because it went 30 days without a reply since it was labeled "Can Close?"

[marczych-zoox]

Can we reopen this?

[pjjw]

fwiw we worked around this by tarring the docker-generated tarball (so a tar in a tar), gzipping that, and then using http_archive to get a repository with the importable tarball.

[marczych-zoox]

Yeah, we did the same thing but I was hoping for a less hacky workaround. If nothing else, the documentation is out of date.