Skip to content

fix: add missing INITIALIZING system status

Description

qblox_instruments is still missing the INITIALIZING flag. It's in the SYS_ST_STATUS_STATE_LIST define in pulsar_src but not in the SystemStatuses enum in qblox_instruments

Leading to HIL failures since we query system state to know when the cluster booted up: https://gitlab.com/qblox/testing/qblox-automation/-/pipelines/2123925425

Here in the JTAG test: https://gitlab.com/qblox/testing/qblox-automation/-/jobs/11906114003#L315

Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Code refactoring (only refactoring of existing code, without new functionality)
  • Backport (cherry-picking fixes/features from development into a release branch)

Issue(s)

https://qblox.atlassian.net/browse/QINST-123

Related MRs

Testing

retesting again the JTAG test with this branch: https://gitlab.com/qblox/testing/qblox-automation/-/jobs/11906169412

Notes

Merge checklist

  • Cleaned up code (correct alignment/spacing/code style, no dead code, etc).
  • Merge request targets correct branch (usually development).
  • Added team label to MR (team::fe, team::ese or team::dd).
  • Added MR dependencies and link to feature MR at highest hierarchy level (usually product-level)
  • Selected assignee (you) and reviewers (team-members).
  • If merging a repository with a release package, updated appropriate CHANGELOG.md:
    • CLUSTER_MM, QBLOX_INSTRUMENTS: file in same repo
    • CLUSTER_QxM: file in CLUSTER_MM repo
  • Merge request has been reviewed by at least one team-member.
  • All pipelines pass, including manually triggered ones.
Edited by Yassine El Khadiri

Merge request reports

Loading