Epic: Hub controller for lighthouse

[donaldh]

Epic Description

The goal of this epic is to implement ServiceImport merging.

Once an mcs-controller has been introduced, the reconciliation logic will be extended to address
the following open issues:

• Mark conflicting ServiceExport objects with the ServiceExportConflict condition
• Merge matching ServiceExport objects into a single ServiceImport object

Related enhancement proposal: #111

Acceptance Criteria

Lighthouse behaviour aligns with MCS specification:

• Mark conflicting ServiceExport objects with the ServiceExportConflict condition
• Merge matching ServiceExport objects into a single ServiceImport object

Definition of Done (Checklist)

• Code complete
• Relevant metrics added
• The acceptance criteria met
• Unit/e2e test added & pass
• CI jobs pass
• Deployed using cloud-prepare+subctl
• Deployed using ACM/OCM addon
• Deploy using Helm
• Deployed on supported platforms (for e.g kind, OCP on AWS, OCP on GCP)
• Run subctl verify, diagnose and gather
• Uninstall
• Troubleshooting (gather/diagnose) added
• Documentation added
• Release notes added

Work Items

[donaldh]

Aggregate ServiceImports
Updating an exported service is not handled
GlobalNet - exported service isn't reachable using desired port after service port edit
GlobalNet- exported service isn't reachable after service re creation

[vthapar]

@donaldh Any specific reason why we need a hub controller and cant make this work without needing a centralized mcscontroller? Having a serverless broker has been one of the core design tenent of submariner. If we want to introduce a central controller to run on broker, it should be with a good enough reason. Note that an mcs controller with come withs own overhead of HA etc.

[donaldh]

@vthapar I have described the reason in enhancement pr #82

[dfarrell07]

It seems like #82 is closed, and this Epic doesn't track #111. Should it be updated?

[donaldh]

Good catch @dfarrell07, I have updated the title and description to match the ER.

[maayanf24]

Talked on the planning call - Not a priority for now