-
Notifications
You must be signed in to change notification settings - Fork 101
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
Get long-haul test cluster working #6936
Conversation
2e1d8ea
to
0952f28
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... |
0952f28
to
560b3fd
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... |
560b3fd
to
281701f
Compare
281701f
to
60fb21c
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... |
60fb21c
to
573e89b
Compare
573e89b
to
07bac27
Compare
07bac27
to
ba800bc
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... |
891aff9
to
538893d
Compare
538893d
to
dbc611e
Compare
dbc611e
to
09d1cf0
Compare
09d1cf0
to
bfe4e40
Compare
bfe4e40
to
1c74069
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... |
cpu: '600m' | ||
memory: '1024Mi' | ||
cpu: '1200m' | ||
memory: '2048Mi' | ||
} | ||
limits: { | ||
cpu: '900m' | ||
memory: '1024Mi' | ||
cpu: '1500m' | ||
memory: '4096Mi' |
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.
Needed to do this update because mssql pod wasn't able to get into READY state.
1c74069
to
0b7a8fe
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... |
Signed-off-by: ytimocin <[email protected]>
Signed-off-by: ytimocin <[email protected]>
0b7a8fe
to
0d58a4d
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... |
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.
👍
# Description Get long-haul test cluster working. ## Type of change - This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional). Fixes: #issue_number ## Auto-generated summary <!-- copilot:all --> ### <samp>🤖[[deprecated]](https://githubnext.com/copilot-for-prs-sunset) Generated by Copilot at 2e1d8ea</samp> ### Summary 🔐📝🧹 <!-- 1. 🔐 - This emoji represents the change to the RBAC rules, which is related to security and permissions. It could also be represented by a 🔑 or a 🛡️ emoji. 2. 📝 - This emoji represents the change to the YAML file, which is related to editing and writing code. It could also be represented by a ✏️ or a 🖋️ emoji. 3. 🧹 - This emoji represents the change to the formatting and style, which is related to cleaning and tidying up the code. It could also be represented by a 🗑️ or a 🚮 emoji. --> This pull request improves the formatting of a GitHub workflow file and updates the RBAC rules for the controller service account in the Helm chart. The formatting changes make the workflow file more consistent and readable. The RBAC changes allow the controller to manage all resources in the `radapp.io` API group, instead of only the recipes. > _`radapp.io` grows_ > _controller needs more access_ > _helm chart changes now_ ### Walkthrough * Expand RBAC permissions for controller service account in Helm chart ([link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-0eb83f283dad161ea34486e10ead98d0c74fcaa93a13e8d182113ebb8de66c63L39-R40)) * Replace single quotes with double quotes for YAML strings in long-running-azure workflow ([link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L48-R52), [link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L71-R73), [link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L79-R82)) * Remove trailing spaces and extra blank lines in long-running-azure workflow ([link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L7-R7), [link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L20-R34), [link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L154-R155), [link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L160-R166), [link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L189-R198), [link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L208-R209), [link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L215-R216)) * Reduce indentation of echo commands in long-running-azure workflow ([link](https://github.com/radius-project/radius/pull/6936/files?diff=unified&w=0#diff-85cd86e7ae6da973e4808fe57f4fea7892ab78a499ba5aec72c0b3751a2cc226L126-R129)) --------- Signed-off-by: ytimocin <[email protected]> Signed-off-by: willdavsmith <[email protected]>
Description
Get long-haul test cluster working.
Type of change
Fixes: #issue_number
Auto-generated summary
🤖[deprecated] Generated by Copilot at 2e1d8ea
Summary
🔐📝🧹
This pull request improves the formatting of a GitHub workflow file and updates the RBAC rules for the controller service account in the Helm chart. The formatting changes make the workflow file more consistent and readable. The RBAC changes allow the controller to manage all resources in the
radapp.io
API group, instead of only the recipes.Walkthrough