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

Better error message needed for the environment cloud provider configuration #6861

Closed
ytimocin opened this issue Nov 27, 2023 · 9 comments
Closed
Assignees
Labels
bug Something is broken or not working as expected good first issue Good for newcomers triaged This issue has been reviewed and triaged

Comments

@ytimocin
Copy link
Contributor

ytimocin commented Nov 27, 2023

Bug information

Getting this error while deploying:

{
"code": "AuthorizationFailed",
"message": "The client 'f96424a0-61a3-4fdf-814e-b15aff634321' with object id 'f96424a0-61a3-4fdf-814e-b15aff634321' does not have authorization to perform action 'Microsoft.ServiceBus/namespaces/write' over scope '/providers/Microsoft.ServiceBus/namespaces/testservicebusnamespace' or the scope is invalid. If access was recently granted, please refresh your credentials.",
"target": "/providers/Microsoft.ServiceBus/namespaces/testservicebusnamespace"
}

This is basically an environment that are missing cloud provider configuration. This needs to be told to the Radius user/operator with a more clear message.

Steps to reproduce (required)

  1. Create an environment without cloud provider configuration.
  2. Deploy a bicep file that has cloud resources (like Azure Cosmos DB)
  3. See the error message.

Observed behavior (required)

Error message not being clear.

Desired behavior (required)

Error message should state that a cloud provider needs to be configured.

Workaround (optional)

System information

rad Version (required)

RELEASE VERSION BICEP COMMIT
0.27.1 v0.27.1 0.27.1 a0fa10b

Operating system (required)

macOS, Sonoma 14.1.1, Apple Chip

Additional context

Discussion on Discord: https://discord.com/channels/1113519723347456110/1176534953345957888/1176534953345957888.

AB#10546

@ytimocin ytimocin added the bug Something is broken or not working as expected label Nov 27, 2023
@radius-triage-bot
Copy link

👋 @ytimocin Thanks for filing this bug report.

A project maintainer will review this report and get back to you soon. If you'd like immediate help troubleshooting, please visit our Discord server.

For more information on our triage process please visit our triage overview

@shalabhms shalabhms added the triaged This issue has been reviewed and triaged label Dec 7, 2023
@radius-triage-bot
Copy link

👍 We've reviewed this issue and have agreed to add it to our backlog. Please subscribe to this issue for notifications, we'll provide updates when we pick it up.

We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

For more information on our triage process please visit our triage overview

@shalabhms shalabhms added the good first issue Good for newcomers label Feb 2, 2024
@radius-triage-bot
Copy link

This issue is a great one to pickup for new contributors. It should only require small changes and not assume a deep knowledge of the Radius architecture.

We always welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

For more information on our triage process please visit our triage overview

@willtsai
Copy link
Contributor

willtsai commented Sep 3, 2024

The root cause of this is missing configuration for the Azure Scope as a part of the environment, thus the error message should indicate as such. Because the scope is missing, we're sending ARM a request something the use didn't intend, hence the authorization error being returned from Azure.

@ckansara16
Copy link
Contributor

/assign

@willtsai
Copy link
Contributor

@ckansara16 - i believe this one might have the same root cause as #7658 which @vishwahiremat is working on currently in #8006 -- the two of you should sync up on the fix and validating the solution!

@ckansara16
Copy link
Contributor

Hello @vishwahiremat - i have reviewed #8006 . its same issue which i have started. if yes than we can close this. let me know if you need additional fix.

@willtsai
Copy link
Contributor

closing as this is a confirmed duplicate of #7658

@willtsai
Copy link
Contributor

Thanks @ckansara16! If you're looking for similar issues to take on, might I suggest doing some investigation into #6302 as I think it might be a separate but related issue with ambiguous errors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken or not working as expected good first issue Good for newcomers triaged This issue has been reviewed and triaged
Projects
None yet
Development

No branches or pull requests

4 participants