Skip to content

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