Skip to content

Fix/discovery server config url#544

Closed
varungopal286 wants to merge 43 commits into
spring-petclinic:mainfrom
PetClinic-Project-Team:fix/discovery-server-config-url
Closed

Fix/discovery server config url#544
varungopal286 wants to merge 43 commits into
spring-petclinic:mainfrom
PetClinic-Project-Team:fix/discovery-server-config-url

Conversation

@varungopal286

Copy link
Copy Markdown

Description

Fixes # (issue)

Type of change

  • Bug fix
  • New feature
  • Documentation update
  • Dependency upgrade

Checklist

  • I have read the CONTRIBUTING guide
  • This is NOT coursework, a student assignment, or homework — I understand that forks of this repo are widely used in courses and that student PRs will be closed without review
  • My change targets the main branch of spring-petclinic/spring-petclinic-microservices, not my own fork
  • I have tested this change locally
  • Existing tests pass (./mvnw test)

⚠️ Are you a student?
If you forked this repo for a course or bootcamp, your changes belong in your own fork, not here.
PRs that are part of a course assignment will be labelled invalid and closed immediately.

Treasurematrix and others added 30 commits May 8, 2026 21:48
…ructure-setup

feat: setup core infrastructure (VPC, IAM, EKS, ECR)
…very-server

- config-server: Deployment + ClusterIP Service (port 8888)
- discovery-server: Deployment + ClusterIP Service (port 8761)
- discovery-server uses initContainer to wait for config-server
- Both services use docker Spring profile
- README with full setup, testing steps and pending items
- Image URLs pending AWS Account ID from P2

Refs #3
- Updated ECR image URLs with AWS account ID
- config-server: deployed and verified on EKS (1/1 Running)
- discovery-server: deployed and verified on EKS (1/1 Running)
- All smoke tests passed (7/7)
- Updated README with completed status and smoke test results
- Removed pending items, no sensitive info in docs

Closes #3
…nfig-discovery

feat(issue-3): Config Server and Discovery Server K8s deployment
…ity-monitoring

Integrated observability and monitoring capabilities into Spring PetClinic Microservices.

Changes included:
- Added Prometheus metrics integration
- Configured Spring Boot Actuator endpoints
- Added Micrometer Prometheus registry
- Enabled monitoring configurations across services
- Added observability documentation
- Validated setup locally using Docker Compose

Outcome:
Improved system visibility, metrics collection, and monitoring readiness.
…stomer/vets-services

Feature/develop customer/vets services
…line

Feature/ci cd pipeline-CI/CD: Add GitHub Actions pipeline for ECR and EKS
1
Updated application name and management endpoint configuration.
2
Consolidate management endpoint exposure settings and enable health details.
3
Updated management endpoints configuration to expose health and prometheus metrics.
…stomer/vets-services

Feature/develop customer/vets services
Theodora-Okonkwo and others added 13 commits May 16, 2026 17:14
…manifests

- All k8s manifests now target the spring-petclinic namespace (matches live cluster)
- Fixed broken shell-variable image refs in visits-service, api-gateway, genai-service, admin-server
- Removed invalid imagePullSecrets entries (ecr-secret does not exist)
- Updated CI/CD K8S_NAMESPACE to spring-petclinic so future deployments target correct namespace
- visits-service and admin-server now use springcommunity public images (consistent with customers/vets)
- api-gateway and genai-service use hardcoded ECR registry URL (consistent with config/discovery)
…spring-petclinic

fixed: correct namespace from petclinic to spring-petclinic across all …
Secret was using placeholder <BASE64_PASSWORD> causing CreateContainerConfigError
on pods that reference mysql-cred. Updated with actual base64-encoded credentials
used by the running cluster (username: petclinic, password: petclinic).
fix: set correct mysql-cred secret credentials in mysql-secret.yaml
…ity-monitoring

Added monitoring and observability validation updates, namespace configuration fixes, and Kubernetes service alignment for Grafana, Prometheus, Zipkin, Config Server, and Discovery Server deployments.
…rofile

fix: enable MySQL profile for all services with per-service datasourc…
…rofile

fix: prevent Hibernate DDL override and add ALB ingress for external …
@sonarqubecloud

Copy link
Copy Markdown

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.

6 participants