search

kubestellar / kubestellar

KubeStellar - a flexible solution for challenges associated with multi-cluster configuration management for edge, multi-cloud, and hybrid cloud
https://kubestellar.io
Apache License 2.0
270 stars 65 forks source link

feature: rename IMBS to transport space #1557

Open nirrozenbaum opened 10 months ago

nirrozenbaum commented 10 months ago

Feature Description

I suggest rename imbs to transport space. imbs is the acronym for inventory + mailbox space. in our terms, mailbox+inventory is exactly the transport abstraction. so we could just call it transport space, which makes it clearer.

Proposed Solution

rename it

Want to contribute?

Additional Context

No response

nirrozenbaum commented 10 months ago

/cc @ezrasilvera /cc @pdettori /cc @MikeSpreitzer /cc @clubanderson

pdettori commented 10 months ago

@nirrozenbaum just to clarify, you are suggesting that imbs1 becomes transport1 in all docs/examples/ arch description ?

nirrozenbaum commented 10 months ago

@nirrozenbaum just to clarify, you are suggesting that imbs1 becomes transport1 in all docs/examples/ arch description ?

yes

nirrozenbaum commented 9 months ago

another point to mention - once we add the cluster inventory abstraction, we might want to host the abstract inventory CRs in a space. this would probably be called inventory space (the inventory of KS, not of the transport specific). therefore, IMBS doesn't make much sense at this point. so potentially we can have - WDS, Transport Space, Inventory Space. and inventory space can run on the same space as WDS or transport to minimize number of spaces.

ezrasilvera commented 9 months ago

I have no strong opinion on that. Transport seems like not accurate to me, but I don't have a good suggestion :-)

ezrasilvera commented 9 months ago

@nirrozenbaum is it urgent ? Can we do a renaming PR later on ? (i.e., during phase1 but after we stabilize our docs a little ) ?