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

Fix list-functions deployment in stack.yml #243

Merged
merged 1 commit into from
Dec 13, 2020

Conversation

Waterdrips
Copy link
Contributor

Description

Fixes: #242
The move to GHCR had a rogue vim replace all (functions --> ghcr.io/)
and thus the list-functions function got some random replacments.

This has been tested on a new cluster and the broken credentials for
list-function have been restored and now it can call the gateway

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

How Has This Been Tested?

New cluster deployment from scratch on master, then change the fix and re-deploy and it works

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 move to GHCR had a rogue vim replace all (functions --> ghcr.io/)
and thus the list-functions function got some random replacments.

This has been tested on a new cluster and the broken credentials for
list-function have been restored and now it can call the gateway

Signed-off-by: Alistair Hey <[email protected]>
@Waterdrips Waterdrips force-pushed the Waterdrips-fix-dashboard branch from ce36be3 to b32da75 Compare December 13, 2020 14:16
@alexellis
Copy link
Member

How can we start picking this kind of error up in CI?

@Waterdrips
Copy link
Contributor Author

We would need to start running deplyoemts/functions/reading the ui/checking the logs etc

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.

Approved

@alexellis alexellis merged commit ed76ca6 into openfaas:master Dec 13, 2020
@alexellis
Copy link
Member

Can you raise an issue and propose how you would have detected this error that the PR fixes?

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.

[support] after running ofc-bootstrap with GitHub config, unable to see functions.
2 participants