Skip to content

Commit 5c37dfe

Browse files
committed
PUDN, static ips: fill out the drawbacks section
Signed-off-by: Miguel Duarte Barroso <[email protected]>
1 parent 3c499d0 commit 5c37dfe

File tree

1 file changed

+11
-23
lines changed

1 file changed

+11
-23
lines changed

enhancements/network/routed-ingress-support-for-primary-udn-attached-vms-with-static-ips.md

Lines changed: 11 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -613,36 +613,24 @@ in OVN-Kubernetes, and more CRs to reconcile.
613613

614614
### Implementation Details/Notes/Constraints
615615

616-
What are some important details that didn't come across above in the
617-
**Proposal**? Go in to as much detail as necessary here. This might be
618-
a good place to talk about core concepts and how they relate. While it is useful
619-
to go into the details of the code changes required, it is not necessary to show
620-
how the code will be rewritten in the enhancement.
616+
TODO
621617

622618
### Risks and Mitigations
623619

624-
What are the risks of this proposal and how do we mitigate. Think broadly. For
625-
example, consider both security and how this will impact the larger OKD
626-
ecosystem.
627-
628-
How will security be reviewed and by whom?
629-
630-
How will UX be reviewed and by whom?
631-
632-
Consider including folks that also work outside your immediate sub-project.
620+
TODO
633621

634622
### Drawbacks
635623

636-
The idea is to find the best form of an argument why this enhancement should
637-
_not_ be implemented.
638-
639-
What trade-offs (technical/efficiency cost, user experience, flexibility,
640-
supportability, etc) must be made in order to implement this? What are the reasons
641-
we might not want to undertake this proposal, and how do we overcome them?
624+
The biggest drawback we have to implementing this feature is lack of clear
625+
asks from customers - we're pretty much guessing what they would want to use.
642626

643-
Does this proposal implement a behavior that's new/unique/novel? Is it poorly
644-
aligned with existing user expectations? Will it be a significant maintenance
645-
burden? Is it likely to be superceded by something else in the near future?
627+
We do not know (for instance):
628+
- is this feature only for VMs ?
629+
- is this feature only about importing VMs ? Should we allow creating new VMs
630+
with dedicated MAC / IP / gateway requests ?
631+
- are the end users (admins) after a centralized IP management alternative ?
632+
Do they want to have a single place to check their network address
633+
assignments ?
646634

647635
## Alternatives (Not Implemented)
648636

0 commit comments

Comments
 (0)