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

test: error handling for arc conformance #799

Merged
merged 40 commits into from
Feb 24, 2022
Merged

Conversation

nilekhc
Copy link
Contributor

@nilekhc nilekhc commented Feb 15, 2022

Reason for Change:

This PR adds additional error handling while running conformance test

Requirements

  • squashed commits
  • included documentation
  • added unit tests and e2e tests (if applicable).

Issue Fixed:

Does this change contain code from or inspired by another project?

  • Yes
  • No

If "Yes," did you notify that project's maintainers and provide attribution?

Special Notes for Reviewers:

* feat: implements outbound proxy support for arc extension

Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
@nilekhc nilekhc requested a review from aramase February 15, 2022 10:06
@codecov-commenter
Copy link

codecov-commenter commented Feb 15, 2022

Codecov Report

Merging #799 (bef9625) into master (132553e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #799   +/-   ##
=======================================
  Coverage   59.44%   59.44%           
=======================================
  Files           7        7           
  Lines         868      868           
=======================================
  Hits          516      516           
  Misses        316      316           
  Partials       36       36           

Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
…anup.

Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
arc/conformance/plugin/arc_conformance.sh Outdated Show resolved Hide resolved
arc/conformance/plugin/arc_conformance.sh Outdated Show resolved Hide resolved
arc/conformance/plugin/arc_conformance.sh Outdated Show resolved Hide resolved
arc/conformance/plugin/arc_conformance.sh Outdated Show resolved Hide resolved
arc/conformance/plugin/arc_conformance.sh Outdated Show resolved Hide resolved
arc/conformance/plugin/arc_conformance.sh Outdated Show resolved Hide resolved
arc/conformance/plugin/arc_conformance.sh Outdated Show resolved Hide resolved
arc/conformance/plugin/arc_conformance.sh Show resolved Hide resolved
arc/conformance/plugin/arc_conformance.sh Outdated Show resolved Hide resolved
@nilekhc nilekhc requested a review from aramase February 23, 2022 11:03
Copy link
Member

@aramase aramase left a comment

Choose a reason for hiding this comment

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

Just one small nit!

Feel free to ping after this temporary change is reverted and I can approve the PR.

arc/conformance/plugin/arc_conformance.sh Outdated Show resolved Hide resolved
@aramase aramase changed the title Error handling test: error handling for arc conformance Feb 23, 2022
Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
@nilekhc nilekhc requested a review from aramase February 24, 2022 07:15
@nilekhc
Copy link
Contributor Author

nilekhc commented Feb 24, 2022

Just one small nit!

Feel free to ping after this temporary change is reverted and I can approve the PR.

I have reverted temporary change. PTAL when you get a change. Thanks.

@nilekhc nilekhc merged commit 20ddcf7 into Azure:master Feb 24, 2022
@nilekhc nilekhc deleted the error-handling branch February 24, 2022 18:15
@nilekhc nilekhc restored the error-handling branch February 24, 2022 19:37
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.

4 participants