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

feat: ability to pull from your own docker registry using a secret #14686

Merged
merged 7 commits into from
May 26, 2021

Conversation

jmistry
Copy link
Contributor

@jmistry jmistry commented May 18, 2021

SUMMARY

Add ability to pull images from remote package registries that need image pull secrets. This is needed in many corporate environments where image copies are kept on the private registries

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

from the root of the repo execute the following
helm template test helm/superset

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

@junlincc junlincc changed the title feature: ability to pull from your own docker registry using a secret feat: ability to pull from your own docker registry using a secret May 18, 2021
Copy link
Member

@craig-rueda craig-rueda left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor Author

@jmistry jmistry left a comment

Choose a reason for hiding this comment

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

I have updated the helm chart version as per the linting and chart test failure

Copy link
Contributor

@danielewood danielewood left a comment

Choose a reason for hiding this comment

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

LGTM

@nytai nytai merged commit 9fe0222 into apache:master May 26, 2021
cccs-RyanS pushed a commit to CybercentreCanada/superset that referenced this pull request Dec 17, 2021
…pache#14686)

* feature: ability to pull from your own docker registry using a secret

* feature: ability to pull from your own docker registry using a secret

* fix: formating

* fix: formating

* use secret for all deployments and jobs that use the helm chart

* update chart version number

Co-authored-by: jaisheel <jaisheel.mistry@partner.bmw.co.za>
QAlexBall pushed a commit to QAlexBall/superset that referenced this pull request Dec 29, 2021
…pache#14686)

* feature: ability to pull from your own docker registry using a secret

* feature: ability to pull from your own docker registry using a secret

* fix: formating

* fix: formating

* use secret for all deployments and jobs that use the helm chart

* update chart version number

Co-authored-by: jaisheel <jaisheel.mistry@partner.bmw.co.za>
cccs-rc pushed a commit to CybercentreCanada/superset that referenced this pull request Mar 6, 2024
…pache#14686)

* feature: ability to pull from your own docker registry using a secret

* feature: ability to pull from your own docker registry using a secret

* fix: formating

* fix: formating

* use secret for all deployments and jobs that use the helm chart

* update chart version number

Co-authored-by: jaisheel <jaisheel.mistry@partner.bmw.co.za>
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.3.0 labels Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/S 🚢 1.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants