🐞 [Bug]: Deleting a VM with IPv4 interface linked to a domain will not remove the associated name and gateway contracts

[sameh-farouk]

Is there an existing issue for this?

• I have searched the existing issues

which package/s did you face the problem with?

Dashboard, grid_client

What happened?

When I click the actions button next to a VM and link a domain with an IP reserved by the VM, then proceed to delete the VM, the client leaves the name contract and gateway workload that were supposedly routing traffic to the now-removed VM.

Steps To Reproduce

1. go to https://staging.dashboard.dev.grid.tf/#/deploy/virtual-machines/full-virtual-machine/
2. Deploy a VM with IPv4 address
3. Add a new domain. Make sure to pick the IPv4 interface
4. Verify that the name contract and the gateway contract were successfully created.
5. Select the VM, from the full-virtual-machine page, and delete it.
6. revisit the contracts page and verify that the name contract and gateway's node contract are left behind.

which network/s did you face the problem on?

Dev

version

NA

Twin ID/s

No response

Node ID/s

No response

Farm ID/s

No response

Contract ID/s

No response

Relevant screenshots/screen records

I tried that twice, and here is the result

Relevant log output

NA

[maayarosama]

This might happen because we check if the backends in the gateway is wiregaurd ip connected to the vm. In this case the solution need to check on all network interfaces

tfgrid-sdk-ts/packages/playground/src/utils/delete_deployment.ts

Line 122 in 205fd89

filter: ip ? gw => gw.backends.some(bk => bk.includes(ip)) : undefined,

[0oM4R]

work completed :
add a function that pass all ips to the delete function and edit the filtration prop

WIP: some gws on interfaces like ipv4 didn't got listed checking..

[A-Harby]

Verified, Devnet ffbb750.

I deployed a VM and add a domain to it and made sure everything is correct.
then deleted it as mention and verified that all the contracts are deleted from contracts page, including vm, name and gateway.

Newly created test case: TC2903 - Deleting VM with domain