RDO tickets are now tracked in Jira https://issues.redhat.com/projects/RDO/issues/
Bug 1400557 - Duplicate declaration: nfs-utils already declared by cinder
Summary: Duplicate declaration: nfs-utils already declared by cinder
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: RDO
Classification: Community
Component: openstack-manila
Version: trunk
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: trunk
Assignee: Tom Barron
QA Contact: Dustin Schoenbrun
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-12-01 13:37 UTC by Christopher Brown
Modified: 2016-12-09 12:19 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-12-09 11:26:40 UTC
Embargoed:


Attachments (Terms of Use)
cinder-netapp-config.yaml (1.66 KB, text/plain)
2016-12-01 16:01 UTC, Christopher Brown
no flags Details
manila-netapp-config.yaml (1.51 KB, text/plain)
2016-12-01 16:02 UTC, Christopher Brown
no flags Details
manila-netapp-config.yaml (1.49 KB, text/plain)
2016-12-01 16:12 UTC, Christopher Brown
no flags Details


Links
System ID Private Priority Status Summary Last Updated
OpenStack gerrit 408658 0 None MERGED Fix duplicate declaration between manila/tripleO 2020-06-04 05:57:29 UTC

Description Christopher Brown 2016-12-01 13:37:12 UTC
Description of problem:

Error: Duplicate declaration: Package[nfs-utils] is already declared in file /etc/puppet/modules/tripleo/manifests/profile/base/cinder/volume/nfs.pp:53; cannot redeclare at /etc/puppet/modules/manila/manifests/backend/netapp.pp:144 on node overcloud-ctrl-0.cluster.local

Comment 1 Christopher Brown 2016-12-01 16:01:43 UTC
Created attachment 1226862 [details]
cinder-netapp-config.yaml

Comment 2 Christopher Brown 2016-12-01 16:02:29 UTC
Created attachment 1226864 [details]
manila-netapp-config.yaml

Comment 3 Christopher Brown 2016-12-01 16:03:32 UTC
openstack overcloud deploy --templates \
 -e /usr/share/openstack-tripleo-heat-templates/environments/puppet-pacemaker.yaml \
 -e /usr/share/openstack-tripleo-heat-templates/environments/tls-endpoints-public-dns.yaml \
 -e ~/tripleo/custom/manila-netapp-config.yaml \
 -e ~/tripleo/custom/network-environment.yaml \
 -e ~/tripleo/custom/enable-tls.yaml \
 -e ~/tripleo/custom/scheduler_hints_env.yaml \
 -e ~/tripleo/custom/cinder-netapp-config.yaml \
 -e ~/tripleo/custom/all_params.yaml \
 --control-scale 3 --compute-scale 2 --ntp-server 0.uk.pool.ntp.org \
 --neutron-network-type vxlan --neutron-tunnel-types vxlan

Comment 4 Christopher Brown 2016-12-01 16:12:31 UTC
Created attachment 1226867 [details]
manila-netapp-config.yaml

Comment 5 Craig 2016-12-07 21:13:07 UTC
2 recommendations:

1.  test on your environment after removing the nfs-utils section from the puppet-manila/netapp.pp file.  If this works and correctly configures manila and your cinder nfs on the same server, then we can look at pulling that code out.  (I am not convinced it is 100% needed there, but I would have to do a lot more research)

2.  explicitly set package_ensure for that module to false/undef in hiera

something like this 

manila::profile::backend::netapp::package_ensure: false

I don't run triple-o, but we are using that particular puppet-manila module w/o any issues.  Keep in mind here we are not running cinder nfs on our control nodes


I know Tom is going to try to test this as well.  If it turns out that we don't need the nfs-utils for that netapp piece, I can pull that and put it up for review

Comment 6 Jan Provaznik 2016-12-08 08:44:41 UTC
The problem is in usage of "package" directive (where puppet doesn't like that Package class for nfs-utils is declared multiple times). 
Using "ensure_packages" (instead of "package") in both puppet modules should solve the problem.

Comment 7 Christopher Brown 2016-12-09 10:15:19 UTC
This isn't a problem for me any more however it isn't clear what resolved it. Potentially I was calling controllerExtraConfigPre twice. Sorry, slightly weird issue.

Comment 8 Christopher Brown 2016-12-09 11:26:40 UTC
Thanks Tom, will close this then.

Comment 9 Craig 2016-12-09 12:19:41 UTC
I posted a change to puppet-manila to use ensure_resource, I believe it merged last night.  

https://review.openstack.org/#/c/408658/

Glad your problem resolved itself.


Note You need to log in before you can comment on or make changes to this bug.