-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Custom Framework API Spec #28798
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
Custom Framework API Spec #28798
Conversation
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.
Hi @nkonjeti there are some formatting snafu's that need fixed along with requested changes in https://github.com/DataDog/datadog-api-spec/pull/3739.
Let me know once fixed and we can re-review. Thanks!
data/api/v2/full_spec.yaml
Outdated
origin: request | ||
source: data.attributes.version | ||
type: unsafe | ||
x-unstable: '**Note**: This endpoint is in beta and may be subject to changes. |
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.
x-unstable: '**Note**: This endpoint is in beta and may be subject to changes. | |
x-unstable: **Note**: This endpoint is in Preview and may be subject to changes. |
data/api/v2/full_spec.yaml
Outdated
type: unsafe | ||
x-unstable: '**Note**: This endpoint is in beta and may be subject to changes. | ||
|
||
Please check the documentation regularly for updates.' |
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.
Please check the documentation regularly for updates.' | |
Please check the documentation regularly for updates. |
data/api/v2/full_spec.yaml
Outdated
- security_monitoring_rules_write | ||
x-undo: | ||
type: idempotent | ||
x-unstable: '**Note**: This endpoint is in beta and may be subject to changes. |
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.
x-unstable: '**Note**: This endpoint is in beta and may be subject to changes. | |
x-unstable: **Note**: This endpoint is in beta and may be subject to changes. |
data/api/v2/full_spec.yaml
Outdated
- security_monitoring_rules_write | ||
x-undo: | ||
type: idempotent | ||
x-unstable: '**Note**: This endpoint is in beta and may be subject to changes. |
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.
x-unstable: '**Note**: This endpoint is in beta and may be subject to changes. | |
x-unstable: **Note**: This endpoint is in beta and may be subject to changes. |
data/api/v2/full_spec.yaml
Outdated
type: idempotent | ||
x-unstable: '**Note**: This endpoint is in beta and may be subject to changes. | ||
|
||
Please check the documentation regularly for updates.' |
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.
Please check the documentation regularly for updates.' | |
Please check the documentation regularly for updates. |
data/api/v2/full_spec_deref.json
Outdated
} | ||
"type": "idempotent" | ||
}, | ||
"x-unstable": "**Note**: This endpoint is in beta and may be subject to changes.\nPlease check the documentation regularly for updates." |
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.
"x-unstable": "**Note**: This endpoint is in beta and may be subject to changes.\nPlease check the documentation regularly for updates." | |
"x-unstable": **Note**: This endpoint is in Preview and may be subject to changes.\nPlease check the documentation regularly for updates." |
data/api/v2/full_spec_deref.json
Outdated
], | ||
"type": "unsafe" | ||
}, | ||
"x-unstable": "**Note**: This endpoint is in beta and may be subject to changes.\nPlease check the documentation regularly for updates." |
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.
"x-unstable": "**Note**: This endpoint is in beta and may be subject to changes.\nPlease check the documentation regularly for updates." | |
"x-unstable": **Note**: This endpoint is in beta and may be subject to changes.\nPlease check the documentation regularly for updates." |
data/api/v2/full_spec_deref.json
Outdated
"x-undo": { | ||
"type": "idempotent" | ||
}, | ||
"x-unstable": "**Note**: This endpoint is in beta and may be subject to changes.\nPlease check the documentation regularly for updates." |
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.
"x-unstable": "**Note**: This endpoint is in beta and may be subject to changes.\nPlease check the documentation regularly for updates." | |
"x-unstable": **Note**: This endpoint is in beta and may be subject to changes.\nPlease check the documentation regularly for updates." |
cef2bb3
to
bf2b140
Compare
cca0147
to
5496930
Compare
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.
One small comment in the DeleteCustomFramework
that is missing a description, but approved
"json": { | ||
"data": { | ||
"attributes": { | ||
"description": "this is an example description", |
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.
"description": "this is an example description", | |
"description": "this is an example description", |
Small suggestion, should this be filled out with the actual description ?
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.
oh the description was "this is an example description" I will try to think of a better example
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.
ok thanks for the context there! I think this is probably fine, I was just missing the context.
5496930
to
b80f645
Compare
b80f645
to
4bf4fd4
Compare
Co-authored-by: ci.datadog-api-spec <[email protected]>
See DataDog/datadog-api-spec#3739
Test branch datadog-api-spec/test/vbarth/neha.konjeti/custom-framework