-
Notifications
You must be signed in to change notification settings - Fork 143
Feat/bi support #2239
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
Feat/bi support #2239
Conversation
4db3c20
to
09a1c54
Compare
ctx := context.WithValue(context.TODO(), logIdKey, logId) | ||
|
||
paramMap := make(map[string]interface{}) | ||
if v, _ := d.GetOk("page_no"); v != nil { |
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.
这里不需要使用 GetOkExists吗
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.
1
} | ||
|
||
if v, _ := d.GetOk("all_page"); v != nil { | ||
paramMap["AllPage"] = helper.Bool(v.(bool)) |
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.
这里不需要使用 GetOkExists吗
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.
1
a4de4fb
to
402c2e2
Compare
ctx := context.WithValue(context.TODO(), logIdKey, logId) | ||
|
||
paramMap := make(map[string]interface{}) | ||
if v, _ := d.GetOk("project_id"); v != nil { |
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.
这里是否需要使用 GetOkExists
paramMap["ProjectId"] = helper.IntInt64(v.(int)) | ||
} | ||
|
||
if v, _ := d.GetOk("all_page"); v != nil { |
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.
这里是否需要使用 GetOkExists
Config: testAccBiDatasourceCloud, | ||
Check: resource.ComposeTestCheckFunc( | ||
resource.TestCheckResourceAttrSet("tencentcloud_bi_datasource_cloud.datasource_cloud", "id"), | ||
), |
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.
补充下 Import 和 update 的 e2e
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.
这个查询接口查询不到vip、vport、vpc_id,无法支持import,e2e不可执行,涉及密码问题
response = bi.NewApplyEmbedIntervalResponse() | ||
biToken string | ||
) | ||
if v, _ := d.GetOk("project_id"); v != nil { |
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.
这里是否需要使用 GetOkExists
request.ProjectId = helper.IntUint64(v.(int)) | ||
} | ||
|
||
if v, _ := d.GetOk("page_id"); v != nil { |
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.
这里是否需要使用 GetOkExists
Example Usage | ||
|
||
```hcl | ||
resource "tencentcloud_bi_embed_interval" "embed_interval" { |
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.
这个是一个operation资源吗,名称上好像不太容易区分
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.
根据接口名,我没想到更好的
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.
resource_tc_bi_token_extend这个可以吗
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.
这个资源的作用是什么呢
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.
延长临时token时间
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.
跟下面的那个 operation 对应,这个是不是可以叫 resource_tc_bi_embed_interval_apply
下面那个叫 resource_tc_bi_embed_token_apply
Example Usage | ||
|
||
```hcl | ||
resource "tencentcloud_bi_embed_token" "embed_token" { |
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.
operation资源是吧,同上
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.
resource_tc_bi_token_apply可以吗
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.
resource_tc_bi_token_apply可以吗
嗯,这个看起来比较容易理解些
response = bi.NewCreateEmbedTokenResponse() | ||
pageId int | ||
) | ||
if v, _ := d.GetOk("project_id"); v != nil { |
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.
这里是否需要使用 GetOkExists
request.ProjectId = helper.IntUint64(v.(int)) | ||
} | ||
|
||
if v, _ := d.GetOk("page_id"); v != nil { |
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.
这里是否需要使用 GetOkExists
), | ||
}, | ||
{ | ||
Config: testAccBiProjectUserRole, |
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.
这个测试步骤是重复了吗
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.
恢复数据使用,
16467c3
to
0d466cd
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.
LGTM
No description provided.