Move remove_tags value to skip_tag_list

https://review.opendev.org/c/openstack/tripleo-ci/+/758373 added
support for removing tags from the job config.

It is basically removing the tags from the whole tag list.
Instead of removing the tag, we can put it in the skip list.

It will help us from running the plays twice irrespective of
tags defined at role level or play level.

It also solves the issues of running build-test-packages on
consumer jobs where build tags in removed, is coming in
consolidation patch of build-test-packages [1].

[1]. https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/790926

Signed-off-by: Chandan Kumar (raukadah) <chkumar@redhat.com>
Change-Id: I072c682ce811871a7c06fb18b31d76f2e9397643

roles/run-test/templates/toci_quickstart.sh.j2

@@ -18,7 +18,7 @@ export DEFAULT_ARGS="--extra-vars local_working_dir=$LOCAL_WORKING_DIR \
     --extra-vars tripleo_root={{ tripleo_root }} \
     --extra-vars working_dir=$WORKING_DIR \
     --extra-vars tripleo_generate_scripts=true \
-    --skip-tags "{{ skip_tags_list }}" \
+    --skip-tags "{{ ','.join(skip_tags_list.split(',') + remove_tags|default([])) }}" \
 "
 
 # --install-deps arguments installs deps and then quits, no other arguments are
@@ -42,10 +42,9 @@ QUICKSTART_VENV_CMD="
     {{ undercloud }}
 "
 
-#REMOVE build from --skip-tags ASAP
 QUICKSTART_INSTALL_CMD="
     $LOCAL_WORKING_DIR/bin/ansible-playbook
-    --tags {{ ','.join((tags + (extra_tags|default([]))) | difference(remove_tags|default([]))) }}
+    --tags {{ ','.join(tags + (extra_tags|default([]))) }}
 "
 
 export QUICKSTART_DEFAULT_RELEASE_ARG="--extra-vars @$LOCAL_WORKING_DIR/config/release/tripleo-ci/{{ ansible_distribution }}-{{ ansible_distribution_major_version }}/$QUICKSTART_RELEASE.yml"