DASB-213 - Fixed issue with getting trace when no spans have been provided #36
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
|
uil9
approved these changes
Jan 19, 2024
mkrystof
added a commit
that referenced
this pull request
Feb 14, 2024
* correctedBranchRepoNaming (#16) * DASB-146 - Dev rework reports (#15) * Report rework in progress * Reference constant for partition key name * Updates to new report creation design --------- Co-authored-by: Matt B Krystof <[email protected]> * Updated postman to use new report design (#19) Co-authored-by: Matt B Krystof <[email protected]> * changedPubPlugin (#20) * DASB-151 - Dev feature load testing (#17) * DASB-151 - Initial version of Python script to do load testing * Updates with load testing scripts to align with new report create design --------- Co-authored-by: Matt B Krystof <[email protected]> * Fix build issues with health endpoint (#18) Co-authored-by: Matt Krystof <[email protected]> * addedPublishing (#21) * Publishing rewrite (#22) * addedPublishing * movingPublishBackend * removePub (#23) * reworkAuth (#24) * DASB-158 - Dev trace rework (#25) * Trace rework, code cleanup and refactoring * Code cleanup * Health check code cleanup * Fixed issue with stage report items count check * Rework of getting traces * Added logic for grabbing optional tags when creating spans * Added missing function header to JSON util method * First cut at combined status * Syncing up code changes * Updates to status model * Send a trace for upload with the load testing script * Added traces for remaining stages to load test script * Updated excluded span tags, code cleanup * Removed output of the service bus connection str in load test script --------- Co-authored-by: Matt B Krystof <[email protected]> * DASB-142 - DEX logger to Processing Status API (#26) * Updates to processing status postman (#27) * Dev cicd finalize (#28) * addAzureInfo * setCITrigger * Dev change gradle asp (#30) * renamedASP * correctedInputforUT * Updated api specification (#31) * Added service bus unit tests, don't escape quote reports JSON, updated load testing (#32) * Initial version of readme for report schemas (#33) Co-authored-by: Matt B Krystof <[email protected]> * Adding api for teh notifications api * Upating tag to notifications * Updating notification open api spec to make fields required and add correct schema for the api calls * DASB-213 - Fixed issue with getting trace when no spans have been provided (#36) Co-authored-by: Matt B Krystof <[email protected]> * Adding type of enum to statusType * Adding enum for status type and adding response code for 403 * DASB-188 - Jaeger health check (#38) * DASB-217 - Temporary implementation of getting trace by uploadId (#40) * DASB-224 - Trace endpoint rework (#41) * Updated openapi spec * Changed implementation of how trace interface works * Adding new function app for notifications api * DASB-224 - Dev feature trace endpoint rework (#43) * Updated openapi spec * Changed implementation of how trace interface works * Added endpoint to query for a span for a given uploadId and stageName * Provide span and trace id even when span is completed * Cleaning open telemetry related config * Merging conflicts * Removing methods for Service bus * DASB-224 - Dev feature trace endpoint rework (#45) * Updated openapi spec * Changed implementation of how trace interface works * Added endpoint to query for a span for a given uploadId and stageName * Provide span and trace id even when span is completed * Updated load test python script for new span start/stop interface * Updated postman for new interface and span retrieval endpoint * Added retry mechanism for searching a trace for a span of given stage name * Adding code to hadle subscriptions and unsibscriptions for websocket and email using in memory caches * Dev feature junit test (#46) * DASB-188 - Jaeger health check * DASB-204 - Junit tests for Function apps * DASB-204 - Junit tests for Function apps * DASB-204 - Junit tests for Function apps * DASB-204 - Added Jacoco for code coverage * DASB-235 - Fix for health check (#47) * DASB-242 - Dev feature add metadata verify (#48) * Added initial version of metdata-verify stage to upload status projection * Format timestamps with milliseconds included * Added missing fields when only upload report is the metadata-verify report * Added missing metadata from report when metadata verify report only --------- Co-authored-by: Matt B Krystof <[email protected]> * Code cleanup * Code cleanup to make sonar happy * Code cleanup and formatting * Adding model and removing message class * Code refactoring to lift the return up * Removing notification type argument from Unsubscribe method * Updating open api spec * Adding postman test csript * Removing HttpTrigger annotation * Adding unit test for function wrapper and requests * DASB-242 - Dev feature add metadata verify (#50) * Added initial version of metdata-verify stage to upload status projection * Format timestamps with milliseconds included * Added missing fields when only upload report is the metadata-verify report * Added missing metadata from report when metadata verify report only * Code cleanup * Added missing class headers, added comments, removed commented out code --------- Co-authored-by: Matt B Krystof <[email protected]> * DASB-247 - Fixed service bus JSON longs being made doubles (#51) Co-authored-by: Matt B Krystof <[email protected]> * Adding unit test cases * Addressing comments * DASB-215 - Updated open spec api and readMe documents. (#49) * DASB-215 - Updated open spec api and readMe documents. * DASB-215 - Updated open spec api and readMe documents. * DASB-215 - Updated open spec api and readMe documents. * DASB-252 - Fix bug where issues has to be missing or null to register as none (#52) Co-authored-by: Matt B Krystof <[email protected]> * DASB-251 - Added support for metadata passed in upload status (#53) Co-authored-by: Matt B Krystof <[email protected]> * Updated to use env vars for the deployment (#54) --------- Co-authored-by: David Payne <[email protected]> Co-authored-by: Matt B Krystof <[email protected]> Co-authored-by: Surya Talla <[email protected]> Co-authored-by: Nagpal <[email protected]> Co-authored-by: Purnima Nagpal <[email protected]>
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.
No description provided.