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

datastore.applyMigrations value is not working #161

Closed
jpnorenam opened this issue Oct 5, 2024 · 2 comments · Fixed by #187
Closed

datastore.applyMigrations value is not working #161

jpnorenam opened this issue Oct 5, 2024 · 2 comments · Fixed by #187

Comments

@jpnorenam
Copy link

jpnorenam commented Oct 5, 2024

openfga-migrate BatchJob is not getting created for some reason. As a workaround, I had to go to the openfga ReplicaSet and modify the openfga container arguments from run to migrate to get the migration done, then change it back to run.

source:
  repoURL: https://openfga.github.io/helm-charts
  targetRevision: 0.2.13
  helm:
    valueFiles:
      - values.yaml
    parameters:
      - name: service.type
        value: ClusterIP
      - name: datastore.engine
        value: postgres
      - name: datastore.uri
        value: >-
          postgres://postgres:[email protected]:5432/postgres?sslmode=disable
      - name: postgresql.enabled
        value: 'true'
      - name: postgresql.auth.postgresPassword
        value: password
      - name: postgresql.auth.database
        value: postgres
      - name: datastore.waitForMigrations
        value: 'false'
      - name: replicaCount
        value: '1'
      - name: datastore.applyMigrations
        value: 'true'
  chart: openfga
@98jan
Copy link

98jan commented Nov 16, 2024

I set in the values.yaml following entry, then the migration did work:

  datastore:
    engine: postgres
    uri: "..."
    migrationType: "job" # espacially this entry was needed, for it to work

nsklikas added a commit to canonical/identity-platform-admin-ui that referenced this issue Nov 21, 2024
nsklikas added a commit to canonical/identity-platform-admin-ui that referenced this issue Nov 21, 2024
@whoisxx
Copy link
Contributor

whoisxx commented Jan 28, 2025

@jpnorenam Hi, I will close this issue since this PR is merged for fix.
you could try next helm chart version. let me know if you still have issue

@whoisxx whoisxx closed this as completed Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants