duplicate key value violates unique constraint « dcim_cabletermination_unique_termination »

[r000m5]

Hello,

I'm using netbox 3.4.4 and the following code to create cables using netbox script :

#!/usr/bin/python3

from math import *
from dcim.choices import *
from dcim.models import *
from extras.scripts import *

class Link(Script):
    def run(self, data, commit):
        nb_int_used_in_sw = floor(48 * 0.8)
        swac_role = DeviceRole.objects.get(name='SWAC')
        for ue in Rack.objects.filter():
            devices = Device.objects.filter(custom_field_data__target=ue.id)
            switches = Device.objects.filter(rack=ue,device_role=swac_role)
            nb_int_in_rack = 0
            if switches:
                for d in enumerate(devices):
                    interfaces = Interface.objects.filter(device=d)
                    for i in interfaces:
                        nb_int_in_rack = nb_int_in_rack + 1
                        swac_bc_index = int(nb_int_in_rack // nb_int_used_in_sw)+1
                        int_index = int(nb_int_in_rack % nb_int_used_in_sw)
                        bc = Device.objects.get(name='BC'+'-'+ue.name+'-'+str(swac_bc_index))
                        c = Cable.objects.create()
                        cta = CableTermination.objects.create(termination=i,cable=c,cable_end="A")
                        ctb = CableTermination.objects.create(termination=RearPort.objects.get(device=bc,name='Rear Port ' + str(int_index)),cable=c,cable_end="B")
                        c.save()

Tough I get this error :

[2023-02-07 16:21:14,265][ERROR] - An exception occurred: `IntegrityError: ERROR:  duplicate key value violates unique constraint « dcim_cabletermination_unique_termination »
DETAIL:  Key « (termination_type_id, termination_id)=(32, 1) » already exists.

I can't understand why two terminations would be created with the same "termination_id". Indeed The "termination=" parameters of the CableTermination.objects.create() functions are different every time the functions are called.
And I couldn't find a way to force the value of "termination_id".

Would anyone have any idea which could help me with this issue ? I'm kind of stuck for now.

Here is the traceback :

Traceback (most recent call last):
  File "/opt/netbox-3.4.4/netbox/extras/management/commands/runscript.py", line 45, in _run_script
    script.output = script.run(data=data, commit=commit)
  File "/opt/netbox-3.4.4/netbox/scripts/LinkSW.py", line 32, in run
    cta = CableTermination.objects.create(termination=i,cable=c,cable_end="A")
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/models/manager.py", line 85, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/models/query.py", line 671, in create
    obj.save(force_insert=True, using=self.db)
  File "/opt/netbox-3.4.4/netbox/dcim/models/cables.py", line 316, in save
    super().save(*args, **kwargs)
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/models/base.py", line 812, in save
    self.save_base(
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/models/base.py", line 863, in save_base
    updated = self._save_table(
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/models/base.py", line 1006, in _save_table
    results = self._do_insert(
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/models/base.py", line 1047, in _do_insert
    return manager._insert(
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/models/manager.py", line 85, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/models/query.py", line 1791, in _insert
    return query.get_compiler(using=using).execute_sql(returning_fields)
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/models/sql/compiler.py", line 1660, in execute_sql
    cursor.execute(sql, params)
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 67, in execute
    return self._execute_with_wrappers(
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 80, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 89, in _execute
    return self.cursor.execute(sql, params)
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/utils.py", line 91, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/opt/netbox-3.4.4/venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 89, in _execute
    return self.cursor.execute(sql, params)
django.db.utils.IntegrityError: ERROR:  duplicate key value violates unique constraint « dcim_cabletermination_unique_termination »
DETAIL:  Key « (termination_type_id, termination_id)=(32, 1) » already exists.

Regards,

Romain

[jsenecal]

Aren't you connecting twice the same interface pairs within your for loop ?

[r000m5]

Thanks for your reply.
I thought I wasn't... but indeed I was. It took me quite a long time to find my error :
That was not correct :
interfaces = Interface.objects.filter(device=d)
The correct one was :
interfaces = Interface.objects.filter(device=d[1])