Load test : Ready for Release #3284
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Sean P. Goggins <s@goggins.com>
Full Load Test of patches adn fixes.
commented out abused API endpoint
1 task
MoralCode
requested changes
Sep 23, 2025
Contributor
MoralCode
left a comment
MoralCode
left a comment
There was a problem hiding this comment.
for reasons that defy logic, disable secondary and facade tasks on some deployment environments
I think it would be preferable to understand why things break first so that the release notes can be more useful to any admins
I made #3286 to see if we can identify the reason why this revert is necessary
Contributor
|
I made #3289 so we can ship the part of this that we agree on and keep the discussion focused on figuring out the issues with the other piece I think this PR should be closed in favor of just merging its constituent parts directly into main |
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR :
augur/tasks/util/collection_util.pythat, for reasons that defy logic, disable secondary and facade tasks on some deployment environments. We have yet to determine WHY this causes issue, but know that it does so are reverting the change for release.