This repository has been archived by the owner on Jun 6, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 41
Realigning our role composition back with cf upstream #1633
Closed
jandubois
wants to merge
8
commits into
develop
from
llqBKY7z-realigning-our-role-composition-back-with-cf-upstream
Closed
Realigning our role composition back with cf upstream #1633
jandubois
wants to merge
8
commits into
develop
from
llqBKY7z-realigning-our-role-composition-back-with-cf-upstream
Commits on Jul 20, 2018
-
Remove "processes" list from all roles
They are not being used for anything anymore. [trello#llqBKY7z]
Configuration menu - View commit details
-
Copy full SHA for ce19645 - Browse repository at this point
Copy the full SHA ce19645View commit details -
Configuration menu - View commit details
-
Copy full SHA for 9c7ac4e - Browse repository at this point
Copy the full SHA 9c7ac4eView commit details -
Configuration menu - View commit details
-
Copy full SHA for bc75611 - Browse repository at this point
Copy the full SHA bc75611View commit details -
Configuration menu - View commit details
-
Copy full SHA for 55dabb7 - Browse repository at this point
Copy the full SHA 55dabb7View commit details -
Combine loggregator and syslog-rlp into log-api role
The old loggregator role was already claiming port 8082 for "grpc", so would potentially conflict with syslog-rlp. However, I think 8082 is the doppler grpc port, which doesn't run on the loggregator role, so this looks like a mistake. [trello#llqBKY7z]
Configuration menu - View commit details
-
Copy full SHA for 41898fe - Browse repository at this point
Copy the full SHA 41898feView commit details -
Merge routing-api role into api
Also move cc_uploader role (from cc-uploader) and file_server role (from diego_access) into api role as well. Open issues/questions: * api needs to talk to routing-api. Since we don't require hairpin support (Azure doesn't support it yet), we cannot use the service name, but need to access it via 127.0.0.1. This means that the routing-api cert also needs to list the loopback address as an alternate name. * routing-api was an active-passive role, so api is also active-passive now, which will affect scalability. [trello#llqBKY7z]
Configuration menu - View commit details
-
Copy full SHA for edef4ec - Browse repository at this point
Copy the full SHA edef4ecView commit details -
Includes (what is still left of) cc-clock, cc-uploader (tps), diego-access, diego-brain, and syslog-scheduler. Open issues/questions: * syslog-scheduler was a headless role; that tag is now lost. * cc-clock and syslog-scheduler were singletons; they can now have up to 3 instances (and will have 2 by default under HA). [trello#llqBKY7z]
Configuration menu - View commit details
-
Copy full SHA for 9320314 - Browse repository at this point
Copy the full SHA 9320314View commit details -
Rename syslog-adapter to adapter and blobstore to singleton-blobstore
Open issue/question: * Upstream rename just the blobstore role, but not the URL, so that the singleton role can be replaced with a scalable HA version via ops files, without having to also update any of the other roles as well. [trello#llqBKY7z]
Configuration menu - View commit details
-
Copy full SHA for 59d3345 - Browse repository at this point
Copy the full SHA 59d3345View commit details
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.