No resource or method named `not_if' for `Chef::Recipe "setup_package_manager"' about ambari HOT 4 CLOSED

I think the issue is with the code itself.

I confirmed the ambari repo was not installed:

[root@ip-10-0-1-78 yum.repos.d]# ls
CentOS-Base.repo  CentOS-CR.repo  CentOS-Debuginfo.repo  CentOS-Media.repo  CentOS-Vault.repo  elrepo.repo  epel.repo  epel-testing.repo

I think you need to bring the guards within the remote_file resource block, like so:

case node['platform']
when "redhat","centos","amazon","scientific"
  case node['platform_version'].to_i
  when 5
    yum_repo = node['ambari']['rhel_5_repo']
  when 6
    yum_repo = node['ambari']['rhel_6_repo']
  end
  remote_file "/etc/yum.repos.d/ambari.repo" do
    source yum_repo
    not_if { File.exists?("/etc/yum.repos.d/ambari.repo") }
  end
when "suse"
  remote_file "/etc/zypp/repos.d/ambari.repo" do
    source node['ambari']['suse_11_repo']
    not_if { File.exists?("/etc/zypp/repos.d/ambari.repo") }
  end

That worked for me. Let me know if you want me to write that up and submit a pull request.

from ambari.

Hey Joe, thanks for your feedback. I would be happy if you can submit a PR
for this. There is not tests yet for this cookbook, so just write up your
changes ;)
On Apr 12, 2015 3:29 PM, "JoeReid-Korrelate" [email protected]
wrote:

I think the issue is with the code itself.

I confirmed the ambari repo was not installed:

[root@ip-10-0-1-78 yum.repos.d]# ls
CentOS-Base.repo CentOS-CR.repo CentOS-Debuginfo.repo CentOS-Media.repo CentOS-Vault.repo elrepo.repo epel.repo epel-testing.repo

I think you need to bring the guards within the remote_file resource
block, like so:

case node['platform']
when "redhat","centos","amazon","scientific"
case node['platform_version'].to_i
when 5
yum_repo = node['ambari']['rhel_5_repo']
when 6
yum_repo = node['ambari']['rhel_6_repo']
end
remote_file "/etc/yum.repos.d/ambari.repo" do
source yum_repo
not_if { File.exists?("/etc/yum.repos.d/ambari.repo") }
end
when "suse"
remote_file "/etc/zypp/repos.d/ambari.repo" do
source node['ambari']['suse_11_repo']
not_if { File.exists?("/etc/zypp/repos.d/ambari.repo") }
end

That worked for me. Let me know if you want me to write that up and submit
a pull request.

—
Reply to this email directly or view it on GitHub
#2 (comment).

from ambari.

Hey JP,

Thanks! Will do.

Regards,
Joe

from ambari.

Thanks for the PR 👍

from ambari.