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.

Sign up for GitHub

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

Jump to bottom

Better upgrade and distro package installation. #180

Merged
merged 4 commits into from
Dec 16, 2014
Merged

Better upgrade and distro package installation. #180

merged 4 commits into from
Dec 16, 2014

Conversation

jjasghar
Copy link
Contributor

@jjasghar jjasghar commented Dec 9, 2014

The rabbitmq cookbook should support both install and upgrade of packages.

Fixes issue#143

wenchma and others added 2 commits December 8, 2014 11:05
Add the packages upgrade support
d53a578 
The rabbitmq cookbook should support both install and upgrade of packages.

Fixes issue#143
This pins the version of the package for upgrade
5fd487c 
So you make sure you don't upgrade anything other than what you.
@jjasghar jjasghar changed the title Better upgrad Better upgrade and distro package installation. Dec 9, 2014
@jjasghar jjasghar mentioned this pull request Dec 9, 2014
Closed
jjasghar
jjasghar reviewed Dec 9, 2014
View reviewed changes
spec/default_spec.rb
@@ -42,10 +42,6 @@
expect(chef_run).to create_remote_file_if_missing("#{Chef::Config[:file_cache_path]}/rabbitmq-server-#{version}.noarch.rpm")
end

it 'installs the rabbitmq-server rpm_package with the default action' do
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This totally didn't work how i expected. I'll need to investigate this.

@jjasghar
Copy link
Contributor Author

jjasghar commented Dec 9, 2014

I havent hooked up travis to do it yet, but it looks like kitchen passes too:

-----> Kitchen is finished. (16m15.09s)
14:26:57 rabbitmq > (better_upgrad) bundle exec kitchen list
Instance                  Driver        Provisioner  Last Action
default-centos-6-5-x64    Digitalocean  ChefZero     <Not Created>
default-debian-7-0-x64    Digitalocean  ChefZero     <Not Created>
default-ubuntu-12-04-x64  Digitalocean  ChefZero     <Not Created>
default-ubuntu-14-04-x64  Digitalocean  ChefZero     <Not Created>
cluster-centos-6-5-x64    Digitalocean  ChefZero     <Not Created>
cluster-debian-7-0-x64    Digitalocean  ChefZero     <Not Created>
cluster-ubuntu-12-04-x64  Digitalocean  ChefZero     <Not Created>
cluster-ubuntu-14-04-x64  Digitalocean  ChefZero     <Not Created>
lwrps-centos-6-5-x64      Digitalocean  ChefZero     <Not Created>
lwrps-debian-7-0-x64      Digitalocean  ChefZero     <Not Created>
lwrps-ubuntu-12-04-x64    Digitalocean  ChefZero     <Not Created>
lwrps-ubuntu-14-04-x64    Digitalocean  ChefZero     <Not Created>
14:28:27 rabbitmq > (better_upgrad)

This was referenced Dec 9, 2014
Closed
Closed
@jjasghar jjasghar merged commit 9fd31ec into master Dec 16, 2014
@jjasghar jjasghar deleted the better_upgrad branch December 16, 2014 21:49
@jjasghar jjasghar mentioned this pull request Dec 16, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
needs tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jjasghar @cheeseplus