darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-24 17:31:34 -05:00
Code Issues Packages Projects Releases Wiki Activity
6,270 Commits 128 Branches 60 Tags
bf4e75da40ad45ffb78be9b25d210049dcca2de5
Commit Graph

6270 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Athanasios Voutsadakis
bf4e75da40 Explicitly target the right space before deploying 2019-04-24 15:55:45 +01:00
Alexey Bezhan
bc7d91daec Merge pull request #2468 from alphagov/local-statsd-exporter 
Local statsd exporter
 2019-04-24 15:27:55 +01:00
Rebecca Law
b1d4af2f4a Merge pull request #2467 from alphagov/populate-go-live-data 
Populate go live user and date
 2019-04-24 14:28:03 +01:00
Alexey Bezhan
528b095fcc Add statsd exporter metric mapping configuration file 
`statsd_mapping.yml` sets all timers to default to a Prometheus
histogram metric type and renames all metrics to start with `notifications_`.
 2019-04-24 13:50:13 +01:00
Alexey Bezhan
570cbc3eab Add statsd_exporter to app PaaS startup scripts 
`statsd_exporter` is only started if `STATSD_HOST` is set to `localhost`.
 2019-04-24 13:50:13 +01:00
Alexey Bezhan
ba2abc9127 Add a local_statsd configuration to PaaS manifest template 
Running `statsd_exporter` alongside the app process allows us to get
StatsD metrics pushed by workers to Prometheus.

This requires adding a route to the worker instances and binding the
RE prometheus discovery service. So this approach won't work for API
and admin since they already have `gunicorn` bound to the `$PORT`.

Since we're not ready to switch all apps to Prometheus metrics at once
and we don't currently have a way to push statsd metrics to multiple
destination we're using a configuration setting in the manifest template
to switch individual workers in specific environments.

`local_statsd` contains a list of environments where the app should
use local `statsd_exporter` for pushing statsd metrics instead of
HostedGraphite.
 2019-04-24 13:50:13 +01:00
Alexey Bezhan
0138eb0cae Make statsd host configurable with an env variable 
Setting `STATSD_HOST` for an env variable allows us to switch to a
local statsd_exporter on a per-app basis.

This also changes `STATSD_ENABLED` to be on when `STATSD_HOST` is set,
avoiding the need to set it separately.
 2019-04-24 13:50:13 +01:00
Alexey Bezhan
394a0244d0 Merge pull request #2469 from alphagov/stub-staging-sms-providers 
Stub out SMS providers on staging for the perf tests
 2019-04-24 13:33:49 +01:00
Rebecca Law
bf42fa0c49 Use the csv.reader instead. 2019-04-24 11:56:44 +01:00
Alexey Bezhan
7520cc46de Stub out SMS providers on staging for the perf tests 
This points MMG and Firetext on staging to a stub service run on
PaaS to avoid text message costs during the load test.
 2019-04-24 11:37:41 +01:00
Alexey Bezhan
dc904f8742 Add a make target to download statsd_exporter binary 
Downloads a linux binary from Github and puts it in the scripts folder.
This should add it to the package CI uploads to CF during deploy.

Unfortunately, since out .cfignore is a symlink to .gitignore we
can't add the file to be ignored by git, so it shows up in untracked
if `cf-deploy` is run locally.
 2019-04-24 11:25:38 +01:00
Rebecca Law
c989310ac5 New command to populate the go live user and date from the beta partners spreadsheet. 2019-04-24 10:56:50 +01:00
Rebecca Law
d8ea30f414 Merge pull request #2465 from alphagov/go-live-user-id-for-services 
Go live data recorded for service
 2019-04-18 15:02:21 +01:00
Rebecca Law
cc17ba3ad9 Merge pull request #2455 from alphagov/pyup-scheduled-update-2019-04-10 
Scheduled weekly dependency update for week 14
 2019-04-17 13:50:42 +01:00
Rebecca Law
55b2fd6d57 Fix the migration file conflicts 2019-04-17 13:39:49 +01:00
Rebecca Law
55a65a22a9 Added test for go_live_user and go_live_at in the response for get_service_by_id 2019-04-17 13:20:44 +01:00
Rebecca Law
60b4970d7f Add new columns to services and services_history. 
go_live_user_id: is the user that requested the service to go live
go_live_at: is the DateTime the service went live.

There will be a data migration from the beta partners spreadsheet to back fill the data.
 2019-04-17 13:20:44 +01:00
Rebecca Law
0730f9d9e1 Merge branch 'master' into pyup-scheduled-update-2019-04-10 2019-04-17 11:37:47 +01:00
Rebecca Law
b30969dbcf Update requirements.txt 2019-04-17 11:35:42 +01:00
Chris Hill-Scott
96bb9e19f9 Merge pull request #2385 from alphagov/raise-if-fail-to-write-history 
Raise exception if history can’t be written
 2019-04-17 11:17:49 +01:00
Chris Hill-Scott
2c57e4a4f6 Merge pull request #2445 from alphagov/remove-domains-from-branding-migration 
Remove domain columns from branding table
 2019-04-17 11:02:27 +01:00
Chris Hill-Scott
916d4e2860 Remove domain columns from branding table 
This relationship is via the `Organisation` now; we don’t use this
column to fudge a relationship based on the user’s email address and the
matching something in these columns.
 2019-04-15 16:14:41 +01:00
Chris Hill-Scott
8929911ad8 Merge pull request #2444 from alphagov/remove-domains-from-branding 
Stop updating or reading domain columns on branding table
 2019-04-15 15:35:09 +01:00
Rebecca Law
6ccb242107 Merge pull request #2458 from alphagov/remove-unused-method 
Remove unused method.
 2019-04-15 12:11:00 +01:00
Rebecca Law
d5ae740f83 Merge pull request #2459 from alphagov/update-response-for-duplicate-domains 
The previous PR added a error handler for the wrong db exception.
 2019-04-15 12:10:44 +01:00
Chris Hill-Scott
c6447c111d Don’t let admin specify branding for new service 
It doesn’t need to because the API code works it all out now by looking
at the organisation.
 2019-04-12 16:42:50 +01:00
Chris Hill-Scott
9bca2432ce Merge pull request #2463 from alphagov/remove-domain-required-letter-branding-schema 
Don’t require a domain when updating the letter branding
 2019-04-12 16:16:00 +01:00
Pea (Malgorzata Tyczynska)
88b31c51f5 Merge pull request #2437 from alphagov/precompiled-overlay-failed-validation 
Precompiled request preview with overlay when validation fails
 2019-04-12 16:14:56 +01:00
Chris Hill-Scott
9c0b9171a1 Don’t require a domain when updating the letter branding 
So the admin is free to stop passing it in.
 2019-04-12 16:05:29 +01:00
Chris Hill-Scott
ee966668bd Remove domain columns from branding table 
This relationship is via the `Organisation` now; we don’t use this
column to fudge a relationship based on the user’s email address and the
matching something in these columns.
 2019-04-12 15:53:17 +01:00
Chris Hill-Scott
a39ed098b8 Merge pull request #2442 from alphagov/default-branding-for-nhs-users 
Set default branding for NHS services
 2019-04-12 15:29:30 +01:00
Alexey Bezhan
ebd12d7e76 Merge pull request #2461 from alphagov/firetext-url-config-env 
Make Firetext URL configurable through the application environment
 2019-04-12 13:40:25 +01:00
Alexey Bezhan
330afab5e2 Make Firetext URL configurable through the application environment 
Similar to MMG, there's a new env variable FIRETEXT_URL that can be
used to override the Firetext api URL.

This will be used to stub out both providers during the load test or
can be used to run a local API against a fake provider endpoint.
 2019-04-12 12:03:58 +01:00
Leo Hemsted
09decfe3a8 Merge pull request #2460 from alphagov/forgot-about-beat 
add delivery-celery-beat to paas_app_wrapper
 2019-04-12 11:25:02 +01:00
Leo Hemsted
95ca533df2 add delivery-celery-beat to paas_app_wrapper 
Forget about beats by dre
 2019-04-12 11:10:18 +01:00
Rebecca Law
22fc4b693a The previous PR added a error handler for the wrong db exception. 
This corrects it to look for the unique constraint.
 2019-04-12 10:55:21 +01:00
Leo Hemsted
5d26724022 Merge pull request #2456 from alphagov/remove-multi-app-manifests 
Remove multi app manifests
 2019-04-12 10:34:49 +01:00
Rebecca Law
1c68e0f565 Remove unused method. 
last_n_days was only being used in a test.
 2019-04-12 10:26:46 +01:00
Rebecca Law
622b3e76f7 Merge pull request #2454 from alphagov/populate-volumne-intentions 
Command to populate the volume intentions for services.
 2019-04-12 09:44:57 +01:00
Rebecca Law
b7ecaa4a40 Merge pull request #2453 from alphagov/update-response-for-duplicate-domains 
Return a useful message if the domain is a duplicate.
 2019-04-12 09:44:29 +01:00
Pea Tyczynska
2d71aea7a1 Get rid of an unneeded flag - destination endpoint is slimmer now and doesnt need it 2019-04-11 16:33:40 +01:00
Chris Hill-Scott
08f67dcfdd Merge pull request #2457 from alphagov/count-live-services-and-organisations 
Add method to count of live services and orgs
 2019-04-11 16:22:54 +01:00
Leo Hemsted
3277c65a7c add jinja2-cli to dev requirements 2019-04-11 15:05:44 +01:00
Leo Hemsted
c786258a60 give instances and NOTIFY_APP_NAME sensible defaults 
NOTIFY_APP_NAME follows precedent and just tries to strip 'notify-'
from the beginning of the string.

instances is not specified at all if not defined - it'll scale up to
the same amount of instances as currently present, and then the
autoscaler will take over anyway
 2019-04-11 14:57:22 +01:00
Pea Tyczynska
dd9e285684 Refactor preview_letter_template_by_notification_id 2019-04-11 14:27:57 +01:00
Pea Tyczynska
63fe210202 Also show overlay on document download of failed letter 
And of course test this new flex
 2019-04-11 14:27:12 +01:00
Pea Tyczynska
2f52a8fd64 Call overlaid preview for precompiled letters that fail validation 
Also test precompiled letter preview with overlay
 2019-04-11 14:26:59 +01:00
Chris Hill-Scott
6bfd999de2 Add method to count of live services and orgs 
So that we don’t have to update the home page manually any more.
 2019-04-11 13:54:43 +01:00
Leo Hemsted
575ec96b6b fix test that checks manifest contains references to all queues 2019-04-10 16:48:21 +01:00
Leo Hemsted
8cc5d40291 remove old manifest files and creation script 2019-04-10 15:21:30 +01:00