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

kubeless function deploy/update: add option to specify node selectors. #1159

Merged
merged 1 commit into from
Aug 24, 2020
Merged

kubeless function deploy/update: add option to specify node selectors. #1159

merged 1 commit into from
Aug 24, 2020

Conversation

varunbpatil
Copy link
Contributor

Issue Ref: #1156

Description:

Add a new option --node-selectors to the kubeless function deploy and kubeless function update commands to specify node-selectors for the function.

TODOs:

  • Ready to review
  • Automated Tests
  • Docs

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 for the PR @varunbpatil ! I just have one question.

cmd/kubeless/function/function.go Show resolved Hide resolved
@andresmgot
Copy link
Contributor

Sorry @varunbpatil, this PR is ready to be merged but the CI is failing because an unrelated issue. There are several references in the file examples/Makefile that relates to the runtime nodejs6. That should be now nodejs10, do you mind updating it so your PR passes the tests?

@varunbpatil
Copy link
Contributor Author

Sorry @varunbpatil, this PR is ready to be merged but the CI is failing because of an unrelated issue. There are several references in the file examples/Makefile that relates to the runtime nodejs6. That should be now nodejs10, do you mind updating it so your PR passes the tests?

Will do.

@varunbpatil
Copy link
Contributor Author

Hi @andresmgot , I've update the Makefile and CI is passing.

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.

Cool, thanks!

@andresmgot andresmgot merged commit 900832a into vmware-archive:master Aug 24, 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