Improvements to VPCs and RDS #814
Draft
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
#781) * show proxy ip address in deployment info * add ProxyInstanceCount and ServerURL configs - `ProxyInstanceCount` will determine how many proxy instances should be deployed. Default to 0, can be 0 or 1. - `ServerURL` determines the URL to use with the Mattermost client from the agent nodes. It takes precedence over the Site URL, proxy IP, etc. * downgrade elasticsearch dependency * ProxyInstanceCount default 0 * faq * feat: add concurrency to configureAndRunAgents * Better log tracing * Update docs/faq.md Co-authored-by: Alejandro García Montoro <[email protected]> * SAML configuration * ltkeycloak * typos * fixed slog calls * updated mattermost-realm.json * saml configuration * increase errorsChan buffer * revert: agent concurrency * docs: updated faq with SiteURL mention * NewPointer * refactor terraform app config * mlog, ctx, worker number * debug log * revert saml changes * make assets * go mod tidy * revert back channel to size 2, return early --------- Co-authored-by: Alejandro García Montoro <[email protected]>
* Update Pyroscope v1 Add support for goroutine and block profiles * Unify service name for targets * Update test
* MM-56605: Create results file and print to file * Separate writter into its own function for later * Mofify writeToFile. Passing the file object. * implement multiWriter so I can double the writes and then send them to cosole as well as file. * Replace file object with interface. * - Remove separate function to create file - Move file creatuin and writting to the file to Comparison function * Draft unti test for PrintResults * Make test runnable by using LoadTests and Status properly. * - Solve missing use of interface in the initail code chnage - Solve whitespace mismatch * Change expected to allign with the output. This solves failing unit test. * Address most several of the code review feedback point. * Use existing file path that is already being provided to the function. * Address linter complaint about use of `Sprintf` for strings that need no formating. * Address review comments * Remove line printing an error as it was redundant. * Update another if block that redundantly prints and returns the same error. * Implement missed correction from Alejandro. If block must return the correct var when returning the error. Ensure this is not occuring on other lines I've added.
Without this line, the deployment fails because it cannot do t.getAsset
* ltkeycloak: added flag to force migrate all users * changed flag * avoid logging and returning error * modified comment * remove duplicated restoreasset * reverted condition to return early * check for nil values * use UpdateUserAuth * rename variable name * Update cmd/ltkeycloak/from_mattermost.go Co-authored-by: Alejandro García Montoro <[email protected]> --------- Co-authored-by: Alejandro García Montoro <[email protected]>
…e version (#813) * customizable database engine version * Use same VPC for all resources * make assets * removed engineversion validation * wait for stdout * use default subnet if not specified
This was referenced Sep 27, 2024
|
Looks like the branch got messed up. |
fmartingr
changed the title
fmartingr
force-pushed
the
elasticsearch-lt
branch
from
7e814a2 to
7c21c7f
Compare
Now that we validate that var.cluster_subnet_ids.xyz's length is larger than 0, we allow both: 1. Unset lists, which are indeed serialized as empty strings. The length of an empty string is still 0. 2. Empty lists, which are serialized as lists, but that are not equal to the empty string. If we only check for the empty string, we lose 2.
We were checking for the length of the `cluster_subnet_ids` variable, not the underlying `redis` or `database` lists, which are the ones we later use in these definitions.
If we don't define the VPC, leaving ClusterVPCID empty or unset, the data.aws_subnets.selected list ends up being empty, since we filter for subnets that are contained in such VPC. To fix this, we retrieve the default VPC, and use its ID to filter the subnets to select if no VPC is explicitly set.
Similar to the VPC change, we filter the selected subnets by their availability zone *only* if the `AWSAvailabilityZone` setting is non-empty.
If the user doesn't specify a subnet for a resource, we don't need to pick one for it, we just set the subnet_id to null.
Subnet groups require 2+ subnets Co-authored-by: Felipe Martin <[email protected]>
fmartingr
force-pushed
the
elasticsearch-lt
branch
from
3c609f5 to
58f362a
Compare
fmartingr
force-pushed
the
elasticsearch-lt
branch
from
58f362a to
2496b65
Compare
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.
List of PRs merged