Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated README and Deployment File #17

Merged
merged 4 commits into from
Jan 28, 2019
Merged

Updated README and Deployment File #17

merged 4 commits into from
Jan 28, 2019

Conversation

embano1
Copy link
Contributor

@embano1 embano1 commented Jan 25, 2019

Signed-off-by: Michael Gasch embano1@live.com

- Added instructions how to deploy to Kubernetes
- Example to show how a VM can be tagged based on a vCenter VM event
- Connector deployment YAML updated with image supporting pending (PR

Signed-off-by: Michael Gasch <embano1@live.com>
- Be more clear about `TAGURN` format
- Use same example password across README

Signed-off-by: Michael Gasch <embano1@live.com>
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated
```yaml
environment:
VC: vcenter.ip # must be reachable/resolvable from OpenFaaS
VCUSERNAME: VCUSER # WIP: migration to secrets
Copy link
Member

Choose a reason for hiding this comment

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

Should we finish the creation of secrets in your function first before merging this PR?

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 definitely would prefer that, but give the time constraints right now and people wanting to try this out, I'd point it out in the docs (so people are aware, not prod ready, etc.) and we'll fix ASAP in another commit.

Copy link
Member

Choose a reason for hiding this comment

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

OK please can you log an issue to cover the technical debt?

README.md Outdated
VCPASSWORD: VCPASSWORD # WIP: migration to secrets
# Replace TAGURN example below with the one you created with govc above
TAGURN: urn:vmomi:InventoryServiceTag:019c0a9e-0672-48f5-ac2a-e394669e2916:GLOBAL
TAGACTION: attach # this function also supports detach
Copy link
Member

Choose a reason for hiding this comment

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

Generally we separate the words with _ like TAG_ACTION

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also changed for VC* and will update the function examples accordingly.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@alexellis
Copy link
Member

This is amazing Michael. I've added some comments on the README that I'd like to be updated before merge.

Thanks for all your effort on this

Alex

- Incorporate review feedback for README
- Enable OpenFaaS auth in the deployment manifest and describe how to
disable if needed

Signed-off-by: Michael Gasch <embano1@live.com>
- detailed information for basic_auth enabled OpenFaaS deployment
- work around the DRS powered on issue

Signed-off-by: Michael Gasch <embano1@live.com>
@alexellis alexellis merged commit d8711ba into openfaas-incubator:master Jan 28, 2019
@alexellis
Copy link
Member

Thank you @embano1 👍 this help is making a huge difference

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants