External-Mirrors/awx
2
0
Fork 0
mirror of https://github.com/ansible/awx.git synced 2026-05-08 01:47:35 -02:30
Code Issues Packages Projects Releases Wiki Activity

Show error if no Execution Environment is found on project sync/job run

Browse Source
This commit is contained in:
beeankha
2021-05-12 11:52:12 -04:00
parent e970620672
commit fd5f3a82d2
2 changed files with 41 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
awx/main/tasks.py
View File

@@ -994,6 +994,9 @@ class BaseTask(object):
env['AWX_PRIVATE_DATA_DIR'] = private_data_dir env['AWX_PRIVATE_DATA_DIR'] = private_data_dir
if self.instance.execution_environment is None:
raise RuntimeError('The project could not sync because there is no Execution Environment.')
ee_cred = self.instance.execution_environment.credential ee_cred = self.instance.execution_environment.credential
if ee_cred: if ee_cred:
verify_ssl = ee_cred.get_input('verify_ssl') verify_ssl = ee_cred.get_input('verify_ssl')
@@ -1709,6 +1712,10 @@ class RunJob(BaseTask):
error = _('Job could not start because it does not have a valid project.') error = _('Job could not start because it does not have a valid project.')
self.update_model(job.pk, status='failed', job_explanation=error) self.update_model(job.pk, status='failed', job_explanation=error)
raise RuntimeError(error) raise RuntimeError(error)
elif job.execution_environment is None:
error = _('Job could not start because no Execution Environment could be found.')
self.update_model(job.pk, status='error', job_explanation=error)
raise RuntimeError(error)
elif job.project.status in ('error', 'failed'): elif job.project.status in ('error', 'failed'):
msg = _('The project revision for this job template is unknown due to a failed update.') msg = _('The project revision for this job template is unknown due to a failed update.')
job = self.update_model(job.pk, status='failed', job_explanation=msg) job = self.update_model(job.pk, status='failed', job_explanation=msg)

34
awx/main/tests/unit/test_tasks.py
View File

@@ -1927,3 +1927,37 @@ def test_notification_job_finished(mocker):
with mocker.patch('awx.main.models.UnifiedJob.objects.get', mocker.MagicMock(return_value=uj)): with mocker.patch('awx.main.models.UnifiedJob.objects.get', mocker.MagicMock(return_value=uj)):
tasks.handle_success_and_failure_notifications(1) tasks.handle_success_and_failure_notifications(1)
uj.send_notification_templates.assert_called() uj.send_notification_templates.assert_called()
@pytest.mark.django_db
class TestNullExecutionEnvironment(TestJobExecution):
def test_job_run_no_ee(self, execution_environment, private_data_dir, job):
org = Organization(pk=1)
proj = Project(pk=1, organization=org)
job = Job(project=proj, organization=org, inventory=Inventory(pk=1))
job.execution_environment = None
task = tasks.RunJob()
task.instance = job
task.update_model = mock.Mock(return_value=job)
task.model.objects.get = mock.Mock(return_value=job)
with mock.patch('awx.main.tasks.copy_tree'):
with pytest.raises(RuntimeError) as e:
task.pre_run_hook(job, private_data_dir)
update_model_call = task.update_model.call_args[1]
assert update_model_call['status'] == 'error'
assert 'Job could not start because no Execution Environment could be found' in str(e.value)
def test_project_update_no_ee(self, execution_environment, job):
org = Organization(pk=1)
proj = Project(pk=1, organization=org)
project_update = ProjectUpdate(pk=1, project=proj, scm_type='git')
project_update.execution_environment = None
task = tasks.RunProjectUpdate()
task.instance = project_update
with pytest.raises(RuntimeError) as e:
task.build_env(job, {})
assert 'The project could not sync because there is no Execution Environment' in str(e.value)