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

overlays builds pipeline fails when using Issue Deliver Container Image #115

Open
pacospace opened this issue Jun 22, 2021 · 12 comments
Open
Labels
human_intervention_required kind/bug Categorizes issue or PR as related to a bug. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.

Comments

@pacospace
Copy link

Describe the bug
Overlays build pipeline fails when run from issue pipeline.

To Reproduce
Steps to reproduce the behavior:

  1. Open Issue with Deliver Container Image in Elyra tutorial and see pipeline failing.

Expected behavior
Pipeline successfully delivers all overlays builds.

Screenshots

Additional context

@pacospace
Copy link
Author

I did some debugging, and it seems base image is not passed to later step. In particular when opening issue for Delivering Container Image the first step to handle configuration does not consider overlays, therefore later steps miss inputs, like base image:
Screenshot from 2021-06-22 11-31-40

We can see error in generate task:

+ '[' -z '' ']'
+ '[' -f Dockerfile ']'
+ '[' -f Containerfile ']'
+ /usr/local/bin/s2i build . --env THAMOS_RUNTIME_ENVIRONMENT= --env THOTH_ADVISE=0 --env THOTH_ERROR_FALLBACK=1 --env THOTH_DRY_RUN=1 --env THAMOS_DEBUG=0 --env THAMOS_VERBOSE=1 --env THOTH_PROVENANCE_CHECK=0 --loglevel=0 --as-dockerfile /gen-source/Containerfile
ERROR: Required value not specified for "builderImage"

and step later fails to give a misleading error, because in reality the ContainerFile has not been created by the previous task

+ buildah bud --storage-driver=overlay --tls-verify=false --layers -f /gen-source/Containerfile -t elyra-aidevsecops-tutorial-166 .
stat /gen-source/Containerfile: no such file or directory

@goern
Copy link
Member

goern commented Jun 23, 2021

/kind bug
/priority critical-urgent
/assign @harshad16

@sesheta sesheta added kind/bug Categorizes issue or PR as related to a bug. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. labels Jun 23, 2021
@harshad16
Copy link
Member

FIxed it with b4f842a
closing it, please reopen if the issue persists.

@pacospace
Copy link
Author

@harshad16 does it solve also the overlays build issue when using "Deliver Container Image" issue?

@harshad16
Copy link
Member

no it doesnt , let me re-open this.

@harshad16 harshad16 reopened this Jun 23, 2021
@goern
Copy link
Member

goern commented Jun 29, 2021

Blocking: thoth-station/ps-cv#2
Blocking: thoth-station/ps-nlp#3

@pacospace
Copy link
Author

Blocking: thoth-station/ps-cv#2
Blocking: thoth-station/ps-nlp#3

Not blocking those, this issue is when someone uses issue Deliver Container Image with overlays builds

@goern
Copy link
Member

goern commented Jun 29, 2021

ACK, thx.

@goern
Copy link
Member

goern commented Sep 15, 2021

is this still valid?

@pacospace pacospace changed the title overlays builds pipeline fails overlays builds pipeline fails when using Issue Deliver Container Image Sep 15, 2021
@sesheta
Copy link
Contributor

sesheta commented Dec 14, 2021

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@sesheta sesheta added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Dec 14, 2021
@sesheta
Copy link
Contributor

sesheta commented Jan 13, 2022

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

/lifecycle rotten

@sesheta sesheta added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Jan 13, 2022
@harshad16
Copy link
Member

yes this is still valid
will get this on lifecycle soon
/lifecycle frozen

@sesheta sesheta added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. labels Jan 14, 2022
@harshad16 harshad16 removed their assignment Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
human_intervention_required kind/bug Categorizes issue or PR as related to a bug. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.
Projects
None yet
Development

No branches or pull requests

4 participants