Majic Projects
previous_open_issue.png
Go to the previous open issue
previous_issue.png
Go to the previous issue (open or closed)
star_faded.png
Please log in to bookmark issues
bug_report_small.png
Closed Bug report MAR-35 Incorrect use of Ansible facts and a couple of other variables
next_issue.png
Go to the next issue (open or closed)
next_open_issue.png
Go to the next open issue
icon_info.png This issue has been closed with status "Closed" and resolution "RESOLVED".
Issue basics
  • Type of issue
    Bug report
  • Category
    Not determined
  • Targetted for
    1.0.0
  • Status
    Closed
  • Progress
  • Priority
    Not determined
User pain
  • Type of bug
    Not triaged
  • Likelihood
    Not triaged
  • Effect
    Not triaged
Affected by this issue (0)
There are no items
People involved
Times and dates
  • Posted at
  • Last updated
  • Estimated time
    Not estimated
  • Time spent
    No time spent
    Click here to see time logged against this issue
Issue details
  • Resolution
    RESOLVED
  • Reproducability
    Not determined
  • Severity
    Not determined
Attachments (0)
There is nothing attached to this issue
Duplicate issues (0)
This issue does not have any duplicates
Description
In the current master branch there is a couple of places where the variables are not used correctly.

In particular:

# The Ansible-provided facts are used via completely wrong names. Plus, they are overcomplicated. In particular, they are mistakenly referenced via '''ansible_facts''' variable instead of '''hostvars''', and there is also no need to reference them like this. They can be reffed via ansible_NAME instead. This is consistently wrong use in both roles and docs.
# The '''common_packages''' variable is referenced as '''packages''' in the common role.
# Documentation for variables '''tls_private_key_dir''' and '''tls_certificate_dir''' seems to be just wrong. These variables seem to be mandatory.
Steps to reproduce this issue
See description.
Comments ()
History