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

Update to 0.13.6 openfaas-cloud #199

Merged
merged 3 commits into from
Mar 4, 2020

Conversation

Waterdrips
Copy link
Contributor

@Waterdrips Waterdrips commented Mar 3, 2020

Signed-off-by: Alistair Hey [email protected]

Description

Update OFC version to 0.13.5

How Has This Been Tested?

This has been tested with:

0.13.1 cluster upgraded to 0.13.5 and then 0.13.6

and 0.13.5 upgraded to 0.13.6.

Clusters up and running!

Checklist:

I have:

  • checked my changes follow the style of the existing code / OpenFaaS repos
  • updated the documentation and/or roadmap in README.md
  • read the CONTRIBUTION guide
  • signed-off my commits with git commit -s
  • added unit tests

The secret type was set to `literals` not files, this means when you
copy the file and set the value_from to a filepath, it actually uses the
"value_from" as the literal value and not the contents of the file.

I have also added a clarifying comment for usage

Signed-off-by: Alistair Hey <[email protected]>
Signed-off-by: Alistair Hey <[email protected]>
@Waterdrips Waterdrips changed the title [WIP] Update to 0.13.5 openfaas-cloud Update to 0.13.6 openfaas-cloud Mar 3, 2020
@Waterdrips Waterdrips closed this Mar 3, 2020
@Waterdrips Waterdrips reopened this Mar 3, 2020
@Waterdrips Waterdrips marked this pull request as ready for review March 3, 2020 15:50
- name: "of-customers"
literals:
Copy link
Member

Choose a reason for hiding this comment

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

Was this a typo on my part?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think so, if the intention was to read it from a file rather than a literal?

Copy link
Member

@alexellis alexellis left a comment

Choose a reason for hiding this comment

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

LGTM

@alexellis alexellis merged commit 1650a9e into openfaas:master Mar 4, 2020
@Waterdrips Waterdrips deleted the 0-13-5-uupgrade branch June 2, 2020 10:21
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