Skip to content
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

[Windows Container] Make sure WindowsZip/LinuxTar remove correct dir when uninstall #4023

Conversation

peterzhuamazon
Copy link
Member

@peterzhuamazon peterzhuamazon commented Sep 15, 2023

Description

[Windows Container] Make sure WindowsZip/LinuxTar remove correct dir when uninstall

Issues Resolved

opensearch-project/opensearch-ci#281
#3743

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@peterzhuamazon
Copy link
Member Author

peterzhuamazon commented Sep 15, 2023

Before:

rm: cannot remove 'C:\Users\Administrator\tmp76kgq8lv\local-test-cluster/opensearch-2.9.0/modules/transport-netty4/netty-buffer-4.1.94.Final.jar': Device or resource busy
rm: cannot remove 'C:\Users\Administrator\tmp76kgq8lv\local-test-cluster/opensearch-2.9.0/modules/transport-netty4/netty-codec-4.1.94.Final.jar': Device or resource busy
rm: cannot remove 'C:\Users\Administrator\tmp76kgq8lv\local-test-cluster/opensearch-2.9.0/modules/transport-netty4/netty-codec-http-4.1.94.Final.jar': Device or resource busy
rm: cannot remove 'C:\Users\Administrator\tmp76kgq8lv\local-test-cluster/opensearch-2.9.0/modules/transport-netty4/netty-common-4.1.94.Final.jar': Device or resource busy
rm: cannot remove 'C:\Users\Administrator\tmp76kgq8lv\local-test-cluster/opensearch-2.9.0/modules/transport-netty4/netty-handler-4.1.94.Final.jar': Device or resource busy
rm: cannot remove 'C:\Users\Administrator\tmp76kgq8lv\local-test-cluster/opensearch-2.9.0/modules/transport-netty4/netty-resolver-4.1.94.Final.jar': Device or resource busy
rm: cannot remove 'C:\Users\Administrator\tmp76kgq8lv\local-test-cluster/opensearch-2.9.0/modules/transport-netty4/netty-transport-4.1.94.Final.jar': Device or resource busy
rm: cannot remove 'C:\Users\Administrator\tmp76kgq8lv\local-test-cluster/opensearch-2.9.0/modules/transport-netty4/netty-transport-native-unix-common-4.1.94.Final.jar': Device
 or resource busy

......

2023-09-15 02:13:09 WARNING  Removing try count: 1/3 for C:\Users\Administrator\tmp76kgq8lv
2023-09-15 02:13:09 WARNING  Exception: [WinError 145] The directory is not empty: 'C:\\Users\\Administrator\\tmp76kgq8lv'
2023-09-15 02:13:09 WARNING  Removing try count: 2/3 for C:\Users\Administrator\tmp76kgq8lv
2023-09-15 02:13:09 WARNING  Exception: [WinError 145] The directory is not empty: 'C:\\Users\\Administrator\\tmp76kgq8lv'
2023-09-15 02:13:09 WARNING  Removing try count: 3/3 for C:\Users\Administrator\tmp76kgq8lv
2023-09-15 02:13:09 WARNING  Exception: [WinError 145] The directory is not empty: 'C:\\Users\\Administrator\\tmp76kgq8lv'
Traceback (most recent call last):
  File "C:\Users\Administrator\opensearch-build-peterzhuamazon\src\run_integ_test.py", line 36, in <module>

After:

2023-09-15 02:08:24 INFO     Sending SIGKILL to PID 13080
2023-09-15 02:08:24 INFO     Process killed with exit code None
2023-09-15 02:08:42 INFO     Recording local cluster logs for mlCommonsDashboards with test configuration as without-security at C:\Users\Administrator\opensearch-build-peterz
huamazon\test-results\ed8898ca2f2a4f77972b12e6479b9c75\integ-test\mlCommonsDashboards\without-security\local-cluster-logs
2023-09-15 02:08:42 INFO     Cleanup C:\Users\Administrator\tmp3zukb0i0\local-test-cluster\opensearch-dashboards-2.9.0 content after the test
2023-09-15 02:08:52 INFO     Sending SIGKILL to PID 14052
2023-09-15 02:08:52 INFO     Process killed with exit code None
2023-09-15 02:09:11 INFO     Recording local cluster logs for mlCommonsDashboards with test configuration as without-security at C:\Users\Administrator\opensearch-build-peterz
huamazon\test-results\ed8898ca2f2a4f77972b12e6479b9c75\integ-test\mlCommonsDashboards\without-security\local-cluster-logs
2023-09-15 02:09:11 INFO     Cleanup C:\Users\Administrator\tmp3zukb0i0\local-test-cluster\opensearch-2.9.0 content after the test
2023-09-15 02:09:11 INFO     Recording component test results for mlCommonsDashboards at C:\Users\Administrator\opensearch-build-peterzhuamazon\test-results\ed8898ca2f2a4f7797
2b12e6479b9c75\integ-test\mlCommonsDashboards\without-security
2023-09-15 02:09:11 INFO     Walking tree from C:\Users\Administrator\opensearch-build-peterzhuamazon\test-results\ed8898ca2f2a4f77972b12e6479b9c75\integ-test\mlCommonsDashboa
rds\without-security
2023-09-15 02:09:11 INFO     Removing C:\Users\Administrator\tmp3zukb0i0
2023-09-15 02:09:11 WARNING  Removing try count: 1/3 for C:\Users\Administrator\tmp3zukb0i0\mlCommonsDashboards\.git\objects\pack\pack-d9a227460d89739d54be0f92bdebcf87290503a7
.idx
2023-09-15 02:09:11 WARNING  Removing try count: 1/3 for C:\Users\Administrator\tmp3zukb0i0\mlCommonsDashboards\.git\objects\pack\pack-d9a227460d89739d54be0f92bdebcf87290503a7
.pack
2023-09-15 02:09:11 WARNING  Removing try count: 1/3 for C:\Users\Administrator\tmp3zukb0i0\mlCommonsDashboards\.git\objects\pack\pack-d9a227460d89739d54be0f92bdebcf87290503a7
.rev
2023-09-15 02:09:14 ERROR    | mlCommonsDashboards  | without-security     | PASS  |

@codecov
Copy link

codecov bot commented Sep 15, 2023

Codecov Report

Merging #4023 (b45b6f8) into main (9ff7a9e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #4023   +/-   ##
=======================================
  Coverage   92.10%   92.10%           
=======================================
  Files         187      187           
  Lines        5674     5674           
=======================================
  Hits         5226     5226           
  Misses        448      448           
Files Changed Coverage Δ
src/test_workflow/integ_test/distribution_tar.py 100.00% <100.00%> (ø)
src/test_workflow/integ_test/distribution_zip.py 100.00% <100.00%> (ø)

@rishabh6788
Copy link
Collaborator

Please fix the failing test @peterzhuamazon

Signed-off-by: Peter Zhu <[email protected]>
@peterzhuamazon
Copy link
Member Author

Please fix the failing test @peterzhuamazon

Fixed now.

@peterzhuamazon peterzhuamazon merged commit 6e89bdb into opensearch-project:main Sep 15, 2023
12 checks passed
@peterzhuamazon peterzhuamazon deleted the windows-integtest-removal-fix branch September 15, 2023 02:33
peterzhuamazon added a commit to peterzhuamazon/opensearch-build that referenced this pull request Sep 15, 2023
@peterzhuamazon peterzhuamazon changed the title Make sure WindowsZip/LinuxTar remove correct dir when uninstall [Windows Container] Make sure WindowsZip/LinuxTar remove correct dir when uninstall Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

3 participants