PR to bump `pulumi-cloudflare` to support Cloudfla...
# contribute
l
PR to bump
pulumi-cloudflare
to support Cloudflare Pages now that upstream Terraform does https://github.com/pulumi/pulumi-cloudflare/pull/287
s
Were those resources just flat-out missing from the provider?
l
They only got added in Terraform in the last release or so, so there was nothing to map to, is my understanding.
The provider recently bumped the TF dependency though so they are now available -- this PR just maps them.
s
Yeah, I saw that. That's a mistake on our end. I'll alert the team to your PR.
l
What's the mistake? Not adding when the dependency was bumped?
No worries if so -- the TF change is literally only a few weeks old (if that)!
We're just literally about to deploy some pages so I figured worth a shot hacking them in
s
Yeah. There's env vars that should be set that will throw an exception if a resource is not mapped.
l
Oh interesting. That's a nice guard.
s
Yeah - we need to set them in GHA so that this stuff gets caught.
In the meanwhile, do you need to use a local version of the provider to get unstuck, and if so, do you need any help with that?
l
We're not blocked per se -- I was wary of doing something like that since I would guess it's a bit of effort, and if the PR is mergeable in the next week or so then it could be a bad use of time (all of this is hypothetical)
s
Well, if you are hot to trot, you can do
make provider
and then add the generated
bin/pulumi-resource-cloudflare
to your PATH. Then do
make build_sdks
and link to the local copy of the SDK in your language of choice.
l
Ah I see -- so basically vendoring that SDK in our repo until we can point to a new release?
s
Yes, but you also need the provider binary in your path.
So it may not be totally feasible for shared development. I can see about getting a release out today.
l
No rush, honestly -- thanks for all the help, Josh!
s
You are most welcome, and thank you for your contribution!