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

SFTP Option should allow a separate file allowed to be configured from a secret. #1931

Open
Jeremy-Boyle opened this issue Jan 4, 2024 · 3 comments
Labels

Comments

@Jeremy-Boyle
Copy link

Jeremy-Boyle commented Jan 4, 2024

Is your feature request related to a problem? Please describe.
Currently the SFTP private key is not configurable and would like to provide my own private key. Currently the SFTP uses the same console tls for SFTP.

Describe the solution you'd like
A value in the CRD that allows you to change the the value from a secret key / path , should automatically change the deployment and mount the secret to the path that is expected when provided.

Helm chart should also be updated to allow easy configuration of the tenants.

Describe alternatives you've considered
Not using minio operator

Suggestion:

spec:
  features:
    enableSFTP: true
    sftp:
      secretName: secret-name
      secretKey: (Default if not provided private.key)

Open to suggestions / changes happy to do the PR to support this.

@Jeremy-Boyle
Copy link
Author

See

if t.Spec.Features != nil && t.Spec.Features.EnableSFTP != nil && *t.Spec.Features.EnableSFTP {
pkFile := filepath.Join(miniov2.MinIOCertPath, certs.PrivateKeyFile)
args = append(args, []string{
"--sftp", fmt.Sprintf("address=:%d", miniov2.MinIOSFTPPort),
"--sftp", "ssh-private-key=" + pkFile,
}...)
containerPorts = append(containerPorts, v1.ContainerPort{
ContainerPort: miniov2.MinIOSFTPPort,
})
}

@harshavardhana
Copy link
Member

This is a very low priority for us. Feel free to work on this and send a PR yourself.

@cniackz
Copy link
Contributor

cniackz commented Apr 8, 2024

We are going to keep it open, but we are still waiting for a PR if you have time to invest and thank you ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants