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

🐛 Fix Pipeline and Deploy #596

Merged
merged 13 commits into from
Feb 11, 2025
Merged

🐛 Fix Pipeline and Deploy #596

merged 13 commits into from
Feb 11, 2025

Conversation

ShanaLMoore
Copy link
Contributor

@ShanaLMoore ShanaLMoore commented Feb 7, 2025

this is a copy of rename-softserv-references-to-notch8 + additional updates.

rename-softserv-references-to-notch8 deploys now 🎉 but test and lint steps fail, so I branched off of it to resolve the latter.

This implementation required custom branches for BL:

ref:

Test and Lint are passing now!

image

Deploy succeeds too ✅

image

Issue:

v0.0.13-rename-ss-to-n8 changed scientist-softserve => nothc8 in the actions repo -  this keeps BL pointing to the same version they're already on
points to v0.0.13-docker-cmd-update and v0.0.6-docker-cmd-update which changes docker-compose commands to docker compose
Add a step to fetch Solr logs in your workflow to help diagnose the issue:
@ShanaLMoore ShanaLMoore requested a review from cziaarm February 7, 2025 19:59
@ShanaLMoore ShanaLMoore changed the title Docker cmd update Fix Pipeline and Deploy Feb 7, 2025
@ShanaLMoore ShanaLMoore marked this pull request as ready for review February 7, 2025 21:17
@ShanaLMoore ShanaLMoore changed the title Fix Pipeline and Deploy 🐛 Fix Pipeline and Deploy Feb 7, 2025
Copy link
Collaborator

@cziaarm cziaarm left a comment

Choose a reason for hiding this comment

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

Thanks Shana, looks like there were a good few more switches to do than I had thought!

I added a bit to get it to pass the lint, so happy to merge :)

@cziaarm cziaarm merged commit 7db568d into main Feb 11, 2025
7 checks passed
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