Skip to content

[New] Managed Databases #592

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

Merged
merged 27 commits into from
Feb 23, 2022
Merged

[New] Managed Databases #592

merged 27 commits into from
Feb 23, 2022

Conversation

bbiggerr
Copy link
Contributor

@bbiggerr bbiggerr commented Feb 3, 2022

No description provided.

@bbiggerr bbiggerr requested a review from yoshizzle February 23, 2022 16:01
Copy link
Collaborator

@yoshizzle yoshizzle left a comment

Choose a reason for hiding this comment

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

There was also a comment in the go/no-go about being explicit to customers that the secondary hostname (for private IP/within DC traffic) is publically viewable/accessible? I think Matt has more info on that but it should absolutely be included.

openapi.yaml Outdated
- $ref: '#/components/parameters/pageOffset'
- $ref: '#/components/parameters/pageSize'
description: |
Display information for a single Managed Database engine type.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Display information for a single Managed Database engine type.
Display information for a single Managed Database engine type and version.

openapi.yaml Outdated
beta.
responses:
'200':
description: Returns information for a single Managed Database engine.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
description: Returns information for a single Managed Database engine.
description: Returns information for a single Managed Database engine and version.

@bbiggerr bbiggerr requested a review from yoshizzle February 23, 2022 18:08
@bbiggerr bbiggerr merged commit ce0d3c8 into linode:development Feb 23, 2022
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