Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

Strict deployment loading from configmap #1199

Merged
merged 3 commits into from
Dec 2, 2020
Merged

Strict deployment loading from configmap #1199

merged 3 commits into from
Dec 2, 2020

Conversation

andrascz
Copy link
Contributor

@andrascz andrascz commented Nov 27, 2020

Issue Ref: #1197

Description:
Fail if a deployment configuration in the ConfigMap has unknown or duplicate keys.

TODOs:

  • Ready to review
  • Automated Tests
  • Docs

@andrascz
Copy link
Contributor Author

@andresmgot is there a place in the docs where we can note this?

Copy link
Contributor

@andresmgot andresmgot left a comment

Choose a reason for hiding this comment

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

Thanks @andrascz! Yes, you can add a bit of documenation about this at https://github.com/kubeless/kubeless/blob/master/docs/function-controller-configuration.md#using-configmap

Once you add that I will merge this PR.

Copy link
Contributor

@andresmgot andresmgot left a comment

Choose a reason for hiding this comment

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

Thanks!

@andresmgot andresmgot merged commit c7ee7b5 into vmware-archive:master Dec 2, 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.

2 participants