Skip to content

#12: Reference Server MariaDB Initial Release #31

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.

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

anasik
Copy link

@anasik anasik commented Mar 18, 2025

Closes #12.

@patrickcoca patrickcoca force-pushed the 26-reference-server-initial-release branch from 310b9cd to 850e063 Compare March 19, 2025 00:07
@darnjo darnjo changed the title #26: Reference Server Initial Release #12: Reference Server MariaDB Initial Release Mar 25, 2025
@darnjo darnjo self-requested a review March 25, 2025 20:40
@darnjo darnjo added the enhancement New feature or request label Mar 25, 2025
@darnjo darnjo added this to the Web API Core Reference Server milestone Mar 25, 2025
Copy link
Member

@darnjo darnjo left a comment

Choose a reason for hiding this comment

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

Made some comments on things to look out for and address. Shouldn't need to spend too much time on fixing things - some are just observations.


## ENVIRNONMENT SPECIFIC NOTES

The build scripts were moved to take place in a Docker container so that they would work consistently across environments.

### Windows

In Windows, running under a Bash shell will work, assuming you meet the above requirements.
Don't forget to have Docker installed for Windows.

There is an `env-default-windows` file you should rename to `.env` before running the build script.

The `docker/docker-builder` file has a line commented out for Windows users, and a line that needs to be commented out.

Copy link
Member

Choose a reason for hiding this comment

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

TODO: @patrickcoca - need to add Mac build instructions and Linux if they're different. Sometimes ARM64 builds are different from AMD64 depending on which version of Linux is being used under the hood. Debian Bullseye seems to work OK.

@@ -1,11 +1,18 @@
FROM ubuntu:20.10
FROM ubuntu:20.04
Copy link
Member

Choose a reason for hiding this comment

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

@patrickcoca - may want to take a look at this Docker Compose file and migrate from Ubuntu to Debian. https://github.com/RESOStandards/reso-certification-utils/blob/main/DockerfileBullseye

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Expand functionality to the Web API Reference Server
4 participants