Force 3 station replications #1085
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
shohamroditimemphis
requested review from
avrhamNeeman and
idanasulin2706
as code owners
avrhamNeeman
reviewed
Jul 9, 2023
| @@ -337,6 +337,9 @@ export const tieredStorageTimeValidator = (value) => { | |||
| }; | |||
|
|
|||
| export const replicasConvertor = (value, stringToNumber) => { | |||
| if (isCloud()) { | |||
There was a problem hiding this comment.
Why you didn't pass it by param?
There was a problem hiding this comment.
resolve in this pr:#1091
idanasulin2706
approved these changes
Jul 9, 2023
idanasulin2706
added a commit
that referenced
this pull request
Jul 10, 2023
* version update * . * build * remove env * remove unused folder * little changes * upgrade go mod packages * small changes * close popover when clicking * Update env variable (#999) * remove user for cloud * update Authenticate - check * update launch json * fixing auth middleware * fix auth middleware * fix Authenticate endpoint (#1000) * Update README.md * Update index.js (#1002) * getMainOverviewDataDetails (#1001) * fix removeStaleStations (#1003) * dashboard connect to endpoints * Move refresh token endpoint (#1004) * fix removeStaleStations * move RefreshToken endpoint * add indentations to reload memphis options * final changes * . * build files * add full name in add user response * add account_id * fix * fix * fix * fix (#1008) * fix * build * build * finished changing the logs format to include the tenant and user name * build * fixing log of disconnection event * Ws password (#1006) * generate password for internal ws use * fix * fix * allowedPasswordChars * Insert user where sys created (#1013) * generate password for internal ws use * insert user where sys created * fixes * fix + comment * add line space * ui build files * fixing issue cause the tiered storage station to not get created * shortened the logs prefix and made the logs more accurate * add internal user per tenant for mgmt purposses * billing page * resolved comments * change default max storage to 95% * fix code owner file * fixed memphisConnection decleration in ts code examples (#1017) * general bugfixes + ui build files * review comments fix * biling changes * added the subject of sdk schema creation to memphis services (#1018) Co-authored-by: daniel-davidd <[email protected]> * final changes * change 401 errors to 500 * build * Create schema fix (#1020) * added the subject of sdk schema creation to memphis services * changed respones listener --------- Co-authored-by: daniel-davidd <[email protected]> * fix some issues * fix UpdateIntegration (#1022) * fix UpdateIntegration * fix update + ccreate integration * fix DisconnectIntegration & GetIntegrationDetails * fix some issues --------- Co-authored-by: idanasulinStrech <[email protected]> * route fixed * cloud flag * fix send integration update (#1028) * ui changes * remove unused code (#1032) * added delayed cgs (#1030) * improve code examples (#1029) * improve code examples * add .Net examples * fix .Net example * add account id in .Net examples * removed the use of c.acc to get tenant name and added tenant and user name to noticef logs (#1034) Co-authored-by: daniel-davidd <[email protected]> * move throughput func to memphis_cloud (#1035) * add missing/wrong acc id error (#1036) * add missing/wrong acc id error * removed the use of c.acc to get tenant name and added tenant and user name to noticef logs (#1034) Co-authored-by: daniel-davidd <[email protected]> * move throughput func to memphis_cloud (#1035) * add missing/wrong acc id error * resolve --------- Co-authored-by: daniel-davidd <[email protected]> Co-authored-by: daniel-davidd <[email protected]> Co-authored-by: Shay Bratslavsky <[email protected]> * bugfix * move send analytics * get max db conns from env * added validation (#1037) * added validation * added returns --------- Co-authored-by: daniel-davidd <[email protected]> * some changes * build * pricing * pricing * Update index.js * . * build * improve handle connect performance * build * added tenent table custom sequence (#1044) * added tenent table custom sequence * added tenant seq initializing to cloud file --------- Co-authored-by: daniel-davidd <[email protected]> * general bugfixes * purge account * handle support * fix GetAllActiveUsers * add support popover * create user pattern * update GetActiveUsers per stations and per schemas * fix GetAllActiveUsers (#1046) * fix GetAllActiveUsers * fix * socket issue * . * build * Reload bugfixes (#1050) * test * hopefully fix reload --------- Co-authored-by: shay23b <[email protected]> * Reload bugfixes (#1051) * test * hopefully fix reload * add memphis_user creation * fix * fix * fixes --------- Co-authored-by: idanasulinStrech <[email protected]> * Reload bugfixes (#1052) * test * hopefully fix reload * add memphis_user creation * fix * fix * fixes * add comments --------- Co-authored-by: shay23b <[email protected]> * support update * disclaimer * clear values in the modal * support box changes * build * edit traces * Update README.md * Move g to memphis (#1056) * move $G to $memphis and reload logic * remove comments * fix reloadAuthorization * fixes * fixes * fixes --------- Co-authored-by: idanasulinStrech <[email protected]> * remove calculation * enhancements * build * fix hex to ascii function * clean code * fix * fix * change availavle replicas to max 3 (#1062) * change availavle replicas to max 5 * undo ui changes * fix * seperate os and cloud * fix hex to ascii function (#1063) * fix hex to ascii function * clean code * fix * fix * add remove old stations (#1038) * add remove old stations * fix * fix * fix * fix * fix * fix * fix * fix dls (#1059) * ui build files * update ui package json * fix * add data location from global account to memphis account * Handle refresh issue * fix issues * fix * joined relive function to one with a batch (#1064) Co-authored-by: daniel-davidd <[email protected]> * update DataLocationFromGlobalAccountToMemphisAccount * fix image upload (#1066) * test * test * remove unnecessary code * remove unnecessary code * fix issues * fix * fix * fix * fix * fix * fix issue * bugfix - fix reload * validate user app password (#1071) * validate user app password * fix validatePassword * Validate user app password (#1072) * validate user app password * fix validatePassword * fix err * err Warnf * Fixed generate password with valid characters * fix error message on bas password * minor issue * pattern password fixed * remove extra line space * version printing correction * create user validation (#1074) * create user validation * move validateUsername * move funcs * move validatePassword * switch traces to memphis stream * switch traces to memphis stream * Create user validation (#1076) * create user validation * move validateUsername * move funcs * move validatePassword * ui build * update getIp() * update go mod * fix issues * change errorf to warnf creation producer * fix issues * fix * Update README.md * change analytics params to map instead of array * test * build * generate password issue fixed * remove unnecessary code * update SendEvent with goroutine * fix timestamp * fix timestamp * fix timestamp to unix * retry on failed traces sending * general bugfixes * email validation * Enable custom endpoint resolver and path-style (#1015) * Force 3 station replications (#1085) * add force 3 replications per station * add force 3 replications per station * update CreateDefaultStation * remove replicas section in cloud * fixes * fix ui * fix validations * message journey issue fixed * build issue * remove support from OSS * change traces prefrences * Update Jenkinsfile (#1090) * UI changes force 3 replications per station (#1091) * fix ui changes regarding force 3 replications per station ticket * changes errors to be more informative in s3 integration * fix * search user issue fixed * system components icon * fix tier storage buttom * fix (#1094) * . * create station on initialize analytics * S3 Integration UI changes * account_id copy * trace system up is up per tenant for cloud (#1098) * trace system up is up per tenant for cloud * fix * fix * fix * fix s3 validations * s3 integrtion * build * fix remove tenant resources (#1100) --------- Co-authored-by: avrham <[email protected]> Co-authored-by: Avraham Neeman <[email protected]> Co-authored-by: svetaStrech <[email protected]> Co-authored-by: shohamroditimemphis <[email protected]> Co-authored-by: Yaniv Ben Hemo <[email protected]> Co-authored-by: Shay Bratslavsky <[email protected]> Co-authored-by: Sveta Gimpelson <[email protected]> Co-authored-by: shohamroditimemphis <[email protected]> Co-authored-by: daniel-davidd <[email protected]> Co-authored-by: daniel-davidd <[email protected]> Co-authored-by: daniel-davidd <[email protected]> Co-authored-by: Bastiaan Schaap <[email protected]> Co-authored-by: Valera Bronshtein <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.