Skip to content
This repository has been archived by the owner on Mar 10, 2023. It is now read-only.

Update chart namespaces to match existing #680

Merged

Conversation

Waterdrips
Copy link
Contributor

Description

The namespaces in the chart were wrong or missing for a lot of
the resources. These have now been corrected along with adding the
annotation for ingress shim for cert-manager to get tls for domains if
they are asked for

Signed-off-by: Alistair Hey alistair@heyal.co.uk

How Has This Been Tested?

Unit tests

@Waterdrips Waterdrips force-pushed the waterdrips-fix-chart-namespaces branch 2 times, most recently from c9d1c12 to 7cee527 Compare October 15, 2020 19:01
The namespaces in the chart were wrong or missing for a lot of
the resources. These have now been corrected along with adding the
annotation for ingress shim for cert-manager to get tls for domains if
they are asked for

Signed-off-by: Alistair Hey <alistair@heyal.co.uk>
@Waterdrips Waterdrips force-pushed the waterdrips-fix-chart-namespaces branch from 7cee527 to 973f68e Compare October 15, 2020 19:02
@alexellis alexellis merged commit 7f4fb91 into openfaas:master Oct 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants