-
-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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
Update nest.markdown - troubleshooting entry #36716
Conversation
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @ShadowAya 👍
../Frenck
📝 WalkthroughWalkthroughThe pull request introduces a troubleshooting note in the Google Nest integration documentation, focusing on issues related to enabling Pub/Sub in the Device Access Console. It highlights a known problem where users may struggle to create a project with Pub/Sub or enable it, and advises users to consult a specific comment on the Google IssueTracker page for additional information. Changes
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
source/_integrations/nest.markdown (1)
605-606
: LGTM! Consider adding a brief description of the manual topic creation process.The new troubleshooting entry is well-placed and provides valuable information about the Pub/Sub enabling issue. The links to the issue tracker and specific comment with the workaround are helpful.
To make it even more user-friendly, consider adding a brief description of the manual topic creation process or linking to relevant Google Cloud documentation about creating Pub/Sub topics manually.
That advice does not work. That just creates a random topic unrelated to SDM right? The SDM Topic needs to match "Pub/Sub topic" in the device access console. The nest team is going to roll out a feature to allow the SDM console to use other pub/sub topics but it has not launched yet. If you do this, you are going to later realize your devices are not receiving any updates/events. |
You're right it doesn't match, but it enabled me to turn the option on (for some reason), which then created the correct topic. It might be appropriate to mention the issue without suggesting a solution, I can update that. I am now testing an automation on my camera, and even when I selected the correct topic everywhere, I don't seem to be receiving any events. This whole thing is broken, classic Google. |
They are basically having an outage right now, and just need to fix the issue. |
Okay, sorry about that! Feel free to dismiss this pull request; still, I think it would be helpful to mention this issue in Home Assistant's docs, might throw some people off. |
Let me reach out to my contacts on the nest team about the current outage and see if it will be prolonged or not. (My impression was that known issue was temporarily resolved until we change how subscribers work, which they are going to rollout this year some time, but i'll see if that is still the case). Thanks, hold tight and i'll report back. |
I'm drafting this PR, as there is a discussion ongoing. Please undraft it, once the changes have been implemented and the PR is ready for review again. |
The nest team is aware of the issue and working to resolve it. I don't think we should do anything here. |
Okay, that's good to hear. I started receiving events yesterday, so the outage may be resolved soon. Thanks for the info! I'm closing the PR. |
The outage is only for creating a new topic through right? I don't think it's related to receiving events. |
Proposed change
Add a Troubleshooting entry to the Google Nest integration page regarding a Device Access Console bug which prevents Pub/Sub topic from being enabled
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
n/a
Checklist
current
branch.next
branch.Summary by CodeRabbit