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

Expose configurability of terraform provider #40

Merged
merged 2 commits into from
Jun 29, 2024
Merged

Conversation

tchinmai7
Copy link
Collaborator

Description of your changes

  1. Exposes all configurable options currently exposed by the linode terraform provider
  2. Adds some validation for invalid configs

Was mostly motivated by the ability to delete buckets with objects in them. Ended up making a generic system. To test force deletes, apply these objects after running make local-deploy on the branch

  1. Your crossplane provider-linode secret
apiVersion: v1
data:
  credentials: $base64({"token": your token}) # Replace this 
kind: Secret
metadata:
  name: linode-api-token
  namespace: default
type: Opaque

  1. ProviderConfig
apiVersion: linode.upbound.io/v1beta1
kind: ProviderConfig
metadata:
  name: default
spec:
  credentials:
    source: Secret
    secretRef:
      name: linode-api-token
      namespace: default
      key: credentials
  config:
    obj_bucket_force_delete: true
    obj_use_temp_keys: true
  1. A bucket
apiVersion: objectstorage.linode.upbound.io/v1alpha1
kind: Bucket
metadata:
  name: test-bucket
spec:
  forProvider:
    label: test-1
    cluster: us-sea-1
  providerConfigRef:
    name: default
  1. Verify that the bucket is created in cloud manager, and upload some objects to it.
  2. k delete bucket test-bucket - Verify that the bucket gets cleaned up

Fixes #

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

@tchinmai7 tchinmai7 merged commit 9fe73c1 into main Jun 29, 2024
@tchinmai7 tchinmai7 deleted the add-configurability branch July 11, 2024 17:47
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.

2 participants