添加快速查询资源 #11
添加快速查询资源 #11
Conversation
WalkthroughWalkthrough新增了一个名为 Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Provider
participant LogService
User ->> Provider: 提交创建/更新保存搜索请求
Provider ->> LogService: 调用DescribeSavedSearch方法
LogService -->> Provider: 返回保存搜索详情
Provider ->> LogService: 调用WaitForSavedSearch方法
LogService -->> Provider: 返回搜索状态
Provider -->> User: 返回操作结果
Poem
TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
8df0321 to
e75c11c
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- alicloud/provider.go (1 hunks)
- alicloud/resource_alicloud_log_savedsearch.go (1 hunks)
- alicloud/service_alicloud_log.go (1 hunks)
Additional comments not posted (8)
alicloud/resource_alicloud_log_savedsearch.go (5)
14-54: 资源 schema 定义和 CRUD 操作绑定资源
alicloud_log_saved_search的 schema 定义和 CRUD 操作绑定看起来很好。所有必要的字段都已包含，并且类型和约束也合适。
56-97: 创建资源函数
resourceAlicloudLogSavedSearchCreate函数处理保存搜索资源的创建。函数使用 SDK 创建保存搜索，并包含针对临时错误的重试逻辑。错误处理和日志记录都很到位。请确保
WaitForSavedSearch函数的使用是正确的。
99-123: 读取资源函数
resourceAlicloudLogSavedSearchRead函数处理保存搜索资源的读取。函数使用 SDK 检索保存搜索并设置资源数据字段。错误处理和字段设置都很到位。请确保错误处理是正确的。
125-161: 更新资源函数
resourceAlicloudLogSavedSearchUpdate函数处理保存搜索资源的更新。函数检查相关字段的更改，并使用 SDK 更新保存搜索。错误处理和更改检测逻辑都很到位。请确保更改检测逻辑是正确的。
163-195: 删除资源函数
resourceAlicloudLogSavedSearchDelete函数处理保存搜索资源的删除。函数使用 SDK 删除保存搜索，并包含针对临时错误的重试逻辑。错误处理和日志记录都很到位。请确保
WaitForSavedSearch函数的使用是正确的。alicloud/service_alicloud_log.go (2)
556-596: 描述保存搜索函数
DescribeSavedSearch函数使用 SDK 检索保存搜索，并包含针对临时错误的重试逻辑。错误处理和日志记录都很到位。请确保错误处理是正确的。
598-621: 等待保存搜索函数
WaitForSavedSearch函数等待保存搜索在给定超时时间内达到指定状态。错误处理和日志记录都很到位。请确保超时逻辑是正确的。
alicloud/provider.go (1)
995-995: 新增资源alicloud_log_saved_search已成功添加新资源
alicloud_log_saved_search，符合现有的资源映射结构。
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- alicloud/provider.go (1 hunks)
- alicloud/resource_alicloud_log_savedsearch.go (1 hunks)
- alicloud/service_alicloud_log.go (1 hunks)
Files skipped from review due to trivial changes (1)
- alicloud/provider.go
Additional comments not posted (7)
alicloud/resource_alicloud_log_savedsearch.go (5)
14-54: 定义资源架构和CRUD操作资源架构和CRUD操作定义看起来是正确的。
99-123: 读取资源数据读取资源数据的实现看起来是正确的，包括错误处理。
125-161: 更新资源数据更新资源数据的实现看起来是正确的，检查特定字段的变化并相应地更新资源。
163-195: 删除资源时验证WaitForSavedSearch的使用删除资源的实现看起来是正确的，包括错误处理和重试机制。
但是，请确保
WaitForSavedSearch的使用是正确的。Verification successful
删除资源时验证
WaitForSavedSearch的使用删除资源的实现看起来是正确的，包括错误处理和重试机制。
WaitForSavedSearch函数的实现也符合预期。
WaitForSavedSearch函数的实现位于alicloud/service_alicloud_log.go文件的第 598 行。Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the implementation of WaitForSavedSearch function. # Test: Search for the function implementation. Expect: Correct implementation of WaitForSavedSearch. rg --type go $'func (s *LogService) WaitForSavedSearch'Length of output: 56
Script:
#!/bin/bash # Description: Verify the implementation of WaitForSavedSearch function or similar functions. # Test: Search for all functions in LogService. Expect: To find WaitForSavedSearch or similar functions. rg --type go --pcre2 'func \(s \*LogService\) \w+'Length of output: 4551
Script:
#!/bin/bash # Description: Retrieve the implementation details of WaitForSavedSearch function. # Test: Get the implementation of the WaitForSavedSearch function in LogService. ast-grep --lang go --pattern $'func (s *LogService) WaitForSavedSearch($_, $_, $_) error { $$$ }'Length of output: 1694
56-97: 创建资源时验证WaitForSavedSearch的使用创建资源的实现看起来是正确的，包括错误处理和重试机制。
但是，请确保
WaitForSavedSearch的使用是正确的。alicloud/service_alicloud_log.go (2)
556-596: 检索保存的搜索检索保存的搜索的实现看起来是正确的，包括错误处理和重试机制。
598-621: 等待保存的搜索达到指定状态等待保存的搜索达到指定状态的实现看起来是正确的，包括错误处理和循环检查状态直到超时。
There was a problem hiding this comment.
Actionable comments posted: 4
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- alicloud/provider.go (1 hunks)
- alicloud/resource_alicloud_log_savedsearch.go (1 hunks)
- alicloud/service_alicloud_log.go (1 hunks)
Files skipped from review due to trivial changes (1)
- alicloud/provider.go
Additional comments not posted (3)
alicloud/resource_alicloud_log_savedsearch.go (3)
14-54: 定义资源 schema 成功资源 schema 定义良好，包含必要的和可选的字段。资源具有创建、读取、更新、删除操作。
99-123: 读取资源操作成功函数结构良好，适当处理了错误情况。
126-161: 更新资源操作成功函数正确检查更改并更新资源，同时适当处理错误。
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- alicloud/provider.go (1 hunks)
- alicloud/resource_alicloud_log_savedsearch.go (1 hunks)
- alicloud/service_alicloud_log.go (1 hunks)
Additional comments not posted (8)
alicloud/resource_alicloud_log_savedsearch.go (5)
14-54: 资源定义实现正确
resourceAlicloudLogSavedSearch函数定义了alicloud_log_saved_search资源的 schema 和 CRUD 操作，字段和属性配置正确。
56-97: 创建资源函数实现正确
resourceAlicloudLogSavedSearchCreate函数处理alicloud_log_saved_search资源的创建操作，包含重试逻辑和错误处理，逻辑实现正确。
99-123: 读取资源函数实现正确
resourceAlicloudLogSavedSearchRead函数处理alicloud_log_saved_search资源的读取操作，检索已保存的搜索并正确设置 schema 字段。
126-161: 更新资源函数实现正确
resourceAlicloudLogSavedSearchUpdate函数处理alicloud_log_saved_search资源的更新操作，检查特定字段的更改并相应地更新已保存的搜索。
163-195: 删除资源函数实现正确
resourceAlicloudLogSavedSearchDelete函数处理alicloud_log_saved_search资源的删除操作，包含重试逻辑和错误处理，逻辑实现正确。alicloud/service_alicloud_log.go (2)
556-596: 描述已保存搜索函数实现正确
DescribeSavedSearch函数通过 ID 检索已保存的搜索，包含重试逻辑和错误处理，逻辑实现正确。
598-621: 等待已保存搜索函数实现正确
WaitForSavedSearch函数在超时时间内等待已保存的搜索达到指定状态，包含重试逻辑和错误处理，逻辑实现正确。alicloud/provider.go (1)
995-995: 新增资源alicloud_log_saved_search看起来不错！该资源已正确添加到
ResourcesMap中。
e75c11c to
7aa2f3c
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- alicloud/provider.go (1 hunks)
- alicloud/resource_alicloud_log_savedsearch.go (1 hunks)
- alicloud/service_alicloud_log.go (1 hunks)
Files skipped from review as they are similar to previous changes (2)
- alicloud/provider.go
- alicloud/resource_alicloud_log_savedsearch.go
Additional comments not posted (2)
alicloud/service_alicloud_log.go (2)
556-596: 建议添加日志记录函数实现了全面的检索操作，包括重试和错误处理。但缺少日志记录，建议添加日志以便更好地追踪。
// 在检索操作中添加日志记录 log.Printf("[INFO] Describing Alicloud Log Saved Search: %s", id)
598-621: 建议添加日志记录函数实现了全面的等待操作，包括超时和错误处理。但缺少日志记录，建议添加日志以便更好地追踪。
// 在等待操作中添加日志记录 log.Printf("[INFO] Waiting for Alicloud Log Saved Search: %s to reach status: %s", id, status)
7aa2f3c to
074631f
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- alicloud/provider.go (1 hunks)
- alicloud/resource_alicloud_log_savedsearch.go (1 hunks)
- alicloud/service_alicloud_log.go (1 hunks)
Files skipped from review as they are similar to previous changes (3)
- alicloud/provider.go
- alicloud/resource_alicloud_log_savedsearch.go
- alicloud/service_alicloud_log.go
074631f to
4ceae76
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- alicloud/provider.go (1 hunks)
- alicloud/resource_alicloud_log_savedsearch.go (1 hunks)
- alicloud/service_alicloud_log.go (1 hunks)
Files skipped from review as they are similar to previous changes (3)
- alicloud/provider.go
- alicloud/resource_alicloud_log_savedsearch.go
- alicloud/service_alicloud_log.go
4ceae76 to
fb37e9c
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (4)
- alicloud/provider.go (1 hunks)
- alicloud/resource_alicloud_log_saved_search.go (1 hunks)
- alicloud/service_alicloud_log.go (1 hunks)
- website/docs/r/log_saved_search.html.markdown (1 hunks)
Files skipped from review as they are similar to previous changes (2)
- alicloud/provider.go
- alicloud/service_alicloud_log.go
Additional comments not posted (9)
website/docs/r/log_saved_search.html.markdown (4)
1-8: 元数据字段正确文档头部的元数据字段设置正确。
10-15: 描述准确描述部分准确地介绍了
alicloud_log_saved_search资源的功能。
16-37: 示例用法正确示例用法部分提供了一个完整的 Terraform 配置，展示了如何使用
alicloud_log_saved_search资源。
40-55: 参数和属性描述正确参数参考和属性参考部分准确且完整地描述了
alicloud_log_saved_search资源的参数和导出的属性。alicloud/resource_alicloud_log_saved_search.go (5)
1-51: 资源定义正确
resourceAlicloudLogSavedSearch函数正确定义了alicloud_log_saved_search资源的 schema 和 CRUD 操作绑定。
53-93: 创建逻辑正确
resourceAlicloudLogSavedSearchCreate函数正确地处理了保存搜索资源的创建逻辑，包括错误处理和重试逻辑。
95-119: 读取逻辑正确
resourceAlicloudLogSavedSearchRead函数正确地处理了保存搜索资源的读取逻辑，包括错误处理和状态设置。
121-160: 更新逻辑正确
resourceAlicloudLogSavedSearchUpdate函数正确地处理了保存搜索资源的更新逻辑，包括变更检测和更新执行。
162-194: 删除逻辑正确
resourceAlicloudLogSavedSearchDelete函数正确地处理了保存搜索资源的删除逻辑，包括错误处理和重试逻辑。
背景
cdk项目需要添加快速查询，但是官方terraform插件里并未提供相关资源
方案
依靠sdk已有的快速查询相关接口，参照logdashboard的相关代码结构新增快速查询资源
Summary by CodeRabbit
新功能
alicloud_log_saved_search资源，用于管理阿里云日志服务中的已保存搜索功能。文档
alicloud_log_saved_search资源的使用指南，包括创建和使用已保存搜索进行查询和分析操作的说明。