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

Use Chef Omnitruck API to get Chef/ChefDK URls #87

Closed
wants to merge 9 commits into from

Conversation

daxgames
Copy link
Contributor

  • Also updated Chef and ChefDK versions to latest stable

@daxgames
Copy link
Contributor Author

Implemented Omnitruck API suggested in #32

Also fixes assumption that x64 versions exist causing download failure older versions.

@daxgames daxgames closed this Dec 11, 2016
@daxgames daxgames reopened this Dec 11, 2016
@daxgames
Copy link
Contributor Author

daxgames commented Dec 14, 2016

Output from an older Chef 12.5.1 client run omnitruck returns an Chef-Client x86 URL:

==> virtualbox-iso: Provisioning with shell script: script/cmtool.bat
    virtualbox-iso: ==> Getting Chef 12.5.1 download URL
    virtualbox-iso: ==> Creating "C:\Users\vagrant\AppData\Local\Temp\chef"
    virtualbox-iso: ==> Downloading "https://packages.chef.io/files/stable/chef/12.5.1/windows/2012r2/chef-client-12.5.1-1-x86.msi" to "C:\Users\vagrant\AppData\Local\Temp\chef\chef-client-12.5.1-1-x86.ms
i"
    virtualbox-iso: ==> Installing Chef client 12.5.1 x86
    virtualbox-iso: 2016-12-14 09:52:00 URL:https://packages.chef.io/files/stable/chef/12.5.1/windows/2012r2/chef-client-12.5.1-1-x86.msi [78307328/78307328] -> "C:/Users/vagrant/AppData/Local/Temp/chef/c
hef-client-12.5.1-1-x86.msi" [1]
    virtualbox-iso:
    virtualbox-iso: Pinging 127.0.0.1 with 32 bytes of data:
    virtualbox-iso: Reply from 127.0.0.1: bytes=32 time<1ms TTL=128
    virtualbox-iso: Reply from 127.0.0.1: bytes=32 time<1ms TTL=128
    virtualbox-iso: Reply from 127.0.0.1: bytes=32 time<1ms TTL=128
    virtualbox-iso: Reply from 127.0.0.1: bytes=32 time<1ms TTL=128
    virtualbox-iso:
    virtualbox-iso: Ping statistics for 127.0.0.1:
    virtualbox-iso: Packets: Sent = 4, Received = 4, Lost = 0 (0% loss),
    virtualbox-iso: Approximate round trip times in milli-seconds:
    virtualbox-iso: Minimum = 0ms, Maximum = 0ms, Average = 0ms
    virtualbox-iso: ==> Script exiting with errorlevel 0

@daxgames
Copy link
Contributor Author

daxgames commented Dec 14, 2016

Output from an newer Chef 12.16.42 client run omnitruck returns an Chef-Client x64 URL:

==> virtualbox-iso: Provisioning with shell script: script/cmtool.bat
    virtualbox-iso: ==> Getting Chef 12.16.42 download URL
    virtualbox-iso: ==> Creating "C:\Users\vagrant\AppData\Local\Temp\chef"
    virtualbox-iso: ==> Downloading "https://packages.chef.io/files/stable/chef/12.16.42/windows/2012r2/chef-client-12.16.42-1-x64.msi" to "C:\Users\vagrant\AppData\Local\Temp\chef\chef-client-12.16.42-1-x64.msi"
    virtualbox-iso: ==> Installing Chef client 12.16.42 x86_64
    virtualbox-iso: 2016-12-14 10:09:30 URL:https://packages.chef.io/files/stable/chef/12.16.42/windows/2012r2/chef-client-12.16.42-1-x64.msi [177176576/177176576] -> "C:/Users/vagrant/AppData/Local/Temp/chef/chef-client-12.16.42-1-x64.msi" [1]
    virtualbox-iso:
    virtualbox-iso: Pinging 127.0.0.1 with 32 bytes of data:
    virtualbox-iso: Reply from 127.0.0.1: bytes=32 time<1ms TTL=128
    virtualbox-iso: Reply from 127.0.0.1: bytes=32 time<1ms TTL=128
    virtualbox-iso: Reply from 127.0.0.1: bytes=32 time<1ms TTL=128
    virtualbox-iso: Reply from 127.0.0.1: bytes=32 time<1ms TTL=128
    virtualbox-iso:
    virtualbox-iso: Ping statistics for 127.0.0.1:
    virtualbox-iso: Packets: Sent = 4, Received = 4, Lost = 0 (0% loss),
    virtualbox-iso: Approximate round trip times in milli-seconds:
    virtualbox-iso: Minimum = 0ms, Maximum = 0ms, Average = 0ms
    virtualbox-iso: ==> Script exiting with errorlevel 0```

@dragetd
Copy link

dragetd commented Feb 27, 2019

We keep defining the download URLs at several spots. It makes them hard to manage and update. I am preparing a PR (should be up in a few minutes) which also updates a lot of URLs (and thus will cause even more merge-issues with this one).

I'd highly welcome an attempt to move all URLs of additional tools to one spot. :-/

@daxgames
Copy link
Contributor Author

I also have an updated version of this that uses this new ps1_download function assuming #179 gets merged.

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.

2 participants