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

Update Workspace limit in docs #4098

Merged
merged 2 commits into from
Sep 26, 2024
Merged

Conversation

tim-p-allen
Copy link
Collaborator

Resolves #3920

What is being addressed

Update Docs to reflect workspace limit and a workaround to extend.

Copy link

github-actions bot commented Sep 26, 2024

Unit Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 83bcf84.

♻️ This comment has been updated with latest results.

@tim-p-allen tim-p-allen marked this pull request as ready for review September 26, 2024 09:41
@tim-p-allen
Copy link
Collaborator Author

/help

Copy link

🤖 pr-bot 🤖

Hello!

You can use the following commands:
    /test - build, deploy and run smoke tests on a PR
    /test-extended - build, deploy and run smoke & extended tests on a PR
    /test-extended-aad - build, deploy and run smoke & extended AAD tests on a PR
    /test-shared-services - test the deployment of shared services on a PR build
    /test-force-approve - force approval of the PR tests (i.e. skip the deployment checks)
    /test-destroy-env - delete the validation environment for a PR (e.g. to enable testing a deployment from a clean start after previous tests)
    /help - show this help

(in response to this comment from @tim-allen-ck)

@tim-p-allen
Copy link
Collaborator Author

/test-force-approve

Copy link

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit e202797)

(in response to this comment from @tim-allen-ck)

Copy link
Member

@marrobi marrobi left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Danny-Cooke-CK Danny-Cooke-CK left a comment

Choose a reason for hiding this comment

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

approved

@tim-p-allen
Copy link
Collaborator Author

/test-force-approve

@tim-p-allen tim-p-allen enabled auto-merge (squash) September 26, 2024 13:34
Copy link

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit 83bcf84)

(in response to this comment from @tim-allen-ck)

@tim-p-allen tim-p-allen merged commit 87cb4f0 into microsoft:main Sep 26, 2024
12 checks passed
@tim-p-allen tim-p-allen deleted the workspace-limit branch September 26, 2024 13:35
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.

TRE cannot scale beyond about 32 projects
3 participants