devfile / api

Kube-native API for cloud development workspaces specification
Apache License 2.0
263 stars 63 forks source link

Non terminating image check does not support parent in devfiles #1568

Open thepetk opened 5 months ago

thepetk commented 5 months ago

/kind bug

Which area is this bug related to?

/area ci /area registry

Bug Summary

The check_non_terminating script is failing in case a registry stack is using a parent devfile.

Describe the bug:

As the devfile is not flattened before checking for non terminating images, the script is unable to find any components.

To Reproduce:

Create a registry stack that has only a metadata & parent section. An example is here: https://github.com/devfile/registry/actions/runs/9259636857/job/25472255936?pr=401

Expected behavior

The check should be passing and the devfile should be flattened during the execution of the script with all the parent components.

Any logs, error output, screenshots etc? Provide the devfile that sees this bug, if applicable

Logs can be found here: https://github.com/devfile/registry/actions/runs/9259636857/job/25472255936?pr=401

Testing /home/runner/work/registry/registry/stacks/ollama/devfile.yaml
  COMMAND: minikube ssh docker pull null

Additional context

Useful context is here: https://github.com/devfile/registry/pull/401#issuecomment-2134023630

Any workaround?

N/A

Suggestion on how to fix the bug

N/A

maysunfaisal commented 5 months ago

One solution would be to invoking the devfile/library cmd and flattening the devfile before examining it. The devfile/library module might not have the flattening available via their main(), so we will need to update that main() a bit.

AObuchow commented 5 months ago

One solution would be to invoking the devfile/library cmd and flattening the devfile before examining it. The devfile/library module might not have the flattening available via their main(), so we will need to update that main() a bit.

+1 I think exposing this functionality might also be useful for future cases where we need to obtain the flattened version of a devfile easily

github-actions[bot] commented 4 days ago

This issue is stale because it has been open for 90 days with no activity. Remove stale label or comment or this will be closed in 60 days.