-
Notifications
You must be signed in to change notification settings - Fork 66
Creating a new Thorntail project fails - build failure due to quota being exceeded #4547
Comments
This is in production - cannot yet recreate on prod-preview as env reset results in Jenkins pod not starting. |
You seem to have run with an infamous OOM which basically mean thorntail is consuming too much memory to run on starter's OSIO restricted environment, if you can it would be nice if you can copy and paste the |
Your guess is correct - see log fragment below. Note that the first step in the test was to reset the user's OpenShift.io env - no other spaces or projects were present when the test was run:
|
@ldimaggi if a project build fails due to OOM, I guess there is nothing the build can do about it. The only way I see is to request for more resources; probably something that will be coved in OSD. So I am going to close it as "Not a Bug" |
If some booster project can’t be built on OSO then it should be removed from the wizard. This is something for the getting started team. I’m reopening the issue. |
Is removal the right thing to do? How about we try to optimize the booster itself? @slemeur any comment on that? Let's keep in mind it was working before - did the runtime version change? Booster was modified? @ldimaggi I have few questions here
|
Yes, if it was working before, it's a regression and we need to look at that. |
@bartoszmajsak Who will look at the booster. |
@GeorgeActon App Creation aka Getting Started with help of booster owner if our initial analysis will lead to a conclusion that we have to improve booster to make it running in our env |
@sbose78 @tinakurian All indications at this point are to look at the Booster. Please sync with @chmouel and figure next steps. |
Yes - this is a regression. It is 100% reproducible. And it blocks the user from creating the booster. As there is no workaround I think that it is a SEV1. Thx! |
I think the first thing would be to look at the booster and see if there was any update for the last couple of weeks and track down the regression between the versions, I am not sure what is the girhub repo but assuming is this one there was a version update thorntail-examples/health-check@6291567 |
It fails for either REST API and for Health check. tried both this morning. |
This is how boosters are mapped:
|
I agree @bartoszmajsak . @corinnekrych , let's take a look at possibly fixing the booster. Btw, If that doesn't happen soon enough, we should temporarily take the booster out and then put it back only after its fixed. You OK with that, @slemeur ? |
Yes that would be fine with me @sbose78 |
+1 to removing the booster until it works - anything visible should be working. |
Let's start with disabling it see PR. |
It's enough. @gastaldi will instantly look at it anyway. He's like a bot ;) |
FYI: Figuring out why something which was working is now breaking, and not owned by us, will take a while. The sev1 might take longer to resolve. |
* Disabled failing booster in osio as per openshiftio/openshift.io#4547 * Update thorntail/community/health-check/booster.yaml Co-Authored-By: corinnekrych <corinnekrych@gmail.com>
We should get this booster only available when building on a OSD cluster. Currently it's failing due of resources limitation for OSD, it may be possible to fit it to run on OSO but the users would never being able to use it :( This may get better tho when we move over the new osio-pipeline library, since we will not be building anymore in the @slemeur i think it's something that should be discussed |
@ldimaggi I've tested v11 (last updated 24th July) and v12 (last updated 16th October) of the thorntail health-check app and both fails on my prod-preview => which confirm that this is not a regression in booster themselves but indeed as @chmouel mention in the ticket an issue with build. @shoubhikbose I'll remove my name from the ticket as I think this ticket will be better handle in build team. Let me know if you need more help on my side. |
Thanks for the investigation @corinnekrych and @chmouel .
I'll give this a shot. |
Thanks @sbose78 this would be a nice use case for osio-pipeline, |
Both the health-check and rest-http boosters work with the new pipeline library since the build happens in the user namespace. Example: Once we finish migrating our Java boosters to use the new pipeline library, then we can update the @rupalibehera let's work together on getting the migration done. |
#4535 would need to be done before we can start using the new pipeline for java projects. |
Raised #4535 as P1 since we are depending on this to fix this issue, |
Issue Overview
Creating a new project with the Thorntail runtime fails with build error.
Expected Behaviour
It should be possible for users to create new projects configured with the Thorntail runtime.
Current Behaviour
See build log fragment below for error.
Steps To Reproduce
Additional Information
The text was updated successfully, but these errors were encountered: