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

Remove repro test from integration tests #3560

Merged
merged 45 commits into from
Oct 9, 2023
Merged

Remove repro test from integration tests #3560

merged 45 commits into from
Oct 9, 2023

Conversation

nharper285
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 and others added 30 commits August 29, 2023 19:57
@AdamL-Microsoft @chkeita
Release 8.7.1 (hotfix) (microsoft#3459)
c69deed 
* Remove the retention policy setting (microsoft#3452)

---------

Co-authored-by: Cheick Keita <chkeita@microsoft.com>
@AdamL-Microsoft
Revert "Release 8.7.1 (hotfix) (microsoft#3459)" (microsoft#3468)
c8986aa 
This reverts commit c69deed.
@AdamL-Microsoft @chkeita
Redo 8.7.1 (microsoft#3469)
7b40402 
* Redo-8.7.1-hotfix

---------

Co-authored-by: Cheick Keita <chkeita@microsoft.com>
@kananb
Support custom ado fields that mark work items as duplicate (microsof…
d999603 
…t#3467)

* Add field to ado config for checking duplicate work items

* Make duplicate fields nullable and add it to python models

* Update broken tests

* Update docs to include new ado_duplicate_fields property
@mgreisen @AdamL-Microsoft
Update readme with archive message (microsoft#3408)
b2435b1 
Co-authored-by: Adam <103067949+AdamL-Microsoft@users.noreply.github.com>
@dependabot
Bump tokio from 1.30.0 to 1.32.0 in /src/proxy-manager (microsoft#3425)
b913074 
Bumps [tokio](https://github.com/tokio-rs/tokio) from 1.30.0 to 1.32.0.
- [Release notes](https://github.com/tokio-rs/tokio/releases)
- [Commits](tokio-rs/tokio@tokio-1.30.0...tokio-1.32.0)

---
updated-dependencies:
- dependency-name: tokio
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump tokio from 1.30.0 to 1.32.0 in /src/agent (microsoft#3424)
14ab36e 
Bumps [tokio](https://github.com/tokio-rs/tokio) from 1.30.0 to 1.32.0.
- [Release notes](https://github.com/tokio-rs/tokio/releases)
- [Commits](tokio-rs/tokio@tokio-1.30.0...tokio-1.32.0)

---
updated-dependencies:
- dependency-name: tokio
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@kananb
Remove unnecessary method argument (microsoft#3473)
f141050
@dependabot
Bump elsa from 1.8.1 to 1.9.0 in /src/agent (microsoft#3411)
d4319d2 
Bumps [elsa](https://github.com/manishearth/elsa) from 1.8.1 to 1.9.0.
- [Commits](Manishearth/elsa@v1.8.1...v1.9.0)

---
updated-dependencies:
- dependency-name: elsa
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump tempfile from 3.7.1 to 3.8.0 in /src/agent (microsoft#3437)
93b16ec 
Bumps [tempfile](https://github.com/Stebalien/tempfile) from 3.7.1 to 3.8.0.
- [Changelog](https://github.com/Stebalien/tempfile/blob/master/CHANGELOG.md)
- [Commits](Stebalien/tempfile@v3.7.1...v3.8.0)

---
updated-dependencies:
- dependency-name: tempfile
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump tempfile from 3.7.1 to 3.8.0 in /src/proxy-manager (microsoft#3436)
7f7ab37 
Bumps [tempfile](https://github.com/Stebalien/tempfile) from 3.7.1 to 3.8.0.
- [Changelog](https://github.com/Stebalien/tempfile/blob/master/CHANGELOG.md)
- [Commits](Stebalien/tempfile@v3.7.1...v3.8.0)

---
updated-dependencies:
- dependency-name: tempfile
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@nharper285
Updating requirements.txt to accept >= onefuzztypes. (microsoft#3477)
b2e6a07 
* Updating requirements.txt to accept >= onefuzztypes.

* Trying to loosen restriction.
@dependabot
Bump notify from 6.0.1 to 6.1.1 in /src/agent (microsoft#3435)
aa9c9ea 
Bumps [notify](https://github.com/notify-rs/notify) from 6.0.1 to 6.1.1.
- [Release notes](https://github.com/notify-rs/notify/releases)
- [Changelog](https://github.com/notify-rs/notify/blob/main/CHANGELOG.md)
- [Commits](notify-rs/notify@notify-6.0.1...notify-6.1.1)

---
updated-dependencies:
- dependency-name: notify
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@Porges
Bump azure_* crates (microsoft#3478)
74475cc
@AdamL-Microsoft
Release 8.8.0 (microsoft#3466)
64699ed 
* Release 8.8.0
@dependabot
Bump clap from 4.3.21 to 4.4.2 in /src/agent (microsoft#3484)
a3fb480 
Bumps [clap](https://github.com/clap-rs/clap) from 4.3.21 to 4.4.2.
- [Release notes](https://github.com/clap-rs/clap/releases)
- [Changelog](https://github.com/clap-rs/clap/blob/master/CHANGELOG.md)
- [Commits](clap-rs/clap@v4.3.21...v4.4.2)

---
updated-dependencies:
- dependency-name: clap
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump gimli from 0.27.3 to 0.28.0 in /src/agent (microsoft#3414)
59c52d6 
Bumps [gimli](https://github.com/gimli-rs/gimli) from 0.27.3 to 0.28.0.
- [Changelog](https://github.com/gimli-rs/gimli/blob/master/CHANGELOG.md)
- [Commits](gimli-rs/gimli@0.27.3...0.28.0)

---
updated-dependencies:
- dependency-name: gimli
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump clap from 4.3.21 to 4.4.2 in /src/proxy-manager (microsoft#3474)
dd9e266 
Bumps [clap](https://github.com/clap-rs/clap) from 4.3.21 to 4.4.2.
- [Release notes](https://github.com/clap-rs/clap/releases)
- [Changelog](https://github.com/clap-rs/clap/blob/master/CHANGELOG.md)
- [Commits](clap-rs/clap@v4.3.21...v4.4.2)

---
updated-dependencies:
- dependency-name: clap
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot @AdamL-Microsoft
Bump winreg from 0.50.0 to 0.51.0 in /src/agent (microsoft#3434)
6e2cb14 
Bumps [winreg](https://github.com/gentoo90/winreg-rs) from 0.50.0 to 0.51.0.
- [Release notes](https://github.com/gentoo90/winreg-rs/releases)
- [Changelog](https://github.com/gentoo90/winreg-rs/blob/master/CHANGELOG.md)
- [Commits](gentoo90/winreg-rs@v0.50.0...v0.51.0)

---
updated-dependencies:
- dependency-name: winreg
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Adam <103067949+AdamL-Microsoft@users.noreply.github.com>
@tevoinea
Starting integration tests (microsoft#3438)
d2d57a8 
* Starting integration tests

* Ready to test the test

* Parametrize test

* checkpoint

* Test works

* Run integration tests in pipeline

* fmt

* .

* -p

* Install clang

* quotes not required in yaml?

* Hopefully fixed windows?

* Try without killondrop

* lint

* small test

* another test

* Reuse core name

* Wrong step

* bump tokio?

* Try with rust

* make build happy

* Bump pete and small clean up

* Clean up and make the test pass regularly

* fix broken ci

* Lower the poll timeout

* Set the timeout in a nicer way

* fix windows

* fmt

* Include and copy pdbs

* Ignore if pdb is missing on linux

* It takes too long for coverage to be generated

* lint

* Only warn on missing coverage since it's flaky

* Fix windows build

* Small clean up

* Try lowering the poll delay

* fix coverage

* PR comments

* .

* Apparently make is missing?

* Remove aggressive step skipping in CI
@nharper285
Fix sed checks for CLI versioning (microsoft#3486)
830b479 
* Fix sed checks for CLI versioning

* Fix.

* Fix.

* Changing build_cli

* Trying greater than

* Tring once more.

* Trying major minor

* trying to replace major minor

* Using major minor
@dependabot
Bump bytes from 1.4.0 to 1.5.0 in /src/agent (microsoft#3488)
896329d 
Bumps [bytes](https://github.com/tokio-rs/bytes) from 1.4.0 to 1.5.0.
- [Release notes](https://github.com/tokio-rs/bytes/releases)
- [Changelog](https://github.com/tokio-rs/bytes/blob/master/CHANGELOG.md)
- [Commits](tokio-rs/bytes@v1.4.0...v1.5.0)

---
updated-dependencies:
- dependency-name: bytes
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@kananb
Improve area/iteration path validation (microsoft#3489)
d34138d 
* Add more comprehensive checks and better error messages to area/iteration path validation

* Join invalid chars with space instead of comma

* Make tree path validation more testable

* Add error code for invalid ADO project in config

* Write unit tests for tree path validation

* Format tree path unit tests

* Merge escape character and control character checks and clarify error message
@tevoinea
Improve handling of unexpected breakpoints (microsoft#3493)
d009476 
* Improve handling of unexpected breakpoints

* fmt
@Porges
Update azure_* crates (microsoft#3503)
18f2b4a
@tevoinea @Porges
Fuzz coverage recording (microsoft#3322)
9ede0de 
* Fuzz coverage recording

* Update cargo.toml

* Update src/agent/coverage/fuzz/fuzz_targets/fuzz_target_record_coverage.rs

Co-authored-by: George Pollard <porges@porg.es>

* Fix fuzz

---------

Co-authored-by: George Pollard <porges@porg.es>
@nharper285
Reporting coverage on task start up (microsoft#3502)
cde6a19 
* Reporting coverage on task start up

* Moving metric up.
@nharper285
Remove feature flag from heartbeat metrics. (microsoft#3505)
1fb1563
@mgreisen
Update archive notice. (microsoft#3507)
c7a9827
@kananb
Add onefuzz service version to job created events (microsoft#3504)
58da7b4
tevoinea and others added 15 commits September 21, 2023 17:31
@tevoinea
Tevoinea/add version checking in local tasks (microsoft#3517)
60766e6 
* Compare task version to service version

* Swallow output when looking for appropriate name
@tevoinea
Create directories if they don't exist in the template (microsoft#3522)
e3c4a40 
* Create directories if they don't exist in the template

* fmt
@Porges
Support for retention policies on containers (microsoft#3501)
d1ccb1e 
- [x] ability to specify a retention period on a container, which applies to newly-created blobs
- [x] specify default retention periods in templates from CLI side 

There's a small breaking change to the Python JobHelper class.
@dependabot
Bump rayon from 1.7.0 to 1.8.0 in /src/agent (microsoft#3520)
7efea43 
Bumps [rayon](https://github.com/rayon-rs/rayon) from 1.7.0 to 1.8.0.
- [Changelog](https://github.com/rayon-rs/rayon/blob/master/RELEASES.md)
- [Commits](rayon-rs/rayon@rayon-core-v1.7.0...rayon-core-v1.8.0)

---
updated-dependencies:
- dependency-name: rayon
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump insta from 1.31.0 to 1.32.0 in /src/agent (microsoft#3521)
f3b7e20 
Bumps [insta](https://github.com/mitsuhiko/insta) from 1.31.0 to 1.32.0.
- [Changelog](https://github.com/mitsuhiko/insta/blob/master/CHANGELOG.md)
- [Commits](mitsuhiko/insta@1.31.0...1.32.0)

---
updated-dependencies:
- dependency-name: insta
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@nharper285
Disable repro and debug VM CLI commands. (microsoft#3494)
d2ba170 
* Disable  and  VM CLI commands.

* Formatting.

* More formatting.

* More formatting.

* Removing Repro check.
@tevoinea
Make modules case insenstive on windows (microsoft#3527)
2c8ecc9 
* Make modules and coverage allowlist case insensitive on Windows

* Tests and fmt

* PR comments

* fmt

* Debugging missing file coverage

* fmt

* Broken linux test

* Add a case insensitive transformer for better perf

* cargo fix
@tevoinea
Update windows interceptor list (microsoft#3528)
e12b41e
@tevoinea
Template creation command (microsoft#3531)
552df45 
* Tasks are selectable

* Almost there

* It works

* fmt

* remove dead code

* Remove unnecessary comments

* Improve instructions

* fix bug

* Add some dummy values for paths
@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
@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
@nharper285
Merge branch 'microsoft:main' into main
3553220
@nharper285
Merge branch 'main' of https://github.com/nharper285/onefuzz
d5ffc10
@nharper285
Merge branch 'main' of https://github.com/nharper285/onefuzz
0bbaad8
@nharper285
Removing repro test.
4209e73
@nharper285 nharper285 marked this pull request as ready for review October 9, 2023 21:07
@codecov-commenter
Copy link

Codecov Report

Merging #3560 (4209e73) into main (22caac1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3560   +/-   ##
=======================================
  Coverage   38.97%   38.97%           
=======================================
  Files         301      301           
  Lines       36848    36848           
=======================================
  Hits        14363    14363           
  Misses      22485    22485

@nharper285 nharper285 merged commit e7b0dd5 into microsoft:main Oct 9, 2023
24 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Porges Porges Porges approved these changes

@chkeita chkeita chkeita approved these changes

@AdamL-Microsoft AdamL-Microsoft AdamL-Microsoft approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
8 participants
@nharper285 @codecov-commenter @Porges @chkeita @AdamL-Microsoft @kananb @mgreisen @tevoinea