Egress Router CNI (OSDOCS-979) #28778
Conversation
mikemckiernan
commented
Jan 22, 2021
mikemckiernan
commented
- The ER CNI plug-in is TP for the 4.7 release.
openshift-ci-robot
added
the
size/S
|
Draft /hold |
openshift-ci-robot
added
the
do-not-merge/hold
|
@jboxman , @danielmellado , @weliang1 , PTAL. If this is technically accurate and the definition of concision, then I'll take an approval or thumbs up now and add a URL to the content after #28609 is merged. |
|
/lgtm |
|
Hi @mikemckiernan , left a few comments also on #28609, sounds good to me! I'll also reply to your JIRA and try to get a slot for a quick meeting on Fri! Thanks! |
|
New changes are detected. LGTM label has been removed. |
|
Deploy preview for osdocs ready! Built with commit 566c398 |
|
@openshift/team-documention, PTAL. After Daniel's review, I rebased and added an xref to the new topic. /hold cancel Applies to enterprise-4.7 and--small guess--milestone Future-Release. Please add the label, milestone, and merge if it looks OK. |
openshift-ci-robot
removed
the
do-not-merge/hold
adellape
left a comment
adellape
left a comment
There was a problem hiding this comment.
Looks good, couple tiny things. 👍
There was a problem hiding this comment.
s/The plug-in enables you to/You can use the plug-in to/
(Per IBMSG)
adellape
added
branch/enterprise-4.7
peer-review-done
* The ER CNI plug-in is TP for the 4.7 release. * Peer review from Alex.
|
@adellape , thank you for the review and corrections. It's ready for merge when you get a chance. |
5 participants
