[Bug]: Error: Plugin did not respond -> panic: interface conversion: interface {} is string, not int #32588

x4team · 2023-07-19T14:15:10Z

Terraform Core Version

1.5.3

AWS Provider Version

5.8.0

Affected Resource(s)

aws_ec2_traffic_mirror_filter_rule

Expected Behavior

Recreate the inbound/output rules with a different "Rule number". For example, from 2 to 1

Actual Behavior

Stack trace from the terraform-provider-aws_v5.8.0_x5 plugin:

panic: interface conversion: interface {} is string, not int

Relevant Error/Panic Output Snippet

│ Error: Plugin did not respond
│ 
│   with module.suricata.aws_ec2_traffic_mirror_filter_rule.rule["all_out"],
│   on ../../../modules/suricata/main.tf line 92, in resource "aws_ec2_traffic_mirror_filter_rule" "rule":
│   92: resource "aws_ec2_traffic_mirror_filter_rule" "rule" {
│ 
│ The plugin encountered an error, and failed to respond to the plugin.(*GRPCProvider).ApplyResourceChange
│ call. The plugin logs may contain more details.
╵
╷
│ Error: Plugin did not respond
│ 
│   with module.suricata.aws_ec2_traffic_mirror_filter_rule.rule["all_in"],
│   on ../../../modules/suricata/main.tf line 92, in resource "aws_ec2_traffic_mirror_filter_rule" "rule":
│   92: resource "aws_ec2_traffic_mirror_filter_rule" "rule" {
│ 
│ The plugin encountered an error, and failed to respond to the plugin.(*GRPCProvider).ApplyResourceChange
│ call. The plugin logs may contain more details.
╵

Stack trace from the terraform-provider-aws_v5.8.0_x5 plugin:

panic: interface conversion: interface {} is string, not int

goroutine 349 [running]:
github.com/hashicorp/terraform-provider-aws/internal/service/ec2.resourceTrafficMirrorFilterRuleUpdate({0xf387580, 0xc00377acf0}, 0xc0043dcd00, {0xe06f840?, 0xc002857860?})
	github.com/hashicorp/terraform-provider-aws/internal/service/ec2/vpc_traffic_mirror_filter_rule.go:259 +0x1134
github.com/hashicorp/terraform-provider-aws/internal/provider.interceptedHandler[...].func1(0x0?, {0xe06f840?, 0xc002857860?})
	github.com/hashicorp/terraform-provider-aws/internal/provider/intercept.go:111 +0x34b
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).update(0xf387580?, {0xf387580?, 0xc003765110?}, 0xd?, {0xe06f840?, 0xc002857860?})
	github.com/hashicorp/terraform-plugin-sdk/[email protected]/helper/schema/resource.go:762 +0x87
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).Apply(0xc0010cc2a0, {0xf387580, 0xc003765110}, 0xc00372b2b0, 0xc0043dcb80, {0xe06f840, 0xc002857860})
	github.com/hashicorp/terraform-plugin-sdk/[email protected]/helper/schema/resource.go:874 +0x845
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*GRPCProviderServer).ApplyResourceChange(0xc00357f2d8, {0xf387580?, 0xc003764ff0?}, 0xc003d09810)
	github.com/hashicorp/terraform-plugin-sdk/[email protected]/helper/schema/grpc_provider.go:1024 +0xe8d
github.com/hashicorp/terraform-plugin-mux/tf5muxserver.muxServer.ApplyResourceChange({0xc002858f90, 0xc004ef5780, 0xc002858fc0, 0xc002858ff0, {0xc00285c860, 0x2, 0x2}}, {0xf387580?, 0xc003764cc0?}, 0xc003d09810)
	github.com/hashicorp/[email protected]/tf5muxserver/mux_server_ApplyResourceChange.go:30 +0x139
github.com/hashicorp/terraform-plugin-go/tfprotov5/tf5server.(*server).ApplyResourceChange(0xc002848d20, {0xf387580?, 0xc0037644b0?}, 0xc0013e6d90)
	github.com/hashicorp/[email protected]/tfprotov5/tf5server/server.go:821 +0x574
github.com/hashicorp/terraform-plugin-go/tfprotov5/internal/tfplugin5._Provider_ApplyResourceChange_Handler({0xdd4fb40?, 0xc002848d20}, {0xf387580, 0xc0037644b0}, 0xc0013e6d20, 0x0)
	github.com/hashicorp/[email protected]/tfprotov5/internal/tfplugin5/tfplugin5_grpc.pb.go:422 +0x170
google.golang.org/grpc.(*Server).processUnaryRPC(0xc000910000, {0xf3943a0, 0xc00292a820}, 0xc001bba900, 0xc002914e10, 0x162634e0, 0x0)
	google.golang.org/[email protected]/server.go:1337 +0xdf3
google.golang.org/grpc.(*Server).handleStream(0xc000910000, {0xf3943a0, 0xc00292a820}, 0xc001bba900, 0x0)
	google.golang.org/[email protected]/server.go:1714 +0xa36
google.golang.org/grpc.(*Server).serveStreams.func1.1()
	google.golang.org/[email protected]/server.go:959 +0x98
created by google.golang.org/grpc.(*Server).serveStreams.func1
	google.golang.org/[email protected]/server.go:957 +0x18c

Error: The terraform-provider-aws_v5.8.0_x5 plugin crashed!

This is always indicative of a bug within the plugin. It would be immensely
helpful if you could report the crash with the plugin's maintainers so that it
can be fixed. The output above should help diagnose the issue.

Terraform Configuration Files

resource "aws_ec2_traffic_mirror_filter_rule" "rule" {
  for_each                 = {
    for key, value in {
      all_in  = { description = "ALL IN",  direction = "ingress" },
      all_out = { description = "ALL OUT", direction = "egress"  },
    } : key => value
  }
  description              = each.value.description
  traffic_mirror_filter_id = aws_ec2_traffic_mirror_filter.filter.id
  destination_cidr_block   = "0.0.0.0/0"
  source_cidr_block        = "0.0.0.0/0"
  rule_number              = 1
  rule_action              = "accept"
  traffic_direction        = each.value.direction
  protocol                 = 0
}

Steps to Reproduce

Create traffic mirror resource "aws_ec2_traffic_mirror_filter" "filter"
Create resource "aws_ec2_traffic_mirror_filter_rule" "rule" with igress/egress rules
Create resource "aws_ec2_traffic_mirror_target" "target"
Create resource "aws_ec2_traffic_mirror_session" "session"
Change rule_number to another number (look point 2 - aws_ec2_traffic_mirror_filter_rule)
terraform apply

Debug Output

No response

Panic Output

No response

Important Factoids

No response

References

No response

Would you like to implement a fix?

None

The text was updated successfully, but these errors were encountered:

github-actions · 2023-07-19T14:15:25Z

Community Note

Voting for Prioritization

Please vote on this issue by adding a 👍 reaction to the original post to help the community and maintainers prioritize this request.
Please see our prioritization guide for information on how we prioritize.
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.

Volunteering to Work on This Issue

If you are interested in working on this issue, please leave a comment.
If this would be your first contribution, please review the contribution guide.

ewbankkit · 2023-07-19T14:43:20Z

terraform-provider-aws/internal/service/ec2/vpc_traffic_mirror_filter_rule.go

Lines 258 to 260 in 71ac1fa

    
           if d.HasChange("rule_number") { 
        
           	input.RuleNumber = aws.Int64(int64(d.Get("rule_action").(int))) 
        
           }

rule_action should be rule_number.

ewbankkit · 2023-07-19T15:47:12Z

I can reproduce this with an updated acceptance test case:

% make testacc TESTARGS='-run=TestAccVPCTrafficMirrorFilterRule_basic' PKG=ec2
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/ec2/... -v -count 1 -parallel 20  -run=TestAccVPCTrafficMirrorFilterRule_basic -timeout 180m
=== RUN   TestAccVPCTrafficMirrorFilterRule_basic
=== PAUSE TestAccVPCTrafficMirrorFilterRule_basic
=== CONT  TestAccVPCTrafficMirrorFilterRule_basic
panic: interface conversion: interface {} is string, not int

goroutine 17226 [running]:
github.com/hashicorp/terraform-provider-aws/internal/service/ec2.resourceTrafficMirrorFilterRuleUpdate({0x105a6280, 0xc0045e1620}, 0xc005e13e00, {0xf163540?, 0xc0040a0dd0?})
	/Users/ewbankkit/altsrc.2/github.com/terraform-providers/terraform-provider-aws/internal/service/ec2/vpc_traffic_mirror_filter_rule.go:259 +0x1134
github.com/hashicorp/terraform-provider-aws/internal/provider.interceptedHandler[...].func1(0x0?, {0xf163540?, 0xc0040a0dd0?})
	/Users/ewbankkit/altsrc.2/github.com/terraform-providers/terraform-provider-aws/internal/provider/intercept.go:111 +0x34b
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).update(0x105a6280?, {0x105a6280?, 0xc0045c9fb0?}, 0xd?, {0xf163540?, 0xc0040a0dd0?})
	/Users/ewbankkit/go/pkg/mod/github.com/hashicorp/terraform-plugin-sdk/[email protected]/helper/schema/resource.go:762 +0x87
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).Apply(0xc001fb7880, {0x105a6280, 0xc0045c9fb0}, 0xc002763d40, 0xc005e13c80, {0xf163540, 0xc0040a0dd0})
	/Users/ewbankkit/go/pkg/mod/github.com/hashicorp/terraform-plugin-sdk/[email protected]/helper/schema/resource.go:874 +0x845
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*GRPCProviderServer).ApplyResourceChange(0xc0042656b0, {0x105a6280?, 0xc0045c9ef0?}, 0xc0026d7ae0)
	/Users/ewbankkit/go/pkg/mod/github.com/hashicorp/terraform-plugin-sdk/[email protected]/helper/schema/grpc_provider.go:1024 +0xe8d
github.com/hashicorp/terraform-plugin-mux/tf5muxserver.muxServer.ApplyResourceChange({0xc00067a510, 0xc0042407e0, 0xc00067a540, 0xc00067a570, {0xc0040aa440, 0x2, 0x2}}, {0x105a6280?, 0xc0045c9c20?}, 0xc0026d7ae0)
	/Users/ewbankkit/go/pkg/mod/github.com/hashicorp/[email protected]/tf5muxserver/mux_server_ApplyResourceChange.go:30 +0x139
github.com/hashicorp/terraform-plugin-go/tfprotov5/tf5server.(*server).ApplyResourceChange(0xc0040794a0, {0x105a6280?, 0xc0045c9350?}, 0xc000c700e0)
	/Users/ewbankkit/go/pkg/mod/github.com/hashicorp/[email protected]/tfprotov5/tf5server/server.go:821 +0x574
github.com/hashicorp/terraform-plugin-go/tfprotov5/internal/tfplugin5._Provider_ApplyResourceChange_Handler({0xee3b780?, 0xc0040794a0}, {0x105a6280, 0xc0045c9350}, 0xc000c70070, 0x0)
	/Users/ewbankkit/go/pkg/mod/github.com/hashicorp/[email protected]/tfprotov5/internal/tfplugin5/tfplugin5_grpc.pb.go:422 +0x170
google.golang.org/grpc.(*Server).processUnaryRPC(0xc004c54b40, {0x105b3380, 0xc005716b60}, 0xc0029977a0, 0xc00067ad80, 0x176f5cc0, 0x0)
	/Users/ewbankkit/go/pkg/mod/google.golang.org/[email protected]/server.go:1337 +0xdf3
google.golang.org/grpc.(*Server).handleStream(0xc004c54b40, {0x105b3380, 0xc005716b60}, 0xc0029977a0, 0x0)
	/Users/ewbankkit/go/pkg/mod/google.golang.org/[email protected]/server.go:1714 +0xa36
google.golang.org/grpc.(*Server).serveStreams.func1.1()
	/Users/ewbankkit/go/pkg/mod/google.golang.org/[email protected]/server.go:959 +0x98
created by google.golang.org/grpc.(*Server).serveStreams.func1
	/Users/ewbankkit/go/pkg/mod/google.golang.org/[email protected]/server.go:957 +0x18c
FAIL	github.com/hashicorp/terraform-provider-aws/internal/service/ec2	52.585s
FAIL
make: *** [testacc] Error 1

github-actions · 2023-08-19T02:00:06Z

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.

x4team added bug Addresses a defect in current functionality. needs-triage Waiting for first response or review from a maintainer. labels Jul 19, 2023

github-actions bot added crash Results from or addresses a Terraform crash or kernel panic. service/vpc Issues and PRs that pertain to the vpc service. labels Jul 19, 2023

ewbankkit removed the needs-triage Waiting for first response or review from a maintainer. label Jul 19, 2023

ewbankkit self-assigned this Jul 19, 2023

ewbankkit mentioned this issue Jul 19, 2023

r/aws_ec2_traffic_mirror_filter_rule: Fix crash when updating rule_number #32594

Merged

ewbankkit closed this as completed in #32594 Jul 19, 2023

github-actions bot locked as resolved and limited conversation to collaborators Aug 19, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Bug]: Error: Plugin did not respond -> panic: interface conversion: interface {} is string, not int #32588

[Bug]: Error: Plugin did not respond -> panic: interface conversion: interface {} is string, not int #32588

x4team commented Jul 19, 2023 •

edited

Loading

github-actions bot commented Jul 19, 2023

ewbankkit commented Jul 19, 2023

ewbankkit commented Jul 19, 2023

github-actions bot commented Aug 19, 2023

[Bug]: Error: Plugin did not respond -> panic: interface conversion: interface {} is string, not int #32588

[Bug]: Error: Plugin did not respond -> panic: interface conversion: interface {} is string, not int #32588

Comments

x4team commented Jul 19, 2023 • edited Loading

Terraform Core Version

AWS Provider Version

Affected Resource(s)

Expected Behavior

Actual Behavior

Relevant Error/Panic Output Snippet

Terraform Configuration Files

Steps to Reproduce

Debug Output

Panic Output

Important Factoids

References

Would you like to implement a fix?

github-actions bot commented Jul 19, 2023

Community Note

ewbankkit commented Jul 19, 2023

ewbankkit commented Jul 19, 2023

github-actions bot commented Aug 19, 2023

x4team commented Jul 19, 2023 •

edited

Loading