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

Release new crate versions #700

Merged
merged 6 commits into from
Jan 18, 2024
Merged

Conversation

dannycjones
Copy link
Contributor

Description of change

There's not many changes here, but let's get what we have out.

Relevant issues: N/A

Does this change impact existing behavior?

This change itself? No.

The crate releases however include one breaking change for mountpoint-s3-crt: "Return type of common::Uri::host_port changed from u16 to u32".

Open question: should this be a minor rather than patch update for mountpoint-s3-crt version?


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I agree to the terms of the Developer Certificate of Origin (DCO).

Signed-off-by: Daniel Carl Jones <[email protected]>
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 10:17 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 10:17 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 10:17 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 10:17 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 10:17 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 10:17 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 11:22 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 11:22 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 11:22 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 11:22 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 11:22 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 11:22 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 12, 2024 11:22 — with GitHub Actions Inactive
passaro
passaro previously approved these changes Jan 12, 2024
Copy link
Contributor

@passaro passaro left a comment

Choose a reason for hiding this comment

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

LGTM

mountpoint-s3-crt/CHANGELOG.md Show resolved Hide resolved
monthonk
monthonk previously approved these changes Jan 15, 2024
@dannycjones dannycjones marked this pull request as draft January 16, 2024 14:55
@dannycjones dannycjones dismissed stale reviews from monthonk and passaro via 8ecb3be January 17, 2024 12:31
@dannycjones dannycjones temporarily deployed to PR integration tests January 17, 2024 12:31 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 17, 2024 12:31 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 17, 2024 12:31 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 17, 2024 12:31 — with GitHub Actions Inactive
Signed-off-by: Daniel Carl Jones <[email protected]>
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 13:49 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 13:49 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 13:49 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 13:49 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 13:49 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 13:49 — with GitHub Actions Inactive
@dannycjones
Copy link
Contributor Author

Crate checks are expected to fail and are non-blocking: #658

@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 14:11 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 14:11 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 14:11 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 14:11 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 14:11 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 14:12 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests January 18, 2024 14:12 — with GitHub Actions Inactive
Copy link
Contributor

@monthonk monthonk left a comment

Choose a reason for hiding this comment

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

LGTM

@dannycjones dannycjones enabled auto-merge January 18, 2024 14:13
@dannycjones dannycjones added this pull request to the merge queue Jan 18, 2024
Merged via the queue into awslabs:main with commit 06aca78 Jan 18, 2024
23 of 24 checks passed
@dannycjones dannycjones deleted the release-crates branch January 18, 2024 14:47
jamesbornholt added a commit to jamesbornholt/mountpoint-s3 that referenced this pull request Mar 7, 2024
This workflow exists because we're worried about the 10MiB limit. The
sys crate is the only one that's in any danger of hitting that limit
because it's packaging a ton of C code, including an entire TLS
implementation. This workflow is also broken (awslabs#700) when updating crate
versions for multiple crates at the same time, which we really want to
do.

Since we only care about the size of the sys crate, let's only check
that one.

Signed-off-by: James Bornholt <[email protected]>
github-merge-queue bot pushed a commit that referenced this pull request Mar 7, 2024
This workflow exists because we're worried about the 10MiB limit. The
sys crate is the only one that's in any danger of hitting that limit
because it's packaging a ton of C code, including an entire TLS
implementation. This workflow is also broken (#700) when updating crate
versions for multiple crates at the same time, which we really want to
do.

Since we only care about the size of the sys crate, let's only check
that one.

Signed-off-by: James Bornholt <[email protected]>
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.

4 participants