Skip to content

misc: add mounted_secrets parameter with defaulting to old behavior#889

Open
isidentical wants to merge 1 commit intomainfrom
expose-mounted-secrets
Open

misc: add mounted_secrets parameter with defaulting to old behavior#889
isidentical wants to merge 1 commit intomainfrom
expose-mounted-secrets

Conversation

@isidentical
Copy link
Collaborator

specify which secrets you want to mount and which not

class MountedSecretsApp(fal.App, keep_alive=300, max_concurrency=1):
    machine_type = "XS"
    mounted_secrets = ["MOUNTED_TEST_SECRET"]

    @fal.endpoint("/")
    def check_secrets(self) -> SecretsOutput:
        return SecretsOutput(
            has_mounted="MOUNTED_TEST_SECRET" in os.environ,
            has_not_mounted="NOT_MOUNTED_SECRET" in os.environ,
        )

by default we mount to mounted_secrets = ["*"].

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant