We don't support advertising multiple operational identities

[bzbarsky-apple]

Problem

Both DiscoveryImplPlatform::Advertise(const OperationalAdvertisingParameters & params) and AdvertiserMinMdns::Advertise(const OperationalAdvertisingParameters & params) override an existing advertisement if there is one. This makes it impossible to advertise multiple operational identities.

Proposed Solution

Fix both codepaths to support multiple operational ids.

See also #8037 and #6141

@andy31415 @cecille @pan-apple

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[bzbarsky-apple]

I believe this is fixed now. AdvertiserMinMdns::Advertise does:

    auto * operationalAllocator = FindOperationalAllocator(nameCheck);

which will use different allocators for different instance names (which are different per operational identity). I don't see anything obvious in DiscoveryImplPlatform::Advertise that would preclude multiple operational identities (though some platform impls could have their own constraints, I suppose).

@vivien-apple @andy31415 can you confirm?

[andy31415]

As far as I am aware, we are able to advertise multiple operational networks and we are able to do operational discovery for specific identities. Multi-admin tests validate that.