Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

YumrepoをResourceTypeに追加する #162

Merged
merged 8 commits into from Jun 23, 2013
Merged

YumrepoをResourceTypeに追加する #162

merged 8 commits into from Jun 23, 2013

Conversation

@kitak
Copy link
Contributor

@kitak kitak commented Jun 23, 2013

YumのパッケージリポジトリをResourceTypeに追加しました。

記述は以下のようになります。

describe yumrepo('epel') do
  it { should exist }
  it { should be_enabled }
end
  • existマッチャでレポジトリが追加されているかどうか確認できます。
  • be_enabledマッチャでレポジトリが有効になっているかどうか確認できます。

Resourceのマッチャとcommandsのテストを記述しました。
bundle exec rspec spec/redhatを実行するとexamplesが全て通ることを確認しています。

@mizzy
Copy link
Owner

@mizzy mizzy commented Jun 23, 2013

お、ナイス。ただ、repositoryだと幅が広すぎるのと、とりあえずRedHat系だけ、ってことなら、yumrepo というリソースタイプ名の方が良さそう。yumrepoPuppet 標準の type にもあるし、Chef の Cookbook も存在するので。

@kitak
Copy link
Contributor Author

@kitak kitak commented Jun 23, 2013

debian系とかも意識して、repositoryにしたのですが、たしかに幅が広いですね...
yumrepoに修正します。

@@ -10,6 +10,14 @@ def check_enabled service
"chkconfig --list #{escape(service)} | grep 3:on"
end

def check_repository(repository)
"yum repolist | grep ^#{escape(repository)}"
Copy link
Owner

@mizzy mizzy Jun 23, 2013

yum repolist -C is better.Because without -C option, yum command tries to download metadata and it may take long time.

@mizzy mizzy merged commit 760fccd into mizzy:master Jun 23, 2013
@mizzy
Copy link
Owner

@mizzy mizzy commented Jun 23, 2013

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants