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

Decouple chef-client installation for windows - using omnitruck API #76

Conversation

Vasu1105
Copy link
Contributor

No description provided.

@adamedx
Copy link

adamedx commented Oct 12, 2015

I'll be really interested to see how this affects download and install time -- let's make sure we don't regress such that we can't get chef installed in time. There is work being done to speed up chef-client installations which will benefit us very soon

@Vasu1105
Copy link
Contributor Author

@adamedx - Tested with internal publish is not affecting download and install time.

@@ -48,7 +48,7 @@ function Get-SharedHelper {
}

function Get-LocalDestinationMsiPath($chefExtensionRoot) {

Choose a reason for hiding this comment

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

I think we can remove argument $chefExtensionRoot here ?

@NimishaS NimishaS force-pushed the vj/decouple_windows_chef_client_install branch from e63909f to 865e793 Compare October 19, 2015 08:38
NimishaS pushed a commit that referenced this pull request Oct 19, 2015
…hef_client_install

Decouple chef-client installation for windows - using omnitruck API
@NimishaS NimishaS merged commit ffdd22f into chef-partners:master Oct 19, 2015
@NimishaS NimishaS deleted the vj/decouple_windows_chef_client_install branch October 19, 2015 09:20
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