-
Notifications
You must be signed in to change notification settings - Fork 101
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
DXE-4326 akamai_edgekv resource always issues a PUT /edgekv/v1/initialize
#589
Comments
Also: the
|
Thanks for reporting this issue. We need analyze impact of this change and we will inform you about progress. BR, |
lsadlon
changed the title
akamai_edgekv resource always issues a
DXE-4326 akamai_edgekv resource always issues a Oct 23, 2024
PUT /edgekv/v1/initialize
PUT /edgekv/v1/initialize
Merged
Hi @aukevanleeuwen this issue has been addressed in recent release. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Terraform and Akamai Terraform Provider Versions
Affected Resource(s)
Please list the resources as a list, for example:
akamai_edgekv
Terraform Configuration Files
Expected Behavior
I would to have this resource run with the least amount of privileges.
Actual Behavior
Looking at my access denied error messages, but more specifically here:
terraform-provider-akamai/pkg/providers/edgeworkers/resource_akamai_edgekv.go
Lines 141 to 144 in e591db9
It appears the EdgeWorker Initialize call is always issued. If I'm reading everything correctly that would need quite a bit of permissions. Even write permission on the root group for CP codes for example. I would want to run this 'initialization' manually through some admin CLI credentials perhaps just once, after which it should ideally work with less permissions.
I'm no Golang expert, but looking at the code it should be quite trivial to first query the initialization status (i.e.
client.GetEdgeKVInitializationStatus(ctx)
and depending on that start the initialization or just skip that call all together. By my testing, theclient.GetEdgeKVInitializationStatus(ctx)
needs a lot less permissions.Steps to Reproduce
Please list the steps required to reproduce the issue, for example:
terraform apply
with the above mentioned resourceThe text was updated successfully, but these errors were encountered: