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] Unable to include folder/dir as deps in elyra pipeline #215

Open
3 tasks
harshad16 opened this issue Sep 19, 2023 · 1 comment
Open
3 tasks

[Bug] Unable to include folder/dir as deps in elyra pipeline #215

harshad16 opened this issue Sep 19, 2023 · 1 comment
Labels
feature/elyra kind/bug Something isn't working priority/normal An issue with the product; fix when possible

Comments

@harshad16
Copy link
Member

harshad16 commented Sep 19, 2023

Using elyra plugin to create a pipeline, when trying to introduce the file system as dependencies.
The inclusion of single files works as expected, however, when trying to include a folder/directory, the elyra send out an error.
Example:

Error details:
{
  "severity": 1,
  "source": "Elyra Pipeline Validation Service",
  "type": "invalidFilePath",
  "message": "Property has an invalid path to a file/dir or the file/dir does not exist.",
  "data": {
    "nodeID": "8c333f6c-f21f-4061-a00a-098cd80c0f81",
    "nodeName": "test.ipynb",
    "propertyName": "dependencies",
    "value": "/opt/app-root/src/test/test"
  }

Even though the directory exists:

Screenshot from 2023-09-19 16-21-02

This disables users' ability to include there filesystem from a directory.

Upstream issue: elyra-ai/elyra#3181

Acceptance criteria

Dev:

  • Based on docs the directory is allowed, Fix the validation here

QA:

  • Extend pipeline execution to include the file dependency kinda pipelines. here

Docs:

Known issues:

  • Include a release note about this issue with a workaround solution.

Resolved: None

@harshad16
Copy link
Member Author

Workaround:
Clone the repo (if on git) or download the folder on runtime in the pipeline to use it.

@harshad16 harshad16 added the kind/bug Something isn't working label Sep 26, 2023
@harshad16 harshad16 moved this from 📋 Backlog to 🔖Todo in ODH IDE Planning Oct 17, 2023
@harshad16 harshad16 added the priority/normal An issue with the product; fix when possible label Oct 17, 2023
@harshad16 harshad16 changed the title Unable to include folder/dir as deps in elyra pipeline [Bug] Unable to include folder/dir as deps in elyra pipeline Oct 17, 2023
harshad16 added a commit to harshad16/odh-notebooks that referenced this issue Apr 11, 2024
[release-2024a] sync release 2024a branch with change from upstream 2024a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/elyra kind/bug Something isn't working priority/normal An issue with the product; fix when possible
Projects
Status: No status
Status: No status
Status: 🔖Todo
Development

No branches or pull requests

1 participant