-
Notifications
You must be signed in to change notification settings - Fork 15
feat(temporal): initial support #391
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
Conversation
c73b1e9 to
96d7fa6
Compare
|
@kosmoz I talked with @robholland about the upstream pr. He is eager to merge it, but as he is traveling it will take at least one more week. In the meantime we should either deploy the helm chart to the glasskube charts repo or build the chart and push it to an oci registry so we are not blocked by the upstream. |
|
@pmig @kosmoz I think we also need this change in the helm chart: temporalio/helm-charts#550 – Right now, no temporal namespace is created automatically. |
96d7fa6 to
d20227e
Compare
|
@pmig OK thanks, a version of the chart that includes the TTL and default namespace features is now available on charts.glasskube.eu @christophenne can you check if the package works for you use case? |
christophenne
left a comment
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.
Thanks, when the one type is fixed, it works! 👍
2ccdbd6 to
63ee96b
Compare
Signed-off-by: Jakob Steiner <jakob.steiner@glasskube.eu>
Signed-off-by: Jakob Steiner <jakob.steiner@glasskube.eu>
Signed-off-by: Jakob Steiner <jakob.steiner@glasskube.eu>
Signed-off-by: Jakob Steiner <jakob.steiner@glasskube.eu>
Co-authored-by: Christoph Enne <christoph.enne@glasskube.eu>
Signed-off-by: Philip Miglinci <pmig@glasskube.eu>
63ee96b to
e9f43c9
Compare
c183b47 to
7d54499
Compare
Signed-off-by: Philip Miglinci <pmig@glasskube.eu>
7d54499 to
c5a5002
Compare
Requires: