edit original migration file, add blank string as acceptable to model

awx/main/migrations/0126_executionenvironment_container_options.py

@@ -13,6 +13,6 @@ class Migration(migrations.Migration):
         migrations.AddField(
             model_name='executionenvironment',
             name='pull',
-            field=models.CharField(choices=[('always', 'Always pull container before running.'), ('missing', 'No pull option has been selected.'), ('never', 'Never pull container before running.')], default='missing', help_text='Pull image before running?', max_length=1024),
+            field=models.CharField(choices=[('always', 'Always pull container before running.'), ('missing', 'No pull option has been selected.'), ('never', 'Never pull container before running.')], blank=True, default='', help_text='Pull image before running?', max_length=16),
         ),
     ]

awx/main/migrations/0128_set_default_pull_to_none.py

@@ -1,18 +0,0 @@
-# Generated by Django 2.2.16 on 2021-02-16 20:38
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('main', '0127_reset_pod_spec_override'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='executionenvironment',
-            name='pull',
-            field=models.CharField(choices=[('always', 'Always pull container before running.'), ('missing', 'No pull option has been selected.'), ('never', 'Never pull container before running.')], default=None, help_text='Pull image before running?', max_length=1024),
-        ),
-    ]

awx/main/models/execution_environments.py

@@ -42,9 +42,10 @@ class ExecutionEnvironment(CommonModel):
         on_delete=models.SET_NULL,
     )
     pull = models.CharField(
-        max_length=1024,
+        max_length=16,
         choices=PULL_CHOICES,
-        default=None,
+        blank=True,
+        default='',
         help_text=_('Pull image before running?'),
     )
 

awx_collection/plugins/modules/tower_execution_environment.py

@@ -54,7 +54,7 @@ options:
       description:
         - determine image pull behavior
       choices: ["always", "missing", "never"]
-      default: None
+      default: ''
       type: str
 extends_documentation_fragment: awx.awx.auth
 '''

awxkit/awxkit/api/pages/execution_environments.py

@@ -21,7 +21,7 @@ class ExecutionEnvironment(HasCreate, base.Base):
     NATURAL_KEY = ('name',)
 
     # fields are name, image, organization, managed_by_tower, credential
-    def create(self, name='', image='quay.io/ansible/ansible-runner:devel', credential=None, pull=None, **kwargs):
+    def create(self, name='', image='quay.io/ansible/ansible-runner:devel', credential=None, pull='', **kwargs):
         # we do not want to make a credential by default
         payload = self.create_payload(name=name, image=image, credential=credential, pull=pull, **kwargs)
         ret = self.update_identity(ExecutionEnvironments(self.connection).post(payload))
@@ -33,13 +33,13 @@ class ExecutionEnvironment(HasCreate, base.Base):
         payload.ds = DSAdapter(self.__class__.__name__, self._dependency_store)
         return payload
 
-    def payload(self, name='', image=None, organization=None, credential=None, pull=None, **kwargs):
+    def payload(self, name='', image=None, organization=None, credential=None, pull='', **kwargs):
         payload = PseudoNamespace(
             name=name or "EE - {}".format(random_title()),
             image=image or random_title(10),
             organization=organization.id if organization else None,
             credential=credential.id if credential else None,
-            pull=pull if pull else None,
+            pull=pull,
             **kwargs
         )