linode / terraform-provider-linode

Terraform Linode provider
https://www.terraform.io/docs/providers/linode/
Mozilla Public License 2.0
197 stars 94 forks source link

test: Fix e2e test firewall syntax and remove ipv6 dependency #1462

Closed ykim-1 closed 3 months ago

ykim-1 commented 4 months ago

📝 Description

Fixing https://github.com/linode/terraform-provider-linode/actions/runs/9389098679

✔️ How to Test

make test

📷 Preview

If applicable, include a screenshot or code snippet of this change. Otherwise, please remove this section.

ykim-1 commented 4 months ago

Forked run - https://github.com/ykim-1/terraform-provider-linode/actions/runs/9389566886

zliang-akamai commented 4 months ago

test run passed: https://github.com/linode/terraform-provider-linode/actions/runs/9391466234

lgarber-akamai commented 3 months ago

E2E test run: https://github.com/linode/terraform-provider-linode/actions/runs/9406523501

ykim-1 commented 3 months ago

Also tested on my forked branch - https://github.com/ykim-1/terraform-provider-linode/actions/runs/9389566886 1 Failure has been addressed with this PR

zliang-akamai commented 3 months ago

Nice job!