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

chore: regenerate reconciler sdk #56

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

mfiedorowicz
Copy link
Member

This pull request includes several updates to the netbox_diode_plugin/reconciler/sdk/v1/reconciler_pb2.py and netbox_diode_plugin/reconciler/sdk/v1/reconciler_pb2.pyi files. The changes primarily focus on adjusting serialized start and end positions, removing unnecessary fields, and updating method options.

Key Changes:

Adjustments to Serialized Positions:

  • Updated the serialized start and end positions for _STATE, _RETRIEVEDEVIATIONSREQUEST, _DEVIATIONERROR, _CHANGE, _DEVIATION, _RETRIEVEDEVIATIONSRESPONSE, _RETRIEVEDEVIATIONBYIDREQUEST, _RETRIEVEDEVIATIONBYIDRESPONSE, and _RECONCILERSERVICE to reflect the new positions after changes. [1] [2]

Removal of Fields:

  • Removed the site field from the RetrieveDeviationsRequest class in the reconciler_pb2.pyi file, including its slot, field number, and references in the constructor.

Signed-off-by: Michal Fiedorowicz <[email protected]>
@mfiedorowicz mfiedorowicz requested a review from ltucker January 23, 2025 10:39
@mfiedorowicz mfiedorowicz self-assigned this Jan 23, 2025
Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1750 1729 99% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 4b241a7 by action🐍

@mfiedorowicz mfiedorowicz merged commit 1fa65a4 into feat-diode-data-plugin Jan 23, 2025
2 checks passed
@mfiedorowicz mfiedorowicz deleted the chore-regen-reconciler-sdk branch January 23, 2025 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants