Skip to content
This repository has been archived by the owner on Jun 22, 2022. It is now read-only.

draft specification for central control plane changes #1

Open
wants to merge 13 commits into
base: devel
Choose a base branch
from

Conversation

elevran
Copy link
Owner

@elevran elevran commented Dec 16, 2021

Signed-off-by: Etai Lev Ran etai@il.ibm.com

@elevran
Copy link
Owner Author

elevran commented Dec 16, 2021

@davidohana @donaldh

submariner/centralized-control-plane.md Outdated Show resolved Hide resolved
submariner/centralized-control-plane.md Outdated Show resolved Hide resolved
submariner/centralized-control-plane.md Outdated Show resolved Hide resolved
submariner/centralized-control-plane.md Outdated Show resolved Hide resolved
submariner/centralized-control-plane.md Outdated Show resolved Hide resolved
submariner/centralized-control-plane.md Show resolved Hide resolved
// ServiceSpec defines the desired state of a Service
type ServiceSpec struct {
// UUID is a globally unique identifier for the Service object
UUID string `json:"id,omitempty"`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why force a UUID and not any other unique string?
Who generates this UUID?

Copy link
Collaborator

@davidohana davidohana Dec 20, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if we stay with a machine generated UUID, at least can add some logical name/description field of the service?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See note above regarding Clusters. It could apply to Services as well:
...(denoted as a UUID below. More descriptive names could also be used as long as they're not ambiguous within a ClusterSet). For example, if each cluster or service has a unique name - we could certainly use that.

I think the UUID might be confusing as a real UUID isn't strictly needed or useful. Changed to ID to avoid confusion.

We can certainly add a description to the Service (in fact, I'd imagine we may want to add more information than just a description, for example: owner, API reference, ...). I'm leaving those out for now and we can add then as needed later. I'm not sure any of these attributes


### Source Code Location

We propose a new toplevel `submariner-io/axon-ccp` (short for for Axon Central Control plane).
Copy link
Collaborator

@davidohana davidohana Dec 20, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't the controller per broker (given broker multi-tenancy)?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's a deployment choice on out part. Either works:

  • global broker instance, running in (e.g.,) "submariner-system", operating on all namespaces
  • local instance in each namespace.

Copy link
Collaborator

@davidohana davidohana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First review iteration done

Signed-off-by: Etai Lev Ran <etai@il.ibm.com>
@elevran elevran force-pushed the centralized_control_plane branch from 05250d2 to 5898bb9 Compare December 20, 2021 18:39
Signed-off-by: Etai Lev Ran <etai@il.ibm.com>
@elevran elevran force-pushed the centralized_control_plane branch from 8838a82 to 8085248 Compare December 22, 2021 08:07
Signed-off-by: Etai Lev Ran <etai@il.ibm.com>
maayanf24 and others added 7 commits January 31, 2022 13:01
This is the preffered place fpr user to open issues
with enhancement requests for the Submariner project

Signed-off-by: Maayan Friedman <maafried@redhat.com>
Signed-off-by: Etai Lev Ran <etai@il.ibm.com>
Signed-off-by: Etai Lev Ran <etail@il.ibm.com>
Signed-off-by: Etai Lev Ran <etail@il.ibm.com>
Commits produced using "git commit --fix" are great for review, but
must be squashed before a PR is merged.

Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Etai Lev Ran <etail@il.ibm.com>
Signed-off-by: Donald Hunter <donald.hunter@redhat.com>
@elevran elevran force-pushed the centralized_control_plane branch from 11dbad1 to 1b28b57 Compare January 31, 2022 11:04
Signed-off-by: Etai Lev Ran <etail@il.ibm.com>
Signed-off-by: Etai Lev Ran <etail@il.ibm.com>
Signed-off-by: Etai Lev Ran <etail@il.ibm.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants