added exception handling for main async loop

[goern]

Signed-off-by: Christoph Görn goern@redhat.com

Related Issues and Dependencies

DNS resolution problems on cluster lead to failed handling of webhooks received.

This introduces a breaking change

• Yes
• No

[todo]

add a error/exception metric inc here, and in general a metric exporter

sefkhet-abwy/aicoe/sesheta/review_manager.py

Lines 407 to 408 in 8c95bc9

	# TODO add a error/exception metric inc here, and in general a metric exporter
	_LOGGER.exception(gai)

---

This comment was generated by todo based on a TODO comment in 8c95bc9 in #72. cc @goern.

[harshad16]

lgtm, thanks

[sesheta]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: harshad16
To complete the pull request process, please assign after the PR has been reviewed.
You can assign the PR to them by writing /assign in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment