knife data bag show groups fails ... with something like undefined method `keys' for ["http://localhost:4000/data/groups/sysadmin"]:Array

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 0.8.2
  • Fix Version/s: 0.8.2
  • Component/s: None
  • Labels:
    None
  • Environment:

    clone of github/opscode/chef.git master as of 1/14/4:37pm on ubuntu karmic koala

Description

See http://gist.github.com/277649

Steps to Reproduce,

1) knife data bag create bob
2) knife create data bag bob jones
... put in whatever
3) knife data bag show bob

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Daniel DeLeo added a comment -

http://github.com/danielsdeleo/chef/tree/CHEF-879

  • Makes chef-server-api return a Hash of the form {data_bag_item_id => URL-endpoint} when showing data/data_bag--this fixes the reported bug
  • Fixes a regression where knife was trying to create data bag items via HTTP PUT--changed to POST
  • basic rspec coverage for data_bag_create and data_bag_show
  • gitignore .rake_tasks~ files created by zsh rake completion
Show
Daniel DeLeo added a comment - http://github.com/danielsdeleo/chef/tree/CHEF-879 Makes chef-server-api return a Hash of the form {data_bag_item_id => URL-endpoint} when showing data/data_bag--this fixes the reported bug Fixes a regression where knife was trying to create data bag items via HTTP PUT--changed to POST basic rspec coverage for data_bag_create and data_bag_show gitignore .rake_tasks~ files created by zsh rake completion
Hide
Permalink
Daniel DeLeo added a comment -

Need to update some cukes before this is merged, mai bahd

Show
Daniel DeLeo added a comment - Need to update some cukes before this is merged, mai bahd
Hide
Permalink
Daniel DeLeo added a comment -

Updated the cukez to expect hash output from the API for the data bags, they now pass.

Show
Daniel DeLeo added a comment - Updated the cukez to expect hash output from the API for the data bags, they now pass.
Hide
Permalink
Adam Jacob added a comment -

This is merged - thanks for the bug, Scott, and the fix, Dan. Available in opscode/master.

Show
Adam Jacob added a comment - This is merged - thanks for the bug, Scott, and the fix, Dan. Available in opscode/master.

People

  • Assignee:
    Adam Jacob
    Reporter:
    Scott M. Likens
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: