Bug 1837717
| Summary: | Ansible temporary files are not removed from the /tmp directory. | |||
|---|---|---|---|---|
| Product: | Red Hat Satellite | Reporter: | Rudnei Bertol Jr. <rbertolj> | |
| Component: | Ansible - Configuration Management | Assignee: | Suraj Patil <supatil> | |
| Status: | CLOSED ERRATA | QA Contact: | Lukas Pramuk <lpramuk> | |
| Severity: | medium | Docs Contact: | ||
| Priority: | unspecified | |||
| Version: | 6.6.0 | CC: | aruzicka, bbuckingham, dleroux, egolov, inecas, mkalyat, oezr, pcreech, rankumar, sadas, supatil, wpinheir | |
| Target Milestone: | 6.9.0 | Keywords: | Triaged | |
| Target Release: | Unused | |||
| Hardware: | x86_64 | |||
| OS: | Linux | |||
| Whiteboard: | ||||
| Fixed In Version: | tfm-rubygem-foreman_ansible_core-4.0.0 | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | ||
| Clone Of: | ||||
| : | 1919415 (view as bug list) | Environment: | ||
| Last Closed: | 2021-04-21 13:14:53 UTC | Type: | Bug | |
| Regression: | --- | Mount Type: | --- | |
| Documentation: | --- | CRM: | ||
| Verified Versions: | Category: | --- | ||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
| Cloudforms Team: | --- | Target Upstream Version: | ||
| Embargoed: | ||||
|
Description
Rudnei Bertol Jr.
2020-05-19 19:49:51 UTC
Created redmine issue https://projects.theforeman.org/issues/30518 from this bug Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/30518 has been resolved. FailedQA.
@Satellite 6.8.0 Snap16
tfm-rubygem-foreman_ansible_core-3.0.3-1.el7sat.noarch
by the following reproducer:
1) Monitor /tmp directory for foreman-proxy content:
# watch "ls -lt /tmp | grep foreman-proxy"
2) Run ansible REX command "sleep 60" against a host while monitoring /tmp directory
Before run:
<empty>
During run:
drwx------. 3 foreman-proxy foreman-proxy 29 Sep 21 19:06 ansible-local-186745lJmoV
drwx------. 5 foreman-proxy foreman-proxy 67 Sep 21 19:06 d20200921-2455-19kslzo
After run:
drwx------. 5 foreman-proxy foreman-proxy 67 Sep 21 19:06 d20200921-2455-19kslzo
>>> d<date>-* dirs are not deleted
Oops.
The fix is not present, rel-eng failure:
# grep -A3 'def close' /opt/theforeman/tfm/root/usr/share/gems/gems/foreman_ansible_core-*/lib/foreman_ansible_core/runner/ansible_runner.rb
<empty>
I miss these lines being added:
---
def close
super
FileUtils.remove_entry(@root) if @tmp_working_dir
end
---
Hey Guys, Thank you for working hard on this BZ, we appreciate it. Working on further case issues, we had the necessity to use those files to troubleshoot some ansible-runner/execution issues. Is there a possibility to add an option on the Satellite settings to keep those files? Like an option to debug those files if we need it. regards rbertol Hi rberol, unfortunatelly this is not possible as of now. Could you please create separate BZ for that? Hey Ondřej, We have filled the RFE https://bugzilla.redhat.com/show_bug.cgi?id=1915394 requesting this new feature. Thank you. rbertol VERIFIED.
@Satellite 6.9.0 Snap13
tfm-rubygem-foreman_ansible_core-4.0.0-1.el7sat.noarch
by the following reproducer:
1) Monitor /tmp directory for foreman-proxy content:
# watch "ls -lt /tmp | grep foreman-proxy"
2) Run ansible REX command "sleep 60" against a host while monitoring /tmp directory
Before run:
<empty>
During run:
drwx------. 3 foreman-proxy foreman-proxy 29 Feb 15 07:26 ansible-local-285640GtC4z
drwx------. 5 foreman-proxy foreman-proxy 67 Feb 15 07:26 d20210215-10703-1uz4wsf
After run:
<empty>
>>> temporary working dirs are deleted after the job finished
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory (Moderate: Satellite 6.9 Release), and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2021:1313 |