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

feat: OBS-399 - netbox diode plugin to listen on post_save signals #51

Conversation

Julio-Oliveira-Encora
Copy link
Contributor

@Julio-Oliveira-Encora Julio-Oliveira-Encora commented Mar 7, 2024

It adds a method to call the SDK when receiving the post-save signal.

Added method "handle_notifify_reconciliation" to receive "post_save" signal.
Added method "handle_notifify_reconciliation" to receive "post_save" signal.
Added method "handle_notifify_reconciliation" to receive "post_save" signal.
Copy link

linear bot commented Mar 7, 2024

OBS-399 NetBox diode plugin to listen on post_save django events/signals

Listen to the NetBox post_save events/signals and notify reconciliation service about observed objects changes via diode python sdk communicating with ReconcilerService.AddObjectState gRPC method.

{
  "site": {
    "id": 1,
    "name": "test-device",
    "slug": "test-device",
    "url": "http://localhost:8000/api/dcim/sites/1/"
  }
}

@Julio-Oliveira-Encora Julio-Oliveira-Encora marked this pull request as ready for review March 11, 2024 13:17
@Julio-Oliveira-Encora Julio-Oliveira-Encora changed the title Obs 399 netbox diode plugin to listen on post save django feat: Obs 399 - netbox diode plugin to listen on post save django Mar 11, 2024
@mfiedorowicz mfiedorowicz changed the title feat: Obs 399 - netbox diode plugin to listen on post save django feat: OBS-399 - netbox diode plugin to listen on post_save signals Mar 11, 2024
@Julio-Oliveira-Encora Julio-Oliveira-Encora merged commit 159079c into develop Mar 11, 2024
14 checks passed
@Julio-Oliveira-Encora Julio-Oliveira-Encora deleted the obs-399-netbox-diode-plugin-to-listen-on-post_save-django branch March 11, 2024 14:48
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.

2 participants