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

[Resources: cloudflare_argo] Tiered Cache Topology selection needed #996

Closed
igloo777 opened this issue Mar 15, 2021 · 7 comments
Closed
Labels
kind/enhancement Categorizes issue or PR as related to improving an existing feature. workflow/pending-upstream-library Indicates an issue or PR requires changes from an upstream library.

Comments

@igloo777
Copy link

Current Terraform version

Terraform version v0.14.4

Description

As in https://registry.terraform.io/providers/cloudflare/cloudflare/latest/docs/resources/argo - cloudflare provider 2.19.2
supports enabling of tiered_caching and smart_routing, But it doesn't support selection of Tiered Cache Topology for tiered_caching.

Use cases

Currently we have to switch the Tiered Cache Topology in web ui, while the state we keep in terraform.

Potential Terraform configuration

variable "tiered_caching_topology" {
  type = string
  # Value may be "generic" or "smart"
  default = "generic"
}


resource "cloudflare_argo" "example" {
  zone_id        = "d41d8cd98f00b204e9800998ecf8427e"
  tiered_caching = "on"
  tiered_caching_topology = "smart"
  smart_routing  = "off"
}

References

No response

Currently we have to switch the Tiered Cache Topology in web ui, while the state we keep in terraform.
The lack of the feature Tiered Cache Topology may be an issue for projects with a large number of domains .

@igloo777 igloo777 added kind/enhancement Categorizes issue or PR as related to improving an existing feature. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Mar 15, 2021
@jacobbednarz
Copy link
Member

At the time of writing, there isn't a public API endpoint support for this so we can't yet add anything to cloudflare-go. Once that lands, we can look at adding support in the Go library before incorporating it here.

@jacobbednarz jacobbednarz added workflow/pending-upstream-library Indicates an issue or PR requires changes from an upstream library. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Mar 17, 2021
@toddschilling
Copy link

Tracking this issue as Seismic Software has a need for this as well.

@AlexKasaku
Copy link

This appears to be supported via an undocumented API endpoint.

To enable smart topology:

PATCH /zones/:zone_id/cache/tiered_cache_smart_topology_enable with {"value":"on"}

To enable smart topology and use generic:

PATCH /zones/:zone_id/cache/tiered_cache_smart_topology_enable with {"value":"off"}

@borisceranic
Copy link

borisceranic commented May 10, 2022

@jacobbednarz any chance to have this feature added to cloudflare-go (and, subsequently, to the CF provider) now that there's an (undocumented) API endpoint for this?

I'm hoping that the fact this API is undocumented should not be an issue, because tiered_caching was also previously implemented against a similarly undocumented API endpoint 😂

@jacobbednarz
Copy link
Member

I'm afraid the endpoint needs to be publicly documented before using it in Terraform or the SDK. I'd recommend opening a support ticket to get it routed to the service team to action.

@evanrappe
Copy link

FYI @jacobbednarz I got this submitted via a support request, and after a few days got this resolution response:

Thanks for your continued patience. I will close this ticket as we have filed an internal request to get this resolved.

While it doesn't seem to be done yet, hopefully it will be soon.

@jacobbednarz
Copy link
Member

this has landed in v3.31.0 via #2101

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Categorizes issue or PR as related to improving an existing feature. workflow/pending-upstream-library Indicates an issue or PR requires changes from an upstream library.
Projects
None yet
Development

No branches or pull requests

6 participants