List (KsqlAdmin) role bindings for user not showing the Logical ksql clusterId

[Panagiotis (Panos) Garefalakis (pgaref)]

Checklist

1. [CRUCIAL] Is the change for CP or CCloud functionalities that are already live in prod?
   • yes: ok

What

https://confluentinc.atlassian.net/browse/KSQL-9550
List role bindings for user not showing the Logical ksql cluster id to differentiate.
Added an extra List call to get pksqlid, output changed from:

    Principal   |                  Email                   |       Role        | Environment | Cloud Cluster | Cluster Type | Logical Cluster | Resource Type | Name | Pattern Type  
----------------+------------------------------------------+-------------------+-------------+---------------+--------------+-----------------+---------------+------+---------------
  User:u-y9gdx6 | pgarefalakis+ksql+test_stag@confluent.io | KsqlAdmin         | env-nx6qz6  | lkc-8gookr    |              |                 |               |      |               
  User:u-y9gdx6 | pgarefalakis+ksql+test_stag@confluent.io | KsqlAdmin         | env-nx6qz6  | lkc-kjzdov    |              |                 |               |      |               
  User:u-y9gdx6 | pgarefalakis+ksql+test_stag@confluent.io | OrganizationAdmin |             |               |              |                 |               |      |   

to:

    Principal   |                  Email                   |       Role        | Environment | Cloud Cluster | Cluster Type | Logical Cluster | Resource Type |          Name          | Pattern Type  
----------------+------------------------------------------+-------------------+-------------+---------------+--------------+-----------------+---------------+------------------------+---------------
  User:u-y9gdx6 | pgarefalakis+ksql+test_stag@confluent.io | KsqlAdmin         | env-nx6qz6  | lkc-8gookr    | ksqlDB       |                 | KSQL          | ksqlDB_cluster_0       | LITERAL       
  User:u-y9gdx6 | pgarefalakis+ksql+test_stag@confluent.io | KsqlAdmin         | env-nx6qz6  | lkc-kjzdov    | ksqlDB       | pksqlc-ynnj6    | KSQL          | panos_ksqlDB_cluster_0 | LITERAL       
  User:u-y9gdx6 | pgarefalakis+ksql+test_stag@confluent.io | OrganizationAdmin |             |               |              |                 |               |                        |     

Note: ksqlDB_cluster_0 is delete thus does not have a pksqlc anymore (this should not happen in practice as we automatically cleanup role binding when a Ksql cluster is being deleted)

References

Test & Review

[Brian Strauch (brianstrauch)]

Panagiotis (Panos) Garefalakis (@pgaref) This PR looks good, but it's untested. Can you add an integration test to tests/iam_test.go?

[Panagiotis (Panos) Garefalakis (pgaref)]

Panagiotis (Panos) Garefalakis (@pgaref) This PR looks good, but it's untested. Can you add an integration test to tests/iam_test.go?

Yes, working on this as we speak! Thanks!