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.

Sign up for GitHub

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

Jump to bottom

Add a NFS system test for tiered storage #11341

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Add a NFS system test for tiered storage #11341

wants to merge 10 commits into from

Conversation

showuon
Copy link
Member

@showuon showuon commented Apr 10, 2025 •
edited by see-quick
Loading

Type of change

  • Enhancement

Description

Add a NFS system test for tiered storage.

  1. Deploy a NFS using nfs-provisioner.
  2. Deploy NFS related resources, ex: services, roleBinding, serviceAccount, StorageClass,...
  3. Mount the additional NFS volume to Kafka pods as remote storage.
  4. Test tiered storage feature using Aiven FileSystem plugin.

Checklist

Please go through this checklist and make sure all applicable tasks have been done

  • Write tests
  • Make sure all tests pass
  • Update documentation
  • Check RBAC rights for Kubernetes / OpenShift roles
  • Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
  • Reference relevant issue(s) and close them after merging
  • Update CHANGELOG.md
  • Supply screenshots for visual changes, such as Grafana dashboards

@showuon showuon force-pushed the fileTest6 branch 2 times, most recently from 3cdcfc7 to 95b3144 Compare April 10, 2025 03:54
@showuon
clean up
df16d00 
Signed-off-by: Luke Chen <[email protected]>
ppatierno
ppatierno approved these changes Apr 11, 2025
View reviewed changes
Copy link
Member

@ppatierno ppatierno left a comment

Choose a reason for hiding this comment

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

LGTM. Just a nit.

@ppatierno ppatierno requested a review from a team April 11, 2025 07:45
@showuon
use NFS_VOLUME constant
b20fdfe 
Signed-off-by: Luke Chen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ppatierno ppatierno ppatierno approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@showuon @ppatierno