Caddy server setup #233
Open
Caddy server setup #233
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…server (eg nginx, caddy) config
…der spec properly
ccjoel
commented
Jan 10, 2024
router/Caddyfile
Outdated
| reverse_proxy {$DOJO_HOST}:80 | ||
| } | ||
|
|
||
| redir /api/ai-docs /api/ai-docs/ |
There was a problem hiding this comment.
We can remove this redirect rule, as boxytas/knowledge is part of the Dojo API now (which is covered by the above /api/dojo rule).
ccjoel
commented
Jan 10, 2024
| @@ -218,7 +220,38 @@ services: | |||
| - dojo-stack | |||
| ports: | |||
| - 8012:8012 | |||
| # profiles: | |||
There was a problem hiding this comment.
Suggestion, where we could enable testmos on local dev only.
ccjoel
commented
Jan 10, 2024
| @@ -193,6 +193,8 @@ services: | |||
| - ${PWD}/data:/data | |||
| networks: | |||
| dojo-stack: {} | |||
| # profiles: | |||
There was a problem hiding this comment.
Suggestion, where we could enable minio on local dev only.
* Once a non-pdf is uploaded, converted, and processed, replace the original non-pdf file references on ES with the PDF paths/filenames, in order to reference and display the PDF on the document knowledge UI (#242) * Small quality-of-life changes (logging of validation errors on api logs); fixed response job get status schema to use valid rq statuses (#244) * Updates data modeling dataset selector to include a filter, makes name column wider, improves expandable datagrid cell across app (#243) * Data Modeling Error Handling (#245) * Adds initial Data Modeling error handling with unstyled error card * Styles modeler error card, changes button to refresh page and send user back to the dataset selection screen * Improves modeling dataset selection screen loading/error state * Slight styling tweaks to data modeling error card, adjust error catching to be broader * Changes IFPRI branding to CGIAR branding (#246) * Changes IFPRI branding to CGIAR branding * Changes 'digital twin' image border color --------- Co-authored-by: Joel Quiles <[email protected]> Co-authored-by: adamgilg <[email protected]>
* Once a non-pdf is uploaded, converted, and processed, replace the original non-pdf file references on ES with the PDF paths/filenames, in order to reference and display the PDF on the document knowledge UI (#242) * Small quality-of-life changes (logging of validation errors on api logs); fixed response job get status schema to use valid rq statuses (#244) * Updates data modeling dataset selector to include a filter, makes name column wider, improves expandable datagrid cell across app (#243) * Data Modeling Error Handling (#245) * Adds initial Data Modeling error handling with unstyled error card * Styles modeler error card, changes button to refresh page and send user back to the dataset selection screen * Improves modeling dataset selection screen loading/error state * Slight styling tweaks to data modeling error card, adjust error catching to be broader * Changes IFPRI branding to CGIAR branding (#246) * Changes IFPRI branding to CGIAR branding * Changes 'digital twin' image border color * CGIAR Whitelabel: Updates landing page font weight --------- Co-authored-by: Joel Quiles <[email protected]> Co-authored-by: adamgilg <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The purpose of this work is to add a built-in prod-like router within the dojo-stack. Currently for staging/prod/deployments we use an external nginx server with a mix of in-code and in-flight configuration. Setting this up would mean we can start the stack without the need to wrap with nginx or other servers separate from our stack.
Uses a
prodv2docker profile. We can/could merge, try it out, and either eventually remove the profile or extend our docker profiles usage.All of it will run by running the following: