Actions
action #103329
closedos-autoinst: unhandled and confusing CI test ouput from git calls size:M
Description
Observation¶
From https://github.com/os-autoinst/os-autoinst/runs/4326980124?check_suite_focus=true#step:3:624 and in every os-autoinst CI test run:
8: [18:02:43]
8: All tests successful.
8: Files=49, Tests=1108, 415 wallclock secs ( 0.68 usr 0.11 sys + 266.31 cusr 21.89 csys = 288.99 CPU)
8: Result: PASS
8: fatal: ambiguous argument ':!COPYING': unknown revision or path not in the working tree.
8: Use '--' to separate paths from revisions, like this:
8: 'git <command> [<revision>...] -- [<file>...]'
8: fatal: ambiguous argument ':!COPYING': unknown revision or path not in the working tree.
8: Use '--' to separate paths from revisions, like this:
8: 'git <command> [<revision>...] -- [<file>...]'
8: fatal: ambiguous argument ':!COPYING': unknown revision or path not in the working tree.
8: Use '--' to separate paths from revisions, like this:
8: 'git <command> [<revision>...] -- [<file>...]'
8: [18:02:44] ./01-style.t ............. ok 406 ms ( 0.00 usr 0.00 sys + 0.33 cusr 0.07 csys = 0.40 CPU)
Acceptance criteria¶
- AC1: Still working xt/01-style.t both locally and CI but no unhandled test output unless the test fails
Updated by livdywan almost 3 years ago
- Subject changed from os-autoinst: unhandled and confusing CI test ouput from git calls to os-autoinst: unhandled and confusing CI test ouput from git calls size:M
- Status changed from New to Workable
Updated by okurz almost 3 years ago
Related to that: https://github.com/os-autoinst/os-autoinst/pull/1871 which fixes a new file that already misses the "Test::Warnings" which xt/01-style.t should check for but only shows this when executed locally, not in GHA
Updated by okurz almost 3 years ago
- Status changed from Workable to In Progress
- Assignee set to okurz
I have an idea
Updated by okurz almost 3 years ago
- Status changed from In Progress to Feedback
Updated by okurz almost 3 years ago
- Status changed from Feedback to Resolved
merged. https://github.com/os-autoinst/os-autoinst/runs/4396177333?check_suite_focus=true#step:3:617 shows the fix effective in master
Updated by okurz almost 3 years ago
- Category changed from Feature requests to Regressions/Crashes
Actions