Skip to content

Conversation

@vamsimanohar
Copy link
Member

…ettings GET API (#1828)

Signed-off-by: Vamsi Manohar [email protected]
(cherry picked from commit a8ecd2f)

  • Fxied DatasourceClusterSettingsIT for 2.x

Signed-off-by: Vamsi Manohar [email protected]


Signed-off-by: Vamsi Manohar [email protected]
Co-authored-by: Vamsi Manohar [email protected]
(cherry picked from commit 98ca9f8)

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • 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.

…ettings GET API (#1828)

* Restrict master key reading from cluster settings API (#1825)

Signed-off-by: Vamsi Manohar <[email protected]>
(cherry picked from commit a8ecd2f)

* Fxied DatasourceClusterSettingsIT for 2.x

Signed-off-by: Vamsi Manohar <[email protected]>

---------

Signed-off-by: Vamsi Manohar <[email protected]>
Co-authored-by: Vamsi Manohar <[email protected]>
(cherry picked from commit 98ca9f8)
@vamsimanohar vamsimanohar changed the title [Backport 2.x] Filter datasource encryption master key from cluster s… [Backport 2.9] Filter datasource encryption master key from cluster s… Jul 11, 2023
@codecov
Copy link

codecov bot commented Jul 11, 2023

Codecov Report

Merging #1844 (4e03281) into 2.9 (2144ddd) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                2.9    #1844   +/-   ##
=========================================
  Coverage     97.35%   97.35%           
  Complexity     4443     4443           
=========================================
  Files           388      388           
  Lines         10998    10998           
  Branches        784      784           
=========================================
  Hits          10707    10707           
  Misses          284      284           
  Partials          7        7           
Flag Coverage Δ
sql-engine 97.35% <ø> (ø)

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

Impacted Files Coverage Δ
...rch/sql/opensearch/setting/OpenSearchSettings.java 100.00% <ø> (ø)

@vamsimanohar vamsimanohar merged commit ce9d4ba into 2.9 Jul 11, 2023
@github-actions github-actions bot deleted the backport/backport-1828-to-2.9 branch July 11, 2023 17:39
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