Skip to content

Conversation

@aldbr
Copy link
Contributor

@aldbr aldbr commented May 9, 2025

Closes #349

I could potentially create a reusable workflow to avoid duplicating some code between deployment.yaml and extension.yaml but this would also add some complexity to extension.yaml. As it can act as an example of CI, I feel like it should stay clear and simple and that we can afford that small duplication (I've tried to minimize it). Please let me know if you have a different opinion.

Here I export both gubbins/client and services, but I think we would just need to use gubbins/services.

Unless you disagree, or if you have a better plan, I will make a quick test on my repo first, and if everything goes well I undraft the PR.

@aldbr aldbr force-pushed the main_FEAT_deploy-gubbins branch 4 times, most recently from 7cab612 to 0b35072 Compare May 12, 2025 15:26
@aldbr
Copy link
Contributor Author

aldbr commented May 12, 2025

So I applied b830754 to test the content of the PR on my fork.

It seems to work:

Note: extensions.yaml cannot pass this CI because it requires an existing gubbins/services:dev image, which does not exist at the moment.

@aldbr aldbr marked this pull request as ready for review May 12, 2025 15:28
@aldbr aldbr force-pushed the main_FEAT_deploy-gubbins branch 5 times, most recently from afd00ac to 714db7c Compare May 22, 2025 11:06
@aldbr
Copy link
Contributor Author

aldbr commented May 22, 2025

I've applied the latest changes, and made things a bit differently in main.yaml: instead of relying on a potential dev image that would be stored in the registry (diracx-services:dev exists but not gubbins-services:dev), I build dev from scratch in the CI for both extensions.

It allows testing potential changes that could occur in the Dockerfiles (diracx and gubbins)

@aldbr aldbr marked this pull request as draft May 27, 2025 12:32
@aldbr
Copy link
Contributor Author

aldbr commented May 27, 2025

Looks like I don't have enough space to run minio as I am now loading the diracx docker image both locally and in kind:
https://github.com/DIRACGrid/diracx/actions/runs/15189075216/job/42716849956#step:13:1171

I am trying to add a prune-loaded-images option to run_demo.sh to remove the local image once loaded in kind, let's see how it goes 🤞

@aldbr
Copy link
Contributor Author

aldbr commented May 27, 2025

I've applied DIRACGrid/diracx-charts#158 to this PR, and it seems to solve the issue.
Once merged - if you validate the solution - I will remove my workaround and undraft this PR.

@aldbr aldbr force-pushed the main_FEAT_deploy-gubbins branch 2 times, most recently from 4b1d400 to bd0f6e3 Compare May 27, 2025 13:43
@aldbr
Copy link
Contributor Author

aldbr commented May 27, 2025

Should we move the content of https://github.com/DIRACGrid/diracx-charts/blob/master/demo/ci_values.yaml in ./diracx_values.yaml and ./extensions/gubbins_values.yaml (or in ./ci_values.yaml)?

@aldbr aldbr force-pushed the main_FEAT_deploy-gubbins branch from bd0f6e3 to a2f221d Compare July 17, 2025 12:04
@aldbr aldbr marked this pull request as ready for review July 17, 2025 12:04
@aldbr aldbr force-pushed the main_FEAT_deploy-gubbins branch 2 times, most recently from 5d195dd to 923ba11 Compare July 17, 2025 15:41
@fstagni fstagni closed this Aug 28, 2025
@fstagni fstagni reopened this Aug 28, 2025
@read-the-docs-community
Copy link

read-the-docs-community bot commented Aug 28, 2025

Documentation build overview

📚 diracx | 🛠️ Build #29811145 | 📁 Comparing 1f81e2e against latest (bc33e1c)


🔍 Preview build

Show files changed (115 files in total): 📝 115 modified | ➕ 0 added | ➖ 0 deleted
File Status
404.html 📝 modified
index.html 📝 modified
REFERENCE/index.html 📝 modified
RUN_PROD/index.html 📝 modified
SECURITY/index.html 📝 modified
SSO/index.html 📝 modified
admin/index.html 📝 modified
dev/index.html 📝 modified
roadmap/index.html 📝 modified
user/index.html 📝 modified
admin/deploy_instance/index.html 📝 modified
admin/explanations/index.html 📝 modified
admin/how-to/index.html 📝 modified
admin/manage_dependencies/index.html 📝 modified
admin/manage_release/index.html 📝 modified
admin/reference/index.html 📝 modified
admin/tutorials/index.html 📝 modified
dev/explanations/index.html 📝 modified
dev/how-to/index.html 📝 modified
dev/manage_extension/index.html 📝 modified
dev/reference/index.html 📝 modified
dev/setup_environment/index.html 📝 modified
dev/tutorials/index.html 📝 modified
dev/web-arch/index.html 📝 modified
developer/contribute/index.html 📝 modified
developer/manage_extension/index.html 📝 modified
developer/setup_environment/index.html 📝 modified
user/explanations/index.html 📝 modified
user/how-to/index.html 📝 modified
user/reference/index.html 📝 modified
user/tutorials/index.html 📝 modified
user/web/index.html 📝 modified
admin/explanations/auth-with-diracx/index.html 📝 modified
admin/explanations/auth-with-external/index.html 📝 modified
admin/explanations/chart-structure/index.html 📝 modified
admin/explanations/configuration/index.html 📝 modified
admin/explanations/database-management/index.html 📝 modified
admin/explanations/opentelemetry/index.html 📝 modified
admin/explanations/sandbox-store/index.html 📝 modified
admin/explanations/user-management/index.html 📝 modified
admin/how-to/debugging/index.html 📝 modified
admin/how-to/install/index.html 📝 modified
admin/how-to/rotate-a-secret/index.html 📝 modified
admin/how-to/upgrading/index.html 📝 modified
admin/reference/env-variables/index.html 📝 modified
admin/reference/security_model/index.html 📝 modified
admin/reference/settings-and-preferences/index.html 📝 modified
admin/reference/values/index.html 📝 modified
admin/tutorials/authentication/index.html 📝 modified
admin/tutorials/run_locally/index.html 📝 modified
dev/explanations/components/index.html 📝 modified
dev/explanations/designing-functionality/index.html 📝 modified
dev/explanations/extensions/index.html 📝 modified
dev/explanations/run_demo/index.html 📝 modified
dev/explanations/testing/index.html 📝 modified
dev/how-to/add-a-cli-command/index.html 📝 modified
dev/how-to/add-a-db/index.html 📝 modified
dev/how-to/add-a-route/index.html 📝 modified
dev/how-to/add-a-setting/index.html 📝 modified
dev/how-to/add-a-task/index.html 📝 modified
dev/how-to/add-a-test/index.html 📝 modified
dev/how-to/add-functionality/index.html 📝 modified
dev/how-to/client-customization/index.html 📝 modified
dev/how-to/client-extension/index.html 📝 modified
dev/how-to/client-generation/index.html 📝 modified
dev/how-to/contribute/index.html 📝 modified
dev/how-to/create_application/index.html 📝 modified
dev/how-to/develop-legacy-dirac/index.html 📝 modified
dev/how-to/extend-diracx/index.html 📝 modified
dev/how-to/use-the-demo/index.html 📝 modified
dev/reference/application-state/index.html 📝 modified
dev/reference/client-metapathfinder/index.html 📝 modified
dev/reference/coding-conventions/index.html 📝 modified
dev/reference/configuration/index.html 📝 modified
dev/reference/db-transaction-model/index.html 📝 modified
dev/reference/dependency-injection/index.html 📝 modified
dev/reference/entrypoints/index.html 📝 modified
dev/reference/env-variables/index.html 📝 modified
dev/reference/pixi-tasks/index.html 📝 modified
dev/reference/security-policies/index.html 📝 modified
dev/reference/security-properties/index.html 📝 modified
dev/reference/test-recipes/index.html 📝 modified
dev/reference/writing-tests/index.html 📝 modified
dev/tutorials/advanced-tutorial/index.html 📝 modified
dev/tutorials/develop-web/index.html 📝 modified
dev/tutorials/getting-started/index.html 📝 modified
dev/tutorials/larger-developments/index.html 📝 modified
dev/tutorials/making-changes/index.html 📝 modified
dev/tutorials/play-with-auth/index.html 📝 modified
dev/tutorials/run-locally/index.html 📝 modified
dev/tutorials/write-docs/index.html 📝 modified
user/reference/client-configuration/index.html 📝 modified
user/reference/known-installations/index.html 📝 modified
user/reference/programmatic-usage/index.html 📝 modified
user/tutorials/getting-started/index.html 📝 modified
user/web/list_and_share_applications/index.html 📝 modified
user/web/login_out/index.html 📝 modified
user/web/monitor_jobs/index.html 📝 modified
admin/how-to/install/convert-cs/index.html 📝 modified
admin/how-to/install/embracing/index.html 📝 modified
admin/how-to/install/install-kubernetes/index.html 📝 modified
admin/how-to/install/installing/index.html 📝 modified
admin/how-to/install/minimal-requirements/index.html 📝 modified
admin/how-to/install/register-a-vo/index.html 📝 modified
admin/how-to/install/register-the-admin-vo/index.html 📝 modified
dev/explanations/components/api/index.html 📝 modified
dev/explanations/components/cli/index.html 📝 modified
dev/explanations/components/client/index.html 📝 modified
dev/explanations/components/db/index.html 📝 modified
dev/explanations/components/routes/index.html 📝 modified
dev/how-to/use-the-demo/swagger/index.html 📝 modified
dev/how-to/use-the-demo/web/index.html 📝 modified
user/reference/programmatic-usage/command-line-interface/index.html 📝 modified
user/reference/programmatic-usage/https-interface/index.html 📝 modified
user/reference/programmatic-usage/python-interface/index.html 📝 modified

@fstagni fstagni force-pushed the main_FEAT_deploy-gubbins branch from 923ba11 to 3232bab Compare October 3, 2025 13:28
@fstagni fstagni force-pushed the main_FEAT_deploy-gubbins branch from 3232bab to 1f81e2e Compare October 3, 2025 13:49
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.

Publish gubbins:dev

2 participants