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

https://issues.jboss.org/browse/WFCORE-145 CLI run-batch command cannot ... #250

Merged
merged 1 commit into from Nov 20, 2014

Conversation

aloubyansky
Copy link
Contributor

...process files with comments or blank lines

Thanks,
Alexey

@wildfly-ci
Copy link

Windows Build 324 is now running using a merge of ad79e36

@wildfly-ci
Copy link

Linux Build 598 is now running using a merge of ad79e36

@wildfly-ci
Copy link

Windows Build 324 outcome was SUCCESS using a merge of ad79e36
Summary: Tests passed: 2688, ignored: 56 Build time: 0:15:14

@wildfly-ci
Copy link

Linux Build 598 outcome was SUCCESS using a merge of ad79e36
Summary: Tests passed: 2688, ignored: 56 Build time: 0:17:01

@bstansberry
Copy link
Contributor

If we're adding a new script, it should have a PowerShell variant.

I wonder if these new scripts should go in bin/client instead of directly in bin. Having different scripts to run the CLI in the same place is a bit confusing.

Either way, bin/client/README-CLI-JCONSOLE.txt needs updating.

I'm not sure why the bin/client dir doesn't exist in core. Maybe just too much trouble for the feature-pack-build? The non-CLI stuff in it doesn't belong in core though.

@bstansberry bstansberry added fixme This PR introduces issues that must be fixed rebase-this This PR must be rebased before it is merged labels Nov 3, 2014
@wildfly-ci
Copy link

Windows Build 468 is now running using a merge of 74edfa9

@wildfly-ci
Copy link

Core - Full Integration Build 37 is now running using a merge of 74edfa9

@wildfly-ci
Copy link

Linux Build 744 is now running using a merge of 74edfa9

@wildfly-ci
Copy link

Windows Build 468 outcome was SUCCESS using a merge of 74edfa9
Summary: Tests passed: 2770, ignored: 58 Build time: 0:16:03

@wildfly-ci
Copy link

Linux Build 744 outcome was SUCCESS using a merge of 74edfa9
Summary: Tests passed: 2770, ignored: 58 Build time: 0:19:54

@aloubyansky
Copy link
Contributor Author

The script got there unintentionally. Sorry for me overlooking and thanks for you paying attention.

@wildfly-ci
Copy link

Core - Full Integration Build 37 outcome was FAILURE using a merge of 74edfa9
Summary: Tests failed: 1 (1 new), passed: 5392, ignored: 347 Build time: 1:00:41

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.smoke.webservices.WSTestCase.testManagementDescriptionMetrics: java.lang.AssertionError: null
    at org.junit.Assert.fail(Assert.java:86)
    at org.junit.Assert.assertTrue(Assert.java:41)

…ot process files with comments or blank lines
@wildfly-ci
Copy link

Core - Full Integration Build 86 is now running using a merge of 66190b2

@wildfly-ci
Copy link

Linux Build 796 is now running using a merge of 66190b2

@wildfly-ci
Copy link

Windows Build 516 is now running using a merge of 66190b2

@wildfly-ci
Copy link

Windows Build 516 outcome was SUCCESS using a merge of 66190b2
Summary: Tests passed: 2772, ignored: 58 Build time: 0:16:10

@wildfly-ci
Copy link

Linux Build 796 outcome was SUCCESS using a merge of 66190b2
Summary: Tests passed: 2772, ignored: 58 Build time: 0:19:36

@wildfly-ci
Copy link

Core - Full Integration Build 86 outcome was SUCCESS using a merge of 66190b2
Summary: Tests passed: 2938, ignored: 303 Build time: 0:52:26

@bstansberry bstansberry removed rebase-this This PR must be rebased before it is merged fixme This PR introduces issues that must be fixed labels Nov 17, 2014
@bstansberry
Copy link
Contributor

retest this please

@wildfly-ci
Copy link

Windows Build 560 is now running using a merge of 66190b2

@wildfly-ci
Copy link

Windows Build 560 outcome was SUCCESS using a merge of 66190b2
Summary: Tests passed: 2783, ignored: 58 Build time: 0:17:17

@wildfly-ci
Copy link

Linux Build 842 is now running using a merge of 66190b2

@wildfly-ci
Copy link

Core - Full Integration Build 141 is now running using a merge of 66190b2

@wildfly-ci
Copy link

Linux Build 842 outcome was SUCCESS using a merge of 66190b2
Summary: Tests passed: 2783, ignored: 58 Build time: 0:20:08

@wildfly-ci
Copy link

Core - Full Integration Build 141 outcome was SUCCESS using a merge of 66190b2
Summary: Tests passed: 2942, ignored: 306 Build time: 0:53:00

@bstansberry bstansberry added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Nov 20, 2014
@kabir kabir merged commit a0645aa into wildfly:master Nov 20, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
4 participants