Skip to content
This repository has been archived by the owner on Nov 7, 2023. It is now read-only.

Update deviceController to reconcile the deviceService between the openyurt and edge #4

Merged
merged 1 commit into from
Sep 22, 2021

Conversation

qclc
Copy link
Member

@qclc qclc commented Sep 8, 2021

  1. deviceService created/deleted in openyurt could sync in edgex-foundry.
  2. deviceService create/deleted in edgex-foundry could sync in openyurt.
  3. can update AdminState field of deviceService
    Note: Can't support update other field of deviceService yet.

@@ -18,6 +18,15 @@ package v1alpha1

import (
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
clusterv1 "sigs.k8s.io/cluster-api/api/v1alpha4"
Copy link
Member

Choose a reason for hiding this comment

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

need to update copyright: kubernetes --> OpenYurt

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for reminding, the license header is changed.

"github.com/edgexfoundry/go-mod-core-contracts/models"
"github.com/go-logr/logr"
coremetacli "github.com/openyurtio/device-controller/clients/core-metadata"
"sigs.k8s.io/controller-runtime/pkg/client"

Copy link
Member

Choose a reason for hiding this comment

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

please add license header. all others files should be check.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for reminding, the license headers are all added.

@rambohe-ch
Copy link
Member

@qclc need to fix conflicting files.

@qclc
Copy link
Member Author

qclc commented Sep 22, 2021

@rambohe-ch all the conflicting files are fixed, please review.

Copy link
Member

@rambohe-ch rambohe-ch left a comment

Choose a reason for hiding this comment

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

LGTM

@rambohe-ch
Copy link
Member

/lgtm
/approve

@openyurt-bot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: qclc, rambohe-ch
To complete the pull request process, please assign
You can assign the PR to them by writing /assign in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rambohe-ch rambohe-ch merged commit 1be1952 into openyurtio:main Sep 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants