[Shared] Refresh shared module params logic #852

bryanchriswhite · 2024-10-02T15:52:59Z

Summary

The "adding_params.md" doc has (or will have) changed substantially (see #839) since the shared module's param logic was last updated. This PR aligns the shared module's param logic with the snippets in the updated docs.

Specifically, this refresh includes:

Moving validation logic from msgServer#UpdateParam() to MsgUpdateParam#ValidateBasic().
Replacing magic strings of param names with their standard variable counterparts (i.e. sharedtypes.Param...).
Improving some local variable names.
Replacing usages of interface{} with any.

Dependencies

Dependents

Issue

[Utility] Minimum staking values for each actor #612

Type of change

Select one or more from the following:

New feature, functionality or library
Consensus breaking; add the consensus-breaking label if so. See [Infra] Automatically add the consensus-breaking label #791 for details
Bug fix
Code health or cleanup
Documentation
Other (specify)

Testing

Documentation: make docusaurus_start; only needed if you make doc changes
Unit Tests: make go_develop_and_test
LocalNet E2E Tests: make test_e2e
DevNet E2E Tests: Add the devnet-test-e2e label to the PR.

Sanity Checklist

I have tested my changes using the available tooling
I have commented my code
I have performed a self-review of my own code; both comments & source code
I create and reference any new tickets, if applicable
I have left TODOs throughout the codebase, if applicable

…me as_type --response params

… name as_type --response params

…resh

## Summary ```bash ignite scaffold message update-param --module supplier --signer authority name as_type --response params ``` Adds the `MsgUpdateParam` message so that the supplier module may update individual parameters following section 0 from the updated docs (see #839). ## Dependencies - #809 - #843 - #844 - #845 - #847 - #848 ## Dependents - #850 - #857 - #852 - #861 - #851 - #863 ## Issue - #612 ## Type of change Select one or more from the following: - [x] New feature, functionality or library - [ ] Consensus breaking; add the `consensus-breaking` label if so. See #791 for details - [ ] Bug fix - [ ] Code health or cleanup - [ ] Documentation - [ ] Other (specify) ## Testing - [ ] **Documentation**: `make docusaurus_start`; only needed if you make doc changes - [x] **Unit Tests**: `make go_develop_and_test` - [ ] **LocalNet E2E Tests**: `make test_e2e` - [ ] **DevNet E2E Tests**: Add the `devnet-test-e2e` label to the PR. ## Sanity Checklist - [x] I have tested my changes using the available tooling - [x] I have commented my code - [x] I have performed a self-review of my own code; both comments & source code - [ ] I create and reference any new tickets, if applicable - [x] I have left TODOs throughout the codebase, if applicable --------- Co-authored-by: Redouane Lakrache <r3d0ne@gmail.com> Co-authored-by: Daniel Olshansky <olshansky.daniel@gmail.com> Co-authored-by: red-0ne <red-0ne@users.noreply.github.com>

…-stake * pokt/HEAD: [Supplier] Add `MsgUpdateParam` to application module (#849) # Conflicts: # x/supplier/keeper/msg_server_update_param.go # x/supplier/types/message_update_param.go # x/supplier/types/message_update_param_test.go

…/staking

## Summary - Adds the `min_stake` param to the supplier module following section 0 from the updated docs (see #839). - Adds missing validation unit test coverage around existing application module param, max_delegated_gateways. - Pushes validation from the MsgUpdateParam handler down into MsgUpdateParam#ValidateBasic(). - Ensures Params#Valid() is called prior to setting params in MsgUpdateParam handler. - Updates error returns in the same handler to ensure all error paths return appropriate gRPC status errors. ## Dependencies - #809 - #843 - #844 - #845 - #847 - #848 - #849 ## Dependents - #857 - #852 - #861 - #851 - #863 ## Issue - #612 ## Type of change Select one or more from the following: - [x] New feature, functionality or library - [ ] Consensus breaking; add the `consensus-breaking` label if so. See #791 for details - [ ] Bug fix - [ ] Code health or cleanup - [ ] Documentation - [ ] Other (specify) ## Testing - [ ] **Documentation**: `make docusaurus_start`; only needed if you make doc changes - [ ] **Unit Tests**: `make go_develop_and_test` - [ ] **LocalNet E2E Tests**: `make test_e2e` - [ ] **DevNet E2E Tests**: Add the `devnet-test-e2e` label to the PR. ## Sanity Checklist - [x] I have tested my changes using the available tooling - [x] I have commented my code - [x] I have performed a self-review of my own code; both comments & source code - [ ] I create and reference any new tickets, if applicable - [x] I have left TODOs throughout the codebase, if applicable --------- Co-authored-by: Redouane Lakrache <r3d0ne@gmail.com> Co-authored-by: Daniel Olshansky <olshansky.daniel@gmail.com> Co-authored-by: red-0ne <red-0ne@users.noreply.github.com>

…king * pokt/main: [Supplier] Add `min_stake` supplier module param (#850)

…resh

## Summary 1. Adds minimum stake validation to the supplier stake message handler (i.e. total stake must be >= minimum stake). 2. Updates error returns in the same handler to ensure all error paths return appropriate gRPC status errors. ## Dependencies - #809 - #843 - #844 - #845 - #847 - #848 - #849 - #850 ## Dependents - #852 - #861 - #851 ## Issue - #612 ## Type of change Select one or more from the following: - [x] New feature, functionality or library - [ ] Consensus breaking; add the `consensus-breaking` label if so. See #791 for details - [ ] Bug fix - [ ] Code health or cleanup - [ ] Documentation - [ ] Other (specify) ## Testing - [ ] **Documentation**: `make docusaurus_start`; only needed if you make doc changes - [ ] **Unit Tests**: `make go_develop_and_test` - [ ] **LocalNet E2E Tests**: `make test_e2e` - [ ] **DevNet E2E Tests**: Add the `devnet-test-e2e` label to the PR. ## Sanity Checklist - [x] I have tested my changes using the available tooling - [x] I have commented my code - [x] I have performed a self-review of my own code; both comments & source code - [ ] I create and reference any new tickets, if applicable - [ ] I have left TODOs throughout the codebase, if applicable --------- Co-authored-by: Redouane Lakrache <r3d0ne@gmail.com> Co-authored-by: Daniel Olshansky <olshansky.daniel@gmail.com> Co-authored-by: red-0ne <red-0ne@users.noreply.github.com>

* pokt/main: [Supplier] Enforce minimum stake when staking (#857)

github-actions · 2024-10-14T14:28:33Z

The CI will now also run the e2e tests on devnet, which increases the time it takes to complete all CI checks.

You may need to run make trigger_ci to submit an empty commit that'll trigger the tests.

GCP workloads (requires changing the namespace to 852)
Grafana network dashboard for devnet-issue-852

bryanchriswhite · 2024-10-14T15:57:35Z

E2E tests passing locally but stalled in CI:

* pokt/main: [Shared] Refresh shared module params logic (#852)

## Summary The "adding_params.md" doc has (or will have) changed substantially (see #839) since the proof module's param logic was last updated. This PR aligns the proof module's param logic with the snippets in the updated docs. Specifically, this refresh includes: - Moving validation logic from `msgServer#UpdateParam()` to `MsgUpdateParam#ValidateBasic()`. - Replacing magic strings of param names with their standard variable counterparts (i.e. `prooftypes.Param...`). - Improving some local variable names. - Replacing usages of `interface{}` with `any`. - Improving validation for all coin type params to be consistent with application, gateway, and supplier coin type validation. # Dependencies - #809 - #843 - #844 - #845 - #847 - #848 - #849 - #850 - #857 - #852 ## Dependents - #861 ## Issue - #612 ## Type of change Select one or more from the following: - [ ] New feature, functionality or library - [ ] Consensus breaking; add the `consensus-breaking` label if so. See #791 for details - [ ] Bug fix - [x] Code health or cleanup - [ ] Documentation - [ ] Other (specify) ## Testing - [ ] **Documentation**: `make docusaurus_start`; only needed if you make doc changes - [ ] **Unit Tests**: `make go_develop_and_test` - [ ] **LocalNet E2E Tests**: `make test_e2e` - [ ] **DevNet E2E Tests**: Add the `devnet-test-e2e` label to the PR. ## Sanity Checklist - [x] I have tested my changes using the available tooling - [ ] I have commented my code - [x] I have performed a self-review of my own code; both comments & source code - [ ] I create and reference any new tickets, if applicable - [ ] I have left TODOs throughout the codebase, if applicable --------- Co-authored-by: Redouane Lakrache <r3d0ne@gmail.com> Co-authored-by: Daniel Olshansky <olshansky.daniel@gmail.com> Co-authored-by: red-0ne <red-0ne@users.noreply.github.com>

## Summary The "adding_params.md" doc has (or will have) changed substantially (see #839) since the shared module's param logic was last updated. This PR aligns the service module's param logic with the snippets in the updated docs. Specifically, this refresh includes: - Moving validation logic from `msgServer#UpdateParam()` to `MsgUpdateParam#ValidateBasic()`. - Replacing magic strings of param names with their standard variable counterparts (i.e. `prooftypes.Param...`). - Improving some local variable names. - Replacing usages of `interface{}` with `any`. - Improving validation for all coin type params to be consistent with application, gateway, and supplier coin type validation. ## Dependencies - #809 - #843 - #844 - #845 - #847 - #848 - #849 - #850 - #857 - #852 - #851 ## Issue - #612 ## Type of change Select one or more from the following: - [ ] New feature, functionality or library - [ ] Consensus breaking; add the `consensus-breaking` label if so. See #791 for details - [ ] Bug fix - [x] Code health or cleanup - [ ] Documentation - [ ] Other (specify) ## Testing - [ ] **Documentation**: `make docusaurus_start`; only needed if you make doc changes - [ ] **Unit Tests**: `make go_develop_and_test` - [ ] **LocalNet E2E Tests**: `make test_e2e` - [ ] **DevNet E2E Tests**: Add the `devnet-test-e2e` label to the PR. ## Sanity Checklist - [ ] I have tested my changes using the available tooling - [ ] I have commented my code - [ ] I have performed a self-review of my own code; both comments & source code - [ ] I create and reference any new tickets, if applicable - [ ] I have left TODOs throughout the codebase, if applicable --------- Co-authored-by: Redouane Lakrache <r3d0ne@gmail.com> Co-authored-by: Daniel Olshansky <olshansky.daniel@gmail.com> Co-authored-by: red-0ne <red-0ne@users.noreply.github.com>

…ed-params * pokt/main: [On-Chain] Refactor `uint64` type individual param update logic (#863) [Service] Refresh service module params logic (#861) [Proof] Refresh proof module params logic (#851) [Shared] Refresh shared module params logic (#852) [Docs] Fix example of how to set rev share precentage (#877) [Tokenomics] Implement difficulty proportional rewards (#880) [Supplier] Enforce minimum stake when staking (#857) [Supplier] Add `min_stake` supplier module param (#850) [Supplier] Add `MsgUpdateParam` to application module (#849) [Application] Enforce minimum stake when burning (#848) [DifficultyHash] Prepare for difficulty multiplier usage (#836) [Application] Enforce minimum stake when staking (#847) [Tokenomics] Prevent GMR to produce zero values (#866)

* pokt/main: [On-Chain] Refactor `uint64` type individual param update logic (#863) [Service] Refresh service module params logic (#861) [Proof] Refresh proof module params logic (#851) [Shared] Refresh shared module params logic (#852) [Docs] Fix example of how to set rev share precentage (#877) [Tokenomics] Implement difficulty proportional rewards (#880) [Supplier] Enforce minimum stake when staking (#857) [Supplier] Add `min_stake` supplier module param (#850) [Supplier] Add `MsgUpdateParam` to application module (#849) [Application] Enforce minimum stake when burning (#848) [DifficultyHash] Prepare for difficulty multiplier usage (#836) [Application] Enforce minimum stake when staking (#847) [Tokenomics] Prevent GMR to produce zero values (#866)

…/application * refactor/testutil/filter-events: chore: rename testevents import refactor: simplify testevents.FilterEvents() [Code Health] refactor: share helpers to consolidate avoid dependency cycles (#855) [Tooling, Docs] Refactor params docs generation command & update params docs (#864) [On-Chain] Refactor `uint64` type individual param update logic (#863) [Service] Refresh service module params logic (#861) [Proof] Refresh proof module params logic (#851) [Shared] Refresh shared module params logic (#852)

bryanchriswhite added 30 commits September 26, 2024 08:56

scaffold: message update-param --module gateway --signer authority na…

d1cd5e6

…me as_type --response params

chore: update gateway MsgUpdateParam fields

91eecfc

chore: update MsgUpdateParamResponse fields

ef62280

chore: comment out autocli for gateway module param updates

f7a6e5b

fix: linter error

8957f1c

revert: premature changes

3016ca3

feat: add min_stake gateway module param

899effb

chore: cleanup makefiles and param update JSON files

83c1d74

chore: add gateway module param make targets

55e6a31

refctor: consolidate param config types

ddd68bb

tests: improve error messaging

fa8caf6

tests: add coverage over gateway min stake param updates

ae006e2

chore: add as_coin as_type type 😉

e92e3b1

chore: add app MsgUpdateParam to genesis authorizations

b2e1267

fix: typo

0373fa2

chore: cleanup makefiles

da7894a

wip: fixing tests

015c537

tests: add tests

1ac6c48

tests: remove erroneous case

db366f6

test: gateway staking below minimum fails

60bef17

feat: gateway min stake validation & grpc status error returns

a21687f

chore: cleanup comments

7e9f857

scaffold: message update-param --module examplemod --signer authority…

4140710

… name as_type --response params

chore: update MsgUpdateParam fields

0a8d922

chore: comment out application MsgUpdateParam autocli

c88edd6

chore: add application MsgUpdateParam to dao genesis authorizations

da51239

chore: update NewMsgUpdateParam constructor

45c5a99

fix: simulations generated code

d987e09

chore: add min_stake param to application module params

0da255d

chore: update params config for param updates test suite

4542628

Merge branch 'issues/612/supplier/staking' into issues/612/shared/ref…

fcaef74

…resh

bryanchriswhite added 2 commits October 14, 2024 12:51

Merge branch 'issues/612/supplier/min-stake' into issues/612/supplier…

413a084

…/staking

bryanchriswhite added 2 commits October 14, 2024 13:40

Merge remote-tracking branch 'pokt/main' into issues/612/supplier/sta…

67f28f6

…king * pokt/main: [Supplier] Add `min_stake` supplier module param (#850)

Merge branch 'issues/612/supplier/staking' into issues/612/shared/ref…

c7aa318

…resh

Merge remote-tracking branch 'pokt/main' into issues/612/shared/refresh

6e2b4db

* pokt/main: [Supplier] Enforce minimum stake when staking (#857)

bryanchriswhite changed the base branch from issues/612/supplier/staking to main October 14, 2024 12:02

bryanchriswhite added 3 commits October 14, 2024 14:18

Empty commit

d85daf7

Merge branch 'main' into issues/612/shared/refresh

3fa5533

Merge branch 'main' into issues/612/shared/refresh

9e06b8e

bryanchriswhite added devnet-test-e2e and removed push-image CI related - pushes images to ghcr.io devnet devnet-test-e2e labels Oct 14, 2024

github-actions bot added devnet push-image CI related - pushes images to ghcr.io labels Oct 14, 2024

Empty commit

972b8df

bryanchriswhite merged commit 2ac298a into main Oct 14, 2024
9 of 10 checks passed

bryanchriswhite added a commit that referenced this pull request Oct 14, 2024

Merge remote-tracking branch 'pokt/main' into issues/612/proof/refresh

3673665

* pokt/main: [Shared] Refresh shared module params logic (#852)

bryanchriswhite deleted the issues/612/shared/refresh branch October 14, 2024 17:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Shared] Refresh shared module params logic #852

[Shared] Refresh shared module params logic #852

bryanchriswhite commented Oct 2, 2024 •

edited

Loading

github-actions bot commented Oct 14, 2024

bryanchriswhite commented Oct 14, 2024

[Shared] Refresh shared module params logic #852

[Shared] Refresh shared module params logic #852

Conversation

bryanchriswhite commented Oct 2, 2024 • edited Loading

Summary

Dependencies

Dependents

Issue

Type of change

Testing

Sanity Checklist

github-actions bot commented Oct 14, 2024

bryanchriswhite commented Oct 14, 2024

bryanchriswhite commented Oct 2, 2024 •

edited

Loading