Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix ToT failure on avahi #9744

Merged
merged 1 commit into from
Sep 16, 2021
Merged

Conversation

cecille
Copy link
Contributor

@cecille cecille commented Sep 16, 2021

Problem

The current ToT code calls PublishService twice in a row during startup - once during a call to open the pairing window, one during the general server startup. There's also a Stop called between them. On ToT this was failing. Root caused to Stop not depopulating the list of services, so we were attempting to update a service that had already been stopped.

Fixes #9695

Change overview

Add a clear to the Stop call in the avahi mdnsImpl

Testing

No unit tests for avahi impl - tested on linux lighting-app, can see error no longer prints, can find on avahi-discover.

@woody-apple
Copy link
Contributor

/rebase

@andy31415
Copy link
Contributor

@saurabhst @LuDuda @msandstedt @jepenven-silabs @chrisdecenzo @jmartinez-silabs ?

This seems to be stuck on restyler (??? ... @woody-apple is there a way to kick it) however if we get enough checkmarks I can admin-merge it. Waiting >1day over a one line fix seems harsh

@woody-apple
Copy link
Contributor

@andy31415 Restyled issues can be filed here: https://github.com/restyled-io/restyler/issues I'll label this restyled-ignore, and merge.

@woody-apple
Copy link
Contributor

Filed: restyled-io/restyler#156

@cecille cecille deleted the avahi_fix branch September 20, 2021 14:52
mleisner pushed a commit to mleisner/connectedhomeip that referenced this pull request Sep 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Avahi error on upstream/master - failing to publish commissionable node
5 participants