postgresql icon indicating copy to clipboard operation
postgresql copied to clipboard

error in README pass: pass

Open Pepan opened this issue 7 years ago • 5 comments

# List of users to be created (optional)
postgresql_users:
  - name: baz
    pass: pass
    encrypted: no       # denotes if the password is already encrypted.

should be

# List of users to be created (optional)
postgresql_users:
  - name: baz
    password: pass
    encrypted: no       # denotes if the password is already encrypted.

Pepan avatar Oct 25 '18 12:10 Pepan

Pull requests for easy doc fixes are always welcome!

otakup0pe avatar Oct 25 '18 18:10 otakup0pe

In tasks/users.yml, it used as {{item.pass}. So, it should be pass. @Pepan

- name: PostgreSQL | Make sure the PostgreSQL users are present                
  postgresql_user:                                                             
    name: "{{item.name}}"                                                      
    password: "{{ item.pass | default(omit) }}"                                
    encrypted: "{{ item.encrypted | default(omit) }}"                          
    port: "{{postgresql_port}}"                                                
    state: present                                                             
    login_user: "{{postgresql_admin_user}}"                                    
  no_log: true                                                                 
  become: yes                                                                  
  become_user: "{{postgresql_admin_user}}"                                     
  with_items: "{{postgresql_users}}"                                           
  when: postgresql_users|length > 0

erdenezul avatar Oct 29 '18 02:10 erdenezul

@otakup0pe If you would like to change from pass to password for better readable, I want to submit PR

erdenezul avatar Oct 29 '18 02:10 erdenezul

Otherwise, I believe we can close this issue

erdenezul avatar Oct 29 '18 02:10 erdenezul

PR's always welcome!

otakup0pe avatar Nov 01 '18 05:11 otakup0pe