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

resource/alicloud_ess_scheduled_task: update launch_expiration_time range. #8028

Merged
merged 1 commit into from
Dec 26, 2024

Conversation

fuliu-zln
Copy link
Contributor

No description provided.

@fuliu-zln fuliu-zln force-pushed the feat/task branch 3 times, most recently from 4a8b196 to b9a7a3f Compare December 25, 2024 02:15
@@ -47,7 +47,7 @@ func resourceAlicloudEssScheduledTask() *schema.Resource {
Type: schema.TypeInt,
Default: 600,
Optional: true,
ValidateFunc: validation.IntBetween(0, 21600),
ValidateFunc: validation.IntBetween(0, 1800),
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个缩减了是非兼容的,会导致客户模板报错。另外直接使用common下的IntBetween即可

Copy link
Contributor Author

@fuliu-zln fuliu-zln Dec 25, 2024

Choose a reason for hiding this comment

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

1.API 后端的约束目前就是0-1800 之前的应该是之前接入有问题

  1. 使用common下的IntBetween即可 done

@fuliu-zln fuliu-zln force-pushed the feat/task branch 2 times, most recently from a560651 to 40454f5 Compare December 25, 2024 07:13
@@ -133,7 +133,7 @@ The following arguments are supported:
The time must be in UTC. You cannot enter a time point later than 90 days from the date of scheduled task creation.
If the `recurrence_type` parameter is specified, the task is executed repeatedly at the time specified by LaunchTime.
Otherwise, the task is only executed once at the date and time specified by LaunchTime.
* `launch_expiration_time` - (Optional) The time period during which a failed scheduled task is retried. Unit: seconds. Valid values: 0 to 21600. Default value: 600
* `launch_expiration_time` - (Optional) The time period during which a failed scheduled task is retried. Unit: seconds. Valid values: 0 to 1800. Default value: 600
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个变更加个说明吧,从哪个版本起取值范围缩减了

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -133,7 +133,7 @@ The following arguments are supported:
The time must be in UTC. You cannot enter a time point later than 90 days from the date of scheduled task creation.
If the `recurrence_type` parameter is specified, the task is executed repeatedly at the time specified by LaunchTime.
Otherwise, the task is only executed once at the date and time specified by LaunchTime.
* `launch_expiration_time` - (Optional) The time period during which a failed scheduled task is retried. Unit: seconds. Valid values: 0 to 21600. Default value: 600
* `launch_expiration_time` - (Optional) The time period during which a failed scheduled task is retried. Unit: seconds. Valid values: 0 to 1800, Available in 1.239.0. Default value: 600
Copy link
Collaborator

Choose a reason for hiding this comment

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

Available since v1.240.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@fuliu-zln fuliu-zln force-pushed the feat/task branch 2 times, most recently from 24811a6 to 46735ee Compare December 26, 2024 02:39
Copy link
Collaborator

@MrWolong MrWolong left a comment

Choose a reason for hiding this comment

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

approved

@fuliu-zln fuliu-zln force-pushed the feat/task branch 2 times, most recently from c2cec51 to 1ba7452 Compare December 26, 2024 07:04
Copy link
Collaborator

@MrWolong MrWolong left a comment

Choose a reason for hiding this comment

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

approved

@ChenHanZhang ChenHanZhang merged commit ab2e8a1 into aliyun:master Dec 26, 2024
18 of 19 checks passed
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.

3 participants