Cumin: PuppetDB, fail better if regex are used on resource parameters
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Volans
	Apr 4 2017, 2:33 PM

Description

PuppetDB APIv3 doesn't support to use regex while querying resource parameters.

Let Cumin detect this and fail with a nice message that tells the user why instead of a 400 from PuppetDB

	Subject	Repo	Branch	Lines +/-
	PuppetDB backend: forbid resource's parameters regex	operations/software/cumin	master	+7 -0
	PuppetDB backend: consistently use InvalidQueryError	operations/software/cumin	master	+12 -12

Mentioned In: rCUMIN42a336958564: PuppetDB backend: consistently use InvalidQueryError
rCUMIN094e9ed43765: PuppetDB backend: forbid resource's parameters regex

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 4 2017, 2:33 PM

Change 346301 had a related patch set uploaded (by Volans):
[operations/software/cumin@master] PuppetDB backend: consistently use InvalidQueryError

Change 346302 had a related patch set uploaded (by Volans):
[operations/software/cumin@master] PuppetDB backend: forbid resource's parameters regex

Change 346301 merged by jenkins-bot:
[operations/software/cumin@master] PuppetDB backend: consistently use InvalidQueryError

Change 346302 merged by jenkins-bot:
[operations/software/cumin@master] PuppetDB backend: forbid resource's parameters regex

Volans closed this task as Resolved.May 16 2017, 9:35 PM

Volans triaged this task as Medium priority.

Volans moved this task from In Code Review to Done on the SRE-tools board.