mirror of
https://github.com/ansible/awx.git
synced 2024-11-01 08:21:15 +03:00
Remove limit on limit
field.
This allows 'relaunch-on-failed' on an arbitrary number of failed hosts.
This commit is contained in:
parent
7333e55748
commit
4b669fb16d
@ -3618,11 +3618,6 @@ class JobRelaunch(RetrieveAPIView):
|
|||||||
'Cannot relaunch because previous job had 0 {status_value} hosts.'
|
'Cannot relaunch because previous job had 0 {status_value} hosts.'
|
||||||
).format(status_value=retry_hosts)}, status=status.HTTP_400_BAD_REQUEST)
|
).format(status_value=retry_hosts)}, status=status.HTTP_400_BAD_REQUEST)
|
||||||
copy_kwargs['limit'] = ','.join(retry_host_list)
|
copy_kwargs['limit'] = ','.join(retry_host_list)
|
||||||
limit_length = len(copy_kwargs['limit'])
|
|
||||||
if limit_length > 1024:
|
|
||||||
return Response({'limit': _(
|
|
||||||
'Cannot relaunch because the limit length {limit_length} exceeds the max of {limit_max}.'
|
|
||||||
).format(limit_length=limit_length, limit_max=1024)}, status=status.HTTP_400_BAD_REQUEST)
|
|
||||||
|
|
||||||
new_job = obj.copy_unified_job(**copy_kwargs)
|
new_job = obj.copy_unified_job(**copy_kwargs)
|
||||||
result = new_job.signal_start(**serializer.validated_data['credential_passwords'])
|
result = new_job.signal_start(**serializer.validated_data['credential_passwords'])
|
||||||
|
25
awx/main/migrations/0058_v350_remove_limit_limit.py
Normal file
25
awx/main/migrations/0058_v350_remove_limit_limit.py
Normal file
@ -0,0 +1,25 @@
|
|||||||
|
# -*- coding: utf-8 -*-
|
||||||
|
# Generated by Django 1.11.16 on 2019-02-05 18:29
|
||||||
|
from __future__ import unicode_literals
|
||||||
|
|
||||||
|
from django.db import migrations, models
|
||||||
|
|
||||||
|
|
||||||
|
class Migration(migrations.Migration):
|
||||||
|
|
||||||
|
dependencies = [
|
||||||
|
('main', '0057_v350_remove_become_method_type'),
|
||||||
|
]
|
||||||
|
|
||||||
|
operations = [
|
||||||
|
migrations.AlterField(
|
||||||
|
model_name='job',
|
||||||
|
name='limit',
|
||||||
|
field=models.TextField(blank=True, default=''),
|
||||||
|
),
|
||||||
|
migrations.AlterField(
|
||||||
|
model_name='jobtemplate',
|
||||||
|
name='limit',
|
||||||
|
field=models.TextField(blank=True, default=''),
|
||||||
|
),
|
||||||
|
]
|
@ -93,8 +93,7 @@ class JobOptions(BaseModel):
|
|||||||
blank=True,
|
blank=True,
|
||||||
default=0,
|
default=0,
|
||||||
)
|
)
|
||||||
limit = models.CharField(
|
limit = models.TextField(
|
||||||
max_length=1024,
|
|
||||||
blank=True,
|
blank=True,
|
||||||
default='',
|
default='',
|
||||||
)
|
)
|
||||||
|
Loading…
Reference in New Issue
Block a user