-
Notifications
You must be signed in to change notification settings - Fork 195
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Update Azure Go SDK #786
Update Azure Go SDK #786
Conversation
41c68f9
to
d4ab975
Compare
See notes in fluxcd/flux2#2696 |
@hiddeco always one step ahead. I was doing this work for the Thanos project so though why not just continue as the information was fresh in my head. |
@phillebaba please rebase with main where we have Go 1.18 |
fd1e121
to
663215d
Compare
Signed-off-by: Philip Laine <philip.laine@gmail.com>
663215d
to
fdf6b01
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
Thanks @phillebaba
@stefanprodan so I have had the same issues with CodeQL in the Thanos project. It also started happening when I updated the Azure SDK. I don't no if you immediately see what is wrong but I have spent some time trying to figure this out now without any luck. |
We can't merge this as the scanning is critical in our release workflow and this week we want to release OCI. |
Signed-off-by: Philip Laine <philip.laine@gmail.com>
@stefanprodan seems like the issue was not Azure SDK related but rather caused by CodeQL not using the correct Go version. Specifically setting the Go version has solved this issue. I found this issue which seems to cover the issue we are seeing. |
Updates Azure Go SDK to stable release.
This PR may have to wait as the stable release requires Go 1.18, not sure if that is a hard or soft requirement.