Skip to content

[CLI-2061, CLI-2062] Add --environment flag to api-key list, improve wrong env error#1479

Merged
Brian Strauch (brianstrauch) merged 1 commit intomainfrom
CLI-2062
Oct 25, 2022
Merged

[CLI-2061, CLI-2062] Add --environment flag to api-key list, improve wrong env error#1479
Brian Strauch (brianstrauch) merged 1 commit intomainfrom
CLI-2062

Conversation

@brianstrauch
Copy link

@brianstrauch Brian Strauch (brianstrauch) commented Oct 25, 2022

Checklist

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

What

When using a --resource not part of the current environment, the user used to see:

Forbidden Access: 403 Forbidden

This error has been improved with suggestions:

Error: Kafka cluster not found or access forbidden: Forbidden Access: 403 Forbidden

Suggestions:
    Ensure the cluster ID you entered is valid.
    Ensure the cluster you are specifying belongs to the currently selected environment with `confluent kafka cluster list`, `confluent environment list`, and `confluent environment use`.

At this point, the user can use --environment to choose the right environment.

References

CLI-2061
CLI-2062

Test & Review

Manually tested, all tests still pass

@brianstrauch Brian Strauch (brianstrauch) requested a review from a team as a code owner October 25, 2022 21:39
@brianstrauch Brian Strauch (brianstrauch) changed the title [CLI-2062] Add --environment flag to api-key list [CLI-2061, CLI-2062] Add --environment flag to api-key list, improve wrong env error Oct 25, 2022
Copy link
Member

Choose a reason for hiding this comment

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

lgtm!

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