Majic Projects
Majic Ansible Roles
Go to the previous open issue
Go to the previous issue (open or closed)
star_faded.png
Please log in to bookmark issues
icon_project.png Majic Ansible Roles / Closed Bug report MAR-35 Incorrect use of Ansible facts and a couple of other variables
Go to the next issue (open or closed)
Go to the next open issue
There are no more issues in that direction.
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
Show / hide more details »
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.
Todos (0 / 0)
Options
Issue created