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

chore: Update ic-agent and ic-agent's reverse dependencies to v0.39 #3282

Merged
merged 4 commits into from
Jan 7, 2025

Conversation

anchpop
Copy link
Contributor

@anchpop anchpop commented Dec 20, 2024

I created this PR by following these instructions from @nathanosdev:

The following crates should use version 3.0.2:

  1. ic-response-verification
  2. ic-certification
  3. ic-certificate-verification
  4. ic-http-certification
  5. ic-cbor

I recommend just setting them to 3 in Cargo.toml and ^3.0.2 in external_crates.bzl so they can be updated more easily in the future. That's how it's setup in ic-http-gateway too.
And ic-http-gateway can be set to version 0.1.0. The git tag reference can be removed now.

I also updated ic-agent to 0.39 because that is the version of ic-agent that ic-http-gateway v0.1.0 depends on. In doing so, I also updated ic-utils and ic-transport-types. I also moved the ic-http-gateway dependency to the workspace-level Cargo.toml.

It seems like there were a number of breaking changes in these crates, so I also went through and fixed all the code that became broken.

@anchpop anchpop requested a review from a team as a code owner December 20, 2024 17:11
@github-actions github-actions bot added the chore label Dec 20, 2024
@anchpop anchpop force-pushed the @anchpop/update-crate-versions branch from 1b2ffd0 to 1289183 Compare December 20, 2024 17:26
@github-actions github-actions bot added the @idx label Dec 20, 2024
@anchpop anchpop force-pushed the @anchpop/update-crate-versions branch from c04cbfc to 7fe8cbc Compare December 20, 2024 17:36
@marko-k0 marko-k0 marked this pull request as draft December 20, 2024 18:33
@anchpop anchpop force-pushed the @anchpop/update-crate-versions branch 18 times, most recently from db85b04 to ec17eb9 Compare January 4, 2025 00:24
@anchpop anchpop force-pushed the @anchpop/update-crate-versions branch 6 times, most recently from 984b7af to 029c50c Compare January 4, 2025 01:16
@anchpop anchpop changed the title chore: Update crate versions chore: Update ic-agent to v0.39 (as well as some other crates that needed upgrading) Jan 4, 2025
@anchpop
Copy link
Contributor Author

anchpop commented Jan 4, 2025

@nikolay-komarevskiy : could you take a look at the failing test? I wasn't able to figure out what's breaking it :(

@nikolay-komarevskiy
Copy link
Contributor

@nikolay-komarevskiy : could you take a look at the failing test? I wasn't able to figure out what's breaking it :(

@anchpop Thanks a lot for doing a very valuable effort of updating this dependency!
It seems there is flakiness in this test, which needs to be understood. For now I disabled flaky steps in this system-test.

Copy link
Member

@mbjorkqvist mbjorkqvist left a comment

Choose a reason for hiding this comment

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

The Rosetta-related changes LGTM, thanks @anchpop !

@anchpop anchpop changed the title chore: Update ic-agent to v0.39 (as well as some other crates that needed upgrading) chore: Update ic-agent and ic-agent's reverse dependencies to v0.39 Jan 6, 2025
@anchpop anchpop force-pushed the @anchpop/update-crate-versions branch from db9ed03 to febda61 Compare January 6, 2025 18:02
@anchpop anchpop enabled auto-merge January 6, 2025 18:42
Copy link
Member

@DSharifi DSharifi left a comment

Choose a reason for hiding this comment

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

Thank you for doing this!

@anchpop anchpop added this pull request to the merge queue Jan 7, 2025
Merged via the queue into master with commit 0cd1ac6 Jan 7, 2025
25 checks passed
@anchpop anchpop deleted the @anchpop/update-crate-versions branch January 7, 2025 12:40
daniel-wong-dfinity-org pushed a commit that referenced this pull request Jan 7, 2025
…3282)

I created this PR by following these instructions from @nathanosdev:

> The following crates should use version 3.0.2:
> 1. `ic-response-verification`
> 2. `ic-certification`
> 3. `ic-certificate-verification`
> 4. `ic-http-certification`
> 5. `ic-cbor`
> 
> I recommend just setting them to `3` in `Cargo.toml` and `^3.0.2` in
`external_crates.bzl` so they can be updated more easily in the future.
That's how it's setup in `ic-http-gateway` too.
> And `ic-http-gateway` can be set to version `0.1.0`. The git tag
reference can be removed now.

I also updated `ic-agent` to 0.39 because that is the version of
`ic-agent` that `ic-http-gateway` v0.1.0 depends on. In doing so, I also
updated `ic-utils` and `ic-transport-types`. I also moved the
`ic-http-gateway` dependency to the workspace-level `Cargo.toml`.

It seems like there were a number of breaking changes in these crates,
so I also went through and fixed all the code that became broken.

---------

Co-authored-by: IDX GitLab Automation <idx@dfinity.org>
github-merge-queue bot pushed a commit that referenced this pull request Jan 7, 2025
Copy link
Contributor

@andrew-lee-work andrew-lee-work left a comment

Choose a reason for hiding this comment

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

LGTM. Left a comment about not checking the cert in Agent.call.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.