-
Notifications
You must be signed in to change notification settings - Fork 677
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
added in preliminary changes for v2 atracker #3724
Conversation
|
||
## Attribute reference | ||
|
||
In addition to all argument references listed, you can access the following attribute references after your resource is created. | ||
|
||
* `id` - The unique identifier of the Activity Tracker Route. |
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.
Generally all attributes shd not have (optional /Required) they are computed may be can you remove that for all attributes..something like below
created - (String) The timestamp of the route creation time. api_version
- (Integer) The API version of the route.
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.
same for all docs
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.
Done
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.
I still see the same
updated_at
- (Required, String) The timestamp of the route last updated time.
For computed we will not have either Required/Optional
it should be something like below
updated_at
- (String) The timestamp of the route last updated time.
* added in preliminary changes for v2 atracker * allow redacted key saves * formatted * fixes acceptance tests * removed debugging log * allow secret * another fake secret * added review corrections * removed required flag * fixes routes not being returned during migration * additional fixes for routes usage in v1 * added example for logdna target * fix for settings acting like put instead of patch * fixes crn constantly showing up as a change as well as region * fix for route always showing recieve global event change * fix route flag being used * added review comments * Only add region if provided * Documentation update * take it from the response instead; Gave better error when ENV are missing * adjusted setting region * defaults empty strings in arrays * env update here as well * fixed comment changes as well as route workflow * Fix route apply * clean up * force new instead of applyonce * forces new target on region change * doc update and update fix for routes * Final clean up additions from acceptance tests * prevent null api numver Co-authored-by: Van Chau Bui <van.bui@ibm.com>
* added in preliminary changes for v2 atracker * allow redacted key saves * formatted * fixes acceptance tests * removed debugging log * allow secret * another fake secret * added review corrections * removed required flag * fixes routes not being returned during migration * additional fixes for routes usage in v1 * added example for logdna target * fix for settings acting like put instead of patch * fixes crn constantly showing up as a change as well as region * fix for route always showing recieve global event change * fix route flag being used * added review comments * Only add region if provided * Documentation update * take it from the response instead; Gave better error when ENV are missing * adjusted setting region * defaults empty strings in arrays * env update here as well * fixed comment changes as well as route workflow * Fix route apply * clean up * force new instead of applyonce * forces new target on region change * doc update and update fix for routes * Final clean up additions from acceptance tests * prevent null api numver Co-authored-by: Van Chau Bui <van.bui@ibm.com>
What has been done?
Acceptance Tests Results
Community Note
Relates OR Closes #0000
Output from acceptance testing: