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

Adding an option to get only envs and nodes. #10

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

sriramgd
Copy link

@sriramgd sriramgd commented Feb 9, 2012

Adding a env_node_only (i.e. -e) option. Uses the Chef RESTful API to first get envs, and their nodes. This is faster than getting all node objects and querying them. Allows hundreds of Chef nodes to be available in Rundeck quickly.

Cost:

  • Some information is lost (e.g. machine type etc.).
  • Machine name is set to unix for any machine.

Usage example:
sudo ./chef-rundeck -c /home/chef/.chef/knife.rb -u username -w
http://servername:4040 -e

PS: I am new to git/ruby

adamhjk and others added 8 commits July 26, 2011 12:45
Please include osFamily identification fix for chef-rundeck in master
Fix for "text/html" incompatibility error
using the Chef RESTful API. This is faster than getting all node objects and
querying them. Allows hundreds of Chef nodes to be available in Rundeck
quickly.

Usage example:
sudo ./chef-rundeck -c /home/chef/.chef/knife.rb -u username -w
http://servername:4040 -e
@josephholsten
Copy link
Contributor

I think this would be best served by using something like partial_search instead

@bscott
Copy link

bscott commented Aug 19, 2013

@sriramgd @josephholsten @leftathome Do we still need this PR?

@sriramgd
Copy link
Author

Hi, I dont work with Chef /Rundeck in my current job and am not up to date
on the latest changes. However have received mails from a couple of people
asking how to access the PR (since it was closed). The Rundeck bridge was
very slow for more than a couple of hundred nodes. We had this problem for
the couple of hundred Windows nodes which were coordinated from Rundeck,
and so I had to add this switch*.
Cc'ing Etienne, who sent a mail few weeks back, in case he has any input on
whether the PR was useful.
Thanks,
Sriram

On Mon, Aug 19, 2013 at 1:16 PM, Brian Scott [email protected]:

@sriramgd https://github.com/sriramgd @josephholstenhttps://github.com/josephholsten
@leftathome https://github.com/leftathome Do we still need this PR?


Reply to this email directly or view it on GitHubhttps://github.com//pull/10#issuecomment-22888207
.

@bscott
Copy link

bscott commented Aug 29, 2013

This PR will have to wait till 0.2.2 release.

@bscott
Copy link

bscott commented Feb 27, 2014

@sriramgd Can you rebase?

@leftathome
Copy link
Contributor

Anecdotal evidence suggests that supporting partial search has solved this issue for others, and doesn't introduce a significantly different collection / rendering code path in the process. My preference is to stick with that approach. Objections?

@bscott
Copy link

bscott commented Oct 15, 2014

@sriramgd Can you rebase this?

@bscott
Copy link

bscott commented Nov 27, 2015

@sriramgd ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants