While deploying v1.2.1 of `flyte-core` through Hel...
# ask-the-community
a
While deploying v1.2.1 of
flyte-core
through Helm on EKS when
workflow_notifications
are enabled it produces ill-formatted yaml. Steps to reproduce, `test-values.yaml`:
Copy code
userSettings:
  accountRegion: us-east-1
  accountNumber: 123123123
  notifications:
    topicName: topic-name
    queueName: queue-name

workflow_notifications:
  enabled: true
  config:
    notifications:
      type: "aws"
      region: "{{ .Values.userSettings.accountRegion }}"
      publisher:
        topicName: "arn:aws:sns:{{ .Values.userSettings.accountRegion }}:{{ .Values.userSettings.accountNumber }}:{{ .Values.userSettings.notifications.topicName }}"
      processor:
        queueName: "{{ .Values.userSettings.notifications.queueName }}"
        accountId: "{{ .Values.userSettings.accountNumber }}"
      emailer:
        subject: "Flyte: {{ project }}/{{ domain }}/{{ launch_plan.name }} has '{{ phase }}'"
        sender: "{{ .Values.userSettings.notifications.sender }}"
        body: |
          "Execution {{ workflow.project }}/{{ workflow.domain }}/{{ workflow.name }}/{{ name }} has {{ phase }}.
           Details: <https://flyte.example.com/console/projects/{{> project }}/domains/{{ domain }}/executions/{{ name }}.
           {{ error }}"
Then run template:
Copy code
helm template admin flyteorg/flyte-core --version v1.2.1 --values test-values.yaml | grep -A 4 "notifications.yaml"
Which outputs:
Copy code
notifications.yaml: |
    notifications:
      type: aws
      region:us-east-1
      publisher:
By yaml spec the key must be separated with space, so
region:us-east-1
breaks the service deployment and should be
region: us-east-1
instead. Simple workaround is to put an extra space inside templated string
region: " {{ .Values.userSettings.accountRegion }}"
.
s
Thanks Andrew for filing the issue. If you could create a PR as well, that'd be great!
a
I’m not that good at helm, so not 100% sure what’s the issue is, but looks like there is a fix on the way https://github.com/flyteorg/flyte/pull/3297