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

Initialize services for infrahub db update-core-schema #5186

Merged
merged 2 commits into from
Dec 10, 2024

Conversation

LucasG0
Copy link
Contributor

@LucasG0 LucasG0 commented Dec 10, 2024

I manually tested that it fixes the issue. Ideally, we should have some automated tests for infrahub db commands.

@github-actions github-actions bot added the group/backend Issue related to the backend (API Server, Git Agent) label Dec 10, 2024
@@ -28,6 +28,7 @@ services:
com.github.job: "${JOB_NAME:-unknown}"
ports:
- "${INFRAHUB_DB_BACKUP_PORT:-6362}:6362"
- "${INFRAHUB_DB_PORT:-7687}:7687"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is necessary to test infrahub db commands against infra started through invoke dev.start

Copy link
Collaborator

Choose a reason for hiding this comment

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

Currently the recommendation is to use an override file to expose the ports that you need
mine looks like this

  database:
    ports:
      - "7474:7474"
      - "7687:7687"
  message-queue:
    ports:
      - "5672:5672"
      - "15672:15672"
  cache:
    ports:
      - "6379:6379"
  task-manager:
    ports:
      - "4200:4200"

@LucasG0 LucasG0 requested a review from a team December 10, 2024 16:32
Copy link

codspeed-hq bot commented Dec 10, 2024

CodSpeed Performance Report

Merging #5186 will not alter performance

Comparing lgu-fix-infrahub-db-update (fe830fb) with stable (afb7c5c)

Summary

✅ 10 untouched benchmarks

@LucasG0 LucasG0 merged commit 6470179 into stable Dec 10, 2024
28 checks passed
@LucasG0 LucasG0 deleted the lgu-fix-infrahub-db-update branch December 10, 2024 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
group/backend Issue related to the backend (API Server, Git Agent)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants