Skip to content

Conversation

@samuelallan72
Copy link
Member

Description

Configuring the volume size is important,
since deployments can vary greatly in size.
This retains the 5Gi default volume size,
but allows configuring it to suit the deployment.

This is only relevant to k8s deployments;
the docker deployments use a directory mounted from the host machine.

Supporting information

Private-ref: https://tasks.opencraft.com/browse/BB-9977

Testing instructions

  • install and enable this plugin in a tutor environment
  • run tutor config save
  • verify the pvc for a 5Gi volume is generated for typesense
  • set TYPESENSE_VOLUME_SIZE to something other than 5Gi in tutor's config.yml
  • run tutor config save again
  • verify the pvc for typesense data now has the size as configured

Deadline

None

Configuring the volume size is important,
since deployments can vary greatly in size.
This retains the 5Gi default volume size,
but allows configuring it to suit the deployment.

This is only relevant to k8s deployments;
the docker deployments use a directory mounted from the host machine.

Private-ref: https://tasks.opencraft.com/browse/BB-9977
Copy link
Member

@bradenmacdonald bradenmacdonald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't easily test this on my devstack, but the change is small and makes sense - and I believe we're testing it on the sandbox anyways, so all good.

@samuelallan72
Copy link
Member Author

Thanks @bradenmacdonald . Yep, I've tested it on the typesense sandbox. :)

@samuelallan72 samuelallan72 merged commit 6ba6000 into main Oct 21, 2025
2 checks passed
@samuelallan72 samuelallan72 deleted the samuel/typesense-volume branch October 21, 2025 01:32
@samuelallan72 samuelallan72 restored the samuel/typesense-volume branch October 21, 2025 01:33
@bradenmacdonald bradenmacdonald deleted the samuel/typesense-volume branch October 21, 2025 01: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.

3 participants