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

Updated JDBC driver in artifacts and download page #1730

Merged
merged 4 commits into from
Jun 23, 2023

Conversation

GumpacG
Copy link
Contributor

@GumpacG GumpacG commented Jun 20, 2023

Description

Updating the version of the JDBC driver on the download page is required for Tableau Certification of OpenSearch. The driver has been published and signed using JarSigner as per opensearch-project/opensearch-build#3614.
There is no signature file in the artifacts bucket for the driver because it is not signed with PGP. I've added a command in https://opensearch.org/verify-signatures.html for verifying the driver using jarsigner.

Issues Resolved

#1721

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 BSD-3-Clause License.

Signed-off-by: Guian Gumpac <guian.gumpac@improving.com>
Signed-off-by: Guian Gumpac <guian.gumpac@improving.com>
_includes/downloads/jdbc.markdown Outdated Show resolved Hide resolved
verify-signatures.markdown Outdated Show resolved Hide resolved
verify-signatures.markdown Show resolved Hide resolved
verify-signatures.markdown Show resolved Hide resolved
_versions/2023-06-20-opensearch-2.8.0.markdown Outdated Show resolved Hide resolved
Signed-off-by: Guian Gumpac <guian.gumpac@improving.com>
@GumpacG
Copy link
Contributor Author

GumpacG commented Jun 22, 2023

Hi @krisfreedain, would you be able to take a look at this? This is beneficial for Tableau certification in order to get users to download the latest and greatest JDBC driver.

verify-signatures.markdown Outdated Show resolved Hide resolved
Signed-off-by: Guian Gumpac <guian.gumpac@improving.com>
Copy link
Collaborator

@dtaivpp dtaivpp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and verified the signature and looks good!

@dtaivpp dtaivpp merged commit 0fa8687 into opensearch-project:main Jun 23, 2023
@krisfreedain
Copy link
Member

thank you @GumpacG & @dtaivpp

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.

5 participants