Alan Rominger

Results 284 comments of Alan Rominger

No, that doesn't fully cover this. This suggestion is much more nuanced. If we do #5518, the need for _this_ will be greatly reduced. I'll adjust tags somewhat to reflect...

Have you tried changing the first playbook to use `no_log`, like ```yaml --- - hosts: localhost no_log: True tasks: - set_stats: data: message: "SECRET VALUE" no_log: true ``` I'm trying...

There was an attempt https://github.com/ansible/awx/blob/889dae357b6448519b3c68c77068bedfe068a6a8/awx/main/models/workflow.py#L254-L257 It was combined into the same data structure used for tracking survey passwords (although since that time, management of survey passwords has undergone substantial changes)....

We did recently introduce instance group roles. This includes a read-level role. https://github.com/ansible/awx/pull/13584

Do you know what method this happens from? I ask, because saving facts post-run is batched to 100 hosts at a time. https://github.com/ansible/awx/blob/f377b5fdde44a73d2e52f33d0574e85ad99156be/awx/main/tasks/facts.py#L138-L140 So then it would be helpful to...

@kdelee you just merged https://github.com/ansible/awx/pull/15978 and that is created for an OOM situation, but it did not attempt to fix or identify the original cause of the OOM. I'm leaving...

``` File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/rest_framework/views.py", line 506, in dispatch response = handler(request, *args, **kwargs) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/awx_devel/awx/api/views/__init__.py", line 3674, in get logging.warning(f"JobEventChildrenSummary: job event 'event' field is unexpectedly empty for job {job.id}")...

Now it gives this which is intended: ``` File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/_pytest/python.py", line 157, in pytest_pyfunc_call result = testfunction(**testargs) ^^^^^^^^^^^^^^^^^^^^^^^^ File "/awx_devel/awx/main/tests/functional/api/test_events.py", line 173, in test_job_job_events_children_summary_empty_event response = get(url, user=objs.superusers.admin, expect=200) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^...

Running the development environment seems to go fine with this. Additionally, making sure methods are well-behaved: ``` In [1]: from awx.main.tasks.system import * In [2]: load_inventory_plugins() In [3]: load_credential_types_feature() ```...

Leaning towards closing this, as talking with @elyezer we would prefer consolidation management commands and doing this in one of those.