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

Updates 1.3 dashboards manifest to have security-dashboards-plugin reference 1.3 instead of main #1702

Merged

Conversation

DarshitChanpura
Copy link
Member

Description

Security-dashboards-plugin currently point to main branch . This PR updates it to now point to 1.3 branch in the yml file

Check List

  • Commits are signed per the DCO using --signoff

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.

@DarshitChanpura DarshitChanpura requested a review from a team as a code owner March 3, 2022 20:45
@DarshitChanpura
Copy link
Member Author

Looks like the build failed during sanity testing of kNN plugin. Here is the stacktrace:
2022-03-03 20:58:54 INFO     Sanity testing k-NN
2022-03-03 20:58:54 INFO     Executing "git init" in /tmp/tmp01gfhvxn/k-NN
2022-03-03 20:58:54 INFO     Executing "git remote add origin https://github.com/opensearch-project/k-NN.git" in /tmp/tmp01gfhvxn/k-NN
2022-03-03 20:58:54 INFO     Executing "git fetch --depth 1 origin main" in /tmp/tmp01gfhvxn/k-NN
2022-03-03 20:58:55 INFO     Executing "git checkout FETCH_HEAD" in /tmp/tmp01gfhvxn/k-NN
2022-03-03 20:58:56 INFO     Executing "git rev-parse HEAD" in /tmp/tmp01gfhvxn/k-NN
2022-03-03 20:58:56 INFO     Checked out https://github.com/opensearch-project/k-NN.git@main into /tmp/tmp01gfhvxn/k-NN at b22a2dc44ee93464bf4c5d17cf26d[1268](https://github.com/opensearch-project/opensearch-build/runs/5413474267?check_suite_focus=true#step:6:1268)54060cd
2022-03-03 20:58:56 INFO     Executing "./gradlew properties -Dopensearch.version=1.3.0-SNAPSHOT -Dbuild.snapshot=true" in /tmp/tmp01gfhvxn/k-NN

FAILURE: Build failed with an exception.

* Where:
Build file '/tmp/tmp01gfhvxn/k-NN/build.gradle' line: 51

* What went wrong:
A problem occurred evaluating root project 'opensearch-knn'.
> Failed to apply plugin class 'org.opensearch.gradle.info.GlobalBuildInfoPlugin'.
   > Could not create plugin of type 'GlobalBuildInfoPlugin'.
      > Could not generate a decorated class for type GlobalBuildInfoPlugin.
         > org/gradle/jvm/toolchain/JavaInstallation

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org/

BUILD FAILED in 8s
2022-03-03 20:59:08 ERROR    CI Manifest check failed

Should we restart the build?

@gaiksaya
Copy link
Member

gaiksaya commented Mar 3, 2022

Looks like the build failed during sanity testing of kNN plugin. Here is the stacktrace:

2022-03-03 20:58:54 INFO     Sanity testing k-NN
2022-03-03 20:58:54 INFO     Executing "git init" in /tmp/tmp01gfhvxn/k-NN
2022-03-03 20:58:54 INFO     Executing "git remote add origin https://github.com/opensearch-project/k-NN.git" in /tmp/tmp01gfhvxn/k-NN
2022-03-03 20:58:54 INFO     Executing "git fetch --depth 1 origin main" in /tmp/tmp01gfhvxn/k-NN
2022-03-03 20:58:55 INFO     Executing "git checkout FETCH_HEAD" in /tmp/tmp01gfhvxn/k-NN
2022-03-03 20:58:56 INFO     Executing "git rev-parse HEAD" in /tmp/tmp01gfhvxn/k-NN
2022-03-03 20:58:56 INFO     Checked out https://github.com/opensearch-project/k-NN.git@main into /tmp/tmp01gfhvxn/k-NN at b22a2dc44ee93464bf4c5d17cf26d[1268](https://github.com/opensearch-project/opensearch-build/runs/5413474267?check_suite_focus=true#step:6:1268)54060cd
2022-03-03 20:58:56 INFO     Executing "./gradlew properties -Dopensearch.version=1.3.0-SNAPSHOT -Dbuild.snapshot=true" in /tmp/tmp01gfhvxn/k-NN

FAILURE: Build failed with an exception.

* Where:
Build file '/tmp/tmp01gfhvxn/k-NN/build.gradle' line: 51

* What went wrong:
A problem occurred evaluating root project 'opensearch-knn'.
> Failed to apply plugin class 'org.opensearch.gradle.info.GlobalBuildInfoPlugin'.
   > Could not create plugin of type 'GlobalBuildInfoPlugin'.
      > Could not generate a decorated class for type GlobalBuildInfoPlugin.
         > org/gradle/jvm/toolchain/JavaInstallation

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org/

BUILD FAILED in 8s
2022-03-03 20:59:08 ERROR    CI Manifest check failed

Should we restart the build?

Build is failing for k-NN. Informed the team here: #1701 (comment)
cc: @opensearch-project/k-nn

@gaiksaya
Copy link
Member

gaiksaya commented Mar 3, 2022

Hi @DarshitChanpura can you rebase with main? The issue should be fixed now. See #1704
Thanks!

…ference 1.3 instead of main

Signed-off-by: Darshit Chanpura <dchanp@amazon.com>
@DarshitChanpura DarshitChanpura force-pushed the update/dashboards-manifest branch from b59c85a to 8235428 Compare March 4, 2022 01:15
@DarshitChanpura
Copy link
Member Author

Done..thank you @gaiksaya .

@gaiksaya gaiksaya merged commit 543ef57 into opensearch-project:main Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants