Skip to content

Validate field names match DCM#161

Merged
digimaun merged 13 commits into
devfrom
unknown repository
Apr 16, 2020
Merged

Validate field names match DCM#161
digimaun merged 13 commits into
devfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Apr 14, 2020

This PR has 2 purposes:

  1. Add ability to retrieve DCM from central, and validate that messages sent by device match this DCM

  2. Propose a new way to organize code in repo:

azext_iot/
|--dps
|--central
  |-- providers
  |-- services
|--hub
...
|-- common
|-- utility

This project has adopted the Microsoft Open Source Code of Conduct. For more information see the Code of Conduct FAQ or contact opencode@microsoft.com with any additional questions or comments.

Thank you for contributing to the IoT extension!

This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • If introducing new functionality or modified behavior, are they backed by unit and integration tests?
  • In the same context as above are command names and their parameter definitions accurate? Do help docs have sufficient content?
  • Have all unit and integration tests passed locally? i.e. pytest <project root> -vv
  • Have static checks passed using the .pylintrc and .flake8 rules? Look at the CI scripts for example usage.

@ghost ghost requested a review from digimaun as a code owner April 14, 2020 20:25
@ghost ghost changed the title Validate field names match DCM [DRAFT] Validate field names match DCM Apr 14, 2020
@ghost ghost changed the title [DRAFT] Validate field names match DCM Validate field names match DCM Apr 15, 2020
@ghost
Copy link
Copy Markdown
Author

ghost commented Apr 15, 2020

Tests passing

image

Comment thread azext_iot/tests/test_iot_utility_unit.py
Comment thread azext_iot/central/services/device_template.py Outdated
Comment thread azext_iot/operations/central.py Outdated
Comment thread azext_iot/central/providers/device_provider.py Outdated
Comment thread azext_iot/constants.py Outdated
Comment thread azext_iot/operations/central.py
Comment thread azext_iot/tests/test_iot_central_unit.py Outdated
Comment thread azext_iot/central/services/device_template.py Outdated
Comment thread azext_iot/central/providers/device_provider.py Outdated
Comment thread azext_iot/commands.py Outdated
Comment thread azext_iot/operations/events3/_parser.py
Comment thread azext_iot/commands.py
@digimaun digimaun merged commit e10bec9 into Azure:dev Apr 16, 2020
@ghost ghost deleted the prbans/validate-telemetry-across-dcm branch May 4, 2020 19:43
c-ryan-k added a commit to c-ryan-k/azure-iot-cli-extension that referenced this pull request Apr 10, 2025
… sets (Azure#161)

* Add support bundle capturing of individual MQ connector stateful sets

* One-off test for mq support bundle stateful sets

* Move aio-mq- into constant AIO_MQ_RESOURCE_PREFIX string
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.

1 participant