|
| 1 | += ADR028: Discovery revision |
| 2 | +Razvan Mihai <razvan.mihai@stackable.tech> |
| 3 | +v0.1, 2023-03-30 |
| 4 | +:status: draft |
| 5 | + |
| 6 | +* Status: {status} |
| 7 | +* Contributors: |
| 8 | +** Felix Hennig |
| 9 | +** Malte Sander |
| 10 | +** Natalie Klestrup Röijezon |
| 11 | +** Razvan Mihai |
| 12 | +** Sebastian Bernauer |
| 13 | +* Date: 2023-02-28 |
| 14 | +
|
| 15 | +== Context and Problem Statement |
| 16 | + |
| 17 | +// Describe the context and problem statement, e.g., in free form using two to three sentences. You may want to articulate the problem in form of a question. |
| 18 | + |
| 19 | +This ADR is written with a specific problems in mind, but the goal is to reach a generic mechanism for the discovery of products. |
| 20 | +The current discovery mechanism is described https://docs.stackable.tech/home/stable/concepts/service_discovery.html[in our docs] (make sure to pick the 23.1 version). |
| 21 | + |
| 22 | +Basically when clients connect to products managed by Stackable, they need to have certain information about how to connect to these products. |
| 23 | +Currently we expose some of these information, but not all (e.g. which ca cert the exposed product uses). |
| 24 | + |
| 25 | +== Decision Drivers |
| 26 | +We have some common use-cases that we need to express via the discovery mechanism: |
| 27 | + |
| 28 | +1. Trino cluster |
| 29 | +* Currently we don't write any discovery CM |
| 30 | +* We need at least |
| 31 | +** Coordinator address (currently only single coordinator is supported) |
| 32 | +*** Ideally split it into the following attributes, so that clients can construct the URI. (trino python client e.g. needs all of these attributes separately and I don't want to rely on parsing and extracting the URI) |
| 33 | +**** protocol (http/https) |
| 34 | +**** host |
| 35 | +**** port |
| 36 | +** What Secretclass must be used to authenticate |
| 37 | +*** null (no SecretClass): Means no authentication at all |
| 38 | +*** (future) static: One of these plain credentials |
| 39 | +*** tls: provides ca.crt that needs to have signed the client |
| 40 | +*** ldap: <whatever> |
| 41 | +*** (future) kerberos: kdc where you can get a ticket from (together with the realm) |
| 42 | +*** (future) oauth: <whatever> |
| 43 | +*** (future) jwt: <whatever> |
| 44 | + |
| 45 | +2. HDFS cluster |
| 46 | +* Currently we expose hdfs-site and core-site |
| 47 | +* We need at least |
| 48 | +** hdfs-site |
| 49 | +** core-site |
| 50 | +** What Secretclass must be used to authenticate |
| 51 | +*** (future) kerberos: kdc where you can get a ticket from (together with the realm) |
| 52 | +** The information about rpc encryption is already in the core-site, so not needed explicitly |
| 53 | +** The information about data encryption is already in the hdfs-site, so not needed explicitly |
| 54 | + |
| 55 | +== Considered Options |
| 56 | + |
| 57 | +=== Option 1 |
| 58 | + |
| 59 | +==== Pros |
| 60 | + |
| 61 | + |
| 62 | +==== Cons |
| 63 | + |
| 64 | +[source,yaml] |
| 65 | +---- |
| 66 | +status: |
| 67 | + conditions: |
| 68 | + - type: Available |
| 69 | + status: "True" |
| 70 | + lastProbeTime: 2023-02-28T14:02:00Z |
| 71 | + lastTransitionTime: 2023-02-28T12:00:00Z |
| 72 | + message: "UI and Postgres DB running" |
| 73 | + - type: Degraded |
| 74 | + status: "True" |
| 75 | + lastProbeTime: 2023-02-28T14:02:00Z |
| 76 | + lastTransitionTime: 2023-02-28T12:00:00Z |
| 77 | + reason: "DruidConnection failed. <Optional: Druid degraded message>" |
| 78 | + - type: Progressing |
| 79 | + status: "True" |
| 80 | + lastProbeTime: 2023-02-28T14:02:00Z |
| 81 | + lastTransitionTime: 2023-02-28T12:00:00Z |
| 82 | + message: "New replicas starting." |
| 83 | + - type: Upgradable |
| 84 | + status: "Unknown" |
| 85 | + lastProbeTime: 2023-02-28T14:02:00Z |
| 86 | + lastTransitionTime: 2023-02-28T12:00:00Z |
| 87 | + - type: Paused |
| 88 | + status: "True" |
| 89 | + lastProbeTime: 2023-02-28T14:02:00Z |
| 90 | + lastTransitionTime: 2023-02-28T12:00:00Z |
| 91 | + message: "User requested reconcile pause." |
| 92 | +---- |
| 93 | + |
| 94 | +== Decision Outcome |
| 95 | + |
| 96 | +TODO |
0 commit comments