< plain lunch 50969> please take a look at the release notes Ory Community #ory-selfhosting

<@U017406ACES> please take a look at the release n...

worried-kitchen-94392

07/21/2022, 6:10 AM

@plain-lunch-50969 please take a look at the release notes for https://github.com/ory/k8s/releases/tag/v0.24.0 😉 there was a change in variables. And yes, if you use the new automigration mechanism, you shouldnt need a custom initContaienr anymore

👍 1

plain-lunch-50969

07/21/2022, 2:03 PM

I think the question was whether or not the automigration would pick up the kratos.config during the migration or not. It's still not clear to me if it does or not.

worried-kitchen-94392

07/25/2022, 7:49 AM

yes it does, and it will now. Some parameters like encoding keys in hydra, are important during the migration procedure

brash-hamburger-57986

07/25/2022, 8:57 AM

hi, thanks for clarifying. i am supplying

kratos.config

from a configmap, generated by kustomize. it should be something wrong on my side, but i was trying to figure out why it didn't get picked up:

Copy code

❯ kc get pods -n kratos
NAME                          READY   STATUS                       RESTARTS      AGE
ory-kratos-67764d5d9f-vpgg6   0/1     CrashLoopBackOff             5 (17s ago)   3m7s
ory-kratos-courier-0          0/1     CreateContainerConfigError   0             3m6s

❯ kc logs pod/ory-kratos-757cf48598-5fr5c -c kratos -n kratos

The configuration contains values or keys which are invalid:
The configuration contains values or keys which are invalid:
identity.schemas: <nil>
                  ^-- one or more required properties are missing

The configuration contains values or keys which are invalid:
courier.smtp.connection_uri: <nil>
                             ^-- one or more required properties are missing

The configuration contains values or keys which are invalid:
selfservice.default_browser_return_url: <nil>
                                        ^-- one or more required properties are missing

kratos-config-ttg2582gch

is the generated one:

Copy code

❯ kc get configmap -n kratos
NAME                       DATA   AGE
kratos-config-ttg2582gch   1      5m56s
kube-root-ca.crt           1      29m
ory-kratos-config          2      53s

in the helmrelease, i see it does get referenced:

Copy code

valuesFrom:
  - kind: ConfigMap
    name: kratos-config-ttg2582gch
    targetPath: spec.values.kratos.config
    valuesKey: kratos.yaml

brash-hamburger-57986

07/25/2022, 9:01 AM

however, what's interesting is this error:

Copy code

identity.schemas: <nil>
                  ^-- one or more required properties are missing

i didn't use ConfigMap for this field, i directly added it as a string

Copy code

kratos:
      automigration:
        enable: true
      identitySchemas:
        "identity.schema.json": |
          {
            ...

so why there is such error...

worried-kitchen-94392

07/25/2022, 11:04 AM

Hi there, what version of kratos are you using? There was a change in the keys at some point

worried-kitchen-94392

07/25/2022, 11:04 AM

compare your values with https://github.com/ory/k8s/blob/master/.circleci/values/kratos.yaml you will see that schemas are references by a different key now

brash-hamburger-57986

07/25/2022, 5:17 PM

i am using the latest kratos

0.10.1

and was referencing the latest master

kratos.yaml

with pinned version

v0.10.1

~~i had thought it is a customizable value~~ 😅 so
"identity.schema.json"
is not a valid key, it should be
"identity.default.schema.json"
? actually it shouldn't matter, after comparing, i the fields which log says missing are defined. i have the following in kratos config:

Copy code

identity:
  default_schema_id: default
  schemas:
    - id: default
      url: file:///etc/config/identity/identity.schema.json

3 Views

Open in Slack

Previous Next