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

913 replace minio s3 with hdfs #930

Merged
merged 5 commits into from
Jan 16, 2025
Merged

Conversation

fjammes
Copy link
Contributor

@fjammes fjammes commented Dec 27, 2024

Linked to issue(s): Closes #913

What changes were proposed in this pull request?

Support HDFS (and Minio)

How was this patch tested?

On CI platforms for both science and noscience

@fjammes fjammes force-pushed the 913-replace-minio-s3-with-hdfs branch 2 times, most recently from 32c0958 to f99bc5b Compare December 27, 2024 12:52
@fjammes fjammes force-pushed the 913-replace-minio-s3-with-hdfs branch from a5321bf to 879fa60 Compare January 6, 2025 09:11
Copy link
Member

@JulienPeloton JulienPeloton left a comment

Choose a reason for hiding this comment

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

Thanks @fjammes -- tested and approved! I only left one comment on the documentation.

```bash
kubectl port-forward -n argocd $(kubectl get pods --selector=app.kubernetes.io/name=argocd-server -n argocd --output=jsonpath="{.items..metadata.name}") 8080
# Login is "admin, Password is set to "password", fix this in production
kubectl -n argocd patch secret argocd-secret -p '{"stringData": {"admin.password": "$2a$10$rRyBsGSHK6.uc8fntPwVIuLVHgsAhAX7TcdrqW/RADU0uh7CaChLa", "admin.passwordMtime": "'$(date +%FT%T%Z)'" }}'
Copy link
Member

Choose a reason for hiding this comment

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

Is this also the prod password?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No it isn't, this doc will be improved wrt to production setup!

- Install stackable operators with argoCD
- Improve ignoreDifferences field in Argo
  * add /spec/names/categories and shortNames
- Install HDFS with argoCD
- Reduce memory for spark pods to 1Gi
- Bump spark-operator to v2.1.0
- Add hdfs init for user 185 inside chart
@fjammes fjammes force-pushed the 913-replace-minio-s3-with-hdfs branch from 879fa60 to dc58e95 Compare January 16, 2025 09:23
@fjammes fjammes force-pushed the 913-replace-minio-s3-with-hdfs branch from dc58e95 to 676bff6 Compare January 16, 2025 10:29
@fjammes fjammes merged commit 06159ce into master Jan 16, 2025
16 checks passed
@fjammes fjammes deleted the 913-replace-minio-s3-with-hdfs branch January 16, 2025 11:22
@fjammes fjammes restored the 913-replace-minio-s3-with-hdfs branch January 16, 2025 11:24
@fjammes fjammes deleted the 913-replace-minio-s3-with-hdfs branch January 16, 2025 11:25
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.

Replace MinIO S3 with HDFS on Kubernetes Cluster for Fink Database Compatibility
2 participants