You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
$ go mod edit -json | jq '.Require[] | select(.Path=="github.com/hashicorp/terraform-plugin-sdk")'
{
"Path": "github.com/hashicorp/terraform-plugin-sdk",
"Version": "v1.4.0"
}
Use-cases
When running something like
$ TEST=./aws SWEEP=us-west-2 SWEEPARGS=-sweep-run=aws_xxx make sweep
the sweeper for resources like aws_xxx_attachment are run even if they are not declared as dependencies of aws_xxx.
This is because filterSweepers() uses strings.Contains() on the sweeper name to determine if it should be run.
Changing this to a regexp match would allow exact matches (^aws_xxx$) as well as being backwards compatible.
The text was updated successfully, but these errors were encountered:
One thing to consider if you look to open a PR (definitely welcome 🙂) is how to handle potential commas in regex statements, since -sweep-run is a comma separated list. We could also consider a new complimentary -sweep-run-regex flag if we want to avoid that problem.
SDK version
Use-cases
When running something like
the sweeper for resources like
aws_xxx_attachment
are run even if they are not declared as dependencies ofaws_xxx
.This is because
filterSweepers()
usesstrings.Contains()
on the sweeper name to determine if it should be run.Changing this to a regexp match would allow exact matches (
^aws_xxx$
) as well as being backwards compatible.The text was updated successfully, but these errors were encountered: