You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by GitBox <gi...@apache.org> on 2022/04/06 01:06:42 UTC
[GitHub] [storm] bipinprasad opened a new pull request, #3470: [STORM-3851] Fix syntax errors and modify to run under python3
bipinprasad opened a new pull request, #3470:
URL: https://github.com/apache/storm/pull/3470
## What is the purpose of the change
*Errors in travis build are not always printer properly*
If errors are encountered in Travis build, then script dev-tools/travis/print-errors-from-test-reports.py is used to examine the reports in target/test-reports and print the errors in test.
However, the script itself has syntax errors and therefore fails to load, especially under python3.
Fix the script so that it can properly parse and print.
## How was the change tested
*Ran this script against storm-core/target/test-reports.*
Old code gave an error:
```
(oath_tools) bprasad@C02XG0WLJGH8 storm-core % python3 ../dev-tools/travis/print-errors-from-test-reports.py target/test-reports
Traceback (most recent call last):
File "/Users/bprasad/sources/community/bipinprasad/storm/storm-core/../dev-tools/travis/print-errors-from-test-reports.py", line 77, in <module>
if sys.argv < 2:
TypeError: '<' not supported between instances of 'list' and 'int'
```
New code ran successfully and displayed error:
```
(oath_tools) bprasad@C02XG0WLJGH8 storm-core % python3 ../dev-tools/travis/print-errors-from-test-reports.py target/test-reports
Checking target/test-reports/org.apache.storm.cluster-test.xml
--------------------------------------------------
classname: org.apache.storm.cluster-test / testname: test-storm-cluster-state-errors
expected: (.contains (:error error) target)
actual: false
at: MultiFn.java:229
-------------------- system-out --------------------
5709 [main] INFO o.a.s.z.Zookeeper - Starting inprocess zookeeper at port 2000 and dir /var/folders/ln/b66p649n02v3x6xyclwndw6w0000gn/T/d1fc7bb4-72d1-4082-a73c-9d126c4c2807
5713 [main] WARN o.a.s.u.Time - AutoCloseable Simulated Time Starting...
5762 [main] INFO o.a.s.z.ClientZookeeper - Starting ZK Curator
5895 [main] INFO o.a.s.z.ClientZookeeper - Starting ZK Curator
IllegalArgumentException => java.lang.RuntimeException
at org.apache.storm.cluster_test$fn__636.invokeStatic(cluster_test.clj:274)
at org.apache.storm.cluster_test$fn__636.invoke(cluster_test.clj:270)
at clojure.test$test_var$fn__9707.invoke(test.clj:717)
at clojure.test$test_var.invokeStatic(test.clj:717)
at clojure.test$test_var.invoke(test.clj:708)
at clojure.test$test_vars$fn__9733$fn__9738.invoke(test.clj:735)
at clojure.test$default_fixture.invokeStatic(test.clj:687)
at clojure.test$default_fixture.invoke(test.clj:683)
at clojure.test$test_vars$fn__9733.invoke(test.clj:735)
at clojure.test$default_fixture.invokeStatic(test.clj:687)
at clojure.test$default_fixture.invoke(test.clj:683)
at clojure.test$test_vars.invokeStatic(test.clj:731)
at clojure.test$test_all_vars.invokeStatic(test.clj:737)
at clojure.test$test_ns.invokeStatic(test.clj:758)
at clojure.test$test_ns.invoke(test.clj:743)
at clojure.core$map$fn__5851.invoke(core.clj:2755)
at clojure.lang.LazySeq.sval(LazySeq.java:42)
at clojure.lang.LazySeq.seq(LazySeq.java:51)
at clojure.lang.Cons.next(Cons.java:39)
at clojure.lang.RT.boundedLength(RT.java:1788)
at clojure.lang.RestFn.applyTo(RestFn.java:130)
at clojure.core$apply.invokeStatic(core.clj:667)
at clojure.test$run_tests.invokeStatic(test.clj:768)
at clojure.test$run_tests.doInvoke(test.clj:768)
at clojure.lang.RestFn.invoke(RestFn.java:408)
at org.apache.storm.testrunner$eval701$iter__702__706$fn__707$fn__708$fn__709.invoke(test_runner.clj:107)
at org.apache.storm.testrunner$eval701$iter__702__706$fn__707$fn__708.invoke(test_runner.clj:53)
at org.apache.storm.testrunner$eval701$iter__702__706$fn__707.invoke(test_runner.clj:52)
at clojure.lang.LazySeq.sval(LazySeq.java:42)
at clojure.lang.LazySeq.seq(LazySeq.java:51)
at clojure.lang.RT.seq(RT.java:531)
at clojure.core$seq__5387.invokeStatic(core.clj:137)
at clojure.core$dorun.invokeStatic(core.clj:3133)
at clojure.core$dorun.invoke(core.clj:3133)
at org.apache.storm.testrunner$eval701.invokeStatic(test_runner.clj:52)
at org.apache.storm.testrunner$eval701.invoke(test_runner.clj:47)
at clojure.lang.Compiler.eval(Compiler.java:7176)
at clojure.lang.Compiler.load(Compiler.java:7635)
at clojure.lang.Compiler.loadFile(Compiler.java:7573)
at clojure.main$load_script.invokeStatic(main.clj:452)
at clojure.main$script_opt.invokeStatic(main.clj:512)
at clojure.main$script_opt.invoke(main.clj:507)
at clojure.main$main.invokeStatic(main.clj:598)
at clojure.main$main.doInvoke(main.clj:561)
at clojure.lang.RestFn.applyTo(RestFn.java:137)
at clojure.lang.Var.applyTo(Var.java:705)
at clojure.main.main(main.java:37)
--------------------------------------------------
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@storm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [storm] bipinprasad merged pull request #3470: [STORM-3851] Fix syntax errors and modify to run under python3
Posted by GitBox <gi...@apache.org>.
bipinprasad merged PR #3470:
URL: https://github.com/apache/storm/pull/3470
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@storm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org