Skip to content

Conversation

@joelverhagen
Copy link
Member

This brings a removal of unused tests to main and unblocks the test build.
This is needed to unblocked deployed since I removed the configuration setting used by these tests.

These have not run in ages (couldn't find how long, likely over a year). The test coverage they provide is duplicated in other tests and they are not even executed as load tests. Even if they were, they would not sufficiently cover our performance hot spots and are not run in a scientific way.
@joelverhagen joelverhagen requested a review from a team as a code owner December 9, 2025 16:03
joelverhagen and others added 10 commits December 10, 2025 11:57
…Redirect (#10650)

* Use Microsoft.CodeDom.Providers.DotNetCompilerPlatform in Gallery
* Use latest language version like .csproj
* Check in debug for CDNRedirect web.config, remove it for release builds, like gallery
* Allow custom tenant for Entra ID auth
* Remove-Item is no longer needed for load test results
* Initial files

* INitial files

* Pushing solution

* Removed unnecessary files

* Removed more files

* Removed more files

* Removed more files

* Properly changed gallery.sln

* Polish

* Removed more files

* Fix nupkg package is not created

* Added column

* Updated entities

* Moved everything to catalogvalidation

* Add batchId

* Revert unnecessary changes

* Added batchid to validation entities (instead of catalogvalidation), made it optional, added indexing

* Removed an unnecessary migration

* Commenting out a check which doesn't apply

* Fixed comment

* Fixed tests

* Proper migration schema

* Clarified changes

* Nit, testing pushed changes

* Remove IsReducedVersionListsEnabled method

Removed unimplemented method IsReducedVersionListsEnabled from FakeFeatureFlagService.

* Remove IsReducedVersionListsEnabled method

Removed the IsReducedVersionListsEnabled method.

* Update comment for bug reference in Job.cs

* Revert "Merge branch 'main' into lpar-validation-change"

This reverts commit aacf762, reversing
changes made to 72dde91.

* Reverting unneccessary changes

* Reverting more changes

* Reverting

* Fixing revert

* Fixed code so that validationDB, table validatostatus doesn't have the batchID column

* Latest nits

---------

Co-authored-by: Lanaparezanin <[email protected]>
Co-authored-by: Lanaparezanin <[email protected]>
* Fixed package icon-title-version alignment on package details page
* Maintaining aspect ratio for non-standard icons and aligned version number with title
* Option to rename output file.
* Preserving paths in source storage.
* Better filename generation for sourceFilename column.
* Not actually logging "was already present" when destination blob was not present.
@chabiss chabiss closed this Jan 6, 2026
@chabiss
Copy link
Contributor

chabiss commented Jan 6, 2026

No need anymore

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.

7 participants