From 239ab69b342ed63eebcf7c197e46f3c3a730f81e Mon Sep 17 00:00:00 2001
From: xpk <xpk@headdesk.me>
Date: Tue, 22 Jan 2019 17:15:28 +0800
Subject: [PATCH] updated readme

---
 README.md      | 5 ++++-
 tasks/main.yml | 7 +++++++
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 89e5bb2..e398d57 100644
--- a/README.md
+++ b/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
diff --git a/tasks/main.yml b/tasks/main.yml
index 162ca9c..b25b7da 100644
--- a/tasks/main.yml
+++ b/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