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

Upgrade terraform-provider-nomad to v2.1.0 #193

Closed
pulumi-bot opened this issue Dec 21, 2023 · 5 comments · Fixed by #226
Closed

Upgrade terraform-provider-nomad to v2.1.0 #193

pulumi-bot opened this issue Dec 21, 2023 · 5 comments · Fixed by #226
Assignees
Labels
blocked The issue cannot be resolved without 3rd party action. kind/enhancement Improvements or new features resolution/fixed This issue was fixed

Comments

@pulumi-bot
Copy link
Contributor

Release details: https://github.com/hashicorp/terraform-provider-nomad/releases/tag/v2.1.0

@pulumi-bot pulumi-bot added the kind/enhancement Improvements or new features label Dec 21, 2023
@pulumi-bot pulumi-bot self-assigned this Dec 21, 2023
@iwahbe iwahbe added the blocked The issue cannot be resolved without 3rd party action. label Dec 21, 2023
@iwahbe
Copy link
Member

iwahbe commented Dec 21, 2023

This version contains BUSL code: github.com/hashicorp/nomad/helper/pointer.

@kevinschoonover
Copy link

kevinschoonover commented Jan 6, 2024

This version contains BUSL code: github.com/hashicorp/nomad/helper/pointer.

@iwahbe I was looking around the repo, but don't see any indication its importing this package. It looks like its bundling its own and tagged it as MPL-2.0. I was looking in their FAQ and they also mention their terraform providers remain MPL-2.0

I admittedly don't understand the finer points of licensing, but this release includes new constructs (such as crons) which I use for my jobs

@iwahbe iwahbe self-assigned this Jan 19, 2024
@iwahbe iwahbe added blocked The issue cannot be resolved without 3rd party action. and removed blocked The issue cannot be resolved without 3rd party action. labels Jan 25, 2024
@iwahbe
Copy link
Member

iwahbe commented Jan 25, 2024

Hi @kevinschoonover. Sorry for not getting back to you sooner. This is my understanding:

Hashi's BUSL license is infectious (transitive), so any code that depends on BUSL code is in fact BUSL. This is like a copyleft licenses.

The terraform-nomad-provider (which pulumi-nomad wraps) imports github.com/hashicorp/nomad/jobspec (src). Ideally, that would be fine, since github.com/hashicorp/nomad/jobspec is explicitly licensed as MPL 2.0... however, github.com/hashicorp/nomad/jobspec imports github.com/hashicorp/nomad/helper/pointer (src). github.com/hashicorp/nomad/helper/pointer is licensed at the repo root, and thus BUSL.

That means that github.com/hashicorp/nomad/jobspec and thus github.com/hashicorp/terraform-nomad-provider can be argued to be BUSL. The license is unclear.

We are not willing to assume liability on an ambiguous license. We can upgrade again if github.com/hashicorp/nomad/jobspec removes their trivial dependency on github.com/hashicorp/nomad/helper/pointer or if Hashi drops an OSS LICENSE file in the path of github.com/hashicorp/nomad/helper/pointer (like they have for github.com/hashicorp/nomad/jobspec).

@kevinschoonover
Copy link

kevinschoonover commented Jan 27, 2024

Hey @iwahbe, the nomad folks should now be tracking this in hashicorp/nomad#19832. Putting it here for reference.

There is an initial PR which should unblock this hashicorp/nomad#19832 when merged.

@iwahbe
Copy link
Member

iwahbe commented Jan 29, 2024

@kevinschoonover That is awesome!

iwahbe added a commit that referenced this issue Jan 31, 2024
This PR was generated via `$ upgrade-provider pulumi/pulumi-nomad
--kind=all --target-bridge-version=latest`.

---

- Upgrading terraform-provider-nomad from 2.0.0  to 2.1.1.
	Fixes #225
	Fixes #193

---------

Co-authored-by: Ian Wahbe <ian@wahbe.com>
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked The issue cannot be resolved without 3rd party action. kind/enhancement Improvements or new features resolution/fixed This issue was fixed
Projects
None yet
3 participants