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

added config option to disable concurrent builds. Fixes #2217 #2224

Merged
merged 1 commit into from
Jan 14, 2020

Conversation

t-gauss
Copy link
Contributor

@t-gauss t-gauss commented Jan 14, 2020

Signed-off-by: Tobias Gauss tobias.gauss@bosch-si.com

Signed-off-by: Tobias Gauss <tobias.gauss@bosch-si.com>
@t-gauss t-gauss force-pushed the bug/issue-2217_throttle_builds branch from 5f37193 to b372e44 Compare January 14, 2020 05:51
@ghost
Copy link

ghost commented Jan 14, 2020

Here the ES tests failed, but in Travis (supposedly isolated) and due to a "different" timeout as discussed.
Had not seen this one before - hopefully just a glitch. Will be restarting this soon.

@ghost
Copy link

ghost commented Jan 14, 2020

ES tests are failing again due to inability to manipulate the temporary Allegro/ES data in the /tmp folder.
Restarting everything as I am not sure whether this was cause by builds running concurrently.

@kolotu kolotu merged commit a7fa9b3 into eclipse:development Jan 14, 2020
@ghost
Copy link

ghost commented Jan 14, 2020

Not sure whether this works yet. When I restart jenkins builds manually, they are running concurrently so that would mean the config change does not work in that context. However, there might be a chance for it to work exclusively when jenkins starts the jobs from new PR detected.
Guess we'll just have to wait and see.

@t-gauss
Copy link
Contributor Author

t-gauss commented Jan 16, 2020

Think it works only in scope of PR in jenkins. But lets try the measures we discussed.

@t-gauss t-gauss deleted the bug/issue-2217_throttle_builds branch January 16, 2020 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants