Skip to content

Commit

Permalink
fix(newrelic): use the new location of uwsgi.ini in the launch command (
Browse files Browse the repository at this point in the history
#144)

The location of the uwsgi.ini file was changed in tutor==18.1.3 so we
have to update the command run by the container. Additionally we remove
an unused variable that was lingering.
  • Loading branch information
MoisesGSalas authored Oct 15, 2024
1 parent 2078d11 commit 67fb1fc
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
1 change: 0 additions & 1 deletion drydock/plugin.py
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,6 @@ def get_sync_waves_for_resource(resource_name: str) -> SYNC_WAVES_ORDER_ATTRS_TY
"INGRESS": False,
"INGRESS_EXTRA_HOSTS": [],
"INGRESS_LMS_EXTRA_HOSTS": [],
"NEWRELIC": False,
"NEWRELIC_LICENSE_KEY": "",
"CUSTOM_CERTS": {},
"DEBUG": False,
Expand Down
2 changes: 1 addition & 1 deletion drydock/templates/drydock/k8s/newrelic.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
path: /spec/template/spec/containers/0/args
value:
- uwsgi
- uwsgi.ini
- /openedx/uwsgi.ini
- op: add
path: /spec/template/spec/containers/0/command
value:
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ classifiers = [
"Programming Language :: Python :: 3.12",
]
dependencies = [
"tutor>=18.0.0,<19.0.0"
"tutor>=18.1.3,<19.0.0"
]

[project.optional-dependencies]
Expand Down

0 comments on commit 67fb1fc

Please sign in to comment.