build icon indicating copy to clipboard operation
build copied to clipboard

mismatched version of python-junit-xml and tap2junit?

Open vtjnash opened this issue 5 years ago • 8 comments

This error is nonsensical to me, unless the version of python-junit-xml being used is pretty old (pre-Oct, 2016), but I'm just reporting what I saw on CI run https://ci.nodejs.org/job/node-test-commit-linux/36408/nodes=centos7-64-gcc8/consoleText

+ tap2junit -i test.tap -o out/junit/test.xml
Traceback (most recent call last):
  File "/usr/bin/tap2junit", line 10, in <module>
    sys.exit(main())
  File "/usr/lib/python2.7/site-packages/tap2junit/__main__.py", line 51, in main
    convert(args.input, args.output)
  File "/usr/lib/python2.7/site-packages/tap2junit/__main__.py", line 42, in convert
    result = parse(input_file, data)
  File "/usr/lib/python2.7/site-packages/tap2junit/__main__.py", line 35, in parse
    junit_tests = [map_yaml_to_junit(t) for t in tap_parser.tests]
  File "/usr/lib/python2.7/site-packages/tap2junit/__main__.py", line 27, in map_yaml_to_junit
    t.add_failure_info(error_message, err_output, err_code)
TypeError: add_failure_info() takes at most 3 arguments (4 given)
POST BUILD TASK : FAILURE

vtjnash avatar Aug 04 '20 17:08 vtjnash

This issue is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made.

github-actions[bot] avatar Jun 01 '21 02:06 github-actions[bot]

bad bot

vtjnash avatar Jun 01 '21 03:06 vtjnash

Your bot is very naughty and closing active issues.

vtjnash avatar Jul 02 '21 00:07 vtjnash

@nodejs/build A comment was made but the bot still closed the issue.

targos avatar Jul 02 '21 08:07 targos

Isn't that the expected behaviour, seeing the bot is configured to close issues after having labelled it with stale and no activity happens for 30 days? https://github.com/nodejs/build/blob/1a77fd78ddd10cea455100f6e0920cf49507d609/.github/workflows/stale.yml#L17

..that's at least how I interpret that configuration and the stale action' v1 source code: actions/stale ./src/main.ts

phillipj avatar Jul 02 '21 09:07 phillipj

so looking into it the feature to unstale upon updating wasnt added untill around v3 https://github.com/actions/stale/pull/58 and we run v1 - will open a PR to update the action

AshCripps avatar Jul 02 '21 09:07 AshCripps

PR here: https://github.com/nodejs/build/pull/2699

AshCripps avatar Jul 02 '21 09:07 AshCripps

I'm told (https://github.com/nodejs/build/issues/2404) that the bots are not supposed to be using Python 2.7 anymore, so this issue may only be a configuration issue on a small number of remaining bots.

vtjnash avatar Jul 05 '21 12:07 vtjnash

Is there a todo on this Python 2 issue or can it be closed?

  • https://github.com/nodejs/tap2junit/releases

cclauss avatar Apr 11 '23 23:04 cclauss

I don't think there's anything left to do. Please reopen or comment if incorrect.

richardlau avatar Apr 12 '23 18:04 richardlau