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.

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

Update REST and CI docker compose scripts to sync with 7.x backend #2967

Merged

Conversation

tdonohue
Copy link
Member

@tdonohue tdonohue commented Apr 19, 2024

Manual port of #2966 to dspace-7_x.

This fixes an error that also occurs if you attempt to run the backend using the guide at https://wiki.lyrasis.org/display/DSPACE/Try+out+DSpace+7 When doing so, the dspacedb container will fail to start with this error:

Error: Database is uninitialized and superuser password is not specified.
        You must specify POSTGRES_PASSWORD to a non-empty value for the
        superuser. For example, "-e POSTGRES_PASSWORD=password" on "docker run".

@tdonohue tdonohue added bug 1 APPROVAL pull request only requires a single approval to merge backend: Docker related to DSpace deployment via Docker labels Apr 19, 2024
@tdonohue tdonohue added this to the 7.6.2 milestone Apr 19, 2024
@tdonohue
Copy link
Member Author

This PR is failing e2e tests because of minor bugs in Backend Dockers Scripts. These bugs are fixed in DSpace/DSpace#9491

@tdonohue
Copy link
Member Author

Backend fixes were applied in DSpace/DSpace#9491 and all Angular tests now pass. Merging immediately.

@tdonohue tdonohue merged commit 679702b into DSpace:dspace-7_x Apr 22, 2024
13 checks passed
@tdonohue tdonohue deleted the update_backend_docker_scripts_7x branch April 22, 2024 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 APPROVAL pull request only requires a single approval to merge backend: Docker related to DSpace deployment via Docker bug
Projects
Development

Successfully merging this pull request may close these issues.

1 participant