Skip to content
This repository was archived by the owner on Jan 29, 2025. It is now read-only.

Helm chart support for ingressClass #1605

Open
gberche-orange opened this issue Apr 8, 2022 · 2 comments
Open

Helm chart support for ingressClass #1605

gberche-orange opened this issue Apr 8, 2022 · 2 comments
Assignees

Comments

@gberche-orange
Copy link
Contributor

gberche-orange commented Apr 8, 2022

Describe the bug

As a service operator,

  • in order to deploy interoperator in a cluster with multiple ingress controllers,
  • I need to be able to specify an ingressClassName on the broker ingress

What's the usual contribution/test process for such small PRs ? Is there an existing automated test suite for the helm chart that SAP maintains in a private build that could be shared ? Is it instead manual testing from the helm sources ?

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/181831250

The labels on this github issue will be updated when the story is started.

@anoopjb anoopjb self-assigned this May 5, 2022
@anoopjb
Copy link
Contributor

anoopjb commented May 5, 2022

Hi, @gberche-orange,

Thank you for the suggestions. We do have some internal test beds based on Gardener project. But unfortunately, it is not easy to share the setup. If you plan to contribute, I can do a sanity check on the changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Waiting for Changes | Open for Contributions
Development

No branches or pull requests

3 participants