Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

DPM 202 asim formatter logging #893

Merged

Conversation

marijnkampf
Copy link
Contributor

Description of change

Use ASIM formatter for logging on production environments.

Test instructions

No visible changes.

@marijnkampf marijnkampf marked this pull request as ready for review April 22, 2024 12:01
@marijnkampf marijnkampf requested a review from a team as a code owner April 22, 2024 12:01
pytest.ini Outdated
@@ -1,3 +1,4 @@
[pytest]
DJANGO_SETTINGS_MODULE = app.settings.djangotest
python_files = tests.py test_*.py
# pythonpath = . tests app app/pingdom/tests pingdom

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No we don't.

Remove comment
@marijnkampf marijnkampf merged commit c25b2fe into migration-deploy Apr 23, 2024
2 checks passed
@marijnkampf marijnkampf deleted the migration/DPM-202-ASIM-formatter-logging branch April 23, 2024 09:55
marijnkampf added a commit that referenced this pull request Apr 23, 2024
* Use ASIM formatter for logging on production environments
* Upgrade Python version in Docker
marijnkampf added a commit that referenced this pull request Apr 23, 2024
* Use ASIM formatter for logging on production environments
* Upgrade Python version in Docker
marijnkampf added a commit that referenced this pull request May 8, 2024
* Use ASIM formatter for logging on production environments
* Upgrade Python version in Docker
cgsunkel pushed a commit that referenced this pull request Jun 25, 2024
* Use ASIM formatter for logging on production environments
* Upgrade Python version in Docker
cgsunkel pushed a commit that referenced this pull request Jul 15, 2024
* Use ASIM formatter for logging on production environments
* Upgrade Python version in Docker
cgsunkel added a commit that referenced this pull request Jul 15, 2024
* update codeowners

* Add sample rate to sentry (#981)

* Add sample_rate to sentry setup

* Add new action for govpaas branch

* Bump django from 4.2.13 to 4.2.14 (#982)

Bumps [django](https://github.com/django/django) from 4.2.13 to 4.2.14.
- [Commits](django/django@4.2.13...4.2.14)

---
updated-dependencies:
- dependency-name: django
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add new fields to sentry config (#983)

* Remove migration actions (#984)

* Update Pingdom health check Database, Redis and Celery (#871)

* Change health check end point to /pingdom/ping.xml.
* Extend health check to include Celery and Redis in addition to Database.

* DPM 202 asim formatter logging (#893)

* Use ASIM formatter for logging on production environments
* Upgrade Python version in Docker

* add build config file

* allow for running on gov.uk paas and dbt platform

* allow any python 3.12 version

* allow nodejs buildpack to be included

* set nodejs version to 20.11

* add collect static command to build

* do not use sentry when dsn unset

* add required variables for local build

* DPM 623 Update DB connection environment variable (#915)

* Update database connection environment variable to use DATABASE_CREDENTIALS

* Update image_build for packaging.
Co-Authored-By: Kuds <kudrotey.khuda@digital.trade.gov.uk>

* Add extra env vars to image build.

* Adding Env vars
Co-Authored-By: Kuds <kudrotey.khuda@digital.trade.gov.uk>

* Update env vars
<yusuf.sheikh@digital.trade.gov.uk>

* Adding Env vars

* Added more env vars

* More Env vars

* More Env Vars

* Remove Duplicate Env Var

* Added extra env var

* fix: remove unnecessary start from package.json

* Update image_build_run.sh to include all env vars

* Add setuptools to requirements.txt

* fix: try downgrade of python

* Reset migration for staging deploy

* Apply admin migration before enquiry (#933)

* Revert migration script changes (#941)

* Remove celery check from pingdom (#944)

* Remove celery check from pingdom

* Remove celery pingdom checks

* fix: always call collectstatic
Co-Authored-By: Paul <paul.wheatcroft@digital.trade.gov.uk>

* Add saas compilation to image build script

* fix: update celery timezone

* chore: update default timezone

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Christopher <christopher.sunkel@digital.trade.gov.uk>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Marijn Kampf <marijn.kampf@digital.trade.gov.uk>
Co-authored-by: Lawrence Goldstien <lawrence.goldstein@digital.trade.gov.uk>
Co-authored-by: markwarriner <mark.warriner@digital.trade.gov.uk>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants