core: fix csi-cephfsplugin pod restart on non-hostnetworking env #9925
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
leseb
changed the title
BlaineEXE
reviewed
Mar 17, 2022
leseb
force-pushed
the
multus-plugin-restart-fix
branch
from
6a1a322
to
d15f5e2
Compare
leseb
force-pushed
the
multus-plugin-restart-fix
branch
from
d15f5e2
to
9bc3d19
Compare
leseb
force-pushed
the
multus-plugin-restart-fix
branch
10 times, most recently
from
a894455
to
ec7870d
Compare
|
CI extracted in #10048 |
leseb
force-pushed
the
multus-plugin-restart-fix
branch
from
ec7870d
to
b31a13b
Compare
|
This pull request has merge conflicts that must be resolved before it can be merged. @leseb please rebase it. https://rook.io/docs/rook/latest/development-flow.html#updating-your-fork |
leseb
force-pushed
the
multus-plugin-restart-fix
branch
2 times, most recently
from
ad3a76c
to
94778a1
Compare
|
This pull request has merge conflicts that must be resolved before it can be merged. @leseb please rebase it. https://rook.io/docs/rook/latest/development-flow.html#updating-your-fork |
leseb
force-pushed
the
multus-plugin-restart-fix
branch
2 times, most recently
from
e976d9b
to
3e692e1
Compare
leseb
force-pushed
the
multus-plugin-restart-fix
branch
4 times, most recently
from
376f6da
to
28074d3
Compare
leseb
force-pushed
the
multus-plugin-restart-fix
branch
5 times, most recently
from
e14576b
to
2f9a695
Compare
Madhu-1
reviewed
Apr 26, 2022
The CSI package needs to load clusterInfo, today this code is in the mon package which makes the call of LoadClusterInfo impossible without having a circular import. Signed-off-by: Sébastien Han <seb@redhat.com>
The CI runs will now collect all the secrets and configmaps available. Signed-off-by: Sébastien Han <seb@redhat.com>
leseb
force-pushed
the
multus-plugin-restart-fix
branch
4 times, most recently
from
ac7c522
to
8ef6819
Compare
Madhu-1
approved these changes
Apr 26, 2022
leseb
force-pushed
the
multus-plugin-restart-fix
branch
3 times, most recently
from
d49bad2
to
44348e3
Compare
Madhu-1
approved these changes
Apr 26, 2022
Implementation of the design proposed in rook#9903. Signed-off-by: Sébastien Han <seb@redhat.com>
leseb
force-pushed
the
multus-plugin-restart-fix
branch
from
44348e3
to
73b1347
Compare
travisn
approved these changes
Apr 26, 2022
mergify bot
added a commit
that referenced
this pull request
Apr 27, 2022
core: fix csi-cephfsplugin pod restart on non-hostnetworking env (backport #9925)
7 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of your changes:
Implementation of the design proposed in
#9903.
Signed-off-by: Sébastien Han seb@redhat.com
Checklist: