vncproxy: removal of tags and steps change from post_upgrade_tasks

Post upgrade tasks are limited to 4-1 steps, so these
tasks are not executed.

Deleting nova-consoleauth is failing with some of the CIs and this is
mostly a best-effort kind of task, so adding ignore_errors.

Also, replacing some yes/nos with boolean as it's best practice.

Related: https://bugzilla.redhat.com/1921112
Change-Id: I0bc39170569c42a0be31233fd7ec068613547790

deployment/nova/nova-vnc-proxy-container-puppet.yaml

@@ -381,26 +381,24 @@ outputs:
                 comment: qemu user
       post_upgrade_tasks:
         - when:
-            - step|int == 4
-          tags:
-            - never
-            - system_upgrade_stop_services
+            - step|int == 3
           environment:
             OS_CLOUD: {get_param: RootStackName}
           block:
             - name: Get nova-consoleauth service ID
               command: openstack compute service list --service nova-consoleauth --column ID  --column Host --format yaml
               register: nova_compute_service_result
-              delegate_to: localhost
-              check_mode: no
-              run_once: yes
+              check_mode: false
+              run_once: true
               changed_when: false
+              failed_when: false
+              delegate_to: undercloud
             - name: Deleting nova-consoleauth
               command: "openstack compute service delete {{ item.ID }}"
-              loop: "{{ nova_compute_service_result.stdout | from_yaml }}"
-              run_once: yes
-              delegate_to: localhost
-              ignore_errors: yes
+              loop: "{{ nova_compute_service_result.stdout | from_yaml | default([]) }}"
+              run_once: true
+              delegate_to: undercloud
+              failed_when: false
       external_upgrade_tasks:
         - when:
             - step|int == 1