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

Legacy DB Migration | Store Client credentials in Gatehouse DB #130

Merged
merged 3 commits into from
Mar 31, 2025

Conversation

AshCorr
Copy link
Member

@AshCorr AshCorr commented Mar 14, 2025

What does this change?

Add 2 new migrations:

  • Create a identity_admin_api user with IAM login credentials to allow Identity Admin API to connect to Gatehoues
  • Create a clients and client_access_tokens table which mimics the existing clients and clientsaccesstokens table in Identity DB.

How to test?

Run ./db/test.sh

@AshCorr AshCorr force-pushed the ash/add-client-credentials branch 2 times, most recently from 631fc77 to a85b1b6 Compare March 14, 2025 15:46
@AshCorr AshCorr force-pushed the ash/add-client-credentials branch from 0639202 to 5028603 Compare March 27, 2025 16:45
@AshCorr AshCorr marked this pull request as ready for review March 27, 2025 16:47
@AshCorr AshCorr requested a review from a team as a code owner March 27, 2025 16:47
@AshCorr AshCorr force-pushed the ash/add-client-credentials branch from 5028603 to 78a1f5a Compare March 27, 2025 17:05
]
environment:
JAVA_ARGS: -XX:UseSVE=0 -XX:+IgnoreUnrecognizedVMOptions
Copy link
Member Author

Choose a reason for hiding this comment

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

Flyby fix, Docker (atleast Colima) is struggling to run Java images right now on our new macbooks: abiosoft/colima#965

The UseSVE flag only exists on Arm versions of Java, hence also requiring IgnoreUnrecognizedVMOptions to allow Java to run on the x86_64 Github Actions runner.

@AshCorr AshCorr force-pushed the ash/add-client-credentials branch from 78a1f5a to bd00e18 Compare March 27, 2025 17:18
@AshCorr AshCorr merged commit 706deb4 into main Mar 31, 2025
3 checks passed
@AshCorr AshCorr deleted the ash/add-client-credentials branch March 31, 2025 09:24
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