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

dbforpostgresql Cluster (Cosmos DB) is missing encryption #3278

Closed
ruben-janssens opened this issue May 13, 2024 · 4 comments
Closed

dbforpostgresql Cluster (Cosmos DB) is missing encryption #3278

ruben-janssens opened this issue May 13, 2024 · 4 comments
Labels
kind/bug Some behavior is incorrect or out of spec resolution/no-repro This issue wasn't able to be reproduced

Comments

@ruben-janssens
Copy link

ruben-janssens commented May 13, 2024

What happened?

The pulumi resource for setting up a cosmos DB with postgresql is missing the dataencryption option even though the Azure ARM template does have this option. The api version is 2023-03-02-preview but when looking in that version of the pulumi variant the option is not available.

Example

from pulumi_azure_native.dbforpostgresql.v20230302preview import Cluster

Cluster(dataencryption=)

data encyrption is not an option but it is available in the arm template

Output of pulumi about

CLI
Version 3.100.0
Go Version go1.21.5
Go Compiler gc

Host
OS ubuntu
Version 22.04
Arch x86_64

Additional context

No response

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@ruben-janssens ruben-janssens added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team labels May 13, 2024
@Arttii
Copy link

Arttii commented May 13, 2024

I am having the same issue

@thomas11
Copy link
Contributor

In the latest provider version and in TypeScript, the property exists.

import * as resources from "@pulumi/azure-native/resources";
import { Cluster, DataEncryptionType } from "@pulumi/azure-native/dbforpostgresql/v20230302preview";

const resourceGroup = new resources.ResourceGroup("resourceGroup");

const cluster = new Cluster("cluster", {
    resourceGroupName: resourceGroup.name,
    location: resourceGroup.location,
    dataEncryption: {
        type: DataEncryptionType.SystemAssigned
    }
});

Looks like you're using Python? Will take a look if there's an issue in the Python SDK.

@thomas11 thomas11 removed the needs-triage Needs attention from the triage team label May 15, 2024
@thomas11
Copy link
Contributor

Looking at the Python SDK, the property's name is data_encryption, not dataencryption.

Our docs for Cluster do not list this property because it's not in the current default API version 2022-11-08. We're aware that other API versions are not present in the docs currently and are tracking improvements in this area.

@ruben-janssens
Copy link
Author

I must have missed it then, because I did explore other versions of the api. Maybe it was recently added?
Anyways thanks for pointing it out! I will close this then.

@thomas11 thomas11 added the resolution/no-repro This issue wasn't able to be reproduced label May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Some behavior is incorrect or out of spec resolution/no-repro This issue wasn't able to be reproduced
Projects
None yet
Development

No branches or pull requests

3 participants