-
Notifications
You must be signed in to change notification settings - Fork 0
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
Docker Storage Support for the Firmware Manager #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me!
@@ -40,7 +40,7 @@ Available REST endpoints: | |||
|
|||
To properly run the firmware_manager microservice the following services are also required: | |||
|
|||
- Cloud based blob storage | |||
- Cloud based blob storage (if not using a directory mounted as a docker volume) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a bit confusing, should the Docker volume be organized the same?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes it should. This has been modified to improve clarity.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple questions, looks good!
This PR is ready for re-review. @payneBrandon |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Problem
The Firmware Manager addon currently only supports storing firmware files in GCP. In a deployment where this isn't desired, we need another way to store the firmware files.
Solution
Support for storing firmware files in a directory mounted as a docker volume has been added to the project.
Testing