Oauth2-proxy: Helm Chart needs critical attention

Created on 23 Nov 2020  路  5Comments  路  Source: oauth2-proxy/oauth2-proxy

It looks like there's a discussion in an existing issue covering this, but the existing issue is broad, the discussion is leisurely, and unfortunately this specific issue has become critical.

https://github.com/oauth2-proxy/oauth2-proxy/issues/608

The "stable" helm repository is now EOL, and should no longer be used. This means the stable/oauth2-proxy chart should also no longer be used.

This is problematic for two reasons:

1) The stable repository, and the stable/oauth2-proxy chart specifically are used by this very project
2) As this project has not provided an official method for Kubernetes deployments, the chart has been the defacto standard for deploying oauth2-proxy with Helm. This means the EOL has rendered oauth2-proxy unusable for most Helm users.

I understand there's a few considerations going into this:

  • The stable/oauth2-proxy chart is not maintained by this repository or this organization, and is a legacy from when this project was owned by bitly.
  • There are potentially licensing issues around the chart
  • There is no repository at the moment to put this chart into.
  • Maintainers for the chart need to be found.

If we can get a clear direction on where to go with this, I would be glad to contribute work towards that direction.

Most helpful comment

Just raised a PR to add the license and introduce CI stuff for lint-testing and making a release using gh-pages. https://github.com/oauth2-proxy/manifests/pull/1

We rely on this chart at eLife and I'm happy to help with further PRs to get the chart up to date with the current release of oauth2-proxy.

All 5 comments

I have created this PR to address thew fact that this repository was still using the old stable chart URL: https://github.com/oauth2-proxy/oauth2-proxy/pull/931

Please keep in mind this is only a dirty fix, and the use of the stable repository is actively advised against by the maintainers.

We have an outstanding question regarding licensing with the original maintainer of the 3rd party chart. Once that's resolved, we have a dedicated repo under the project ready for the chart community to help migrate it over: https://github.com/oauth2-proxy/manifests

I'd actually suggest at this point that we just try and do the migration without attempting to change the license. I don't think it's going to be an issue having one license for the manifests and a different license for the project itself.

CC @desaintmartin @tlawrie

FYI I created a branch that copies the old chart over preserving the history, we need to add the license to this, and I assume a bunch of CI stuff etc, but this has the basic chart at least, perhaps someone who knows helm better can advise what we need to do now https://github.com/oauth2-proxy/manifests/tree/migration

Just raised a PR to add the license and introduce CI stuff for lint-testing and making a release using gh-pages. https://github.com/oauth2-proxy/manifests/pull/1

We rely on this chart at eLife and I'm happy to help with further PRs to get the chart up to date with the current release of oauth2-proxy.

Was this page helpful?
0 / 5 - 0 ratings