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

Update rust dependencies #2816

Merged
merged 2 commits into from
Feb 24, 2023
Merged

Conversation

jpculp
Copy link
Member

@jpculp jpculp commented Feb 18, 2023

Issue number:

Closes #2762

Description of changes:

Ran cargo update for sources and tools, avoided some updates that pull unnecessary extra versions, and adjusted the deny and clarify TOMLs as necessary.

Also removed unnecessary patch versions in Cargo.tomls for more flexibility.

Testing done:

  • Built aws-ecs-1 (x86_64). Launched AMI and ran various host container tests.
  • Built aws-k8s-1.24 (aarch64). Launched AMI and ran through some basic operations to verify functionality. (thanks @stmcginnis)

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

@jpculp jpculp requested a review from webern February 18, 2023 01:54
Copy link
Contributor

@stmcginnis stmcginnis left a comment

Choose a reason for hiding this comment

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

This looks good to me. To check for any unintended side effect, built and deployed an aws-k8s-1.24-aarch64 node. Ran through some basic operations to verify functionality:

$ k get nodes
NAME                                           STATUS   ROLES    AGE     VERSION
ip-192-168-76-121.us-east-2.compute.internal   Ready    <none>   8m17s   v1.24.10-eks-08ad9cc
$ k apply -f ../testing/test-pod.yaml
pod/webserver created
$ k get all -A
NAMESPACE     NAME                           READY   STATUS    RESTARTS   AGE
default       pod/webserver                  1/1     Running   0          6s
kube-system   pod/aws-node-7qdnq             1/1     Running   0          8m36s
kube-system   pod/coredns-5c5677bc78-hzhbc   1/1     Running   0          3d16h
kube-system   pod/coredns-5c5677bc78-vz8zt   1/1     Running   0          3d16h
kube-system   pod/kube-proxy-xr6xb           1/1     Running   0          8m36s

NAMESPACE     NAME                 TYPE        CLUSTER-IP    EXTERNAL-IP   PORT(S)         AGE
default       service/kubernetes   ClusterIP   10.100.0.1    <none>        443/TCP         4d19h
kube-system   service/kube-dns     ClusterIP   10.100.0.10   <none>        53/UDP,53/TCP   4d18h

NAMESPACE     NAME                        DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR   AGE
kube-system   daemonset.apps/aws-node     1         1         1       1            1           <none>          4d18h
kube-system   daemonset.apps/kube-proxy   1         1         1       1            1           <none>          4d18h

NAMESPACE     NAME                      READY   UP-TO-DATE   AVAILABLE   AGE
kube-system   deployment.apps/coredns   2/2     2            2           4d18h

NAMESPACE     NAME                                 DESIRED   CURRENT   READY   AGE
kube-system   replicaset.apps/coredns-5c5677bc78   2         2         2       4d18h

@jpculp jpculp marked this pull request as ready for review February 20, 2023 18:30
Copy link
Contributor

@webern webern left a comment

Choose a reason for hiding this comment

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

Looks good

@jpculp jpculp merged commit 1bbe74e into bottlerocket-os:develop Feb 24, 2023
@jpculp jpculp deleted the rust-updates branch February 24, 2023 23:12
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.

v1.13.0 Rust updates
4 participants