Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

Sync from CVP fails due to multiple tag "Container" #43

Closed
u1735067 opened this issue Sep 7, 2021 · 2 comments
Closed

Sync from CVP fails due to multiple tag "Container" #43

u1735067 opened this issue Sep 7, 2021 · 2 comments
Assignees

Comments

@u1735067
Copy link

u1735067 commented Sep 7, 2021

Environment

  • Python version: 3.9
  • Nautobot version: 1.0.3
  • aristacv-sync version: 1.0.1
  • CVP: 2021.1.1

Expected Behavior

Sync works

Observed Behavior

Sync fails

Steps to Reproduce

  1. Create group/containers hierarchy in CVP
  2. Put device in the leaf
  3. Start sync

Additional details

In CVP 2021.1.1 (at least), when you place devices in nested container, the device inherits multiple Container tags, which results in errors when syncing:

Traceback (most recent call last):
  File "/opt/nautobot/.local/lib/python3.9/site-packages/nautobot_ssot_aristacv/diffsync/fromcv/cloudvision.py", line 39, in load
    self.add(self.cf)
  File "/opt/nautobot/.local/lib/python3.9/site-packages/diffsync/__init__.py", line 657, in add
    raise ObjectAlreadyExists(f"Object {uid} already present")
diffsync.exceptions.ObjectAlreadyExists: Object arista_Container__<device_name> already present
@qduk qduk self-assigned this Sep 13, 2021
@qduk
Copy link
Contributor

qduk commented Sep 13, 2021

Thanks for the issue @u1735067. I'll look into this.

qduk added a commit that referenced this issue Sep 17, 2021
qduk added a commit that referenced this issue Sep 20, 2021
@qduk
Copy link
Contributor

qduk commented Sep 20, 2021

Hey @u1735067 after talking with Arista engineers, it was decided that the container tag is something we will not be syncing from CloudVision. The container tag is now skipped.

@qduk qduk closed this as completed Sep 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants