Skip to content

Moves away from Omnitruck to Chef Community + adds License param needed for commercial endpoint#35

Open
san983 wants to merge 3 commits intoactionshub:mainfrom
dnsimple:main
Open

Moves away from Omnitruck to Chef Community + adds License param needed for commercial endpoint#35
san983 wants to merge 3 commits intoactionshub:mainfrom
dnsimple:main

Conversation

@san983
Copy link

@san983 san983 commented May 29, 2025

Description

As Omnitruck is scheduled to reach its End of Life (EOL) by mid-2025. I'm pushing this PR to move away from Omnitruck endpoint, defaulting to Chef Community endpoint and adding the license param required for Chef Commercial endpoint

Issues Resolved

  • Omnitruck deprecation

Check List

  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

@san983
Copy link
Author

san983 commented May 31, 2025

@damacus could you please take a look here? Thanks!

@damacus
Copy link
Contributor

damacus commented May 31, 2025

@xorima is actually looking at this area. So I'll hand over to him 👍

@xorima
Copy link
Contributor

xorima commented Jun 1, 2025

Hey, there is a branch for this already (which I'm very struggling with getting working, no lie) the aim is to support the new chef ways of installing

BUT

Chef 19's new install method is not yet there for every OS, so will need some time and I need to work with Cinc to ensure they are also supported

PR in question: #33

@xorima xorima added the Release: Major Release is a semver.org major label Jun 1, 2025
@san983
Copy link
Author

san983 commented Jun 9, 2025

Hey, there is a branch for this already (which I'm very struggling with getting working, no lie) the aim is to support the new chef ways of installing

BUT

Chef 19's new install method is not yet there for every OS, so will need some time and I need to work with Cinc to ensure they are also supported

PR in question: #33

Thanks for your input. So may I ask if it makes sense to keep this PR? Actually I was not able to review #33 as it's a huge diff.

version:
description: 'version to install, default is latest'
required: false
omnitruckUrl:
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we keep this and if this is set to something different then default we use omnitruckUrl otherwise we use chefDownloadUrl to maintain backwards compatability, else when this is merged I fear a few customers will have things break

@xorima
Copy link
Contributor

xorima commented Oct 6, 2025

this seems sensible enough to keep until chef 19 is working for everyone,

Hey, there is a branch for this already (which I'm very struggling with getting working, no lie) the aim is to support the new chef ways of installing
BUT
Chef 19's new install method is not yet there for every OS, so will need some time and I need to work with Cinc to ensure they are also supported
PR in question: #33

Thanks for your input. So may I ask if it makes sense to keep this PR? Actually I was not able to review #33 as it's a huge diff.

I'm happy to get this in and working, just want a minor backwards compatability fix for now and when chef 19 is fully supported a wider how this should work can be discussed :)

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

Labels

Release: Major Release is a semver.org major

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants