-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Indexserver & nameserver roles in cluster sites #2311
Conversation
e2891f8
to
a8d6ef2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code apart, it looks good to me @CDimonaco !. For the sake of authenticity we might want to update the story book for HANA scale-out taking into account:
- in a given site, there can only be one master nameserver, the rest are slaves
- in a given site, there can only be one master indexserver, the rest are slaves or standby
- in the nodes that are not part of a site (other), there is no nameserver nor indexserver.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really nice job @CDimonaco
I put some comments, but it is great in general. No need for a new review (unless you have some question)
test/e2e/cypress/fixtures/hana-cluster-details/available_hana_cluster.js
Outdated
Show resolved
Hide resolved
d21ef23
to
bee9855
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔝
This reverts commit 09a8649.
1a3a12d
to
8fbb7f0
Compare
Description
This PR adds the support to nameserver and indexserver roles in cluster nodes.
The roles are extracted from node attributes.
The roles are separated by ":", the role string is splitted on the ":" character.
In scale up
4:P:master1:master:worker:master
Nameserver is at index 3, and Indexserver at index 5
In scale out
master1:master:worker:master
Nameserver is at index 1 and Indexserver at index 3
How was this tested?
Automated tests, frontend, backend and e2e