Skip to content

Commit

Permalink
test: enable kubernetes tests for python-language-pdf2parquet
Browse files Browse the repository at this point in the history
also updates to latest from data-prep-kit

Signed-off-by: Nick Mitchell <[email protected]>
  • Loading branch information
starpit committed Nov 19, 2024
1 parent bbbbedb commit 53ae115
Show file tree
Hide file tree
Showing 8 changed files with 217 additions and 88 deletions.
4 changes: 3 additions & 1 deletion pkg/be/local/shell/ok.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,9 @@ func isCompatibleImage(image string) bool {
func IsCompatible(c llir.ShellComponent) error {
switch {
case c.Application.Spec.Image != "" && !isCompatibleImage(c.Application.Spec.Image):
return fmt.Errorf("Unable to target the local backend because a component '%s' needs to run in a container: %s", c.C(), c.Application.Spec.Image)
if slices.IndexFunc(c.Application.Spec.Needs, func(need hlir.Needs) bool { return need.Requirements != "" }) < 0 {
return fmt.Errorf("Unable to target the local backend because a component '%s' needs to run in a container: %s", c.C(), c.Application.Spec.Image)
}

case c.Application.Spec.SecurityContext.RunAsUser != 0 ||
c.Application.Spec.SecurityContext.RunAsGroup != 0 ||
Expand Down
1 change: 1 addition & 0 deletions tests/tests/python-language-pdf2parquet/pail/app.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ metadata:
spec:
image: docker.io/python:3.12
command: python3 ./main.py
minSize: sm
needs:
- name: python
version: latest
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
docling-core==1.3.0
docling-ibm-models==1.1.7
deepsearch-glm==0.21.0
docling==1.11.0
docling-core==2.3.0
docling-ibm-models==2.0.3
deepsearch-glm==0.26.1
docling==2.3.1
filetype >=1.2.0, <2.0.0

# we can probably update to 18+, but we will have to re-generate expected output as pyarrow 18 seems to have resulted in a binary format change
Expand Down
Loading

0 comments on commit 53ae115

Please sign in to comment.