Page MenuHomeSoftware Heritage

Add a script for a 'monthly roadmap report' bot email
Needs ReviewPublic

Authored by vlorentz on Thu, Oct 14, 11:24 AM.

Details

Reviewers
ardumont
Group Reviewers
System administrators
Reviewers
Test Plan

Not tested :|

Diff Detail

Repository
rSPSITE puppet-swh-site
Branch
monthly-report
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24457
Build 38162: arc lint + arc unit

Event Timeline

vlorentz created this revision.

Good idea.

Do we have the same for the weekly report already?

site-modules/profile/files/weekly_report_bot/monthly-report-bot
18

You can make it parametrized on the year as well.
That way, no need to update it next year.

Do we have the same for the weekly report already?

yes, we do, same link as the comment [1]

[1] https://forge.softwareheritage.org/source/puppet-swh-site/browse/production/site-modules/role/manifests/swh_sysadmin.pp$28

site-modules/profile/manifests/monthly_report_bot.pp
22

That profile must be installed in the role somewhere otherwise, that won't do anything.
I'd say probably pergamon again [1]

[1] https://forge.softwareheritage.org/source/puppet-swh-site/browse/production/site-modules/role/manifests/swh_sysadmin.pp$28

site-modules/profile/files/weekly_report_bot/monthly-report-bot
18

actual suggestion with a diff ;)

I'm fine with the idea.
I've suggested a couple of fix, one actually be blocking.

olasd added inline comments.
site-modules/profile/files/weekly_report_bot/monthly-report-bot
18

The 2022 roadmap won't exist on the first month of 2022, so I guess it's fine to keep the year hardcoded.

site-modules/profile/files/weekly_report_bot/monthly-report-bot
18

Unless someone actually installs an empty template very early in the year.
Either way that's work so fine with either road we are taking.

site-modules/profile/files/weekly_report_bot/monthly-report-bot
18

Either way that's work so fine with either road we are taking.

Either way is fine with me*

site-modules/profile/files/weekly_report_bot/monthly-report-bot
18

once we have at least two roadmaps, we will probably add an index of roadmaps in the doc, and the email can link to that.