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.

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

version: update builds and CI to go 1.22 #82

Merged
merged 5 commits into from
Sep 3, 2024
Merged

version: update builds and CI to go 1.22 #82

merged 5 commits into from
Sep 3, 2024

Conversation

vsoch
Copy link
Member

@vsoch vsoch commented Sep 1, 2024

Problem: the upstream scheduler-plugins is now using 1.22.0.
Solution: we should do the same.

Other Changes:

  • The webhook interface changing pointers to be proper interfaces (no longer builds with pointers)
  • Boost system is no longer provided in the container or needed for bindings, will result in error if not removed
  • Entrypoint changed again
  • Build logic in hack/build-images.sh again changed - I decided to move the docker build commands into our Makefile to both simplify and prevent these future bugs. Most of them are just tweaks/changes to environment variables.

Problem: the upstream scheduler-plugins is now using 1.22.0.
Solution: we should do the same.
Signed-off-by: vsoch <[email protected]>
Problem: boost_system is no longer required (or built into the base image)
Solution: remove it.
Signed-off-by: vsoch <[email protected]>
@vsoch
Copy link
Member Author

vsoch commented Sep 1, 2024

I think what is happening are upstream changes (many in the last 2 months) https://github.com/kubernetes-sigs/scheduler-plugins/commits/master/ so fluence builds but isn't functioning as expected. I'm too zonked to debug now or soon, but if someone wants to jump on it, please do!

The hack build images is changing too often to make it
easy to maintain, and we anticipated this. Instead
we can keep the build logic in our Makefile.

Signed-off-by: vsoch <[email protected]>
There was a change upstream that switched the kube-scheduler back to being
in bin (in the Dockerfile) but the corresponding manifest was not
updated.

Signed-off-by: vsoch <[email protected]>
@vsoch vsoch requested a review from cmisale September 2, 2024 05:16
Copy link
Collaborator

@cmisale cmisale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, very straightforward

@vsoch
Copy link
Member Author

vsoch commented Sep 3, 2024

Thanks @cmisale !

@vsoch vsoch merged commit 33a854a into main Sep 3, 2024
8 checks passed
@vsoch vsoch deleted the update-go-version branch September 3, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants