Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[release/0.9] Backport test/pkg directory #1913

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

helsaawy
Copy link
Contributor

Backport testing support functions in the test/pkg directory. This includes changes from:

Rather than cherry-pick them, only changes to test/pkg are included, since tests themselves will require significant changes to bring up-to-date.

The goal is to expose testing functions so that tests can be moved out of the repo.

Updated go version in test to 1.8, as required by test/pkg/flag.

Backport testing support functions in the `test/pkg` directory.
This includes changes from:
 - microsoft#1536
 - microsoft#1704
 - microsoft#1853
 - microsoft#1893

Rather than cherry-pick them, only changes to `test/pkg` are included,
since tests themselves will require significant changes to bring
up-to-date.

The goal is to expose testing functions so that tests can be moved out
of the repo.

Updated go version in `test` to 1.8, as required by `test/pkg/flag`.

Signed-off-by: Hamza El-Saawy <hamzaelsaawy@microsoft.com>
@helsaawy helsaawy requested a review from a team as a code owner September 20, 2023 19:16
@@ -0,0 +1,10 @@
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
<description>containerd-shim-runhcs-v1</description>
Copy link
Contributor

@yyatmsft yyatmsft Sep 27, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

containerd-shim-runhcs-v1

update description?
#Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i totally missed that when i made the original PR
since its also in main, id rather get in a PR to main first that updates the description, then backport that to 0.9 and 0.11, if thats allright

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(#1919)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah back porting makes more sense

@helsaawy helsaawy merged commit e3d012a into microsoft:release/0.9 Sep 28, 2023
14 checks passed
@helsaawy helsaawy deleted the release0.9-test-update branch September 28, 2023 17:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants