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: allowed ccv,globalfee stargate queries #NTRN-408 #765

Merged
merged 2 commits into from
Nov 27, 2024

Conversation

swelf19
Copy link
Contributor

@swelf19 swelf19 commented Nov 15, 2024

PR includes:

  1. Adding new Stargate requests to the whitelist.
  2. Correcting typos in existing requests.
  3. Changing the Hermes configuration. This is done to align the local Hermes configuration with the version that has long been used in integration tests (https://github.com/neutron-org/neutron-integration-tests/blob/main/setup/hermes/create-conn.sh#L13). This configuration is used only when manually launching chains using the commands 'make init && make start-rly'.

Task definition: Add updateParamsPersimmon to chain manager (https://github.com/neutron-org/neutron-dao/tree/main/contracts/dao/neutron-chain-manager) for the following modules:

  • consumer
  • dynamicfees
  • globalfee

All prs:
neutron-org/neutronjs#10
neutron-org/neutronjsplus#69
neutron-org/neutron-integration-tests#368
neutron-org/neutron-dao#116
neutron-org/neutron-std#8
#765



# a short living client is needed for client update proposal test
echo "Initiating short living client..."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it possible to move it into the integration tests?
Cause this seems unnecessary in Neutron itself

Copy link
Contributor

Choose a reason for hiding this comment

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

💯

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We have two sets of configurations for Hermes:

  1. For use with the cosmospark in integration tests: https://github.com/neutron-org/neutron-integration-tests/blob/main/setup/hermes/create-conn.sh#L13
  2. For local relay run using the commands 'make init && make start-rly'.

I standardized them, as the command has been present in the hermes (docker) configuration for quite some time.

joldie777
joldie777 previously approved these changes Nov 21, 2024


# a short living client is needed for client update proposal test
echo "Initiating short living client..."
Copy link
Contributor

Choose a reason for hiding this comment

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

💯

wasmbinding/stargate_allowlist.go Show resolved Hide resolved
NeverHappened
NeverHappened previously approved these changes Nov 26, 2024
@swelf19 swelf19 dismissed stale reviews from NeverHappened and joldie777 via 66a8aa5 November 26, 2024 18:33
@pr0n00gler pr0n00gler merged commit c74f625 into main Nov 27, 2024
12 checks passed
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.

5 participants