Skip to content

Conversation

@kiersten-stokes
Copy link
Contributor

@kiersten-stokes kiersten-stokes commented Sep 27, 2024

Description

This PR adds a new configuration property for an OAUTH2 server uri: iceberg.rest.auth.oauth2.uri

Motivation and Context

Fixes #23491

See linked issue for context

Impact

Adds a iceberg.rest.auth.oauth2.uri configurable property

Test Plan

Configuration tests have been updated

Contributor checklist

  • Please make sure your submission complies with our development, formatting, commit message, and attribution guidelines.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

Iceberg Connector Changes
* Add iceberg.rest.auth.oauth2.uri configurable property :pr:`23739` 

steveburnett
steveburnett previously approved these changes Sep 27, 2024
Copy link
Contributor

@steveburnett steveburnett left a comment

Choose a reason for hiding this comment

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

LGTM! (docs)

Pull branch, local doc build, looks good. Thanks!

agrawalreetika
agrawalreetika previously approved these changes Sep 30, 2024
Copy link
Member

@agrawalreetika agrawalreetika left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ZacBlanco ZacBlanco left a comment

Choose a reason for hiding this comment

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

It would be good to see a unit test where we can verify the catalog config has properly set the OAUTH2_SERVER_URI as well.

@kiersten-stokes kiersten-stokes merged commit 266ccec into prestodb:master Oct 1, 2024
@jaystarshot jaystarshot mentioned this pull request Nov 1, 2024
25 tasks
@tdcmeehan tdcmeehan added the from:IBM PR from IBM label Dec 13, 2024
@prestodb-ci prestodb-ci requested review from a team, ScrapCodes and aaneja and removed request for a team December 13, 2024 15:31
@prestodb-ci
Copy link
Contributor

Saved that user @kiersten-stokes is from IBM

@steveburnett
Copy link
Contributor

Formatting nit for the release note entry

== RELEASE NOTES ==

Iceberg Connector Changes
* Add ``iceberg.rest.auth.oauth2.uri`` configurable property. :pr:`23739` 

Copy link
Contributor

@steveburnett steveburnett left a comment

Choose a reason for hiding this comment

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

LGTM! (docs)

Pull branch, local doc build, looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

from:IBM PR from IBM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Iceberg REST configurable OAuth Endpoint

7 participants