tree 1ec58d70b0105a43ee1e2e9ae36cfd52418ddbfb
parent 017a9db0d63d2fe2fa3d7140757844e665f76d35
author Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com> 1703100367 -0500
committer GitHub <noreply@github.com> 1703100367 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJlgz/PCRBK7hj4Ov3rIwAABeUIAEt8roBW5GHXTh7caEopEAWH
 fObgvEM31MCYPRaerd2K0/9vZUHXQH6GofSt05dn9lCBrU9R6OJFt8mI/kns3I4Z
 UROVuB0S7Zp0Jo7E9AosEIT8e3REW/cYNTHQr/L9NpEN+XPxpbfdPnSeJPXiycgW
 aXS2xTKSu5ekxQFhKCKbtDkH1sc4XtJrFC4ex7Ev3zwQvdB9rAGLl1SXXo4X/uFs
 9ymInXTdDtVBTJzcOJUzb5v3YnLiSf2yqj8Co+j+Q84apbnkupMe9XCPM2eVYtup
 HVLyLyvGmwTZrtwbFZsDpIPHaTy+Fad0hK3z2p/fTTBiX09KUxHyGmSbD2ogn2s=
 =mscE
 -----END PGP SIGNATURE-----
 

Introduce a Status + Cluster Code abstraction (#31121)

* Introduce a Status + Cluster Code abstraction

- Existing code always has to split IM status and
  cluster-specific status codes, which causes clumsy
  implementation of clusters (see issue #31120).
- This PR introduces a value that encapsulates both
  the IM status and cluster-specific status, in an
  easy-to-pass-around abstraction, which can be directly
  used in place of Status.
- Subsequent PR will implement usage in IM with overloads
  for common cases, such as for AddStatus.

Issue #31120

Testing done:
- Added unit tests
- Other tests still pass

* Restyled by clang-format

* Restyled by gn

* Address review comments

* Rename ClusterStatus to ClusterStatusCode

* Fix build deps

* Restyled by gn

---------

Co-authored-by: Restyled.io <commits@restyled.io>