Skip to content

Refactor away STATUS_MAP

Thong Kuah requested to merge fix_spelling_rollout_status_status_map into master

What does this MR do?

Refactor away STATUS_MAP - It's only used internally and is really a copy of the constants in Gitlab::Kubernetes::Pod

Most of STATUS_MAP seems to be un-used, except for STATUS_MAP[:running]. There was also a typo in RolloutStatus.STATUS_MAP[:unkonw]

Does this MR meet the acceptance criteria?

Conformity

Performance and testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by 🤖 GitLab Bot 🤖

Merge request reports