-
Notifications
You must be signed in to change notification settings - Fork 9
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
✨ Add issuer/verifier image_url
to trustregistry
#983
Conversation
cf2195b
to
c2aa5ba
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is alembic
?
Is the script just for a once-off migration, and can then be removed?
116a468
to
abea83d
Compare
|
7760330
to
d9a8c50
Compare
trustregistry/migrations/versions/5bcfb2c0bc05_actor_image_url.py
Dismissed
Show dismissed
Hide dismissed
7226e2c
to
64affc5
Compare
image_url
to trustregistryimage_url
to trustregistry
64affc5
to
64aed40
Compare
K8s Regression Test Coverage
|
|
Add
image_url
to actor modelsHandel new
image_url
in update tenant call and update call on TRFix/update tests
Update pre-commit
Added
alembic
a lightweight database migration tool for usage with SQLAlchemyDid the following for setup:
alembic init migrations
sqlalchemy.url
inalembic.ini
Base
fromtrustregistry/db.py
totrustregistry/migrations/env.py
set
target_metadata = Base.metadata
in env.py (alembic can see models now)alembic revision --autogenerate -m "Actor image_url"
to create migration.Can see migrations generated in
trustregistry/migrations/versions
Can now run:
alembic upgrade head
to run migrationor
alembic downgrade -1
to undo last migration