Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lightspeed: use the code value from the 403 #1024

Conversation

goneri
Copy link
Contributor

@goneri goneri commented Dec 5, 2023

The 403 error can now return a code value to specify the nature of the
error.
permission_denied_wca_api_key_is_missing is the first case we can get.
The error happens when the access is denied because the user has a seat
but the organization has not been fully configured yet.

Depends-On: #1026
Depends-On: #1029

@ganeshrn
Copy link
Member

ganeshrn commented Dec 6, 2023

@goneri Can you please fix the CI failure

The 403 error can now return a `code` value to specify the nature of the
error.
`permission_denied_wca_api_key_is_missing` is the first case we can get.
The error happens when the access is denied because the user has a seat
but the organization has not been fully configured yet.
Handle the `permission_denied__org_not_ready_because_wca_not_configured`
error and returns the following error to the user:

```
Contact your administrator to configure IBM watsonx Code Assistant model settings for your organization.
```
@goneri goneri force-pushed the goneri/lightspeed-use-the-code-value-from-the-403_8454 branch from 649d9e7 to 399c636 Compare December 8, 2023 13:56
@goneri
Copy link
Contributor Author

goneri commented Dec 8, 2023

The CI gods are against me.

@goneri
Copy link
Contributor Author

goneri commented Dec 8, 2023

@ajinkyau could you also take a look at #1029. It's a CI fix that address the err 127 that we get time to time when Task tries to call yarn.

@ssbarnea ssbarnea enabled auto-merge (squash) December 9, 2023 15:08
@ssbarnea ssbarnea merged commit 4517575 into ansible:main Dec 9, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants