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 #1750

Merged
merged 5 commits into from Sep 17, 2021
Merged

Update Rust dependencies #1750

merged 5 commits into from Sep 17, 2021

Conversation

tjkirch
Copy link
Contributor

@tjkirch tjkirch commented Sep 15, 2021

Description of changes:

tools: cargo update
tools: update nonzero_ext to 0.3

Update serde-plain to 1.0
    The update_metadata crate is shared between the sources/ and tools/
    workspaces.  This update covers both so the tools/ workspace doesn't
    need to be updated again after sources/.

sources: cargo update
sources: update serde-xml-rs

Not updated: gptman, hyper (in sources/), x509-parser

Testing done:

Unit tests pass. Built AWS k8s 1.21 and ECS AMIs for x86_64 and aarch64. Confirmed systemctl status running, dmesg and journal OK. Ran a pod/task OK. Made API changes OK.

Confirmed that cargo make repo and cargo make ssm still work OK.

Built vmware-k8s-1.21 and confirmed that XML in an OVF still provides user data to it correctly, with the serde-xml-rs update.

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.

@tjkirch
Copy link
Contributor Author

tjkirch commented Sep 15, 2021

^ Add a commit that locks to httptest 0.15.2 for now, since 0.15.3's IPv6 support doesn't yet work in CI.

@tjkirch
Copy link
Contributor Author

tjkirch commented Sep 16, 2021

I'll rebase this onto #1751 when it's done so we don't need the httptest locking.

@tjkirch tjkirch marked this pull request as draft September 16, 2021 17:35
The update_metadata crate is shared between the sources/ and tools/
workspaces.  This update covers both so the tools/ workspace doesn't
need to be updated again after sources/.
@tjkirch
Copy link
Contributor Author

tjkirch commented Sep 16, 2021

^ Rebase on develop to get #1751 so I could remove the commit locking httptest.

@tjkirch tjkirch marked this pull request as ready for review September 16, 2021 23:21
@tjkirch tjkirch merged commit df7c522 into bottlerocket-os:develop Sep 17, 2021
@tjkirch tjkirch deleted the rust-deps branch September 17, 2021 20:22
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.

None yet

4 participants