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.

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

updating readme to chefdk #247

Merged
merged 2 commits into from
Nov 3, 2015
Merged

updating readme to chefdk #247

merged 2 commits into from
Nov 3, 2015

Conversation

vinyar
Copy link

@vinyar vinyar commented Oct 29, 2015

ok to delete branch when done

@chef-supermarket
Copy link

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.

Non-GitHub Verified Committers

There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.

Please sign the CLA here.

via:

gem install chef
https://downloads.chef.io/chef-dk/

This plugin is distributed as a Ruby Gem. To install it, run:

Choose a reason for hiding this comment

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

I think If you are using chef-dk in that case ruby gems are install using -
chef gem install <name>. And also chef-dk comes with inbuilt knife azure, we have to verify how chef gem install knife-azure will work here, As recently we found some gem dependency related issues with chef-dk.

Thank you for sharing this.

Choose a reason for hiding this comment

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

@vinyar Please sign CLA.

Choose a reason for hiding this comment

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

👍 to the comment about updating the installation command to chef gem install knife-azure. I haven't found any dependency conflicts with installing knife-azure into the chef-dk, what conflicts are you referring to @siddheshwar-more?

Copy link
Author

Choose a reason for hiding this comment

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

@siddheshwar-more done. Thanks

@siddheshwar-more
Copy link

@andrewelizondo
The current chef-dk comes with knife-windows v.0.8 and chef gem install knife-azure installs the knife-windows v1.1.1 .
Here's issue chef-boneyard/chef-dk#411 related to gem dependency. I think we should verify this kind of dependency issues!

Thanks!

@siddheshwar-more
Copy link

I think above issue doesn't exists anymore ! Just need to replace https://github.com/chef/knife-azure/pull/247/files#diff-04c6e90faac2675aa89e2176d2eec7d8R17 by chef gem install knife-azure

siddheshwar-more pushed a commit that referenced this pull request Nov 3, 2015
updating readme to chefdk
@siddheshwar-more siddheshwar-more merged commit b0b82c1 into chef:master Nov 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants