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
HELP-56667: Revert changes in PR #1967 for DiskIOPS for Kubernetes deployment #2812
Conversation
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.
hold
internal/pointer/pointer.go
Outdated
@@ -44,3 +44,11 @@ func GetNonZeroValue[T constraints.Integer](val T) *T { | |||
|
|||
return nil | |||
} | |||
|
|||
func IntToInt64(source *int) *int64 { |
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.
can you move this closer to the kube package, these methods are a bit of a hack for converting between the SDK and the operator internal and not really helpful for them to be exposed to all the code base
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.
Fixed. Moved to .../kubernetes/operator/convert/
package
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.
thanks
5845fb4
5845fb4
to
4ef85a5
Compare
4ef85a5
to
1f67c53
Compare
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.
LGTM
Proposed changes
Jira ticket: HELP-56667
Closes #[issue number]
Checklist
make fmt
and formatted my codeFurther comments