-
Notifications
You must be signed in to change notification settings - Fork 14
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: Use file-server container for db upgrade job #307
Conversation
node-red:2.2.3-main-linux-amd64 scan results26 tests 0 ✅ 0s ⏱️ For more details on these failures, see this check. Results for commit 71218ab. ♻️ This comment has been updated with latest results. |
node-red:3.0.2-main-linux-amd64 scan results 1 files 4 suites 0s ⏱️ For more details on these failures, see this check. Results for commit 71218ab. ♻️ This comment has been updated with latest results. |
node-red:3.1.x-main-linux-amd64 scan results8 tests 0 ✅ 0s ⏱️ For more details on these failures, see this check. Results for commit 71218ab. ♻️ This comment has been updated with latest results. |
file-server:main-linux-amd64 scan results1 tests 0 ✅ 0s ⏱️ For more details on these failures, see this check. Results for commit 71218ab. ♻️ This comment has been updated with latest results. |
node-red:3.0.2-main-linux-arm64 scan results 1 files 4 suites 0s ⏱️ For more details on these failures, see this check. Results for commit 71218ab. ♻️ This comment has been updated with latest results. |
node-red:2.2.3-main-linux-arm64 scan results26 tests 0 ✅ 0s ⏱️ For more details on these failures, see this check. Results for commit 71218ab. ♻️ This comment has been updated with latest results. |
node-red:3.1.x-main-linux-arm64 scan results8 tests 0 ✅ 0s ⏱️ For more details on these failures, see this check. Results for commit 71218ab. ♻️ This comment has been updated with latest results. |
file-server:main-linux-arm64 scan results1 tests 0 ✅ 0s ⏱️ For more details on these failures, see this check. Results for commit 71218ab. ♻️ This comment has been updated with latest results. |
forge-k8s:main-linux-amd64 scan results1 tests ±0 0 ✅ ±0 0s ⏱️ ±0s For more details on these failures, see this check. Results for commit 71218ab. ± Comparison against base commit b3cd583. ♻️ This comment has been updated with latest results. |
forge-k8s:main-linux-arm64 scan results1 tests ±0 0 ✅ ±0 0s ⏱️ ±0s For more details on these failures, see this check. Results for commit 71218ab. ± Comparison against base commit b3cd583. ♻️ This comment has been updated with latest results. |
I assume the validation failed because the file-server container wasn't built? https://github.com/FlowFuse/helm/actions/runs/7870917055/job/21473105621?pr=307#step:10:561 |
@ppawlowski Just checking, the failed job looks to be using the latest containers from the docker hub, not the ones built as part of the PR test, so the failure is because the new version of the container is not published yet? |
I'll test this when I get chance locally |
Co-authored-by: Ben Hardill <ben@flowforge.com>
fixes #306
Description
While testing installing on OpenShift the db upgrade job fails because it's no longer run as the root user so can't install the psql command into the alpine container image.
This adds the psql command to the file-server image and uses this to run the job.
Related Issue(s)
#306
Checklist
flowforge.yml
?FlowFuse/helm
to update ConfigMap TemplateFlowFuse/CloudProject
to update values for Staging/ProductionLabels
backport
labelarea:migration
label