Page MenuHomePhabricator

enable email for tickets in domains project?
Closed, ResolvedPublic

Description

@RobH so we just got a project created to handle the tickets we formerly had in the RT queue "domains" which wasn't imported.

(T87465)

what are your thoughts about getting MarkMonitor mail on there? can we do this like we did for the datacenter projects for vendors?

Event Timeline

Dzahn raised the priority of this task from to Needs Triage.
Dzahn updated the task description. (Show Details)
Dzahn added subscribers: Dzahn, RobH.
Dzahn renamed this task from enable email for tickets in domains project to enable email for tickets in domains project?.Feb 6 2015, 9:20 PM
Dzahn updated the task description. (Show Details)
Dzahn set Security to None.
Dzahn triaged this task as Medium priority.Feb 6 2015, 10:43 PM
gerritbot added a subscriber: gerritbot.

Change 189140 had a related patch set uploaded (by Dzahn):
phab: direct_comments_allowed for Domains tickets

https://gerrit.wikimedia.org/r/189140

Patch-For-Review

there's a patch now trying to enable the direct_comments, please see above. i'm currently not sure why jenkins dislikes it

Change 189140 merged by Dzahn:
phab: direct_comments_allowed for Domains tickets

https://gerrit.wikimedia.org/r/189140

reason was that 'Domains' was capitalized. but it's still unclear why that was considered a syntax error by operations-puppet-pplint-HEAD.

anyways, project tag was renamed to 'domains' in phab. also added wikimedia.org to be able to test.

https://gerrit.wikimedia.org/r/#/c/190383/

@RobH wanna test?

reason was that 'Domains' was capitalized.

That was intentional, because by default projects are capitalized in Phab (except for when there are good reasons, like package names).

anyways, project tag was renamed to 'domains' in phab.

So the next person renaming the project in Phab and not being aware of this conversation will break this again? Might be worth to add some notice to the project description why it needs to be lower case?

That was intentional, because by default projects are capitalized in Phab (except for when there are good reasons, like package names).

none of the other ops tags, operations, ops-eqiad, ops-codfw etc are capitalized. but maybe they all need to be renamed then. i dunno, but there is a good reason, it lets jenkins vote down my puppet change :p

Might be worth to add some notice to the project description why it needs to be lower case?

done

@Dzahn: Do you know if there is something left to do here?

chasemp claimed this task.

Closing as inactive and AFAIK it looks like what was wanted is accomplished. @Dzahn, please reopen if I'm mistaken.

It should be resolved, it just wasn't technically tested. the difference to other queues where it works and has been tested is that it specifies domain names instead of just *. The real test would be if somebody from MarkMonitor would reply by mail into a ticket. But it's ok that it's closed as inactive for now.