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

feat: Create UCR for using URIs to load files #938

Merged
merged 4 commits into from
Jan 25, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 9 additions & 8 deletions docs_src/design/TOC.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,16 @@
!!! note
UCRs are listed in alphabetical order by title.

| Name/Link | Short Description |
| ------------------------------------------------------------ |---------------------------------------------------------------------|
| [Common Configuration](./ucr/Common Configuration.md) | Use Case for having Common configuration used by all EdgeX services |
| [Device Parent-Child Relationships](./ucr/Device-Parent-Child-Relationships.md) | Use Case for Device Parent-Child Relationships |
| [Extending Device Data](./ucr/Extending-Device-Data.md) | Use Case for Extending of Device Data by Application Services |
| Name/Link | Short Description |
|-------------------------------------------------------------------------------------|---------------------------------------------------------------------|
| [Common Configuration](./ucr/Common Configuration.md) | Use Case for having Common configuration used by all EdgeX services |
| [Device Parent-Child Relationships](./ucr/Device-Parent-Child-Relationships.md) | Use Case for Device Parent-Child Relationships |
| [Extending Device Data](./ucr/Extending-Device-Data.md) | Use Case for Extending of Device Data by Application Services |
| [Provision Watch via Device Metadata](./ucr/Provision-Watch-via-Device-Metadata.md) | Use Case for Provision Watching via Additional Device Metadata |
| [Record and Replay](./ucr/Record-and-Replay.md) | Use Case for Recording and Replaying event/readings |
| [System Events for Devices](./ucr/System-Events-for-Devices.md) | Use Case for System Events for Device add/update/delete |
| [Microservice Authentication](./ucr/Microservice-Authentication.md) | Use Case for Microservice Authentication |
| [Record and Replay](./ucr/Record-and-Replay.md) | Use Case for Recording and Replaying event/readings |
| [System Events for Devices](./ucr/System-Events-for-Devices.md) | Use Case for System Events for Device add/update/delete |
| [Microservice Authentication](./ucr/Microservice-Authentication.md) | Use Case for Microservice Authentication |
| [URIs for files](.//ucr/URIs-for-Files.md) | Use Case for loading service files from URIs |

## Architectural Design Records (ADRs)

Expand Down
56 changes: 56 additions & 0 deletions docs_src/design/ucr/URIs-for-Files.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,56 @@
## URIs for files
### Submitters
- Lenny Goodell (Intel)

## Change Log
- [pending](URL of PR) (2023-01-18)

### Market Segments
- All

### Motivation
Deployment at scale, i.e. identical or almost identical deployments across many locations, would benefit from the ability to loaded service files from a central location. This would allow the maintainer to make changes once to a shared file and have them apply to all or a subset of deployments. The following are some EdgeX service files that would benefit for this capability:
lenny-goodell marked this conversation as resolved.
Show resolved Hide resolved

- [Unit of Measure](https://github.com/edgexfoundry/edgex-go/blob/v2.3.0/cmd/core-metadata/res/uom.toml) file used by Core Metadata

- Location of this file is specified in configuration [here](https://github.com/edgexfoundry/edgex-go/blob/v2.3.0/cmd/core-metadata/res/configuration.toml#L50)

- Service Configuration files
- Location of these files are currently defaulted to be `./res/configuration.toml`, but can be overridden via -cf/--configFile command line flag.
- The Common Configuration ADR adds a new [common configuration file](https://docs.edgexfoundry.org/3.0/design/adr/0026-Common%20Configuration/#specifying-the-common-configuration-location) specified via the future -cc/--commonConfig command line flag.

- Device Profiles, Device Definition and Provision Watchers

- These files can reside in a device services local file system and are pushed to Core Metadata the first time the service starts. Example [here](https://github.com/edgexfoundry/device-onvif-camera/tree/v2.3.0/cmd/res)

- These files are found by scanning the folders specified in configuration [here](https://github.com/edgexfoundry/device-sdk-go/blob/main/example/cmd/device-simple/res/configuration.toml#L94-L96)

!!! note
These files are only pushed to Core Metadata the first time the device service is loaded. They are not currently re-pushed once they exist in Core Metadata even when the files have changed locally. Thus updating the files locally or in a shared location will not result in changing the contents of these files in Core Metadata. They still benefit from this capability during initial deployment and when new files are added.


Currently all files loaded by services are expected to be on the local file system, thus are duplicated many times when deploying at scale.

### Target Users
- Software Deployer
- Software Integrator

### Description
This UCR proposes to enhance loading of files in EdgeX by allowing the location of the file to be optionally specified as an URI.

### Existing solutions
Loading shared files via a URI is not new in the software industry. Here is the Wiki page for [Uniform Resource Identifier](https://en.wikipedia.org/wiki/Uniform_Resource_Identifier)

### Requirements
- Single EdgeX service files shall optionally be loaded via a specified URI.
- Sets of EdgeX service files (i.e. device service files) shall optionally be loaded via a URI. Details on how are left to the ADR.
- The URIs specified shall follow the [Uniform Resource Identifier](https://en.wikipedia.org/wiki/Uniform_Resource_Identifier) spec when authentication not required or using **basic-auth** in plain text in the URI, i.e. `username:password@`
lenny-goodell marked this conversation as resolved.
Show resolved Hide resolved
- Only the `http`, `https`, and `file` schemes from the above spec shall be supported.
lenny-goodell marked this conversation as resolved.
Show resolved Hide resolved

- The URI spec shall be extended to allow the specifying of EdgeX service secrets from the service's Secret Store in order to avoid credentials in plain text. Details on how are left to the ADR.

### Other Related Issues
- [Common Configuration ADR](https://docs.edgexfoundry.org/3.0/design/adr/0026-Common%20Configuration/): This ADR specifies that the common configuration file specified by the `-cc/--commonConfig` flag can be a URI to a remote files. The implementation of this portion of the ADR is dependent on the UCR and following ADR.

### References
- None