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

Decouple CD for stage/UAT and connect to Geoblacklight v4 branch #1041

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

thatbudakguy
Copy link
Member

This will allow iterative refinement of the Geoblacklight-4
compatible version of earthworks without needing to pause
dependency updates in production.

It's expected that this work will be finished and reverted by
the start of the Earthworks workcycle on 6/24.

If this is merged, https://github.com/sul-dlss/shared_configs/pull/2194
and https://github.com/sul-dlss/shared_configs/pull/2193 are
also needed to point the app at new compatible solr cores using
the Aardvark schema.

@thatbudakguy thatbudakguy marked this pull request as ready for review June 5, 2024 21:31
@cbeer
Copy link
Member

cbeer commented Jun 5, 2024

Do you also need to tell cap which branch to deploy, or is it getting that magically?

This will allow iterative refinement of the Geoblacklight-4
compatible version of earthworks without needing to pause
dependency updates in production.

It's expected that this work will be finished and reverted by
the start of the Earthworks workcycle on 6/24.

If this is merged, sul-dlss/shared_configs#2194
and sul-dlss/shared_configs#2193 are
also needed to point the app at new compatible solr cores using
the Aardvark schema.
@thatbudakguy
Copy link
Member Author

Good catch, thanks! I think this is the right way?

@cbeer cbeer merged commit b0045ff into main Jun 5, 2024
3 checks passed
@cbeer cbeer deleted the decouple-cd branch June 5, 2024 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants