Skip to content

Add abilty to configure postStart handler for teleport-cluster chart#7168

Merged
webvictim merged 10 commits intogravitational:masterfrom
stefansedich:teleport-cluster-postStart
Aug 20, 2021
Merged

Add abilty to configure postStart handler for teleport-cluster chart#7168
webvictim merged 10 commits intogravitational:masterfrom
stefansedich:teleport-cluster-postStart

Conversation

@stefansedich
Copy link
Copy Markdown
Contributor

This PR introduces the ability to configure a postStart handler. We are using this to configure a script that will run tctl create of our roles and provider config when the server starts.

@russjones
Copy link
Copy Markdown
Contributor

@webvictim Can you review?

Copy link
Copy Markdown
Contributor

@webvictim webvictim left a comment

Choose a reason for hiding this comment

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

@stefansedich Changes look good to me.

Please also add this value to the Helm chart reference at https://github.com/gravitational/teleport/blob/master/docs/pages/kubernetes-access/helm/reference.mdx so that it's kept up to date.

Thanks for the PR!

@stefansedich stefansedich force-pushed the teleport-cluster-postStart branch from c2a8b90 to df46530 Compare June 25, 2021 17:58
@stefansedich
Copy link
Copy Markdown
Contributor Author

@stefansedich Changes look good to me.

Please also add this value to the Helm chart reference at https://github.com/gravitational/teleport/blob/master/docs/pages/kubernetes-access/helm/reference.mdx so that it's kept up to date.

Thanks for the PR!

@webvictim that should be done.

@webvictim
Copy link
Copy Markdown
Contributor

@stefansedich Thanks - this will be merged once it has another approving review.

Comment thread docs/pages/kubernetes-access/helm/reference.mdx
@webvictim webvictim enabled auto-merge (squash) June 30, 2021 17:10
auto-merge was automatically disabled July 28, 2021 20:47

Head branch was pushed to by a user without write access

@stefansedich
Copy link
Copy Markdown
Contributor Author

@klizhentas anything I need to do to get CI running?

@webvictim
Copy link
Copy Markdown
Contributor

@stefansedich External PRs require approval to run, I just hit the start button.

@webvictim webvictim enabled auto-merge (squash) August 19, 2021 21:30
@webvictim webvictim merged commit 8fb89a2 into gravitational:master Aug 20, 2021
webvictim added a commit that referenced this pull request Aug 20, 2021
webvictim added a commit that referenced this pull request Aug 26, 2021
* Add abilty to configure postStart handler for teleport-cluster chart (#7168)

* Add support for PDB with the teleport-cluster helm chart (#7138)

* Allow teleport-cluster-agent chart to use an existing volume for the data directory (#7096)

* Mount teleport-tls to the init container for the teleport-cluster helm chart (#7166)

* add support for running agent helm chart on persistent volume (#7123)

* helm: Make auth type configurable (#7508)

* Add imagePullSecrets in kube-agent chart (#6941)

Co-authored-by: Gus Luxton <gus@goteleport.com>
Co-authored-by: Stefan Sedich <stefan.sedich@gmail.com>
Co-authored-by: Kevin Nisbet <kevin@gravitational.com>
Co-authored-by: Antonio Gurgel <66754096+antonio-te@users.noreply.github.com>
Co-authored-by: Aymen Memni <7290430+amemni@users.noreply.github.com>
zmb3 pushed a commit that referenced this pull request Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants