Distributed tracing update #1611
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
release_2019_10_07_after_bump_version
…t message_sender.c and add UT for it.
…nug 800-min limit
Move longhaul timeoutInSeconds to correct place in yml, change to a snug 800-min limit
c-ares e2e testing
* Check correct timeout time * Fix MQTT timeout issues Fix regression introduced on overly aggressive MQTT timeouts. To fix and cleanup code a bit: * Remove concept of enqueue time and instead have a message creation time. * Set this message creation time ALWAYS, regardless of whether it's a PUT or GET. * Check this publish time on timeouts. * Create a common function to create TWIN requests and cleanup some dup'd code. On test side: * Fix up UT's required of course * Change the default timer that UT uses to start at 30 minutes, not 0 minutes. If we had done this it would've caught initial bug where a message creation time was never init'd but close enough to 0 to have tests pass.
…l fast and generate a better log. [issue #1220]
…tforms The longhaul tests on Windows have been cancelled by devops after 12h, because not all Windows LH tests run in parallel. There are 10 tests, but on our scripts we restrict the max number of parallel tests to 8. We could fix this in different ways, but this is the cheapest and most straightforward. Also, on Linux we run up to 16 tests in parallel, so increasing by that much on Windows should be acceptable.
Fix the cancellation of the longhaul jobs in the gate for Windows platforms
…ereceiver state cb (issue #1254)
Handle unexpected states in dev provisioning messagesender and messagereceiver state cb (issue #1254)
#1274) * added a define that can be used as an option to reduce memory footprint on embedded devices Note: esp8266 may run out of memory without this define. Add in #defines for loading only specific certs. * Update linux_c_option_test.sh Added option to test build with -Duse_baltimore_cert=ON * Update certs.c Update to certificate descriptions. Name conventions set. * Update CMakeLists.txt pass cert defines down to compiler
Remove incorrect const qualifier in serializer methodreturn
Updating references from Device Explorer to Azure IoT Explorer and some minor documentation changes.
… treated (gh#1309) Explicitly explain how the memory used by IOTHUB_MESSAGE_HANDLE is copied by the *_SendEventAsync function, allowing the calling application to destroy the message argument right after the *_SendEventAsync function returns.
Improve *SendEventAsync documentation about how the message handle is treated (gh#1309)
RIoT Update to utilize new Crypto lib
IoTHub has changed the convention for indicating a device's ModelId. Instead of passing `digital-twin-model-id`, we now pass `model-id`. This succeeds on latest canary deployments.
Implement a simple PnP thermometer sample that implements the interface defined [here](https://github.com/Azure/opendigitaltwins-dtdl/blob/master/DTDL/v2/samples/Thermostat.json).
Update submodules
* sendPendingGetTwinRequests only after the suback is received * Update unit tests for fix on MQTT transport/sendPendingGetTwinRequests Co-authored-by: Ewerton Scaboro da Silva <ewertons@microsoft.com>
* Add TemperatureController model for a more complex PnP sample. * Expand the common library specifically with PnP parsing helpers. * Fix up fit and finish issues in the already checked in simple thermostat, so it's a bit more inline stylistically with rest of files under `./samples/pnp`.
* Update arduino-esp.yml for Azure Pipelines * Update esp_arduino_uart_interface.py * Update serial_connect.py * Delete mxchip_settings.py * try save env variable * ok ok ok * Update azure_test_firmware_errors.py
release_2020_07_19_after_bump_version
Update Long Term Support information
Fix format string passed to strftime to use desired format. Matches what was already in the more complex component sample and embedded C samples.
Fix incorrect handle name in telemetry sample (#1599)
Adds DPS support to PnP temperature controller sample.
Naming functions and file name with `helper` wasn't particularly helpful. Cleaning up.
Further cleanup of PNP samples, most importantly returning an empty JSON blob on commands (hub/explorer aren't happy otherwise the way C SDK works).
danewalton-msft
approved these changes
Jul 31, 2020
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.
Checklist
devdocfolder and added or modified requirements.masterbranch.masterbranch prior to submission and re-merged as needed after I took any feedback.Reference/Link to the issue solved with this PR (if any)
Description of the problem
public-preview branch hasn't been updated for months, lacking critical fixes.
Description of the solution