Project

General

Profile

Bug #14738

reprepro-related cron mails shouldn't be sent to sysadmins

Added by groente 2 months ago. Updated 14 days ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Infrastructure
Target version:
Start date:
09/29/2017
Due date:
% Done:

100%

QA Check:
Pass
Feature Branch:
Type of work:
Sysadmin
Blueprint:
Starter:
Affected tool:

Description

reprepro-time-based-snapshots cronjobs currently mail to tails-sysadmins, whereas tails-rm would be a more suitable destination. Same for the "Deleting vanished identifier" email from tails-update-reprepro-config.


Related issues

Blocks Tails - Feature #13242: Core work 2017Q4: Sysadmin (Maintain our already existing services) Confirmed 06/29/2017

History

#1 Updated by intrigeri 2 months ago

  • Description updated (diff)
  • Target version set to Tails_3.3

I'll try to take a look during this cycle as it should be trivial, but no big deal if that gets postponed a bit.

#2 Updated by intrigeri 2 months ago

  • Blocks Feature #13242: Core work 2017Q4: Sysadmin (Maintain our already existing services) added

#3 Updated by intrigeri 2 months ago

  • Status changed from Confirmed to In Progress
  • % Done changed from 0 to 10
  • QA Check set to Ready for QA

Hopefully fixed. I'm not 100% sure this change will be sufficient; we'll see.

#4 Updated by intrigeri 2 months ago

  • QA Check changed from Ready for QA to Dev Needed

Oops, I forgot the "Deleting vanished identifier" email from tails-update-reprepro-config.

#5 Updated by intrigeri 2 months ago

  • QA Check changed from Dev Needed to Ready for QA

intrigeri wrote:

Oops, I forgot the "Deleting vanished identifier" email from tails-update-reprepro-config.

Implemented, let's see how it goes next time a branch is deleted. I'll push a branch, wait for its APT suite to be created, and then will delete it.

#6 Updated by intrigeri 2 months ago

  • Assignee changed from intrigeri to groente
  • % Done changed from 10 to 80

Works!

groente, if you want to review this:

#7 Updated by anonym 28 days ago

  • Target version changed from Tails_3.3 to Tails_3.5

#8 Updated by groente 15 days ago

  • Assignee changed from groente to intrigeri
  • QA Check changed from Ready for QA to Info Needed

i like the consistent recipient parameter, but shouldn't it be set to instead of root or am i missing something here?

#9 Updated by intrigeri 15 days ago

  • Assignee changed from intrigeri to groente
  • QA Check changed from Info Needed to Ready for QA

groente wrote:

i like the consistent recipient parameter, but shouldn't it be set to instead of root or am i missing something here?

I disagree about setting the default value for this parameter to one of our mailing lists: I want new contributors who replicate our setup locally to receive the corresponding email, instead of sending it to us.

Keep in mind this is merely the default value. We override it in our production setup: as my merge request said, see "commit 6a891cac0981ad8ad9df3af25fa9a808bf6a636f in our manifests repo" (and perhaps learn a tiny bit about Hiera? it's a cool tool and we're using it a lot :)

#10 Updated by groente 15 days ago

  • Status changed from In Progress to Resolved
  • % Done changed from 80 to 100
  • QA Check changed from Ready for QA to Pass

well, that answers the question whether i missed something :) thanks!

#11 Updated by intrigeri 14 days ago

well, that answers the question whether i missed something :) thanks!

:)

Also available in: Atom PDF