Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DBTP-457 Add DBT PaaS CodeBuild configuration #464

Merged
merged 66 commits into from
Oct 9, 2023
Merged

DBTP-457 Add DBT PaaS CodeBuild configuration #464

merged 66 commits into from
Oct 9, 2023

Conversation

WillGibson
Copy link
Contributor

No description provided.

@WillGibson
Rename celery stuff in Procfile
933b9fd
@WillGibson
Merge branch 'main' into DBTP-457-add-dbt-paas-codebuild-configuratio…
f0ab58e 
…n-e2e
@WillGibson
Merge branch 'main' into DBTP-457-add-dbt-paas-codebuild-configuratio…
6fb4fd2 
…n-e2e
@WillGibson
Add CodeBuild badge to README
98df597
@WillGibson
Add initial-codebuild-configuration
0238806
@WillGibson
Set PAKETO_BUILDER_VERSION and LIFECYCLE_VERSION
0980bb8
@WillGibson
Try just building a single celery image
441cb2b
@WillGibson
Sort pii-ner-exclude.txt
a2b6e27
@WillGibson
Try compliling assets during build
dc7dc54
@WillGibson
Exclude some false Named Entity Recognition positives
4c83f0c
@WillGibson
cd src to collectstatic
7ab0ad2
@WillGibson
Add config.settings/build.py
3a9d23b
@WillGibson
Skip celery for now
0364817
@WillGibson
Separate out config/settings/base_after_django_environ.py
7b871dc
@WillGibson
Add secret key, lose authbroker stuff
82f0189
@WillGibson
Add ELASTICSEARCH_DSL
5107e11
@WillGibson
Add ClamAV stuff
2f4bdd0
@WillGibson
Add APP_ENV
623b4f3
@WillGibson
Add DATABASES
37a2026
@WillGibson
Delete comented out stuff
146f13b
@WillGibson
Add a little debug output
4ebad3c
@WillGibson
Correct import in src/config/settings/env.py
ed5f86d
@WillGibson
Don't initialise_db_dict() for build
f9f78dd
@WillGibson
Let black do the reformatting
de35afb
@WillGibson
Fix ruff checks
277525f
@WillGibson WillGibson force-pushed the DBTP-457-add-dbt-paas-codebuild-configuration-e2e branch from 3ceda65 to 277525f Compare September 25, 2023 11:51
antroy-madetech and others added 17 commits September 26, 2023 16:06
@antroy-madetech
Trigger build
caecd7f
@antroy-madetech
Removed elastic search app and config
2c088c7
@antroy-madetech
Append ssl parameter to the CELERY_BROKER_URL
ea0fa88
@acodeninja @yusufsheiqh
Only use AWS_ACCESS_KEY_ID and SECRET_ACCESS_KEY when in GovPaaS
8fdbb93 
Co-Authored-By: Yusuf <[email protected]>
@acodeninja
Dump opensearch query logs
a9a0437
@acodeninja
Better debug logging
0baa29d
@acodeninja
Logger instance not writing to stdout
d9f6027
@acodeninja
WSGIRequest is not JSON serializable
855bc7b
@acodeninja
Remove query from debug
af2f25f
@antroy-madetech
More logging
da2d4d1
@acodeninja @antroy-madetech
Revert to existing requirements
3f39e4f 
Co-Authored-By: Ant <[email protected]>
@CamLamb
Run make fix to correct the formatting
c13f8ad
@CamLamb
Remove ssl_cert_reqs=required from local and testing scenarios
8cfdf87
@CamLamb
Remove django_extensions from testing INSTALLED_APPS
facfab2
@acodeninja
Test if order matters between Procfile and process.yml
88576da
@antroy-madetech
Merge branch 'main' into DBTP-457-add-dbt-paas-codebuild-configuratio…
56d6952 
…n-e2e
@antroy-madetech
Remove codebuild check for now
649260e
@antroy-madetech antroy-madetech marked this pull request as ready for review October 6, 2023 12:56
CamLamb
CamLamb reviewed Oct 6, 2023
View reviewed changes
Copy link
Contributor

@CamLamb CamLamb left a comment

Choose a reason for hiding this comment

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

I've added a couple of comments 😄

src/search/templatetags/search.py Outdated Show resolved Hide resolved
src/config/settings/base.py Show resolved Hide resolved
marcelkornblum
marcelkornblum reviewed Oct 6, 2023
View reviewed changes
src/config/settings/env.py Outdated Show resolved Hide resolved
src/config/settings/env.py Outdated Show resolved Hide resolved
src/config/settings/test.py Show resolved Hide resolved
src/extended_search/apps.py Show resolved Hide resolved
CamLamb
CamLamb approved these changes Oct 9, 2023
View reviewed changes
Copy link
Contributor

@CamLamb CamLamb left a comment

Choose a reason for hiding this comment

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

LGTM

@CamLamb CamLamb merged commit eb62598 into main Oct 9, 2023
1 check passed
@CamLamb CamLamb deleted the DBTP-457-add-dbt-paas-codebuild-configuration-e2e branch October 9, 2023 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@antroy-madetech antroy-madetech antroy-madetech left review comments

@marcelkornblum marcelkornblum marcelkornblum left review comments

@CamLamb CamLamb CamLamb approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@WillGibson @CamLamb @marcelkornblum @antroy-madetech @acodeninja