Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

[KOGITO 2723] Post-release updates #439

Merged
merged 1 commit into from
Jul 15, 2020
Merged

Conversation

radtriste
Copy link
Contributor

@radtriste radtriste commented Jul 9, 2020

https://issues.redhat.com/browse/KOGITO-2723

After testing of 0.12.0, there were some small adjustments on deploy pipeline.

Many thanks for submiting your Pull Request ❤️!

Please make sure that your PR meets the following requirements:

  • You have read the contributors guide
  • Pull Request title is properly formatted: [KOGITO-XYZ] Subject
  • Pull Request contains link to the JIRA issue
  • Pull Request contains description of the issue
  • Pull Request does not include fixes for issues other than the main ticket
  • Your feature/bug fix has a unit test that verifies it
  • You've tested the new feature/bug fix in an actual OpenShift cluster

@radtriste radtriste added the WIP 👷‍♂️ Pull Request is on work in progress label Jul 9, 2020
@radtriste radtriste added this to the v0.13.0 milestone Jul 9, 2020
@codecov
Copy link

codecov bot commented Jul 9, 2020

Codecov Report

Merging #439 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #439   +/-   ##
=======================================
  Coverage   43.67%   43.67%           
=======================================
  Files         144      144           
  Lines        8691     8691           
=======================================
  Hits         3796     3796           
  Misses       4469     4469           
  Partials      426      426           
Flag Coverage Δ
#cli 68.08% <ø> (ø)
#operator 38.29% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a787e8...096f1b8. Read the comment docs.

@@ -131,7 +131,7 @@ pipeline {
}
steps {
script {
sh "make run-tests timeout=360 load_factor=3 concurrent=6 image_cache_mode=always smoke=${params.SMOKE_TESTS_ONLY} ${getBDDParameters()}"
sh "make run-tests timeout=360 load_factor=3 concurrent=3 image_cache_mode=always smoke=${params.SMOKE_TESTS_ONLY} ${getBDDParameters()}"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Set back to 3 for now. Anyway with scenario concurrency, it should already be a bit faster

@radtriste radtriste force-pushed the kogito-2723 branch 2 times, most recently from a734e0a to 4980f29 Compare July 9, 2020 15:13
Examples:
| native | minutes |
| enabled | 20 |
| native | minutes | build-request-cpu | build-limit-cpu | build-request-memory |
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like the native with persistence and events is resources demanding. I put the same values as for onboarding

@radtriste radtriste force-pushed the kogito-2723 branch 2 times, most recently from 7d32484 to 34e5eb3 Compare July 10, 2020 08:32
@radtriste radtriste marked this pull request as ready for review July 10, 2020 09:57
@radtriste radtriste requested a review from sutaakar July 10, 2020 09:57
@radtriste radtriste added bdd-tests 🧪 PR is related to the BDD test framework needs review 🔍 Pull Request that needs reviewers and removed WIP 👷‍♂️ Pull Request is on work in progress labels Jul 10, 2020
@radtriste radtriste added ready 🚀 PR is ready to be merged and removed needs review 🔍 Pull Request that needs reviewers labels Jul 15, 2020
@radtriste radtriste merged commit 817ce40 into apache:master Jul 15, 2020
@radtriste radtriste deleted the kogito-2723 branch July 15, 2020 07:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bdd-tests 🧪 PR is related to the BDD test framework ready 🚀 PR is ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants