Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Fix integration test.py multitest case #3595

Closed
wants to merge 175 commits into from
Closed

Fix integration test.py multitest case #3595

wants to merge 175 commits into from

Conversation

AdamL-Microsoft
Copy link
Contributor

Summary of the Pull Request

What is this about?

PR Checklist

  • Applies to work item: #xxx
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

@AdamL-Microsoft
re-add sync-fork.yml
cdcd3b7
@AdamL-Microsoft
Merge pull request #316 from microsoft/main
d78d47b 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #318 from microsoft/main
e97517e 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #319 from microsoft/main
fa01963 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #321 from microsoft/main
7e28393 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #322 from microsoft/main
47a1de9 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #324 from microsoft/main
d7487d5 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #325 from microsoft/main
ba5c160 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #326 from microsoft/main
2d8cef5 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #328 from microsoft/main
89cb764 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #329 from microsoft/main
253f759 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #330 from microsoft/main
f9d0b59 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #331 from microsoft/main
4969769 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #332 from microsoft/main
8e20d19 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #333 from microsoft/main
8f30d2c 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #335 from microsoft/main
094fb8c 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #336 from microsoft/main
a3edb3b 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #341 from microsoft/main
442d92b 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #342 from microsoft/main
c79d8c9 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #343 from microsoft/main
22c46f3 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #345 from microsoft/main
09dbbe8 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #346 from microsoft/main
1f8806d 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #347 from microsoft/main
8cba9a5 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #348 from microsoft/main
1641b7d 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #349 from microsoft/main
a1a28ef 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #351 from microsoft/main
2fd7774 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #352 from microsoft/main
e385f68 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #353 from microsoft/main
16df970 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #355 from microsoft/main
28c9336 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #357 from microsoft/main
b5b3ee1 
Fork Sync: Update from parent repository
chkeita and others added 26 commits September 28, 2023 17:54
@chkeita
Terminate process on timeout in windows for the coverage task (micros…
e3b1e0e 
…oft#3529)

* Terminate process on timeout in windows for the coverage task

* set the timeout before we start the debugger

* split the target launch from the debugger initialization
wait for the process to finish on a separate thread

* fix build

* move comments
@AdamL-Microsoft
Merge branch 'microsoft:main' into main
056c674
@chkeita
Ignore regression update when the work item is in some states (micros…
434a435 
…oft#3532)

* Ignore regression update when the work item is in some states

* format

* formatting

* don't hide messages in the poison queue

* fix typo

* update regression logic
update test_template to support regression

* build fix

* mypy fix

* build fix

* move regression ignore state under ADODuplicateTemplate

* replace extend with append

* update set_tcp_keepalive

* mke mypy happy

* copy ADODuplicateTemplate.OnDuplicate.RegressionIgnoreStates
@AdamL-Microsoft
Merge pull request #450 from microsoft/main
53ffd92 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #451 from microsoft/main
7eddd15 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #452 from microsoft/main
292e0a5 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #453 from microsoft/main
5bd49b0 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #454 from microsoft/main
0268308 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #455 from microsoft/main
60396bf 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge branch 'microsoft:main' into main
10d1c08
@AdamL-Microsoft
Merge pull request #456 from microsoft/main
450c8bb 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #457 from microsoft/main
af42e0a 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #458 from microsoft/main
689c0eb 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #459 from microsoft/main
ce01238 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #460 from microsoft/main
8cf1a31 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #461 from microsoft/main
cefff2a 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #462 from microsoft/main
e7332d2 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #463 from microsoft/main
f4f64b1 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #464 from microsoft/main
8b8414e 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #465 from microsoft/main
0ff4868 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #466 from microsoft/main
97d102e 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #467 from microsoft/main
d73c966 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #468 from microsoft/main
dab9f8f 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #469 from microsoft/main
29fd625 
Fork Sync: Update from parent repository
@AdamL-Microsoft
Merge pull request #470 from microsoft/main
c780347 
Fork Sync: Update from parent repository
@AdamL-Microsoft
testing
b4e2f6d
@codecov-commenter
Copy link

codecov-commenter commented Oct 23, 2023 •
edited
Loading

Codecov Report

Merging #3595 (16bc0dc) into main (44c2c39) will increase coverage by 8.45%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3595      +/-   ##
==========================================
+ Coverage   39.96%   48.42%   +8.45%     
==========================================
  Files         303      139     -164     
  Lines       37635    17861   -19774     
  Branches     1721        0    -1721     
==========================================
- Hits        15042     8649    -6393     
+ Misses      22274     9212   -13062     
+ Partials      319        0     -319

see 195 files with indirect coverage changes

@AdamL-Microsoft
Copy link
Contributor Author

This turned out to be unnecessary, issue was in ADO not the integration-test.py script.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@AdamL-Microsoft @codecov-commenter @kananb @mgreisen @nharper285 @Porges @tevoinea @chkeita