Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

The task "store facts from all hosts in a variable" is failing when the target machine is fedora #472

Closed
elyezer opened this issue Nov 14, 2017 · 0 comments
Assignees
Milestone

Comments

@elyezer
Copy link

elyezer commented Nov 14, 2017

Specify type:

  • Bug

Bug severity (if applicable):

  • High

Description:

The task store facts from all hosts in a variable is failing when scanning for redhat-packages.is_redhat when the target machine is not RHEL.

This makes rho scan return a non-zero which indicates it has failed.


Bug Report

Version of rho:

0.30 (master at 7d67baa)

Expected behavior:

rho should be able to complete a scan no matter what subset of facts is provided

Actual behavior:

rho is not able to complete a scan when a subset of facts is provided

Steps to reproduce:

Run rho scan --profile 1 --facts redhat-packages.is_redhat --reportfile report.csv where profile 1 have at least one Fedora host.

Environment information:

Both the scanning system and the target were Fedora 26 systems. Also faced that when both were Fedora 25.

The other setup was scanning system Fedora 26 and target systems were Fedora 26 and RHEL 7.4. In this case RHEL7 worked and Fedora 26 failed.

@chambridge chambridge self-assigned this Nov 14, 2017
@chambridge chambridge added this to the Sprint 11 milestone Nov 14, 2017
chambridge added a commit that referenced this issue Nov 29, 2017
* master:
  Add jboss.eap.init-files to JBOSS_FACTS. Closes #478. (#479)
  Add default dictionaries to handle lost host. Handle ansible rc if csv was successfully created. Closes #475. (#476)
  Fix addition of certs for wrong when condition. Closes #472. (#474)
  Present more status during discovery for larger environments. Closes #442. (#467)
  revert default logging change. Closes #460. (#461)
  Check for empty vault password. Closes #444. (#457)
  Highlight potential issues in the console output. Closes #443. (#458)
  Default verbosity to 1 for improved log data. Closes #441. (#456)
  Turn off colors in the ansible log. Closes #448. (#455)
  Fix task that can hang if systemctl paginates
  Update playbook to use free strategy. Closes #445. (#453)
  Merge 0.30 version into master branch (#451)

# Conflicts:
#	rho/ansible_utils.py
#	rho/facts.py
#	rho/inventory_scan.py
#	rho/postprocessing.py
#	rho/scancommand.py
#	roles/jboss_eap/tasks/main.yml
#	roles/write/tasks/main.yml
#	test/test_postprocessing.py
chambridge added a commit that referenced this issue Nov 29, 2017
* Merge branch 'master' into dev

* master:
  Add jboss.eap.init-files to JBOSS_FACTS. Closes #478. (#479)
  Add default dictionaries to handle lost host. Handle ansible rc if csv was successfully created. Closes #475. (#476)
  Fix addition of certs for wrong when condition. Closes #472. (#474)
  Present more status during discovery for larger environments. Closes #442. (#467)
  revert default logging change. Closes #460. (#461)
  Check for empty vault password. Closes #444. (#457)
  Highlight potential issues in the console output. Closes #443. (#458)
  Default verbosity to 1 for improved log data. Closes #441. (#456)
  Turn off colors in the ansible log. Closes #448. (#455)
  Fix task that can hang if systemctl paginates
  Update playbook to use free strategy. Closes #445. (#453)
  Merge 0.30 version into master branch (#451)

# Conflicts:
#	rho/ansible_utils.py
#	rho/facts.py
#	rho/inventory_scan.py
#	rho/postprocessing.py
#	rho/scancommand.py
#	roles/jboss_eap/tasks/main.yml
#	roles/write/tasks/main.yml
#	test/test_postprocessing.py
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants