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

Initialize all ansible variables prior to collecting facts #475

Closed
mdvickst opened this issue Nov 14, 2017 · 0 comments
Closed

Initialize all ansible variables prior to collecting facts #475

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

Comments

@mdvickst
Copy link

Opening a new issue


Specify type:

  • Bug

Bug severity (if applicable):

  • Medium

Description:

Some systems may lose connectivity or other unexpected errors may occur during a scan so if there are ansible variables that are expected to be initialized by the report generation task we need to initialize those first before any facts are collected that way if something happens the report generation will still be successful even if all facts are not collected.


\

@chambridge chambridge added this to the Sprint 11 milestone Nov 15, 2017
@chambridge chambridge self-assigned this Nov 15, 2017
chambridge added a commit that referenced this issue Nov 15, 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