forked from apache/openwhisk
-
Notifications
You must be signed in to change notification settings - Fork 1
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
mesos support for NodeStats events #4
Open
tysonnorris
wants to merge
142
commits into
ow-invoker-backpressure-mesos
Choose a base branch
from
invoker-backpressure-mesos
base: ow-invoker-backpressure-mesos
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
mesos support for NodeStats events #4
tysonnorris
wants to merge
142
commits into
ow-invoker-backpressure-mesos
from
invoker-backpressure-mesos
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
…pache#4294) 1. Fixed 2 typos that refer to non-existing response 'TooManyRequsts' 2. Updated description of PUT method for rule 3. Updated description of PUT method for trigger 4. Created new definition ActivationBrief 5. Changed response type for GET activations to refer to ActivationBrief instead of EntityBrief, which didn't match the actual response 6. Added attribute statusCode to definition ActivationBrief and Activation 7. Modified description of GET activations apache#2271 8. Added operationId to GET activation logs and result (apache#318) 9. Removed the init attribute from the ActionExec definition (apache#4201) 10. Use of 2 spaces for indentation, which is the default when working with the Swagger editor, instead of the four spaces used.
* Improve container factory documentation Explicitly list some of the expectations and requirements an actual container factory / container implementation needs to satisfy. Addresses apache#4002. * Address review feedback
…Apache Hadoop clusters. (apache#4129)
…ryTests (apache#4305) * Look up runtime tag for YARN tests * Scala formatting
* Use custom manifest for YARN tests * Remove imageTag method
…ily (apache#4299) * wait for sentinel only in case no timeout occurred * introduce timeout annotation * nodejs:6 logging timeout action w/o busy loop * do conversion from FiniteDuration to Duration
…pache#4275) * Collect and log CosmosDB query metrics when extra logging is enabled Fixes apache#4268
If there is an deployment with several controllers, the test `Whisk rules should preserve rule status when a rule is updated` fails intermittently. The reason is, that there are several modifications on the rule. These modifications are already retried. But enabling and disabling the rule also modifies the database document. But that change is currently not retried. This PR puts the enabling/disabling of the rule into the retry block as well to make this test more stable against our eventual consistency. Co-authored-by: Somaya Jamil <[email protected]>
Codecov Report
@@ Coverage Diff @@
## ow-invoker-backpressure-mesos #4 +/- ##
==================================================================
- Coverage 78.23% 42.01% -36.23%
==================================================================
Files 168 176 +8
Lines 8152 8406 +254
Branches 554 595 +41
==================================================================
- Hits 6378 3532 -2846
- Misses 1774 4874 +3100
Continue to review full report at Codecov.
|
Records the clusterId (if configured) with the updated document while persisting in ComosDB. This is an optional property. By default no `clusterId` info is saved
* Update to CosmosDB SDK 2.4.2 * Remove logic related to storing attachments in CosmosDB * Reduce throughput for test to 400
…ache#4284) * Factor out some annotation names to WhiskAction singleton. * Omit API key unless requested explicitly. * Add an annotation provide-api-key which causes an API key to be passed to the action context. * Modify Parameters.isTruthy to allow for caller to specify how it wants to treat a missing key. Default is missing key -> false (preserving behavior). Add test for isTruthy. * Treat a missing provide-key-annotation as truthy annotation. Actions that already exist without the annotation will receive the key as they might already expect. * Add container proxy test. * Update tests for new system annotation. * Allow pre-existing actions to be exempt from new system annotations. * Update docs.
* Kamon Gauge abstraction * Switch to gauge metric
… with new format. (apache#4346) * Adapts test harness to parses activation ids from wsk activation list with new format.
…4348) Ignores the json records which have some of the required fields missing and replaces them with the error message from the parsing
…rceManager impls; handle ClusterResourceError in case of reaching cluster capacity
…s, total memory, max memory
…dle containers; only remove idle containers that match the actual host/port of selected container
…nup of containers
tysonnorris
force-pushed
the
invoker-backpressure-mesos
branch
from
July 5, 2019 20:10
c30cdf4
to
dc127ab
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.
Description
Related issue and scope
My changes affect the following components
Types of changes
Checklist: