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.

Sign up for GitHub

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

Jump to bottom

chore: adding logs in chart repo code #3810

Merged

iamayushm merged 1 commit into main from chart-repo-logs

Aug 22, 2023

Contributor

iamayushm commented Aug 22, 2023 •

edited

Description

Adding logs in chart repo save flow for debugging purpose. In some clusters there is issue in saving chart repo. These logs will help in debugging that issue.

Checklist:

The title of the PR states what changed and the related issues number (used for the release note).
Does this PR requires documentation updates?
I've updated documentation as required by this PR.
I have performed a self-review of my own code.
I have commented my code, particularly in hard-to-understand areas.
I have tested it for all user roles.
I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?


          adding logs in chart repo save flow

e569391

prakarsh-dt added the PR:Ready-to-Review label

sonarcloud bot commented Aug 22, 2023

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

vikramdevtron approved these changes

View reviewed changes

iamayushm merged commit 8d4389f into main

7 checks passed

iamayushm deleted the chart-repo-logs branch

August 22, 2023 08:52

kamal-devtron mentioned this pull request

SYNC: OSS sync for 12c92dcd kamal-test/devtron-enterprise#2

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment