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

knife launch should create a chef node if not present #62

Closed
mrflip opened this issue Oct 25, 2011 · 6 comments
Closed

knife launch should create a chef node if not present #62

mrflip opened this issue Oct 25, 2011 · 6 comments
Milestone

Comments

@mrflip
Copy link
Member

mrflip commented Oct 25, 2011

Knife should create the chef node if it's missing.

This causes problems with sync and lots besides when it doesn't

flip

@temujin9
Copy link
Contributor

I think there were unresolved permissions issues which prevented this, the last time it was tried by Chris.

@mrflip
Copy link
Member Author

mrflip commented Oct 25, 2011

Ah, right.

The client that creates the node is the one that then owns it.

It would probably be best if knife force-created the client and the node
on launch -- we would also then be pushing in the client.pem and not the
activation.pem, which is much nicer. But also much more complicated, and
something that feels like it should be in knife core. I shall sail the IRCs
and see what @opscode thinks.

Now I remember why this wasn't simple.

On Tue, Oct 25, 2011 at 2:17 PM, Nathaniel Eliot <
reply@reply.github.com>wrote:

I think there were unresolved permissions issues which prevented this, the
last time it was tried by Chris.

Reply to this email directly or view it on GitHub:
https://github.com/infochimps/cluster_chef/issues/62#issuecomment-2521655

@temujin9
Copy link
Contributor

Well, if it were simple, we'd have done it already . . .

I agree that bootstrapping in the pre-made client definition seems more correct than relying on the client to make client and node itself.

@mrflip
Copy link
Member Author

mrflip commented Oct 25, 2011

this might work:

  • ensure the client exists. If it doesn't, make the client on my side, and
    just discard the key.
  • ensure the node exists, and ensure that the eponymous client has the right
    permissions.
  • when the machine spins up, I think it will just blow away the old client
    key and assume its identity, leaving the permissions alone.

If so, Bob's our uncle.

flip

On Tue, Oct 25, 2011 at 2:36 PM, Nathaniel Eliot <
reply@reply.github.com>wrote:

Well, if it were easy, we'd have done it already . . .

I agree that bootstrapping in the pre-made client definition seems more
correct than relying on the client to make client and node itself.

Reply to this email directly or view it on GitHub:
https://github.com/infochimps/cluster_chef/issues/62#issuecomment-2521893

infochimps.com - discover data

@mrflip
Copy link
Member Author

mrflip commented Oct 29, 2011

Fixed.

If the client doesn't exist, knife creates it and saves the client key.
If the node exists, life is good.
If the node doesn't exist, and we have the client key (saved or newly-created), knife creates the node using that client -- making it the owner, life is good.

If the node doesn't exist, and you don't have the client key, knife will error out. You have to delete the client, or create the node yourself and then adjust permissions.

@mrflip mrflip closed this as completed Oct 29, 2011
@temujin9
Copy link
Contributor

+1

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

No branches or pull requests

2 participants