search

bflad / chef-stash

Chef Cookbook for Atlassian Stash
Other
37 stars 42 forks source link

.kitchen.yml: Rename Vagrant boxes "chef" -> "bento" #134

Closed legal90 closed 8 years ago

legal90 commented 8 years ago

All Vagrant boxes from "chef" account on Atlas have been moved to the "bento" namespace: https://atlas.hashicorp.com/chef

P.s. By the way, is it still reasonable to freeze Chef Client version in .kitchen.yml?

require_chef_omnibus: 11.16.2

https://github.com/bflad/chef-stash/blob/8c8dda6b5b25873bd87a8e5c2de75168a1b05adc/.kitchen.yml#L4

maj-translucent commented 8 years ago

LGTM

linc01n commented 8 years ago

@legal90 after removing the version pinning on omnibus I get this warning when running kitchen test

  WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING

   You are installing an omnibus package without a version pin.  If you are installing
   on production servers via an automated process this is DANGEROUS and you will
   be upgraded without warning on new releases, even to new major releases.
   Letting the version float is only appropriate in desktop, test, development or
   CI/CD environments.

   WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNIN
legal90 commented 8 years ago

@linc01n Ah, OK, I've seen this message. So, the last sentence means that this is OK for test environment. Also, it's useful to test the cookbook on the latest Chef Client version, isn't it?

linc01n commented 8 years ago

@legal90 You are right. I want to test the cookbook against the latest chef client. Let me remove the version pinning.