[ 
https://issues.apache.org/jira/browse/BIGTOP-1508?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14269539#comment-14269539
 ] 

Evans Ye commented on BIGTOP-1508:
----------------------------------

The patch fixed the leftover warnings in BIGTOP-1047 instead of the below one:
{code}
==> bigtop1: Warning: The use of 'import' is deprecated at 
/tmp/vagrant-puppet-2/manifests/site.pp:46. See 
http://links.puppetlabs.com/puppet-import-deprecation
==> bigtop1:    (at 
/usr/lib/ruby/site_ruby/1.8/puppet/parser/parser_support.rb:110:in `import')
{code}

The 'import warning" can also be fixed by doing following steps:
* remove import "cluster.pp" in line 44, {{site.pp}}
* specify a directory as your main manifest when doing puppet apply. i.e.
{code}
puppet apply -d --modulepath="bigtop-deploy/puppet/modules:/etc/puppet/modules" 
bigtop-deploy/puppet/manifests/
{code}

Since the the way to specify directory as manifest is currently not supported 
by Vagrant yet, changing the behavior will break vagrant-puppet provisioner.
I think the gradew bundler for puppet apply that [~cos] mentioned in mailing 
list can come to a rescue.
Once the feature implemented, I'll update the vagrant-puppet and docker-puppet 
to use the bundler subsequently. 

> fix Puppet warnings under Puppet 3 
> -----------------------------------
>
>                 Key: BIGTOP-1508
>                 URL: https://issues.apache.org/jira/browse/BIGTOP-1508
>             Project: Bigtop
>          Issue Type: Bug
>          Components: deployment
>    Affects Versions: 0.8.0
>            Reporter: Konstantin Boudnik
>             Fix For: 0.9.0
>
>         Attachments: BIGTOP-1508.1.patch, BIGTOP-1508.2.patch
>
>
> Once BIGTOP-1047 is committed, let's fix the [leftover 
> warnings|https://issues.apache.org/jira/browse/BIGTOP-1047?focusedCommentId=14190686&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14190686]



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to