Skip to content

Conversation

@grahamalama
Copy link
Contributor

Related #645

Adds / modifies logging for a few Jira service methods

  • get_issue
  • create_issue
  • update_issue_resolution

The important things we want to check here:

  • things are logged at the appropriate level (Debug)
  • we include the rid everywhere, as this is what will help us correlate logs for an action workflow
  • We test this in a sane way

This PR is getting a temperature-check for the logging / testing patterns before I continue further

@grahamalama grahamalama requested a review from a team as a code owner November 17, 2023 17:09
@grahamalama grahamalama changed the title Better jira logging Start process to improve Jira logging Nov 17, 2023
Copy link
Contributor

@leplatrem leplatrem left a comment

Choose a reason for hiding this comment

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

👌
(you can ignore any of my comments if they don't make sense to you)

grahamalama and others added 2 commits November 21, 2023 12:32
`test_create_jira_issue_returns_list`
->
`test_create_jira_issue_when_list_is_returned`

Co-authored-by: Mathieu Leplatre <[email protected]>
@grahamalama grahamalama merged commit ff16faa into main Nov 21, 2023
@grahamalama grahamalama deleted the better-jira-logging branch November 21, 2023 22:02
@grahamalama grahamalama removed the request for review from leplatrem November 27, 2023 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants