Skip to content
This repository was archived by the owner on Jan 16, 2024. It is now read-only.

Conversation

@baby-gnu
Copy link
Contributor

No description provided.

@baby-gnu
Copy link
Contributor Author

Hello,

I was not sure to fallback on root if ${USERNAME} does not exist.

For the use case, when preparing an image for context, we define which user can do what, then the ${USERNAME} is set directly in the template and ${PASSWORD} as a user input.

We have a custom, per distrib, context script which do whatever needed to create ${USERNAME} and set sudo authorization.

Regards.

@baby-gnu
Copy link
Contributor Author

Hello,

Is there anything missing to make this pull request mergeable?

Regards.

jfontan added a commit that referenced this pull request Jan 27, 2016
@jfontan jfontan merged commit bbcadaa into OpenNebula:master Jan 27, 2016
@jfontan
Copy link
Contributor

jfontan commented Jan 27, 2016

I think the patch is OK but maybe I would also add a CRYPTED_PASSWORD or something similar in case someone doesn't want it plain in the VM information. I'll merge this now.

Thank you and sorry for the delay.

@baby-gnu
Copy link
Contributor Author

+1

I'll handle the CRYPTED_PASSWORD soon and make a pull request.

Regards.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants