Skip to content
This repository has been archived by the owner on Sep 26, 2022. It is now read-only.

Merge ms #273

Merged
merged 3 commits into from Sep 10, 2019
Merged

Merge ms #273

merged 3 commits into from Sep 10, 2019

Conversation

ceache
Copy link
Contributor

@ceache ceache commented Sep 10, 2019

No description provided.

Morgan-Stanley and others added 3 commits September 10, 2019 18:13
 * Admin command to manage presence.
 * Revert krb forward cmd.
 * api: Fix overly restrictuve 'match' pattern.
 * Fix docker logs - handle both bytes and str.
 * adminapi: Import list performance
 * Expose TREADMILL_UNIQUEID in container env.
 * Fix nodes blacking out due to failing cleanup apps.
 * Fix websocket error handling - exception while sending a message
   (client closed connection for example) crashes websocket server.
 * treadmill admin keytab2 fetch to fix keytab fetching issue in node start
 * kerberos not working in docker container in treadmill instance
* ms:
  Sync with 2019.09.04-1
@ceache ceache requested review from andreikeis, mswiecicki-ms and a team September 10, 2019 22:19
@codecov-io
Copy link

codecov-io commented Sep 10, 2019

Codecov Report

Merging #273 into master will decrease coverage by 0.07%.
The diff coverage is 30.56%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #273      +/-   ##
==========================================
- Coverage    66.7%   66.62%   -0.08%     
==========================================
  Files         583      584       +1     
  Lines       44138    44261     +123     
  Branches     4496     4501       +5     
==========================================
+ Hits        29443    29490      +47     
- Misses      13568    13642      +74     
- Partials     1127     1129       +2
Impacted Files Coverage Δ
lib/python/treadmill/runtime/linux/image/native.py 70.23% <ø> (ø) ⬆️
lib/python/treadmill/runtime/docker/runtime.py 69.33% <ø> (ø) ⬆️
lib/python/treadmill/cli/admin/krb/forward.py 0% <0%> (ø) ⬆️
lib/python/treadmill/cli/admin/master.py 0% <0%> (ø) ⬆️
lib/python/treadmill/sproc/docker.py 34.78% <0%> (-0.44%) ⬇️
lib/python/treadmill/cli/krb/forward.py 0% <0%> (ø) ⬆️
lib/python/treadmill/rest/api/app.py 84.37% <100%> (+84.37%) ⬆️
lib/python/treadmill/api/app.py 80.85% <100%> (-3.36%) ⬇️
...hon/treadmill/tests/runtime/linux/manifest_test.py 93.75% <100%> (+0.2%) ⬆️
lib/python/treadmill/tests/api/app_test.py 97.01% <100%> (-0.49%) ⬇️
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b9a646f...86a583e. Read the comment docs.

@ceache ceache merged commit f18267c into morganstanley:master Sep 10, 2019
@ceache ceache deleted the merge_ms branch September 10, 2019 22:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants