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

Support Kubernetes 1.28 #285

Merged
merged 7 commits into from
May 1, 2024
Merged

Support Kubernetes 1.28 #285

merged 7 commits into from
May 1, 2024

Conversation

terassyi
Copy link
Contributor

@terassyi terassyi commented Apr 26, 2024

This PR supports Kubernetes 1.28.
And this also updates dependencies.

Please notice that CNI library update is not contained in this PR.
I plan to upgrade CNI version in the other PR.

Signed-off-by: terasihma [email protected]

Signed-off-by: terasihma <[email protected]>
@terassyi terassyi self-assigned this Apr 26, 2024
@terassyi terassyi requested a review from yokaze May 1, 2024 06:48
@terassyi terassyi marked this pull request as ready for review May 1, 2024 06:52
"net"
"os"
"strconv"
"time"

v2 "github.com/cybozu-go/coil/v2"
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't mind how the packages are organized into blocks, but at least all the packages from Coil should appear in a same block.

v2 "github.com/cybozu-go/coil/v2"
coilv2 "github.com/cybozu-go/coil/v2/api/v2"
"github.com/cybozu-go/coil/v2/pkg/constants"
"github.com/cybozu-go/coil/v2/pkg/indexing"
"github.com/cybozu-go/coil/v2/pkg/ipam"
"github.com/cybozu-go/coil/v2/runners"

@terassyi terassyi requested a review from yokaze May 1, 2024 07:26
Copy link
Contributor

@yokaze yokaze left a comment

Choose a reason for hiding this comment

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

Could you please check all the change, as there are same issue around five?

@terassyi
Copy link
Contributor Author

terassyi commented May 1, 2024

Sorry, I forgot checking all files.
I'll fix it.

Signed-off-by: terashima <[email protected]>
Copy link
Contributor

@yokaze yokaze left a comment

Choose a reason for hiding this comment

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

LGTM!

@terassyi terassyi merged commit 5faaa12 into main May 1, 2024
7 checks passed
@terassyi terassyi deleted the k8s-1.28 branch May 1, 2024 08:39
@terassyi terassyi mentioned this pull request May 28, 2024
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