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

[Helm] Add Lightbend's helm chart definition #234

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pedro93
Copy link

@pedro93 pedro93 commented May 23, 2021

Adds Lightbend's helm chart definition of this repository as the original repository is no longer maintained as mentioned in this issue and following a suggestion from the slack channel.

This is a direct copy, no changes to the underlying code have been made.
Disclaimer: I am not the owner or maintainer of either repository.

Feel free to suggest appropriate changes, I will do my best to follow through.
Thank you.

@anandswaminathan
Copy link
Contributor

@maghamravi Can you take a 👀 as well ?

- flink
home: https://github.com/lyft/flinkk8soperator
maintainers:
- name: Lightbend Cloudflow Team
Copy link
Contributor

Choose a reason for hiding this comment

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

I respect Lightbend Cloudflow team for their contribution. Curious, do we a have their approval to be listed as maintainers ?

name: flink-operator
description: A Helm chart for Flink operator
version: 0.8.2
appVersion: 0.4.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Thoughts on having the chart config and templates against master version 0.5.0 ? Ideal to keep it in sync to avoid customer surprises.

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