feat: service registration with granular iface/ip conditions

[twizansk]

suggested update to mdns-sd to support granular service registration.

We would find it very useful to be able to register services with specific conditions:

• associate them with with specific interfaces.
• return only link-local ips

@keepsimple1 if this looks like a reasonable update to you, I can finish up the pr (cleanup, tests, etc.) and take it out of draft

issue: #399

[keepsimple1]

Thanks for your PR! At the high level I'm open to add such granular conditions. Please feel free to move it out of draft. I've a couple of comments inline.

[twizansk]

pr is out of draft

[keepsimple1]

Overall looks good to me! A few minor comments inline.

[keepsimple1]

If possible (and ready), could you also add integration test for the new public APIs?

[twizansk]

If possible (and ready), could you also add integration test for the new public APIs?

added. FYI I noticed that the tests don't run on the ubuntu build. the workflow file has

- name: Run tests with debugs
   if: matrix.os == 'ubuntu-20.04' || matrix.os == 'macos-14'

but we're running on ubuntu 22.04

[keepsimple1]

but we're running on ubuntu 22.04

You can update the workflow to use ubuntu 22.04.

[twizansk]

but we're running on ubuntu 22.04

You can update the workflow to use ubuntu 22.04.

getting test failures on 22.04: test_disable_interface_cache. tested on mainline and still failing (admittedly I tested on 24.04 not 22.04) so it seems unrelated. ok if we leave this as a follow-up?

[twizansk]

but we're running on ubuntu 22.04

You can update the workflow to use ubuntu 22.04.

getting test failures on 22.04: test_disable_interface_cache. tested on mainline and still failing (admittedly I tested on 24.04 not 22.04) so it seems unrelated. ok if we leave this as a follow-up?

FWIW, on my setup looks like it's failing because, in the test, we disabled only the ipv4 interface address

however, the iface returns an ipv6 address as well which does not get disabled.

here we send out a query to both ipv6 and ipv4 if the MyIntf instance still exists

[keepsimple1]

Looks good. Only minor comments inline. Thanks for the update!

[keepsimple1]

ok if we leave this as a follow-up?

Yes, sounds good.

[keepsimple1]

One more minor comment: could you mention the issue number that triggered this PR in the Description section? such that GitHub will automatically link this PR to that issue.

[twizansk]

One more minor comment: could you mention the issue number that triggered this PR in the Description section? such that GitHub will automatically link this PR to that issue.

done

[keepsimple1]

LGTM, thanks!