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.

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

Bug: pod for downloading archive datasets does not support arm64 architecture #367

Closed
AlessandroPomponio opened this issue Jul 12, 2024 · 3 comments · Fixed by #376
Closed
Assignees
Labels
bug Something isn't working

Comments

@AlessandroPomponio
Copy link
Collaborator

What happened: After creating a dataset of type ARCHIVE, I see no content in the pod that mounts the dataset. This is because the download pod fails with

exec /bin/sh: exec format error

What you expected to happen: everything should work

How to reproduce it (as minimally and precisely as possible):

  1. Run
    kubectl apply -n dlf -f https://github.com/AlessandroPomponio/datashim/blob/ap_365_readme_updates/examples/minio/minio.yaml
  2. Run
    cat <<EOF | kubectl apply -f -
    apiVersion: v1
    kind: Pod
    metadata:
      name: nginx
      labels:
        dataset.0.id: "archive-dataset"
        dataset.0.useas: "mount"
    spec:
      containers:
        - name: nginx
          image: nginx
    EOF
  3. Run
    kubectl exec nginx -- ls mnt/datasets/archive-dataset/

Anything else we need to know?:

Environment:

  • Datashim version:0.4.0
  • Kubernetes version (use kubectl version):
  • Kubernetes distribution:
    Client Version: v1.30.0 Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3 Server Version: v1.30.0
  • Cloud provider or hardware configuration: Macbook M1, using KIND
  • Install tools: helm
@AlessandroPomponio AlessandroPomponio added the bug Something isn't working label Jul 12, 2024
@srikumar003
Copy link
Collaborator

srikumar003 commented Jul 30, 2024

The function here:

func getPodDataDownload(dataset *datasets.Dataset, operatorNamespace string) (*batchv1.Job, string) {

references a docker image that is not built with multi-arch support. The Dockerfile for that image is here:
RUN yum update -y && yum install wget tar gzip -y

To fix this issue:

  • Update the image building scripts to include the docker image for archive
  • Upload the archive docker image to quay.io/datashim-io

@AlessandroPomponio AlessandroPomponio self-assigned this Jul 30, 2024
@srikumar003 srikumar003 linked a pull request Jul 30, 2024 that will close this issue
@AlessandroPomponio
Copy link
Collaborator Author

@srikumar003
Copy link
Collaborator

Closing this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants