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.

Sign up for GitHub

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

Jump to bottom

ARO-12463: *: use go 1.22.9 #4069

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

ARO-12463: *: use go 1.22.9 #4069

wants to merge 2 commits into from
+83 −64

Conversation

stevekuznetsov
Copy link
Contributor

No description provided.

@stevekuznetsov stevekuznetsov changed the title ARO-12463: *: use go 1.22.11 ARO-12463: *: use go 1.22.9 Jan 29, 2025
@stevekuznetsov
*: use go 1.22.9
15bcdab 
Signed-off-by: Steve Kuznetsov <[email protected]>
@stevekuznetsov
Copy link
Contributor Author

stevekuznetsov commented Jan 29, 2025 •
edited
Loading 

go generate ./...
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xa2cd2f]

goroutine 102 [running]:
go/types.(*Checker).handleBailout(0xc00010f600, 0xc00004fd40)
	/usr/lib/golang/src/go/types/check.go:367 +0x88
...
created by sigs.k8s.io/controller-tools/pkg/loader.(*TypeChecker).check in goroutine 92
	/github/home/go/pkg/mod/sigs.k8s.io/[email protected]/pkg/loader/refs.go:260 +0x1c5
pkg/operator/generate.go:7: running "controller-gen": exit status 2

Need to go for now, but likely need to bump the controller-gen version.

@stevekuznetsov
*: update controller-gen
19336bf 
   bingo get sigs.k8s.io/controller-tools/cmd/[email protected]
   make generate

Signed-off-by: Steve Kuznetsov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kimorris27 kimorris27 kimorris27 approved these changes

@cadenmarchese cadenmarchese cadenmarchese approved these changes

@jewzaam jewzaam Awaiting requested review from jewzaam jewzaam is a code owner

@bennerv bennerv Awaiting requested review from bennerv bennerv is a code owner

@hawkowl hawkowl Awaiting requested review from hawkowl hawkowl is a code owner

@rogbas rogbas Awaiting requested review from rogbas rogbas is a code owner

@petrkotas petrkotas Awaiting requested review from petrkotas petrkotas is a code owner

@jharrington22 jharrington22 Awaiting requested review from jharrington22 jharrington22 is a code owner

@cblecker cblecker Awaiting requested review from cblecker cblecker is a code owner

@UlrichSchlueter UlrichSchlueter Awaiting requested review from UlrichSchlueter UlrichSchlueter is a code owner

@SudoBrendan SudoBrendan Awaiting requested review from SudoBrendan SudoBrendan is a code owner

@yjst2012 yjst2012 Awaiting requested review from yjst2012 yjst2012 is a code owner

@jaitaiwan jaitaiwan Awaiting requested review from jaitaiwan jaitaiwan is a code owner

@anshulvermapatel anshulvermapatel Awaiting requested review from anshulvermapatel anshulvermapatel is a code owner

@hlipsig hlipsig Awaiting requested review from hlipsig hlipsig is a code owner

@mociarain mociarain Awaiting requested review from mociarain mociarain is a code owner

@tsatam tsatam Awaiting requested review from tsatam tsatam is a code owner

@bitoku bitoku Awaiting requested review from bitoku bitoku is a code owner

@fahlmant fahlmant Awaiting requested review from fahlmant fahlmant is a code owner

@sankur-codes sankur-codes Awaiting requested review from sankur-codes sankur-codes is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stevekuznetsov @cadenmarchese @kimorris27