feat: added support for in app logging archival from CI runner #3467
Conversation
This file contains hidden or 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
kripanshdevtron
previously approved these changes
Jun 7, 2023
|
Kudos, SonarCloud Quality Gate passed!
|
kripanshdevtron
approved these changes
Jun 8, 2023
gireesh-naidu
pushed a commit
that referenced
this pull request
Jun 16, 2023
* archive logs sync * added type for cdstage event * test * cleanup * adding log file path to db on trigger * cleanup * cleanup * cleanup * cleanup * exposed pod name in cd workflow details * log path fix * fix * addresing pr comments * addresing pr comments2 * addresing pr comments3 * fix * Update CiHandler.go
adi6859
pushed a commit
that referenced
this pull request
Jun 16, 2023
* archive logs sync * added type for cdstage event * test * cleanup * adding log file path to db on trigger * cleanup * cleanup * cleanup * cleanup * exposed pod name in cd workflow details * log path fix * fix * addresing pr comments * addresing pr comments2 * addresing pr comments3 * fix * Update CiHandler.go
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.
Description
Fixes #
Added support and logic to handle log location based on the key we provide for both CI and CD workflows and passed additional data to CI runner request
How Has This Been Tested?
Manual testing with CI runner changes
Checklist:
Does this PR introduce a user-facing change?