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

Create Helm chart for Nessie #1677

Merged
merged 2 commits into from
Aug 5, 2021
Merged

Create Helm chart for Nessie #1677

merged 2 commits into from
Aug 5, 2021

Conversation

nastra
Copy link
Collaborator

@nastra nastra commented Jul 30, 2021

This is just a fairly simple MVP version so that Nessie can be installed via Helm.


This change is Reviewable

@nastra nastra marked this pull request as draft July 30, 2021 09:48
@codecov
Copy link

codecov bot commented Jul 30, 2021

Codecov Report

Merging #1677 (d83fc08) into main (d01133f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #1677   +/-   ##
=========================================
  Coverage     79.74%   79.74%           
  Complexity     2399     2399           
=========================================
  Files           297      297           
  Lines         14117    14117           
  Branches       1050     1050           
=========================================
  Hits          11258    11258           
  Misses         2349     2349           
  Partials        510      510           
Flag Coverage Δ
java 79.24% <ø> (ø)
python 86.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d01133f...d83fc08. Read the comment docs.

@nastra nastra marked this pull request as ready for review July 30, 2021 13:11
@nastra nastra requested a review from rymurr August 2, 2021 09:05
helm/README.md Show resolved Hide resolved
helm/nessie/templates/NOTES.txt Outdated Show resolved Hide resolved
helm/nessie/templates/deployment.yaml Outdated Show resolved Hide resolved
# runAsNonRoot: true
# runAsUser: 1000

service:
Copy link
Collaborator

Choose a reason for hiding this comment

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

An example of configuring this with a real ingest would be good for the docs

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll try and add a script that generates some data/commits and update the docs in a separate PR

Copy link
Collaborator

Choose a reason for hiding this comment

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

by ingest I meant nginx and possibly ssl as opposed to ingesting data w/ nessie

helm/nessie/values.yaml Outdated Show resolved Hide resolved
helm/nessie/values.yaml Show resolved Hide resolved
servers/quarkus-server/pom.xml Outdated Show resolved Hide resolved
servers/quarkus-server/pom.xml Outdated Show resolved Hide resolved
@nastra nastra force-pushed the k8s branch 2 times, most recently from 4ab9465 to 270000c Compare August 4, 2021 09:48
@nastra nastra requested a review from rymurr August 5, 2021 07:56
- name: AWS_REGION
value: {{ .Values.dynamodb.region }}
{{- end }}
- name: AWS_ACCESS_KEY_ID
Copy link
Collaborator

Choose a reason for hiding this comment

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

will these environment variables show up in logs anywhere? Or in a docker ps type command? I believe the image is set up to read from all the standard locations for AWS config so we could also write to ~/.aws/credentials if these env variables would be visible anywhere.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm not aware that those env vars would be visible. They will only show up on the Deployment/Pod like this:

    Environment:
      NESSIE_VERSION_STORE_TYPE:  DYNAMO
      AWS_REGION:                 us-west-2
      AWS_ACCESS_KEY_ID:          <set to the key 'aws_access_key_id' in secret 'awscreds'>      Optional: false
      AWS_SECRET_ACCESS_KEY:      <set to the key 'aws_secret_access_key' in secret 'awscreds'>  Optional: false

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

actually inside the POD the secret will be exposed as a normal env var as described in https://kubernetes.io/docs/concepts/configuration/secret/#consuming-secret-values-from-environment-variables

@nastra nastra merged commit f0bdb68 into projectnessie:main Aug 5, 2021
@nastra nastra deleted the k8s branch August 5, 2021 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants