Skip to content

Conversation

@mvoitko
Copy link
Contributor

@mvoitko mvoitko commented Jun 22, 2021

SUMMARY

Add possibility to add env variables from multiple secrets in the Helm chart

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

helm install superset --values ./values.yaml

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@mvoitko mvoitko force-pushed the feature/add-env-variables-from-multiple-secrets-in-helm-chart branch from be66dc6 to 2c82312 Compare June 23, 2021 21:29
@mvoitko mvoitko changed the title Add possibility to add env variables from multiple secrets in the Helm chart feat: add possibility to add env variables from multiple secrets in the Helm chart Jun 24, 2021
@mvoitko mvoitko force-pushed the feature/add-env-variables-from-multiple-secrets-in-helm-chart branch from 2c82312 to f1fc3d6 Compare June 27, 2021 10:09
@amitmiran137 amitmiran137 requested a review from craig-rueda June 27, 2021 20:20
@mvoitko mvoitko force-pushed the feature/add-env-variables-from-multiple-secrets-in-helm-chart branch from f1fc3d6 to 7732901 Compare June 28, 2021 13:19
kgabryje and others added 12 commits June 28, 2021 16:31
apache#15385)

* feat(native-filters): Hide non-numeric columns in numeric range filter

* Return true if type_generic undefined

* Code review comments

* Replace any with string

* fix tests

* add missing columns to select

Co-authored-by: Ville Brofeldt <ville.v.brofeldt@gmail.com>
…pache#15407)

* fix removed flask_app property can break derived class

* Update __init__.py
…ore view (Allow more than 50 columns to be shown) (apache#15153)

* Adding a show all button to the column/metrics list in the explore view

* Update superset-frontend/src/explore/components/DatasourcePanel/index.tsx

Co-authored-by: cccs-rc <62034438+cccs-rc@users.noreply.github.com>

* Update superset-frontend/src/explore/components/DatasourcePanel/index.tsx

Co-authored-by: cccs-rc <62034438+cccs-rc@users.noreply.github.com>

* Fixing typo

Co-authored-by: cccs-rc <62034438+cccs-rc@users.noreply.github.com>
* initial

* updated release notes with more content, more media. Added unorganized list of PRs to full changelog.

* Finalize release notes and organized changelog for 1.2

* lint removal

* Update RELEASING/release-notes-1-2/README.md

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>

* Update RELEASING/release-notes-1-2/README.md

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>

* Update RELEASING/release-notes-1-2/README.md

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>

* Update RELEASING/release-notes-1-2/README.md

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>

* Update RELEASING/release-notes-1-2/README.md

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>

* Update RELEASING/release-notes-1-2/README.md

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>

Co-authored-by: Srini Kadamati <skadamat@gmail.com>
Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>
* slovenian translation - config file

* slovenian translation - po, json
* [SIP] Update SIP template

* Update sip.md

* Update .github/ISSUE_TEMPLATE/sip.md

Co-authored-by: Erik Ritter <erik.ritter@airbnb.com>

Co-authored-by: Amit Miran <47772523+amitmiran137@users.noreply.github.com>
Co-authored-by: Erik Ritter <erik.ritter@airbnb.com>
apache#15411)

* fix(native-filters): show human readable time grain label in indicator

* lint

* simplify
* refactor load configuration

* refactor init configuration to enable recursive merge

* Update config.py

* Update app.py

* fix lint and black issue
@mvoitko mvoitko force-pushed the feature/add-env-variables-from-multiple-secrets-in-helm-chart branch from d092255 to 19c6e65 Compare June 28, 2021 13:31
@mvoitko mvoitko closed this Jun 28, 2021
@mvoitko mvoitko deleted the feature/add-env-variables-from-multiple-secrets-in-helm-chart branch June 28, 2021 13:45
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.

10 participants