Provide a Common MQTT Mapper

[fuchendou]

This PR introduces a universal MQTT Mapper designed to handle different serialization formats (JSON, YAML, and XML) during MQTT transmissions. The goal is to enhance flexibility and scalability for device communication.

Main Changes:

devicetype.go:
Improved ConfigData and VisitorConfigData structs to handle data before and after MQTT parsing.

driver.go:
Implemented methods for the ConfigData struct, including logic for serialization and parsing.

This PR fixes the issue: PR #链接(kubeedge/kubeedge#5765)

[kubeedge-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign luogangyi after the PR has been reviewed.
You can assign the PR to them by writing /assign @luogangyi 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:

• OWNERS

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

[wbc6080]

It seems that this PR can't pass the CI check. Can you try to find the reason?

[fuchendou]

Thanks to your letter, I observed the returned error, changed the file permissions and resubmitted the code.

…

On Sat, Oct 12, 2024 at 11:06 AM Elias Wang ***@***.***> wrote: ***@***.**** commented on this pull request. It seems that this PR can't pass the CI check. Can you try to find the reason? — Reply to this email directly, view it on GitHub <#130 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A62LQCMYZ6HSSYANYB3D2S3Z3CG2FAVCNFSM6AAAAABPBUNBI2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRTG4ZDKNBWGM> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[wbc6080]

It seems that there is still a problem with CI. I think it is due to a dependency mismatch caused by the existence of v1alpha2 and v1beta1 mappers in this directory. I suggest placing this mapper in the newly created directory mappers-go/mappers/device-v1beta1/, which should solve the current CI error problem.

[wbc6080]

The name of the package is incorrect. You should set the mapper name to mqtt instead of kubeedge-v1.17.0 when executing the make generate command to generate the mapper project.

[fuchendou]

Thanks for your advice. I will remake a new mapper project using make generate.

[wbc6080]

remove this file

[fuchendou]

Thanks for your advice. I will modify the file structure.

[fuchendou]

Thanks for the feedback! I'll make changes based on your comments.

…

On Tue, Oct 15, 2024 at 3:49 PM Elias Wang ***@***.***> wrote: ***@***.**** commented on this pull request. It seems that there is still a problem with CI. I think it is due to a dependency mismatch caused by the existence of v1alpha2 and v1beta1 mappers in this directory. I suggest placing this mapper in the newly created directory mappers-go/mappers/device-v1beta1/, which should solve the current CI error problem. ------------------------------ In mappers/mqtt-mapper/go.mod <#130 (comment)>: > @@ -0,0 +1,41 @@ +module github.com/kubeedge/kubeedge-v1.17.0 The name of the package is incorrect. You should set the mapper name to mqtt instead of kubeedge-v1.17.0 when executing the make generate command to generate the mapper project. ------------------------------ In mappers/mqtt-mapper/Dockerfile <#130 (comment)>: > +FROM golang:1.20.10-alpine3.18 AS builder + +WORKDIR /build + +ENV GO111MODULE=on \ + GOPROXY=https://goproxy.cn,direct + +COPY . . + remove this file — Reply to this email directly, view it on GitHub <#130 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A62LQCOLXFGOY5KLGCWLGKLZ3TCJVAVCNFSM6AAAAABPBUNBI2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRYGQ2DMMJXGM> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[fuchendou]

It seems that this PR can't pass the CI check. Can you try to find the reason?

[wbc6080]

Good job! Thanks for your contribution!
/lgtm

[fuchendou]

Thank you for your support!

…

On Thu, Oct 17, 2024 at 4:28 PM Elias Wang ***@***.***> wrote: ***@***.**** commented on this pull request. Good job! Thanks for your contribution! /lgtm — Reply to this email directly, view it on GitHub <#130 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A62LQCM644L44EKA2FAZRZLZ35YJ3AVCNFSM6AAAAABPBUNBI2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNZUGUZDAMZRGA> . You are receiving this because you modified the open/close state.Message ID: ***@***.***>

[fuchendou]

Could you please help review my proposal "Add a Proposal about Provide a Common MQTT Mapper" (PR #5765 <kubeedge/kubeedge#5765>)? The detail implementation has been successfully tested(PR #130 <#130>), and I am looking forward to your valuable feedback to proceed further. Thanks for your time and support! On Fri, Oct 18, 2024 at 9:12 AM mengxiao zhang ***@***.***> wrote:

…

Thank you for your support! On Thu, Oct 17, 2024 at 4:28 PM Elias Wang ***@***.***> wrote: > ***@***.**** commented on this pull request. > > Good job! Thanks for your contribution! > /lgtm > > — > Reply to this email directly, view it on GitHub > <#130 (review)>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/A62LQCM644L44EKA2FAZRZLZ35YJ3AVCNFSM6AAAAABPBUNBI2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNZUGUZDAMZRGA> > . > You are receiving this because you modified the open/close state.Message > ID: ***@***.***> >