Skip to content
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

DOC-2727 Inventory Public api documentation #744

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

techsalvator
Copy link
Collaborator

@techsalvator techsalvator commented Feb 13, 2025

@techsalvator techsalvator requested a review from a team as a code owner February 13, 2025 14:39
anyOf:
- type: boolean
- type: 'null'
description: None returns all the properties, True returns only the default properties, False returns only the non-default properties
Copy link
Collaborator

Choose a reason for hiding this comment

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

None returns all the properties. True returns only the default properties. False returns only the non-default properties.

- type: boolean
- type: 'null'
description: None returns all the properties, True returns only the default properties, False returns only the non-default properties
description: None returns all the properties, True returns only the default properties, False returns only the non-default properties
Copy link
Collaborator

Choose a reason for hiding this comment

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

None returns all the properties. True returns only the default properties. False returns only the non-default properties.

anyOf:
- type: boolean
- type: 'null'
description: None returns all the properties, True returns only the selectable properties, False returns only the non-selectable properties
Copy link
Collaborator

Choose a reason for hiding this comment

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

None returns all the properties. True returns only the properties that can be selected. False returns only the properties that cannot be selected.

- type: boolean
- type: 'null'
description: None returns all the properties, True returns only the selectable properties, False returns only the non-selectable properties
description: None returns all the properties, True returns only the selectable properties, False returns only the non-selectable properties
Copy link
Collaborator

Choose a reason for hiding this comment

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

None returns all the properties. True returns only the properties that can be selected. False returns only the properties that cannot be selected.

- type: string
format: date-time
- type: 'null'
description: If specified, snapshot greater than this timestamp.
Copy link
Collaborator

@RachelHooley RachelHooley Feb 17, 2025

Choose a reason for hiding this comment

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

If specified, the inventory must have a snapshot after this timestamp.

format: date-time
- type: 'null'
description: If specified, snapshot greater than this timestamp.
description: If specified, snapshot greater than this timestamp.
Copy link
Collaborator

@RachelHooley RachelHooley Feb 17, 2025

Choose a reason for hiding this comment

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

If specified, the inventory must have a snapshot after this timestamp.

- type: string
format: date-time
- type: 'null'
description: If specified, snapshot less than this timestamp.
Copy link
Collaborator

@RachelHooley RachelHooley Feb 17, 2025

Choose a reason for hiding this comment

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

If specified, the inventory must have a snapshot before this timestamp.

format: date-time
- type: 'null'
description: If specified, snapshot less than this timestamp.
description: If specified, snapshot less than this timestamp.
Copy link
Collaborator

@RachelHooley RachelHooley Feb 17, 2025

Choose a reason for hiding this comment

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

If specified, the inventory must have a snapshot before this timestamp.

tags:
- Inventory Resources
summary: Get Resources
description: Returns a list of resources processed in the given snapshot ID and service ID.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Returns a list of resources processed in the snapshot ID and service ID.

required: false
schema:
type: boolean
description: Include the common properties in addition to any properties specified in select
Copy link
Collaborator

Choose a reason for hiding this comment

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

what does it mean - in select?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@RachelHooley In select is referring to another parameter. Line #57

Copy link
Collaborator

@RachelHooley RachelHooley Feb 19, 2025

Choose a reason for hiding this comment

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

@techsalvator Include the common properties, in addition to the properties you chose for the resource.
?

type: boolean
description: Include the common properties in addition to any properties specified in select
default: true
description: Include the common properties in addition to any properties specified in select
Copy link
Collaborator

Choose a reason for hiding this comment

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

what does it mean - in select?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@RachelHooley In select is referring to another parameter. Line #57

Copy link
Collaborator

Choose a reason for hiding this comment

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

@techsalvator Include the common properties, in addition to the properties you chose for the resource.
?

required: false
schema:
type: boolean
description: Include the data collected from the cloud provider (as is); structure not guaranteed.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Include the data from the cloud provider as is.

type: boolean
description: Include the data collected from the cloud provider (as is); structure not guaranteed.
default: false
description: Include the data collected from the cloud provider (as is); structure not guaranteed.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Include the data from the cloud provider as is.

snapshotDate:
type: string
format: date-time
description: Timestamp used to "best represent" the point in time that the data being viewed represents.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not sure I understand this one. is it:
The time the data was viewed.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@RachelHooley Timestamp represents time the data was collected from customer

Copy link
Collaborator

Choose a reason for hiding this comment

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

@techsalvator When the snapshot was taken from the ____
this is for the customer so we can't say customer. where does the data come from?

Copy link
Collaborator

Choose a reason for hiding this comment

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

The time the data was pulled from your cloud account.

properties:
select:
type: boolean
description: Property can be selected in the inventory list API.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not sure I understand this one. is it:
This property can be selected in the inventory list API.

description: Property can be selected in the inventory list API.
filter:
type: boolean
description: Property can be filtered in the inventory list API.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not sure I understand this one. is it:
This property can be selected in the inventory list API.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@RachelHooley This property can be filtered in the inventory list API.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@techsalvator You can filter the inventories according to this field.
?

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.

2 participants