proxsync 2.0.7__tar.gz → 2.0.8__tar.gz

{proxsync-2.0.7/proxsync.egg-info → proxsync-2.0.8}/PKG-INFO

@@ -1,12 +1,12 @@
 Metadata-Version: 2.4
 Name: proxsync
-Version: 2.0.7
+Version: 2.0.8
 Summary: NetBox plugin for Proxmox VE inventory synchronization
 Author-email: lixiujie85 <lixiujie85@gmail.com>
 License-Expression: MIT
-Project-URL: Homepage, https://github.com/lixiujie85/proxsync
-Project-URL: Source, https://github.com/lixiujie85/proxsync
-Project-URL: Issues, https://github.com/lixiujie85/proxsync/issues
+Project-URL: Homepage, https://github.com/xiujie2/proxsync
+Project-URL: Source, https://github.com/xiujie2/proxsync
+Project-URL: Issues, https://github.com/xiujie2/proxsync/issues
 Classifier: Development Status :: 4 - Beta
 Classifier: Framework :: Django
 Classifier: Intended Audience :: System Administrators

{proxsync-2.0.7 → proxsync-2.0.8/proxsync.egg-info}/PKG-INFO

@@ -1,12 +1,12 @@
 Metadata-Version: 2.4
 Name: proxsync
-Version: 2.0.7
+Version: 2.0.8
 Summary: NetBox plugin for Proxmox VE inventory synchronization
 Author-email: lixiujie85 <lixiujie85@gmail.com>
 License-Expression: MIT
-Project-URL: Homepage, https://github.com/lixiujie85/proxsync
-Project-URL: Source, https://github.com/lixiujie85/proxsync
-Project-URL: Issues, https://github.com/lixiujie85/proxsync/issues
+Project-URL: Homepage, https://github.com/xiujie2/proxsync
+Project-URL: Source, https://github.com/xiujie2/proxsync
+Project-URL: Issues, https://github.com/xiujie2/proxsync/issues
 Classifier: Development Status :: 4 - Beta
 Classifier: Framework :: Django
 Classifier: Intended Audience :: System Administrators

{proxsync-2.0.7 → proxsync-2.0.8}/pve_sync.py

@@ -125,25 +125,12 @@ class OptimizedPVEToNetBoxSync:
         self.error_log = []
 
     # ---------- Telegram ----------
-    # Max Telegram messages per sync run; excess are counted and reported in the summary.
-    _MAX_TELEGRAM_PER_SYNC = 20
-
-    def send_telegram_notification(self, message: str) -> bool:
-        if not getattr(self, 'notify_on_sync', True):
-            return False
-        if not self.telegram_bot_token or not self.telegram_chat_id:
-            return False
-        # Throttle: cap messages per sync run to avoid flooding on bulk changes.
-        sent = getattr(self, '_telegram_sent', 0)
-        if sent >= self._MAX_TELEGRAM_PER_SYNC:
-            self._telegram_suppressed = getattr(self, '_telegram_suppressed', 0) + 1
-            return False
+    def send_telegram_notification(self, message: str):
         try:
             url = f"https://api.telegram.org/bot{self.telegram_bot_token}/sendMessage"
             payload = {'chat_id': self.telegram_chat_id, 'text': message, 'parse_mode': 'HTML'}
             response = requests.post(url, data=payload, timeout=10)
             if response.status_code == 200:
-                self._telegram_sent = sent + 1
                 print(f"✓ Telegram 通知已發送")
                 return True
             else:
@@ -174,6 +161,28 @@ class OptimizedPVEToNetBoxSync:
         self.send_telegram_notification(message)
         print(f"📧 已發送 IP 衝突通知: {vm_name} - {ip_address}")
 
+    def log_mac_conflict_error(self, vm_name: str, mac_address: str, error_message: str):
+        error_info = {
+            'timestamp': time.strftime("%Y-%m-%d %H:%M:%S"),
+            'vm_name': vm_name,
+            'ip_address': mac_address,
+            'error': error_message
+        }
+        self.error_log.append(error_info)
+        message = f"""
+🚨 <b>PVE-NetBox 同步 MAC 位址衝突警告</b>
+
+📅 時間: {error_info['timestamp']}
+🖥️ 虛擬機: {vm_name}
+🔌 MAC 位址: {mac_address}
+❌ 錯誤: {error_message}
+
+⚠️ 該網卡的 MAC 位址與另一台已同步的 VM 重複（通常是複製 VM 時未重新產生 MAC），
+   IP 位址仍會照常同步，但 MAC 關聯需要手動處理
+"""
+        self.send_telegram_notification(message)
+        print(f"📧 已發送 MAC 衝突通知: {vm_name} - {mac_address}")
+
     def send_enhanced_summary(self):
         if not hasattr(self, 'stats'):
             return
@@ -212,9 +221,6 @@ class OptimizedPVEToNetBoxSync:
                 message += f"• {error['vm_name']} - {error['ip_address']}\n"
             if len(self.error_log) > 5:
                 message += f"• ... 還有 {len(self.error_log) - 5} 個錯誤\n"
-        suppressed = getattr(self, '_telegram_suppressed', 0)
-        if suppressed:
-            message += f"\n📵 本輪通知已達上限，{suppressed} 則訊息被壓制（詳見 NetBox 漂移事件記錄）"
         self.send_telegram_notification(message)
 
     # ---------- API 連接 ----------
@@ -301,7 +307,6 @@ class OptimizedPVEToNetBoxSync:
             self.nb_cache['platforms'][platform.name.lower()] = platform
         for role in self.nb_api.dcim.device_roles.all():
             self.nb_cache['roles'][role.name.lower()] = role
-            self.nb_cache['device_roles'][role.name.lower()] = role
         for cluster in self.nb_api.virtualization.clusters.all():
             self.nb_cache['clusters'][str(cluster.id)] = cluster
         for site in self.nb_api.dcim.sites.all():
@@ -310,6 +315,8 @@ class OptimizedPVEToNetBoxSync:
             self.nb_cache['manufacturers'][manufacturer.name.lower()] = manufacturer
         for device_type in self.nb_api.dcim.device_types.all():
             self.nb_cache['device_types'][device_type.model.lower()] = device_type
+        for device_role in self.nb_api.dcim.device_roles.all():
+            self.nb_cache['device_roles'][device_role.name.lower()] = device_role
         for cluster_type in self.nb_api.virtualization.cluster_types.all():
             self.nb_cache['cluster_types'][cluster_type.name.lower()] = cluster_type
         elapsed = time.time() - start_time
@@ -366,13 +373,13 @@ class OptimizedPVEToNetBoxSync:
                     vm['node'] = node_name
                     all_vms.append(vm)
                 self.pve_cache['vms_by_node'][node_name] = all_vms
-                for vm in all_vms:
+                for vm in all_vms[:10]:
                     try:
                         if vm['type'] == 'qemu':
                             vm['config'] = self.pve_api.nodes(node_name).qemu(vm['vmid']).config.get()
                         else:
                             vm['config'] = self.pve_api.nodes(node_name).lxc(vm['vmid']).config.get()
-                    except Exception:
+                    except:
                         pass
             except Exception as e:
                 print(f"  加載節點 {node_name} 的虛擬機失敗: {e}")
@@ -939,33 +946,43 @@ class OptimizedPVEToNetBoxSync:
         return {}, {}
 
     # ---------- MAC 位址設定（NetBox 4.6+ 使用獨立 MACAddress 物件） ----------
-    def set_vm_interface_mac(self, vm_interface, mac_address: str):
-        """Create/update a dcim.MACAddress for the VMInterface and set it as primary."""
-        mac_lower = mac_address.lower()
-        existing_mac = self.nb_cache['mac_addresses'].get(mac_lower)
-        if existing_mac:
-            # Already exists — ensure it's assigned to this interface and set as primary
-            needs_save = False
-            if getattr(existing_mac, 'assigned_object_id', None) != vm_interface.id:
-                existing_mac.assigned_object_type = 'virtualization.vminterface'
-                existing_mac.assigned_object_id = vm_interface.id
-                existing_mac.save()
-            # Set primary_mac_address on the interface if not already set
-            current_primary = getattr(vm_interface, 'primary_mac_address', None)
-            if not current_primary or (hasattr(current_primary, 'id') and current_primary.id != existing_mac.id):
-                vm_interface.primary_mac_address = existing_mac.id
+    def set_vm_interface_mac(self, vm_interface, mac_address: str, vm_name: str = None):
+        """Create/update a dcim.MACAddress for the VMInterface and set it as primary.
+
+        Two different PVE VMs can legitimately report the same hardware-address
+        (e.g. a clone whose NIC MAC was never regenerated). NetBox refuses to
+        move a MACAddress that is another object's primary MAC, so failures here
+        are caught and reported instead of raised — a MAC conflict must never
+        block syncing the interface's IP address.
+        """
+        try:
+            mac_lower = mac_address.lower()
+            existing_mac = self.nb_cache['mac_addresses'].get(mac_lower)
+            if existing_mac:
+                # Already exists — ensure it's assigned to this interface and set as primary
+                if getattr(existing_mac, 'assigned_object_id', None) != vm_interface.id:
+                    existing_mac.assigned_object_type = 'virtualization.vminterface'
+                    existing_mac.assigned_object_id = vm_interface.id
+                    existing_mac.save()
+                # Set primary_mac_address on the interface if not already set
+                current_primary = getattr(vm_interface, 'primary_mac_address', None)
+                if not current_primary or (hasattr(current_primary, 'id') and current_primary.id != existing_mac.id):
+                    vm_interface.primary_mac_address = existing_mac.id
+                    vm_interface.save()
+            else:
+                # Create new MACAddress assigned to this VMInterface
+                mac_obj = self.nb_api.dcim.mac_addresses.create(
+                    mac_address=mac_address,
+                    assigned_object_type='virtualization.vminterface',
+                    assigned_object_id=vm_interface.id,
+                )
+                self.nb_cache['mac_addresses'][mac_lower] = mac_obj
+                # Set it as primary
+                vm_interface.primary_mac_address = mac_obj.id
                 vm_interface.save()
-        else:
-            # Create new MACAddress assigned to this VMInterface
-            mac_obj = self.nb_api.dcim.mac_addresses.create(
-                mac_address=mac_address,
-                assigned_object_type='virtualization.vminterface',
-                assigned_object_id=vm_interface.id,
-            )
-            self.nb_cache['mac_addresses'][mac_lower] = mac_obj
-            # Set it as primary
-            vm_interface.primary_mac_address = mac_obj.id
-            vm_interface.save()
+        except Exception as e:
+            print(f"      設定 MAC 位址失敗 {mac_address}: {e}")
+            self.log_mac_conflict_error(vm_name or "Unknown", mac_address, str(e))
 
     # ---------- VM 介面處理 ----------
     def process_vm_interfaces(self, vm, vm_config: Dict, agent_interfaces: Dict, mac_to_interface: Dict,
@@ -1001,7 +1018,7 @@ class OptimizedPVEToNetBoxSync:
                     if enabled_changed:
                         vm_interface.enabled = enabled
                     if mac_address:
-                        self.set_vm_interface_mac(vm_interface, mac_address)
+                        self.set_vm_interface_mac(vm_interface, mac_address, vm_name=vm.name)
                         # set_vm_interface_mac only saves the interface in some branches;
                         # ensure the enabled change is always persisted
                         if enabled_changed:
@@ -1015,7 +1032,7 @@ class OptimizedPVEToNetBoxSync:
                     vm_interface = self.nb_api.virtualization.interfaces.create(**iface_create_data)
                     self.nb_cache['vm_interfaces'].setdefault(vm.id, {})[config_key] = vm_interface
                     if mac_address:
-                        self.set_vm_interface_mac(vm_interface, mac_address)
+                        self.set_vm_interface_mac(vm_interface, mac_address, vm_name=vm.name)
                 iface_data = {
                     'name': config_key, 'mac_address': mac_address or '', 'bridge': bridge_name
                 }
@@ -1099,11 +1116,6 @@ class OptimizedPVEToNetBoxSync:
                 disks[key] = m.group(1) if m else val_str[:80]
         return disks
 
-    @staticmethod
-    def _norm_desc(s) -> str:
-        """Normalize description: unify line endings, strip whitespace."""
-        return (s or '').replace('\r\n', '\n').replace('\r', '\n').strip()
-
     def _write_drift_event(self, vm_name: str, vmid: int, drift_type: str,
                            field_name: str, old_value: str, new_value: str,
                            notified: bool = False):
@@ -1126,13 +1138,13 @@ class OptimizedPVEToNetBoxSync:
         except Exception as e:
             print(f"  ⚠ 無法寫入漂移事件記錄: {e}")
 
-    def detect_new_vm(self, vm_id: int, vm_name: str, node_name: str, last_config=None):
+    def detect_new_vm(self, vm_id: int, vm_name: str, node_name: str):
         """若此 vmid 在先前同步中不存在（但叢集有先前記錄），視為新增 VM。"""
         if not self.enhanced_mode or not self.state_db:
             return
         if not self._known_vmids:
             return  # 首次同步，無先前記錄，不觸發
-        if last_config:
+        if self.state_db.get_last_vm_config(vm_id, self.cluster_name):
             return  # 已有記錄，不是新增
         print(f"🆕 VM {vm_name} 新增偵測 (ID: {vm_id}, 節點: {node_name})")
         self.stats['config_drifts_detected'] += 1
@@ -1271,29 +1283,15 @@ class OptimizedPVEToNetBoxSync:
                 print(f"  🧹 Log 清理：刪除 {d_del} 筆漂移事件、{t_del} 筆操作記錄（>{days} 天）")
         except Exception as exc:
             print(f"  ⚠️ Log 清理失敗: {exc}")
-        # Also clean up state_db vm_config_history (keep latest row per VM, remove old history)
-        if self.state_db:
-            try:
-                from pve_sync_plugin.models import PvePluginSettings
-                days = PvePluginSettings.load().log_retention_days
-                if days:
-                    h_del = self.state_db.cleanup_old_history(self.cluster_name, days)
-                    if h_del:
-                        print(f"  🧹 state_db 清理：刪除 {h_del} 筆舊配置歷史（>{days} 天）")
-            except Exception as exc:
-                print(f"  ⚠️ state_db 歷史清理失敗: {exc}")
 
     def sync_vm_tasks(self):
         """從 PVE 各節點抓取 VM 操作記錄（建立/刪除/Clone/遷移）並同步到 NetBox。"""
-        import calendar
-        import datetime as _dt
         try:
             from django.utils import timezone
             from pve_sync_plugin.models import PveVmTaskLog
         except ImportError:
             return
 
-        _utc = _dt.timezone.utc
         VM_TASK_TYPES = {
             "qmcreate", "qmdestroy", "qmclone", "qmmigrate", "qmrestore",
             "qmstart", "qmstop", "qmshutdown", "qmreboot",
@@ -1327,18 +1325,10 @@ class OptimizedPVEToNetBoxSync:
             cluster_name=self.cluster_name
         ).order_by("-start_time").values_list("start_time", flat=True).first()
         if latest:
+            import calendar
             since_ts = int(calendar.timegm(latest.utctimetuple())) + 1
         else:
             since_ts = int(time.time()) - 30 * 86400
-
-        # Pre-fetch existing UPIDs in the window → O(1) lookup instead of N+1 queries
-        since_dt = _dt.datetime.fromtimestamp(since_ts, tz=_utc)
-        existing_upids: set = set(
-            PveVmTaskLog.objects
-            .filter(cluster_name=self.cluster_name, start_time__gte=since_dt)
-            .values_list("upid", flat=True)
-        )
-        new_upids: set = set()  # UPIDs inserted this run (avoid duplicates within same run)
         new_count = 0
 
         for node_name in self.pve_cache.get("vms_by_node", {}):
@@ -1362,8 +1352,8 @@ class OptimizedPVEToNetBoxSync:
                 except (ValueError, TypeError):
                     continue
 
-                # UPID 去重（使用預先撈取的集合，避免 N+1 查詢）
-                if upid in existing_upids or upid in new_upids:
+                # UPID 去重
+                if PveVmTaskLog.objects.filter(upid=upid).exists():
                     continue
 
                 task_type = task.get("type")
@@ -1373,6 +1363,8 @@ class OptimizedPVEToNetBoxSync:
                 status    = task.get("status", "") or ""
                 vm_name   = vmid_to_name.get(vmid, f"VM-{vmid}")
 
+                import datetime as _dt
+                _utc = _dt.timezone.utc
                 start_dt = _dt.datetime.fromtimestamp(start_ts, tz=_utc) if start_ts else timezone.now()
                 end_dt   = _dt.datetime.fromtimestamp(end_ts,   tz=_utc) if end_ts   else None
 
@@ -1389,7 +1381,6 @@ class OptimizedPVEToNetBoxSync:
                     status=status,
                     notified_telegram=False,
                 )
-                new_upids.add(upid)
                 new_count += 1
 
                 # Telegram 通知（僅近期且已完成的操作）
@@ -1417,12 +1408,11 @@ class OptimizedPVEToNetBoxSync:
 
     def detect_config_drift(self, vm_id: int, vm_name: str, vm_config: Dict[str, Any],
                             current_tags: List[str], network_interfaces: List[Dict] = None,
-                            current_node: str = None, last_config=None) -> List[Dict]:
+                            current_node: str = None) -> List[Dict]:
         if not self.enhanced_mode or not self.state_db:
             return []
         drifts = []
-        if last_config is None:
-            last_config = self.state_db.get_last_vm_config(vm_id, self.cluster_name)
+        last_config = self.state_db.get_last_vm_config(vm_id, self.cluster_name)
         if not last_config:
             return drifts
         current_memory = int(vm_config.get('memory', 0))
@@ -1503,8 +1493,8 @@ class OptimizedPVEToNetBoxSync:
                     self._write_drift_event(vm_name, vm_id, 'disk_change', k, old_v, new_v, notified=True)
 
         # VM 描述變更偵測
-        current_desc = self._norm_desc(vm_config.get('description'))[:500].strip()
-        old_desc = self._norm_desc(last_config.get('description'))[:500].strip()
+        current_desc = (vm_config.get('description') or '').strip()[:500]
+        old_desc = (last_config.get('description') or '').strip()
         if old_desc and old_desc != current_desc:
             self.stats['config_drifts_detected'] += 1
             print(f"📝 VM {vm_name} 描述變更")
@@ -1544,7 +1534,7 @@ class OptimizedPVEToNetBoxSync:
         return drifts
 
     def should_sync_vm(self, vm_id: int, vm_config: Dict[str, Any], tags: List[str],
-                       network_interfaces: List[Dict] = None, last_config=None) -> bool:
+                       network_interfaces: List[Dict] = None) -> bool:
         if not self.sync_config.get('incremental', True):
             return True
         if self.sync_config.get('force_full_sync', False):
@@ -1552,35 +1542,34 @@ class OptimizedPVEToNetBoxSync:
         current_hash = compute_config_hash(vm_config, tags, network_interfaces)
         if not self.state_db:
             return True
-        if last_config is None:
-            last_config = self.state_db.get_last_vm_config(vm_id, self.cluster_name)
+        last_config = self.state_db.get_last_vm_config(vm_id, self.cluster_name)
         if not last_config:
             return True
         if current_hash != last_config.get('config_hash'):
             return True
         return False
 
-    def detect_tag_changes(self, vm_id: int, vm_name: str, current_tags: List[str],
-                           last_config=None) -> List[Dict]:
+    def detect_tag_changes(self, vm_id: int, vm_name: str, current_tags: List[str]) -> List[Dict]:
         if not self.enhanced_mode or not self.state_db:
             return []
-        if last_config is None:
-            last_config = self.state_db.get_last_vm_config(vm_id, self.cluster_name)
+        changes = []
+        last_config = self.state_db.get_last_vm_config(vm_id, self.cluster_name)
         if not last_config:
-            return []
+            return changes
         old_tags = set(last_config.get('tags', []))
         new_tags = set(current_tags)
         added = new_tags - old_tags
         removed = old_tags - new_tags
-        changes = []
         for tag in added:
             changes.append({'type': 'added', 'tag': tag})
             self.stats['tag_changes'] += 1
             print(f"🏷️  VM {vm_name} 新增標籤: {tag}")
+            self._write_drift_event(vm_name, vm_id, 'tag_change', 'tags', '', tag, notified=False)
         for tag in removed:
             changes.append({'type': 'removed', 'tag': tag})
             self.stats['tag_changes'] += 1
             print(f"🏷️  VM {vm_name} 移除標籤: {tag}")
+            self._write_drift_event(vm_name, vm_id, 'tag_change', 'tags', tag, '', notified=False)
         if changes:
             added_list = [c['tag'] for c in changes if c['type'] == 'added']
             removed_list = [c['tag'] for c in changes if c['type'] == 'removed']
@@ -1596,12 +1585,7 @@ class OptimizedPVEToNetBoxSync:
                 f"📅 時間: {time.strftime('%Y-%m-%d %H:%M:%S')}\n\n"
                 f"{detail}\n請確認是否為預期變更。"
             )
-            notified = self.send_telegram_notification(tag_msg)
-            for c in changes:
-                if c['type'] == 'added':
-                    self._write_drift_event(vm_name, vm_id, 'tag_change', 'tags', '', c['tag'], notified=notified)
-                else:
-                    self._write_drift_event(vm_name, vm_id, 'tag_change', 'tags', c['tag'], '', notified=notified)
+            self.send_telegram_notification(tag_msg)
         return changes
 
     def detect_ip_change(self, vm_id: int, vm_name: str, current_ip: str):
@@ -1669,16 +1653,10 @@ class OptimizedPVEToNetBoxSync:
                             tag_ids.append(tag.id)
                         except Exception as e:
                             print(f"  建立標籤失敗 {tag_name}: {e}")
-        # Read last_config once; share with detect_new_vm / detect_tag_changes /
-        # detect_config_drift / should_sync_vm to avoid repeated SQLite round-trips.
-        last_config = (
-            self.state_db.get_last_vm_config(int(vm_id), self.cluster_name)
-            if (self.enhanced_mode and self.state_db) else None
-        )
         if self.enhanced_mode:
-            self.detect_new_vm(int(vm_id), original_vm_name, node_name, last_config=last_config)
+            self.detect_new_vm(int(vm_id), original_vm_name, node_name)
         if self.enhanced_mode and tag_names:
-            self.detect_tag_changes(int(vm_id), original_vm_name, tag_names, last_config=last_config)
+            self.detect_tag_changes(int(vm_id), original_vm_name, tag_names)
         vm_status = vm_data.get('status', '')
         if tag_names and vm_status in ('running', 'stopped'):
             first_tag = tag_names[0].lower()
@@ -1761,9 +1739,8 @@ class OptimizedPVEToNetBoxSync:
                 })
         if self.enhanced_mode:
             self.detect_config_drift(int(vm_id), original_vm_name, vm_config, tag_names,
-                                     network_interfaces, current_node=node_name, last_config=last_config)
-            if not force and not self.should_sync_vm(int(vm_id), vm_config, tag_names, network_interfaces,
-                                                     last_config=last_config):
+                                     network_interfaces, current_node=node_name)
+            if not force and not self.should_sync_vm(int(vm_id), vm_config, tag_names, network_interfaces):
                 cached_vm = self.nb_cache['virtual_machines_by_serial'].get(f"{vm_id}::{cluster['id']}")
                 if cached_vm is None:
                     # VM 在 NetBox 中不存在（例如隨節點被連帶刪除），
@@ -1804,7 +1781,7 @@ class OptimizedPVEToNetBoxSync:
                                 vm_id=int(vm_id), cluster_name=self.cluster_name, config_hash=config_hash,
                                 memory=memory, vcpus=vcpus_save, tags=tag_names,
                                 node=node_name, primary_ip=cur_ip, vm_name=original_vm_name,
-                                description=self._norm_desc(vm_config.get('description'))[:500].strip(),
+                                description=(vm_config.get('description') or '').strip()[:500],
                                 disk_summary=json.dumps(self._parse_disk_summary(vm_config)),
                             )
                         except Exception as _snap_err:
@@ -1861,7 +1838,7 @@ class OptimizedPVEToNetBoxSync:
                 update_data = {
                     'name': vm_name, 'cluster': cluster['id'], 'device': device.id, 'role': role_id,
                     'vcpus': vcpus, 'memory': int(vm_config.get('memory', 0)), 'status': status,
-                    'description': self._norm_desc(vm_config.get('description'))[:200], 'platform': platform_id,
+                    'description': (vm_config.get('description') or '')[:200], 'platform': platform_id,
                     'start_on_boot': boot_choice
                 }
                 if tag_ids:
@@ -1880,7 +1857,7 @@ class OptimizedPVEToNetBoxSync:
                 vm_data_dict = {
                     'serial': vm_id, 'name': vm_name, 'cluster': cluster['id'], 'device': device.id,
                     'role': role_id, 'vcpus': vcpus, 'memory': int(vm_config.get('memory', 0)),
-                    'status': status, 'description': self._norm_desc(vm_config.get('description'))[:200],
+                    'status': status, 'description': (vm_config.get('description') or '')[:200],
                     'platform': platform_id, 'start_on_boot': boot_choice
                 }
                 if tag_ids:
@@ -1917,7 +1894,7 @@ class OptimizedPVEToNetBoxSync:
                     vm_id=int(vm_id), cluster_name=self.cluster_name, config_hash=config_hash,
                     memory=memory, vcpus=vcpus_save, tags=tag_names,
                     node=node_name, primary_ip=primary_ip_str, vm_name=original_vm_name,
-                    description=self._norm_desc(vm_config.get('description'))[:500].strip(),
+                    description=(vm_config.get('description') or '').strip()[:500],
                     disk_summary=json.dumps(self._parse_disk_summary(vm_config)),
                 )
             return True
@@ -1942,7 +1919,7 @@ class OptimizedPVEToNetBoxSync:
                             node=node_name,
                             primary_ip=_ip,
                             vm_name=original_vm_name,
-                            description=self._norm_desc(vm_config.get('description'))[:500].strip(),
+                            description=(vm_config.get('description') or '').strip()[:500],
                             disk_summary=json.dumps(self._parse_disk_summary(vm_config)),
                         )
                 except Exception:
@@ -1985,18 +1962,6 @@ class OptimizedPVEToNetBoxSync:
         print("=" * 50)
         if self.enhanced_mode and self.state_db:
             self.sync_log_id = self.state_db.start_sync_log(self.cluster_name)
-        # Load notify_on_sync from PveClusterConfig (DB setting takes precedence over default True)
-        self.notify_on_sync = True
-        try:
-            from pve_sync_plugin.models import PveClusterConfig
-            cfg = PveClusterConfig.objects.filter(name=self.cluster_name).first()
-            if cfg is not None:
-                self.notify_on_sync = cfg.notify_on_sync
-        except Exception:
-            pass
-        # Reset per-sync Telegram counters
-        self._telegram_sent = 0
-        self._telegram_suppressed = 0
         sync_mode = "增量" if self.sync_config.get('incremental', True) else "全量"
         start_message = f"""
 🔄 <b>PVE-NetBox 同步開始</b>

{proxsync-2.0.7 → proxsync-2.0.8}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "proxsync"
-version = "2.0.7"
+version = "2.0.8"
 description = "NetBox plugin for Proxmox VE inventory synchronization"
 readme = "README.md"
 requires-python = ">=3.10"
@@ -36,9 +36,9 @@ dependencies = [
 ]
 
 [project.urls]
-Homepage = "https://github.com/lixiujie85/proxsync"
-Source = "https://github.com/lixiujie85/proxsync"
-Issues = "https://github.com/lixiujie85/proxsync/issues"
+Homepage = "https://github.com/xiujie2/proxsync"
+Source = "https://github.com/xiujie2/proxsync"
+Issues = "https://github.com/xiujie2/proxsync/issues"
 
 [project.optional-dependencies]
 standalone = [

{proxsync-2.0.7 → proxsync-2.0.8}/state_db.py

@@ -154,7 +154,7 @@ class StateDB:
             with sqlite3.connect(self.db_path) as conn:
                 conn.execute(
                     """
-                    INSERT OR REPLACE INTO vm_config_history
+                    INSERT INTO vm_config_history
                     (vm_id, cluster_name, config_hash, memory, vcpus, tags_json,
                      node, primary_ip, vm_name, description, disk_summary)
                     VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
@@ -256,28 +256,6 @@ class StateDB:
                     })
                 return changes
 
-    def cleanup_old_history(self, cluster_name: str, days: int) -> int:
-        """Delete vm_config_history rows older than `days`, keeping the latest row per VM."""
-        import datetime as _dt
-        cutoff = (_dt.datetime.utcnow() - _dt.timedelta(days=days)).strftime('%Y-%m-%d %H:%M:%S')
-        with self._lock:
-            with sqlite3.connect(self.db_path) as conn:
-                result = conn.execute(
-                    """
-                    DELETE FROM vm_config_history
-                    WHERE cluster_name = ?
-                      AND sync_time < ?
-                      AND sync_time NOT IN (
-                          SELECT MAX(sync_time) FROM vm_config_history
-                          WHERE cluster_name = ?
-                          GROUP BY vm_id
-                      )
-                    """,
-                    (cluster_name, cutoff, cluster_name),
-                )
-                conn.commit()
-                return result.rowcount
-
     # ========== 节点状态历史 ==========
 
     def save_node_status(self, node_name: str, cluster_name: str, status: str):
@@ -529,7 +507,7 @@ def compute_config_hash(vm_config: Dict[str, Any], tags: List[str], network_inte
         'ostype': vm_config.get('ostype', ''),
         'cores': vm_config.get('cores', 1),
         'sockets': vm_config.get('sockets', 1),
-        'description': (vm_config.get('description') or '').replace('\r\n', '\n').replace('\r', '\n').strip()[:200].strip(),
+        'description': vm_config.get('description', '')[:200],  # 限制长度
         'tags': sorted(tags),  # 排序以确保一致性
         'networks': net_config  # 添加网络接口配置
     }