-
Notifications
You must be signed in to change notification settings - Fork 75
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
-f absolute path fails on windows uploader v0.1.15 with: "No paths matched existing files!" #590
Comments
It's supposed to already skip the blocklist. Lines 194 to 209 in 08140be
However, it doesn't look like you are getting those logs. 🤔 Do you get them on the Linux one? (goes to do some tests) |
Confirmed that 08140be does not display the new logging on Windows. 👎 |
@ben-spiller, it looks like your Windows slashes are backwards. Could you try running the uploader with them swapped? |
It's not easy to switch to backslashes in my environment actually - and from the comments it looks like you're able to repro this failure mode in your environment anyway? If not let me know and I'll try and find some time to try it. If the problem was caused by the slash direction, then why would the relative path "tests/__coverage_python.win-py3.10/coverage.xml" work? Either way, the action should be able to cope with both slash directions - as workflows need to be designed to be cross-platform, we don't want people to have to create separate workflows for windows and linux just to avoid slash problems. And we definitely would benefit from need more detailed logging in verbose mode to help debug these kinds of issues. |
Any updates on this - I think you said you can already reproduce it without more info from me? ("Confirmed that 08140be does not display the new logging on Windows.") If think it's the forward slashes on windows causing the problem, the action could just normalize them for the current OS? Would be great to have a robust action in place for this, rather than requiring end-user workarounds. Thanks |
I'll bring this up with the team, @ben-spiller. There is certainly a lot of value in handling slashes of both directions. I've opened up codecov/codecov-action#638 to track the issue in the codecov/codecov-action repository. |
Closing as a won't do. I would recommend migrating tot he CLI and seeing if the issue still exists |
Describe the bug
After switching from the v1 to the latest v2 uploader, my Linux upload job works but my identical Windows upload fails with the output below. I've turned on verbose logging to get insight into what is happening but it seems verbose logging doesn't mention it. If nothing else, please add to the verbose logging enough info to see why a file was ignored.
I am explicitly specifying the filename with
-f D:/a/xpybuild/xpybuild/tests/__coverage_python.win-py3.10/coverage.xml
- but although that file definitely exists, I get theNo paths matched existing files!
error.I also tried specifying the same filename with a relative path
-f tests/__coverage_python.win-py3.10/coverage.xml
and that works.If I instead use a directory search with an absolute path
-s D:/a/xpybuild/xpybuild/tests/__coverage_python.win-py3.10
it works.And as mentioned, using the absolute -f path on linux works.
To Reproduce
I'm using GitHub Actions, it seems to be the uploader that has the issue.
Using v0.1.15 uploader, execute a command line:
D:\a\_actions\codecov\codecov-action\v2.1.0\dist\codecov.exe -n win-py3.10 -Q github-action-2.1.0 -Z -s D:/a/xpybuild/xpybuild/tests/__coverage_python.win-py3.10 -v
Expected behavior
It should upload the file given I specified, given that it exists. Also given this works on linux it should work the same on Windows.
If not, it should give an error message describing why the file I specified was ignored. Given there is no documentation on what patterns are excluded, verbose logging is especially important to help people debug this kind of thing.
Additional context
https://github.com/xpybuild/xpybuild/runs/4872059624?check_suite_focus=true (as you see I've been hacking around trying to get it working on this branch)
The text was updated successfully, but these errors were encountered: