Skip to content

Commit

Permalink
[Internal] Decouple serving and oauth2 package (#1110)
Browse files Browse the repository at this point in the history
## What changes are proposed in this pull request?

Package `oauth2` contains a `DataPlaneService` that is not intended for
users to use directly. This service is not auto-generated and is only
used in the `serving` package. To remove the dependency between the
`serving` package and the `oauth2` package, this PR moves the
`DataPlaneService` from the `oauth2` to the `serving` package.

## How is this tested?
Existing unit tests
  • Loading branch information
parthban-db authored Jan 7, 2025
1 parent 83db3cb commit adc94ca
Show file tree
Hide file tree
Showing 15 changed files with 231 additions and 75 deletions.
2 changes: 1 addition & 1 deletion .codegen/_openapi_sha
Original file line number Diff line number Diff line change
@@ -1 +1 @@
a6a317df8327c9b1e5cb59a03a42ffa2aabeef6d
1668b0db17e23605f8c9d29fb3b674c01590732d
2 changes: 2 additions & 0 deletions .gitattributes
Original file line number Diff line number Diff line change
Expand Up @@ -77,9 +77,11 @@ experimental/mocks/service/marketplace/mock_provider_provider_analytics_dashboar
experimental/mocks/service/marketplace/mock_provider_providers_interface.go linguist-generated=true
experimental/mocks/service/ml/mock_experiments_interface.go linguist-generated=true
experimental/mocks/service/ml/mock_model_registry_interface.go linguist-generated=true
experimental/mocks/service/oauth2/mock_account_federation_policy_interface.go linguist-generated=true
experimental/mocks/service/oauth2/mock_custom_app_integration_interface.go linguist-generated=true
experimental/mocks/service/oauth2/mock_o_auth_published_apps_interface.go linguist-generated=true
experimental/mocks/service/oauth2/mock_published_app_integration_interface.go linguist-generated=true
experimental/mocks/service/oauth2/mock_service_principal_federation_policy_interface.go linguist-generated=true
experimental/mocks/service/oauth2/mock_service_principal_secrets_interface.go linguist-generated=true
experimental/mocks/service/pipelines/mock_pipelines_interface.go linguist-generated=true
experimental/mocks/service/provisioning/mock_credentials_interface.go linguist-generated=true
Expand Down
12 changes: 12 additions & 0 deletions service/apps/model.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 4 additions & 2 deletions service/catalog/model.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

26 changes: 25 additions & 1 deletion service/jobs/api.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

14 changes: 13 additions & 1 deletion service/jobs/interface.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

83 changes: 71 additions & 12 deletions service/jobs/model.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit adc94ca

Please sign in to comment.