search

sous-chefs / consul

Development repository for the consul cookbook
https://supermarket.chef.io/cookbooks/consul
Apache License 2.0
192 stars 244 forks source link

Move webui installation to poise-archive #378

Closed tanner-bruce closed 7 years ago

tanner-bruce commented 7 years ago

Add spec for webui suite

Fixes #377

legal90 commented 7 years ago

@tanner-bruce Thanks! LGTM. 👍 Then we will be able to fix this spec too: https://github.com/johnbellone/consul-cookbook/blob/master/test/spec/libraries/consul_installation_webui_spec.rb

tanner-bruce commented 7 years ago

@legal90 I added in the missing checksum for 0.7.1 and fixed up the specs. Not sure why the build is failing randomly, it passes locally.

legal90 commented 7 years ago

It appears the last build job has been failed due to the channel we use to install ChefDK in TravisCI (chef-current-trusty). Currently, it installs ChefDK v1.1.3 which looks unstable.

legal90 commented 7 years ago

Quote from the Chef slack chat, about this TravisCI build failure:

I just spun up Chef-DK 1.1.3 and ran rspec in it. Everything was fine. I then checkout out your repo, did a bundle install, and then ran rspec. It blew up. So there's a conflict between something in your Gemfile and newer versions of Chef-DK

It's the poise-boiler gem. it might need to update versions of its deps

cc @johnbellone

legal90 commented 7 years ago

JFYI: #382 is gonna fix the TravisCI build failure mentioned above

lock[bot] commented 4 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.