Currently, adding additional route-target imports will overwrite the existing imports. I'd like to see the behavior of creating additional imports be an append action to existing imports instead of rewriting the existing import. This is more in line with how enterprise gear would handle this.

Someone could cause an outage if they’re not aware of how that behavior differs from what they’ve known.

For instance, if this already exists:

set vrf name red protocols bgp address-family ipv4-unicast route-target vpn import '65500:200'

And I type this:

set vrf name red protocols bgp address-family ipv4-unicast route-target vpn import '65500:300'

It should create this instead of rewriting the original import:

set vrf name red protocols bgp address-family ipv4-unicast route-target vpn import '65500:200 65500:300'

Maybe add a switch to the end of the command that allows the append behavior. Something like:

set vrf name red protocols bgp address-family ipv4-unicast route-target vpn import '65500:300' append

Performing a delete action should simply remove that route-target from the existing list, unless it is the only route-target in the list, in which case the import is removed entirely.