updated readme

README.md

@@ -2,4 +2,7 @@
 URL: https://xpk.headdesk.me/git/xpk/role.vncserver
 
 ## Required variables
-- vnc.user
+Please set the ```vncuser``` in your ansible inventory or playbook. The vncserver service will be started by this user.
+
+## Tested on
+- RHEL7

tasks/main.yml

@@ -32,6 +32,7 @@
 - name: Create xstartup file
   copy:
           dest: "~{{vncuser}}/.vnc/xstartup"
+          mode: 0755
           content: |
                   #!/bin/bash
                   # Uncomment the following two lines for normal desktop:
@@ -82,6 +83,12 @@
     state: started
     enabled: yes
 
+- name: Disable firewalld
+  service:
+    name: firewalld
+    state: stopped
+    enabled: no
+
 - name: Display vncpass
   debug:
     var: vncpass