Codecov Report

Merging #1322 (5729d08) into 1.3 (0a2b86f) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##              1.3    #1322   +/-   ##
=======================================
  Coverage   72.33%   72.33%           
=======================================
  Files          88       88           
  Lines        1916     1916           
  Branches      250      245    -5     
=======================================
  Hits         1386     1386           
  Misses        474      474           
  Partials       56       56           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Location: /ci/dbc/distribution-build-opensearch/1.3.9/7067/windows/x64/zip/builds/opensearch/plugins/opensearch-security-1.3.9.0.zip [following]
--2023-02-10 23:12:30--  https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/1.3.9/7067/windows/x64/zip/builds/opensearch/plugins/opensearch-security-1.3.9.0.zip
Reusing existing connection to ci.opensearch.org:443.
HTTP request sent, awaiting response... 403 Forbidden
2023-02-10 23:12:31 ERROR 403: Forbidden.

Looks like we need to fix this build to latest, @RyanL1997 can you look into this?

@peternied Thanks for the review! I think it is the latest: https://github.com/opensearch-project/security-dashboards-plugin/actions/runs/4148396846/jobs/7231566838#step:9:66. The reason we are getting this 403 is that we are still awaiting for the version increment of the security backend and building artifact. There is no changes need to be applied for this one.

@RyanL1997 We can't use maven because the 1.3 security plugin doesn't support publishing to maven #1331 (comment)

If you figure out how to get this supported in 1.3 then we can start publishing and consuming via maven.

Awaiting for the merge of opensearch-project/opensearch-build#3263, so that we can get the distribution build to unblock this PR

@peternied Got it. For this one, we have added the security plugin to the manifest, and the CI is passing. I think we are good to merge this PR now.

@peternied The change requested was addressed and latest is being pulled:

From the latest integration test output:

Run wget 'https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/1.3.9/latest/windows/x64/zip/builds/opensearch/plugins/opensearch-security-1.3.9.0.zip' --retry-connrefused --tries=3 --no-dns-cache --progress=bar:force:noscroll --verbose
--2023-03-08 01:44:16-- https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/1.3.9/latest/windows/x64/zip/builds/opensearch/plugins/opensearch-security-1.3.9.0.zip