-
Notifications
You must be signed in to change notification settings - Fork 514
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
Add helm chart #1503
Open
patoarvizu
wants to merge
56
commits into
notaryproject:master
Choose a base branch
from
patoarvizu:add_helm_chart
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Add helm chart #1503
Changes from 1 commit
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
3f734d2
First attempt at chart
patoarvizu aa873f4
Minimize duplication by moving if / else logic
patoarvizu 5922c38
Consolidate storage values in a single map
patoarvizu c7a05b0
Only deploy signer if 'trust' is set to 'remote'
patoarvizu c01f19c
Remove unnecessary template
patoarvizu 794014c
Take signer alias secret as a parameter and reference it from the Dep…
patoarvizu 6614999
Add option to configure 'gun_prefixes'
patoarvizu de33e4c
Use values for ports
patoarvizu 637670a
Only create PVC if 'storage' is not 'memory'
patoarvizu e871562
Parameterize PVC size
patoarvizu 1a4ceed
Parameterize and pin versions of server, signer, and migrate
patoarvizu d0485c3
Missed some ports
patoarvizu 40732f8
Add annotations to the ingress via values
patoarvizu a16ef65
Fix signer service name to 'notarysigner'
patoarvizu 58456ed
Remove 'fullname' prefix from all resources to make it consistent wit…
patoarvizu 125ad55
Add better labels to Deployments and Services
patoarvizu 27aa2fb
Enable load balancer service too
patoarvizu 611a7d1
Set 'command' to 'notary-signer' directly
patoarvizu 1b3e89f
Add '.tpl' extension to config files
patoarvizu 0a306d6
Hard-code not-so-secret aliasSecret
patoarvizu a609888
Add Helm section to README
patoarvizu f570e96
Merge branch 'master' into add_helm_chart
patoarvizu 8fed2fb
Merge branch 'master' into add_helm_chart
patoarvizu 6b19514
Merge branch 'master' into add_helm_chart
patoarvizu 7d93c5f
Add 'storage.className' parameter
patoarvizu 88fe64d
Conditionally set 'loadBalancerIP' on the notaryserver service
patoarvizu 1b39517
Add comments to the chart's values.yaml file
patoarvizu a161240
Add option to provide custom certificates or let Helm auto-generate them
patoarvizu aac6862
Deleting symlink to fixtures
patoarvizu 541a81a
Point 'client_ca_file' to the root CA
patoarvizu 12ba72b
Move server and signer configuration directly into the configmap temp…
patoarvizu fcac30b
Add authentication options
patoarvizu 1361d97
Add caching options
patoarvizu e88e83c
Rewrite section about the chart not being prod-ready
patoarvizu eec0f16
Split storage options to allow for remote storage
patoarvizu 647c622
Add options to configure server and signer to use remote storage
patoarvizu 53bd14c
Add option to specify secret to get the alias passphrase instead of a…
patoarvizu 60b4f42
Move migrations to jobs instead of init containers on deployments
patoarvizu c268c82
Start parameterizing database passwords
patoarvizu be04c2e
Parameterize the initdb passwords so they're not in plain text in the…
patoarvizu 5efb93a
Now parameterize Notary storage configuration
patoarvizu 1734495
Remove unneeded volumes
patoarvizu 872c053
Use double '%' everywhere for consistency
patoarvizu e5774ab
Clean up storage container so thereare no hard-coded credentials; fix…
patoarvizu e14c189
Use 'template' instead of templating inline
patoarvizu 35e573c
Add comments to storage credential values
patoarvizu 78edc48
Simplify database configuration
patoarvizu 57d2753
Merge branch 'master' into add_helm_chart
patoarvizu c925f19
Add hyphen to be consistent
patoarvizu dd87d94
Parameterize number of replicas
patoarvizu 78665b1
Add pod anti-affinity options
patoarvizu f689f91
Add node affinity options
patoarvizu 8685262
Rename some default secrets and make them non-optional
patoarvizu adcd28b
Merge branch 'master' into add_helm_chart
patoarvizu 81efdc9
PR comments
patoarvizu e2fc091
Merge branch 'master' into add_helm_chart
patoarvizu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Add pod anti-affinity options
Signed-off-by: Pato Arvizu <patoarvizu@gmail.com>
- Loading branch information
commit 78665b15adffc7ac949316b5a4fd572e58a3e38c
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In notary-server component, you are assigning signer.replicas instead of server.replicas