-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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
Manual changes to aws_ssm_maintenance_window schedule value are not reverted by terraform #14653
Labels
Comments
stack72
added a commit
that referenced
this issue
May 19, 2017
Fixes: #14653 I was originally calling the wrong API method and only some of the values were being persisted to state. By changing the API method, we can now get all of the values and therefore can detech manual drift ``` % make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSSSMMaintenanceWindow_' ==> Checking that code complies with gofmt requirements... go generate $(go list ./... | grep -v /terraform/vendor/) 2017/05/19 16:56:27 Generated command/internal_plugin_list.go TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSSSMMaintenanceWindow_ -timeout 120m === RUN TestAccAWSSSMMaintenanceWindow_basic --- PASS: TestAccAWSSSMMaintenanceWindow_basic (41.39s) PASS ok github.com/hashicorp/terraform/builtin/providers/aws 41.419s ```
stack72
added a commit
that referenced
this issue
May 19, 2017
Fixes: #14653 I was originally calling the wrong API method and only some of the values were being persisted to state. By changing the API method, we can now get all of the values and therefore can detech manual drift ``` % make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSSSMMaintenanceWindow_' ==> Checking that code complies with gofmt requirements... go generate $(go list ./... | grep -v /terraform/vendor/) 2017/05/19 16:56:27 Generated command/internal_plugin_list.go TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSSSMMaintenanceWindow_ -timeout 120m === RUN TestAccAWSSSMMaintenanceWindow_basic --- PASS: TestAccAWSSSMMaintenanceWindow_basic (41.39s) PASS ok github.com/hashicorp/terraform/builtin/providers/aws 41.419s ```
I have just pushed a PR for this right now and I can see it behaves as follows:
|
stack72
added a commit
that referenced
this issue
May 19, 2017
Fixes: #14653 I was originally calling the wrong API method and only some of the values were being persisted to state. By changing the API method, we can now get all of the values and therefore can detech manual drift ``` % make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSSSMMaintenanceWindow_' ==> Checking that code complies with gofmt requirements... go generate $(go list ./... | grep -v /terraform/vendor/) 2017/05/19 16:56:27 Generated command/internal_plugin_list.go TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSSSMMaintenanceWindow_ -timeout 120m === RUN TestAccAWSSSMMaintenanceWindow_basic --- PASS: TestAccAWSSSMMaintenanceWindow_basic (41.39s) PASS ok github.com/hashicorp/terraform/builtin/providers/aws 41.419s ```
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
ghost
locked and limited conversation to collaborators
Apr 12, 2020
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Terraform Version
0.9.5
Affected Resource(s)
Terraform Configuration Files
Expected Behavior
If the maintenance window's schedule value is modified in the AWS console it should be reverted back to the value in the terraform config file on the next apply.
Actual Behavior
Manual changes to the schedule value in the AWS console are not reverted by terraform.
Steps to Reproduce
Please list the steps required to reproduce the issue, for example:
terraform apply
terraform plan
- no changes are foundterraform apply
- the cron value is unchanged and does not match the value in the configuration fileReferences
The text was updated successfully, but these errors were encountered: