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

feat(teleport): update monolithic microk8s example to show how to deploy OPA #23

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
51 changes: 39 additions & 12 deletions ansible/monolithic_mk8s_example/teleport-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,9 @@ appTeleport:
project: default

source:
repoURL: https://harbor.ukserp.ac.uk/chartrepo/dare
repoURL: https://harbor.ukserp.ac.uk/chart/dare
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be repoURL: harbor.ukserp.ac.uk/dare/chart

chart: teleport
targetRevision: 1.5.1
targetRevision: 1.10.1

values:
appGuacamole:
Expand All @@ -23,7 +23,7 @@ appTeleport:
source:
repoURL: https://hub.jupyter.org/helm-chart/
chart: jupyterhub
targetRevision: 3.0.2
targetRevision: 3.1.0
values:
singleuser:
lifecycleHooks:
Expand All @@ -45,25 +45,36 @@ appTeleport:
profileList:
- display_name: "basic image"
kubespawner_override:
image: "harbor.ukserp.ac.uk/dare/jupyter:1.1.5"
image: "harbor.ukserp.ac.uk/dare/jupyter:pr-59-2f7bf0e"
default_url: '/lab'
cmd: ['jupyterhub-singleuser', '--ContentsManager.allow_hidden=True']

appTrino:
enabled: true
namespace: trino
source:
repoURL: https://harbor.ukserp.ac.uk/chartrepo/dare
repoURL: harbor.ukserp.ac.uk/chart/dare
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be repoURL: harbor.ukserp.ac.uk/dare/chart

chart: trino
targetRevision: 1.3.1
targetRevision: 1.6.0
values:
image:
repository: harbor.ukserp.ac.uk/dare/trino
tag: "1.3.1"
tag: 1.5.0

server:
workers: 2
config:
spillToDisk:
enabled: false
# THIS IS THE NEW SECTION
accessControl:
enabled: true
properties:
access-control.name: "tech.stackable.trino.opa.OpaAuthorizer"
# This assumes that opa-operator is deployed in the same cluster as Trino, in a namespace called "dare-opa".
# If your opa operator is deployed in a different namespace, then you'll need to change this.
opa.policy.uri: "http://opa-cluster.dare-opa.svc.cluster.local:8081/v1/data/trino/allow"
security.refresh-period: "15s"
query:
maxMemory: "2GB"
# maxmemorypernode + heapHeadroomPerNode <= coordination|worker.jvm.maxheapsize
Expand Down Expand Up @@ -138,13 +149,13 @@ appTeleport:
# hive needs to go into the trino namespace
namespace: trino
source:
repoURL: https://harbor.ukserp.ac.uk/chartrepo/dare
repoURL: harbor.ukserp.ac.uk/chart/dare
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be repoURL: harbor.ukserp.ac.uk/dare/chart

chart: hive
targetRevision: 1.0.0
targetRevision: 1.3.3
values:
image:
name: harbor.ukserp.ac.uk/dare/hive
tag: "1.0.0"
tag: "1.3.3"

# In the below block, make sure you change these:
# accessKey: 06LVN2MwcfLzsFr7BZ
Expand Down Expand Up @@ -209,9 +220,25 @@ appTeleport:
appVault:
enabled: false

# don't deploy OPA, not needed
# DO deploy OPA
appOpa:
enabled: false
enabled: true
namespace: opa
source:
repoURL: harbor.ukserp.ac.uk/dare/chart
chart: opa
targetRevision: 1.4.6
values:
rules:
host.rego: |
package trino

import future.keywords.if
import future.keywords.in

# as this is a sample allow rule, it just lets anyone in
default allow := true


# this app isn't complete yet
appControl:
Expand Down