-
Notifications
You must be signed in to change notification settings - Fork 15
add tutorial for release v0.2.0 to support EdgeX 2.1 #35
Conversation
@wawlian: The label(s) In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
I think we should put the tutorial of device-controller and edge manager together. Give the use all-in-one doc to install, deployment, test rather than a separate docs in many places |
@lwmqwer agree +1 |
5e7f556
to
fd3a281
Compare
@rambohe-ch @lwmqwer Provide an all-in-one tutorial is fine. But maybe openyurt.io is a better place. This tutorial as an updated version puts more emphasis on how to manage edge devices through yurt-device-controller from OpenYurt/K8S side. |
apiVersion: device.openyurt.io/v1alpha1 | ||
kind: DeviceService | ||
metadata: | ||
name: openyurt-created-deviceservuce-virtual |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deviceservuce -> deviceservice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
other: | ||
Address: openyurt-created-device-virtual-bool-01 | ||
Port: "300" | ||
serviceName: openyurt-created-deviceservuce-virtual |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deviceservuce -> deviceservice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@wawlian correct, neither of device or edgex is suit for this tutorial . But I mean is that we can just write a all-in-one and give the link in edgex and device is enough. |
fd3a281
to
2fab930
Compare
/LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: qclc, wawlian The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #34
Special notes for your reviewer:
/assign @lwmqwer @gnunu
Does this PR introduce a user-facing change?
NONE