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

Cached properties in CableTermination not updated #9788

Closed
mzbroch opened this issue Jul 20, 2022 · 3 comments
Closed

Cached properties in CableTermination not updated #9788

mzbroch opened this issue Jul 20, 2022 · 3 comments
Assignees
Labels
beta Concerns a bug/feature in a beta release status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application

Comments

@mzbroch
Copy link
Contributor

mzbroch commented Jul 20, 2022

NetBox version

v3.3-beta

Python version

3.8

Steps to Reproduce

  1. Populate NetBox with the data mandatory for adding two different Devices: Manufacturer, Device Type, Site (site1), ...
  2. Create two devices ("device1", "device2"), each having set of interfaces ("intf[1-5]") - assign devices to site1.
  3. Connect intf1 of device1 to intf1 of device2
  4. Navigate to cable list view (connections), check out the assigned site - expected site1
  5. Create a new site (site2)
  6. Edit device1 and device2 - change site to site2
  7. Navigate to cable list view (connections), check out the assigned site - expected site2

Expected Behavior

Respective cable between intf1 and intf1 (in a cables list view (connections)) shows site2 as it replaced site1

Observed Behavior

Respective cable between intf1 and intf1 (in a cables list view (connections)) shows cached (invalid) site site1

@mzbroch mzbroch added the type: bug A confirmed report of unexpected behavior in the application label Jul 20, 2022
@jeremystretch
Copy link
Member

Thank you for opening a bug report. Unfortunately, the information you have provided is not sufficient for someone else to attempt to reproduce the reported behavior. Remember, each bug report must include detailed steps that someone else can follow on a clean, empty NetBox installation to reproduce the exact problem you're experiencing. These instructions should include the creation of any involved objects, any configuration changes, and complete accounting of the actions being taken. Also be sure that your report does not reference data on the public NetBox demo, as that is subject to change at any time by an outside party and cannot be relied upon for bug reports.

@jeremystretch jeremystretch added status: revisions needed This issue requires additional information to be actionable beta Concerns a bug/feature in a beta release labels Jul 20, 2022
@mzbroch
Copy link
Contributor Author

mzbroch commented Jul 20, 2022

Thank you for opening a bug report. Unfortunately, the information you have provided is not sufficient for someone else to attempt to reproduce the reported behavior.

Updated the report.

@jeremystretch jeremystretch added status: under review Further discussion is needed to determine this issue's scope and/or implementation and removed status: revisions needed This issue requires additional information to be actionable labels Jul 20, 2022
@jeremystretch
Copy link
Member

This looks like a typical denormalization issue, where we just need the cached values up to date. We could handle this with dedicated signal receivers as we do elsehwere in NetBox, but it's probably worth investigating more robust solutions.

@jeremystretch jeremystretch self-assigned this Jul 25, 2022
@jeremystretch jeremystretch added status: accepted This issue has been accepted for implementation and removed status: under review Further discussion is needed to determine this issue's scope and/or implementation labels Aug 2, 2022
jeremystretch added a commit that referenced this issue Aug 2, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
beta Concerns a bug/feature in a beta release status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Development

No branches or pull requests

2 participants