-
Notifications
You must be signed in to change notification settings - Fork 53
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
Deploy OTP to AWS load balancer and manage OTP servers in separate collection (and misc other fixes) #225
Merged
Merged
Deploy OTP to AWS load balancer and manage OTP servers in separate collection (and misc other fixes) #225
Changes from 20 commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
ac0b00c
WIP spin up EC2 (no user data)
landonreed 9c5e92c
Merge branch 'remove-r5' into deploy-to-ec2
landonreed d34b96a
Merge branch 'dev' into deploy-to-ec2
landonreed 560c4c6
refactor(snapshot): remove legacy MapDB-based snapshot jobs
landonreed dbbd130
fix(delete): delete SQL namespace when feed version/snapshot deleted
landonreed c01eadc
feature(deploy-ec2): deployment enhancements for load balancers
landonreed f777b02
fix(user-mgmt): better error handling when Auth0 cannot update/create…
landonreed 8898376
fix: move toGtfsDate from deleted class to FeedTx
landonreed fa5ce83
refactor: fix whitespace
landonreed e0a1eb6
refactor: add missing aws pom entry
landonreed 61c04d2
build(pom): update gtfs-lib dependency
landonreed 7231a95
feature(server-mgmt): manage deployment servers at the application level
landonreed b372303
refactor(server-job): attach just the project ID to the merge feeds job
landonreed 7c92c1d
Merge branch 'dev' into deploy-to-ec2
landonreed aa0553b
refactor(deploy): shuffle deploy job code for clarity
landonreed c52d6f0
Merge pull request #133 from ibi-group/dev
landonreed 56f7642
Merge branch 'dev' into deploy-to-ec2
landonreed 6ae2055
refactor: fix issues resulting from merge
landonreed c33c290
refactor(deployment): tweak user script and update default config
landonreed 05ec4df
refactor(deployment): improve validation of server fields
landonreed b6d7363
refactor(deployments): modify OtpServer fields and refactor server cr…
landonreed 7a020c8
refactor: remove unused import
landonreed c753a31
refactor(ServerController): add comment about checking S3 permissions
landonreed b793727
refactor(ServerController): add missing exceptions to logMessageAndHalt
landonreed a3ed73c
refactor(deploy): fix check for s3 graph object
landonreed a07935a
refactor(deploy): revert to default instance type if none specified
landonreed a177e79
refactor(deploy): make instance profile arn optional
landonreed 2507ab5
refactor(deploy): use set method rather than with for instance profile
landonreed e1fe1a3
Merge branch 'dev' into deploy-to-ec2
landonreed 4a1ef29
refactor(deploy): move ec2 config into OtpServer
landonreed 9b957dd
refactor(deploy): tweak deployJob for NPE fix and fix server delete
landonreed bf0f1bc
ci(config): update server.yml.tmp for e2e
landonreed ade0b40
refactor(EC2InstanceSummary): add empty constructor for serialization
landonreed 95a2333
Merge branch 'dev' into deploy-to-ec2
landonreed c273350
test(.gitignore): don't ignore test config
landonreed 3493570
test(mtc): fix broken MTC feed merge test with new test config
landonreed 4992b32
refactor(ServerController): isolate jackson parse to utility method
landonreed 3441fa2
refactor(ServerController): surround validation method calls in try/c…
landonreed a36a7d9
refactor(deploy-to-ec2): address PR comments
landonreed 18bd9d3
refactor(deploy-to-ec2): add json property latest; add server ID to s…
landonreed c28c215
Merge branch 'dev' into deploy-to-ec2
landonreed 41c21a8
refactor(deploy): fix check for S3 jar
landonreed 7e1528b
refactor(deploy-to-ec2): address PR comments
landonreed f34affb
refactor(deploy-to-ec2): surround s3 checks in try/catch
landonreed 4cc9b68
refactor(deploy-to-ec2): actually skip termination request
landonreed fb44a61
refactor(deploy): fix duration calc
landonreed 101b7f9
refactor(deploy): use onboard nginx to signal ec2 deploy status
landonreed 523801d
refactor(deploy): bump default otp version to 1.4
landonreed c399189
refactor(deploy): add terminate EC2 instance HTTP endpoint
landonreed 240a6e0
refactor(deploy): refine terminate instances endpoint and check for g…
landonreed File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
94 changes: 0 additions & 94 deletions
94
src/main/java/com/conveyal/datatools/editor/jobs/ProcessGtfsSnapshotExport.java
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
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.
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.
Are we sure that this is information we'd want to include as part of the
appinfo
response?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.
Some of this config info might actually make more sense to store in the
OtpServer
object... that might make more sense than splitting up the AWS info by putting thetargetGroupArn
in theOtpServer
and this stuff inserver.yml
.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.
Also, there may be more than one EC2 environment we're deploying to, so that just strengthens the argument to place this info into
OtpServer
.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.
I concur.