-
Notifications
You must be signed in to change notification settings - Fork 97
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
Changing permissions of UCP - removing unused permissions #8080
Conversation
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
4957ab2
to
d4ee350
Compare
d4ee350
to
4d81571
Compare
- '*' | ||
verbs: | ||
- '*' | ||
- create | ||
--- | ||
apiVersion: rbac.authorization.k8s.io/v1 | ||
kind: ClusterRoleBinding |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing this causes this error (an error that we have all seen before but didn't know why):
Radius release not found. Proceeding with installation.
*** Installing Radius to Kubernetes ***
Installing Radius version 8d1442d-dirty to namespace: radius-system...
*** Create workspace, group and environment for test ***
Creating workspace...
Set "kind-kind" as current workspace
Error: An unknown error was returned while testing Radius API status:
Status Code: 503
Response Headers:
Audit-Id: 863b6ad6-02a3-4ba6-9c51-61f03dc66cf2
Cache-Control: no-cache, private
Content-Length: 20
Content-Type: text/plain; charset=utf-8
Date: Sun, 24 Nov 2024 16:36:20 GMT
X-Content-Type-Options: nosniff
X-Kubernetes-Pf-Flowschema-Uid: db65b004-bc64-4949-a3b2-2a0[80](https://github.com/radius-project/radius/actions/runs/11997783030/job/33443799631?pr=8080#step:18:81)1c4da00
X-Kubernetes-Pf-Prioritylevel-Uid: fe22ed3a-e9f4-4f37-9901-d2347e099c15
Response Body:
service unavailable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The logs will have the real error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ytimocin did you get a chance to look into the logs for error as Ryan recommended?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately, I missed that window. It is a resource permission issue but not exactly sure which one. I will keep an eye out for this error.
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
6423ed2
to
9f6550b
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
9f6550b
to
71044d1
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
71044d1
to
c45be27
Compare
deploy/Chart/templates/ucp/rbac.yaml
Outdated
- ucp.dev | ||
resources: | ||
- queuemessages | ||
- resources |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
c45be27
to
a8d2635
Compare
a8d2635
to
83be037
Compare
b3386e6
to
3768f45
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
3768f45
to
b35b7b5
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
b35b7b5
to
a70366b
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
a70366b
to
494731d
Compare
494731d
to
f967f56
Compare
Signed-off-by: ytimocin <[email protected]>
Signed-off-by: ytimocin <[email protected]>
f967f56
to
1accbb7
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8080 +/- ##
=======================================
Coverage 59.65% 59.65%
=======================================
Files 577 577
Lines 38573 38573
=======================================
Hits 23010 23010
Misses 13916 13916
Partials 1647 1647 ☔ View full report in Codecov by Sentry. |
Description
Changing permissions of UCP - removing unused permissions.
Type of change
Fixes: Update and remove unnecessary permissions of UCP component #8082
Contributor checklist
Please verify that the PR meets the following requirements, where applicable: