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

Explore challenging intermittent nightly test failures #3148

Closed
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
68 commits
Select commit Hold shift + click to select a range
da45bcb
add some debug code to Statistics.Split
bspratt Sep 5, 2024
759fe1c
try to stabilize these tests by using AbstractFunctionalTestEx.OpenDo…
bspratt Sep 5, 2024
c074520
per MattC email, try eliminating call to DiaUmpire.Config.GetConfigF…
bspratt Sep 9, 2024
737aff4
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Sep 9, 2024
0c1e803
trying to determine with TestMS1Tutorial is hanging - added Program.R…
bspratt Sep 9, 2024
ff5c256
Merge branch 'Skyline/work/20240905_what_is_up_with_tests_on_jason_xp…
bspratt Sep 9, 2024
2be4d07
TestRunner launches ProcDump (if installed, see https://learn.microso…
bspratt Sep 30, 2024
b0c7b95
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Oct 1, 2024
b8cb553
Merge branch 'Skyline/work/20240905_what_is_up_with_tests_on_jason_xp…
bspratt Oct 1, 2024
d95f715
SkylineNightly: set git's http.postBuffer size to 500MB to help with …
bspratt Oct 1, 2024
e5ac552
per Nick suggestion, aply the http buffer size change only for the du…
bspratt Oct 1, 2024
78c4729
add an informative message about SysInternals ProcDump when it isn't …
bspratt Oct 2, 2024
9e13daf
slightly more verbose launch for ProcDump
bspratt Oct 7, 2024
da1aa53
try to build as release with debug symbols retained
bspratt Oct 8, 2024
006a64a
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Oct 8, 2024
88265ee
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
brendanx67 Oct 16, 2024
9d3c717
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Oct 17, 2024
f4e8579
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Oct 17, 2024
39158cd
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Oct 18, 2024
a7d7e03
trying to leave debug symbols in confuses bjam
bspratt Oct 19, 2024
568acfe
Merge branch 'Skyline/work/20240905_what_is_up_with_tests_on_jason_xp…
bspratt Oct 19, 2024
fe9334f
Another try at a TC release build with debug symbols on - backs out a…
bspratt Oct 22, 2024
0a88db1
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Oct 24, 2024
6a426e0
Add leak-related diagnotics to the SkylineTester log stats comparison…
bspratt Oct 24, 2024
e0ab70f
Merge branch 'Skyline/work/20240905_what_is_up_with_tests_on_jason_xp…
bspratt Nov 7, 2024
dcc84da
more debug output in Ms1FullScanFilteringTutorial
bspratt Nov 7, 2024
1871f77
a bit more granularity on this test's debug logging
bspratt Nov 8, 2024
051db07
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Nov 8, 2024
9384ea3
trying to enable debug symbols in the pwiz CLI
bspratt Nov 12, 2024
b167c12
Fix a parsing problem in build-apps.bat that would mistake "debug-sym…
bspratt Nov 12, 2024
932fd94
backing out mistaken change to Jamroot.jam
bspratt Nov 12, 2024
a81d191
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Nov 14, 2024
e7c0390
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Nov 15, 2024
ed5f312
add msvc "address sanitizer" to try to chase down memory problem
bspratt Nov 16, 2024
d9357ab
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Nov 18, 2024
20c2fb8
address sanitizer makes for big object files: "fatal error C1128: num…
bspratt Nov 18, 2024
2842502
Fix a bad merge
bspratt Nov 19, 2024
c6b2c87
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Nov 20, 2024
72030f3
Don't include address sanitizer unless debug-symbols=on
bspratt Nov 20, 2024
e73fdbe
introducing WaitForMultiButtonMsgDlg(string messageFormat) to replace…
bspratt Nov 20, 2024
715973b
fix a typo in jamroot.jam
bspratt Nov 21, 2024
993db45
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Nov 21, 2024
9bbd051
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 2, 2024
1e4d700
When lockmass correction is requested for non-Waters data, just issue…
bspratt Dec 3, 2024
4752f8a
fix a race condition in TestAssayLibraryImport and TestAssayLibraryIm…
bspratt Dec 3, 2024
7bebc79
Add a few memory-oriented columns to SkylineTester's RunStats tab for…
bspratt Dec 3, 2024
0d21a08
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 3, 2024
3682ccc
try skipping tests that use MSAmanda, to see if that's implicated in …
bspratt Dec 4, 2024
a3b987f
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 4, 2024
173324e
set Program.ReportTutorialTestProgress to write screenshot info to co…
bspratt Dec 4, 2024
71674d3
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 5, 2024
efb3c74
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 9, 2024
3d89896
As previously done with the spectrum lockmass refiner, when lockmass …
bspratt Dec 10, 2024
c2e0965
disable address sanitizer - it seems to be its own point of failure a…
bspratt Dec 15, 2024
56172b0
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 15, 2024
a08fa77
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 17, 2024
700cfa8
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 18, 2024
e8058d3
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 18, 2024
74b7af7
re-enable MSAmanda to see if that's what's causing memory issues
bspratt Dec 19, 2024
10bd8c8
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 19, 2024
7f78bce
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Dec 22, 2024
55ea8b2
back out some debug cruft
bspratt Dec 22, 2024
9719351
added a TODO to a comment blocking out potentially problematic code
bspratt Dec 30, 2024
03c1d76
Skipping MS-Amanda tests again to convince myself that's the memory i…
bspratt Jan 3, 2025
cea04a7
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Jan 3, 2025
628bf4a
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Jan 7, 2025
4b85049
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Jan 9, 2025
19b83bc
Merge branch 'master' into Skyline/work/20240905_what_is_up_with_test…
bspratt Jan 10, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions Jamroot.jam
Original file line number Diff line number Diff line change
Expand Up @@ -362,6 +362,8 @@ project pwiz
<toolset>msvc:<cxxflags>/wd4456 # warning: declaration of 'type' hides previous local declaration
<toolset>msvc:<cxxflags>/wd4459 # warning: declaration of 'type' hides global declaration
<toolset>msvc:<cxxflags>/wd4458 # warning: declaration of 'type' hides class member
#<toolset>msvc,<debug-symbols>on:<cxxflags>-fsanitize=address # address sanitizer
#<toolset>msvc,<debug-symbols>on:<cxxflags>/bigobj # address sanitizer makes for big object files: "fatal error C1128: number of sections exceeded object file format limit"
<toolset>msvc,<debug-symbols>off:<linkflags>"/OPT:REF,ICF"
<toolset>msvc,<toolset-msvc:version>12.0:<define>_HAS_TR1
<toolset>msvc,<toolset-msvc:version>14.0:<define>_HAS_TR1
Expand Down
4 changes: 3 additions & 1 deletion pwiz_tools/Skyline/Model/DiaUmpireDdaConverter.cs
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,7 @@ public override bool Run(IProgressMonitor progressMonitor, IProgressStatus statu

// CONSIDER: read the file description to see what settings were used to generate the file;
// if the same settings were used, we can re-use the file, else regenerate
/* TODO(bspratt) temp removal - is this the trouble spot causing memory errors? TODO restore this?
if (MsDataFileImpl.IsValidFile(outputFilepath))
{
var outputFileConfig = DiaUmpire.Config.GetConfigFromDiaUmpireOutput(outputFilepath);
Expand Down Expand Up @@ -139,7 +140,8 @@ public override bool Run(IProgressMonitor progressMonitor, IProgressStatus statu
}
}
}

end temp removal */

if (File.Exists(outputFilepath))
FileEx.SafeDelete(outputFilepath);

Expand Down
1 change: 1 addition & 0 deletions pwiz_tools/Skyline/Program.cs
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@ public static bool SkylineOffscreen
} // Set true to move Skyline windows offscreen.

public static bool DemoMode { get; set; } // Set to true in demo mode (main window is full screen and pauses at screenshots)
public static bool ReportTutorialTestProgress { get; set; } // When True, write PauseForScreenshot messages to console even if offscreen and even if not pausing for screenshots
public static bool NoVendorReaders { get; set; } // Set true to avoid calling vendor readers.
public static bool UseOriginalURLs { get; set; } // Set true to use original URLs for downloading tools instead of our S3 copies
public static bool IsPassZero { get { return NoVendorReaders; } } // Currently the only time NoVendorReaders gets set is pass0
Expand Down
36 changes: 36 additions & 0 deletions pwiz_tools/Skyline/SkylineTester/SkylineTesterWindow.Designer.cs

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 6 additions & 0 deletions pwiz_tools/Skyline/SkylineTester/SkylineTesterWindow.resx
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,9 @@
<metadata name="TestName.UserAddedColumn" type="System.Boolean, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089">
<value>True</value>
</metadata>
<metadata name="FirstAppearance.UserAddedColumn" type="System.Boolean, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089">
<value>True</value>
</metadata>
<metadata name="Iterations.UserAddedColumn" type="System.Boolean, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089">
<value>True</value>
</metadata>
Expand All @@ -172,6 +175,9 @@
<metadata name="menuStrip1.TrayLocation" type="System.Drawing.Point, System.Drawing, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a">
<value>17, 17</value>
</metadata>
<metadata name="DetectedLeak.UserAddedColumn" type="System.Boolean, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089">
<value>True</value>
</metadata>
<metadata name="$this.TrayHeight" type="System.Int32, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089">
<value>25</value>
</metadata>
Expand Down
8 changes: 5 additions & 3 deletions pwiz_tools/Skyline/SkylineTester/TabBuild.cs
Original file line number Diff line number Diff line change
Expand Up @@ -196,15 +196,17 @@ public static bool CreateBuildCommands(

commandShell.Add("#@ Checking out {0} source files...\n", branchName);
commandShell.Add("# Checking out {0} source files...", branchName);

// Add the --progress flag for richer logging - git leaves out most progress info when it isn't writing to an actual terminal
// Add the "-c http.postBuffer=524288000" argument to help with clone on unstable networks
if (branchName.Contains("master"))
{
commandShell.AddWithRetry("{0} clone {1} --recurse-submodules --progress {2}", git.Quote(), branchUrl.Quote(), buildRoot.Quote());
commandShell.AddWithRetry("{0} -c http.postBuffer=524288000 clone {1} --recurse-submodules --progress {2}", git.Quote(), branchUrl.Quote(), buildRoot.Quote());
}
else
{
var branch = branchUrl.Split(new[] {"tree/"}, StringSplitOptions.None)[1];
commandShell.AddWithRetry("{0} clone {1} --recurse-submodules --progress -b {2} {3}", git.Quote(), GetMasterUrl().Quote(), branch.Quote(), buildRoot.Quote());
commandShell.AddWithRetry("{0} -c http.postBuffer=524288000 clone {1} --recurse-submodules --progress -b {2} {3}", git.Quote(), GetMasterUrl().Quote(), branch.Quote(), buildRoot.Quote());
}
}

Expand All @@ -214,7 +216,7 @@ public static bool CreateBuildCommands(
foreach (int architecture in architectures)
{
commandShell.Add("#@ Building Skyline {0} bit...\n", architecture);
commandShell.Add("{0} {1} {2} --i-agree-to-the-vendor-licenses {3} nolog",
commandShell.Add("{0} {1} {2} --i-agree-to-the-vendor-licenses {3} debug-symbols=on nolog",
Path.Combine(buildRoot, @"pwiz_tools\build-apps.bat").Quote(),
architecture,
runBuildTests ? "pwiz_tools/Skyline" : "pwiz_tools/Skyline//Skyline.exe",
Expand Down
Loading
Loading