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.

Sign up for GitHub

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

Jump to bottom

Fix add_mlmodel_lineage call #13906

Closed
pmbrull opened this issue Nov 9, 2023 · 0 comments · Fixed by #13920
Closed

Fix add_mlmodel_lineage call #13906

pmbrull opened this issue Nov 9, 2023 · 0 comments · Fixed by #13920
Assignees
@pmbrull
Labels
bug Something isn't working Ingestion P0 Highest priority

Comments

@pmbrull
Copy link
Collaborator

pmbrull commented Nov 9, 2023

Which brings

Traceback (most recent call last):
  File "examples/cicd/ci_cd.py", line 195, in <module>
    run_workflow()
  File "examples/cicd/ci_cd.py", line 190, in run_workflow
    update_openmetadata(raw_meta)
  File "examples/cicd/ci_cd.py", line 160, in update_openmetadata
    metadata.add_mlmodel_lineage(mlmodel)
  File "examples/cicd/.venv/lib/python3.9/site-packages/metadata/ingestion/ometa/mixins/mlmodel_mixin.py", line 79, in add_mlmodel_lineage
    AddLineageRequest(
  File "pydantic/main.py", line 341, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for AddLineageRequest
description
  extra fields not permitted (type=value_error.extra)

after the changes on the AddLineageRequest class
@pmbrull pmbrull self-assigned this Nov 9, 2023
@pmbrull pmbrull added bug Something isn't working Ingestion P0 Highest priority labels Nov 9, 2023
pmbrull added a commit to pmbrull/OpenMetadata that referenced this issue Nov 9, 2023
@pmbrull
Fix open-metadata#13906 - Fix add_mlmodel_lineage description field
3b1b9e6
@pmbrull pmbrull mentioned this issue Nov 9, 2023
Merged
9 tasks
pmbrull added a commit to pmbrull/OpenMetadata that referenced this issue Nov 9, 2023
@pmbrull
Fix open-metadata#13906 - Fix add_mlmodel_lineage description field
3ba4399
pmbrull added a commit that referenced this issue Nov 10, 2023
@pmbrull
Fix #13699 - Add separator for Storage Container manifest (#13924)
d4f25d2 
* Fix #13699 - Add separator for Storage Container manifest

* Fix #13906 - Fix add_mlmodel_lineage description field

* Add separator

* Add separator
ayush-shah pushed a commit that referenced this issue Nov 10, 2023
@pmbrull
Fix #13906 - Fix add_mlmodel_lineage description field (#13920)
8891a9a
pmbrull added a commit that referenced this issue Nov 10, 2023
@pmbrull
Fix #13906 - Fix add_mlmodel_lineage description field (#13920)
2bdbe44
ulixius9 pushed a commit that referenced this issue Nov 10, 2023
@pmbrull @ulixius9
Fix #13699 - Add separator for Storage Container manifest (#13924)
8c6ba93 
* Fix #13699 - Add separator for Storage Container manifest

* Fix #13906 - Fix add_mlmodel_lineage description field

* Add separator

* Add separator
ulixius9 pushed a commit that referenced this issue Nov 10, 2023
@pmbrull @ulixius9
Fix #13906 - Fix add_mlmodel_lineage description field (#13920)
565f96c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pmbrull pmbrull

Labels
bug Something isn't working Ingestion P0 Highest priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #13906 - Fix add_mlmodel_lineage description field pmbrull/OpenMetadata
1 participant
@pmbrull