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!: completed throttle v2 (provider changes + migration + testing) #1321

Merged
merged 21 commits into from
Oct 6, 2023

Commits on Sep 6, 2023

  1. merge main into throttle feat branch (#1267)

    * build(deps): bump actions/checkout from 3 to 4 (#1257)
    
    Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
    - [Release notes](https://github.com/actions/checkout/releases)
    - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
    - [Commits](actions/checkout@v3...v4)
    
    ---
    updated-dependencies:
    - dependency-name: actions/checkout
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    
    * build(deps)!: bump github.com/cosmos/ibc-go/v7 from 7.2.0 to 7.3.0 (#1258)
    
    * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.2.0 to 7.3.0
    
    Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.2.0 to 7.3.0.
    - [Release notes](https://github.com/cosmos/ibc-go/releases)
    - [Changelog](https://github.com/cosmos/ibc-go/blob/main/CHANGELOG.md)
    - [Commits](cosmos/ibc-go@v7.2.0...v7.3.0)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/cosmos/ibc-go/v7
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    
    * add changelog entries
    
    ---------
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: mpoke <marius.poke@posteo.de>
    
    * build(deps): bump github.com/cosmos/cosmos-sdk from 0.47.4 to 0.47.5 (#1259)
    
    * build(deps): bump github.com/cosmos/cosmos-sdk from 0.47.3 to 0.47.5
    
    Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.47.3 to 0.47.5.
    - [Release notes](https://github.com/cosmos/cosmos-sdk/releases)
    - [Changelog](https://github.com/cosmos/cosmos-sdk/blob/main/CHANGELOG.md)
    - [Commits](cosmos/cosmos-sdk@v0.47.3...v0.47.5)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/cosmos/cosmos-sdk
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    
    * add changelog entries
    
    ---------
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: mpoke <marius.poke@posteo.de>
    
    * chore: Separate semver (#1217)
    
    separate semver
    
    * docs: cleanup changelog (#1260)
    
    fix changelog
    
    * fix!: validate MsgTransfer before calling Transfer() (#1244)
    
    * validate MsgTransfer
    
    * add TestSendRewardsToProvider
    
    * update DefaultConsumerUnbondingPeriod to 14 days
    
    * update changelog
    
    * fix linter
    
    * fix test
    
    * apply review suggestions
    
    * update changelog
    
    * docs: Create adr-012-separate-releasing.md (#1229)
    
    * Create adr-011-separate-releasing.md
    
    * Update docs/docs/adrs/adr-011-separate-releasing.md
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    
    * adr 12 not 11
    
    * correct that we use postfix not prefix
    
    * explanation on example release flow
    
    ---------
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    
    * fix: remove addr validation for provider fee pool addr param (#1262)
    
    * fix: remove validation for provider chain address since we cannot validate it properly on consumer
    
    * add changelog entry
    
    ---------
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: mpoke <marius.poke@posteo.de>
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    Co-authored-by: Dmitry Kolupaev <dmitry.klpv@gmail.com>
    5 people committed Sep 6, 2023
    Configuration menu
    Copy the full SHA
    0b1d429 View commit details
    Browse the repository at this point in the history

Commits on Sep 14, 2023

  1. feat!: throttle with retries provider changes (#1230)

    * wip, tests not fixed yet
    
    * rm packet query on provider
    
    * rm unneeded UTs
    
    * rm tests from relay_test
    
    * rm query and more tests
    
    * rm more tests
    
    * builds again and rm debug tests
    
    * lint
    
    * fix handling of slash packet and integration test
    
    * Fix TestMultiConsumerSlashPacketThrottling
    
    * fix two more slashing integration tests
    
    * Update TestSlashRetries, cleanup neededc
    
    * cleaned up TestSlashRetries
    
    * UT for TestOnRecvDowntimeSlashPacket
    
    * cleans
    
    * use helper in throttle test
    
    * lintz
    
    * Revert "rm packet query on provider"
    
    This reverts commit a10a239.
    
    * cmd file too
    
    * fully restore query
    
    * Revert "Merge branch 'main' into shawn/throttle-with-retries-provider-changes"
    
    This reverts commit 73db33b, reversing
    changes made to 78a8269.
    
    * make e2e test pass, with todos
    
    * clean
    
    * Revert "Revert "Merge branch 'main' into shawn/throttle-with-retries-provider-changes""
    
    This reverts commit 5bfccc3.
    
    * lint
    
    * Update CHANGELOG.md
    
    * slightly longer buffer
    
    * build(deps): bump actions/checkout from 3 to 4 (#1257)
    
    Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
    - [Release notes](https://github.com/actions/checkout/releases)
    - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
    - [Commits](actions/checkout@v3...v4)
    
    ---
    updated-dependencies:
    - dependency-name: actions/checkout
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    
    * build(deps)!: bump github.com/cosmos/ibc-go/v7 from 7.2.0 to 7.3.0 (#1258)
    
    * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.2.0 to 7.3.0
    
    Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.2.0 to 7.3.0.
    - [Release notes](https://github.com/cosmos/ibc-go/releases)
    - [Changelog](https://github.com/cosmos/ibc-go/blob/main/CHANGELOG.md)
    - [Commits](cosmos/ibc-go@v7.2.0...v7.3.0)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/cosmos/ibc-go/v7
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    
    * add changelog entries
    
    ---------
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: mpoke <marius.poke@posteo.de>
    
    * build(deps): bump github.com/cosmos/cosmos-sdk from 0.47.4 to 0.47.5 (#1259)
    
    * build(deps): bump github.com/cosmos/cosmos-sdk from 0.47.3 to 0.47.5
    
    Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.47.3 to 0.47.5.
    - [Release notes](https://github.com/cosmos/cosmos-sdk/releases)
    - [Changelog](https://github.com/cosmos/cosmos-sdk/blob/main/CHANGELOG.md)
    - [Commits](cosmos/cosmos-sdk@v0.47.3...v0.47.5)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/cosmos/cosmos-sdk
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    
    * add changelog entries
    
    ---------
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: mpoke <marius.poke@posteo.de>
    
    * chore: Separate semver (#1217)
    
    separate semver
    
    * docs: cleanup changelog (#1260)
    
    fix changelog
    
    * fix!: validate MsgTransfer before calling Transfer() (#1244)
    
    * validate MsgTransfer
    
    * add TestSendRewardsToProvider
    
    * update DefaultConsumerUnbondingPeriod to 14 days
    
    * update changelog
    
    * fix linter
    
    * fix test
    
    * apply review suggestions
    
    * update changelog
    
    * docs: Create adr-012-separate-releasing.md (#1229)
    
    * Create adr-011-separate-releasing.md
    
    * Update docs/docs/adrs/adr-011-separate-releasing.md
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    
    * adr 12 not 11
    
    * correct that we use postfix not prefix
    
    * explanation on example release flow
    
    ---------
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    
    * fix: remove addr validation for provider fee pool addr param (#1262)
    
    * fix: remove validation for provider chain address since we cannot validate it properly on consumer
    
    * add changelog entry
    
    * Revert "Merge branch 'main' into shawn/throttle-with-retries-provider-changes"
    
    This reverts commit 6bdfff9, reversing
    changes made to d8f5690.
    
    * fmt
    
    * Update steps_downtime.go
    
    * Update tests/e2e/steps_downtime.go
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    
    * waittime instead of sleep
    
    * Update x/ccv/provider/client/cli/query.go
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    
    * Update x/ccv/provider/client/cli/query.go
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    
    * rm throttled packets from query
    
    * rm provider query
    
    * whoopsies forgot to rm some boilerplate
    
    ---------
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: mpoke <marius.poke@posteo.de>
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    Co-authored-by: Dmitry Kolupaev <dmitry.klpv@gmail.com>
    5 people committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    727e731 View commit details
    Browse the repository at this point in the history

Commits on Sep 18, 2023

  1. feat: consumer throttle state query (#1285)

    * wip, tests not fixed yet
    
    * rm packet query on provider
    
    * rm unneeded UTs
    
    * rm tests from relay_test
    
    * rm query and more tests
    
    * rm more tests
    
    * builds again and rm debug tests
    
    * lint
    
    * fix handling of slash packet and integration test
    
    * Fix TestMultiConsumerSlashPacketThrottling
    
    * fix two more slashing integration tests
    
    * Update TestSlashRetries, cleanup neededc
    
    * cleaned up TestSlashRetries
    
    * UT for TestOnRecvDowntimeSlashPacket
    
    * cleans
    
    * use helper in throttle test
    
    * lintz
    
    * Revert "rm packet query on provider"
    
    This reverts commit a10a239.
    
    * cmd file too
    
    * fully restore query
    
    * Revert "Merge branch 'main' into shawn/throttle-with-retries-provider-changes"
    
    This reverts commit 73db33b, reversing
    changes made to 78a8269.
    
    * make e2e test pass, with todos
    
    * clean
    
    * Revert "Revert "Merge branch 'main' into shawn/throttle-with-retries-provider-changes""
    
    This reverts commit 5bfccc3.
    
    * lint
    
    * Update CHANGELOG.md
    
    * slightly longer buffer
    
    * build(deps): bump actions/checkout from 3 to 4 (#1257)
    
    Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
    - [Release notes](https://github.com/actions/checkout/releases)
    - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
    - [Commits](actions/checkout@v3...v4)
    
    ---
    updated-dependencies:
    - dependency-name: actions/checkout
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    
    * build(deps)!: bump github.com/cosmos/ibc-go/v7 from 7.2.0 to 7.3.0 (#1258)
    
    * build(deps): bump github.com/cosmos/ibc-go/v7 from 7.2.0 to 7.3.0
    
    Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.2.0 to 7.3.0.
    - [Release notes](https://github.com/cosmos/ibc-go/releases)
    - [Changelog](https://github.com/cosmos/ibc-go/blob/main/CHANGELOG.md)
    - [Commits](cosmos/ibc-go@v7.2.0...v7.3.0)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/cosmos/ibc-go/v7
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    
    * add changelog entries
    
    ---------
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: mpoke <marius.poke@posteo.de>
    
    * build(deps): bump github.com/cosmos/cosmos-sdk from 0.47.4 to 0.47.5 (#1259)
    
    * build(deps): bump github.com/cosmos/cosmos-sdk from 0.47.3 to 0.47.5
    
    Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.47.3 to 0.47.5.
    - [Release notes](https://github.com/cosmos/cosmos-sdk/releases)
    - [Changelog](https://github.com/cosmos/cosmos-sdk/blob/main/CHANGELOG.md)
    - [Commits](cosmos/cosmos-sdk@v0.47.3...v0.47.5)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/cosmos/cosmos-sdk
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    
    * add changelog entries
    
    ---------
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: mpoke <marius.poke@posteo.de>
    
    * chore: Separate semver (#1217)
    
    separate semver
    
    * docs: cleanup changelog (#1260)
    
    fix changelog
    
    * fix!: validate MsgTransfer before calling Transfer() (#1244)
    
    * validate MsgTransfer
    
    * add TestSendRewardsToProvider
    
    * update DefaultConsumerUnbondingPeriod to 14 days
    
    * update changelog
    
    * fix linter
    
    * fix test
    
    * apply review suggestions
    
    * update changelog
    
    * docs: Create adr-012-separate-releasing.md (#1229)
    
    * Create adr-011-separate-releasing.md
    
    * Update docs/docs/adrs/adr-011-separate-releasing.md
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    
    * adr 12 not 11
    
    * correct that we use postfix not prefix
    
    * explanation on example release flow
    
    ---------
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    
    * fix: remove addr validation for provider fee pool addr param (#1262)
    
    * fix: remove validation for provider chain address since we cannot validate it properly on consumer
    
    * add changelog entry
    
    * Revert "Merge branch 'main' into shawn/throttle-with-retries-provider-changes"
    
    This reverts commit 6bdfff9, reversing
    changes made to d8f5690.
    
    * fmt
    
    * protobuf gen
    
    * impl query resp
    
    * add query assertion to e2e tests
    
    * lint
    
    * add consumer to pending packets name
    
    * add panic for invalid bytes
    
    ---------
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    Co-authored-by: mpoke <marius.poke@posteo.de>
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    Co-authored-by: Dmitry Kolupaev <dmitry.klpv@gmail.com>
    5 people committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    f060f48 View commit details
    Browse the repository at this point in the history
  2. feat: retry delay period as param (#1308)

    * boilerplate
    
    * fix test
    
    * Update proposal_test.go
    
    * apply new param to keeper code
    
    * Update params_test.go
    shaspitz committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    c1c3fdb View commit details
    Browse the repository at this point in the history

Commits on Sep 19, 2023

  1. tests: throttle retry e2e tests (#1312)

    * boilerplate
    
    * fix test
    
    * Update proposal_test.go
    
    * apply new param to keeper code
    
    * wip
    
    * add action
    
    * incorporate queue assertions. Also make retry delay period 200s
    
    * correct queue size assertion
    
    * make wait time just 30 seconds
    
    * change naming
    shaspitz committed Sep 19, 2023
    Configuration menu
    Copy the full SHA
    a0b1e5e View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2023

  1. feat!: throttle v2 provider migration (#1317)

    * migration.go and incr consensus version
    
    * throttle v1 code for migration
    
    * old methods
    
    * finish test
    
    * register those migs
    
    * Update migration_test.go
    
    * lint
    
    * consumer keeper -> provider keeper
    shaspitz committed Sep 20, 2023
    Configuration menu
    Copy the full SHA
    ead21ec View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    2318279 View commit details
    Browse the repository at this point in the history
  3. some fixes

    shaspitz committed Sep 20, 2023
    Configuration menu
    Copy the full SHA
    01b001f View commit details
    Browse the repository at this point in the history
  4. fix the rest o the shit

    shaspitz committed Sep 20, 2023
    Configuration menu
    Copy the full SHA
    cb21ebd View commit details
    Browse the repository at this point in the history
  5. lint

    shaspitz committed Sep 20, 2023
    Configuration menu
    Copy the full SHA
    a3a3ccd View commit details
    Browse the repository at this point in the history

Commits on Sep 22, 2023

  1. Addressing simon's comments

    shaspitz committed Sep 22, 2023
    Configuration menu
    Copy the full SHA
    ca39b1d View commit details
    Browse the repository at this point in the history

Commits on Sep 26, 2023

  1. whoopies

    shaspitz committed Sep 26, 2023
    Configuration menu
    Copy the full SHA
    f0ffb47 View commit details
    Browse the repository at this point in the history
  2. height assertion

    shaspitz committed Sep 26, 2023
    Configuration menu
    Copy the full SHA
    3cb1f47 View commit details
    Browse the repository at this point in the history
  3. Update x/ccv/provider/keeper/migration.go

    Co-authored-by: Simon Noetzlin <simon.ntz@gmail.com>
    shaspitz and sainoe committed Sep 26, 2023
    Configuration menu
    Copy the full SHA
    a1e1b63 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    5522b7c View commit details
    Browse the repository at this point in the history
  5. action boilerplate

    shaspitz committed Sep 26, 2023
    Configuration menu
    Copy the full SHA
    0d0709c View commit details
    Browse the repository at this point in the history

Commits on Sep 27, 2023

  1. feat: remove max throttled packets param (#1332)

    * rm param
    
    * Update proto/interchain_security/ccv/provider/v1/provider.proto
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    
    ---------
    
    Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
    shaspitz and p-offtermatt committed Sep 27, 2023
    Configuration menu
    Copy the full SHA
    42c8f0e View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    8bd15ff View commit details
    Browse the repository at this point in the history
  3. merge fixes

    shaspitz committed Sep 27, 2023
    Configuration menu
    Copy the full SHA
    656d837 View commit details
    Browse the repository at this point in the history
  4. update e2e jsons

    shaspitz committed Sep 27, 2023
    Configuration menu
    Copy the full SHA
    48e7591 View commit details
    Browse the repository at this point in the history

Commits on Oct 6, 2023

  1. merge

    shaspitz committed Oct 6, 2023
    Configuration menu
    Copy the full SHA
    28212b5 View commit details
    Browse the repository at this point in the history