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

Add ConfigurationAdmin Integration Functionality to DS RFC #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pelliott-mathworks
Copy link
Collaborator

Added a description of the Configuration Admin functionality to the Requirements, Functional Design and Architectural Design sections of the RFC. Signed-off-by: The MathWorks, Inc. [email protected]

 Added a description of the Configuration Admin functionality to the Requirements, Functional Design and Architectural Design sections of the RFC. Signed-off-by: The MathWorks, Inc. <[email protected]>
Copy link
Member

@jeffdiclemente jeffdiclemente left a comment

Choose a reason for hiding this comment

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

I had envisioned that there would be a separate RFC for this enhancement instead of it being rolled into the existing DS RFC. I do see the benefit of rolling the DS specific changes that allow integration with Config Admin into the existing DS RFC, however there are other parts of the design that are not part of the DS design and architecture. This is why I feel that a separate RFC is necessary.

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.

2 participants