Removed EOL Flocker plugin reference from plugin documentations.#6804
Removed EOL Flocker plugin reference from plugin documentations.#6804icemc wants to merge 1 commit intodocker:masterfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR removes references to the deprecated Flocker plugin from Docker plugin documentation in response to issue #6783. The Flocker repository was archived in May 2018 and should no longer be used as an example in current documentation.
Changes:
- Replaced Flocker-specific examples with generic VolumeDriver examples in plugin API documentation
- Removed Flocker reference and link from volume plugin command-line examples
- Updated example socket paths and plugin names throughout affected documentation
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| docs/extend/plugins_volume.md | Replaced Flocker example commands with generic VolumeDriver examples and updated the volume driver section description |
| docs/extend/plugin_api.md | Replaced Flocker socket path examples with generic VolumeDriver socket path examples |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@icemc can you address Copilot comments? |
|
Hello @vvoland I implemented all suggestions. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
thaJeztah
force-pushed
the
update-docs-flocker-plugin
branch
2 times, most recently
from
b5debcc to
090c1b7
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
thaJeztah
force-pushed
the
update-docs-flocker-plugin
branch
from
090c1b7 to
a3317a3
Compare
- Removed EOL Flocker plugin reference from plugin documentations. - docs: use generic myplugin example instead of VolumeDriver Co-authored-by: Ludovic Temgoua Abanda <abandaludovic500@gmail.com> Signed-off-by: Ludovic Temgoua Abanda <abandaludovic500@gmail.com> Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
thaJeztah
force-pushed
the
update-docs-flocker-plugin
branch
from
a3317a3 to
33790e8
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Closes #6783
- What I did
I removed references of the Flocker plugin from the plugin documentations.
- How I did it
Manually check references of this plugin throughout the codebase.
- How to verify it
- Human readable description for the release notes
Removed references of the EOL Flocker plugin from the plugin documentations.
- A picture of a cute animal (not mandatory but encouraged)