-
Notifications
You must be signed in to change notification settings - Fork 104
Rewrite the data-loader in go #2254
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
Rewrite the data-loader in go #2254
Conversation
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
6da4b0b to
8313e41
Compare
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
nicmorales9
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would definitely test the multi-cluster path but otherwise LGTM
9ac94c1 to
19bbc19
Compare
Result of fdb-kubernetes-operator-pr on Linux RHEL 9
|
19bbc19 to
1718916
Compare
Result of fdb-kubernetes-operator-pr on Linux RHEL 9
|
Result of fdb-kubernetes-operator-pr on Linux RHEL 9
|
7618f0d to
bf9db79
Compare
johscheuer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed all the other changes and made this a 1:1 rewrite. We can add the other features in the future again.
Result of fdb-kubernetes-operator-pr on Linux RHEL 9
|
Result of fdb-kubernetes-operator-pr on Linux RHEL 9
|
Description
Rewrite the data-loader in go.
Type of change
Discussion
Rewrite of the data loader in Golang.
Testing
Ran the tests manually with the new image.
Documentation
Follow-up