diff --git a/openstack/metalsmith/metalsmith.spec.j2 b/openstack/metalsmith/metalsmith.spec.j2
index 7aae2754a..cb54d8532 100644
--- a/openstack/metalsmith/metalsmith.spec.j2
+++ b/openstack/metalsmith/metalsmith.spec.j2
@@ -1,5 +1,5 @@
 {% set pypi_name = 'metalsmith' %}
-{% set upstream_version = upstream_version('1.1.0') %}
+{% set upstream_version = upstream_version('1.2.0') %}
 {% set rpm_release = '1' %}
 {% set source = url_pypi() %}
 
@@ -11,9 +11,9 @@ License:        {{ license('Apache-2.0') }}
 Group:          Development/Languages/Python
 URL:            https://launchpad.net/{{ pypi_name }}
 Source0:        {{ source }}
+BuildRequires:  {{ py3('ansible') }}
 BuildRequires:  openstack-macros
 BuildRequires:  {{ py3('PrettyTable') }}
-BuildRequires:  {{ py3('mock') }}
 BuildRequires:  {{ py3('openstacksdk') }}
 BuildRequires:  {{ py3('pbr') }}
 BuildRequires:  {{ py3('setuptools') }}
@@ -27,6 +27,7 @@ OpenStack Ironic.
 
 %package -n python3-metalsmith
 Summary:        Bare metal provisioner using Ironic
+Requires:       {{ py3('ansible') }}
 Requires:       {{ py3('PrettyTable') }}
 Requires:       {{ py3('openstacksdk') }}
 Requires:       {{ py3('pbr') }}
@@ -49,7 +50,6 @@ Documentation for Metalsmith
 
 %package -n ansible-role-{{ py2name() }}-deployment
 Summary:        Ansible role for Metalsmith deployment
-Requires:       ansible
 Requires:       {{ py3('metalsmith') }}
 
 %description -n ansible-role-{{ py2name() }}-deployment
@@ -85,7 +85,7 @@ python3 -m stestr.cli run
 
 %files -n ansible-role-{{ py2name() }}-deployment
 %license LICENSE
-%doc roles/metalsmith_deployment/README.rst
+%doc metalsmith_ansible/roles/metalsmith_deployment/README.rst
 %{_datadir}/ansible
 %{_datadir}/ansible/roles
 %{_datadir}/ansible/roles/metalsmith_deployment