windows CI for AD

[amitgalitz]

Description

Adds windows CI for main branch

1. Changed the usage of getPath() to first use .toURI() in some of the test cases. This is because just getting the path like we used to was returning a string like this: \C:\Sample\sample.txt. This isn't supported on Windows. Converting with toUri() works on all platforms.

Issues Resolved

resolves #274

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.

[codecov-commenter]

Codecov Report

Merging #703 (80914a8) into main (7a05826) will decrease coverage by 0.46%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #703      +/-   ##
============================================
- Coverage     79.25%   78.79%   -0.47%     
+ Complexity     4255     4229      -26     
============================================
  Files           301      301              
  Lines         17872    17872              
  Branches       1897     1897              
============================================
- Hits          14165    14082      -83     
- Misses         2810     2888      +78     
- Partials        897      902       +5     

Flag	Coverage Δ
plugin	78.79% <ø> (-0.47%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...java/org/opensearch/ad/task/ADBatchTaskRunner.java	81.76% <0.00%> (-3.96%)	⬇️
...c/main/java/org/opensearch/ad/util/ParseUtils.java	73.92% <0.00%> (-3.93%)	⬇️
...va/org/opensearch/ad/feature/SearchFeatureDao.java	82.84% <0.00%> (-3.85%)	⬇️
...rch/ad/transport/ForwardADTaskTransportAction.java	94.06% <0.00%> (-3.39%)	⬇️
...ava/org/opensearch/ad/task/ADHCBatchTaskCache.java	90.12% <0.00%> (-2.47%)	⬇️
...ain/java/org/opensearch/ad/model/ModelProfile.java	70.90% <0.00%> (-1.82%)	⬇️
...ain/java/org/opensearch/ad/task/ADTaskManager.java	75.69% <0.00%> (-1.67%)	⬇️
...rch/ad/transport/AnomalyResultTransportAction.java	79.95% <0.00%> (-0.93%)	⬇️

[amitgalitz]

This is currently failing because a change was made in main to now allow Opensearch to run as root user. This was reverted an hour with this PR: opensearch-project/OpenSearch#4949. Will try running CI again once validated this change is in latest snapshot

[amitgalitz]

@ohltyler @jackiehanyang Lets merge this into main and then merge following backport PR. I will close the other windows-ci PR when this is merged and passing CI.

[ohltyler]

Nice! LGTM. Just recommend changing the PR description from 2.x -> main

[ohltyler]

Also, if you add closes or resolves keyword in front of the issue number it will automatically link to the PR