40 branch

[tiawaz]

When installing from the 40 branch on Centos7

  roles:
    - role: pdns-ansible
  vars:
    pdns_config:
      local-address: '{{ ansible_eth0.ipv4.address }}'
      allow-recursion: 'xxx.xxx.xxx.xxx'
      recursor: '{{ ansible_eth0.ipv4.address }}:5300'
    pdns_backends_mysql_credential:
      gmysql:
        priv_user: root
        priv_password: ***
        priv_host:
          - "%"
    pdns_backends:
      gmysql:
        host: 'localhost'
        port: 3306
        user: xxx
        password: ***
        dbname: pdns
    pdns_installation_type: 'packages'
    pdns_repo_provider: 'powerdns'
    pdns_repo_branch: '40'

i ran into the following problems:

1. The packages use etc/pdns as config dir, but the role defaults to /etc/powerdns
   2)The packages put the schema into /usr/share/doc/pdns-backend-mysql-4.0.1/schema.mysql.sql, but the role defaults to /usr/share/doc/pdns-backend-mysql/schema.mysql.sql

[tiawaz]

both issues are easily worked around by fudging the vars in the role, simply adding

- name: Load variables
  include_vars: '{{item}}'
  with_first_found:
    - files:
      - '{{ansible_os_family}}.yml'
      - 'default.yml'
      paths:
        - ../vars

to the main task and the adding my own vars/RedHat.yml

[tiawaz]

From a cursory look at the code it seems to me that a proper fix will be more involved, since there seems to be little support for the branch idea throughout the code?

Is it intentional that the 40 branch uses different Dirs than the regular packages?

[pieterlexis]

hi @tiawaz,

It indeed looks like the role only takes into account the Debian/Ubuntu default directories. I quickly checked the regular CentOS packages and they also use /etc/pdns/ for their configs. It would appear this role was wrong for CentOS-based distros. We'll fix this!

The schema-location will be somewhat harder, as it has to be calculated based on the version. But this is doable.

[tiawaz]

If the directory /etc/pdns is correct for all RedHat packages i can put up my changes (using the vars file as described above) as a PR

[tiawaz]

#21

[pieterlexis]

Thank you!

[tiawaz]

Just to clarify, since my comments were not entirely clear, the PR only fixes the config dir issue.#

I have an Idea about how to properly deal with the schema issue, but will not be able to have a look at it before next week.