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

feat(subnet): Update resolver to new contracts [NET-619] #1891

Merged
merged 12 commits into from
Nov 15, 2023

Conversation

kmd-fl
Copy link
Contributor

@kmd-fl kmd-fl commented Nov 9, 2023

Description

Tested the nox and decider with new deal lifecycle update.

On deploy to stage/testnet/kras, need to set new addresses in the configs.

Proposed Changes

The format of the call was changed. Now the function named getComputeUnits and the result doesn't have the index field (which was ignored anyway)

Additional Notes

What I did:

  • Built fcli from the PR above. Started local network.
  • Started local nox with the updated matcher contract address.
  • Registered local nox.
  • Deployed a deal from the quickstart.
  • Re-deployed to update
  • Looked at logs

Results:

  • decider found and installed the deal
  • decider couldn't find the update since it didn't move the queried window (created a task in linear). After manually changing the block number in KV, decider found and installed the update
  • deal logs didn't work; needed to change subnet.resolve. After the change, it worked.

crates/subnet-resolver/src/resolve.rs Outdated Show resolved Hide resolved
kmd-fl and others added 2 commits November 13, 2023 10:53
Co-authored-by: Aleksey Proshutinskiy <justprosh@users.noreply.github.com>
@kmd-fl kmd-fl added the e2e Run e2e workflow label Nov 13, 2023
@kmd-fl kmd-fl enabled auto-merge (squash) November 13, 2023 09:56
.github/workflows/e2e.yml Outdated Show resolved Hide resolved
@kmd-fl kmd-fl merged commit 0de0556 into master Nov 15, 2023
17 checks passed
@kmd-fl kmd-fl deleted the update-to-new-deals branch November 15, 2023 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e Run e2e workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants