Bryan McLellan btm

Organizations

@chef @opscode-cookbooks
btm commented on pull request chef/ohai#585
@btm

I don't see a test for this case. Is this legit? Can we get a test and a comment with what platform this might catch?

btm commented on pull request chef/ohai#585
@btm

Why the || ""?

btm commented on pull request chef/ohai#585
@btm

Is this a breaking change? It looks like we used to get wrlinux before, now we're explicitly setting Cisco-wrlinux. Is this platform family so uniq…

btm commented on pull request chef/ohai#585
@btm

So this is where I was having a hard time. While I get now how we would end up here, we shouldn't have both this else and the next one. If we are …

btm commented on pull request chef/ohai#585
@btm

I think we could simplify this set of nested conditionals. First, os_release_file_is_cisco? Is doing a confusing double duty as both checking for a…

btm commented on pull request chef/ohai#585
@btm

If I'm reading this right, this would be a lot easier to understand if cisco_release_info was something like cisco_release_file.

@btm
  • @btm 4b0fcc1
    "Fix missing change of validate!/validate_options! :(
@btm
@btm
negotiate warning for non-windows platforms is late
btm merged pull request chef/knife-windows#260
@btm
Negotiate warning for non-windows platforms is late
3 commits with 16 additions and 7 deletions
btm commented on pull request chef/omnibus-chef#424
@btm

:+1: weird bug.

@btm
negotiate warning for non-windows platforms is late
btm commented on pull request chef/chef#3591
@btm

@sethvargo I believe @thommay has been working on having curry auto-assign PRs needing review. I'm not sure if it's been talked about outside of th…

btm commented on pull request chef/chef#3620
@btm

@danielsdeleo what's the significant change to lib/chef/run_list/versioned_recipe_list.rb?

btm commented on issue chef/knife-windows#257
@btm

@fabioxgn What does which gem return, i.e. is the rubygems installation that you're installing the knife-windows gem the same one that comes in the…

btm commented on issue chef/knife-windows#256
@btm

@Infydevops Is the system running jenkins using ChefDK? If that's your only ruby environment (that's the source of knife) then you can install the …

@btm
Windows Bootstrap Fails
btm commented on issue chef/knife-windows#242
@btm

@Infydevops, I'm going to close this issue since we haven't heard back from you. Let us know if you're still having issues, or what your resolution…

@btm
How to install?
btm commented on issue chef/knife-windows#243
@btm

@spuder I haven't heard back from you so I'm going to close this issue. I'd make sure you're running at least ChefDK 0.6.2. We're going to release …

@btm
Catch errors when knife-windows 1.0.0 is used with chefdk version v0.6.*
btm commented on issue chef/knife-windows#253
@btm

Yeah, unfortunately the errors are in rubygems loading, and I don't think we can catch that in knife-windows because knife-windows isn't actually l…

btm closed pull request chef/knife-windows#254
@btm
Updated release notes for chef-dk 0.6.* version compatibility issue
btm commented on pull request chef/knife-windows#254
@btm

So after much debugging, this turns out to be a dependency issue. See chef/chef-dk#411 for more information. I've added a note to the release notes…

@btm
btm closed issue chef/chef-dk#411
@btm
rubygems undefined method `activate' for nil:NilClass
btm commented on issue chef/chef-dk#411
@btm

Okay, this is all dependency issues, which one thing ChefDK 0.7.0 is meant to bump. To get knife-windows 1.0.0+ working on ChefDK 0.6.2: 1) chef ge…

btm commented on issue chef/chef-dk#411
@btm

irb(main):006:0> Gem::Specification.find_by_name("knife-windows") => nil irb(main):007:0> Gem::Specification.find_all_by_name("knife-windows") => […

btm commented on issue chef/chef-dk#411
@btm

For google, this is the error you get with ChefDK 0.6.2, after you install knife-windows 1.0.0.rc.1 but you have not removed knife-windows 0.8.5: P…

btm opened issue chef/chef-dk#443
@btm
0.7.0, gems in user install not showing up: