Skip to content

Remove odre-server#631

Open
basseche wants to merge 1 commit into
mainfrom
remove-odre-server
Open

Remove odre-server#631
basseche wants to merge 1 commit into
mainfrom
remove-odre-server

Conversation

@basseche

@basseche basseche commented Jul 9, 2026

Copy link
Copy Markdown

PR Summary

Remove odre-server

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 28573647-5f21-4d0f-8b24-bf85d2e5c42a

📥 Commits

Reviewing files that changed from the base of the PR and between 173e206 and f0c0c1c.

📒 Files selected for processing (8)
  • .github/workflows/main.yml
  • README.md
  • docker-compose/study/docker-compose.study.yml
  • k8s/live/local/kustomization.yaml
  • k8s/live/local/odre-server-ingress.yaml
  • k8s/resources/study/kustomization.yaml
  • k8s/resources/study/odre-server-deployment.yaml
  • k8s/resources/study/odre-server-service.yaml
💤 Files with no reviewable changes (7)
  • k8s/resources/study/odre-server-deployment.yaml
  • k8s/live/local/odre-server-ingress.yaml
  • k8s/resources/study/odre-server-service.yaml
  • k8s/live/local/kustomization.yaml
  • .github/workflows/main.yml
  • k8s/resources/study/kustomization.yaml
  • docker-compose/study/docker-compose.study.yml

📝 Walkthrough

Walkthrough

The odre-server service is removed across the CI workflow's repository_dispatch trigger, Docker Compose study configuration, Kubernetes deployment/service/ingress manifests, and their kustomization resource lists. The kustomization configMapGenerator swaps its odre-server config entry for a cgmes-gl-server-configmap-specific one. README documentation is updated to remove odre-server references and adjust data initialization instructions.

Changes

odre-server removal

Layer / File(s) Summary
CI trigger cleanup
.github/workflows/main.yml
Removes the odre_server_updated repository_dispatch event type from the workflow trigger list.
Docker Compose service removal
docker-compose/study/docker-compose.study.yml
Deletes the odre-server service definition, so compose content flows directly from cgmes-gl-server to gridstudy-app.
Kubernetes manifests and kustomization cleanup
k8s/live/local/kustomization.yaml, k8s/live/local/odre-server-ingress.yaml, k8s/resources/study/kustomization.yaml, k8s/resources/study/odre-server-deployment.yaml, k8s/resources/study/odre-server-service.yaml
Removes the odre-server-ingress.yaml, deployment, and service manifests along with their kustomization resource entries; replaces the odre-server-configmap-specific generator with cgmes-gl-server-configmap-specific sourcing config/cgmes-gl-server-application.yml.
Documentation updates
README.md
Removes odre-server from the Docker Compose profiles table and Swagger URL lists (localhost and Minikube), and updates data initialization prerequisites/commands to rely on postgres, geo-data-server, and network-modification-server instead of odre-server.

Suggested reviewers: EtienneLt

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and accurately summarizes the main change: removing odre-server.
Description check ✅ Passed The description is related to the changeset and matches the removal of odre-server.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@basseche basseche self-assigned this Jul 9, 2026
@basseche
basseche requested a review from ghazwarhili July 13, 2026 09:06
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