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

Mount input-artifacts into initContainers #11519

Open
djmcgreal-cc opened this issue Aug 4, 2023 · 0 comments
Open

Mount input-artifacts into initContainers #11519

djmcgreal-cc opened this issue Aug 4, 2023 · 0 comments
Labels
area/artifacts S3/GCP/OSS/Git/HDFS etc solution/workaround There's a workaround, might not be great, but exists type/feature Feature request

Comments

@djmcgreal-cc
Copy link

djmcgreal-cc commented Aug 4, 2023

Summary

Currently initContainers can't access input: artifacts.

Workaround:

    podSpecPatch: | #can't specify it directly because then it fails validation
      initContainers:
        - name: summation
          volumeMounts:
            - mountPath: /data
              name: input-artifacts #have to manually add the data directory to initContainers, unfortunately - hope Argo don't change their internals on this
              subPath: data

Given the structure of the Workflow yaml, I'd naturally assumed that the input-artifacts would be mounted into initContainers also.

Use Cases

Due to a lack of a when clause on output artifacts I have a initContainer doing the real work which needs the artifacts and a container whose code conditionally copies them to a different file path and then making the artifacts optional.

I also commonly use an initContainer pattern as storage is difficult in EKS where we can't assume Workflow Pods will be scheduled in the same availability zone to use EBS backed PVCs.


Message from the maintainers:

Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

@djmcgreal-cc djmcgreal-cc added the type/feature Feature request label Aug 4, 2023
@agilgur5 agilgur5 added the area/artifacts S3/GCP/OSS/Git/HDFS etc label Aug 10, 2023
@agilgur5 agilgur5 added the solution/workaround There's a workaround, might not be great, but exists label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/artifacts S3/GCP/OSS/Git/HDFS etc solution/workaround There's a workaround, might not be great, but exists type/feature Feature request
Projects
None yet
Development

No branches or pull requests

2 participants