-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[pkg/stanza/fileconsumer] TestExcludeOlderThanFilter/file_not_present test failing on Windows #32464
Labels
Comments
Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
djaglowski
pushed a commit
that referenced
this issue
Apr 17, 2024
**Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> Test error message: ``` Error: Error "CreateFile C:\\Users\\RUNNER~1\\AppData\\Local\\Temp\\TestExcludeOlderThanFilterfile_not_present318488423\\001\\b.log: The system cannot find the file specified." does not contain "b.log: no such file or directory" ``` This is simply saying the error message on Windows doesn't contain the expected error message. However, we can tell from the output that the test is working as intended. This updates the test to use the proper error message on Windows to determine if it's successful. **Link to tracking Issue:** <Issue number if applicable> Resolves #32464 **Testing:** <Describe what testing was performed and which tests were added.> I've added the `Run Windows` label to confirm this is working as intended.
rimitchell
pushed a commit
to rimitchell/opentelemetry-collector-contrib
that referenced
this issue
May 8, 2024
…32474) **Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> Test error message: ``` Error: Error "CreateFile C:\\Users\\RUNNER~1\\AppData\\Local\\Temp\\TestExcludeOlderThanFilterfile_not_present318488423\\001\\b.log: The system cannot find the file specified." does not contain "b.log: no such file or directory" ``` This is simply saying the error message on Windows doesn't contain the expected error message. However, we can tell from the output that the test is working as intended. This updates the test to use the proper error message on Windows to determine if it's successful. **Link to tracking Issue:** <Issue number if applicable> Resolves open-telemetry#32464 **Testing:** <Describe what testing was performed and which tests were added.> I've added the `Run Windows` label to confirm this is working as intended.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Component(s)
pkg/stanza
Describe the issue you're reporting
Failed CI/CD run
Failure output:
Potentially related to #32299, where the solution was to use
filepath.Join
for paths to work properly on Windows.The text was updated successfully, but these errors were encountered: