Adding open access for a module to fix pa plugin jdk 17 issue #1947
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Sagar Upadhyaya <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #1947 +/- ##
=========================================
Coverage 94.45% 94.46%
- Complexity 20 22 +2
=========================================
Files 179 179
Lines 3645 3649 +4
Branches 29 29
=========================================
+ Hits 3443 3447 +4
Misses 196 196
Partials 6 6
Continue to review full report at Codecov.
|
sruti1312
approved these changes
Apr 8, 2022
dblock
approved these changes
Apr 8, 2022
prudhvigodithi
pushed a commit
to prudhvigodithi/opensearch-build
that referenced
this pull request
Apr 13, 2022
…arch-project#1947) Signed-off-by: Sagar Upadhyaya <[email protected]> Adding code block for maven plugin zips Signed-off-by: pgodithi <[email protected]> Adding code block for maven plugin zips Signed-off-by: pgodithi <[email protected]> zip maven publish Signed-off-by: pgodithi <[email protected]> added zipsmaven dir check Signed-off-by: pgodithi <[email protected]> Adding better gradle project dir Signed-off-by: pgodithi <[email protected]> test maven snapshots Signed-off-by: pgodithi <[email protected]> test maven snapshots Signed-off-by: pgodithi <[email protected]> test maven snapshots Signed-off-by: pgodithi <[email protected]> test maven snapshots Signed-off-by: pgodithi <[email protected]> test maven snapshots Signed-off-by: pgodithi <[email protected]> test maven snapshots Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> Removing build script for PA (opensearch-project#1952) Signed-off-by: Sagar Upadhyaya <[email protected]> update release issue with steps to update ansible and helm (opensearch-project#1942) * update release issue with steps to update ansible and helm Signed-off-by: Abhinav Gupta <[email protected]> * updated test as sample PR Signed-off-by: Abhinav Gupta <[email protected]> * adding periods Signed-off-by: Abhinav Gupta <[email protected]> Fix the path for maven sign and staging job (opensearch-project#1954) * Fix the path for staging maven Signed-off-by: Zelin Hao <[email protected]> * Add the distribution folder tar to the artifact path and update test Signed-off-by: Zelin Hao <[email protected]> [OSD][2.0.0] use rc1 qualifier (opensearch-project#1949) Signed-off-by: Kawika Avilla <[email protected]> install createrepo in docker image for RPM yum repo support (opensearch-project#1955) * install createrepo in docker image for RPM yum repo support Signed-off-by: Tianle Huang <[email protected]> * fix Signed-off-by: Tianle Huang <[email protected]> * update opensearch to use new image Signed-off-by: Tianle Huang <[email protected]> Add systemd docker for rpm service test (opensearch-project#1958) * Add systemd based image for yum install testing and more Signed-off-by: Peter Zhu <[email protected]> * More packages to add to the images Signed-off-by: Peter Zhu <[email protected]> lower freq for build for 1.3.1 opensearch-dashboards (opensearch-project#1941) * lower freq for build for 1.3.1 opensearch-dashboards Signed-off-by: Abhinav Gupta <[email protected]> * removed build for 1.2.x and 1.3.1 Signed-off-by: Abhinav Gupta <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> Added notifications-core and notifications to 2.0 and add integTest manifest (opensearch-project#1957) * Added notifications-core and notifications to 2.0. Signed-off-by: dblock <[email protected]> * Remove scripts from opensearch-build. Signed-off-by: dblock <[email protected]> * Remove dashboards notifications scripts to use the default one Signed-off-by: Peter Zhu <[email protected]> * Remove CI checks for lack of gradlew. Signed-off-by: dblock <[email protected]> * Fix the second subfolder location after core Signed-off-by: Peter Zhu <[email protected]> * Add notifications related build scripts to build repo Signed-off-by: Peter Zhu <[email protected]> * Tweak build scripts to correctly move the zips Signed-off-by: Peter Zhu <[email protected]> Co-authored-by: Peter Zhu <[email protected]> Make sure notifications is built correctly while passing ci checks (opensearch-project#1959) Signed-off-by: Peter Zhu <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]> test snapshot build Signed-off-by: pgodithi <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Sagar Upadhyaya [email protected]
Description
This change adds open access(via 'add-opens') to jdk.attach module as PA plugin fails to start otherwise in jdk 17 (which had breaking changes and not allowing illegal runtime access to jdk internal classes)
We have opened an internal in RCA to exploring changing logic of taking thread dumps and avoid using "add-open" access.
opensearch-project/performance-analyzer-rca#161
Issues Resolved
opensearch-project/performance-analyzer#144 - Though this states 1.3 as we had earlier planned JDK 17 for 1.3 release.
Check List
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.