Skip to content
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

(maint) Emit more debug output when mod sync fails #1334

Conversation

justinstoller
Copy link
Member

Please add all notable changes to the "Unreleased" section of the CHANGELOG in the format:

- (JIRA ticket) Summary of changes. [Issue or PR #](link to issue or PR)

@justinstoller justinstoller requested a review from a team as a code owner March 15, 2023 20:21
@justinstoller justinstoller reopened this Mar 15, 2023
updated = mod.sync
updated_modules << mod.name if updated
rescue Exception => e
logger.debug _("Module %{mod_name} failed to synchronize due to %{message}") % {mod_name: mod.name, message: e.message}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but this should be at error or warn level so that the information is always visible without having to re-run with debug enabled (difficult to do in Code Manager, and doubly so with ephemeral environments like Impact Analysis branches).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

error level seems the most appropriate as the sort of occurrence causes r10k to exit with a non-0 status, which is fatal to the code deployment process.

@github-actions
Copy link

This PR has been marked stale because it has had no activity for 60 days. If you are still interested in getting this merged, please comment and we'll try to move it forward. Otherwise, it will be closed in 7 days.

@justinstoller
Copy link
Member Author

Couldn't reopen this PR so I created #1347

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants