This change has been marked as completed.

Describe the completed change (optional):

Pushed to release-5.x (93afe2b)

Summary

Add log timers and better logging around DMARC record fetching.

Review Request #14366 — Created March 11, 2025 and submitted 1 day, 6 hours ago

Information

Owner

chipx86*

Repository

Djblets

Branch

release-5.x

Bugs

Depends On

~~14365~~

Reviewers

Groups

djblets

People

Description*

In the event of a DNS issue, it's helpful to know how long a process
took to attempt the resolution, and to know what the failure was. To
help with this, we now wrap the DNS fetching with log_timed() and have
logging for different failure types.

Testing Done

Unit tests pass.

Commits

Summary	ID
Add log timers and better logging around DMARC record fetching. In the event of a DNS issue, it's helpful to know how long a process took to attempt the resolution, and to know what the failure was. To help with this, we now wrap the DNS fetching with `log_timed()` and have logging for different failure types.	1981f63955518750456003b41b8a5a2d0ceea7be

Description	From	Last Updated
There are no open issues

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Ship it!

```
Ship It!
```

Change Summary:

Included the new timer trace ID in the result log messages.

Commits:

	Summary	ID
	Add log timers and better logging around DMARC record fetching. In the event of a DNS issue, it's helpful to know how long a process took to attempt the resolution, and to know what the failure was. To help with this, we now wrap the DNS fetching with `log_timed()` and have logging for different failure types.	d17f120f18cd84f6b7c34d8f7dd0de2f4f426a56
	Add log timers and better logging around DMARC record fetching. In the event of a DNS issue, it's helpful to know how long a process took to attempt the resolution, and to know what the failure was. To help with this, we now wrap the DNS fetching with `log_timed()` and have logging for different failure types.	1981f63955518750456003b41b8a5a2d0ceea7be

Diff:

Revision 2 (+78 -24)

Show changes

djblets/mail/dmarc.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-5.x (93afe2b)