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

Capture more content in parsed summary for all parsers #87

Open
glennmatthews opened this issue Sep 27, 2021 · 4 comments
Open

Capture more content in parsed summary for all parsers #87

glennmatthews opened this issue Sep 27, 2021 · 4 comments

Comments

@glennmatthews
Copy link
Contributor

Environment

  • circuit_maintenance_parser version: 2.0.1

Proposed Functionality

All parsers should err on the side of including more information in the summary field versus trying to narrowly extract a single sentence or paragraph from the overall notification.

Use Case

Notifications often include many additional details beyond what we're currently capturing in most summary fields. In integrating the output of this parser library with other systems, it's better to preserve information rather than discarding it.

@glennmatthews
Copy link
Contributor Author

Optionally could add an entirely new field, full_message or something to the Maintenance model, if we want to keep the current behavior of summary as an extremely brief excerpt.

@chadell
Copy link
Collaborator

chadell commented Oct 1, 2021

The idea of the summary field is to capture any other data related to the maintenance that is not one of the key fields, so it's a matter of the parsers to include more information into this field.
I mean, there is no limitation from the library, it's up to each parser implementation.
Do you have specific parsers that should be updated? or it's just a guideline for new parsers?

@glennmatthews
Copy link
Contributor Author

If nautobot/nautobot-app-circuit-maintenance#116 can be implemented to capture the email "body" rather than a truncated set of headers into the RawNotification, I think that would meet my needs as an alternative / better solution to this issue in general.

Specifically, for Lumen notifications, I believe I've seen that we capture the "Summary" (which is the same across all updates to a scheduled maintenance) but not the "Updates" that are separately included in the email, for example:

Summary:

Lumen intends to carry out internal maintenance within its network. This has been designated as ESSENTIAL. The nature of this work is to perform a traffic-affecting circuit re-route. Work is required to augment infrastructure to ensure the continued integrity of the network. During the maintenance window you may experience one period of outage to the maximum specified below.

Lumen would like to apologise for any inconvenience caused by this maintenance.

Updates:

2021-10-01 02:16:04 GMT - The scheduled maintenance window 12345678-1 has ended.
2021-09-30 22:00:24 GMT - The scheduled maintenance work has begun.

I can verify this behavior and submit a PR for the Lumen parser specifically when I find some time to do so.

@chadell
Copy link
Collaborator

chadell commented Oct 1, 2021

I think that if the info that you are interested can be extracted from the notification, and stored in the object.summary it's always better, and it's relatively easy to do with a small change on the parser to take into account also the Updates text along with the Summary.

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

No branches or pull requests

2 participants