-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
r/aws_cloudwatch_event_target & r/aws_cloudwatch_event_rule: Support partner event bus names #18491
Closed
ewbankkit
wants to merge
13
commits into
hashicorp:main
from
ewbankkit:f-aws_cloudwatch_event_rule-partner-event
Closed
r/aws_cloudwatch_event_target & r/aws_cloudwatch_event_rule: Support partner event bus names #18491
ewbankkit
wants to merge
13
commits into
hashicorp:main
from
ewbankkit:f-aws_cloudwatch_event_rule-partner-event
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ghost
added
size/XL
Managed by automation to categorize the size of a PR.
documentation
Introduces or discusses updates to documentation.
provider
Pertains to the provider itself, rather than any interaction with AWS.
service/cloudwatchevents
tests
PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
labels
Mar 30, 2021
…erEventBus'. Acceptance test output: $ EVENT_BRIDGE_PARTNER_EVENT_BUS_NAME=aws.partner/datadog.com/TerraformTesting make testacc TEST=./aws/ TESTARGS='-run=TestAccAWSCloudWatchEventRule_PartnerEventBus' ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAWSCloudWatchEventRule_PartnerEventBus -timeout 180m === RUN TestAccAWSCloudWatchEventRule_PartnerEventBus === PAUSE TestAccAWSCloudWatchEventRule_PartnerEventBus === CONT TestAccAWSCloudWatchEventRule_PartnerEventBus resource_aws_cloudwatch_event_rule_test.go:495: Step 1/2 error: Error running apply: exit status 1 2021/03/29 13:31:03 [DEBUG] Using modified User-Agent: Terraform/0.12.26 HashiCorp-terraform-exec/0.13.0 Error: error reading CloudWatch Events Rule (aws.partner/datadog.com/TerraformTesting/tf-acc-test-rule-500421321801107394): unexpected format for ID ("aws.partner/datadog.com/TerraformTesting/tf-acc-test-rule-500421321801107394"), expected <event-bus-name>/<rule-name> or <rule-name> on terraform_plugin_test.tf line 2, in resource "aws_cloudwatch_event_rule" "test": 2: resource "aws_cloudwatch_event_rule" "test" { testing_new.go:63: Error running post-test destroy, there may be dangling resources: exit status 1 2021/03/29 13:31:05 [DEBUG] Using modified User-Agent: Terraform/0.12.26 HashiCorp-terraform-exec/0.13.0 Error: unexpected format for ID ("aws.partner/datadog.com/TerraformTesting/tf-acc-test-rule-500421321801107394"), expected <event-bus-name>/<rule-name> or <rule-name> --- FAIL: TestAccAWSCloudWatchEventRule_PartnerEventBus (5.18s) FAIL FAIL github.com/terraform-providers/terraform-provider-aws/aws 5.283s FAIL GNUmakefile:27: recipe for target 'testacc' failed make: *** [testacc] Error 1
Acceptance test output: $ EVENT_BRIDGE_PARTNER_EVENT_BUS_NAME=aws.partner/datadog.com/TerraformTesting make testacc TEST=./aws/ TESTARGS='-run=TestAccAWSCloudWatchEventRule_' ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAWSCloudWatchEventRule_ -timeout 180m === RUN TestAccAWSCloudWatchEventRule_basic === PAUSE TestAccAWSCloudWatchEventRule_basic === RUN TestAccAWSCloudWatchEventRule_EventBusName === PAUSE TestAccAWSCloudWatchEventRule_EventBusName === RUN TestAccAWSCloudWatchEventRule_role === PAUSE TestAccAWSCloudWatchEventRule_role === RUN TestAccAWSCloudWatchEventRule_description === PAUSE TestAccAWSCloudWatchEventRule_description === RUN TestAccAWSCloudWatchEventRule_pattern === PAUSE TestAccAWSCloudWatchEventRule_pattern === RUN TestAccAWSCloudWatchEventRule_ScheduleAndPattern === PAUSE TestAccAWSCloudWatchEventRule_ScheduleAndPattern === RUN TestAccAWSCloudWatchEventRule_NamePrefix === PAUSE TestAccAWSCloudWatchEventRule_NamePrefix === RUN TestAccAWSCloudWatchEventRule_Name_Generated === PAUSE TestAccAWSCloudWatchEventRule_Name_Generated === RUN TestAccAWSCloudWatchEventRule_tags === PAUSE TestAccAWSCloudWatchEventRule_tags === RUN TestAccAWSCloudWatchEventRule_IsEnabled === PAUSE TestAccAWSCloudWatchEventRule_IsEnabled === RUN TestAccAWSCloudWatchEventRule_PartnerEventBus === PAUSE TestAccAWSCloudWatchEventRule_PartnerEventBus === CONT TestAccAWSCloudWatchEventRule_basic === CONT TestAccAWSCloudWatchEventRule_NamePrefix === CONT TestAccAWSCloudWatchEventRule_PartnerEventBus === CONT TestAccAWSCloudWatchEventRule_IsEnabled === CONT TestAccAWSCloudWatchEventRule_tags === CONT TestAccAWSCloudWatchEventRule_Name_Generated === CONT TestAccAWSCloudWatchEventRule_description === CONT TestAccAWSCloudWatchEventRule_ScheduleAndPattern === CONT TestAccAWSCloudWatchEventRule_pattern === CONT TestAccAWSCloudWatchEventRule_role === CONT TestAccAWSCloudWatchEventRule_EventBusName --- PASS: TestAccAWSCloudWatchEventRule_Name_Generated (50.87s) --- PASS: TestAccAWSCloudWatchEventRule_PartnerEventBus (51.28s) --- PASS: TestAccAWSCloudWatchEventRule_ScheduleAndPattern (51.51s) --- PASS: TestAccAWSCloudWatchEventRule_NamePrefix (52.07s) --- PASS: TestAccAWSCloudWatchEventRule_role (60.54s) --- PASS: TestAccAWSCloudWatchEventRule_description (69.58s) --- PASS: TestAccAWSCloudWatchEventRule_pattern (69.88s) --- PASS: TestAccAWSCloudWatchEventRule_IsEnabled (84.21s) --- PASS: TestAccAWSCloudWatchEventRule_basic (85.22s) --- PASS: TestAccAWSCloudWatchEventRule_EventBusName (87.87s) --- PASS: TestAccAWSCloudWatchEventRule_tags (94.29s) PASS ok github.com/terraform-providers/terraform-provider-aws/aws 94.393s
…ission_PartnerEventBus'. Acceptance test output: $ EVENT_BRIDGE_PARTNER_EVENT_BUS_NAME=aws.partner/datadog.com/TerraformTesting make testacc TEST=./aws/ TESTARGS='-run=TestAccAWSCloudWatchEventPermission_PartnerEventBus' ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAWSCloudWatchEventPermission_PartnerEventBus -timeout 180m === RUN TestAccAWSCloudWatchEventPermission_PartnerEventBus === PAUSE TestAccAWSCloudWatchEventPermission_PartnerEventBus === CONT TestAccAWSCloudWatchEventPermission_PartnerEventBus resource_aws_cloudwatch_event_permission_test.go:328: Step 1/2 error: Error running apply: exit status 1 2021/03/29 17:37:22 [DEBUG] Using modified User-Agent: Terraform/0.12.26 HashiCorp-terraform-exec/0.13.0 Error: Creating CloudWatch Events permission failed: ValidationException: 1 validation error detected: Value 'aws.partner/datadog.com/TerraformTesting' at 'eventBusName' failed to satisfy constraint: Member must satisfy regular expression pattern: [\.\-_A-Za-z0-9]+ status code: 400, request id: 8329ec8c-828c-4048-a665-60aaafd76f0d on terraform_plugin_test.tf line 2, in resource "aws_cloudwatch_event_permission" "test": 2: resource "aws_cloudwatch_event_permission" "test" { --- FAIL: TestAccAWSCloudWatchEventPermission_PartnerEventBus (3.87s) FAIL FAIL github.com/terraform-providers/terraform-provider-aws/aws 3.961s FAIL GNUmakefile:27: recipe for target 'testacc' failed make: *** [testacc] Error 1
Acceptance test output: $ EVENT_BRIDGE_PARTNER_EVENT_BUS_NAME=aws.partner/datadog.com/TerraformTesting make testacc TEST=./aws/ TESTARGS='-run=TestAccAWSCloudWatchEventPermission_' ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAWSCloudWatchEventPermission_ -timeout 180m === RUN TestAccAWSCloudWatchEventPermission_basic === PAUSE TestAccAWSCloudWatchEventPermission_basic === RUN TestAccAWSCloudWatchEventPermission_EventBusName === PAUSE TestAccAWSCloudWatchEventPermission_EventBusName === RUN TestAccAWSCloudWatchEventPermission_Action === PAUSE TestAccAWSCloudWatchEventPermission_Action === RUN TestAccAWSCloudWatchEventPermission_Condition === PAUSE TestAccAWSCloudWatchEventPermission_Condition === RUN TestAccAWSCloudWatchEventPermission_Multiple === PAUSE TestAccAWSCloudWatchEventPermission_Multiple === RUN TestAccAWSCloudWatchEventPermission_Disappears === PAUSE TestAccAWSCloudWatchEventPermission_Disappears === RUN TestAccAWSCloudWatchEventPermission_PartnerEventBus === PAUSE TestAccAWSCloudWatchEventPermission_PartnerEventBus === CONT TestAccAWSCloudWatchEventPermission_basic === CONT TestAccAWSCloudWatchEventPermission_Multiple === CONT TestAccAWSCloudWatchEventPermission_PartnerEventBus === CONT TestAccAWSCloudWatchEventPermission_Disappears === CONT TestAccAWSCloudWatchEventPermission_Action === CONT TestAccAWSCloudWatchEventPermission_Condition === CONT TestAccAWSCloudWatchEventPermission_EventBusName === CONT TestAccAWSCloudWatchEventPermission_PartnerEventBus resource_aws_cloudwatch_event_permission_test.go:328: Step 1/2 error: Error running apply: exit status 1 2021/03/29 17:47:27 [DEBUG] Using modified User-Agent: Terraform/0.12.26 HashiCorp-terraform-exec/0.13.0 Error: Creating CloudWatch Events permission failed: ValidationException: 1 validation error detected: Value 'aws.partner/datadog.com/TerraformTesting' at 'eventBusName' failed to satisfy constraint: Member must satisfy regular expression pattern: [\.\-_A-Za-z0-9]+ status code: 400, request id: 0975effd-27f5-4d2d-9b5f-d4a2d0b1a318 on terraform_plugin_test.tf line 2, in resource "aws_cloudwatch_event_permission" "test": 2: resource "aws_cloudwatch_event_permission" "test" { --- FAIL: TestAccAWSCloudWatchEventPermission_PartnerEventBus (10.31s) --- PASS: TestAccAWSCloudWatchEventPermission_EventBusName (27.23s) --- PASS: TestAccAWSCloudWatchEventPermission_Action (28.79s) --- PASS: TestAccAWSCloudWatchEventPermission_Multiple (36.03s) --- PASS: TestAccAWSCloudWatchEventPermission_Condition (37.43s) --- PASS: TestAccAWSCloudWatchEventPermission_basic (47.07s) --- PASS: TestAccAWSCloudWatchEventPermission_Disappears (138.42s) FAIL FAIL github.com/terraform-providers/terraform-provider-aws/aws 138.532s FAIL GNUmakefile:27: recipe for target 'testacc' failed make: *** [testacc] Error 1
…names." This reverts commit 9147433726b98896a2c367cda5499d71f973080d.
…ventPermission_PartnerEventBus'." This reverts commit 10feef37ef808c78b7eeac16233fc3e2a5686fc6.
…artnerEventBus'. Acceptance test output: $ EVENT_BRIDGE_PARTNER_EVENT_BUS_NAME=aws.partner/datadog.com/TerraformTesting make testacc TEST=./aws/ TESTARGS='-run=TestAccAWSCloudWatchEventTarget_PartnerEventBus' ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAWSCloudWatchEventTarget_PartnerEventBus -timeout 180m === RUN TestAccAWSCloudWatchEventTarget_PartnerEventBus === PAUSE TestAccAWSCloudWatchEventTarget_PartnerEventBus === CONT TestAccAWSCloudWatchEventTarget_PartnerEventBus map[string]string{"%":"16", "arn":"arn:aws:sns:us-west-2:123456789012:tf-acc-test-8868416352113624735", "batch_target.#":"0", "dead_letter_config.#":"0", "ecs_target.#":"0", "event_bus_name":"aws.partner/datadog.com/TerraformTesting", "id":"aws.partner/datadog.com/TerraformTesting-tf-acc-test-8868416352113624735-tf-acc-test-8868416352113624735", "input":"", "input_path":"", "input_transformer.#":"0", "kinesis_target.#":"0", "retry_policy.#":"0", "role_arn":"", "rule":"tf-acc-test-8868416352113624735", "run_command_targets.#":"0", "sqs_target.#":"0", "target_id":"tf-acc-test-8868416352113624735"} resource_aws_cloudwatch_event_target_test.go:646: Step 2/2 error running import: exit status 1 2021/03/30 14:26:29 [DEBUG] Using modified User-Agent: Terraform/0.12.26 HashiCorp-terraform-exec/0.13.0 Error: unexpected format for ID ("aws.partner/datadog.com/TerraformTesting/tf-acc-test-8868416352113624735/tf-acc-test-8868416352113624735"), expected <event-bus-name>/<rule-name>/<target-id> or <rule-name>/<target-id> --- FAIL: TestAccAWSCloudWatchEventTarget_PartnerEventBus (15.08s) FAIL FAIL github.com/terraform-providers/terraform-provider-aws/aws 15.433s FAIL GNUmakefile:27: recipe for target 'testacc' failed make: *** [testacc] Error 1
Acceptance test output: $ EVENT_BRIDGE_PARTNER_EVENT_BUS_NAME=aws.partner/datadog.com/TerraformTesting make testacc TEST=./aws/ TESTARGS='-run=TestAccAWSCloudWatchEventTarget_' ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAWSCloudWatchEventTarget_ -timeout 180m === RUN TestAccAWSCloudWatchEventTarget_basic === PAUSE TestAccAWSCloudWatchEventTarget_basic === RUN TestAccAWSCloudWatchEventTarget_EventBusName === PAUSE TestAccAWSCloudWatchEventTarget_EventBusName === RUN TestAccAWSCloudWatchEventTarget_GeneratedTargetId === PAUSE TestAccAWSCloudWatchEventTarget_GeneratedTargetId === RUN TestAccAWSCloudWatchEventTarget_RetryPolicy_DeadLetterConfig === PAUSE TestAccAWSCloudWatchEventTarget_RetryPolicy_DeadLetterConfig === RUN TestAccAWSCloudWatchEventTarget_full === PAUSE TestAccAWSCloudWatchEventTarget_full === RUN TestAccAWSCloudWatchEventTarget_disappears === PAUSE TestAccAWSCloudWatchEventTarget_disappears === RUN TestAccAWSCloudWatchEventTarget_ssmDocument === PAUSE TestAccAWSCloudWatchEventTarget_ssmDocument === RUN TestAccAWSCloudWatchEventTarget_ecs === PAUSE TestAccAWSCloudWatchEventTarget_ecs === RUN TestAccAWSCloudWatchEventTarget_ecsWithBlankTaskCount === PAUSE TestAccAWSCloudWatchEventTarget_ecsWithBlankTaskCount === RUN TestAccAWSCloudWatchEventTarget_batch === PAUSE TestAccAWSCloudWatchEventTarget_batch === RUN TestAccAWSCloudWatchEventTarget_kinesis === PAUSE TestAccAWSCloudWatchEventTarget_kinesis === RUN TestAccAWSCloudWatchEventTarget_sqs === PAUSE TestAccAWSCloudWatchEventTarget_sqs === RUN TestAccAWSCloudWatchEventTarget_input_transformer === PAUSE TestAccAWSCloudWatchEventTarget_input_transformer === RUN TestAccAWSCloudWatchEventTarget_inputTransformerJsonString === PAUSE TestAccAWSCloudWatchEventTarget_inputTransformerJsonString === RUN TestAccAWSCloudWatchEventTarget_PartnerEventBus === PAUSE TestAccAWSCloudWatchEventTarget_PartnerEventBus === CONT TestAccAWSCloudWatchEventTarget_basic === CONT TestAccAWSCloudWatchEventTarget_ecsWithBlankTaskCount === CONT TestAccAWSCloudWatchEventTarget_PartnerEventBus === CONT TestAccAWSCloudWatchEventTarget_inputTransformerJsonString === CONT TestAccAWSCloudWatchEventTarget_input_transformer === CONT TestAccAWSCloudWatchEventTarget_sqs === CONT TestAccAWSCloudWatchEventTarget_kinesis === CONT TestAccAWSCloudWatchEventTarget_batch === CONT TestAccAWSCloudWatchEventTarget_full === CONT TestAccAWSCloudWatchEventTarget_ecs === CONT TestAccAWSCloudWatchEventTarget_ssmDocument === CONT TestAccAWSCloudWatchEventTarget_disappears === CONT TestAccAWSCloudWatchEventTarget_GeneratedTargetId === CONT TestAccAWSCloudWatchEventTarget_RetryPolicy_DeadLetterConfig === CONT TestAccAWSCloudWatchEventTarget_EventBusName --- PASS: TestAccAWSCloudWatchEventTarget_disappears (36.09s) --- PASS: TestAccAWSCloudWatchEventTarget_PartnerEventBus (38.80s) --- PASS: TestAccAWSCloudWatchEventTarget_GeneratedTargetId (40.40s) --- PASS: TestAccAWSCloudWatchEventTarget_sqs (40.45s) --- PASS: TestAccAWSCloudWatchEventTarget_ssmDocument (42.26s) --- PASS: TestAccAWSCloudWatchEventTarget_inputTransformerJsonString (50.73s) --- PASS: TestAccAWSCloudWatchEventTarget_ecsWithBlankTaskCount (55.82s) --- PASS: TestAccAWSCloudWatchEventTarget_ecs (55.97s) --- PASS: TestAccAWSCloudWatchEventTarget_EventBusName (62.47s) --- PASS: TestAccAWSCloudWatchEventTarget_input_transformer (63.21s) --- PASS: TestAccAWSCloudWatchEventTarget_basic (72.88s) --- PASS: TestAccAWSCloudWatchEventTarget_RetryPolicy_DeadLetterConfig (78.52s) --- PASS: TestAccAWSCloudWatchEventTarget_kinesis (80.00s) --- PASS: TestAccAWSCloudWatchEventTarget_full (80.40s) --- PASS: TestAccAWSCloudWatchEventTarget_batch (173.11s) PASS ok github.com/terraform-providers/terraform-provider-aws/aws 173.215s
ewbankkit
force-pushed
the
f-aws_cloudwatch_event_rule-partner-event
branch
from
April 9, 2021 20:04
acfa53b
to
7c92d94
Compare
This was referenced Apr 9, 2021
resource/aws_cloudwatch_event_target: Intermittent acceptance test failures with active tasks
#18601
Closed
Merged into #19072. |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
documentation
Introduces or discusses updates to documentation.
provider
Pertains to the provider itself, rather than any interaction with AWS.
size/XL
Managed by automation to categorize the size of a PR.
tests
PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Community Note
Closes #18431.
The
EVENT_BRIDGE_PARTNER_EVENT_BUS_NAME
environment variable must be set to an existing partner event bridge name to run the new acceptance tests.Output from acceptance testing: