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.

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

[WOR-1385] Lower FP monitor log msgs to warn #2649

Merged
merged 1 commit into from
Dec 4, 2023
Merged

[WOR-1385] Lower FP monitor log msgs to warn #2649

merged 1 commit into from
Dec 4, 2023

Conversation

aherbst-broad
Copy link
Contributor

Ticket
These log messages do not seem actionable but emit error spam to sentry.

PR checklist

  • Include the JIRA issue number in the PR description and title
  • Make sure Swagger is updated if API changes
    • ...and Orchestration's Swagger too!
  • If you changed anything in model/, then you should publish a new official rawls-model and update rawls-model in Orchestration's dependencies.
  • Get two thumbsworth of PR review
  • Verify all tests go green, including CI tests
  • Squash commits and merge to develop (branches are automatically deleted after merging)
  • Inform other teams of any substantial changes via Slack and/or email

@aherbst-broad aherbst-broad requested a review from tlangs December 4, 2023 14:45
@aherbst-broad aherbst-broad marked this pull request as ready for review December 4, 2023 14:45
@aherbst-broad aherbst-broad requested review from a team, cahrens and marctalbott and removed request for a team December 4, 2023 14:45
@aherbst-broad
Copy link
Contributor Author

502 from orch :(
jenkins retest

@aherbst-broad
Copy link
Contributor Author

jenkins retest

@aherbst-broad aherbst-broad merged commit 636c101 into develop Dec 4, 2023
11 checks passed
@aherbst-broad aherbst-broad deleted the arh-wor-1385-fp-logs branch December 4, 2023 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marctalbott marctalbott marctalbott approved these changes

@cahrens cahrens cahrens approved these changes

@tlangs tlangs Awaiting requested review from tlangs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@aherbst-broad @cahrens @marctalbott