-
Notifications
You must be signed in to change notification settings - Fork 61
basic_auth removal + secrets filters break deployment #85
Comments
Please use an older version prior to the change from @martindekov. I'll ask the community to look into this for you in the meantime. Alex |
No problem, I have it working locally now with the modifications I suggested. Just letting you know there is an issue atm |
Well I'm a bit confused because the guys tested their changes before having them merged. I using the latest release or are you building fresh from master? |
-- |
sorry, I'm building fresh from master |
Can you please retry after adding a feature list to features:
- basic_auth
- auth
- scm_github I think that is the reason |
@viveksyngh - there is no I've moved the filter to |
Hello @mbarlow I apolagize for the confusion, had my own |
Expected Behaviour
Cloning the project, copying example.init.yaml to init.yaml (modify config options for TLS, Oauth, github app, etc), and running ./ofc-bootstrap should deploy to cluster the full installation without errors.
Current Behaviour
During the installation w/ the current master, the ./scripts/deploy-cloud-components.sh will fail on missing basic-auth secret. Kubectl port forward will fail, the logic for "Checking if OpenFaas GW is up" will fail, etc.
Possible Solution
Add basic_auth=true back into example.init.yaml, modify filters for secrets to set basic_auth to default, add BasicAuth bool back into pkg/types/types.go Plan struct.
Steps to Reproduce (for bugs)
Context
Cannot access the openfaas-cloud install, missing github functions, clusterissuer deployments also fail and need to be manually applied.
Your Environment
FaaS-CLI version ( Full output from:
faas-cli version
): 0.8.3Docker version
docker version
(e.g. Docker 17.0.05 ): 18.09.2Are you using Docker Swarm or Kubernetes (FaaS-netes)? Kubernetes
Operating System and version (e.g. Linux, Windows, MacOS): MacOS Mojave
Link to your project or a code example to reproduce issue:
Please also follow the troubleshooting guide and paste in any other diagnostic information you have:
The text was updated successfully, but these errors were encountered: