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

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

Open
thepetk opened this issue May 28, 2024 · 5 comments
Open

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

thepetk opened this issue May 28, 2024 · 5 comments
Assignees
Labels
area/ci area/registry Devfile registry for stacks and infrastructure kind/bug Something isn't working

Comments

@thepetk
Copy link
Contributor

thepetk commented May 28, 2024

/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: devfile/registry#401 (comment)

Any workaround?

N/A

Suggestion on how to fix the bug

N/A

@openshift-ci openshift-ci bot added kind/bug Something isn't working area/ci area/registry Devfile registry for stacks and infrastructure labels May 28, 2024
@thepetk thepetk changed the title Non terminating image doe Non terminating image check does not support parent in devfiles May 28, 2024
@thepetk thepetk moved this to Refinement in Devfile Project May 28, 2024
@maysunfaisal
Copy link
Member

maysunfaisal commented May 29, 2024

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.

@maysunfaisal maysunfaisal moved this from Refinement to Backlog in Devfile Project May 29, 2024
@AObuchow
Copy link

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

Copy link

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.

@github-actions github-actions bot added the lifecycle/stale Stale items. These items have not been updated for 90 days. label Nov 14, 2024
@ibuziuk
Copy link

ibuziuk commented Jan 9, 2025

@thepetk hi, any ETA for this ?

@github-actions github-actions bot removed the lifecycle/stale Stale items. These items have not been updated for 90 days. label Jan 10, 2025
@thepetk
Copy link
Contributor Author

thepetk commented Jan 10, 2025

@thepetk hi, any ETA for this ?

Hi @ibuziuk ! The truth is it was left behind due to other priorities. I'm going to include it in the next sprint, so I'm aiming to have everything set mid-late february. Let me know if that works for you.

cc @elsony

@thepetk thepetk moved this from Backlog to To Do 📝 in Devfile Project Jan 20, 2025
@thepetk thepetk self-assigned this Jan 20, 2025
@thepetk thepetk moved this from To Do 📝 to In Progress 🚧 in Devfile Project Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci area/registry Devfile registry for stacks and infrastructure kind/bug Something isn't working
Projects
Status: In Progress 🚧
Development

No branches or pull requests

4 participants