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

CDK itests stabilization on jenkins #1783

Merged
merged 1 commit into from Aug 2, 2018

Conversation

odockal
Copy link
Member

@odockal odockal commented Jul 25, 2018

Signed-off-by: Ondrej Dockal odockal@redhat.com

Pull Request Checklist

General

  • Is this a blocking issue or new feature? If yes, QE needs to +1 this PR

Code

  • Are method-/class-/variable-names meaningful?
  • Are methods concise, not too long?
  • Are catch blocks catching precise Exceptions only (no catch all)?

Testing

  • Are there unit-tests?
  • Are there integration tests (or at least a jira to tackle these)?
  • Is the non-happy path working, too?
  • Are other parts that use the same component still working fine?

Function

  • Does it work?

@odockal odockal force-pushed the cdk-stabilization branch 2 times, most recently from 4c3fefa to 46f664b Compare July 25, 2018 13:21
@@ -233,12 +237,12 @@ protected void startServerAdapter(Runnable cond, boolean rethrow) {
protected void startServerAdapter(Server server, Runnable cond, boolean rethrow) {
log.info("Starting server adapter");
// Workaround for CDK-216
new WaitUntil(new NeverFulfilledCondition(), TimePeriod.getCustom(120), false);
// new WaitUntil(new NeverFulfilledCondition(), TimePeriod.getCustom(120), false);
Copy link
Member

Choose a reason for hiding this comment

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

@odockal can it be deleted? CDK-216 is closed.

Copy link
Member Author

Choose a reason for hiding this comment

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

Probably yes!

Signed-off-by: Ondrej Dockal <odockal@redhat.com>
@jkopriva jkopriva merged commit d7f2bcc into jbosstools:master Aug 2, 2018
adietish pushed a commit to bdshadow/jbosstools-openshift that referenced this pull request Aug 9, 2018
Signed-off-by: Ondrej Dockal <odockal@redhat.com>
adietish pushed a commit to bdshadow/jbosstools-openshift that referenced this pull request Aug 9, 2018
Signed-off-by: Ondrej Dockal <odockal@redhat.com>
@odockal odockal deleted the cdk-stabilization branch January 15, 2020 07:24
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