fix(agw): Enhanced debugging in CI for agw#13423
Merged
pruthvihebbani merged 5 commits intomagma:masterfrom Aug 11, 2022
Merged
fix(agw): Enhanced debugging in CI for agw#13423pruthvihebbani merged 5 commits intomagma:masterfrom
pruthvihebbani merged 5 commits intomagma:masterfrom
Conversation
Signed-off-by: VinashakAnkitAman <[email protected]>
Contributor
|
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
Signed-off-by: VinashakAnkitAman <[email protected]>
Contributor
Contributor
Contributor
Signed-off-by: VinashakAnkitAman <[email protected]>
Signed-off-by: VinashakAnkitAman <[email protected]>
Signed-off-by: VinashakAnkitAman <[email protected]>
pull-request-size
bot
added
size/S
VinashakAnkitAman
requested review from
ardzoht,
pruthvihebbani,
rsarwad and
ulaskozat
pruthvihebbani
suggested changes
Aug 9, 2022
ardzoht
reviewed
Aug 9, 2022
pruthvihebbani
approved these changes
Aug 11, 2022
MagmaCIBot
pushed a commit
that referenced
this pull request
Aug 24, 2022
* fix(agw): Enhanced debugging in CI for agw Signed-off-by: VinashakAnkitAman <[email protected]> * Fixed CI status update for failing CI runs Signed-off-by: VinashakAnkitAman <[email protected]> Signed-off-by: VinashakAnkitAman <[email protected]> (cherry picked from commit dabdd02)
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
tmdzk
pushed a commit
that referenced
this pull request
Aug 24, 2022
* fix(agw): Enhanced debugging in CI for agw Signed-off-by: VinashakAnkitAman <[email protected]> * Fixed CI status update for failing CI runs Signed-off-by: VinashakAnkitAman <[email protected]> Signed-off-by: VinashakAnkitAman <[email protected]> (cherry picked from commit dabdd02) Co-authored-by: Ankit Kumar Aman <[email protected]>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.
Title
fix(agw): Enhanced debugging in CI for agw
Summary
During the CI run, many test cases are there which restart MME causing multiple MME log files generation in the Dev VM. However, after execution of lte integ test, we are fetching only the last MME log. This PR adds code to fetch all MME logs from Dev VM. Moreover the integ test execution is not updating the status properly after execution because of which artifact logs are not published when the CI runs are failing. This PR fixes the integ test status issue. The PR changes does the following:
Test plan
Verified with sanity and in CI
Successful CI execution log:
https://github.com/VinashakAnkitAman/magma/actions/runs/2740579763
https://github.com/VinashakAnkitAman/magma/actions/runs/2803524233
Signed-off-by: VinashakAnkitAman [email protected]