refactor: make WG summary view into a task #7529
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This moves the work in the
wg_summary_by_area
andwg_summary_by_acronym
views into the newgenerate_wg_summary_files_task()
. The views are refactored to pass the file through instead of generating it on request.The old views were used by
Night-runner
and run nightly. I've made these hourly instead because the task is not terribly expensive and this matches the handling of the WG charter file generation. It's easy to change it to daily, though.Generating these files is fast enough we could refactor further and generate the files both in a task and in the view instead of doing the file pass-through. For the charter files, we had caching in place that meant we were effectively serving the generated files anyway. For the summary files, there was no such caching so this is more of a change. Unless there's a strong need for it, though, I'd rather keep these views the same.
Fixes https://github.com/ietf-tools/infra-cloud/issues/204