Skip to content

Support ingressClassName, ingress tls, and extraObjects #23

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Apr 20, 2024
Merged

Support ingressClassName, ingress tls, and extraObjects #23

merged 2 commits into from
Apr 20, 2024

Conversation

nissessenap
Copy link
Contributor

Solves #20

@nissessenap nissessenap mentioned this pull request Apr 19, 2024
Open
@nissessenap
Copy link
Contributor Author

FYI @nscuro , this is a simplification of #17.
It solves the same issue and I think it will be easier for the community to maintain in the long run, and I still think it's easy enough to use. Especially since it follows how other helm charts normally look.

@nscuro nscuro added the enhancement New feature or request label Apr 19, 2024
@nscuro
Copy link
Member

nscuro commented Apr 19, 2024

Thanks @nissessenap, really like the simplicity!

Even if we end up adding more options in the future (not sold on that), I think it's great to start with the minimum viable version proposed here.

Do you mind bumping the chart version to 0.2.0?

@nissessenap
bump chart 0.2.0
6d79172 
Signed-off-by: Edvin Norling <[email protected]>
@nissessenap
Copy link
Contributor Author

Ahh, missed that one. Done
Do you want me to squash the commits as well @nscuro ?

nscuro
nscuro approved these changes Apr 20, 2024
View reviewed changes
Copy link
Member

@nscuro nscuro left a comment

Choose a reason for hiding this comment

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

Thanks!

No need to squash yourself, the repo has been set up to only allow squash merges.

@nscuro nscuro changed the title ingress improvment & extraObjects Support ingressClassName, ingress tls, and extraObjects Apr 20, 2024
@nscuro nscuro merged commit cec9a54 into DependencyTrack:main Apr 20, 2024
nscuro added a commit that referenced this pull request Jun 12, 2024
@nscuro @pawelmrowka
@nissessenap @jazark @Wihrt @stexandev @tvannahl @SergK
Port enhancement from dependency-track chart to hyades
6cad4c1 
Includes changes from:

* #13
* #22
* #23
* #33
* #34
* #42
* #52
* #55
* #57
* #83
* #87

Co-authored-by: Pawel Mrowka <[email protected]>
Co-authored-by: Edvin Norling <[email protected]>
Co-authored-by: Jaz Ark <[email protected]>
Co-authored-by: Arnaud Hatzenbuhler <[email protected]>
Co-authored-by: Stefan Hynek <[email protected]>
Co-authored-by: Theodor van Nahl <[email protected]>
Co-authored-by: Sergiy Kulanov <[email protected]>
Signed-off-by: nscuro <[email protected]>
@nscuro nscuro mentioned this pull request Jun 12, 2024
Merged
nscuro added a commit that referenced this pull request Jun 12, 2024
@nscuro @pawelmrowka
@nissessenap @jazark @Wihrt @stexandev @tvannahl @SergK
Port enhancement from dependency-track chart to hyades
81e3fc5 
Includes changes from:

* #13
* #22
* #23
* #33
* #34
* #42
* #52
* #55
* #57
* #83
* #87

Co-authored-by: Pawel Mrowka <[email protected]>
Co-authored-by: Edvin Norling <[email protected]>
Co-authored-by: Jaz Ark <[email protected]>
Co-authored-by: Arnaud Hatzenbuhler <[email protected]>
Co-authored-by: Stefan Hynek <[email protected]>
Co-authored-by: Theodor van Nahl <[email protected]>
Co-authored-by: Sergiy Kulanov <[email protected]>
Signed-off-by: nscuro <[email protected]>
nscuro added a commit that referenced this pull request Jun 12, 2024
@nscuro @pawelmrowka
@nissessenap @jazark @Wihrt @stexandev @tvannahl @SergK
Port enhancement from dependency-track chart to hyades
29d3478 
Includes changes from:

* #13
* #22
* #23
* #33
* #34
* #42
* #52
* #55
* #57
* #83
* #87

Co-authored-by: Pawel Mrowka <[email protected]>
Co-authored-by: Edvin Norling <[email protected]>
Co-authored-by: Jaz Ark <[email protected]>
Co-authored-by: Arnaud Hatzenbuhler <[email protected]>
Co-authored-by: Stefan Hynek <[email protected]>
Co-authored-by: Theodor van Nahl <[email protected]>
Co-authored-by: Sergiy Kulanov <[email protected]>
Signed-off-by: nscuro <[email protected]>
nscuro added a commit that referenced this pull request Jun 12, 2024
@nscuro @pawelmrowka
@nissessenap @jazark @Wihrt @stexandev @tvannahl @SergK
Port enhancement from dependency-track chart to hyades (#91)
f9336b6 
Includes changes from:

* #13
* #22
* #23
* #33
* #34
* #42
* #52
* #55
* #57
* #83
* #87

Signed-off-by: nscuro <[email protected]>
Co-authored-by: Pawel Mrowka <[email protected]>
Co-authored-by: Edvin Norling <[email protected]>
Co-authored-by: Jaz Ark <[email protected]>
Co-authored-by: Arnaud Hatzenbuhler <[email protected]>
Co-authored-by: Stefan Hynek <[email protected]>
Co-authored-by: Theodor van Nahl <[email protected]>
Co-authored-by: Sergiy Kulanov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nscuro nscuro nscuro approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nissessenap @nscuro