Skip to content
This repository has been archived by the owner on Jul 23, 2020. It is now read-only.

PR builds on boosters do not pass #763

Closed
sbose78 opened this issue Sep 6, 2017 · 11 comments
Closed

PR builds on boosters do not pass #763

sbose78 opened this issue Sep 6, 2017 · 11 comments

Comments

@sbose78
Copy link
Collaborator

sbose78 commented Sep 6, 2017

https://gist.github.com/anonymous/7c1c32ad5e8b8ca1ba398074575f2fd9

@rawlingsj
Copy link

that looks like the integration test.. is that right? If so can you check the resources in the environment that it is running in, I think now that should be your users home namespace.

The error looks like the pod didn't start in time during the integration test but I could be wrong. A bit more info is needed.

@sbose78
Copy link
Collaborator Author

sbose78 commented Sep 6, 2017

Thanks @rawlingsj for looking into this -
In https://console.starter-us-east-2.openshift.com/console/project/shbose/quota
Resource usage is at 50% for cpu and memory, with 2 out of 5 services.

You are correct, that's the integration test:

-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Picked up _JAVA_OPTIONS: -Duser.home=/root/ -XX:+UseParallelGC                                 -XX:MinHeapFreeRatio=20                                 -XX:MaxHeapFreeRatio=40                                 -XX:GCTimeRatio=4                                 -XX:AdaptiveSizePolicyWeight=90                                 -Xmx256m
Running io.openshift.booster.OpenShiftIT
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
application deployed, 3 object(s) created.
Route url: http://booster-rest-http-vertx-shbose.8a09.starter-us-east-2.openshiftapps.com
Deleting application : DeploymentConfig
Tests run: 2, Failures: 1, Errors: 1, Skipped: 0, Time elapsed: 306.268 sec <<< FAILURE! - in io.openshift.booster.OpenShiftIT
testThatWeAreReady(io.openshift.booster.OpenShiftIT)  Time elapsed: 300.032 sec  <<< ERROR!
com.jayway.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.openshift.booster.test.OpenShiftTestAssistant was not fulfilled within 5 minutes.
	at com.jayway.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:122)
	at com.jayway.awaitility.core.CallableCondition.await(CallableCondition.java:79)
	at com.jayway.awaitility.core.CallableCondition.await(CallableCondition.java:27)
	at com.jayway.awaitility.core.ConditionFactory.until(ConditionFactory.java:764)
	at com.jayway.awaitility.core.ConditionFactory.until(ConditionFactory.java:741)
	at io.openshift.booster.test.OpenShiftTestAssistant.awaitApplicationReadinessOrFail(OpenShiftTestAssistant.java:141)
	at io.openshift.booster.OpenShiftIT.testThatWeAreReady(OpenShiftIT.java:39)

testThatWeServeAsExpected(io.openshift.booster.OpenShiftIT)  Time elapsed: 2.838 sec  <<< FAILURE!

@bartoszmajsak
Copy link
Contributor

bartoszmajsak commented Feb 15, 2018

This might be fixed with fabric8io/fabric8-pipeline-library#381

@sbose78 give it a spin when merged or use @Library('github.com/bartoszmajsak/fabric8-pipeline-library@cube-properties') as that's what this PR brings.

@piyush-garg
Copy link
Collaborator

@sbose78 Is this still happening? Can you confirm please?

@hrishin
Copy link

hrishin commented Jul 6, 2018

PR build is failing for Integration Tests

  1. vertex app build logs:
    https://pastebin.com/N2X5YMEF

  2. Springboot app build logs:
    https://pastebin.com/sD7TPwfZ

@hrishin hrishin changed the title PR builds on booster-rest-http-vertx doesn't pass PR builds on boosters doesn't pass Jul 6, 2018
@sthaha sthaha changed the title PR builds on boosters doesn't pass PR builds on boosters do not pass Jul 9, 2018
@sthaha sthaha assigned sthaha and unassigned sthaha Jul 24, 2018
@piyush-garg piyush-garg self-assigned this Jul 30, 2018
@piyush-garg
Copy link
Collaborator

@sthaha This is an issue which is reproducible in the current state also.

@piyush-garg piyush-garg removed their assignment Jul 30, 2018
@bartoszmajsak
Copy link
Contributor

Another issue leading to that problem (reproducible if you have Configmap booster imported) #3134 (comment)

@bartoszmajsak
Copy link
Contributor

bartoszmajsak commented Aug 1, 2018

I managed to fix my issue and I see that reports from @hrishin are showing the same symptoms #763 (comment). I'm happy to provide a PR to the pipeline library based on the outcome of the discussion here #3134 (comment)

hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 13, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 13, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 13, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 14, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 14, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
This patch adds `runTestsInUserNamespace` to use username namespace as defualt default environment to run
Integration Tests.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 14, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
This patch adds `runTestsInUserNamespace` to use username namespace as defualt default environment to run
Integration Tests.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 14, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
This patch adds `runTestsInUserNamespace` to use username namespace as defualt default environment to run
Integration Tests.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 14, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
This patch adds `runTestsInUserNamespace` to use username namespace as defualt default environment to run
Integration Tests.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 14, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
This patch adds `runTestsInUserNamespace` to use username namespace as defualt default environment to run
Integration Tests.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 14, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
This patch adds `runTestsInUserNamespace` to use username namespace as defualt default environment to run
Integration Tests.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 14, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
This patch adds `runTestsInUserNamespace` to use username namespace as defualt default environment to run
Integration Tests.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 14, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
This patch adds `runTestsInUserNamespace` to use username namespace as defualt default environment to run
Integration Tests.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 16, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
It has added a util method to retrive default test namespace.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 16, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
It has added a util method to retrive default test namespace.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-pipeline-library that referenced this issue Aug 16, 2018
This patch allows the `mavenCI{}` to over maven command in order
to allow `mavenIntegrationTes{}` to pass specific command to execute.
It has added a util method to retrive default test namespace.

Fixes: openshiftio/openshift.io#763
Fixes: openshiftio/openshift.io#3134
Related to: fabric8io#412
hrishin added a commit to hrishin/fabric8-jenkinsfile-library that referenced this issue Aug 20, 2018
Maven integration tests are failing because it tries to create a
test namespace (project) on openshift, hence call to openshift API gets
fail due to permission issue.
This patch fixes the `mavenCI` step by inputing maven command to run
integration test along with valid test namespace.

Resolves:
 - openshiftio/openshift.io#3134
 - openshiftio/openshift.io#763
@hrishin hrishin self-assigned this Aug 21, 2018
@hrishin hrishin reopened this Aug 24, 2018
@piyush-garg
Copy link
Collaborator

Fixed with fabric8io/fabric8-jenkinsfile-library#37

@bartoszmajsak
Copy link
Contributor

@piyush1594 did we run PR builds to verify that? Can you point me to the booster repos?

@piyush-garg
Copy link
Collaborator

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

9 participants