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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Properties Missing for Cloudflare Pages Projects #419

Open
fa-bwilliams opened this issue May 24, 2023 · 3 comments
Open

New Properties Missing for Cloudflare Pages Projects #419

fa-bwilliams opened this issue May 24, 2023 · 3 comments
Labels
awaiting-upstream Awaiting upstream dependency kind/enhancement Improvements or new features

Comments

@fa-bwilliams
Copy link

fa-bwilliams commented May 24, 2023

Hello!

  • Vote on this issue by adding a 馃憤 reaction
  • If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

Cloudflare introduced a new build system version property that is available in their API as build_image_major_version accepting a value of 1 or 2. This property is unavailable for use in the pulumi-cloudflare provider. As we are building the rest of our Clouflare Pages project with Pulumi, it would be nice to be able to include this property instead of having to PATCH directly against the Cloudflare API outside of the pulumi project.

@fa-bwilliams fa-bwilliams added kind/enhancement Improvements or new features needs-triage Needs attention from the triage team labels May 24, 2023
@fa-bwilliams
Copy link
Author

As a workaround for now, I used Pulumi Command and interpolated a curl command to patch the property directly with the Cloudflare Pages API.

@fa-bwilliams
Copy link
Author

fa-bwilliams commented May 25, 2023

I found the same issue regarding another new API property for Smart Placement of Cloudflare Pages Functions. I created a workaround in the same way, until the pulumi-cloudflare provider supports this.

@aq17
Copy link
Contributor

aq17 commented May 25, 2023

@fa-bwilliams thanks for the update and for demonstrating a workaround! We'll keep track of this update in the upstream provider, and release a new version once it is available.

@aq17 aq17 added awaiting-feedback awaiting-upstream Awaiting upstream dependency and removed needs-triage Needs attention from the triage team awaiting-feedback labels May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-upstream Awaiting upstream dependency kind/enhancement Improvements or new features
Projects
None yet
Development

No branches or pull requests

2 participants