Skip to content

Chore(Charts): seq-faucet bech32m chart update#1301

Merged
quasystaty1 merged 3 commits intomainfrom
quasystaty1/seq-faucet-bech32
Aug 23, 2024
Merged

Chore(Charts): seq-faucet bech32m chart update#1301
quasystaty1 merged 3 commits intomainfrom
quasystaty1/seq-faucet-bech32

Conversation

@quasystaty1
Copy link
Copy Markdown
Contributor

Summary

Updates sequencer-faucet chart to support the changes made to the sequencer faucet image for bech32m compatibility.

Background

With the introduction of bech32m prefixed addresses, an update to the sequencer faucet was required. This PR updates the sequencer-faucet helm chart to use the new faucet.

Changes

  • ADDED SEQUENCER_FAUCET_MINUTE - rate limit on requests, default 1 day
  • ADDED SEQUENCER_ADDRESS_PREFIX - sequencer prefix address, default astria
  • ADDED SEQUENCER_NATIVE_DENOM - asset to transfer upon request, default nria

Testing

Tested locally against the latest kubernetes cluster

@quasystaty1 quasystaty1 requested a review from a team as a code owner July 26, 2024 08:37
@github-actions github-actions bot added the cd label Jul 26, 2024
@quasystaty1 quasystaty1 force-pushed the quasystaty1/seq-faucet-bech32 branch from ca46361 to fc1f6ca Compare August 23, 2024 14:52
@quasystaty1 quasystaty1 added this pull request to the merge queue Aug 23, 2024
Merged via the queue into main with commit 7339da2 Aug 23, 2024
@quasystaty1 quasystaty1 deleted the quasystaty1/seq-faucet-bech32 branch August 23, 2024 15:35
steezeburger added a commit that referenced this pull request Aug 26, 2024
* main:
  fix(test): use correct conditions to dermine smoke test success (#1395)
  fix(test): fail smoke test scripts eagerly if a command fails (#1394)
  fix(cli, tests): add force flag to overwrite withdrawal target path (#1393)
  release(charts): update with biweekly image cuts (#1399)
  Chore(Charts): seq-faucet bech32m chart update (#1301)
  chore: preview environment with astria-geth changes (#1401)
  release: biweekly release cut (#1398)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants