-
Notifications
You must be signed in to change notification settings - Fork 217
ADD userguide/managingApp/deployment-trace #6413
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
ADD userguide/managingApp/deployment-trace #6413
Conversation
Signed-off-by: rahulshendre <rahulshendre789@gmail.com>
docs/content/en/docs-v1.0.x/user-guide/managing-application/deployment-trace.md
Outdated
Show resolved
Hide resolved
Signed-off-by: Rahul Shendre <144231863+rahulshendre@users.noreply.github.com>
Updated the title and description for clarity. Improved wording in several sections for better understanding. Signed-off-by: Eeshaan Sawant <100678386+eeshaanSA@users.noreply.github.com>
|
Hey @rahulshendre, I commited my changes, but I cant see them here. Can you see them? Here is the link to the changes I made: rahulshendre@ba4c012 Weird. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #6413 +/- ##
==========================================
+ Coverage 28.87% 37.07% +8.20%
==========================================
Files 560 75 -485
Lines 59955 6500 -53455
==========================================
- Hits 17313 2410 -14903
+ Misses 41321 3952 -37369
+ Partials 1321 138 -1183
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@eeshaanSA, I can see the changes. |
|
@rahulshendre @eeshaanSA this page is Deployment Trace while the title and description is Deployment Chain? 👀 |
|
Thanks for pointing this out @khanhtc1202. In v0, Deployment Chain refers to the postSync orchestration feature. Using “Deployment Trace” seemed appropriate here, what do you think? |
|
Hey, sorry for missing it. The title and the body needs to be changed. The files changed, however, only add docs for Deployment Trace. @rahulshendre please keep a look out for such mistakes, I do miss it sometimes. 😃 |
|
Thanks @eeshaanSA, I have updated the PR title and description to Deployment Trace. |
What this PR does:
File by file commit from the bigger PR - #6345
Adds the “Deployment trace” page under User Guide → Managing Application.
Related to #6395
Why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: