sync to master (ignore this)#1
Merged
binary1230 merged 195 commits intobinary1230:masterfrom Dec 6, 2017
Merged
Conversation
…one. Makes having more than one plugin work.
In some cases, Sideboard was comparing a full path (e.g. '/home/vagrant/uber/sideboard/plugins/uber') with a plugin name (e.g. 'uber') to sort plugins. Doing so would break priority plugins settings. Fixes #13.
Make priority_plugins work regardless of setup vagaries
Pull in priority_plugins fix from upstream
since glob() returns an ordering that is os-dependent, we need to sort here in order to ensure that all OS'es load the non-prioritized plugins in the same order this change does not fix any loading issues, BUT it will ensure that if there's an issue on one server, there will be an issue everywhere (no inconsistent issues on different OSes)
sort glob() output for consistent results
- this was causing output to syslog no matter what development.ini said
- this is not enabled by default and has to be turned on in the INI - this enables external apps to easily parse multiline logfile output as one continuous entry - each line of the log will be indented
Centralized logging - add multiline indent, remove syslog output by default
- doesn't add much spam but does report which files are actually being served up, which is good data to collect
turn up all logging thresholds to the highest levels
added @request_cached_property decorator
- this code monkeypatches the start of each thread to call prctl() which sets the thread name - this allows external utilities, like top and ps, to show thread names - this is great because if a particular thread is going off the rails, we can now see which one using external tools - requires build-package and libcap to be installed on the system
- this is useful to add a more descriptive name to be shown to the OS
- required for use in prctl module calls (thread naming setting)
- required for use in prctl module calls (thread naming setting)
add override for thread names for DaemonTasks
set thread names on startup
- cleanup existing debugging stuff - gather linux-specific thread ID so we can correlate threads with their system process info in /proc, top, etc - create general purpose framework for registering more diagnostics functions - add very detailed snapshot printout of per-thread stack traces, run status, thread name, etc (these are heavy weapons for diagnosing hung threads / deadlocks) - add basic memory reporting, more coming later
enable system diagnostics package to show threading info
- turns out this is needed for pycharm - and is a decent idea to do this anyway
upgrade pytest to latest version
passthru subscriptions now get cleaned up properly See merge request !57
Upstream needs to sync up with MAGFest's sideboard. Even if this isn't merged, we need it to create a mergeable PR.
Sync with MAGFest (no conflicts)
Add Dockerfile from upstream
The GPG server is no longer reachable, so we're copy-pasting new instructions from https://github.com/nodejs/docker-node/blob/master/7.10/Dockerfile which should theoretically get around that.
* Exposes cherrypy profiler on http://servername/profiler/ * Adds improved implementation of cherrypy profiler * Adds some profiler docs, improves the profiler file naming scheme by including the function name * Adds test to verify that profile decorator is a noop if profiling is disabled * Fixes tests in test_configuration * Fixes docs per code review * Changes requested in code review
…tests for each individual plugin. I guess you can't run the sideboard tests from one of our vagrant deploys. We could fix this, but it would be a change in every single plugin repository and I just don't have the energy for that.
Resyncing with public
Adding missing unicode_literals imports
Pull in updates from Magfest's Sideboard
* Makes jsonrpc_handler more flexible * Fixes unit tests * Makes autolog a little smarter about self=None * Reverts self=None
looks like these were left off in a merge error, adding them back in
* Adds basic support for pip environment markers to setup.py * Slightly terser code
Update Dockerfile
Owner
Author
|
PERSONAL REPO ONLY, EVERYONE IGNORE THIS |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.