Adding retries to the DPS test. #2250
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ericwolz
reviewed
Mar 1, 2022
ericwolz
reviewed
Mar 1, 2022
...oning_client/tests/prov_x509_client_openssl_engine_e2e/prov_x509_client_openssl_engine_e2e.c
Outdated
Show resolved
Hide resolved
ericwolz
approved these changes
Mar 1, 2022
Co-authored-by: Eric Wolz <ericwol@microsoft.com>
jspaith
added a commit
that referenced
this pull request
Mar 4, 2022
* longhaul: handle IoTHubDeviceTwin_GetTwin network error (#2189) * handle IoTHubDeviceTwin_GetTwin network error * pr feedback * Fix valgrind issues in E2E tests (#2192) * handle IoTHubDeviceTwin_GetTwin network error * pr feedback * Fix valgrind issues in E2E tests * update RIoT to update bouncycastle dependency (#2197) * Bump version from 1.8.0 to 1.9.0. Add LTS Table entry for LTS_01_2022_Ref01 (#2199) * update RIoT to update bouncycastle dependency (#2202) * update Component Governance for pip dependencies (#2203) * add codeowner file (#2204) * Add vaavva to CODEOWNERS (#2208) * Refactor of device twin e2e tests (#2158) * shutdown of AMQP - logging causes crash due to status_description is NULL (#2210) #2200 * Update CMake Linux samples to remove warnings (#2212) * add ios notice for authentication (#2217) * Add macros includes to the build include directories (#2218) * added telemetry timeout filter to statistics (#2221) * added telemetry timeout filter to statistics Added network tracing for get twin tests * fix build break * Ericwol/longhaul3 (#2223) * added telemetry timeout filter to statistics Added network tracing for get twin tests * fix build break * pr feedback * Improve blob unit test performance (#2225) * Added documentation on device application CMAKE (#2224) * Added documentation on device application CMAKE * pr feedback * rename cmake doc * remove old hsm devices from iot hub (#2232) * remove dash from device model ID (#2237) * Adding X509 PEM support for Provisioning Client (#2229) * Adding DPS X509 public APIs. Splitting RIOT vs X509 tests. * Adding UT for x509 HSM. * Adding E2E tests. * Adding OpenSSL ENGINE tests for DPS. * Adding DevOps infra. * Doc changes. * Apply suggestions from code review Co-authored-by: Eric Wolz <ericwol@microsoft.com> * Fixing cmake integration issue * Fixing vsts job name. * Fixing Windows build. Removing DPS TPM and SAS for now. * Adding HSM type configurations for all HSMs. * Enabling DPS client by default. * Addressing feedback around const and malloc/free. * Fixing prov service client comment bug. Disabling prov_client on XCode. * Fixing x509 sample when using trusted certificates. * Fixing options test * Adding test result publishing to all VSTS jobs. * Changing test scripts to output ctest Test.xml logs. * Disabling Provisioning Service Client tests failing on OSX. * Fixing build scripts and test yaml definition. * Apply suggestions from code review Co-authored-by: Dane Walton <dawalton@microsoft.com> * Resolving PR comments. * Disabling two SFC tests failing leak checking. Tracked by #2238 Co-authored-by: Eric Wolz <ericwol@microsoft.com> Co-authored-by: Dane Walton <dawalton@microsoft.com> * Test fixes (#2240) * Removing test publishing for jobs not running tests. * Fixing Twin SFC test memory leak. * Permanently disabling Provisioning Service Client for Apple. * Removing the OpenSSL Valgrind workaround. * Adding E2E service client module creation/update retries. * Add more e2e tests service client retries (#2241) * Adding e2e test method invoke retry. * Removing unused 'edison' cmake config. * Fixing Windows build. * Increasing the DPS E2E randomized start delay between 0 and 1min. * Addressing PR feedback. * Updating Parson to v1.3.1. (#2242) * Test stabilization (#2245) * Increasing methods E2E device connect timeout. * Initializing seed for Prov E2E random back-off algorithm. * Update setting-up-vcpkg documentation (#2246) * Adding retries to the DPS test. (#2250) * Adding retries to the DPS test. * Apply suggestions from code review Co-authored-by: Eric Wolz <ericwol@microsoft.com> Co-authored-by: Eric Wolz <ericwol@microsoft.com> * Adding -Wstrict-prototypes gate validation (#2251) * Partial fixes for -Wstrict-prototypes * Adding strict-prototypes gate check. * Updating submodules. * Update CMakeLists.txt Co-authored-by: Eric Wolz <ericwol@microsoft.com> Co-authored-by: Eric Wolz <ericwol@microsoft.com> * Add instruction to sign certs with intermediate CA (#2186) * Add instruction to sign certs with intermediate cA Default method "create_device_certificate" will create a device certificate which is chained up to the root certificate directly. In group enrollment scenario of Device Provisioning Service, users may want the device certs to be chained up to the intermediate certificate. So "create_device_certificate_from_intermediate" should be noted. * Adding multi-level client certificate documentation * Addressing PR feedback Co-authored-by: CIPop <Cristian.Pop@microsoft.com> * Fix merge typo && fix issue found by more gate tests brought in by merge Co-authored-by: Eric Wolz <ericwol@microsoft.com> Co-authored-by: Mollie Munoz <mollie.munoz@microsoft.com> Co-authored-by: Dane Walton <dane.walton@microsoft.com> Co-authored-by: Cristian Pop <CIPop@users.noreply.github.com> Co-authored-by: Dane Walton <dawalton@microsoft.com> Co-authored-by: Hiroki Nakagami <nkgami@gamid.net> Co-authored-by: CIPop <Cristian.Pop@microsoft.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Precondition failed.server error caused by using the same identity on two test machines at the same time.The provisioning tests failed 3 or 4 times over the weekend (~98% test pass rate over the last 7 days).