Skip to content

fix: llmqType must be equal to one of the allowed values#884

Merged
strophy merged 1 commit into
v0.24-devfrom
fix/dashmate-config-llmqtype
Apr 4, 2023
Merged

fix: llmqType must be equal to one of the allowed values#884
strophy merged 1 commit into
v0.24-devfrom
fix/dashmate-config-llmqtype

Conversation

@strophy
Copy link
Copy Markdown
Collaborator

@strophy strophy commented Apr 4, 2023

Issue being fixed or feature implemented

Dashmate integration on testnet was failing due to inability to select llmqType 6, which is required on testnet.

fatal: [hp-masternode-3]: FAILED! => changed=false 
  cmd:
  - yarn
  - dashmate
  - restart
  delta: '0:00:11.555743'
  end: '2023-04-04 13:05:12.249023'
  msg: non-zero return code
  rc: 1
  start: '2023-04-04 13:05:00.693280'
  stderr: |2-
        InvalidConfigFileFormatError: Invalid
        '/home/dashmate/.dashmate/config.json' config format:
        config/platform/drive/abci/validatorSet/llmqType must be equal to one of
        the allowed values
  stderr_lines: <omitted>
  stdout: ''
  stdout_lines: <omitted>

What was done?

Update enum llmqType

How Has This Been Tested?

In testing

Breaking Changes

None

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@strophy strophy added this to the v0.24.0 milestone Apr 4, 2023
@strophy strophy requested review from antouhou and shumkov as code owners April 4, 2023 13:14
Copy link
Copy Markdown
Collaborator

@shumkov shumkov left a comment

Choose a reason for hiding this comment

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

👍

@strophy strophy merged commit 5e5f5dd into v0.24-dev Apr 4, 2023
@strophy strophy deleted the fix/dashmate-config-llmqtype branch April 4, 2023 15:36
@markin-io markin-io mentioned this pull request Apr 26, 2023
5 tasks
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.

2 participants