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

Support shareShadow option for connect #103

Merged
merged 16 commits into from Mar 25, 2020
Merged

Conversation

cui-liqiang
Copy link
Collaborator

Only connect got affected.

  1. If option is given, deployment name will be fixed as "kt-connect-daemon-connect-shared" instead of having random suffix
  2. The shared shadow deployment will have a label named "refCount" to indicate how many clients are using it.
  3. Store both public and private key in config map, so that latter clients can reuse it.
  4. When client reuses the shadow, increase the refCount label value
  5. When tear down, read the refCount label and decide to delete it or reduce the refCount.

* github/master:
  ci: disable unused stage define
  ci: deploy only on tag
  ci: deploy only on tag
  ci: test matrix
  ci:test test matrix build
  ci:test script deploy
  ci: parallelizing cross platform
  ci: test deploy
  ci: disable on tags condition for test deploy
  test: enable notifications
  ci: split cli/shadow/dashboard
  ci: disable golangci-lint
  ci: fixed config.yaml.enc not found issue
  ci: add code analysis with golangci-lint
  ci: split ci task with stage
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


liqiang.clq seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

cmd/ktctl/main.go Outdated Show resolved Hide resolved
@cui-liqiang cui-liqiang merged commit 3586a42 into alibaba:master Mar 25, 2020
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.

None yet

4 participants