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

Chef workstation and Use Omnitruck API to get Chef URLs #180

Closed
wants to merge 1 commit into from

Conversation

daxgames
Copy link
Contributor

@daxgames daxgames commented Mar 23, 2019

Use Omnitruck API to get Chef URLs

Add Chef Workstation as a CM parameter

  • Add Chef Workstation 0.2.48 as CM and CM_VERSION parameter.
  • Code de-duplication by combining Chef Client, Chef DK, and Chef Workstation downloaders into a single code block that use the above Omnitruck API to retrieve download URLs.

Additional filename patterns in .gitignore.

@arizvisa arizvisa added the enhancement This will introduce or improve an already existing capability label Jan 3, 2020
@arizvisa
Copy link
Contributor

arizvisa commented Jan 3, 2020

@daxgames, I know this is a pretty old PR, but this _is_ adding functionality (i.e. adding more provisioners), right? Like, rather, are we presently provisioning with chef incorrectly?

Sorry, I'm not too familiar with chef and so I want to make sure I prioritize this PR correctly if you're still interested in merging it after all this time.

cleanup

ignore *.*.*

ignore *.*.*

cleanup

README.md

cleanup

cleanup

cleanup

cleanup

add chef workstation

fixes
arizvisa added a commit to arizvisa/boxcutter-windows that referenced this pull request Jan 6, 2020
…oner.

Revert "Applied changes in .gitignore from daxgames/boxcutter_windows@357a5db for updating the Chef provisioner."

This reverts commit b9c1098.
@arizvisa
Copy link
Contributor

arizvisa commented Jan 6, 2020

I split up your single commit into distinctly separate commits and re-factored its logic a bit.

I'ma close this PR as it will be superseded by PR #200 once I'm done testing it. I'll post the status of its merging here tho once its done in case you're still interested in using it.

@arizvisa arizvisa closed this Jan 6, 2020
@arizvisa arizvisa added the PR Superseded This particular PR has been superseded by a different PR mentioned in the discussion label Jan 6, 2020
arizvisa added a commit that referenced this pull request Jan 6, 2020
Revert "Applied changes in .gitignore from daxgames/boxcutter_windows@357a5db for updating the Chef provisioner."

This reverts commit b9c1098.
@arizvisa
Copy link
Contributor

arizvisa commented Jan 6, 2020

Ok. This was merged as PR #200. Thanks!

daxgames pushed a commit to daxgames/boxcutter_windows that referenced this pull request Feb 26, 2020
…oner.

Revert "Applied changes in .gitignore from 357a5db for updating the Chef provisioner."

This reverts commit b9c1098.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This will introduce or improve an already existing capability PR Superseded This particular PR has been superseded by a different PR mentioned in the discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants