-
Notifications
You must be signed in to change notification settings - Fork 487
chore(api): sync Management API OpenAPI spec #5752
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
base: develop
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2539,6 +2539,7 @@ export const V1GetOrganizationEntitlementsOutput = Schema.Struct({ | |
| "integrations.github_connections", | ||
| "dedicated_pooler", | ||
| "observability.dashboard_advanced_metrics", | ||
| "api.members.invitations", | ||
| "api.members.roles", | ||
| ]), | ||
| type: Schema.Literals(["boolean", "numeric", "set"]), | ||
|
|
@@ -2906,6 +2907,12 @@ export const V1GetPostgrestServiceConfigOutput = Schema.Struct({ | |
| }).check(Schema.isInt()), | ||
| Schema.Null, | ||
| ]), | ||
| db_pool_acquisition_timeout: Schema.Union([ | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Fresh evidence in the tree for Useful? React with 👍 / 👎. |
||
| Schema.Number.annotate({ | ||
| description: "If `null`, the value is automatically configured to 10.", | ||
| }).check(Schema.isInt()), | ||
| Schema.Null, | ||
| ]), | ||
|
jgoux marked this conversation as resolved.
jgoux marked this conversation as resolved.
Comment on lines
+2910
to
+2915
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Fresh evidence after the earlier thread: in this target tree, Useful? React with 👍 / 👎. |
||
| jwt_secret: Schema.optionalKey(Schema.String), | ||
| }); | ||
| export const V1GetProfileInput = Schema.Struct({}); | ||
|
|
@@ -5789,6 +5796,11 @@ export const V1UpdatePostgrestServiceConfigInput = Schema.Struct({ | |
| .check(Schema.isGreaterThanOrEqualTo(0)) | ||
| .check(Schema.isLessThanOrEqualTo(1000)), | ||
| ), | ||
| db_pool_acquisition_timeout: Schema.optionalKey( | ||
| Schema.Number.check(Schema.isInt()) | ||
| .check(Schema.isGreaterThanOrEqualTo(0)) | ||
| .check(Schema.isLessThanOrEqualTo(60)), | ||
| ), | ||
| }); | ||
| export const V1UpdatePostgrestServiceConfigOutput = Schema.Struct({ | ||
| db_schema: Schema.String, | ||
|
|
@@ -5800,6 +5812,12 @@ export const V1UpdatePostgrestServiceConfigOutput = Schema.Struct({ | |
| }).check(Schema.isInt()), | ||
| Schema.Null, | ||
| ]), | ||
| db_pool_acquisition_timeout: Schema.Union([ | ||
| Schema.Number.annotate({ | ||
| description: "If `null`, the value is automatically configured to 10.", | ||
| }).check(Schema.isInt()), | ||
| Schema.Null, | ||
| ]), | ||
| }); | ||
| export const V1UpdateProjectApiKeyInput = Schema.Struct({ | ||
| ref: Schema.String.check(Schema.isMinLength(20)) | ||
|
|
@@ -8792,7 +8810,13 @@ export const operationDefinitions = { | |
| requestBody: { | ||
| kind: "json", | ||
| contentType: "application/json", | ||
| fields: ["db_extra_search_path", "db_schema", "max_rows", "db_pool"], | ||
| fields: [ | ||
| "db_extra_search_path", | ||
| "db_schema", | ||
| "max_rows", | ||
| "db_pool", | ||
| "db_pool_acquisition_timeout", | ||
| ], | ||
| }, | ||
| response: { kind: "json" }, | ||
| inputSchema: V1UpdatePostgrestServiceConfigInput, | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fresh evidence after the earlier thread: in this target tree,
apps/cli/src/legacy/commands/config/push/push.integration.test.tsstill definesPOSTGREST_DISABLEDwithoutdb_pool_acquisition_timeout, and the recorded/scenarioGET /postgrestfixtures underapps/cli-e2e/fixturesstill omit it as well. Becausepackages/api/src/internal/client.tsdecodes JSON responses throughdefinition.outputSchema, making this key required makes those config-push integration/e2e replay paths fail schema decoding before reconciliation; update the mocks/fixtures or keep the response field optional during rollout.Useful? React with 👍 / 👎.