-
Notifications
You must be signed in to change notification settings - Fork 20
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
upgrade kubebuilder to plugin/v4 #418
Conversation
Tests currently failing will take a look |
@sanposhiho After upgrading kubebuilder, controller with name tortoise already exists. Controller names must be unique to avoid multiple controllers reporting to the same metric. This error appears for every controller test after the first test. The way the controller tests are written for tortoise is very different from the documentation so i'm not sure how to fix this.. do you know if its possible to delete this controller in the cleanup? |
What if you move startController() like this? runTest := func(path string) {
initializeResourcesFromFiles(ctx, k8sClient, filepath.Join(path, "before"))
- stopFunc = startController(ctx)
if os.Getenv("UPDATE_TESTCASES") == "true" {
generateTestCases(path)
} else {
checkWithWantedResources(path)
}
cleanUp()
}
+ BeforeEach(func() {
+ stopFunc = startController(ctx)
+ }) |
@sanposhiho I fixed it with this |
@sanposhiho Not sure what this exec error is for the tortoisectl test would you have any clue? |
cmd/tortoisectl/main.go
Outdated
|
||
import "github.com/mercari/tortoise/cmd/tortoisectl/commands" | ||
|
||
func main() { | ||
func tortoisectl() { |
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.
why renamed like this? It might be the reason
@sanposhiho all the tests are passing but somehow lint is getting killed on the runner.. likely to be OOM? seems like the linter is using a lot of memory on the runner compared to my local |
Looks like go version issue. Specified go1.22.6 and it works now |
What this PR does / why we need it:
In place upgrade for kubebuilder to plugin v4
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer: