Merge pull request #60 from chadell/release-v1.2.0

Release v1.2.0

Author: chadell

.github/workflows/ci.yml

@@ -103,8 +103,8 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        python-version: ["3.6", "3.7", "3.8", "3.9"]
-        nautobot-version: ["1.0.3", "1.1.4"]
+        python-version: ["3.7", "3.8", "3.9"]
+        nautobot-version: ["1.2.11", "stable"]
     runs-on: "ubuntu-20.04"
     env:
       INVOKE_NAUTOBOT_SSOT_PYTHON_VER: "${{ matrix.python-version }}"

development/Dockerfile

@@ -3,6 +3,8 @@ ARG PYTHON_VER
 ARG NAUTOBOT_VER
 FROM ghcr.io/nautobot/nautobot-dev:${NAUTOBOT_VER}-py${PYTHON_VER}
 
+ARG NAUTOBOT_VER
+
 ENV prometheus_multiproc_dir=/prom_cache
 
 ARG NAUTOBOT_ROOT=/opt/nautobot
@@ -19,17 +21,26 @@ RUN poetry config virtualenvs.create false \
 # Install Nautobot Plugin
 # -------------------------------------------------------------------------------------
 
-WORKDIR /source
+WORKDIR /tmp/install
 
 # Copy in only pyproject.toml/poetry.lock to help with caching this layer if no updates to dependencies
-COPY poetry.lock pyproject.toml /source/
+COPY poetry.lock pyproject.toml /tmp/install/
+
+# Add the version of Nautobot to the pyproject toml for testing
+RUN if [ "$NAUTOBOT_VER" = "stable" ]; \
+    then poetry add nautobot@latest; \
+    else poetry add nautobot==$NAUTOBOT_VER; \
+    fi
 
 # --no-root declares not to install the project package since we're wanting to take advantage of caching dependency installation
 # and the project is copied in and installed after this step
 RUN poetry install --no-interaction --no-ansi --no-root
 
 # Copy in the rest of the source code and install local Nautobot plugin
+WORKDIR /source
 COPY . /source
-RUN poetry install --no-interaction --no-ansi
+# Copy updated Poetry files to override the Poetry files from the project.
+# This will make sure that the correct Nautobot version is used.
+RUN cp /tmp/install/* /source/ && poetry install --no-interaction --no-ansi
 
 COPY development/nautobot_config.py /opt/nautobot/nautobot_config.py

docs/changelog.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## v1.2.0 - 2022-09-30
+
+### Fixed
+
+- [#58](https://github.com/nautobot/nautobot-plugin-ssot/pull/58) - Change `message` and `object_repr` from 200 `CharField` field, to a `TextField`
+
+### Changed
+
+- [#56](https://github.com/nautobot/nautobot-plugin-ssot/pull/56) - Drop Python 3.6 support
+
 ## v1.1.2 - 2022-05-09
 
 ### Fixed

nautobot_ssot/__init__.py

@@ -1,5 +1,5 @@
 """Plugin declaration for nautobot_ssot."""
-
+# flake8: noqa
 try:
     from importlib import metadata
 except ImportError:

nautobot_ssot/filters.py

@@ -8,7 +8,7 @@
 from .models import Sync, SyncLogEntry
 
 
-class SyncFilter(BaseFilterSet):
+class SyncFilterSet(BaseFilterSet):
     """Filter capabilities for SyncOverview instances."""
 
     class Meta:
@@ -18,7 +18,7 @@ class Meta:
         fields = ["dry_run", "job_result"]
 
 
-class SyncLogEntryFilter(BaseFilterSet):
+class SyncLogEntryFilterSet(BaseFilterSet):
     """Filter capabilities for SyncLogEntry instances."""
 
     q = django_filters.CharFilter(method="search", label="Search")

nautobot_ssot/jobs/base.py

@@ -4,6 +4,7 @@
 import traceback
 import tracemalloc
 from typing import Iterable
+from packaging.version import Version
 
 from django.forms import HiddenInput
 from django.templatetags.static import static
@@ -16,6 +17,7 @@
 from diffsync.enum import DiffSyncFlags
 import structlog
 
+from nautobot import __version__ as nautobot_version
 from nautobot.extras.jobs import BaseJob, BooleanVar
 
 from nautobot_ssot.choices import SyncLogEntryActionChoices
@@ -268,9 +270,12 @@ def __init__(self):
         # Default diffsync flags. You can overwrite them at any time.
         self.diffsync_flags = DiffSyncFlags.CONTINUE_ON_FAILURE | DiffSyncFlags.LOG_UNCHANGED_RECORDS
 
-    def as_form(self, data=None, files=None, initial=None):
+    def as_form(self, data=None, files=None, initial=None, approval_view=False):
         """Render this instance as a Django form for user inputs, including a "Dry run" field."""
-        form = super().as_form(data=data, files=files, initial=initial)
+        if Version(nautobot_version) < Version("1.2"):
+            form = super().as_form(data=data, files=files, initial=initial)
+        else:
+            form = super().as_form(data=data, files=files, initial=initial, approval_view=approval_view)
         # Set the "dry_run" widget's initial value based on our Meta attribute, if any
         form.fields["dry_run"].initial = getattr(self.Meta, "dry_run_default", True)
         # Hide the "commit" widget to reduce user confusion

nautobot_ssot/jobs/examples.py

@@ -1,4 +1,7 @@
 """Sample data-source and data-target Jobs."""
+# Skip colon check for multiple statements on one line.
+# flake8: noqa: E701
+
 from typing import Optional, Mapping
 from uuid import UUID
 from django.contrib.contenttypes.models import ContentType

nautobot_ssot/migrations/0003_alter_synclogentry_textfields.py

@@ -0,0 +1,23 @@
+# Generated by Django 3.2.15 on 2022-09-27 13:52
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("nautobot_ssot", "0002_performance_metrics"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="synclogentry",
+            name="message",
+            field=models.TextField(blank=True),
+        ),
+        migrations.AlterField(
+            model_name="synclogentry",
+            name="object_repr",
+            field=models.TextField(blank=True, default="", editable=False),
+        ),
+    ]

nautobot_ssot/models.py

@@ -161,9 +161,9 @@ class SyncLogEntry(BaseModel):
     synced_object_id = models.UUIDField(blank=True, null=True)
     synced_object = GenericForeignKey(ct_field="synced_object_type", fk_field="synced_object_id")
 
-    object_repr = models.CharField(max_length=200, blank=True, default="", editable=False)
+    object_repr = models.TextField(blank=True, default="", editable=False)
 
-    message = models.CharField(max_length=511, blank=True)
+    message = models.TextField(blank=True)
 
     class Meta:
         """Metaclass attributes of SyncLogEntry."""

nautobot_ssot/tests/__init__.py

@@ -1 +1,7 @@
 """Unit tests for nautobot_ssot plugin."""
+
+from django.conf import settings
+
+if "job_logs" in settings.DATABASES:
+    settings.DATABASES["job_logs"] = settings.DATABASES["job_logs"].copy()
+    settings.DATABASES["job_logs"]["TEST"] = {"MIRROR": "default"}